Skip to content

Commit

Permalink
Merge pull request #9259 from parth-gr/deviceClass
Browse files Browse the repository at this point in the history
osd: update existing OSDs with deviceClass
  • Loading branch information
leseb committed Nov 29, 2021
2 parents bf662ec + cbe505d commit 83f7c2b
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion pkg/operator/ceph/cluster/osd/update.go
Expand Up @@ -126,7 +126,9 @@ func (c *updateConfig) updateExistingOSDs(errs *provisionErrors) {
}

// backward compatibility for old deployments
if osdInfo.DeviceClass == "" {
// Checking DeviceClass with None too, because ceph-volume lvm list return crush device class as None
// Tracker https://tracker.ceph.com/issues/53425
if osdInfo.DeviceClass == "" || osdInfo.DeviceClass == "None" {
deviceClassInfo, err := cephclient.OSDDeviceClasses(c.cluster.context, c.cluster.clusterInfo, []string{strconv.Itoa(osdID)})
if err != nil {
logger.Errorf("failed to get device class for existing deployment %q. %v", depName, err)
Expand Down

0 comments on commit 83f7c2b

Please sign in to comment.