Skip to content

troubleshooting: document keep-id size option#28631

Merged
Honny1 merged 1 commit into
containers:mainfrom
eriksjolund:size-troubleshooting-tip
May 4, 2026
Merged

troubleshooting: document keep-id size option#28631
Honny1 merged 1 commit into
containers:mainfrom
eriksjolund:size-troubleshooting-tip

Conversation

@eriksjolund
Copy link
Copy Markdown
Contributor

@eriksjolund eriksjolund commented May 1, 2026

Checklist

Ensure you have completed the following checklist for your pull request to be reviewed:

  • Certify you wrote the patch or otherwise have the right to pass it on as an open-source patch by signing all
    commits. (git commit -s). (If needed, use git commit -s --amend). The author email must match
    the sign-off email address. See CONTRIBUTING.md
    for more information.
  • Referenced issues using Fixes: #00000 in commit message (if applicable)
  • Tests have been added/updated (or no tests are needed)
  • Documentation has been updated (or no documentation changes are needed)
  • All commits pass make validatepr (format/lint checks)
  • Release note entered in the section below (or None if no user-facing changes)

Does this PR introduce a user-facing change?

None

side note: I also tried --userns=nomap:size=2000 but such a size option does not exist.

$ podman run --rm -d --userns=nomap:size=2000 alpine sleep 3600
Error: unrecognized namespace mode nomap:size=2000 passed

Signed-off-by: Erik Sjölund <erik.sjolund@gmail.com>
@packit-as-a-service
Copy link
Copy Markdown

Cockpit tests failed for commit 8a06465. @martinpitt, @jelly, @mvollmer please check.

@martinpitt
Copy link
Copy Markdown
Contributor

@lsm5 Over the weekend, the cockpit revdeps tests started to fail (first one that I saw was in #28621). They all fail in TestApplication.testDownloadImage. I can reproduce this locally with

dnf copr enable rhcontainerbot/podman-next
dnf update --repo='copr*'

but not with the released Fedora packages. I do some investigation.

@martinpitt
Copy link
Copy Markdown
Contributor

@Luap99 I tested cockpit-project/cockpit-podman#2525 and this indeed fixes the problem, thanks a lot! I'll see to shepherding this quickly.

Copy link
Copy Markdown
Member

@Luap99 Luap99 left a comment

Choose a reason for hiding this comment

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

LGTM

@martinpitt
Copy link
Copy Markdown
Contributor

cockpit-project/cockpit-podman#2525 landed, so every cockpit revdeps test from now on should work again. Thanks @Luap99 !

@Luap99
Copy link
Copy Markdown
Member

Luap99 commented May 4, 2026

/packit retest-failed

Copy link
Copy Markdown
Member

@Honny1 Honny1 left a comment

Choose a reason for hiding this comment

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

LGTM

@Honny1 Honny1 merged commit 8dcb5fb into containers:main May 4, 2026
42 checks passed
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