Skip to content

fix: improve network isolation#789

Merged
mergify[bot] merged 1 commit intopython-wheel-build:mainfrom
tiran:improve-network-isolation
Sep 25, 2025
Merged

fix: improve network isolation#789
mergify[bot] merged 1 commit intopython-wheel-build:mainfrom
tiran:improve-network-isolation

Conversation

@tiran
Copy link
Copy Markdown
Collaborator

@tiran tiran commented Sep 25, 2025

Use check_output instead of check_call in network isolation detection helper. It captures the output to stdout and avoid spurious warnings about missing commands.

Add debug code to run_network_isolation.sh script. The script detects missing ip command and prints debug message + arguments on error.

Use `check_output` instead of `check_call` in network isolation
detection helper. It captures the output to stdout and avoid spurious
warnings about missing commands.

Add debug code to `run_network_isolation.sh` script. The script detects
missing `ip` command and prints debug message + arguments on error.

Signed-off-by: Christian Heimes <cheimes@redhat.com>
@tiran tiran requested a review from a team as a code owner September 25, 2025 07:36
Copy link
Copy Markdown
Contributor

@pmattsso pmattsso 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
Copy Markdown
Contributor

@EmilienM EmilienM 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
Copy Markdown
Contributor

@rd4398 rd4398 left a comment

Choose a reason for hiding this comment

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

Looks good!

@mergify mergify Bot merged commit fbf8f78 into python-wheel-build:main Sep 25, 2025
112 checks passed
@tiran tiran deleted the improve-network-isolation branch October 6, 2025 08:03
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.

5 participants