Skip to content

chore(opa): Update versions ahead of 25.7.0 #1103

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

Merged
merged 9 commits into from
May 7, 2025

Conversation

NickLarsenNZ
Copy link
Member

@NickLarsenNZ NickLarsenNZ commented May 6, 2025

Part of #1084.

  • Remove 0.67.1
  • Add 1.4.2

Definition of Done Checklist

Note

Not all of these items are applicable to all PRs, the author should update this template to only leave the boxes in that are relevant.

Please make sure all these things are done and tick the boxes

  • All added packages (via microdnf or otherwise) have a comment on why they are added
  • Things not downloaded from Red Hat repositories should be mirrored in the Stackable repository and downloaded from there
  • All packages should have (if available) signatures/hashes verified
  • Add an entry to the CHANGELOG.md file
  • Integration tests ran successfully
TIP: Running integration tests with a new product image

The image can be built and uploaded to the kind cluster with the following commands:

bake --product <product> --image-version <stackable-image-version>
kind load docker-image <image-tagged-with-the-major-version> --name=<name-of-your-test-cluster>

See the output of bake to retrieve the image tag for <image-tagged-with-the-major-version>.

NOTE: The dnf package was 1.23.6 and OPA required 1.23.8
NOTE: I tried making the version paramaterized, but bake wouldn't allow
it (it worked fine with `docker build` and `docker build xbuild`).
@NickLarsenNZ NickLarsenNZ force-pushed the chore/bump-opa-pre-25.7.0 branch from 71cb241 to 972d4a3 Compare May 7, 2025 09:32
@NickLarsenNZ
Copy link
Member Author

Things not downloaded from Red Hat repositories should be mirrored in the Stackable repository and downloaded from there

@lfrancke, this PR changes the Go installation from dnf to using an image in Docker Hub.
Should this instead be mirrored?

@NickLarsenNZ
Copy link
Member Author

NickLarsenNZ commented May 7, 2025

Smoke tests pass:

--- PASS: kuttl (122.65s)
    --- PASS: kuttl/harness (0.00s)
        --- PASS: kuttl/harness/smoke_opa-1.0.1,local-opa_1.0.1_openshift-false (60.79s)
        --- PASS: kuttl/harness/smoke_opa-1.4.2,local-opa_1.4.2_openshift-false (61.84s)
PASS

@NickLarsenNZ NickLarsenNZ moved this to Development: Waiting for Review in Stackable Engineering May 7, 2025
@NickLarsenNZ NickLarsenNZ marked this pull request as ready for review May 7, 2025 09:37
@lfrancke
Copy link
Member

lfrancke commented May 7, 2025

Should this instead be mirrored?

Yes. Especially if it comes from DockerHub with its rate limits.

@NickLarsenNZ
Copy link
Member Author

Should this instead be mirrored?

Yes. Especially if it comes from DockerHub with its rate limits.

I added it to the mirror workflow in 8e755a3, and then referenced it in b14edeb (refactored in 52f4431).

@NickLarsenNZ NickLarsenNZ added this pull request to the merge queue May 7, 2025
@Techassi Techassi moved this from Development: Waiting for Review to Development: In Review in Stackable Engineering May 7, 2025
Merged via the queue into main with commit bfd6bc9 May 7, 2025
3 checks passed
@NickLarsenNZ NickLarsenNZ deleted the chore/bump-opa-pre-25.7.0 branch May 7, 2025 10:51
@Techassi Techassi moved this from Development: In Review to Development: Done in Stackable Engineering May 7, 2025
labrenbe pushed a commit that referenced this pull request May 8, 2025
* chore(opa): Remove 0.67.1

* chore(opa): Remove legacy bundle-builder

* chore(opa): Bump ubi9 base image

* chore(opa): Add 1.4.2

* fix(opa): Manually install Go version

NOTE: The dnf package was 1.23.6 and OPA required 1.23.8
NOTE: I tried making the version paramaterized, but bake wouldn't allow
it (it worked fine with `docker build` and `docker build xbuild`).

* ci(mirror): Add golang

* chore(opa): Use mirrored golang image

* chore(opa): Parameterise golang container version

* chore: Update changelog
@lfrancke lfrancke moved this from Development: Done to Done in Stackable Engineering May 9, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Development

Successfully merging this pull request may close these issues.

3 participants