fix for "error on subcontainer 'ia_addr' insert (-1)" messages in syslog

[gittygoo]

trafficstars

Fix for https://bugs.launchpad.net/ubuntu/+source/net-snmp/+bug/1246347

[ghoneycutt]

@gittygoo thanks for the contribution! This is failing because the spec tests also need to be updated to match your change.

Check out https://github.com/voxpupuli/puppet-snmp/blob/master/spec/classes/snmp_init_spec.rb#L615

[gittygoo]

@ghoneycutt unfortunately i was a little bit too fast in the trigger, maybe you can help a bit extra also. Problem is that although this does change correctly on /etc/default/snmpd it is not changing for me on /lib/systemd/system/snmpd.service where else would i need to change for it to be applied there?

[bastelfreak]

hey @gittygoo . Is this change still requried? if so, could you please rebase against our latest master?

[kenyon]

This change should actually be made for all versions of Debian and Ubuntu (I have experienced this problem myself). So I've updated the patch. (This may apply to RedHat family too, but I don't use it these days, so don't know for sure.)

[kenyon]

@ghoneycutt unfortunately i was a little bit too fast in the trigger, maybe you can help a bit extra also. Problem is that although this does change correctly on /etc/default/snmpd it is not changing for me on /lib/systemd/system/snmpd.service where else would i need to change for it to be applied there?

@gittygoo for that you need to add a systemd drop-in file. https://forge.puppet.com/modules/camptocamp/systemd#drop-in-files

[elmobp]

@ghoneycutt unfortunately i was a little bit too fast in the trigger, maybe you can help a bit extra also. Problem is that although this does change correctly on /etc/default/snmpd it is not changing for me on /lib/systemd/system/snmpd.service where else would i need to change for it to be applied there?

@gittygoo for that you need to add a systemd drop-in file. forge.puppet.com/modules/camptocamp/systemd#drop-in-files

The systemd ones by default do not use an EnvironmentFile which is why it does not reflect as said above if you use a drop in with that it should work

[vox-pupuli-tasks[bot]]

Dear @gittygoo, thanks for the PR!

This is Vox Pupuli Tasks, your friendly Vox Pupuli GitHub Bot. I noticed that your pull request contains merge conflict. Can you please rebase?

You can find my sourcecode at voxpupuli/vox-pupuli-tasks