Skip to content

Update to new MySQL GPG Key #808

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

Merged
merged 4 commits into from
Jan 19, 2022
Merged

Conversation

ltangvald
Copy link
Collaborator

As part of the 8.0.28 release, MySQL is introducing a new gpg key. It's a new key instead of extending the duration of the old one because the encryption algorithm has been changed.

@ltangvald
Copy link
Collaborator Author

The build failure is expected, and should (fingers-crossed) be resolved when the next MySQL release is out.

@ltangvald
Copy link
Collaborator Author

One complication I didn't consider is that we no longer update 5.6, so that signature is old. We could make it so 5.6 uses the old key, but that key expires soon and will not be updated (because 5.6 is no longer supported). I would suggest removing 5.6 builds.

Since the key change is part of the regular release, we need
to up the version to 5.7.37 and 8.0.28
@rfay
Copy link

rfay commented Jan 18, 2022

What I've been doing with unsupported upstreams is just to remove the key and the /etc/apt/sources.list.d entry from them after build. Then the key can't cause trouble, but other things in the image may still be updatable by child images.

@tianon
Copy link
Member

tianon commented Jan 18, 2022

Ahhh, I've looked and looked for a canonical place that says "5.6 is EOL" and all I can find is wishy-washy "EOL on X" notices, presumably because it's still technically "supported" but maybe only if you have an active support contract or something? If the public/open version is EOL, we should definitely just remove it from the repository entirely. 😅

Edit: opened a PR at #810 to completely remove 5.6

@tianon tianon mentioned this pull request Jan 18, 2022
@tianon
Copy link
Member

tianon commented Jan 18, 2022

@ltangvald I've cherry-picked #810 into this one so we can kill both birds here, but I'm happy to pull that back out if you prefer? ❤️

@rfay
Copy link

rfay commented Jan 18, 2022

Please don't remove 5.5 or 5.6 from dockerhub, as ddev uses those for upstream. They're only used by people who are resurrecting and updating old sites, and only used for local dev.

@tianon
Copy link
Member

tianon commented Jan 18, 2022

Removing tags here will remove them from the "Supported" section on the Hub readme (and will prevent official images from spending cycles rebuilding them on the official build servers), but the tags will still be available to users who want them. (See https://github.com/docker-library/official-images#library-definition-files for more detail on this.)

@ltangvald
Copy link
Collaborator Author

Since the release is out and all looks good, I'll merge this now. As mentioned, removing 5.6 from the code just means we stop building it. The image will still be there (it does mean you'll need to keep the base os layer updated yourself).

@ltangvald ltangvald merged commit f4032a1 into docker-library:master Jan 19, 2022
docker-library-bot added a commit to docker-library-bot/official-images that referenced this pull request Jan 19, 2022
Changes:

- docker-library/mysql@f4032a1: Merge pull request docker-library/mysql#808 from ltangvald/master
- docker-library/mysql@aa60002: Remove EOL 5.6
- docker-library/mysql@25b68c6: Update server versions
- docker-library/mysql@e0359cc: Apply template changes
- docker-library/mysql@2fc7349: Update to new MySQL GPG Key
- docker-library/mysql@37cf404: Merge pull request docker-library/mysql#798 from ducksecops/master
- docker-library/mysql@4207add: update GOSU to 1.14
@JustinAvaLabs
Copy link

Since the release is out and all looks good, I'll merge this now. As mentioned, removing 5.6 from the code just means we stop building it. The image will still be there (it does mean you'll need to keep the base os layer updated yourself).

You broke the GPG key and now the official docker images don't work.
#811

@ltangvald
Copy link
Collaborator Author

Since the release is out and all looks good, I'll merge this now. As mentioned, removing 5.6 from the code just means we stop building it. The image will still be there (it does mean you'll need to keep the base os layer updated yourself).

You broke the GPG key and now the official docker images don't work. #811

Not actually related to this PR. The official mysql apt repo is using a new signing key. The old one uses DSA1024, which is no longer considered secure.

@JustinAvaLabs
Copy link

Got it, my mistake. Is there any insight on how long until the official images are fixed?

@ltangvald
Copy link
Collaborator Author

I'm not clear on when/how rebuilds happen for official images. @tianon ?

@yosifkit
Copy link
Member

Short answer is they are built once the commits here are referenced in the library/mysql file in https://github.com/docker-library/official-images. So after docker-library/official-images#11700 is merged.

More info: https://github.com/docker-library/mysql/tree/f4032a1af40618ab81ecfe03ce0366aeb9f5af5e#see-a-change-merged-here-that-doesnt-show-up-on-docker-hub-yet

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.

5 participants