-
Notifications
You must be signed in to change notification settings - Fork 42
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
OSD replication factor applies to all pools #385
Comments
Can we use the LXD client to query for both the |
Even that isn't quite enough because we also need to make sure the replication size isn't already customized to a higher value. Otherwise we would reset them all back to 3 each time a node is added. |
mseralessandri
added
the
Jira
Allows the synchronization of a GitHub issue in Jira
label
Sep 4, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
We adjust the replication factor for OSDs up to 3 when adding systems to a MicroCloud.
We do this by specifying
Pools: []string{"*"}
which applies the replication factor to all pools. However, if these pools' sizes were manually modified, MicroCloud will not have this context and will overwrite the user configuration on each run ofmicrocloud add
,microcloud service add
, and evenmicrocloud init
with an existing MicroCeph.We will need to add a
GET /1.0/pools-op
to MicroCeph to fetch current pool sizes from ceph.With this endpoint, we can inspect the current pool configuration and make inferences about whether we should change it:
on
microcloud init
, if the total OSD count is fewer than 3, we will set all "managed" OSD pool sizes tomax(3, count(OSDs))
.on
microcloud add
, andmicrocloud service add
, present a list of "unmanaged" OSD pools whose size is smaller (or larger) than the new OSD count. The user will have to select the pools they want to increase in replication size. "Managed" pools will automatically be updated.A "managed" pool is one that MicroCloud sets up. So that means
lxd_remote
,lxd_cephfs
,lxd_cephfs_meta
,lxd_cephfs_data
, and.mgr
.The text was updated successfully, but these errors were encountered: