Skip to content

Add TEST_ONLY build arg for each distro#251

Merged
crazy-max merged 1 commit intodocker:mainfrom
crazy-max:filter-distros
Aug 20, 2025
Merged

Add TEST_ONLY build arg for each distro#251
crazy-max merged 1 commit intodocker:mainfrom
crazy-max:filter-distros

Conversation

@crazy-max
Copy link
Copy Markdown
Member

@crazy-max crazy-max commented Aug 20, 2025

alternative to and closes #245

Signed-off-by: CrazyMax <1951866+crazy-max@users.noreply.github.com>
@crazy-max crazy-max changed the title RELEASE bake var to filter out unsupported distros https://github.com/docker/packaging/pull/251/commits Aug 20, 2025
@crazy-max crazy-max changed the title https://github.com/docker/packaging/pull/251/commits Add TEST_ONLY build arg for each distro Aug 20, 2025
@crazy-max crazy-max merged commit 9e5cd16 into docker:main Aug 20, 2025
542 of 543 checks passed
@crazy-max crazy-max deleted the filter-distros branch August 20, 2025 11:42
DISTRO_ID = ""
DISTRO_SUITE = ""
DISTRO_IMAGE = DISTRO_IMAGE != null && DISTRO_IMAGE != "" ? DISTRO_IMAGE : "debian:bookworm"
TEST_ONLY = "0"
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Silly thought; would it make sense to do the reverse, i.e. currently it's opt-out when adding a distro; wondering if it should be "opt-in" to be deployed.

Or we could even have a CHANNELS param to allow having upcoming distros in our test channel, but not in stable.

Copy link
Copy Markdown
Collaborator

@vvoland vvoland Aug 20, 2025

Choose a reason for hiding this comment

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

Or we could even have a CHANNELS param to allow having upcoming distros in our test channel, but not in stable.

Yeah this is basically what it does, except less mouthy. TEST_ONLY will just deploy to test channel.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Oh! I thought it was "test in CI only, but don't deploy"

FWIW, we had some examples of that in the other repositories, where we test "Fedora Rawhide" and "Debian sid" (testing) in CI.

Those should never, ever be deployed (not even in the "test" channel), but are good to have to validate if upcoming distro releases need changes in packaging.

So that was my thinking behind a "channels" arg; which could be empty if a distro should not be deployed at all

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.

3 participants