From acfca3ad74f5affae39ba6ca851b5e2ee55d4bc4 Mon Sep 17 00:00:00 2001 From: Steven Hawkins Date: Wed, 19 Feb 2025 05:25:37 -0500 Subject: [PATCH] fix: making the deadline override applicable to all remote tests (#37408) closes: #37392 Signed-off-by: Steve Hawkins --- .github/workflows/operator-ci.yml | 1 - .../operator/testsuite/integration/BaseOperatorTest.java | 6 ++++-- 2 files changed, 4 insertions(+), 3 deletions(-) diff --git a/.github/workflows/operator-ci.yml b/.github/workflows/operator-ci.yml index 5777bad3f44..c25a72c6440 100644 --- a/.github/workflows/operator-ci.yml +++ b/.github/workflows/operator-ci.yml @@ -141,7 +141,6 @@ jobs: -Dkc.operator.keycloak.image=keycloak:${{ env.version_remote }} \ -Dquarkus.kubernetes.env.vars.kc-operator-keycloak-image-pull-policy=Never \ -Dtest.operator.custom.image=custom-keycloak:${{ env.version_remote }} \ - -Dquarkus.kubernetes.env.vars.kc-operator-keycloak-update-pod-deadline-seconds=60 \ --no-transfer-progress -Dtest.operator.deployment=remote test-olm: diff --git a/operator/src/test/java/org/keycloak/operator/testsuite/integration/BaseOperatorTest.java b/operator/src/test/java/org/keycloak/operator/testsuite/integration/BaseOperatorTest.java index 1b3b889c4ea..72815cd8bc0 100644 --- a/operator/src/test/java/org/keycloak/operator/testsuite/integration/BaseOperatorTest.java +++ b/operator/src/test/java/org/keycloak/operator/testsuite/integration/BaseOperatorTest.java @@ -17,15 +17,14 @@ package org.keycloak.operator.testsuite.integration; +import io.fabric8.kubernetes.api.model.EnvVar; import io.fabric8.kubernetes.api.model.HasMetadata; import io.fabric8.kubernetes.api.model.MicroTime; import io.fabric8.kubernetes.api.model.NamespaceBuilder; import io.fabric8.kubernetes.api.model.Pod; import io.fabric8.kubernetes.api.model.Secret; -import io.fabric8.kubernetes.api.model.Service; import io.fabric8.kubernetes.api.model.apps.Deployment; import io.fabric8.kubernetes.api.model.apps.StatefulSet; -import io.fabric8.kubernetes.api.model.batch.v1.Job; import io.fabric8.kubernetes.api.model.events.v1.Event; import io.fabric8.kubernetes.api.model.rbac.ClusterRoleBinding; import io.fabric8.kubernetes.api.model.rbac.RoleBinding; @@ -178,6 +177,9 @@ public class BaseOperatorTest implements QuarkusTestAfterEachCallback { ((ClusterRoleBinding)obj).getSubjects().forEach(s -> s.setNamespace(namespace)); } else if (obj instanceof RoleBinding && "keycloak-operator-view".equals(((RoleBinding)obj).getMetadata().getName())) { return null; // exclude this role since it's not present in olm + } else if (obj instanceof Deployment) { + // set values useful for testing - TODO: could drive this in some way from the test/resource/application.properties + ((Deployment)obj).getSpec().getTemplate().getSpec().getContainers().get(0).getEnv().add(new EnvVar("KC_OPERATOR_KEYCLOAK_UPDATE_POD_DEADLINE_SECONDS", "60", null)); } return obj; });