Skip to content

Commit 22ee321

Browse files
Remove FSS max-pvscsi-targets-per-vm (#3400)
1 parent f03e1fe commit 22ee321

File tree

4 files changed

+1
-11
lines changed

4 files changed

+1
-11
lines changed

pkg/common/unittestcommon/utils.go

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -72,7 +72,6 @@ func GetFakeContainerOrchestratorInterface(orchestratorType int) (commonco.COCom
7272
"pv-to-backingdiskobjectid-mapping": "false",
7373
"cnsmgr-suspend-create-volume": "true",
7474
"topology-preferential-datastores": "true",
75-
"max-pvscsi-targets-per-vm": "true",
7675
"multi-vcenter-csi-topology": "true",
7776
"listview-tasks": "true",
7877
"storage-quota-m2": "false",

pkg/csi/service/common/commonco/k8sorchestrator/k8sorchestrator.go

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -364,7 +364,6 @@ func getReleasedVanillaFSS() map[string]struct{} {
364364
common.ListVolumes: {},
365365
common.CnsMgrSuspendCreateVolume: {},
366366
common.TopologyPreferentialDatastores: {},
367-
common.MaxPVSCSITargetsPerVM: {},
368367
common.MultiVCenterCSITopology: {},
369368
common.CSIInternalGeneratedClusterID: {},
370369
common.TopologyAwareFileVolume: {},

pkg/csi/service/common/constants.go

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -418,8 +418,6 @@ const (
418418
// TopologyPreferentialDatastores is the feature gate for preferential
419419
// datastore deployment in topology aware environments.
420420
TopologyPreferentialDatastores = "topology-preferential-datastores"
421-
// MaxPVSCSITargetsPerVM enables support for 255 volumes per node vm
422-
MaxPVSCSITargetsPerVM = "max-pvscsi-targets-per-vm"
423421
// MultiVCenterCSITopology is the feature gate for enabling multi vCenter topology support for vSphere CSI driver.
424422
MultiVCenterCSITopology = "multi-vcenter-csi-topology"
425423
// CSIInternalGeneratedClusterID enables support to generate unique cluster

pkg/csi/service/node.go

Lines changed: 1 addition & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,6 @@ import (
3535
)
3636

3737
const (
38-
maxAllowedBlockVolumesPerNode = 59
3938
// vCenter 8.0 supports attaching max 255 volumes to Node
4039
// Previous vSphere releases supports attaching a max of 59 volumes to Node VM.
4140
// Deployment YAML file for Node DaemonSet has ENV MAX_VOLUMES_PER_NODE set to 59 for vsphere-csi-node container
@@ -410,12 +409,7 @@ func (driver *vsphereCSIDriver) NodeGetInfo(
410409
}
411410

412411
var maxVolumesPerNode int64
413-
var maxAllowedVolumesPerNode int64
414-
if commonco.ContainerOrchestratorUtility.IsFSSEnabled(ctx, common.MaxPVSCSITargetsPerVM) {
415-
maxAllowedVolumesPerNode = maxAllowedBlockVolumesPerNodeInvSphere8
416-
} else {
417-
maxAllowedVolumesPerNode = maxAllowedBlockVolumesPerNode
418-
}
412+
var maxAllowedVolumesPerNode int64 = maxAllowedBlockVolumesPerNodeInvSphere8
419413
if v := os.Getenv("MAX_VOLUMES_PER_NODE"); v != "" {
420414
if value, err := strconv.ParseInt(v, 10, 64); err == nil {
421415
if value < 0 {

0 commit comments

Comments
 (0)