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.

Sign up for GitHub

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

Jump to bottom

v1.11: Update Go to 1.17.6 #18416

Merged
merged 2 commits into from
Jan 12, 2022
Merged

v1.11: Update Go to 1.17.6 #18416

merged 2 commits into from
Jan 12, 2022

Conversation

tklauser
Copy link
Member

No description provided.

@tklauser tklauser added release-note/misc This PR makes changes that have no direct user impact. backport/1.11 This PR represents a backport for Cilium 1.11.x of a PR that was merged to main. labels Jan 10, 2022
@maintainer-s-little-helper maintainer-s-little-helper bot added the kind/backports This PR provides functionality previously merged into master. label Jan 10, 2022
@tklauser tklauser temporarily deployed to release-base-images January 10, 2022 09:53 Inactive
@tklauser tklauser force-pushed the pr/tklauser/v1.11-update-go-1.17.6 branch from e11d6b8 to b65af96 Compare January 10, 2022 10:03
@tklauser tklauser temporarily deployed to release-base-images January 10, 2022 10:03 Inactive
@tklauser tklauser temporarily deployed to release-base-images January 10, 2022 10:03 Inactive
@tklauser tklauser force-pushed the pr/tklauser/v1.11-update-go-1.17.6 branch from b65af96 to 4568520 Compare January 10, 2022 10:24
@tklauser tklauser marked this pull request as ready for review January 10, 2022 10:24
@tklauser tklauser requested review from a team as code owners January 10, 2022 10:24
@tklauser
Copy link
Member Author

tklauser commented Jan 10, 2022 •
edited by maintainer-s-little-helper bot

/test-backport-1.11

Job 'Cilium-PR-K8s-GKE' failed and has not been observed before, so may be related to your PR:

Click to show.

Test Name

K8sDatapathConfig Host firewall With native routing and endpoint routes

Failure Output

FAIL: Managed to reach 10.120.2.1:69 from testclient-host-5ls8p

If it is a flake, comment /mlh new-flake Cilium-PR-K8s-GKE so I can create a new GitHub issue to track it.

Job 'Cilium-PR-K8s-1.22-kernel-4.19' failed and has not been observed before, so may be related to your PR:

Click to show.

Test Name

K8sServicesTest Checks graceful termination of service endpoints Checks client terminates gracefully on service endpoint deletion

Failure Output

FAIL: Timed out after 60.000s.

If it is a flake, comment /mlh new-flake Cilium-PR-K8s-1.22-kernel-4.19 so I can create a new GitHub issue to track it.

Job 'Cilium-PR-K8s-1.19-kernel-4.9' failed and has not been observed before, so may be related to your PR:

Click to show.

Test Name

K8sServicesTest Checks service across nodes Tests NodePort (kube-proxy)

Failure Output

FAIL: Request from testclient-5f9f5 pod to service tftp://[fd04::11]:30428/hello failed

If it is a flake, comment /mlh new-flake Cilium-PR-K8s-1.19-kernel-4.9 so I can create a new GitHub issue to track it.

@tklauser
Update Go to 1.17.6
7589985 
Signed-off-by: Tobias Klauser <tobias@cilium.io>
@tklauser
images: update cilium-{runtime,builder} for Go 1.17.6
4859dee 
Signed-off-by: Tobias Klauser <tobias@cilium.io>
@tklauser tklauser force-pushed the pr/tklauser/v1.11-update-go-1.17.6 branch from 4568520 to 4859dee Compare January 12, 2022 09:11
@tklauser tklauser temporarily deployed to release-base-images January 12, 2022 09:11 Inactive
@tklauser tklauser temporarily deployed to release-base-images January 12, 2022 09:11 Inactive
@tklauser
Copy link
Member Author

tklauser commented Jan 12, 2022 •
edited

/test-backport-1.11

rebased to pick up python3 fix

@tklauser
Copy link
Member Author

tklauser commented Jan 12, 2022 •
edited

/test-1.16-netnext

vm provisioning failed: https://jenkins.cilium.io/job/Cilium-PR-K8s-1.16-net-next/2187

@tklauser
Copy link
Member Author

Travis CI failed due to dockerhub rate limiting, all other tests passed and reviews are in. Marking as ready to merge.

@tklauser tklauser added the ready-to-merge This PR has passed all tests and received consensus from code owners to merge. label Jan 12, 2022
@aditighag aditighag merged commit fd641b8 into v1.11 Jan 12, 2022
@aditighag aditighag deleted the pr/tklauser/v1.11-update-go-1.17.6 branch January 12, 2022 22:43
@joestringer joestringer mentioned this pull request Jan 18, 2022
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@rolinh rolinh rolinh approved these changes

@pchaigno pchaigno pchaigno approved these changes

@nbusseneau nbusseneau nbusseneau approved these changes

Assignees
No one assigned
Labels
backport/1.11 This PR represents a backport for Cilium 1.11.x of a PR that was merged to main. kind/backports This PR provides functionality previously merged into master. 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
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
5 participants
@tklauser @rolinh @pchaigno @nbusseneau @aditighag