Reverse logic in valid_placement_nodes

Author: jvilarru

CHANGELOG.md

@@ -9,6 +9,7 @@ All notable changes to this project will be documented in this file.
 - Migrate from network overrides in bulkInsert to honoring instance templates.
 - Add additional_networks support to instance template and partition_nodes.
 - Support Tier 1 networking in instance templates.
+- Reverse logic in valid_placement_nodes
 
 ## \[5.8.0\]
 

scripts/resume.py

@@ -445,14 +445,14 @@ def valid_placement_nodes(job_id, nodelist):
         for node in nodelist
     }
     fail = False
-    valid_types = ["a2", "a3", "c2", "c2d", "c3", "n2", "n2d"]
+    invalid_types = ["e2", "t2d", "n1", "t2a", "m1", "m2", "m3"]
     for prefix, machine_type in machine_types.items():
-        if machine_type.split("-")[0] not in valid_types:
+        if machine_type.split("-")[0] in invalid_types:
             log.warn(f"Unsupported machine type for placement policy: {machine_type}.")
             fail = True
     if fail:
         log.warn(
-            f"Please use a valid machine type with placement policy: ({','.join(valid_types)})"
+            f"Please do not use any the following machine types with placement policy: ({','.join(invalid_types)})"
         )
         return False
     return True