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

Updated docker-compose.yml: replaced broken images with latest #120

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

TAFKAMW
Copy link

@TAFKAMW TAFKAMW commented Nov 15, 2023

The docker-compose.yml contained references to 3 images with version '4.1.0'. However, those images don't exist (anymore?).

Therefore I updated these 3 references to point to version 'latest'. At this moment this will effectively result in version '4.0.0' being downloaded.

Purpose

Resolves issues:
wso2/docs-apim#7405
https://github.com/wso2/samples-apim/issues/118

Goals

In stead of referencing a non-existent version of 3 images, this fix points to the 'latest' version of these images.

Approach

Replaced 3 entries of '4.1.0' in the docker-compose.yml to version 'latest'. At this moment this will effectively result in version '4.0.0' being downloaded

User stories

Enables https://apim.docs.wso2.com/en/latest/tutorials/scenarios/scenario-overview to works again.

Release note

Fixed broken image(-link)s.

Documentation

https://apim.docs.wso2.com/en/latest/tutorials/scenarios/scenario-overview

Training

https://apim.docs.wso2.com/en/latest/tutorials/scenarios/scenario-overview

Certification

N/A. The docker-compose was simply broken.

Marketing

N/A

Automation tests

N/A

Security checks

Samples

N/A

Related PRs

N/A

Migrations (if applicable)

Just clone latest version.

Test environment

N/A

Learning

https://github.com/wso2/samples-apim/issues/118#issuecomment-1779138182

Fixed issues:
wso2/docs-apim#7405
https://github.com/wso2/samples-apim/issues/118

The docker-compose.yml contained references to 3 images with version '4.1.0'. However, those images don't exist (anymore?).

Therefore I updated these 3 references to point to version 'latest'. At this moment this will effectively result in version '4.0.0' being downloaded.
@CLAassistant
Copy link

CLAassistant commented Nov 15, 2023

CLA assistant check
All committers have signed the CLA.

Copy link

@YBriukhov YBriukhov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I ran into the same situation described and after applying the suggested changes to docker-compose.yml I was able to move forward.
However, the next error (may or may not be related) was: "Error response from daemon: Conflict. The container name "/buildx_buildkit_default" is already in use by container "86c510e92e8c0c80faf8fdbbb97e974245b37aec4d30b6d205103e063b0d864b". You have to remove (or rename) that container to be able to reuse that name."
Full output here apim-tutorial.log

@YBriukhov
Copy link

I ran into the same situation described and after applying the suggested changes to docker-compose.yml I was able to move forward. However, the next error (may or may not be related) was: "Error response from daemon: Conflict. The container name "/buildx_buildkit_default" is already in use by container "86c510e92e8c0c80faf8fdbbb97e974245b37aec4d30b6d205103e063b0d864b". You have to remove (or rename) that container to be able to reuse that name." Full output here apim-tutorial.log

UPD: solved buildx_buildkit_default error by updating docker, which was installed by the snap using this command:
sudo snap refresh docker --channel=latest/edge

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