eliminate dependency on workload.codeflare.dev/appwrapper label

Author: dgrove-oss

src/codeflare_sdk/templates/base-template.yaml

@@ -16,7 +16,6 @@ spec:
         kind: RayCluster
         metadata:
           labels:
-            workload.codeflare.dev/appwrapper: "aw-kuberay"
             controller-tools.k8s.io: "1.0"
             # A unique identifier for the head node and workers of this cluster.
           name: kuberay-cluster

src/codeflare_sdk/utils/generate_yaml.py

@@ -82,7 +82,6 @@ def update_names(yaml, item, appwrapper_name, cluster_name, namespace):
     metadata["name"] = appwrapper_name
     metadata["namespace"] = namespace
     lower_meta = item.get("generictemplate", {}).get("metadata")
-    lower_meta["labels"]["workload.codeflare.dev/appwrapper"] = appwrapper_name
     lower_meta["name"] = cluster_name
     lower_meta["namespace"] = namespace
 
@@ -228,16 +227,9 @@ def write_components(
     with open(output_file_name, "a") as outfile:
         for component in components:
             if "generictemplate" in component:
-                if (
-                    "workload.codeflare.dev/appwrapper"
-                    in component["generictemplate"]["metadata"]["labels"]
-                ):
-                    del component["generictemplate"]["metadata"]["labels"][
-                        "workload.codeflare.dev/appwrapper"
-                    ]
-                    labels = component["generictemplate"]["metadata"]["labels"]
-                    labels.update({"kueue.x-k8s.io/queue-name": lq_name})
-                    labels.update(cluster_labels)
+                labels = component["generictemplate"]["metadata"]["labels"]
+                labels.update({"kueue.x-k8s.io/queue-name": lq_name})
+                labels.update(cluster_labels)
                 outfile.write("---\n")
                 yaml.dump(
                     component["generictemplate"], outfile, default_flow_style=False
@@ -258,16 +250,9 @@ def load_components(
     cluster_labels = labels
     for component in components:
         if "generictemplate" in component:
-            if (
-                "workload.codeflare.dev/appwrapper"
-                in component["generictemplate"]["metadata"]["labels"]
-            ):
-                del component["generictemplate"]["metadata"]["labels"][
-                    "workload.codeflare.dev/appwrapper"
-                ]
-                labels = component["generictemplate"]["metadata"]["labels"]
-                labels.update({"kueue.x-k8s.io/queue-name": lq_name})
-                labels.update(cluster_labels)
+            labels = component["generictemplate"]["metadata"]["labels"]
+            labels.update({"kueue.x-k8s.io/queue-name": lq_name})
+            labels.update(cluster_labels)
             component_list.append(component["generictemplate"])
 
     resources = "---\n" + "---\n".join(

tests/test-case-bad.yaml

@@ -13,7 +13,6 @@ spec:
         kind: RayCluster
         metadata:
           labels:
-            workload.codeflare.dev/appwrapper: unit-test-cluster
             controller-tools.k8s.io: '1.0'
           name: unit-test-cluster
           namespace: ns

tests/test-case.yaml

@@ -12,7 +12,6 @@ spec:
         metadata:
           labels:
             controller-tools.k8s.io: '1.0'
-            workload.codeflare.dev/appwrapper: unit-test-cluster
           name: unit-test-cluster
           namespace: ns
         spec:

tests/test-default-appwrapper.yaml

@@ -12,7 +12,6 @@ spec:
         metadata:
           labels:
             controller-tools.k8s.io: '1.0'
-            workload.codeflare.dev/appwrapper: unit-test-default-cluster
           name: unit-test-default-cluster
           namespace: opendatahub
         spec:

tests/unit_test.py

@@ -968,7 +968,6 @@ def get_ray_obj(group, version, namespace, plural, cls=None):
                     "labels": {
                         "controller-tools.k8s.io": "1.0",
                         "resourceName": "quicktest",
-                        "workload.codeflare.dev/appwrapper": "quicktest",
                         "orderedinstance": "m4.xlarge_g4dn.xlarge",
                     },
                     "name": "quicktest",
@@ -1275,7 +1274,6 @@ def get_ray_obj(group, version, namespace, plural, cls=None):
                     "creationTimestamp": "2023-02-22T16:26:07Z",
                     "generation": 1,
                     "labels": {
-                        "workload.codeflare.dev/appwrapper": "quicktest2",
                         "controller-tools.k8s.io": "1.0",
                         "resourceName": "quicktest2",
                         "orderedinstance": "m4.xlarge_g4dn.xlarge",
@@ -1466,7 +1464,6 @@ def get_aw_obj(group, version, namespace, plural):
                                     "kind": "RayCluster",
                                     "metadata": {
                                         "labels": {
-                                            "workload.codeflare.dev/appwrapper": "quicktest1",
                                             "controller-tools.k8s.io": "1.0",
                                         },
                                         "name": "quicktest1",
@@ -1670,7 +1667,6 @@ def get_aw_obj(group, version, namespace, plural):
                                     "kind": "RayCluster",
                                     "metadata": {
                                         "labels": {
-                                            "workload.codeflare.dev/appwrapper": "quicktest2",
                                             "controller-tools.k8s.io": "1.0",
                                         },
                                         "name": "quicktest2",