Skip to content

More routing cleanups - Vtep - #1869

Open
Fredi-raspall wants to merge 10 commits into
pr/fredi/ifindex_translationfrom
pr/fredi/routing_cleanups_2
Open

Fredi-raspall wants to merge 10 commits into
pr/fredi/ifindex_translationfrom
pr/fredi/routing_cleanups_2

Conversation

@Fredi-raspall

@Fredi-raspall Fredi-raspall commented Oct 1, 2026 •

Copy link
Copy Markdown
Contributor

@Fredi-raspall
Fredi-raspall requested a review from a team as a code owner October 1, 2026 15:03
@Fredi-raspall
Fredi-raspall requested review from sergeymatov and removed request for a team October 1, 2026 15:03
@coderabbitai

coderabbitai Bot commented Oct 1, 2026 •

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Essentials

Run ID: 03902e97-ca52-4742-bc54-3dd975ee7462

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
  • Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Autopilot is currently an internal CodeRabbit preview.


Comment @coderabbitai help to get the list of available commands.

@Fredi-raspall Fredi-raspall changed the title More routing cleanups More routing cleanups - Vtep Oct 1, 2026
@Fredi-raspall
Fredi-raspall requested a review from mvachhar October 1, 2026 15:08
@codecov

codecov Bot commented Oct 1, 2026 •

Copy link
Copy Markdown

@Fredi-raspall
Fredi-raspall force-pushed the pr/fredi/routing_cleanups_2 branch from 4c659f5 to 20d58e2 Compare October 1, 2026 17:41
@Fredi-raspall Fredi-raspall added the ci:+vlab Enable VLAB tests label Oct 1, 2026
@Fredi-raspall
Fredi-raspall force-pushed the pr/fredi/ifindex_translation branch from 17b7fb0 to eab45cf Compare October 1, 2026 20:08
@Fredi-raspall
Fredi-raspall force-pushed the pr/fredi/routing_cleanups_2 branch 2 times, most recently from 063fd98 to fa35cfd Compare October 2, 2026 11:22
@Fredi-raspall
Fredi-raspall force-pushed the pr/fredi/ifindex_translation branch from eab45cf to eaceb87 Compare October 2, 2026 11:27
Signed-off-by: Fredi Raspall <fredi@githedgehog.com>
Signed-off-by: Fredi Raspall <fredi@githedgehog.com>
@Fredi-raspall
Fredi-raspall force-pushed the pr/fredi/routing_cleanups_2 branch from fa35cfd to 6ef30a9 Compare October 2, 2026 11:36
Replace the current vtep definition with one that always has an
IP and a MAC.

Signed-off-by: Fredi Raspall <fredi@githedgehog.com>
Signed-off-by: Fredi Raspall <fredi@githedgehog.com>
Signed-off-by: Fredi Raspall <fredi@githedgehog.com>
Signed-off-by: Fredi Raspall <fredi@githedgehog.com>
Previously, we were showing the vtep information intended by the
config received by the router. Augment that so that it also displays
the vtep information seen by dataplane workers.

Signed-off-by: Fredi Raspall <fredi@githedgehog.com>
When applying a new config and reconfiguring vteps, verify that
the information has been correctly applied.

Signed-off-by: Fredi Raspall <fredi@githedgehog.com>
The logs are pretty chatty and the important ones are recored.

Signed-off-by: Fredi Raspall <fredi@githedgehog.com>
Signed-off-by: Fredi Raspall <fredi@githedgehog.com>
@Fredi-raspall
Fredi-raspall force-pushed the pr/fredi/routing_cleanups_2 branch from 6ef30a9 to b1cbdbf Compare October 2, 2026 14:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci:+vlab Enable VLAB tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant