Skip to content

Conversation

Copy link

Copilot AI commented Jan 29, 2026

Description

Current implementation requires at least one side of every VETH pair to remain in the host namespace. Both ends cannot be assigned to different containers (confd skips setup for container interfaces; if both skip, the pair is never created).

Added limitation notes to:

  • container.md: IMPORTANT callout in "Container Host Interface" section before VETH examples
  • infix-if-veth.yang: Module-level description
  • infix-if-container.yang: host identity description

The actual fix to support inter-container VETH pairs is tracked in #941.

Checklist

Tick relevant boxes, this PR is-a or has-a:

  • Bugfix
    • Regression tests
    • ChangeLog updates (for next release)
  • Feature
    • YANG model change => revision updated?
    • Regression tests added?
    • ChangeLog updates (for next release)
    • Documentation added?
  • Test changes
    • Checked in changed Readme.adoc (make test-spec)
    • Added new test to group Readme.adoc and yaml file
  • Code style update (formatting, renaming)
  • Refactoring (please detail in commit messages)
  • Build related changes
  • Documentation content changes
    • ChangeLog updated (for major changes)
  • Other (please describe):

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Copilot AI and others added 2 commits January 29, 2026 19:59
Add documentation noting that at least one side of a VETH pair must
remain in the host namespace. Both ends cannot be assigned to different
containers.

Updates:
- container.md: Added IMPORTANT note in Container Host Interface section
- infix-if-veth.yang: Added note in module description
- infix-if-container.yang: Added note in host identity description

Fixes: #947
Related: #941

Co-authored-by: troglobit <183517+troglobit@users.noreply.github.com>
Co-authored-by: troglobit <183517+troglobit@users.noreply.github.com>
Copilot AI changed the title [WIP] Assess and propose a solution for issue #947 Document VETH pair limitation with containers Jan 29, 2026
Copilot AI requested a review from troglobit January 29, 2026 20:02
@troglobit troglobit requested review from jovatn and removed request for troglobit January 29, 2026 20:05
@troglobit troglobit linked an issue Jan 29, 2026 that may be closed by this pull request
@troglobit troglobit marked this pull request as ready for review January 29, 2026 20:08
@troglobit troglobit removed their assignment Jan 29, 2026
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.

Document limitation using veth pairs between containers

2 participants