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: Use images built on host in k8s-all job #14292

Merged
merged 2 commits into from
Dec 8, 2020

Conversation

nebril
Copy link
Member

@nebril nebril commented Dec 7, 2020

Since we started building images in vms some time ago it looks like the
timeout for provisioning is not enough. Let's increase it while we
decide whether it's feasible to disable building images in vms again.

Since we started building images in vms some time ago it looks like the
timeout for provisioning is not enough. Let's increase it while we
decide whether it's feasible to disable building images in vms again.

Signed-off-by: Maciej Kwiek <maciej@isovalent.com>
@nebril nebril requested a review from a team as a code owner December 7, 2020 12:05
@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 Dec 7, 2020
@maintainer-s-little-helper maintainer-s-little-helper bot added this to In progress in 1.10.0 Dec 7, 2020
@nebril
Copy link
Member Author

nebril commented Dec 7, 2020

test-missed-k8s

@nebril nebril added the release-note/ci This PR makes changes to the CI. label Dec 7, 2020
@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 Dec 7, 2020
@nebril
Copy link
Member Author

nebril commented Dec 7, 2020

test-missed-k8s

@nebril nebril changed the title Increase provisioning timeout for k8s-all job ci: Use images built on host in k8s-all job Dec 7, 2020
@nebril nebril requested a review from aanm December 7, 2020 13:12
@nebril
Copy link
Member Author

nebril commented Dec 7, 2020

test-missed-k8s

@nebril
Copy link
Member Author

nebril commented Dec 7, 2020

test-me-please

@nebril
Copy link
Member Author

nebril commented Dec 7, 2020

test-runtime

CILIUM_REGISTRY is passed properly, which results in images not being
built inside vms, which should help with build times (images are built
once on host, not 6 times in each vm provision phase).

Signed-off-by: Maciej Kwiek <maciej@isovalent.com>
@nebril nebril force-pushed the pr/incrase-k8s-all-provisioning-timeout branch from f95829c to 691d160 Compare December 7, 2020 17:42
@nebril
Copy link
Member Author

nebril commented Dec 7, 2020

test-me-please

@nebril
Copy link
Member Author

nebril commented Dec 7, 2020

test-missed-k8s

@nebril
Copy link
Member Author

nebril commented Dec 8, 2020

gke failed due to a flake, otherwise looks like CI builds passed correctly

@nebril nebril added the ready-to-merge This PR has passed all tests and received consensus from code owners to merge. label Dec 8, 2020
@nebril nebril merged commit 43d4133 into master Dec 8, 2020
@nebril nebril deleted the pr/incrase-k8s-all-provisioning-timeout branch December 8, 2020 09:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready-to-merge This PR has passed all tests and received consensus from code owners to merge. release-note/ci This PR makes changes to the CI.
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

None yet

3 participants