Skip to content

ci: Add Docker image runtime validation - #17

Merged
yashdotdev13 merged 2 commits into
mainfrom
ci/10-docker-image-validation
Oct 12, 2026
Merged

yashdotdev13 merged 2 commits into
mainfrom
ci/10-docker-image-validation

Conversation

@yashdotdev13

Copy link
Copy Markdown
Owner

Summary

Closes #10

This PR extends the existing GitHub Actions workflow to validate that the built Docker image starts successfully and responds to the application's health endpoint.

Changes

  • Build the Docker image as part of the existing CI workflow.
  • Start a container using the image built during CI.
  • Configure the container with the required environment variables.
  • Poll /actuator/health until the application becomes available or the timeout is reached.
  • Print container logs if the application exits unexpectedly or fails to become healthy.
  • Automatically remove the test container on success or failure.
  • Preserve the existing Maven verification and Redis integration-test setup.

Validation

The CI workflow will verify that:

  • Maven verification completes successfully.
  • The Docker image builds successfully.
  • The application starts inside the container.
  • The Actuator health endpoint responds successfully.
  • The test container is cleaned up after validation.

No AWS credentials, deployment secrets, or deployment steps are introduced.

Scope

This change adds Docker runtime smoke testing to CI. It does not deploy the application or replace the existing automated test suite.

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Oct 12, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-10-12T10:06:30.360518Z 783fd5a PR opened
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 783fd5a905

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread .github/workflows/ci.yml Outdated
--name "$CONTAINER" \
-p 127.0.0.1::8081 \
-e SERVER_PORT=8081 \
-e REDIS_HOST=127.0.0.1 \

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P1 Badge Connect the runtime container to the CI Redis service

127.0.0.1 here is the application container itself, not the runner where the Actions Redis service port is published; GitHub documents that this mapping is accessible from the runner host. Startup unconditionally calls GlobalCapacityBootstrap.run(), whose RedisLeaseCoordinator.registerCapacity() performs a Redis operation, so the container exits with a Redis connection failure and this new validation fails on every hosted-runner CI run. Attach the container to the service network or provide an address that reaches the runner-published Redis port.

Useful? React with 👍 / 👎.

@sonarqubecloud

Copy link
Copy Markdown

@yashdotdev13
yashdotdev13 merged commit 1d16a12 into main Oct 12, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add CI Docker image validation

1 participant