-
Notifications
You must be signed in to change notification settings - Fork 26
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
Update dependencies to latest versions #84
Conversation
@anderseknert I've updated the dependencies and tested it. all tests were green and I was able to reproduce successfully the tutorial in the docs. This PR is related to: |
Thanks Denis! Looks like you may need to run You'll also need to sign-off on the commit for the DCO check, i..e:
|
Signed-off-by: Denis Rendler <connect@rendler.net>
good morning, @anderseknert ! done on all accounts :D thank you for the quick reply on this one :) |
@rendler-denis I just pushed 0.9 with your changes included. Try it out and let me know if things work as expected :) |
I was just about to look if the new version is up. Will do and comeback with results. Thanks for the quick response. |
@anderseknert I tried installing the plugin on a x64 machine and I get an error that says:
and when I look at the file on the disk I see this:
Am I reading this correctly, is it installing an ARM version? This is the command I used to install the plugin: docker plugin install openpolicyagent/opa-docker-authz-v2:0.9 opa-args="-config-file /opa/config/config.yaml" |
Doh! 🤦 We don't have a publishing pipeline set up for this, so of course... I'll need to cross compile for Linux AMD64 and have that pushed (I'm on an M1 mac myself). I'll look into it. Sorry about that! |
@anderseknert no worries. I suspected that. I'll take a look and maybe contribute with a pipeline as well :) But first I need to learn a bit more about the plugins architecture |
@rendler-denis it's hard contributing anything where credentials is involved. I might take a stab at it at some point but this plugin has been pushed infrequently enough that it's not been worth the effort up til this point at least. I've just pushed v0.9 again, this time using GOOS=linux and GOARCH=amd64. Could you please try again to see if it works better now? At least I got the exec format error when trying to run the binary now, so whatever it is it isn't Mac ARM 😄 |
@anderseknert I can confirm, now it works. Thank you. |
No description provided.