feedc0de/my-rook-config
1
0 Fork 0
forked from rook/rook
Code Pull Requests Activity

Merge pull request #15242 from sp98/fix-osd-migration

Browse Source 
osd: use bluestore if store type is empty
This commit is contained in:
Santosh Pillai authored and GitHub committed 2025-01-09 09:24:24 +05:30
2 parents 54c2ba5071 e44dd3ad74
commit b977bd0d39
1 file changed
+3 -2
Download Patch File Download Diff File Expand all files Collapse all files
pkg/operator/ceph/cluster/osd/migrate.go
+3 -2
View File
@@ -105,14 +105,15 @@ func (m *migrationConfig) migrateForEncryption(c *Cluster, osdDeployments *appsv
// migrateForOSDStore gets all the OSDs that require migration due to change in the cephCluster OSD storeType setting
func (m *migrationConfig) migrateForOSDStore(c *Cluster, osdDeployments *appsv1.DeploymentList) error {
desiredOSDStore := c.spec.Storage.GetOSDStore()
for i := range osdDeployments.Items {
if osdStore, ok := osdDeployments.Items[i].Labels[osdStore]; ok {
if osdStore != string(c.spec.Storage.Store.Type) {
if osdStore != desiredOSDStore {
osdInfo, err := c.getOSDInfo(&osdDeployments.Items[i])
if err != nil {
return errors.Wrapf(err, "failed to details about the OSD %q", osdDeployments.Items[i].Name)
}
logger.Infof("migration is required for OSD.%d to update storeType from %q to %q", osdInfo.ID, osdStore, c.spec.Storage.Store.Type)
logger.Infof("migration is required for OSD.%d to update storeType from %q to %q", osdInfo.ID, osdStore, desiredOSDStore)
if _, exists := m.osds[osdInfo.ID]; !exists {
m.osds[osdInfo.ID] = &osdInfo
}