Enable DCV for us-iso regions

Signed-off-by: Hanwen <[email protected]>

Author: hanwen-cluster

cli/src/pcluster/config/cluster_config.py

@@ -1635,7 +1635,6 @@ def _register_validators(self, context: ValidatorContext = None):  # noqa: D102
             imds_support=self.imds.imds_support,
         )
         if self.head_node.dcv:
-            self._register_validator(FeatureRegionValidator, feature=Feature.DCV, region=self.region)
             self._register_validator(
                 DcvValidator,
                 instance_type=self.head_node.instance_type,
@@ -2969,7 +2968,6 @@ def login_nodes_subnet_ids(self):
 
     def _register_login_node_validators(self):
         """Register all login node validators to ensure that the resource parameters are valid."""
-        has_dcv_configured = False
         # Check if all subnets(head node, Login nodes, compute nodes) are in the same VPC and support DNS.
         self._register_validator(
             SubnetsValidator,
@@ -3002,10 +3000,6 @@ def _register_login_node_validators(self):
                     os=self.image.os,
                     architecture=pool.architecture,
                 )
-                has_dcv_configured = True
-
-        if has_dcv_configured:
-            self._register_validator(FeatureRegionValidator, feature=Feature.DCV, region=self.region)
 
     def _register_validators(self, context: ValidatorContext = None):  # noqa: C901
         super()._register_validators(context)

cli/src/pcluster/constants.py

@@ -244,7 +244,6 @@ class Feature(Enum):
 
 UNSUPPORTED_FEATURES_MAP = {
     Feature.BATCH: ["ap-northeast-3", "us-iso"],
-    Feature.DCV: ["us-iso"],
     Feature.FSX_LUSTRE: ["us-isob"],
     Feature.FILE_CACHE: ["us-iso"],
     Feature.FSX_ONTAP: ["us-iso"],

cli/tests/pcluster/test_utils.py

@@ -579,10 +579,6 @@ async def async_method(self, param):
         (Feature.BATCH, "us-iso-west-1", False),
         (Feature.BATCH, "us-isob-east-1", False),
         (Feature.BATCH, "us-isoWHATEVER", False),
-        (Feature.DCV, "us-iso-east-1", False),
-        (Feature.DCV, "us-iso-west-1", False),
-        (Feature.DCV, "us-isob-east-1", False),
-        (Feature.DCV, "us-isoWHATEVER", False),
         (Feature.FSX_LUSTRE, "us-isob-east-1", False),
         (Feature.FSX_LUSTRE, "us-isobWHATEVER", False),
         (Feature.FSX_ONTAP, "us-iso-east-1", False),
@@ -596,7 +592,6 @@ async def async_method(self, param):
         (Feature.SLURM_DATABASE, "us-isoWHATEVER", True),
         (Feature.CLUSTER_HEALTH_METRICS, "us-isoWHATEVER", False),
         (Feature.BATCH, "WHATEVER-ELSE", True),
-        (Feature.DCV, "WHATEVER-ELSE", True),
         (Feature.FSX_LUSTRE, "WHATEVER-ELSE", True),
         (Feature.FSX_ONTAP, "WHATEVER-ELSE", True),
         (Feature.FSX_OPENZFS, "WHATEVER-ELSE", True),

cli/tests/pcluster/validators/test_feature_validators.py

@@ -23,8 +23,6 @@
     [
         (Feature.BATCH, True, None),
         (Feature.BATCH, False, "AWS Batch scheduler is not supported in region 'WHATEVER-REGION'"),
-        (Feature.DCV, True, None),
-        (Feature.DCV, False, "Amazon DCV is not supported in region 'WHATEVER-REGION'"),
         (Feature.FSX_LUSTRE, True, None),
         (Feature.FSX_LUSTRE, False, "FSx Lustre is not supported in region 'WHATEVER-REGION'"),
         (Feature.FSX_ONTAP, True, None),