Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Automated backport of #3156: Explicitly disable BGP export for Calico IPPools #3162

Conversation

yboaron
Copy link
Contributor

@yboaron yboaron commented Sep 11, 2024

Backport of #3156 on release-0.17.

#3156: Explicitly disable BGP export for Calico IPPools

For details on the backport process, see the backport requests page.

We recently noticed that sometimes (probably after node reboots)
static routes to remote cluster CIDRs added by RouteAgent are overridden
by Calico (proto = bird) even though IPPool is set with Disabled=true.
Routes added by Calico point inter-cluster traffic to default interface
instead of vx-submariner which breaks datapath.

Adding DisableBGPExport=true to IPPool solves this issue.

Signed-off-by: Yossi Boaron <yboaron@redhat.com>
@submariner-bot
Copy link
Contributor

🤖 Created branch: z_pr3162/yboaron/automated-backport-of-#3156-origin-release-0.17
🚀 Full E2E won't run until the "ready-to-test" label is applied. I will add it automatically once the PR has 2 approvals, or you can add it manually.

@tpantelis tpantelis enabled auto-merge (rebase) September 11, 2024 12:47
@yboaron yboaron added the ready-to-test When a PR is ready for full E2E testing label Sep 15, 2024
@yboaron
Copy link
Contributor Author

yboaron commented Sep 17, 2024

cc @aswinsuryan @vthapar

@tpantelis tpantelis merged commit 63bfb2a into submariner-io:release-0.17 Sep 17, 2024
44 checks passed
@submariner-bot
Copy link
Contributor

🤖 Closed branches: [z_pr3162/yboaron/automated-backport-of-#3156-origin-release-0.17]

yboaron added a commit to yboaron/submariner-website that referenced this pull request Sep 22, 2024
submariner-io/submariner#3139
submariner-io/submariner#3162

Signed-off-by: Yossi Boaron <yboaron@redhat.com>
dfarrell07 pushed a commit to submariner-io/submariner-website that referenced this pull request Sep 23, 2024
submariner-io/submariner#3139
submariner-io/submariner#3162

Signed-off-by: Yossi Boaron <yboaron@redhat.com>
@tpantelis tpantelis added release-note-needed Should be mentioned in the release notes release-note-handled labels Sep 23, 2024
tpantelis pushed a commit to tpantelis/submariner-website that referenced this pull request Oct 9, 2024
submariner-io/submariner#3139
submariner-io/submariner#3162

Signed-off-by: Yossi Boaron <yboaron@redhat.com>
tpantelis pushed a commit to submariner-io/submariner-website that referenced this pull request Oct 9, 2024
submariner-io/submariner#3139
submariner-io/submariner#3162

Signed-off-by: Yossi Boaron <yboaron@redhat.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automated-backport ready-to-test When a PR is ready for full E2E testing release-note-handled release-note-needed Should be mentioned in the release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants