* [incubator/druid] update chart repo dependency references
Signed-off-by: Scott Rigby <scott@r6by.com>
* Update chart testing config for new repo locations
Signed-off-by: Scott Rigby <scott@r6by.com>
* Attempt helm dependency update with helm 2 to test CI
Signed-off-by: Scott Rigby <scott@r6by.com>
* [Documentation] Document how to test a Chart in the Review Guidelines.
Signed-off-by: Cédric de Saint Martin <cdesaintmartin@wiremind.fr>
* Oops, documentation already exists: improve it, points to it.
Signed-off-by: Cédric de Saint Martin <cdesaintmartin@wiremind.fr>
* eg instead of example.
Signed-off-by: Cédric de Saint Martin <cdesaintmartin@wiremind.fr>
* helm chart test != this test tool but still related.
Signed-off-by: Cédric de Saint Martin <cdesaintmartin@wiremind.fr>
* Better english.
Signed-off-by: Cédric de Saint Martin <cdesaintmartin@wiremind.fr>
* Better way to state that we need empty file for default values.
Signed-off-by: Cédric de Saint Martin <cdesaintmartin@wiremind.fr>
* English.
Signed-off-by: Cédric de Saint Martin <cdesaintmartin@wiremind.fr>
* After repo move to helm/helm, update steps for Owning and Maintaining A Chart
* Change repo URL from kubernetes/helm to helm/helm everywhere above chart directories (those may be separate PRs)
* Fix earlier missing apostrophe and incorrect case
* Add missing k8s/ > helm/ string changes
* [CI] Refactor repo sync
With the upcoming refactoring of the chart testing, Shellcheck is added
to CircleCI for shell scripts. In order to prepare for this, several
adjustments are necessary so it passes. On top of that, the script is
refactored as follows:
* Reorganize code into several functions for better readability and to
eliminate code redundancies
* Use curl instead of wget for intalling Helm because it's already
available in the Docker image
* Treat index.yaml separately and copy it to the bucket after syncing
charts in order to avoid chart fetch errors
* Fix repo bucket urls
* Only log errors but don't exit with non-zero exit code
* Enhancing lint.sh with additional checks
* reverting post local testing changes
* addressing reviews.
removing yq installetion
chaning yq with yaml for Chart.yaml parsing.
modifing Erro message for not github account check.
also modifing lint.sh so it can be used locally
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 fixes flaky readiness checks by double-checking readiness. It could
happen that the script deemed all pods ready when at the moment the
check is executed only ready pods are reported but not all replicas are
running yet.
