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

Migrate nginx images to be build from Dockerfile same as the varnishes #511

Merged
merged 1 commit into from
Aug 16, 2024

Conversation

awmackowiak
Copy link
Contributor

The nginx images were pulled from the docker.io which was built 8 years ago, so the format of the image (nginx) now stops being supported by the Docker

(master) $ docker-compose -f docker-compose.yml -f docker-compose.override.yml up --build
WARN[0000] /Users/artur.mackowiak/Projects/Pluton/vaas_software/vaas/docker-compose.yml: `version` is obsolete
WARN[0000] /Users/artur.mackowiak/Projects/Pluton/vaas_software/vaas/docker-compose.override.yml: `version` is obsolete
[+] Running 2/6
 ✘ nginx-5 Error   context 5.5s
 ⠼ nginx-1 Pulling 5.5s
 ⠼ nginx-4 Pulling 5.5s
 ⠼ nginx-3 Pulling 5.5s
 ✘ nginx-2 Error   context canceled 5.5s
 ⠼ nginx-0 Pulling 5.5s
[DEPRECATION NOTICE] Docker Image Format v1 and Docker Image manifest version 2, schema 1 support is disabled by default and will be removed in an upcoming release. Suggest the author of docker.io/allegro/vaas-nginx:latest to upgrade the image to the OCI Format or Docker Image manifest v2, schema 2. More information at https://docs.docker.com/go/deprecated-image-specs/
(master) $ 

What has been done:

  • add the build fields in docker compose definitions for nginx that it starts to be built locally based on Docker file
  • get rid of version key in compose definitions because of obsoletion

@jtaczanowski jtaczanowski merged commit 51b19fd into master Aug 16, 2024
4 checks passed
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.

3 participants