Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

test: fix Infinite loop during VM provisioning #17031

Merged
merged 1 commit into from Aug 18, 2021
Merged

Conversation

h3llix
Copy link
Contributor

@h3llix h3llix commented Aug 2, 2021

Fixes: #16871

An environment variable in the Jenkins file has been set up to provide a timeout for polling of Cilium Pods.

Signed-off-by: Gaurav Genani h3llix.pvt@gmail.com

@h3llix h3llix requested a review from a team as a code owner August 2, 2021 19:08
@h3llix h3llix requested a review from christarazi August 2, 2021 19:08
@maintainer-s-little-helper maintainer-s-little-helper bot added the dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. label Aug 2, 2021
jenkinsfiles/kubernetes-upstream.Jenkinsfile Outdated Show resolved Hide resolved
test/kubernetes-test.sh Show resolved Hide resolved
@h3llix h3llix force-pushed the fixK8stest branch 2 times, most recently from 2db54bd to cf00c46 Compare August 10, 2021 09:59
Copy link
Member

@christarazi christarazi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

test/kubernetes-test.sh Outdated Show resolved Hide resolved
@christarazi christarazi added the release-note/misc This PR makes changes that have no direct user impact. label Aug 11, 2021
@maintainer-s-little-helper maintainer-s-little-helper bot removed the dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. label Aug 11, 2021
@christarazi christarazi added area/CI-improvement Topic or proposal to improve the Continuous Integration workflow kind/bug/CI This is a bug in the testing code. release-note/ci This PR makes changes to the CI. and removed release-note/misc This PR makes changes that have no direct user impact. labels Aug 11, 2021
Fixes: cilium#16871

Signed-off-by: Gaurav Genani <h3llix.pvt@gmail.com>
@christarazi
Copy link
Member

#17031 (comment) wasn't addressed :)

@h3llix
Copy link
Contributor Author

h3llix commented Aug 15, 2021

#17031 (comment) wasn't addressed :)

Hi @christarazi. I have fixed it here Here.

@christarazi
Copy link
Member

test-me-please

@aanm aanm merged commit c44ff1b into cilium:master Aug 18, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/CI-improvement Topic or proposal to improve the Continuous Integration workflow kind/bug/CI This is a bug in the testing code. release-note/ci This PR makes changes to the CI.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

CI: Potential infinite loop during VM provisioning
3 participants