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 warning about anonymous volume #83

Closed
wants to merge 1 commit into from
Closed

Add warning about anonymous volume #83

wants to merge 1 commit into from

Conversation

cskwrd
Copy link

@cskwrd cskwrd commented Jan 5, 2021

The Dockerfile for the image uses an anonymous volume to persist the transformed config. This presents a problem in situations where a compose file is used. Volumes are not destroyed when changes are made to the compose file. This is in turn results in stale configuration. Additionally, it is rather difficult to debug when the configuration doesn't update as expected.

The `Dockerfile` for the image uses an anonymous volume to persist the transformed config. This presents a problem in situations where a compose file is used. Volumes are not destroyed when changes are made to the compose file. This is in turn results in stale configuration. Additionally, it is rather difficult to debug when the configuration doesn't update as expected.
@BertrandGouny
Copy link
Member

Thanks :)

@maxswjeon
Copy link

Please add warning to docker-compose users.

docker-compose Guide

When using docker-compose, docker-compose up -d --force-recreate does not reset anonymous volume. Run docker-compose stop [service name] and docker-compose rm [service name] and then you should run docker-compose up -d.

@Oisins

This comment has been minimized.

@cskwrd
Copy link
Author

cskwrd commented Jan 6, 2022

Is something more required for this to be merged?

@cskwrd cskwrd closed this by deleting the head repository May 22, 2023
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.

4 participants