feat: containerize linux-sysmonitor health-check script#18
Conversation
gayathrii12
left a comment
There was a problem hiding this comment.
1)Rename dockerfile to Dockerfile, so the usual docker build . command works without extra options.
2)Add README instructions for building and running the container.
3)The health-check script currently has known incompatibilities with top and journalctl inside the container. Since these are part of the health check, please fix them or clearly document the limitation before merge.
4)Please document that the container monitors its own isolated environment, rather than the Docker host. If host monitoring is the goal, the container will need additional mounts/configuration to access host system information.
|
Summary
Containerized the
linux-sysmonitorhealth-check script to provide a consistent and portable execution environment using Docker.Changes
debian:bookworm-slimas the base image and documented the rationale.bash,procps,iputils-ping, andiproute2)./appas the working directory.health-check.shexecutable.health-check.shusingENTRYPOINT.Validation
topoutput parsing andjournalctl) were identified during validation and will be addressed in a follow-up bug fix PR.