1. BUILD_NUMBER is deprecated and needed to be replaced for prow jobs
2. PULL_NUMBER is not available when batch jobs, including merges,
are executed. Need a fallback to avoid errors.
* Fixing orphaned helm deploys left by CI
The CI system has cases where helm delete and namespace cleanup
are skipped if there is an error getting logs on a pod. This causes
the pod logs to always return true and avoid a premature exit before
cleanup is run
* Fixing issue where pods not yet running to not have delay before rechecking
When pods are not yet in a running state there is no delay before
rechecking. This is causing timeouts to happen waiting for pods.
This change does 3 things:
1. Gets the logs from the test pods
2. Prints the output of helm status
3. Gets the logs from the pods running for a release
The bot that merges PRs re-runs the e2e job before merging a pull
request. By also having it here (circleci provides fast response
to PR authors) we make sure the version has been incremented in
the chart prior to it being merged.
The previous version of the sync script ran `helm dep update` which
would recreate the requirements.lock file. This caused new builds
of charts as a total package with different versions of dependencies
but the same chart version. The package was mutating.
This change works towards our goal of immutable charts for a chart
at a version.
The changed.sh script tried to find the merge base but the merge
base branch was not available. This change provides it so that
a common ancestor can be found.
The previous diff was a comparison against master. If master had
changed since the current PR was created the changed charts would
be re-tested. This could create a problem when helm was upgraded
and lint changed causing previously passing charts to fail. Since
they were not associated with the current PR it was an outlier
problem for the PR author.
The new diff does a comparison to the merge-base which ties to look
for changes introduced by the current pull request.
This change does the following:
- Adds a circleci config file
- Has 2 scripts. One for installing tools and a separate one for
executing tests
- Removes lint testing from the changed.sh script since it is
performed elsewhere
- For linting changes the git diff mechanism to look at the merge
base rather than all differences from master. The intent is to
look for changes in this request rather than including those
in PRs that may have already been merged to master while this
one lingered.
Ref #2337
* Add basic verification of release
* Set BUILD_NUMBER in pr-review.sh
* Set default verification pause if not set
* Work around default pause value with -u
* Fix default pause value in pr-review
* Use kubconfig for tests if possible
* Add basic verification of release
* Add basic verification of release
* Set BUILD_NUMBER in pr-review.sh
* Set default verification pause if not set
* Work around default pause value with -u
* Fix default pause value in pr-review
* Use kubconfig for tests if possible
* Add basic verification of release
* Ensure kubeconfig works standalone
* ci(*): fix reference to master and bash options
test-infra no longer has a reference to master so we need to fetch it to
see what's been changed
* Skip if no chart changes
Reinhard Nägele
Matt Farina
Josh Dolitsky
Adnan Abdulhussein
Michael Goodness
Vic Iglesias
Anirudh Ramanathan
Adnan Abdulhussein