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

Fix OPA Integration systemtests #10980

Merged
merged 1 commit into from
Dec 22, 2024
Merged

Fix OPA Integration systemtests #10980

merged 1 commit into from
Dec 22, 2024

Conversation

scholzj
Copy link
Member

@scholzj scholzj commented Dec 21, 2024

Type of change

  • Bugfix

Description

The OPA tests are broken after OPA 1.0.0 release yesterday which brought some breaking changes (removed some old stuff). This PR fixes the issues and also fixes the OPA version used in the STs to 1.0.0 instead of using latest to avoid such surprises in the furure.

Checklist

  • Make sure all tests pass
  • Try your changes from Pod inside your Kubernetes and OpenShift cluster, not just locally

@scholzj scholzj added this to the 0.46.0 milestone Dec 21, 2024
@scholzj scholzj requested a review from im-konge December 21, 2024 18:30
@scholzj scholzj requested a review from a team December 21, 2024 18:31
@scholzj
Copy link
Member Author

scholzj commented Dec 21, 2024

/azp run regression

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

Copy link
Member

@im-konge im-konge left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks. And sorry for the issues

@scholzj scholzj merged commit 851cff1 into strimzi:main Dec 22, 2024
19 of 21 checks passed
@scholzj scholzj deleted the fix-OPA-STs branch December 22, 2024 00:47
scholzj added a commit to scholzj/strimzi-kafka-operator that referenced this pull request Jan 13, 2025
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