Skip to content

Conversation

@mtodor
Copy link
Collaborator

@mtodor mtodor commented Nov 25, 2025

Description

This PR is adding the following changes:

  • Dockerfile to build MCP server image
  • /health endpoint to support K8s deployment readiness
  • new target in Makefile
  • documentation update in README
  • bump go version to 1.25
  • add dockerfile linter

Validation

  • build image locally
  • run docker container and test with MCP inspector
  • validate that /health endpoint works

Info about layer caching

[1/2] STEP 1/10: FROM registry.access.redhat.com/ubi10/ubi:latest AS builder
[1/2] STEP 2/10: ARG TARGETOS=linux
--> Using cache 9bd8d6b87dc7fe94cf6e9a71a1c5404ab3985d6a4bd3e1920e646d69b9b9daf5
--> 9bd8d6b87dc7
[1/2] STEP 3/10: ARG TARGETARCH=amd64
--> Using cache 08722e7b3772a42b01126f9db37606cb52716dd6efebbed841d804f6cdb1126d
--> 08722e7b3772
[1/2] STEP 4/10: ARG VERSION=dev
--> Using cache 7fd092f50e973c653046241b78d6afcecdac5093a747a7c4ce24f5e509a6ce37
--> 7fd092f50e97
[1/2] STEP 5/10: RUN dnf install -y golang && dnf clean all
--> Using cache 44ca1995aab21363aa88b96ac6e4924ae48126d2bbcc72818520926b8a0b2433
--> 44ca1995aab2
[1/2] STEP 6/10: WORKDIR /workspace
--> Using cache 5b91f34d8e6ce6f53babce4f40afdf19546180bb9cc1b82a2d9f55d6908a786e
--> 5b91f34d8e6c
[1/2] STEP 7/10: COPY go.mod go.sum ./
--> Using cache 6ab9bc41fe53041c20bf836a7adf851b8415614afe28d58c72f397c628680597
--> 6ab9bc41fe53
[1/2] STEP 8/10: RUN go mod download
--> Using cache b2c875fa3b3bf75749bfc4ccb3091dd6784f1700872cfa3d422b5bbfc0d38c54
--> b2c875fa3b3b
[1/2] STEP 9/10: COPY . .
--> 1d085c167014

@mtodor mtodor requested a review from janisz as a code owner November 25, 2025 17:10
@codecov-commenter
Copy link

codecov-commenter commented Nov 25, 2025

❌ 2 Tests Failed:

Tests completed Failed Passed Skipped
158 2 156 0
View the top 2 failed test(s) by shortest run time
github.com/stackrox/stackrox-mcp/internal/testutil::
Stack Traces | 0s run time
Build error
github.com/stackrox/stackrox-mcp/internal/toolsets/mock::
Stack Traces | 0s run time
Build error

To view more test analytics, go to the Test Analytics Dashboard
📋 Got 3 mins? Take this short survey to help us improve Test Analytics.

@mtodor mtodor force-pushed the mtodor/ROX-31485-add-dockerfile branch from c804787 to 853b1f1 Compare November 26, 2025 10:49
@mtodor mtodor changed the title WiP: ROX-31485: Add Dockerfile ROX-31485: Add Dockerfile Nov 26, 2025
@mtodor mtodor force-pushed the mtodor/ROX-31485-add-dockerfile branch 2 times, most recently from 063f7ca to 280e3da Compare November 28, 2025 16:33
@mtodor mtodor requested a review from janisz November 28, 2025 16:34
@mtodor mtodor force-pushed the mtodor/ROX-31485-add-dockerfile branch from 280e3da to 5efa378 Compare December 1, 2025 10:23
@mtodor mtodor merged commit 520e95f into main Dec 2, 2025
2 checks passed
@mtodor mtodor deleted the mtodor/ROX-31485-add-dockerfile branch December 2, 2025 10:06
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.

3 participants