Skip to content

Conversation

@ashley-cui
Copy link
Member

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?

The --network-cmd-path option has been removed, as part of the slirp4netns removal.

As part of slirp removal.
RUN-3569

Signed-off-by: Ashley Cui <[email protected]>
@ashley-cui ashley-cui added the No New Tests Allow PR to proceed without adding regression tests label Dec 19, 2025
@ashley-cui ashley-cui marked this pull request as ready for review December 19, 2025 23:24
@packit-as-a-service
Copy link

[NON-BLOCKING] Packit jobs failed. @containers/packit-build please check. Everyone else, feel free to ignore.

Copy link
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

Copy link
Member

@lsm5 lsm5 left a comment

Choose a reason for hiding this comment

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

/lgtm

@lsm5 lsm5 merged commit 38017c4 into containers:main Dec 22, 2025
78 checks passed
@ashley-cui
Copy link
Member Author

Could we merge containers/container-libs#564 as well? :)

@lsm5
Copy link
Member

lsm5 commented Dec 22, 2025

Could we merge containers/container-libs#564 as well? :)

i don't have merge rights there, so I guess not until Jan 2.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

No New Tests Allow PR to proceed without adding regression tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants