[CORE-12279] Live migration for OpenStack#2633
Conversation
✅ Deploy Preview for calico-docs-preview-next ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
✅ Deploy Preview succeeded!
To edit notification comments on pull requests, go to your Netlify project configuration. |
nelljerram
force-pushed
the
openstack-live-migration
branch
from
2f5026c to
6a847e9
Compare
There was a problem hiding this comment.
Pull request overview
Adds new Calico OpenStack documentation describing how to support VM live migration with minimal network disruption, and makes it discoverable in the Calico OpenStack sidebar section.
Changes:
- Adds a new OpenStack “Live migration” MDX page covering route priority propagation via BIRD (iBGP/eBGP), Nova settings, and troubleshooting logs.
- Links the new page into the Calico docs sidebar under “Calico networking for OpenStack”.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| sidebars-calico.js | Adds the new OpenStack live migration doc to the OpenStack sidebar category. |
| calico/networking/openstack/live-migration.mdx | Introduces the new live migration guide (BGP priority propagation, Nova integration, monitoring). |
nelljerram
force-pushed
the
openstack-live-migration
branch
8 times, most recently
from
551b54c to
a02ffc2
Compare
nelljerram
force-pushed
the
openstack-live-migration
branch
from
a02ffc2 to
1dc6c4b
Compare
song-jiang
left a comment
song-jiang
left a comment
There was a problem hiding this comment.
Looks good but with one question.
| ``` | ||
|
|
||
| For IPv6, make the same changes in your `bird6.conf`, replacing `0.0.0.0/0` | ||
| with `::/0` in the `export_bgp` filter. |
There was a problem hiding this comment.
Why we need to reference 0.0.0.0/0 in the export_bgp filter?
There was a problem hiding this comment.
D'oh, good catch! It used to be there, in Claude's first draft, but then I removed it because it wasn't part of what this doc is trying to explain.
In Claude's first draft the filter code included "0.0.0.0/0", but then I removed it because it wasn't part of what this doc is trying to explain.
ctauchen
left a comment
ctauchen
left a comment
There was a problem hiding this comment.
@nelljerram A few small things, otherwise LGTM
| description: Configure live migration support. | ||
| --- | ||
|
|
||
| # Live migration |
There was a problem hiding this comment.
Best probably to include OpenStack in the title. For example, "Live migration for OpenStack VMs".
| - With OpenStack $[prodname] itself does not generate the BGP configuration - | ||
| whereas with Kubernetes it does - so instead this is a customer | ||
| responsibility. |
There was a problem hiding this comment.
Something like this might be a bit easier to parse:
On OpenStack, you need to change your BGP configuration yourself before performing the migration. For migrations on Kubernetes, $[prodname] generates the new configuration automatically.
There was a problem hiding this comment.
I've tweaked this but with a slightly different wording - LMK what you think. Key thing is to avoid the impression that BGP configuration update is needed for every live migration. It's not; rather, BGP configuration is needed, once only, to enable the live migration capability for the whole cluster.
Co-authored-by: Christopher Tauchen <tauchen@gmail.com>
Co-authored-by: Christopher Tauchen <tauchen@gmail.com>
Co-authored-by: Christopher Tauchen <tauchen@gmail.com>
|
@ctauchen I've addressed your comments - PTAL and let me know what you think. |
|
LGTM, thanks @nelljerram |
No description provided.