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

OPA v1.0 Updates #57

Merged
merged 3 commits into from
Dec 24, 2024
Merged

OPA v1.0 Updates #57

merged 3 commits into from
Dec 24, 2024

Conversation

christophwille
Copy link
Owner

@christophwille christophwille commented Dec 22, 2024

#56

Delta of capabilities (meant adjusting concat-capabilities.ps1):

image

@christophwille
Copy link
Owner Author

christophwille commented Dec 22, 2024

Not fixed / working yet:

  • simple-custom-builtincall.rego
  • custom-builtincall.rego
  • math-builtin.rego

Eg ./opa build -t wasm -e "math/builtins/result" --capabilities unittest.capabilities.json math-builtin.rego yields

error: load error: 1 error occurred during loading: rego_parse_error: illegal capabilities: rego_v1 feature required for parsing v1 Rego

Need to check https://www.openpolicyagent.org/docs/latest/v0-compatibility/ and https://github.com/open-policy-agent/opa/releases/tag/v1.0.0 (not been watching OPA changes since 0.68 to v1)

@srenatus could you be so nice to point me in the right direction of builtin changes & the adaptions I'd need to make for v1 transition?

@srenatus
Copy link

Can you please file an issue upstream? I can't fully engage right now and you'll have better chances at getting assitance.

@christophwille
Copy link
Owner Author

@christophwille christophwille merged commit fbfe07b into master Dec 24, 2024
2 checks passed
@christophwille christophwille deleted the fix/56 branch December 24, 2024 09:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants