Merge pull request #23 from coveooss/def-1334

def-1334: support no group
coveooss · Jun 6, 2023 · c757042 · c757042
2 parents 6bb7274 + 5af966e
commit c757042
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/src/kubernetes_operator/iam_mapping.py b/src/kubernetes_operator/iam_mapping.py
@@ -58,7 +58,7 @@ async def create_mapping(spec: dict, diff: list, **_) -> None:
     configmap = API.read_namespaced_config_map("aws-auth", "kube-system")
 
     arn_field = spec["userarn"] if spec.get("userarn") else spec["rolearn"]
-    logger.info("Mapping for user %s as %s to %s", arn_field, spec["username"], spec["groups"])
+    logger.info("Mapping for user %s as %s to %s", arn_field, spec["username"], spec.get("groups", "(no group)"))
 
     identities = get_cm_identity_mappings(configmap)
     updated_mapping = ensure_identity(sanitize_spec, identities)
@@ -74,7 +74,7 @@ async def delete_mapping(spec: dict, **_) -> None:
     configmap = API.read_namespaced_config_map("aws-auth", "kube-system")
 
     arn_field = spec["userarn"] if spec.get("userarn") else spec["rolearn"]
-    logger.info("Delete mapping for user %s as %s to %s", arn_field, spec["username"], spec["groups"])
+    logger.info("Delete mapping for user %s as %s to %s", arn_field, spec["username"], spec.get("groups", "(no group)"))
 
     identity_mappings = get_cm_identity_mappings(configmap)
     updated_mappings = delete_identity(spec, identity_mappings)