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

[release/1.5] ci: remove GOPROXY environment variable due to https://github.com/go-… #7300

Merged
merged 1 commit into from
Aug 16, 2022
Merged

[release/1.5] ci: remove GOPROXY environment variable due to https://github.com/go-… #7300

merged 1 commit into from
Aug 16, 2022

Conversation

samuelkarp
Copy link
Member

Cherry-pick #7293

@samuelkarp
ci: remove GOPROXY environment variable due to go-yaml/yaml#887
d3d97cc 
Signed-off-by: Samuel Karp <samuelkarp@google.com>
(cherry picked from commit 396d2a7)
Signed-off-by: Samuel Karp <samuelkarp@google.com>
@samuelkarp
Copy link
Member Author

Prow failure is unrelated:

++ gcloud auth activate-service-account --key-file /etc/service-account/service-account.json --project=k8s-cri-containerd
ERROR: (gcloud.auth.activate-service-account) There was a problem refreshing your current auth tokens: ('invalid_grant: Invalid JWT Signature.', {'error': 'invalid_grant', 'error_description': 'Invalid JWT Signature.'})
Please run:
  $ gcloud auth login
to obtain new credentials.

@dims
Copy link
Member

dims commented Aug 16, 2022

/retest

@dims
Copy link
Member

dims commented Aug 16, 2022

/test pull-containerd-node-e2e

@kzys kzys merged commit 8def637 into containerd:release/1.5 Aug 16, 2022
@samuelkarp samuelkarp deleted the ci-workflow-goproxy/1.5 branch August 16, 2022 19:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kzys kzys kzys approved these changes

@cpuguy83 cpuguy83 cpuguy83 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@samuelkarp @dims @kzys @cpuguy83