-
Notifications
You must be signed in to change notification settings - Fork 317
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
[ECS] [request]: Take Docker image HEALTHCHECK into account when available #1041
Comments
@mwarkentin It is only supported if you define the healthcheck in the container definition. This feature request is about supporting it without repeating the healthcheck definition that Docker already knows about. |
Ah, thanks - didn't realize that. |
Not an issue. See #1443 (comment) I think the "right" thing to do is for users to remove HEALTHCHECKs from their Dockerfiles and move them to docker-compose and ECS task defs as needed: https://nickjanetakis.com/blog/docker-tip-85-define-healthcheck-in-your-docker-compose-file The Dockerfile HEALTHCHECK (which was added for Swarm) is automatically disabled in Kubernetes since v1.8: AWS added the https://docs.aws.amazon.com/AmazonECS/latest/developerguide/ecs-agent-config.html I suggest removing HEALTHCHECKs from Dockerfiles. Failing that, add |
This issue is more than a year old. Will it ever be tackled? |
@joebowbeer I know that the issue is old but I did not follow your response. ECS supposedly supports it by default but doesn't actually show it on the interface so it seems to me it is a bug, or am I missing a lot of somethings? 😄 I think it is really a useful tool and if your deployment needs a different approach you can override the command by the underlying platform (ECS, K8s, etc). |
@npassaro I'm saying that this is a misfeature that does not have legs. It doesn't make sense for ECS to improve support for a feature that is for all practical purposes deprecated. That's my view considering EKS and Kubernetes in addition to ECS. I assume that containerd and other CRI runtimes ignore this healthcheck as well. According to one respondent, this assumption is correct. |
Community Note
Tell us about your request
Use the
HEALTHCHECK
defined in the Docker image instead of forcing us to repeat ourself and add the same information to the task definition once again.Which service(s) is this request for?
Fargate, ECS
Tell us about the problem you're trying to solve. What are you trying to do, and why is it hard?
In my Dockerfile, I have configured a
HEALTHCHECK
. Unfortunately, ECS doesn't pick up this information. I see that the healtchecks are performed in the container logs but ECS does not seem to fetch this information. It would be so much easier if ECS would just use the information that is already available.Are you currently working around this issue?
I just don't use healthchecks. Configuring them in CloudFormation for 3 containers with parameters is just crazy (15 parameters needed out of 60 available...).
The text was updated successfully, but these errors were encountered: