Skip to content

Conversation

rastislavs
Copy link
Contributor

@rastislavs rastislavs commented May 2, 2025

Adds a test that peers IPv6-only peering and IPv6-only advertisement.
Enhances existing auth test to validate advertised routes.
Fixes comments in svc-path-attributes test.

@maintainer-s-little-helper maintainer-s-little-helper bot added the dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. label May 2, 2025
@rastislavs rastislavs added release-note/ci This PR makes changes to the CI. area/bgp Impacts the Border Gateway Protocol feature. labels May 2, 2025
@maintainer-s-little-helper maintainer-s-little-helper bot removed the dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. label May 2, 2025
rastislavs added 2 commits May 2, 2025 10:39
Add a test that tests IPv6-only peering and IPv6-only advertisement.
Enhance existing auth test to validate advertised routes.

Signed-off-by: Rastislav Szabo <rastislav.szabo@isovalent.com>
Comments were mentioning incorrect types of advertisemnt
(caused probably by a copy-paste from another test).

Signed-off-by: Rastislav Szabo <rastislav.szabo@isovalent.com>
@rastislavs
Copy link
Contributor Author

/test

@rastislavs rastislavs marked this pull request as ready for review May 2, 2025 13:43
@rastislavs rastislavs requested a review from a team as a code owner May 2, 2025 13:43
@rastislavs rastislavs requested a review from harsimran-pabla May 2, 2025 13:43
@rastislavs
Copy link
Contributor Author

ping @harsimran-pabla , PTAL

Copy link
Contributor

@harsimran-pabla harsimran-pabla left a comment

Choose a reason for hiding this comment

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

Sorry, missed this PR. LGTM

@maintainer-s-little-helper maintainer-s-little-helper bot added the ready-to-merge This PR has passed all tests and received consensus from code owners to merge. label May 20, 2025
@julianwiedmann julianwiedmann added this pull request to the merge queue May 20, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks May 20, 2025
@julianwiedmann julianwiedmann added this pull request to the merge queue May 20, 2025
Merged via the queue into cilium:main with commit c483285 May 20, 2025
70 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/bgp Impacts the Border Gateway Protocol feature. ready-to-merge This PR has passed all tests and received consensus from code owners to merge. release-note/ci This PR makes changes to the CI.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants