Skip to content

Add Debian Trixie support for debian-base - #4501

Merged
kubernetes-prow[bot] merged 3 commits into
kubernetes:masterfrom
wespanther:debian-base-trixie
Aug 25, 2026
Merged

Add Debian Trixie support for debian-base#4501
kubernetes-prow[bot] merged 3 commits into
kubernetes:masterfrom
wespanther:debian-base-trixie

Conversation

@wespanther

@wespanther wespanther commented Aug 25, 2026

Copy link
Copy Markdown
Contributor

What type of PR is this?

/kind feature

What this PR does / why we need it:

This adds support to build the debian-base image based on Debian Trixie. This is required because the previous version (Bookworm) is out of support as of 2026-06-10, and will no longer receive patches or updated packages (e.g. python 3.12). This was originally proposed in #4312.

I listed all of the dependencies using debian:trixie-slim to verify I had them correct. I successfully built the image using:

CONFIG=trixie IMAGE_VERSION=trixie-v1.0.0 make build

Note: we may have to revisit the cloud build timeouts based on the other PR (though it failed to build). We will also need to update the defaults in the main images/build/debian-base makefile. I'll send a followup for this one.

Which issue(s) this PR fixes:

N/A

Special notes for your reviewer:

N/A

Does this PR introduce a user-facing change?

Add Debian Trixie support for debian-base

@wespanther
debian-base: copy bookworm to trixie
f5329de 
Starting with a copy from Bookworm to keep the commits clean. The
Trixie-specific changes will be in a following commit.
@wespanther
debian-base: add debian trixie support
2b840d5 
Added libext2fs2t64 and the entry in variants.yaml. Verified other
package names in debian:trixie-slim.
@kubernetes-prow kubernetes-prow Bot added release-note Denotes a PR that will be considered when it comes time to generate release notes. kind/feature Categorizes issue or PR as related to a new feature. needs-priority cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. labels Aug 25, 2026
@kubernetes-prow

kubernetes-prow Bot commented Aug 25, 2026

Copy link
Copy Markdown
Contributor

Hi @wespanther. Thanks for your PR.

I'm waiting for a kubernetes member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work.

Tip

We noticed you've done this a few times! Consider joining the org to skip this step and gain /lgtm and other bot rights. We recommend asking approvers on your previous PRs to sponsor you.

Once the patch is verified, the new status will be reflected by the ok-to-test label.

I understand the commands that are listed here.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

@kubernetes-prow kubernetes-prow Bot added the needs-ok-to-test Indicates a PR that requires an org member to verify it is safe to test. label Aug 25, 2026
@kubernetes-prow
kubernetes-prow Bot requested a review from jrsapi August 25, 2026 05:48
@kubernetes-prow kubernetes-prow Bot added the area/release-eng Issues or PRs related to the Release Engineering subproject label Aug 25, 2026
@kubernetes-prow
kubernetes-prow Bot requested a review from salaxander August 25, 2026 05:48
@kubernetes-prow kubernetes-prow Bot added sig/release Categorizes an issue or PR as relevant to SIG Release. size/L Denotes a PR that changes 100-499 lines, ignoring generated files. labels Aug 25, 2026
@BenTheElder

BenTheElder commented Aug 25, 2026

Copy link
Copy Markdown
Member

/ok-to-test

@kubernetes-prow kubernetes-prow Bot added ok-to-test Indicates a non-member PR verified by an org member that is safe to test. and removed needs-ok-to-test Indicates a PR that requires an org member to verify it is safe to test. labels Aug 25, 2026
@wespanther

wespanther commented Aug 25, 2026

Copy link
Copy Markdown
Contributor Author

The build made it through all of the bookworm variants and 4 of the trixie variants, but timed out on s390x (the last one). I'll up the Cloud Build timeout slightly, we can drop it back down if / when we remove the bookworm image.

@wespanther
Bump the Cloud Build timeout
923add8 
Adding 5 minutes to the Cloud Build timeout to account for building the
trixie variants. This can be adjusted when trixie becomes default and
bookworm is removed.
@wespanther

wespanther commented Aug 25, 2026

Copy link
Copy Markdown
Contributor Author

/retest

BenTheElder
BenTheElder reviewed Aug 25, 2026
View reviewed changes

@BenTheElder BenTheElder left a comment

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.

/lgtm
/approve

@kubernetes-prow kubernetes-prow Bot added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Aug 25, 2026
@kubernetes-prow

kubernetes-prow Bot commented Aug 25, 2026

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: BenTheElder, wespanther

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@kubernetes-prow kubernetes-prow Bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Aug 25, 2026
@kubernetes-prow
kubernetes-prow Bot merged commit 3bc894a into kubernetes:master Aug 25, 2026
6 checks passed
@wespanther wespanther mentioned this pull request Aug 26, 2026
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jrsapi jrsapi Awaiting requested review from jrsapi
@salaxander salaxander Awaiting requested review from salaxander
1 more reviewer
@BenTheElder BenTheElder BenTheElder left review comments
Reviewers whose approvals may not affect merge requirements

Assignees

@BenTheElder BenTheElder

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. area/release-eng Issues or PRs related to the Release Engineering subproject cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. kind/feature Categorizes issue or PR as related to a new feature. lgtm "Looks good to me", indicates that a PR is ready to be merged. needs-priority ok-to-test Indicates a non-member PR verified by an org member that is safe to test. release-note Denotes a PR that will be considered when it comes time to generate release notes. sig/release Categorizes an issue or PR as relevant to SIG Release. size/L Denotes a PR that changes 100-499 lines, ignoring generated files.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@wespanther @BenTheElder