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.

Sign up for GitHub

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

Jump to bottom

Upgrade to opa v1.0.1 #3407

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Upgrade to opa v1.0.1 #3407

wants to merge 4 commits into from

Conversation

Pushpalanka
Copy link
Collaborator

@Pushpalanka Pushpalanka commented Feb 13, 2025 •
edited
Loading

Upgrade from OPA 0.70.0 to 1.0.1

@Pushpalanka Pushpalanka added dependencies Pull requests that update a dependency file major moderate risk, for example new API, small filter changes that have no risk like refactoring or logs labels Feb 13, 2025
@Pushpalanka Pushpalanka force-pushed the upgrade-opa-v1.0.1 branch 3 times, most recently from 1600aed to dba5d43 Compare February 13, 2025 17:17
@Pushpalanka Pushpalanka marked this pull request as ready for review February 17, 2025 13:02
@mjungsbluth
Copy link
Collaborator

LGTM

@mjungsbluth
Copy link
Collaborator

Let's just align internally before merging this to validate all existing policies are ready.

@Pushpalanka
Upgrade to opa v1.0.1
a257d63 
Signed-off-by: Pushpalanka Jayawardhana <[email protected]>
@Pushpalanka
Rebase on master
6267c0d 
Signed-off-by: Pushpalanka Jayawardhana <[email protected]>
@Pushpalanka Pushpalanka requested a review from wisinghe March 10, 2025 07:44
@Pushpalanka
Copy link
Collaborator Author

👍

1 similar comment
@wisinghe
Copy link
Collaborator

👍

@Pushpalanka
Copy link
Collaborator Author

👍

@AlexanderYastrebov
Copy link
Member

Let's just align internally before merging this to validate all existing policies are ready.

LGTM, lets rebase and merge when you are ready.

@AlexanderYastrebov
Copy link
Member

👍

@Pushpalanka
Copy link
Collaborator Author

Pushpalanka commented Mar 17, 2025 •
edited
Loading

Let's just align internally before merging this to validate all existing policies are ready.

LGTM, lets rebase and merge when you are ready.

Internal validations are completed to make sure existing bundles are compatible.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mjungsbluth mjungsbluth Awaiting requested review from mjungsbluth

@MustafaSaber MustafaSaber Awaiting requested review from MustafaSaber

@wisinghe wisinghe Awaiting requested review from wisinghe

Assignees
No one assigned
Labels
dependencies Pull requests that update a dependency file major moderate risk, for example new API, small filter changes that have no risk like refactoring or logs
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@Pushpalanka @mjungsbluth @wisinghe @AlexanderYastrebov