cephcluster: tune disks according to plaftorm

set tunefastdeviceclass according to plaftorm
which are not correctly detected by ceph.
modify tc according to the changes.

Signed-off-by: crombus <pkundra@redhat.com>

controllers/storagecluster/cephcluster.go

@@ -641,6 +641,14 @@ func (r *StorageClusterReconciler) checkTuneStorageDevices(ds ocsv1.StorageDevic
 		return dt.speed, nil
 	}
 
+	tuneFastDevices, err := r.DevicesDefaultToFastForThisPlatform()
+	if err != nil {
+		return diskSpeedUnknown, err
+	}
+	if tuneFastDevices {
+		return diskSpeedFast, nil
+	}
+
 	// not a known disk type, don't tune
 	return diskSpeedUnknown, nil
 }

controllers/storagecluster/platform_detection.go

@@ -18,6 +18,13 @@ var AvoidObjectStorePlatforms = []configv1.PlatformType{
 	configv1.AzurePlatformType,
 }
 
+// TuneFastPlatforms is a list of all PlatformTypes where TuneFastDeviceClass has to be set True.
+var TuneFastPlatforms = []configv1.PlatformType{
+	configv1.OvirtPlatformType,
+	configv1.IBMCloudPlatformType,
+	configv1.AzurePlatformType,
+}
+
 // Platform is used to get the CloudPlatformType of the running cluster in a thread-safe manner
 type Platform struct {
 	platform configv1.PlatformType
@@ -55,3 +62,19 @@ func avoidObjectStore(p configv1.PlatformType) bool {
 	}
 	return false
 }
+
+func (r *ReconcileStorageCluster) DevicesDefaultToFastForThisPlatform() (bool, error) {
+	c := r.client
+	platform, err := r.platform.GetPlatform(c)
+	if err != nil {
+		return false, err
+	}
+
+	for _, tfplatform := range TuneFastPlatforms {
+		if platform == tfplatform {
+			return true, nil
+		}
+	}
+
+	return false, nil
+}

controllers/storagecluster/storagecluster_controller_test.go

@@ -311,11 +311,12 @@ func TestThrottleStorageDevices(t *testing.T) {
 							StorageClassName: &fakestorageClassName,
 						},
 					},
-					Portable: true,
+					Portable:   true,
+					DeviceType: "hdd",
 				},
 			},
 			storageCluster: &api.StorageCluster{},
-			expectedSpeed:  diskSpeedUnknown,
+			expectedSpeed:  diskSpeedSlow,
 		},
 		{
 			label: "Case 3", // storageclass is managed-premium