Skip to content

feat: containerize linux-sysmonitor health-check script#18

Merged
PuneethM-06 merged 16 commits into
mainfrom
feat/dockerfile
Jul 18, 2026
Merged

feat: containerize linux-sysmonitor health-check script#18
PuneethM-06 merged 16 commits into
mainfrom
feat/dockerfile

Conversation

@PuneethM-06

Copy link
Copy Markdown
Owner

Summary

Containerized the linux-sysmonitor health-check script to provide a consistent and portable execution environment using Docker.

Changes

  • Added a Dockerfile to package the application.
  • Used debian:bookworm-slim as the base image and documented the rationale.
  • Installed the required runtime dependencies (bash, procps, iputils-ping, and iproute2).
  • Configured /app as the working directory.
  • Copied the project into the container image.
  • Made health-check.sh executable.
  • Configured the container to execute health-check.sh using ENTRYPOINT.

Validation

  • Successfully built the Docker image.
  • Successfully started the container using the configured entrypoint.
  • Runtime compatibility issues related to environment-specific command behavior (top output parsing and journalctl) were identified during validation and will be addressed in a follow-up bug fix PR.

@gayathrii12 gayathrii12 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.

@PuneethM-06

Copy link
Copy Markdown
Owner Author

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.

  • This is been tracked by a seprate ticket. Will update once the ticket is resolved

@PuneethM-06
PuneethM-06 merged commit 4baa5fe into main Jul 18, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants