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

use GOPROXY=direct for CI builds #1120

Closed
wants to merge 3 commits into from

Conversation

bcressey
Copy link
Contributor

Issue number:
N/A

Description of changes:
Build jobs have started timing out after spending 15-30 minutes on the go mod vendor step.

This sets GOPROXY=direct in the environment and passes it through to the Docker container that runs the command.

Testing done:
None, yet.

Terms of contribution:

By submitting this pull request, I agree that this contribution is dual-licensed under the terms of both the Apache License, version 2.0, and the MIT license.

This allows the Go module proxy to be specified or disabled through
the local environment.

Signed-off-by: Ben Cressey <[email protected]>
Otherwise requests end up throttled and jobs time out.

Signed-off-by: Ben Cressey <[email protected]>
@bcressey
Copy link
Contributor Author

Discarding this in favor of #1121.

@bcressey bcressey deleted the goproxy-direct-ci branch September 16, 2020 17:51
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.

1 participant