-
Notifications
You must be signed in to change notification settings - Fork 107
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
OPA v1 support #275
base: master
Are you sure you want to change the base?
OPA v1 support #275
Conversation
eb2ae6b
to
15c6afc
Compare
15c6afc
to
e8767d7
Compare
Hello @ashutosh-narkar I've added a fix for one of the failing tests, but there's still one of the tests failing - The test uses following opa:
services:
controller:
url: 'https://www.openpolicyagent.org'
bundles:
quickstart:
service: controller
resource: /bundles/helm-kubernetes-quickstart
default_decision: /helm_kubernetes_quickstart/main The idea is to setup static bundle when installing OPA and then the test that bundle is added to OPA rules. You can check error logs, and they say that loading that bundle failed Can you plz check this and fix the published bundle ? |
* upgrade OPA to v1 * update authz policy to be v1 compatible * fix E2E tests Fixes: #272 Signed-off-by: Ashutosh Narkar <[email protected]> Signed-off-by: Ievgenii Shepeliuk <[email protected]>
e8767d7
to
5fc8d4e
Compare
Thnx for updates. As soon as this change available - I will try to update PR to check if tests are passing, so we can eventually merge this PR. |
@eshepelyuk we merged open-policy-agent/opa#7342. Thanks for your help! |
Hello @ashutosh-narkar services:
- name: controller
url: 'https://www.openpolicyagent.org'
bundles:
quickstart:
service: controller
resource: /bundles/helm-kubernetes-quickstart And I run it with opa run -c values.yaml -s In logs I see following errors
|
No description provided.