Skip to content

libpod: improve createRootlessContainer#25575

Merged
openshift-merge-bot[bot] merged 1 commit intocontainers:mainfrom
giuseppe:test-not-safe-for-parallel
Mar 13, 2025
Merged

libpod: improve createRootlessContainer#25575
openshift-merge-bot[bot] merged 1 commit intocontainers:mainfrom
giuseppe:test-not-safe-for-parallel

Conversation

@giuseppe
Copy link
Copy Markdown
Member

do not run the expensive pmount.GetMounts() function if it is not needed.

As a follow-up for commit c9c44d4, do not restore the propagation flag for the parent mount to shared unless it was changed to slave first.

Does this PR introduce a user-facing change?

None

@giuseppe giuseppe added the No New Tests Allow PR to proceed without adding regression tests label Mar 13, 2025
@openshift-ci openshift-ci bot added release-note-none approved Indicates a PR has been approved by an approver from all required OWNERS files. labels Mar 13, 2025
do not run the expensive pmount.GetMounts() function if it is not
needed.

As a follow-up for commit c9c44d4, do
not restore the propagation flag for the parent mount to shared unless
it was changed to slave first.

Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
@giuseppe giuseppe force-pushed the test-not-safe-for-parallel branch from c7786c1 to a2953da Compare March 13, 2025 15:55
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

@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci bot commented Mar 13, 2025

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: giuseppe, Luap99

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

@mheon
Copy link
Copy Markdown
Member

mheon commented Mar 13, 2025

Sure, LGTM

@mheon
Copy link
Copy Markdown
Member

mheon commented Mar 13, 2025

/lgtm

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Mar 13, 2025
@openshift-merge-bot openshift-merge-bot bot merged commit 79e05ca into containers:main Mar 13, 2025
78 of 79 checks passed
@stale-locking-app stale-locking-app bot added the locked - please file new issue/PR Assist humans wanting to comment on an old issue or PR with locked comments. label Jun 12, 2025
@stale-locking-app stale-locking-app bot locked as resolved and limited conversation to collaborators Jun 12, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. lgtm Indicates that a PR is ready to be merged. locked - please file new issue/PR Assist humans wanting to comment on an old issue or PR with locked comments. No New Tests Allow PR to proceed without adding regression tests release-note-none

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants