Skip to content
This repository was archived by the owner on Oct 1, 2026. It is now read-only.

Add container liveness and runtime smoke test - #48

Merged
elkampu merged 2 commits into
KalyteraSystems:mainfrom
iibaranov-IG:ci/container-runtime-smoke
Sep 8, 2026
Merged

elkampu merged 2 commits into
KalyteraSystems:mainfrom
iibaranov-IG:ci/container-runtime-smoke

Conversation

@iibaranov-IG

Copy link
Copy Markdown
Contributor

Summary

  • add a dependency-free local-only GET /healthz endpoint returning an empty 204 response
  • install curl explicitly and declare a Docker health check against loopback
  • load the amd64 PR image and verify healthy startup, non-root UID, and an actual create/delete probe under /data
  • print inspect/log diagnostics on failure and always remove the uniquely named smoke container
  • distinguish liveness from detailed System diagnostics in the README

Verification

  • dotnet test IPCamLapse.sln -c Release: 105 tests pass (58 IPCamLapse + 47 OpenCamInterop)
  • dotnet format IPCamLapse.sln --verify-no-changes --no-restore: pass
  • local docker build: pass
  • live container health/non-root/data create-delete smoke: pass
  • workflow YAML parse and git diff --check: pass
  • signed commit: 1014957

Closes #35.

@elkampu elkampu 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.

Approved at exact head 1014957. I reviewed the local-only liveness contract and workflow permissions, reproduced locked restore, Release build with zero warnings/errors, all 105 tests, formatting verification, and the dependency audit. CI, CodeQL, and the container build/runtime smoke (including non-root and writable-data probes) are green.

@elkampu elkampu 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.

Re-approved at updated exact head 27fe12c after the main merge. The resulting branch contains only the reviewed #48 delta over current main. I reran locked restore, Release build (zero warnings/errors), all 113 combined tests, formatting, and dependency audit; refreshed CI, CodeQL, and the complete container runtime smoke are green.

@elkampu
elkampu merged commit 564e398 into KalyteraSystems:main Sep 8, 2026
5 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add a container health check and runtime smoke test

2 participants