Graylog2 / puppet-graylog

Install and configure a Graylog system via Puppet
Apache License 2.0
8 stars 29 forks source link

Problem with /etc/apt/apt.conf.d/01Proxy #31

Closed identw closed 5 years ago

identw commented 5 years ago

OS: ubuntu 16.04 puppet-agent version: 5.5.10 puppetserver: 5.3.7 Issue: Apt repo class does not work properly For code

class { 'graylog::repository':
  version => '3.0'
}->
class { 'graylog::server':
  package_version => '3.0.0-12',
  config          => {
    'password_secret'       => 'password',
    'root_password_sha2' => '3b18da9f4e1ddd0439e0f76f7414724234091ba1353e09e1e9e42b494c2a052d',
  }
}

I get this error:

Info: Retrieving pluginfacts
Info: Retrieving plugin
Info: Retrieving locales
Info: Loading facts
Info: Caching catalog for ubuntu16
Info: Applying configuration version '1558548639'
Error: /Stage[main]/Graylog::Repository::Apt/File_line[Remove graylog config from apt proxy file]: Could not evaluate: No such file or directory @ rb_sysopen - /etc/apt/apt.conf.d/01proxy
Error: Could not update: Execution of '/usr/bin/apt-get -q -y -o DPkg::Options::=--force-confold --force-yes install graylog-server=3.0.0-12' returned 100: Reading package lists...
Building dependency tree...
Reading state information...
E: Unable to locate package graylog-server
Error: /Stage[main]/Graylog::Server/Package[graylog-server]/ensure: change from 'purged' to '3.0.0-12' failed: Could not update: Execution of '/usr/bin/apt-get -q -y -o DPkg::Options::=--force-confold --force-yes install graylog-server=3.0.0-12' returned 100: Reading package lists...
Building dependency tree...
Reading state information...
E: Unable to locate package graylog-server

I fixed this problem

CLAassistant commented 5 years ago

CLA assistant check
All committers have signed the CLA.

rsynnest commented 5 years ago

Pretty critical PR here, Issue #30 makes the graylog puppet module undeployable and broken by default for us on Ubuntu 16.04 and 18.04, this simple PR fixes that issue. We've been running a patched version for a couple of months based on this PR. Thanks very much @identw ! Hope this gets some eyes on it (@bernd ? ) and hope to see a merge soon.

bernd commented 5 years ago

@rsynnest Thanks for the heads up! @identw Thanks for the PR and thanks for the patience!