v2: Upgrade OPA#4268
Conversation
Signed-off-by: Mark Phelps <[email protected]>
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## v2 #4268 +/- ##
=======================================
Coverage 50.65% 50.65%
=======================================
Files 122 122
Lines 13200 13200
=======================================
Hits 6686 6686
Misses 5950 5950
Partials 564 564
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
I guess we need to address this othewise UI is not working |
ah good call. will do that now |
Signed-off-by: Mark Phelps <[email protected]>
|
The latest Buf updates on your PR. Results from workflow Proto / proto-lint (pull_request).
|
|
Added to I kept them all as read actions until we have a better way of handling branched env permissions |
erka
left a comment
There was a problem hiding this comment.
nice!
The main challenge was updating the package declaration in my existing policy.rego file to package flipt.authz.v2.
It would be helpful to include this as a migration tip in the v2 documentation for users upgrading from earlier v1 versions.
I'll create a new issue to track notes we should add to migration docs |
upgrade OPA rego/storage libraries to v1.x https://github.com/open-policy-agent/opa
From the looks of it we should be backward compatible as we dont import the
future.keywords: https://www.openpolicyagent.org/docs/v0-compatibilitysee: #4267 as well