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

Update Docker APT references in Vagrant #1343

Merged
merged 1 commit into from
Mar 17, 2020

Conversation

adamczi
Copy link
Contributor

@adamczi adamczi commented Mar 17, 2020

Hi, I noticed Vagrant version of JuiceShop does not provision, due to errors with apt repository:

default: W: The repository 'https://apt.dockerproject.org/repo ubuntu-xenial Release' does not have a Release file.
default: E: Failed to fetch https://apt.dockerproject.org/repo/dists/ubuntu-xenial/main/binary-amd64/Packages  404  Not Found
default: E: Some index files failed to download. They have been ignored, or old ones used instead.

and therefore:

default: E: Package 'docker-engine' has no installation candidate

Entering https://apt.dockerproject.org/repo/ shows a notice that dockerproject.org shuts down in the end of March 2020:

Notice: Shutting down dockerproject.org APT and YUM repos 2020-03-31

Docker will be shutting down the deprecated APT and YUM repositories hosted at "dockerproject.org" and "dockerproject.com" on the 31st of March 2020.

We noticed that this project is referencing one of these repositories, and recommend updating to use the "download.docker.com" repository to prevent disruption.

More info: https://www.docker.com/blog/changes-dockerproject-org-apt-yum-repositories/ 

So I:

  • changed public key reference to official Ubuntu keyserver
  • changed apt repository line to docker.com
  • replaced deprecated docker-engine installation to docker-ce
  • added set -e script option to stop on errors (so with previous errors it would stop on dockerproject.org repo error and not go further trying to download docker-engine)

Docker repository address is changing in the end of March 2020. Also added current public key.

Signed-off-by: adamczi <adamczi@users.noreply.github.com>
@bkimminich
Copy link
Member

Cool, thanks a lot! As Vagrant seems to be so rarely used, I'm even more glad you found and fixed this proactively!

@bkimminich
Copy link
Member

Please send me an email or Gitter-PM with your home address and I'll send you a nice collection of stickers!

@bkimminich bkimminich merged commit 822ac70 into juice-shop:master Mar 17, 2020
@github-actions github-actions bot locked and limited conversation to collaborators Aug 12, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants