Skip to content

Commit

Permalink
Add a dry-run 'kubectl apply' in prepare-for-integration-tests.sh s…
Browse files Browse the repository at this point in the history
…o 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 <moyerm@vmware.com>
  • Loading branch information
mattmoyer committed Sep 2, 2021
1 parent f0a1555 commit c7a8c42
Showing 1 changed file with 4 additions and 2 deletions.
6 changes: 4 additions & 2 deletions hack/prepare-for-integration-tests.sh
Original file line number Diff line number Diff line change
Expand Up @@ -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

Expand All @@ -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

Expand Down Expand Up @@ -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

Expand Down Expand Up @@ -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

Expand Down

0 comments on commit c7a8c42

Please sign in to comment.