Skip to content

Commit 7e4193a

Browse files
committed
Move placement kuttl tests under default kuttl namespace
Currently, zuul only runs default kuttl tests, thus placement kuttl tests were not getting validated. This commit ensure they are also tested. Signed-off-by: Amartya Sinha <amsinha@redhat.com>
1 parent 13c7e53 commit 7e4193a

50 files changed

Lines changed: 25 additions & 119 deletions

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

Makefile

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -411,7 +411,7 @@ kuttl-test-prep:
411411

412412
.PHONY: kuttl-test-run
413413
kuttl-test-run:
414-
oc kuttl test --v 1 --start-kind=false --config $(KUTTL_SUITE_DIR)/config.yaml
414+
KUTTL_SUITE_DIR=$(KUTTL_SUITE_DIR) oc kuttl test --v 1 --start-kind=false --config $(KUTTL_SUITE_DIR)/config.yaml
415415

416416
.PHONY: kuttl-test
417417
kuttl-test: kuttl-test-prep kuttl-test-run
@@ -424,6 +424,7 @@ kuttl-test-cleanup:
424424
# result in errors in mariadb- and keystone-operator and then
425425
# finalizer removal get stuck blocking the namespace deletion.
426426
if [ "${namespace_exists}" != "" ]; then \
427+
oc delete --wait=true --all=true -n $(KUTTL_NAMESPACE) --timeout=120s PlacementAPI; \
427428
oc delete --wait=true --all=true -n $(KUTTL_NAMESPACE) --timeout=120s Nova; \
428429
oc delete --wait=true --all=true -n $(KUTTL_NAMESPACE) --timeout=120s OpenStackControlPlane; \
429430
oc delete --wait=true namespace $(KUTTL_NAMESPACE); \

README.md

Lines changed: 5 additions & 4 deletions

test/kuttl/test-suites/placement/go_templates/apiEndpoints.gotemplate renamed to test/kuttl/test-suites/default/go_templates/apiEndpoints.gotemplate

File renamed without changes.

test/kuttl/test-suites/placement/lib/helper_functions.sh renamed to test/kuttl/test-suites/default/lib/helper_functions.sh

File renamed without changes.

test/kuttl/test-suites/placement/common/assert_sample_deployment.yaml renamed to test/kuttl/test-suites/default/placement-tests/common/assert_sample_deployment.yaml

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@ spec:
1717
secret: osp-secret
1818
serviceUser: placement
1919
status:
20-
databaseHostname: openstack.placement-kuttl-tests.svc
20+
databaseHostname: openstack.nova-kuttl-default.svc
2121
readyCount: 1
2222
conditions:
2323
- message: Setup complete
@@ -259,14 +259,14 @@ kind: TestAssert
259259
namespaced: true
260260
commands:
261261
- script: |
262-
. $PLACEMENT_KUTTL_DIR/../lib/helper_functions.sh
263-
apiEndpoints=$(oc get -n $NAMESPACE KeystoneEndpoint placement -o go-template-file=$PLACEMENT_KUTTL_DIR/../go_templates/apiEndpoints.gotemplate)
262+
. ${KUTTL_SUITE_DIR}/lib/helper_functions.sh
263+
apiEndpoints=$(oc get -n $NAMESPACE KeystoneEndpoint placement -o go-template-file=${KUTTL_SUITE_DIR}/go_templates/apiEndpoints.gotemplate)
264264
assert_regex $apiEndpoints 'http:\/\/placement-internal\..+\.svc.*'
265265
assert_regex $apiEndpoints 'http:\/\/placement-public\..+\.svc.*'
266266
267267
# when using image digests the containerImage URLs are SHA's so we verify them with a script
268268
tupleTemplate='{{ range (index .spec.template.spec.containers 0).env }}{{ .name }}{{ "#" }}{{ .value}}{{"\n"}}{{ end }}'
269-
imageTuples=$(oc get -n openstack-operators deployment placement-operator-controller-manager -o go-template="$tupleTemplate")
269+
imageTuples=$(oc get -n openstack-operators deployment nova-operator-controller-manager -o go-template="$tupleTemplate")
270270
# format of imageTuple is: RELATED_IMAGE_PLACEMENT_<service>#<image URL with SHA> separated by newlines
271271
for ITEM in $(echo $imageTuples); do
272272
# it is an image
@@ -280,7 +280,7 @@ commands:
280280
;;
281281
esac
282282
if [ "$SERVICE_IMAGE" != "$IMG_FROM_ENV" ]; then
283-
echo "$NAME image does not equal $VALUE"
283+
echo "$NAME image does not equal $IMG_FROM_ENV (got $SERVICE_IMAGE)"
284284
exit 1
285285
fi
286286
fi

test/kuttl/test-suites/placement/common/cleanup-placement.yaml renamed to test/kuttl/test-suites/default/placement-tests/common/cleanup-placement.yaml

File renamed without changes.

test/kuttl/test-suites/placement/common/errors_cleanup_placement.yaml renamed to test/kuttl/test-suites/default/placement-tests/common/errors_cleanup_placement.yaml

File renamed without changes.

test/kuttl/test-suites/placement/common/patch_placement_deploy.yaml renamed to test/kuttl/test-suites/default/placement-tests/common/patch_placement_deploy.yaml

File renamed without changes.
Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
../../../../../../config/samples/placement_v1beta1_placementapi.yaml

test/kuttl/test-suites/placement/common/tls_certificates.yaml renamed to test/kuttl/test-suites/default/placement-tests/common/tls_certificates.yaml

File renamed without changes.

0 commit comments

Comments
 (0)