-
Notifications
You must be signed in to change notification settings - Fork 311
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
Using Compose with build statement a container is not detected as running and recreated with each run #955
Comments
If you need more information please let me know. |
I think you are using a different module. There is no |
Yes you were right, we had another wrapper around docker_compose, I did not notice that, I've updated the setup |
Puppet 5 is EOL, and v3.10.2 of this module is very old. |
As i stated, a more recent version was behaving the same. |
And the changed line that fixes the issue is the same for all affected version. |
I'm not sure whether the issue is a bug or just requires an enhancement for supporting docker-compose > 2.8. One could check the version of docker-compose and use |
Sounds like this is a duplicate of #891. |
You are probably right on this, i did not notice that |
I still encounter this bug with |
Describe the Bug
Using
docker_compose
together with adocker-compose.yaml
with a custom build image like for example:And having docker-compose >= 2.8.0 installed, one can observe the following issue.
Puppet does not detect the docker-container as already running and tries to recreates it with every puppet run.
The following message can be observed in the logs of puppet with each run
Expected Behavior
The container is not attempted to be created as a corrective measure with each
puppet agent
run when it is already running.Steps to Reproduce
Steps to reproduce the behavior:
Environment
Additional Context
During further analysis I was able to track the issue down to following changes:
_
to-
with this commit: Docker compose_
not being a valid character for DNS-names.Workaround:
puppetlabs-docker/lib/puppet/provider/docker_compose/ruby.rb
Line 71 in 65e36c1
-
, the issue cannot be observed anymore.The text was updated successfully, but these errors were encountered: