Skip to content

Commit 629193f

Browse files
committed
e2e: check all SGConfig related jobs are cleaned up
1 parent 58ff2b0 commit 629193f

3 files changed

Lines changed: 25 additions & 18 deletions

File tree

stackgres-k8s/e2e/spec/dbops-security-upgrade-olm

Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -339,6 +339,13 @@ EOF
339339
}
340340

341341
e2e_upgrade_operator() {
342+
SGCONFIG_NAMESPACE="$OPERATOR_NAMESPACE"
343+
if [ "x$E2E_ALLOWED_NAMESPACES" != x ] \
344+
&& ! printf ' %s ' "$E2E_ALLOWED_NAMESPACES" | grep -qF " $OPERATOR_NAMESPACE "
345+
then
346+
SGCONFIG_NAMESPACE="$CLUSTER_NAMESPACE"
347+
fi
348+
342349
echo "Upgrading operator bundle ($STACKGRES_VERSION)"
343350

344351
if [ "$STACKGRES_PREVIOUS_VERSION" = 1.6.0 ]
@@ -480,13 +487,6 @@ INNER_EOF
480487
)
481488
EOF
482489

483-
SGCONFIG_NAMESPACE="$OPERATOR_NAMESPACE"
484-
if [ "x$E2E_ALLOWED_NAMESPACES" != x ] \
485-
&& ! printf ' %s ' "$E2E_ALLOWED_NAMESPACES" | grep -qF " $OPERATOR_NAMESPACE "
486-
then
487-
SGCONFIG_NAMESPACE="$CLUSTER_NAMESPACE"
488-
fi
489-
490490
EXTRA_OPTS="$(printf %s "
491491
-Dquarkus.log.category.\"io.stackgres\".level=DEBUG
492492
-Dquarkus.log.category.\"io.quarkus\".level=INFO
@@ -515,12 +515,12 @@ EOF
515515
{"name":"JAVA_OPTS","value":$EXTRA_OPTS},
516516
{"name":"APP_OPTS","value":$EXTRA_OPTS},
517517
{"name":"SGCONFIG","value":$(yq --arg sgconfig "${E2E_OPERATOR_BUNDLE_SGCONFIG:-{\}}" '($sgconfig | fromjson) * . | tostring' "$LOG_PATH/sgconfig-patch.yaml")}
518-
,
519518
$(
520519
if [ "x$E2E_ALLOWED_NAMESPACES" != x ] \
521520
&& ! printf ' %s ' "$E2E_ALLOWED_NAMESPACES" | grep -qF " $OPERATOR_NAMESPACE "
522521
then
523522
cat << INNER_EOF
523+
,
524524
{"name":"SGCONFIG_NAMESPACE","value":"$CLUSTER_NAMESPACE"}
525525
INNER_EOF
526526
fi
@@ -578,7 +578,7 @@ EOF
578578
echo "SUCCESS. The StackGres restapi service was available."
579579
fi
580580

581-
if ! wait_until eval 'kubectl get job -n "$SGCONFIG_NAMESPACE" -l "app=StackGresConfig" -o name | grep -qxF 0'
581+
if ! wait_until eval 'kubectl get job -n "$SGCONFIG_NAMESPACE" -l "app=StackGresConfig" -o name | wc -l | grep -qxF 0'
582582
then
583583
echo "FAILED. The StackGres jobs still running."
584584
return 1

stackgres-k8s/e2e/spec/operator-olm

Lines changed: 15 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -51,6 +51,13 @@ e2e_test() {
5151
}
5252

5353
check_operator_install() {
54+
SGCONFIG_NAMESPACE="$OPERATOR_NAMESPACE"
55+
if [ "x$E2E_ALLOWED_NAMESPACES" != x ] \
56+
&& ! printf ' %s ' "$E2E_ALLOWED_NAMESPACES" | grep -qF " $OPERATOR_NAMESPACE "
57+
then
58+
SGCONFIG_NAMESPACE="$CLUSTER_NAMESPACE"
59+
fi
60+
5461
install_operator_olm
5562

5663
check_operator_installed
@@ -202,13 +209,6 @@ EOF
202209
-Dquarkus.log.category.\"io.stackgres.operator.conciliation\".level=TRACE
203210
" | tr -s ' \n' ' ' | jq -s -R .)"
204211

205-
SGCONFIG_NAMESPACE="$OPERATOR_NAMESPACE"
206-
if [ "x$E2E_ALLOWED_NAMESPACES" != x ] \
207-
&& ! printf ' %s ' "$E2E_ALLOWED_NAMESPACES" | grep -qF " $OPERATOR_NAMESPACE "
208-
then
209-
SGCONFIG_NAMESPACE="$CLUSTER_NAMESPACE"
210-
fi
211-
212212
cat << EOF > "$LOG_PATH/stackgres-subscription-$STACKGRES_VERSION.yaml"
213213
apiVersion: operators.coreos.com/v1alpha1
214214
kind: Subscription
@@ -303,7 +303,7 @@ check_operator_installed() {
303303
echo "SUCCESS. The StackGres restapi service was available."
304304
fi
305305

306-
if ! wait_until eval 'kubectl get job -n "$SGCONFIG_NAMESPACE" -l "app=StackGresConfig" -o name | grep -qxF 0'
306+
if ! wait_until eval 'kubectl get job -n "$SGCONFIG_NAMESPACE" -l "app=StackGresConfig" -o name | wc -l | grep -qxF 0'
307307
then
308308
echo "FAILED. The StackGres jobs still running."
309309
return 1
@@ -431,6 +431,13 @@ create_or_replace_cluster_from_backup() {
431431
}
432432

433433
check_operator_delete() {
434+
SGCONFIG_NAMESPACE="$OPERATOR_NAMESPACE"
435+
if [ "x$E2E_ALLOWED_NAMESPACES" != x ] \
436+
&& ! printf ' %s ' "$E2E_ALLOWED_NAMESPACES" | grep -qF " $OPERATOR_NAMESPACE "
437+
then
438+
SGCONFIG_NAMESPACE="$CLUSTER_NAMESPACE"
439+
fi
440+
434441
kubectl delete subscription -n "$OPERATOR_NAMESPACE" stackgres
435442

436443
kubectl delete clusterserviceversion -n "$OPERATOR_NAMESPACE" "stackgres.v$BUNDLE_STACKGRES_VERSION"

stackgres-k8s/e2e/utils/operator

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -830,7 +830,7 @@ before_operator_install_or_upgrade() {
830830
after_operator_install_or_upgrade() {
831831
wait_services_available "$OPERATOR_NAMESPACE" 1 '^stackgres-operator$'
832832
wait_services_available "$OPERATOR_NAMESPACE" 1 '^stackgres-restapi$'
833-
wait_until eval 'kubectl get job -n "$SGCONFIG_NAMESPACE" -l "app=StackGresConfig" -o name | grep -qxF 0'
833+
wait_until eval 'kubectl get job -n "$OPERATOR_NAMESPACE" -l "app=StackGresConfig" -o name | wc -l | grep -qxF 0'
834834
if [ "$OPERATOR_FROM" != true ] && [ "$EXTENSIONS_CACHE_ENABLED" = true ]
835835
then
836836
wait_services_available "$OPERATOR_NAMESPACE" 1 '^stackgres-operator-extensions-cache$'

0 commit comments

Comments
 (0)