Skip to content

Conversation

tklauser
Copy link
Member

This pulls in additional fixes for working around #11455
and will also get rid of two transitive dependencies:
github.com/kardianos/osext and github.com/shirou/w32

@tklauser tklauser added the release-note/misc This PR makes changes that have no direct user impact. label Jun 25, 2020
@tklauser tklauser requested a review from a team as a code owner June 25, 2020 10:00
@tklauser tklauser requested a review from a team June 25, 2020 10:00
@tklauser tklauser requested review from a team as code owners June 25, 2020 10:00
@tklauser
Copy link
Member Author

test-me-please

This pulls in additional fixes for working around #11455
and will also get rid of two transitive dependencies:
github.com/kardianos/osext and github.com/shirou/w32

Signed-off-by: Tobias Klauser <tklauser@distanz.ch>
@tklauser tklauser force-pushed the pr/tklauser/vendor-update-gops branch from d9c7fe6 to bb296d3 Compare June 25, 2020 10:12
@tklauser
Copy link
Member Author

test-me-please

@coveralls
Copy link

Coverage Status

Coverage increased (+0.001%) to 37.191% when pulling bb296d3 on pr/tklauser/vendor-update-gops into e9233ce on master.

@tklauser
Copy link
Member Author

tklauser commented Jun 25, 2020

retest-runtime (vm provisioning failure due to docker pull timeouts)

@aanm
Copy link
Member

aanm commented Jun 25, 2020

The runtime test failed to be provisioned but going to merge this since #12287 will retest everything again

@aanm aanm merged commit 3384892 into master Jun 25, 2020
@aanm aanm deleted the pr/tklauser/vendor-update-gops branch June 25, 2020 13:02
@jdef
Copy link

jdef commented Sep 8, 2020

any chance of the gops bump being backported to 1.7.x or 1.8.x?

google/gops#97 is missing from gops 0.3.6 and is generating TONS of "use of a closed connection" noise in our logs.

@tgraf tgraf added needs-backport/1.8 kind/bug This is a bug in the Cilium logic. labels Sep 8, 2020
tklauser added a commit that referenced this pull request Sep 9, 2020
[ upstream commit bb296d3 ]

Manual backport of #12283 due to update of cilium-runtime and
cilium-builder images.

This pulls in additional fixes for working around #11455
and will also get rid of two transitive dependencies:
github.com/kardianos/osext and github.com/shirou/w32

Signed-off-by: Tobias Klauser <tklauser@distanz.ch>
aanm pushed a commit that referenced this pull request Sep 10, 2020
[ upstream commit bb296d3 ]

Manual backport of #12283 due to update of cilium-runtime and
cilium-builder images.

This pulls in additional fixes for working around #11455
and will also get rid of two transitive dependencies:
github.com/kardianos/osext and github.com/shirou/w32

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

any chance of the gops bump being backported to 1.7.x or 1.8.x?

google/gops#97 is missing from gops 0.3.6 and is generating TONS of "use of a closed connection" noise in our logs.

@jdef this PR was backported to v1.8 in #13128 and will be included in the next 1.8.x release.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug This is a bug in the Cilium logic. release-note/misc This PR makes changes that have no direct user impact.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants