Merge pull request #554 from GrahamDumpleton/lookup-service-selector-fix

Fix match expressions bug in lookup service.
educates · Aug 20, 2024 · ef3953d · ef3953d
2 parents e087051 + 45fd96a
commit ef3953d
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/lookup-service/service/caches/tenants.py b/lookup-service/service/caches/tenants.py
@@ -47,6 +47,7 @@ def allowed_access_to_portal(self, portal: TrainingPortal) -> bool:
         resource = {
             "metadata": {
                 "name": portal.name,
+                "uid": portal.uid,
                 "labels": {item["name"]: item["value"] for item in list(portal.labels)},
             },
         }

diff --git a/lookup-service/service/helpers/selectors.py b/lookup-service/service/helpers/selectors.py
@@ -115,7 +115,9 @@ def convert_to_label_selector(label_selector_dict: dict) -> LabelSelector:
 
     match_expressions = [
         LabelSelectorRequirement(
-            key=expr["key"], operator=expr["operator"], values=expr.get("values")
+            key=expr["key"],
+            operator=Operator(expr["operator"]),
+            values=expr.get("values"),
         )
         for expr in match_expressions_data
     ]