[Validators] Remove features from SlurmSettings deny list. That property name was wrong: the correct one is feature, not features. We do not fix the typo because there is no reason to block the customization of feature.

gmarciani · gmarciani · commit 0c3cbf612ed0 · 2025-12-04T12:49:56.000-05:00
diff --git a/cli/src/pcluster/validators/slurm_settings_validator.py b/cli/src/pcluster/validators/slurm_settings_validator.py
@@ -50,7 +50,7 @@
         "Global": ["nodes", "partitionname", "resumetimeout", "state", "suspendtime", "oversubscribe"],
     },
     "ComputeResource": {
-        "Global": ["cpus", "features", "gres", "nodeaddr", "nodehostname", "nodename", "state", "weight"],
+        "Global": ["cpus", "gres", "nodeaddr", "nodehostname", "nodename", "state", "weight"],
     },
 }
 

Original file line number	Diff line number	Diff line change
`@@ -50,7 +50,7 @@`
`50`	`50`	`"Global": ["nodes", "partitionname", "resumetimeout", "state", "suspendtime", "oversubscribe"],`
`51`	`51`	`},`
`52`	`52`	`"ComputeResource": {`
`53`		`- "Global": ["cpus", "features", "gres", "nodeaddr", "nodehostname", "nodename", "state", "weight"],`
	`53`	`+ "Global": ["cpus", "gres", "nodeaddr", "nodehostname", "nodename", "state", "weight"],`
`54`	`54`	`},`
`55`	`55`	`}`
`56`	`56`