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

Update Go to 1.17.3 #17792

Merged
merged 1 commit into from
Nov 9, 2021
Merged

Update Go to 1.17.3 #17792

merged 1 commit into from
Nov 9, 2021

Conversation

tklauser
Copy link
Member

@tklauser tklauser commented Nov 5, 2021

No description provided.

@tklauser tklauser added the release-note/misc This PR makes changes that have no direct user impact. label Nov 5, 2021
@tklauser tklauser temporarily deployed to release-base-images November 5, 2021 09:36 Inactive
@tklauser tklauser temporarily deployed to release-base-images November 5, 2021 09:46 Inactive
@tklauser tklauser temporarily deployed to release-base-images November 5, 2021 09:46 Inactive
@tklauser tklauser temporarily deployed to release-base-images November 5, 2021 10:17 Inactive
@tklauser tklauser temporarily deployed to release-base-images November 5, 2021 10:17 Inactive
@tklauser tklauser marked this pull request as ready for review November 5, 2021 10:22
@tklauser tklauser requested review from a team as code owners November 5, 2021 10:22
@tklauser
Copy link
Member Author

tklauser commented Nov 5, 2021 •
edited by maintainer-s-little-helper bot
Loading

/test

Job 'Cilium-PR-K8s-GKE' has 1 failure but they might be new flake since it also hit 1 known flake: #17270 (93.51)

@tklauser tklauser mentioned this pull request Nov 5, 2021
Merged
@maintainer-s-little-helper maintainer-s-little-helper bot mentioned this pull request Nov 5, 2021
Closed
@tklauser
Update Go to 1.17.3
20b5486 
Signed-off-by: Tobias Klauser <tobias@cilium.io>
@tklauser
Copy link
Member Author

tklauser commented Nov 9, 2021 •
edited by maintainer-s-little-helper bot
Loading

/test

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

Click to show.

Test Name

K8sServicesTest Checks ClusterIP Connectivity Checks service on same node

Failure Output

FAIL: cannot curl to service IP from host: time-> DNS: '0.000022()', Connect: '0.000000',Transfer '0.000000', total '5.001622'command terminated with exit code 28

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

@pchaigno
Copy link
Member

pchaigno commented Nov 9, 2021

k8s-1.22-kernel-4.9 failed with known flake #17766. Team review requests are covered. Merging.

@pchaigno pchaigno added the ready-to-merge This PR has passed all tests and received consensus from code owners to merge. label Nov 9, 2021
@pchaigno pchaigno merged commit 956c4d6 into master Nov 9, 2021
@pchaigno pchaigno deleted the pr/tklauser/update-go-1.17.3 branch November 9, 2021 21:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@pchaigno pchaigno pchaigno approved these changes

@aanm aanm aanm approved these changes

@michi-covalent michi-covalent Awaiting requested review from michi-covalent michi-covalent was automatically assigned from cilium/build

@nebril nebril Awaiting requested review from nebril nebril was automatically assigned from cilium/ci-structure

Assignees

@nebril nebril

@michi-covalent michi-covalent

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
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
5 participants
@tklauser @pchaigno @aanm @nebril @michi-covalent