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

Add "pgp-happy-eyeballs" in Travis to help cut down on gpg-related issues #471

Merged
merged 1 commit into from
Jul 26, 2018

Conversation

tianon
Copy link
Member

@tianon tianon commented Jul 25, 2018

@rbreslow
Copy link

I feel nervous about the amount of messing around pgp-happy-eyeballs does with Docker internals.

In our experience across all the Docker official images, the ipv4 pool isn't any more reliable on the whole than the ha pool (and in fact is made up of a number of the same servers). I think you probably just happened to hit a different server via ipv4 than you did via ha.

This hasn't been my experience so far. We haven't had a single failure in our builds. Why would the IPv4 pool contain the same IPv6 servers from the ha pool?

@yosifkit
Copy link
Member

messing around pgp-happy-eyeballs does with Docker internals

That's just adding a DNS server so that things in a docker build use that to look up pgp servers and get the caching proxy instead of the real server.

Why would the IPv4 pool contain the same IPv6 servers from the ha pool?

Because any of the IPv6 servers are also IPv4 servers: https://sks-keyservers.net/status/. I don't see any indication that they are IPv6 only. And ha, currently, contains one server that supports IPv6 and one that does not.

We haven't had a single failure in our builds.

Lucky you 😄 We see it all too often. Like this one which is not the IPv6 Cannot assign requested address error (quoted below for when the travis build log goes away).

+ gpg --keyserver ha.pool.sks-keyservers.net --recv-keys B97B0AFCAA1A47F044F244A07FCC7D46ACCC4CF8
gpg: keybox '/tmp/tmp.FxfD4RbclK/pubring.kbx' created
gpg: keyserver receive failed: No data

@rbreslow
Copy link

Thank you for adding more clarity @yosifkit!

@yosifkit yosifkit merged commit 2cab504 into docker-library:master Jul 26, 2018
@yosifkit yosifkit deleted the pgp-happy-eyeballs branch July 26, 2018 18:54
tianon added a commit to infosiftr/stackbrew that referenced this pull request Jul 26, 2018
- `gcc`: 8.2.0
- `ghost`: 1.25.2
- `mongo`: minor comment update
- `mysql`: `gpgconf --kill all` (docker-library/mysql#459)
- `postgres`: `gpgconf --kill all` (docker-library/postgres#471)
- `python`: remove `ca-certificates` from Alpine 3.7+ (docker-library/python#307)
- `redis`: suite aliases (redis/docker-library-redis#154)
- `rocket.chat`: 0.67.0
- `tomcat`: jessie arch removals
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.

3 participants