Skip to content
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 user and group name change from snmp to Debian-snmp in Ubuntu 18.04 #125

Closed
wants to merge 1 commit into from

Conversation

david-barbion
Copy link

No description provided.

@cmezzetti
Copy link

@razorsedge please merge this PR, it would be wonderful to have this module working on Ubuntu 18.04 without the need to fork
TIA

@bastelfreak
Copy link
Member

Hi @david-barbion, this module got recently migrated to Vox Pupuli. This is similar to #130. I'm fine with merging with whatever is ready first. Please add all supported operating systems to the metadata.json file, also a unit test with rspec-puppet-facts would be nice and a rebase is required.

@cliff-wakefield
Copy link

I don't believe this or PR #130 are needed anymore after PR #134

@alexjfisher
Copy link
Member

I think this is still needed.
See #151

@cliff-wakefield
Copy link

PR #130 rebased.

@Dan33l
Copy link
Member

Dan33l commented Dec 4, 2018

Looks to be done : by #130
7ab970b

@Dan33l Dan33l closed this Dec 4, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants