-
Notifications
You must be signed in to change notification settings - Fork 3
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
chore(opa): Bump to 1.0.0 #677
Conversation
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.
You missed one OpaCluster ;)
Also the import rego.v1
should now be able to go 😅
Last but not least we need to update the supported versions list
diff --git a/docs/modules/opa/pages/usage-guide/user-info-fetcher.adoc b/docs/modules/opa/pages/usage-guide/user-info-fetcher.adoc
index 81cf504..6c9f90b 100644
--- a/docs/modules/opa/pages/usage-guide/user-info-fetcher.adoc
+++ b/docs/modules/opa/pages/usage-guide/user-info-fetcher.adoc
@@ -16,7 +16,7 @@ metadata:
name: opa
spec:
image:
- productVersion: 0.67.1
+ productVersion: 1.0.0
clusterConfig:
userInfo: # <1>
backend:
@@ -132,8 +132,6 @@ For example, the following rule allows access for users in the `/admin` group:
----
package test
-import rego.v1
-
default allow := false
allow if {
diff --git a/docs/modules/opa/partials/supported-versions.adoc b/docs/modules/opa/partials/supported-versions.adoc
index f7b0d81..f221808 100644
--- a/docs/modules/opa/partials/supported-versions.adoc
+++ b/docs/modules/opa/partials/supported-versions.adoc
@@ -2,5 +2,5 @@
// This is a separate file, since it is used by both the direct OPA documentation, and the overarching
// Stackable Platform documentation.
-* 0.67.1
-* 0.66.0 (deprecated)
+* 1.0.0
+* 0.67.1 (deprecated)
@sbernauer addressed some with b0ad785 |
Co-authored-by: Sebastian Bernauer <sebastian.bernauer@stackable.de>
Co-authored-by: Sebastian Bernauer <sebastian.bernauer@stackable.de>
…bletech/opa-operator into chore/update-opa-to-1.0.0
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.
Thanks LGTM!
Please watch out for pre-commit
Description
Part of stackabletech/docker-images#969
Definition of Done Checklist
Author
Reviewer
Acceptance