Skip to content

fix: Regexp of network devices to remove#533

Merged
urunc-bot[bot] merged 1 commit intomainfrom
fix_regexp_in_network_removal
Mar 20, 2026
Merged

fix: Regexp of network devices to remove#533
urunc-bot[bot] merged 1 commit intomainfrom
fix_regexp_in_network_removal

Conversation

@cmainas
Copy link
Copy Markdown
Contributor

@cmainas cmainas commented Mar 13, 2026

Description

Fix the regexp that compares against the network interfaces to find urunc tap devices. Make it follow the correct naming convention we use for tap devices in urunc.

Related issues

How was this tested?

In a k3s environment deployed a urunc container, killed the monitor process and checked if the restarted pod is responsive to network requests.

LLM usage

N/A

Checklist

  • I have read the contribution guide.
  • The linter passes locally (make lint).
  • The e2e tests of at least one tool pass locally (make test_ctr, make test_nerdctl, make test_docker, make test_crictl).
  • If LLMs were used: I have read the llm policy.

@netlify
Copy link
Copy Markdown

netlify Bot commented Mar 13, 2026

Deploy Preview for urunc canceled.

Name Link
🔨 Latest commit f077a2e
🔍 Latest deploy log https://app.netlify.com/projects/urunc/deploys/69bd18889fbbe700084d5273

@cmainas cmainas force-pushed the fix_regexp_in_network_removal branch from 83b8fd0 to e0d07ed Compare March 13, 2026 15:40
@cmainas cmainas requested a review from ananos March 13, 2026 15:59
@cmainas cmainas force-pushed the fix_regexp_in_network_removal branch from e0d07ed to f3a9af2 Compare March 20, 2026 08:25
Copy link
Copy Markdown
Contributor

@ananos ananos left a comment

Choose a reason for hiding this comment

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

thanks @cmainas! LGTM!

Fix the regexp that compares the network interfaces names to find
urunc tap devices. Make it follow the correct naming convention we use
for tap devices in urunc.

PR: #533
Signed-off-by: Charalampos Mainas <charalampos.mainas@gmail.com>
Reviewed-by: Anastassios Nanos <ananos@nubificus.co.uk>
Approved-by: Anastassios Nanos <ananos@nubificus.co.uk>
@github-actions github-actions Bot force-pushed the fix_regexp_in_network_removal branch from f3a9af2 to f077a2e Compare March 20, 2026 09:51
@urunc-bot urunc-bot Bot merged commit f64191a into main Mar 20, 2026
2 of 4 checks passed
@cmainas cmainas deleted the fix_regexp_in_network_removal branch March 27, 2026 15:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants