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

Update K8s dependencies to v1.20.7 #104

Merged
merged 1 commit into from
Jun 4, 2021
Merged

Update K8s dependencies to v1.20.7 #104

merged 1 commit into from
Jun 4, 2021

Conversation

hiddeco
Copy link
Member

@hiddeco hiddeco commented Jun 4, 2021

This ensures we are no longer affected by GO-2021-0053.

In an initial try an attempt was made to also deal with GO-2020-0018
by updating the github.com/satori/go.uuid dependency to
v1.2.1-0.20181028125025-b2ce2384e17b.

This is however at present not possible due to a change of the NewV4
method signature (single value return -> multiple-value), and the
impact of this should be low (if not non-existent) as the
gittestserver package is only used during tests.

@hiddeco hiddeco requested a review from stefanprodan June 4, 2021 09:01
This ensures we are no longer affected by GO-2021-0053.

In an initial try an attempt was made to also deal with GO-2020-0018
by updating the `github.com/satori/go.uuid` dependency to
`v1.2.1-0.20181028125025-b2ce2384e17b`.

This is however at present not possible due to a change of the `NewV4`
method signature (single value return -> multiple-value), and the
impact of this should be low (if not non-existent) as the
`gittestserver` package is only used during tests.

Signed-off-by: Hidde Beydals <hello@hidde.co>
@hiddeco hiddeco changed the title Update K8s and UUID deps Update K8s dependencies to v1.20.7 Jun 4, 2021
@hiddeco hiddeco merged commit f0418f8 into main Jun 4, 2021
@hiddeco hiddeco deleted the k8s-update branch June 4, 2021 11:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants