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

ci: fix timeout vagrant up timeout #11798

Merged
merged 1 commit into from Jun 2, 2020
Merged

Conversation

nebril
Copy link
Member

@nebril nebril commented Jun 1, 2020

Due to jenkins bug: https://issues.jenkins-ci.org/browse/JENKINS-51454
pipeline was failing after first ci boot timeout.

This change changes timeout mechanism to timeout cli call instead

@nebril nebril requested a review from a team as a code owner June 1, 2020 13:22
@maintainer-s-little-helper
Copy link

Please set the appropriate release note label.

@nebril
Copy link
Member Author

nebril commented Jun 1, 2020

test-4.9

@nebril nebril force-pushed the pr/fix-inner-timeout-jenkins branch from c4465f6 to a47c031 Compare June 1, 2020 13:41
@nebril
Copy link
Member Author

nebril commented Jun 1, 2020

test-4.9

Due to jenkins bug: https://issues.jenkins-ci.org/browse/JENKINS-51454
pipeline was failing after first ci boot timeout.

This change changes timeout mechanism to `timeout` cli call instead

Signed-off-by: Maciej Kwiek <maciej@isovalent.com>
@nebril nebril force-pushed the pr/fix-inner-timeout-jenkins branch from a47c031 to 1caaf58 Compare June 1, 2020 13:43
@nebril
Copy link
Member Author

nebril commented Jun 1, 2020

test-4.9

@nebril nebril added the release-note/ci This PR makes changes to the CI. label Jun 1, 2020
@coveralls
Copy link

coveralls commented Jun 1, 2020

Coverage Status

Coverage increased (+0.009%) to 36.912% when pulling 1caaf58 on pr/fix-inner-timeout-jenkins into b7be1c0 on master.

@nebril nebril changed the title ci: clean up vagrant-ci-start call, fix timeout ci: fix timeout vagrant up timeout Jun 1, 2020
@b3a-dev b3a-dev added this to In Progress (Cilium) in CI Force via automation Jun 1, 2020
@nebril nebril merged commit 630df50 into master Jun 2, 2020
1.8.0 automation moved this from In progress to Merged Jun 2, 2020
CI Force automation moved this from In Progress (Cilium) to Fixed / Done Jun 2, 2020
@nebril nebril deleted the pr/fix-inner-timeout-jenkins branch June 2, 2020 09:07
@maintainer-s-little-helper maintainer-s-little-helper bot added this to Needs backport from master in 1.8.0 Jun 2, 2020
@maintainer-s-little-helper maintainer-s-little-helper bot moved this from Needs backport from master to Backport pending to v1.8 in 1.8.0 Jun 3, 2020
@joestringer joestringer moved this from Backport pending to v1.8 to Backport done to v1.8 in 1.8.0 Jun 6, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release-note/ci This PR makes changes to the CI.
Projects
No open projects
1.8.0
  
Merged
1.8.0
Backport done to v1.8
CI Force
  
Fixed / Done
Development

Successfully merging this pull request may close these issues.

None yet

4 participants