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

Attempt to pull from a custom cache of images #2009

Closed
wants to merge 1 commit into from
Closed

Attempt to pull from a custom cache of images #2009

wants to merge 1 commit into from

Conversation

onetechnical
Copy link
Contributor

@onetechnical onetechnical commented Mar 24, 2021

Summary

Docker Hub now limits the number of pulls against its repository for unauthenticated users. While we can login to Travis in job using secret variables, these are not available in forked repositories. As a result, PRs coming from forks that hit the docker pull limit will start failing. This is most notable in integration tests.

To work around this, we cache the images requested to our custom public repository that does not require authentication. This is not a good solution, but will help work around pulls in Travis for forked repositories for now. Bandwidth usage should be monitored, both from activity as well as for potential for abuse.

Test Plan

See if integration tests can pull images.

This is not a good solution, but will help work around pulls in Travis
for forked repositories for now.
@algorandskiy
Copy link
Contributor

weird, although 3.4.7 was released few hours ago.

Building wheel for cryptography (PEP 517): finished with status 'error'
8495
  error: can't find Rust compiler

They switched to Rust 1.41 quite long ago in 3.4.5

3.4.5 - 2021-02-13
Various improvements to type hints.
Lower the minimum supported Rust version (MSRV) to >=1.41.0. This change improves compatibility with system-provided Rust on several Linux distributions.

@onetechnical
Copy link
Contributor Author

Using the approach in #2012 instead.

@onetechnical onetechnical deleted the onetechnical/docker-registry branch March 26, 2021 20:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants