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

vagrant: bump box versions, again #15129

Merged
merged 1 commit into from
Mar 4, 2021
Merged

vagrant: bump box versions, again #15129

merged 1 commit into from
Mar 4, 2021

Conversation

tklauser
Copy link
Member

@tklauser tklauser commented Feb 26, 2021

The previous box version bump in commit bc9998f ("vagrant: bump box
versions") did not include the Go 1.16 update, despite mentioning it in
the commit message.

This time update it to a box containing Go 1.16 for real.

Note that this also bumps the patch version for kernel 4.9 and 4.19 in
the VM images to 4.9.212 (see
cilium/packer-ci-build#257) and 4.19.177,
respectively (see cilium/packer-ci-build#256).
The reason for this is that the old versions are no longer available
from https://kernel.ubuntu.com/~kernel-ppa/mainline

@tklauser tklauser added the release-note/misc This PR makes changes that have no direct user impact. label Feb 26, 2021
@tklauser tklauser requested a review from a team as a code owner February 26, 2021 14:25
@tklauser tklauser requested a review from kkourt February 26, 2021 14:25
@maintainer-s-little-helper maintainer-s-little-helper bot added this to In progress in 1.10.0 Feb 26, 2021
@tklauser
Copy link
Member Author

test-me-please

@tklauser
Copy link
Member Author

tklauser commented Mar 1, 2021

test-1.19-4.19

previous failure: https://jenkins.cilium.io/job/Cilium-PR-K8s-1.19-kernel-4.19/4819/

@tklauser
Copy link
Member Author

tklauser commented Mar 1, 2021

test-1.20-4.9

previous failure: https://jenkins.cilium.io/job/Cilium-PR-K8s-1.20-kernel-4.9/800/

@tklauser
Copy link
Member Author

tklauser commented Mar 1, 2021

test-me-please

@tklauser
Copy link
Member Author

tklauser commented Mar 1, 2021

4.19 tests are broken with the current image because it does contain a 4.15 kernel rather than a 4.19 kernel: https://jenkins.cilium.io/job/Cilium-PR-K8s-1.19-kernel-4.19/4827

This is due to the micro version in use no longer being available from Ubuntu Kernel PPA. I've sent cilium/packer-ci-build#256 which should hopefully fix this. Once the new VM image is build, I'll bump the version in this PR.

The 4.19 tests seem broken because of #15141

@tklauser tklauser closed this Mar 2, 2021
@tklauser tklauser reopened this Mar 2, 2021
@tklauser
Copy link
Member Author

tklauser commented Mar 2, 2021

test-me-please

@tklauser
Copy link
Member Author

tklauser commented Mar 2, 2021

Same issue as #15129 (comment) for the 4.9 tests: the image was built without a 4.9 kernel being installed. I've sent cilium/packet-ci-build#257 with a fix.

@tklauser tklauser added the dont-merge/waiting-for-upstream Only merge once upstream library or kernel features have landed label Mar 2, 2021
@tklauser tklauser force-pushed the pr/update-packer-ci-build branch 2 times, most recently from 0c0a67b to b53fa41 Compare March 3, 2021 13:46
@tklauser tklauser removed the dont-merge/waiting-for-upstream Only merge once upstream library or kernel features have landed label Mar 3, 2021
The previous box version bump in commit bc9998f ("vagrant: bump box
versions") did not include the Go 1.16 update, despite mentioning it in
the commit message.

This time update it to a box containing Go 1.16 for real.

Note that this also bumps the patch version for kernel 4.9 and 4.19 in
the VM images to 4.9.212 (see
cilium/packer-ci-build#257) and 4.19.177,
respectively (see cilium/packer-ci-build#256).
The reason for this is that the old versions are no longer available
from https://kernel.ubuntu.com/~kernel-ppa/mainline

Signed-off-by: Tobias Klauser <tklauser@distanz.ch>
@tklauser
Copy link
Member Author

tklauser commented Mar 3, 2021

test-me-please

@tklauser
Copy link
Member Author

tklauser commented Mar 3, 2021

@maintainer-s-little-helper maintainer-s-little-helper bot added the ready-to-merge This PR has passed all tests and received consensus from code owners to merge. label Mar 3, 2021
@aanm aanm merged commit 9ff6d52 into master Mar 4, 2021
1.10.0 automation moved this from In progress to Done Mar 4, 2021
@aanm aanm deleted the pr/update-packer-ci-build branch March 4, 2021 10:27
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/misc This PR makes changes that have no direct user impact.
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

None yet

3 participants