-
Notifications
You must be signed in to change notification settings - Fork 33
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
Add choria::group
to manage groups
#324
Conversation
b2240b4
to
d5c8ecf
Compare
CI on Ruby 2.4 can probably be dropped (anything older than 2.7 is EOL and 2.7 reach EOL in 1 month). Puppet 6 ship with Ruby 2.5. Puppet 7 ship with Ruby 2.7. Puppet 8 will ship Ruby 3.2. Maybe we can just remove 2.4 for now, when Puppet 6 reach EOL (soon), we can remove 2.5 too. We can also add 3.1 / 3.2 anytime. But maybe this is for another PR? |
looks great, yeah feel free to do CI changes here, squash these commits to one then add one for the CI version changes etc. I havnt tested if all works on ruby 3 yet though myself |
10780d2
to
2324376
Compare
I will review this one soon, been on a business trip! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM thank you!
This defined type allow to conveniently manage groups of choria users.
Remove oldest version of Ruby (Puppet 6 is too modern for Ruby 2.4) and add recent versions. While here, switch from the legacy circleci ruby images to the new ones: https://circleci.com/developer/images/image/cimg/ruby
You can merge whenever you're ready @smortex |
Merged. I'll keep the branch around until I update our Puppet config to not track it at $WORK next week. |
This is a follow-up to a previous comment and recent discussion on slack. This PR add a Puppet defined type to help managing groups of users in choria policies.