This repository has been archived by the owner on Feb 9, 2022. It is now read-only.
[ci/backports/1.2] backport ci updates #516
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
(cherry picked from commit 14ff119)
|
👎 Rejected by PR status |
|
bors retry |
bors bot
added a commit
that referenced
this pull request
Apr 29, 2019
516: [ci/backports/1.2] backport ci updates r=falfaro a=sameersbn Backported: b5859ca - tests: optimize retry timeouts 705ec25 - tests: [fix] retry monitoring/exporters test a8d901e - ci: ignore cluster deletion errors for EKS d1fcdcb - ci: fix aks k8s version selection 422d434 - tests: retry Exporters tests on failures Co-authored-by: Sameer Naik <sameer@bitnami.com>
Build failed |
|
bors retry |
bors bot
added a commit
that referenced
this pull request
Apr 29, 2019
516: [ci/backports/1.2] backport ci updates r=falfaro a=sameersbn Backported: b5859ca - tests: optimize retry timeouts 705ec25 - tests: [fix] retry monitoring/exporters test a8d901e - ci: ignore cluster deletion errors for EKS d1fcdcb - ci: fix aks k8s version selection 422d434 - tests: retry Exporters tests on failures Co-authored-by: Sameer Naik <sameer@bitnami.com>
Build failed |
added 4 commits
April 30, 2019 08:32
the `kversion` variable was not being properly bound because of closures as a result of which the wrong k8s version was being picked for the aks tests (cherry picked from commit 46fa8f2)
(cherry picked from commit 6fc54d2)
(cherry picked from commit 33d6f6d)
Successful passes of the ci tests generally conclude in 5 - 10mins. It has been observered that when it takes longer the tests are bound to fail. Some of our tests use 30m retry timeouts which is lot of time wasted hoping that a failing test will pass. This commit optimizes the retry timeouts for all tests to more realistic values thereby reducing the time spent by a job in the CI environment (cherry picked from commit f4d09df)
sameersbn
force-pushed
the
backports-ci-1.2-20190429
branch
from
b5859ca to
1ca0d2c
Compare
|
bors retry |
bors bot
added a commit
that referenced
this pull request
Apr 30, 2019
516: [ci/backports/1.2] backport ci updates r=falfaro a=sameersbn Backported: b5859ca - tests: optimize retry timeouts 705ec25 - tests: [fix] retry monitoring/exporters test a8d901e - ci: ignore cluster deletion errors for EKS d1fcdcb - ci: fix aks k8s version selection 422d434 - tests: retry Exporters tests on failures Co-authored-by: Sameer Naik <sameer@bitnami.com>
Build succeeded |
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
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.
Backported:
b5859ca - tests: optimize retry timeouts
705ec25 - tests: [fix] retry monitoring/exporters test
a8d901e - ci: ignore cluster deletion errors for EKS
d1fcdcb - ci: fix aks k8s version selection
422d434 - tests: retry Exporters tests on failures