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

PL-131857 rbd pool migration #18

Merged
merged 12 commits into from
Oct 23, 2024
Merged

PL-131857 rbd pool migration #18

merged 12 commits into from
Oct 23, 2024

Conversation

ctheune
Copy link
Member

@ctheune ctheune commented Oct 17, 2024

We now support automatically migrating VM root disks between different RBD tools using Ceph's "RBD live migration" feature.

Also includes a number of refactorings and paying down some technical debt.

PL-131857

@ctheune ctheune changed the title Pl 131857 rbd pool migration PL-31857 rbd pool migration Oct 17, 2024
@ctheune ctheune requested a review from sysvinit October 17, 2024 21:00
@ctheune ctheune changed the title PL-31857 rbd pool migration PL-131857 rbd pool migration Oct 17, 2024
This uses Ceph's RBD live migration and thus is quite fast and
we continuously maintain the flattening during subsequent ensure()
calls. Existing snapshots are kept.

This includes a large refactoring to make the Ceph abstraction code
easier to understand with the different requirements we now juggle.

We basically have a relatively abstract "Ceph" object that is the
entrance point to the cluster and some convenience APIs. Volumes are
specific RBD volumes and their API and "VolumeSpecs" capture the
logic we have around Root/Tmp/Swap volumes and their details.

They get embedded into a streamlined process of
starting/ensuring/stopping so the agent and ceph manager objects
can handle them pretty "blindly". They are reachable as well known
names, though, to make it easier to handle the few special cases
in the higher layers.

re PL-131857
(test separation part 2, record intermittent changes)
@ctheune ctheune force-pushed the PL-131857-rbd-pool-migration branch from 482bd55 to 1fcf21c Compare October 23, 2024 20:50
@ctheune ctheune force-pushed the PL-131857-rbd-pool-migration branch from 1fcf21c to 960184c Compare October 23, 2024 20:52
@ctheune
Copy link
Member Author

ctheune commented Oct 23, 2024

@sysvinit and I reviewed this together last week and I only did a few fixups afterwards.

@ctheune ctheune merged commit 4aa4369 into master Oct 23, 2024
1 check passed
@ctheune ctheune deleted the PL-131857-rbd-pool-migration branch October 23, 2024 20:54
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.

1 participant