Skip to content

Build docker test containers on release#943

Merged
fdupress merged 2 commits intomainfrom
ci-build-docker-test-on-release
Mar 20, 2026
Merged

Build docker test containers on release#943
fdupress merged 2 commits intomainfrom
ci-build-docker-test-on-release

Conversation

@fdupress
Copy link
Member

No description provided.

@fdupress
Copy link
Member Author

Quick note: I'd like to merge this before the 2026.03 release. But I'd also like to revisit this to try and reuse the fact that the base and build boxes could be relied upon instead of being rebuilt by the test job. If time permits, before merging; otherwise, later.

test-box always built `main` even when tagged, this
uses the tag as a version selector.
@fdupress fdupress force-pushed the ci-build-docker-test-on-release branch from c0fb5f9 to aa7a341 Compare March 17, 2026 19:09
@fdupress fdupress requested a review from strub March 17, 2026 19:30
@fdupress
Copy link
Member Author

The container is tested and functional. (Tested on WSL docker, and on a standard Linux via docker and podman.)

@fdupress fdupress force-pushed the ci-build-docker-test-on-release branch 2 times, most recently from 6d59992 to 346557f Compare March 19, 2026 19:04
@fdupress fdupress force-pushed the ci-build-docker-test-on-release branch from 346557f to d29eef0 Compare March 19, 2026 19:36
@fdupress
Copy link
Member Author

This is properly ready to go now.

  • Building the test box doesn't rebuild the whole stack of boxes;
  • The test box is not built on the main branch; and
  • The exclusion condition has been tested.

In addition, I've streamlined the jobs a little. This could be done as a matrix forced to run sequentially (not sure how I'd do the exclusion for the test box, then), but this feels simple enough to maintain.

@strub --- quick review ahead of the release? (REVERT ME will be reverted before merge; it's only there to test the workflow and its exclusion condition before merge)

@fdupress fdupress force-pushed the ci-build-docker-test-on-release branch from d29eef0 to a5d3d35 Compare March 20, 2026 10:24
@fdupress fdupress merged commit 7035747 into main Mar 20, 2026
16 checks passed
@fdupress fdupress deleted the ci-build-docker-test-on-release branch March 20, 2026 13:33
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