-
-
Notifications
You must be signed in to change notification settings - Fork 131
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix for "error on subcontainer 'ia_addr' insert (-1)" messages in syslog #191
base: master
Are you sure you want to change the base?
Conversation
@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 |
@ghoneycutt unfortunately i was a little bit too fast in the trigger, maybe you can help a bit extra also. |
hey @gittygoo . Is this change still requried? if so, could you please rebase against our latest master? |
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.) |
@gittygoo for that you need to add a systemd drop-in file. https://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 |
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 |
Fix for https://bugs.launchpad.net/ubuntu/+source/net-snmp/+bug/1246347