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

Use a - as a separator to fix issues with already running detection #990

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

Conversation

vb-linetco
Copy link

@vb-linetco vb-linetco commented Aug 22, 2024

Summary

Fix an issue where services are wrongly restarted as a corrective measure, due to images built with docker compose not detected as running.

Additional Context

Add any additional context about the problem here.

  • Root cause and the steps to reproduce. (If applicable)
  • Thought process behind the implementation.

Related Issues (if any)

Fixes #955, for more details see the issue, where this fix already is proposed.

Checklist

  • Manually verified. (For example puppet apply)

@vb-linetco vb-linetco requested a review from a team as a code owner August 22, 2024 06:35
@CLAassistant
Copy link

CLAassistant commented Aug 22, 2024

CLA assistant check
All committers have signed the CLA.

@vb-linetco vb-linetco force-pushed the 955-fix-build-image-name-detection branch from 6d6943e to d724ee1 Compare September 18, 2024 13:33
@judzk
Copy link

judzk commented Oct 10, 2024

Hi, can someone check on this ? i really need this bug fixed :)
Regards !

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.

Using Compose with build statement a container is not detected as running and recreated with each run
3 participants