Remove fc from expandable plugins

jsafrane · jsafrane · commit 0ecbdf362208 · 2024-10-31T09:08:21.000+01:00
FibreChannel volume plugin does not implement ExpandableVolumePlugin.
diff --git a/cmd/kube-controller-manager/app/plugins.go b/cmd/kube-controller-manager/app/plugins.go
@@ -72,7 +72,6 @@ func ProbeExpandableVolumePlugins(logger klog.Logger, config persistentvolumecon
 	if err != nil {
 		return allPlugins, err
 	}
-	allPlugins = append(allPlugins, fc.ProbeVolumePlugins()...)
 	return allPlugins, nil
 }
 
diff --git a/cmd/kube-controller-manager/app/plugins_test.go b/cmd/kube-controller-manager/app/plugins_test.go
@@ -53,7 +53,7 @@ func TestProbeExpandableVolumePlugins(t *testing.T) {
 	if err != nil {
 		t.Fatalf("TestProbeExpandableVolumePlugins failed: %s", err)
 	}
-	checkPlugins(t, plugins, []string{"kubernetes.io/fc", "kubernetes.io/portworx-volume"})
+	checkPlugins(t, plugins, []string{"kubernetes.io/portworx-volume"})
 }
 
 func TestProbeControllerVolumePlugins(t *testing.T) {

Original file line number	Diff line number	Diff line change
`@@ -72,7 +72,6 @@ func ProbeExpandableVolumePlugins(logger klog.Logger, config persistentvolumecon`
`72`	`72`	`if err != nil {`
`73`	`73`	`return allPlugins, err`
`74`	`74`	`}`
`75`		`- allPlugins = append(allPlugins, fc.ProbeVolumePlugins()...)`
`76`	`75`	`return allPlugins, nil`
`77`	`76`	`}`
`78`	`77`
Original file line number	Diff line number	Diff line change
`@@ -53,7 +53,7 @@ func TestProbeExpandableVolumePlugins(t *testing.T) {`
`53`	`53`	`if err != nil {`
`54`	`54`	`t.Fatalf("TestProbeExpandableVolumePlugins failed: %s", err)`
`55`	`55`	`}`
`56`		`- checkPlugins(t, plugins, []string{"kubernetes.io/fc", "kubernetes.io/portworx-volume"})`
	`56`	`+ checkPlugins(t, plugins, []string{"kubernetes.io/portworx-volume"})`
`57`	`57`	`}`
`58`	`58`
`59`	`59`	`func TestProbeControllerVolumePlugins(t *testing.T) {`