snmpd_options parameter does not work with Debian 9 about puppet-snmp HOT 8 CLOSED

That is correct... Here we added a workaround, but it would be nice to see the logic directly into the module:

  case $::lsbdistcodename {
    'stretch': {
      augeas { 'systemd-snmpd-conf':
        context => '/files/lib/systemd/system/snmpd.service/Service/ExecStart',
        changes => [
          'set arguments/1 -LS2d',
        ],
        require => Package['snmpd'],
        notify  => [ Exec['systemd-reload'], Service['snmpd'], ],
      }
      exec { 'systemd-reload':
        command     => '/bin/systemctl daemon-reload',
         refreshonly => true,
      }
    }
    default: { }
  }

With the exec due to systemd being unable to read system conf files directly, as it seems.

from puppet-snmp.

Hi,

I have the same issue.
@bastelfreak is there any work in progress on this issue ?

Maybe the fastest way to fix this is to provide for debian 9 a template for /lib/systemd/system/snmpd.service ?

from puppet-snmp.

@hdep can you provide a patch for this? Also the preferred way custom systemd unit files is /etc/systemd/system/ and not /lib/systemd/system.

from puppet-snmp.

Since this module is declared Debian9 compatible and with acceptance tests, they should fail. But they pass.

@hep with your patch can you update acceptance tests to tackle this issue ?

from puppet-snmp.

@bastelfreak on debian 9 the file in /etc/systemd is a link to /lib/systemd

$ ls -al /etc/systemd/system/multi-user.target.wants/snmpd.service
lrwxrwxrwx 1 root root 33 janv.  4 10:02 /etc/systemd/system/multi-user.target.wants/snmpd.service -> /lib/systemd/system/snmpd.service

So I can't do that

from puppet-snmp.

I give up because I think fixing rpsec should have been done in another PR.
if anyone can fix the bug this will be great.

from puppet-snmp.

bug still present in 5.0.0

from puppet-snmp.

fixed in #216

from puppet-snmp.