Repository navigation
feat(infrahub-mcp): update to infrahub-mcp v1.2.1 and expose its settings - #110
Open
FragmentedPacket wants to merge 3 commits into
Open
FragmentedPacket wants to merge 3 commits into
FragmentedPacket wants to merge 3 commits into
Conversation
…ings - appVersion v1.1.6 -> v1.2.1, version 0.1.0 -> 0.2.0 - Add values for the settings that v1.2.1 reads and the chart did not set: INFRAHUB_USERNAME/INFRAHUB_PASSWORD, MCP_HOST, INFRAHUB_MCP_TOKEN_PASSTHROUGH_HEADER, INFRAHUB_MCP_AUTH_SCOPES_WRITE, INFRAHUB_MCP_OIDC_* and INFRAHUB_MCP_SCHEMA_EXPAND_PEERS. - Set MCP_PORT from service.targetPort so the server listens on the container port the chart declares. - Correct the mcp.rateLimit.burst description: 0 means twice rps. - Add a production example to the README: token-passthrough, read-only, API token from an existing Secret, ingress on /mcp. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…e chart Carry infrahub-mcp 0.2.0 (image v1.2.1) up the dependency chain: - infrahub: re-pin infrahub-mcp 0.1.0 -> 0.2.0, version 4.35.0 -> 4.36.0 - infrahub-enterprise: re-pin infrahub 4.35.0 -> 4.36.0, version 4.23.0 -> 4.24.0 Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
This was referenced Oct 28, 2026
…mple In token-passthrough mode the server calls Infrahub with each client's token and never reads INFRAHUB_API_TOKEN, so the example no longer creates a Secret for it or sets infrahub.apiToken. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Moves
charts/infrahub-mcpto infrahub-mcp v1.2.1, adds values for the server settings the chart did not set, and re-pins the chart ininfrahubandinfrahub-enterprise.infrahub-mcp chart (0.1.0 → 0.2.0)
appVersion:v1.1.6→v1.2.1. The imageregistry.opsmill.io/opsmill/infrahub-mcp:v1.2.1exists.src/infrahub_mcp/config.py,server.pyand the configuration reference:infrahub.usernameandinfrahub.password(INFRAHUB_USERNAME,INFRAHUB_PASSWORD), with the samevalue/existingSecretpattern asapiTokenmcp.host(MCP_HOST)mcp.tokenPassthroughHeader(INFRAHUB_MCP_TOKEN_PASSTHROUGH_HEADER)mcp.authScopesWrite(INFRAHUB_MCP_AUTH_SCOPES_WRITE)mcp.oidc.*(the sixINFRAHUB_MCP_OIDC_*variables; the client secret can come from an existing Secret)mcp.schemaExpandPeers(INFRAHUB_MCP_SCHEMA_EXPAND_PEERS)MCP_PORTis now set fromservice.targetPort. Before, changingtargetPortchanged the container port but not the port the server listens on.mcp.rateLimit.burstis corrected:0means twicerps, not "disabled".extraEnvcovers them:INFRAHUB_MCP_ENV_FILE: the pod has no.envfile.INFRAHUB_TIMEOUT,INFRAHUB_TLS_*andINFRAHUB_PROXY.README.md.gotmplhas a production example:authMode: token-passthrough,readOnly: true, the in-cluster server URL and an ingress on/mcp. It sets no shared API token: intoken-passthroughmode the server calls Infrahub with each client's token and never readsINFRAHUB_API_TOKEN(server.py, configuration reference). An unused token in the pod would give every client its access ifauthModewere set back tonone.README.mdis regenerated with helm-docs 1.14.2.Re-pin in infrahub (4.35.0 → 4.36.0) and infrahub-enterprise (4.23.0 → 4.24.0)
infrahubpinsinfrahub-mcp0.2.0;infrahub-enterprisepinsinfrahub4.36.0. This follows the last re-pin, 70d0746 (infrahub-backup1.4.0).CI fails on
infrahubandinfrahub-enterpriseuntil 0.2.0 is publishedhelm dependency updatecannot findinfrahub-mcp0.2.0 orinfrahub4.36.0 in the registry, so the lint and end-to-end jobs of those two charts fail on this pull request. The pins stay in this pull request on purpose. With their dependencies pointed at the local charts, both charts passhelm lint.Tests
make lintpasses forinfrahub-mcp,infrahub-backupandinfrahub-observability.helm templateof the README production example renders the expected env vars, Secret reference and ingress.uv run pytest -v -m mcp tests/e2edid not run locally: thevclusterCLI is not installed on the machine used. Thee2e-mcpCI job runs it.Related pull requests
Merge order: #110, then #111, then opsmill/infrahub-mcp#221.
infrahub-mcp-releasedevent after a release🤖 Generated with Claude Code