-
Notifications
You must be signed in to change notification settings - Fork 32
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
Wrong rights on lib/puppet/provider/gms_webhook/github.rb #28
Comments
@mvisonneau Well, that is lame. My apologies. Will cut another release soon. Thanks for the heads up. Confirmed the permissions issue when I did a puppet module install abrader/gms as well. |
No worries, thanks! |
@abrader can you push a new release for this? It's currently blocking this module from working properly.. |
@walterheck will do today. |
Did this ever get fixed? ...in master? Still seeing this in the |
Still seeing this in |
If you're using Code Manager and seeing this issue even on The fix for me was to delete the That same bug might also affect |
I am indeed using |
I'm using r10k. The bug (a minor oversight, really) seems to be in the module itself as packaged on the Forge, not in Puppet or r10k. The zipfile of 1.0.2 from github looks like it has the right permissions, which makes me think the problem is in the package the Forge is supplying. @mvisonneau's work-around, even if temporary, is the right one: setting the read bit. No exec permissions are needed. |
@mvisonneau @rjhornsby @djjudas21 @natemccurdy @walterheck Please inspect release 1.0.3 for similar permission issues. I inspected tarball prior to upload to Forge and confirmed all were set correctly. |
@abrader |
1.0.3 looks like that fixed it. Thanks! |
@abrader works for me as well ! Closing the issue then, thanks for the fix :) |
Hey,
Got this error after upgrading to 1.0.2 :
Adding missing read right on others solves it :
Cheers!
The text was updated successfully, but these errors were encountered: