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.

Sign up for GitHub

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

Jump to bottom

Question: After using the latest image - 2 Nov 2023 - health check returns unhealthy. #1078

Closed
leonid-deriv opened this issue Nov 2, 2023 · 2 comments · Fixed by #1079
Closed

Question: After using the latest image - 2 Nov 2023 - health check returns unhealthy. #1078

leonid-deriv opened this issue Nov 2, 2023 · 2 comments · Fixed by #1079

Comments

@leonid-deriv
Copy link

Anything changed which can affect this?
morucci added a commit to morucci/monocle that referenced this issue Nov 2, 2023
@morucci
docker - healthcheck assumes /bin/sh
9b57ce7 
Here is the command executed in the container context:

```
/bin/sh -c curl --silent --fail localhost:9001/health || exit 1
```

Since the move to UBI 8 image, sh is available under /usr/bin/sh.
This change ensures a link from /bin/sh to /usr/bin/sh.

Attempt to fix change-metrics#1078
@morucci morucci mentioned this issue Nov 2, 2023
Merged
morucci added a commit to morucci/monocle that referenced this issue Nov 2, 2023
@morucci
docker - healthcheck assumes /bin/sh
a85f188 
Here is the command executed in the container context:

```
/bin/sh -c curl --silent --fail localhost:9001/health || exit 1
```

Since the move to UBI 8 image, sh is available under /usr/bin/sh.
This change ensures a link from /bin/sh to /usr/bin/sh.

Attempt to fix change-metrics#1078
@morucci
Copy link
Collaborator

morucci commented Nov 2, 2023

related #1080

@morucci
Copy link
Collaborator

morucci commented Nov 2, 2023

This is due to a recent change how the container image was built #1069

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

docker - healthcheck assumes /bin/sh morucci/monocle
2 participants
@morucci @leonid-deriv