fix: correct infrahub-mcp Dockerfile and README #7

Merged
Damien merged 2 commits from fix/infrahub-mcp-dockerfile into main 2026-03-15 10:28:04 +00:00
Owner

What changed

Fixes the build failure caused by wrong upstream repo URL and missing default branch.

Dockerfile

  • Repo URL: opsmill/infrahub-mcp (was opsmill/infrahub-mcp-server)
  • Default branch: clone without --branch when INFRAHUB_MCP_VERSION is unset, so the upstream default branch is used automatically
  • Secrets warning: removed ENV INFRAHUB_API_TOKEN="" — the token should be passed at runtime via -e, not baked into the image
  • Comments: rewritten in English

README

  • Fixed upstream repo link
  • Clarified that INFRAHUB_API_TOKEN is required at runtime
## What changed Fixes the build failure caused by wrong upstream repo URL and missing default branch. ### Dockerfile - **Repo URL**: `opsmill/infrahub-mcp` (was `opsmill/infrahub-mcp-server`) - **Default branch**: clone without `--branch` when `INFRAHUB_MCP_VERSION` is unset, so the upstream default branch is used automatically - **Secrets warning**: removed `ENV INFRAHUB_API_TOKEN=""` — the token should be passed at runtime via `-e`, not baked into the image - **Comments**: rewritten in English ### README - Fixed upstream repo link - Clarified that `INFRAHUB_API_TOKEN` is required at runtime
Damien added 2 commits 2026-03-15 10:27:20 +00:00
- Fix repo URL: opsmill/infrahub-mcp (not infrahub-mcp-server)
- Clone default branch when INFRAHUB_MCP_VERSION is not set
- Remove INFRAHUB_API_TOKEN from ENV to avoid Docker secrets warning
- Rewrite comments in English
Damien merged commit 755423cf4e into main 2026-03-15 10:28:04 +00:00
Damien deleted branch fix/infrahub-mcp-dockerfile 2026-03-15 10:28:05 +00:00
Sign in to join this conversation.
No Reviewers
No Label
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: Damien/docker-images#7