From c7a8c429eda86fa420af93b01a2707fe19797064 Mon Sep 17 00:00:00 2001 From: Matt Moyer Date: Thu, 2 Sep 2021 16:53:49 -0500 Subject: [PATCH] Add a dry-run 'kubectl apply' in `prepare-for-integration-tests.sh` so we can be sure that our manifests pass API validation. We had this for some components, but not the ones that mattered the most. Signed-off-by: Matt Moyer --- hack/prepare-for-integration-tests.sh | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/hack/prepare-for-integration-tests.sh b/hack/prepare-for-integration-tests.sh index 1cbd8ca18..90faf5bdf 100755 --- a/hack/prepare-for-integration-tests.sh +++ b/hack/prepare-for-integration-tests.sh @@ -219,8 +219,8 @@ ytt --file . \ --data-value "image_repo=$registry_repo" \ --data-value "image_tag=$tag" >"$manifest" -kubectl apply --dry-run=client -f "$manifest" # Validate manifest schema. kapp deploy --yes --app local-user-authenticator --diff-changes --file "$manifest" +kubectl apply --dry-run=client -f "$manifest" # Validate manifest schema. popd >/dev/null @@ -238,8 +238,8 @@ ytt --file . \ --data-value "pinny_bcrypt_passwd_hash=$(htpasswd -nbBC 10 x "$dex_test_password" | sed -e "s/^x://")" \ >"$manifest" -kubectl apply --dry-run=client -f "$manifest" # Validate manifest schema. kapp deploy --yes --app tools --diff-changes --file "$manifest" +kubectl apply --dry-run=client -f "$manifest" # Validate manifest schema. popd >/dev/null @@ -281,6 +281,7 @@ ytt --file . \ >"$manifest" kapp deploy --yes --app "$supervisor_app_name" --diff-changes --file "$manifest" +kubectl apply --dry-run=client -f "$manifest" # Validate manifest schema. popd >/dev/null @@ -308,6 +309,7 @@ ytt --file . \ --data-value "discovery_url=$discovery_url" >"$manifest" kapp deploy --yes --app "$concierge_app_name" --diff-changes --file "$manifest" +kubectl apply --dry-run=client -f "$manifest" # Validate manifest schema. popd >/dev/null