Skip to content

podman: update to 6.1.4 - #4364

Merged
t-lo merged 3 commits into
mainfrom
t-lo/podman-6.1
Nov 6, 2026
Merged

t-lo merged 3 commits into
mainfrom
t-lo/podman-6.1

Conversation

@t-lo

@t-lo t-lo commented Nov 6, 2026 •

Copy link
Copy Markdown
Member

6.0.0 is missing from Gentoo; our package keywords cause build breakage when trying to emerge podman.

Update: added update changelog and CVEs fixed; 2 critical, one high.

@t-lo
t-lo requested a review from a team as a code owner November 6, 2026 10:40
Copilot AI balanced review requested due to automatic review settings November 6, 2026 10:40

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟡 Changes recommended

Invalid wildcarded lower-bound Portage atoms cause keyword overrides to be ignored, potentially leaving Podman dependencies keyword-masked and blocking the build.

1 open issue
What changed in this PR

Updates Flatcar’s Podman keyword overrides toward 6.1.4 to address build failures caused by Gentoo no longer providing 6.0.0.

Changes:

  • Accepts Podman 6.1-series versions and adjusts dependency keyword ranges.
  • Adds a changelog entry linking to the Podman 6.1.4 release.
File Description
sdk_container/​src/​third_party/​coreos-overlay/​profiles/​coreos/​base/​package.accept_keywords Updates Podman and dependency keyword overrides.
changelog/​updates/​2026-11-06-podman-6.1.4.md Records the Podman 6.1.4 update.

Review effort: Balanced

@t-lo

t-lo commented Nov 6, 2026

Copy link
Copy Markdown
Member Author

@chewi

Copilot is right. Also, Krzesimir has traditionally kept these very specific with the aim of undoing them later.

I meant to use =app-containers/aardvark-dns-2.1*, will fix.

I'd argue to keep this to get automated patch level updates (and maybe work with comments in mask and allow_keywords in the future), particularly since we're testing podman in CI now.

Copilot AI balanced review requested due to automatic review settings November 6, 2026 11:31

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟢 Approval recommended

No active review issues remain for the Podman keyword override and changelog updates.

1 open issue

Review effort: Balanced

@t-lo
t-lo requested review from a team and chewi November 6, 2026 11:40
Signed-off-by: Thilo Fromm <thilofromm@microsoft.com>
Copilot AI balanced review requested due to automatic review settings November 6, 2026 14:41

@krnowak krnowak 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.

Should get a security entry for CVE-2026-94603 (flatcar/Flatcar#2441).

@krnowak

krnowak commented Nov 6, 2026

Copy link
Copy Markdown
Member

I normally don't use wildcard versions for accept keywords, but I think the exception can be made for container stuff, for which we probably want to have bleeding edge versions in Flatcar.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟡 Changes recommended

The security release notes contain a trailing empty advisory placeholder (,[]()), which should be removed so the entry lists only the three supplied advisories.

1 open issue
1 resolved since last review

Review effort: Balanced

Co-authored-by: Krzesimir Nowak <knowak@microsoft.com>
Signed-off-by: Thilo Fromm <thilo.alexander@gmail.com>
Copilot AI balanced review requested due to automatic review settings November 6, 2026 15:04
Fix trailing empty link in podman CVE entries.

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Signed-off-by: Thilo Fromm <thilo.alexander@gmail.com>

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🔵 Needs a closer look

The security changelog contains a trailing ,[]() placeholder that creates an empty advisory link and a dangling comma after the three CVEs.

No open issues 🎉

1 resolved since last review

Review effort: Balanced

Copilot AI balanced review requested due to automatic review settings November 6, 2026 15:07
@t-lo

t-lo commented Nov 6, 2026

Copy link
Copy Markdown
Member Author

Should get a security entry for CVE-2026-94603 (flatcar/Flatcar#2441).

Done, thanks!

I normally don't use wildcard versions for accept keywords, but I think the exception can be made for container stuff, for which we probably want to have bleeding edge versions in Flatcar.

I think we can be more relaxed towards patch level changes in the future for ebuilds in gentoo. Trying to keep the manual work down to a minimum and automate more.

@t-lo
t-lo requested a review from krnowak November 6, 2026 15:09

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟢 Approval recommended

No active review issues were identified for the Podman 6.1.4 update.

No open issues 🎉

Review effort: Balanced

@t-lo
t-lo merged commit db3c622 into main Nov 6, 2026
1 of 3 checks passed
@t-lo
t-lo deleted the t-lo/podman-6.1 branch November 6, 2026 15:45

This branch is waiting to be deployed

1 waiting deployment
development — f35871a6 Waiting Nov 6, 2026 by t-lo via Wait for approval #6091
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.

4 participants