This repository has been archived by the owner on May 16, 2023. It is now read-only.
[elasticsearch] Add upgrade integration test #115
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This test will install the latest chart release and then attempt to
upgrade to what is currently in git. This is designed to test two
things.
by Kubernetes. An example of this happening is bump es version, remove default storageClassName #94 where the change
was not caught as part of pull request testing but was luckily caught
during manual testing before the release.
cluster at the end. The test will always override the
"terminationGracePeriod" to make sure that a rolling upgrade is always
done even if there are no changes.
Kubectl and helm have also been updated to the latest releases. Oddly
enough there was a breaking change in the mock value used for release
names where it is now lowercased for some reason.
The kubectl upgraded was needed to get the handy timeout flag for the
kubectl rollout status
command to make sure that jobs don't hangforever if something does actually go wrong. This command is needed
because the
helm --wait
flag only worksapps/v1
resources. Seehelm/helm#3173 for more information.