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

Move to Vox Pupuli #29

Closed
roidelapluie opened this issue Oct 4, 2016 · 25 comments
Closed

Move to Vox Pupuli #29

roidelapluie opened this issue Oct 4, 2016 · 25 comments
Labels
enhancement New feature or request

Comments

@roidelapluie
Copy link
Member

What do you think about moving puppet-acl to Vox Pupuli to get more people working on it, and get higher visibility plus better standards?

@dobbymoodge
Copy link

dobbymoodge commented Oct 4, 2016 via email

@roidelapluie
Copy link
Member Author

It is there: https://voxpupuli.org/docs/

@dobbymoodge
Copy link

I suspect we should resolve #26 first. Any idea on how we could do that without breaking existing deployments?

@roidelapluie
Copy link
Member Author

The only way to do #26 is by bumping major version so pple will know it's breaking

@roidelapluie
Copy link
Member Author

puppet-posixacl and posix_acl type?

@dobbymoodge
Copy link

sounds good to me

@roidelapluie
Copy link
Member Author

Well this module is not meant to be only for posix acl, so I think keeping acl is fine ?

@claflico
Copy link

claflico commented Oct 9, 2016

@roidelapluie Does that mean that enhancements to this module will be capable of what the puppetlabs/acl Windows module does?

@tequeter
Copy link

I don't think keeping acl is fine: until the module and its type are renamed, #26 will be a blocking issue for multi-platform shops. I'll have to fork this module and rename the day I need to manage Windows ACLs on the same Puppet server.

If you have a strong opinion on supporting Windows ACL (or some other breed of ACL?), then at the very least make it "acl2". But IMHO it is fine to have this module as a POSIX specialist.

@roidelapluie
Copy link
Member Author

ok guys.

posix_acl it will be then.

@dobbymoodge
Copy link

@roidelapluie I still haven't gotten started on this, unfortunately. I'll try to get working on #26 this weekend, but if you have something in some stage of development, you could add it as a branch on this repo and we could finish it together.

@dobbymoodge dobbymoodge added the enhancement New feature or request label Oct 27, 2016
@dobbymoodge
Copy link

dobbymoodge commented Feb 7, 2017

@roidelapluie actually I think file_acl would be a better name.

*** actually no, posix_acl is probably better

@ekohl
Copy link
Member

ekohl commented Aug 21, 2017

Any progress on this? It looks like we might need a module to maintain ACLs in the foreman installer and this I could help if needed.

@dobbymoodge
Copy link

@ekohl Real Life and Paying Work have kept this task at the bottom of my list. If you want to submit a PR for the module rename, that would help things move along a bit though. :D

@vandelin586
Copy link

What are people using in the interim for a solution ?

@dobbymoodge
Copy link

@vandelin586 I don't know what alternatives exist. I'm really sorry I've let this issue dangle for over a year now. I've just merged #33 so if anyone wants to help with the next steps, I'd be thrilled. For now I'm still pretty much underwater with my Real Job so I'm not sure when I'll have time to invest in this particular issue :(

@vandelin586
Copy link

If I knew where to start or what to do , I would try my best. A person I work with copied this code and redesigned the naming so that the module named was not acl as puppetlabs-acl is. Just changing the module name was not all that is involved and this did not work, I believe what I seen in your latest merge #33 is more of what we need. or perhaps a combination of the two. When trying to use your most recent type name posix_acl I get back an error saying could not find class acl::requirements

I dont know , im a newb at this stuff , I will work hard at trying to understand it over this next weekend, right now setting linux acl is a show stopper for me, might revert to some exec or something if I cant figure anything out

@vandelin586
Copy link

nevermind posix_acl type works just had stale data from the older module

@ekohl
Copy link
Member

ekohl commented Oct 21, 2017

@dobbymoodge I can help with the migration aspect of it and send some PRs to update code but you'll have to do the github transfer and upload a module on the forge that states it's deprecated.

@bastelfreak
Copy link
Member

Hi @dobbymoodge, I sent you an invitation to the Vox Pupuli org on github. After accepting it, you're able to migrate the repositoy.

@claflico
Copy link

@bastelfreak I'm moving on from Puppet. Can I get an invite too so I can migrate my winlogbeat repo?

@bastelfreak
Copy link
Member

@claflico invited you as well. Feel free to join our IRC channel #voxpupuli on freenode for further discussions.

@ekohl
Copy link
Member

ekohl commented Dec 13, 2017

Looks like this has been resolved now.

@ekohl ekohl closed this as completed Dec 13, 2017
@crispygoth
Copy link

This issue got closed and a 0.0.5 release has been tagged on GitHub, but I can't seem to see a voxpupuli release on the Forge? https://forge.puppet.com/puppet/posix_acl gives a "page not found" error. Is there something that needs doing to make it appear there?

@bastelfreak
Copy link
Member

@crispygoth it is now available: https://forge.puppet.com/puppet/posix_acl

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

8 participants