Skip to content

helm: update AWS MCP addon for native Streamable HTTP (2.1.0) - #2222

Merged
moshemorad merged 2 commits into
masterfrom
claude/holmesgpt-aws-mcp-2-1-0-ilz001
Jun 24, 2026
Merged

moshemorad merged 2 commits into
masterfrom
claude/holmesgpt-aws-mcp-2-1-0-ilz001

Conversation

@moshemorad

@moshemorad moshemorad commented Jun 23, 2026 •

Copy link
Copy Markdown
Collaborator

Summary

The built-in AWS MCP addon images (robusta-dev/holmes-mcp-integrations#30, merged) were migrated off supergateway/SSE to run awslabs.aws-api-mcp-server's native MCP Streamable HTTP transport, released as tag 2.1.0. The native server serves only at /mcp. The chart still pointed at the old 2.0.1 image and generated an MCP URL with no /mcp path and no transport mode, so Holmes' streamable-http client hit :8000 root and got 404/406 — it couldn't connect.

Changes

  • helm/holmes/templates/toolset-config.yaml — in the aws_api MCP server config, append /mcp to the url and add "mode" "streamable-http".
  • helm/holmes/values.yaml — bump both image tags under mcpAddons.aws: aws-api-mcp-server and multiAccount.image (multi-aws-api-mcp-server) from 2.0.1 → 2.1.0.

The tcpSocket health probes, ServiceAccount/IRSA, and READ_OPERATIONS_ONLY (read-only mode) env are left unchanged.

Verification

$ helm template r ./helm/holmes --set mcpAddons.aws.enabled=true | grep -A5 aws_api
      aws_api:
        config:
          icon_url: https://raw.githubusercontent.com/gilbarbara/logos/.../aws.svg
          mode: streamable-http
          url: http://r-aws-mcp-server.default.svc.cluster.local:8000/mcp
        description: AWS API MCP Server - comprehensive AWS service access...

Rendered URL ends in /mcp, mode: streamable-http is present, and the rendered image tag is aws-api-mcp-server:2.1.0.

🤖 Generated with Claude Code

https://claude.ai/code/session_01P2aD44gnGDWY68YNzpCkuq


Generated by Claude Code

Summary by CodeRabbit

  • Chores
    • Upgraded AWS MCP container images to version 2.1.0 for enhanced reliability
    • Updated AWS MCP server configuration to use optimized HTTP streaming mode

The AWS MCP images were migrated off supergateway/SSE to the native
awslabs.aws-api-mcp-server Streamable HTTP transport (tag 2.1.0), which
serves only at /mcp. Update the addon to match:

- toolset-config.yaml: append /mcp to the aws_api MCP url and set
  "mode" "streamable-http" so Holmes' streamable-http client connects
  to the correct endpoint instead of hitting :8000 root (404/406).
- values.yaml: bump aws-api-mcp-server and multi-aws-api-mcp-server
  image tags 2.0.1 -> 2.1.0.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01P2aD44gnGDWY68YNzpCkuq
Signed-off-by: Claude <noreply@anthropic.com>
@netlify

netlify Bot commented Jun 23, 2026 •

Copy link
Copy Markdown

✅ Deploy Preview for holmes-docs ready!

Name Link
🔨 Latest commit e18ab9a
🔍 Latest deploy log https://app.netlify.com/projects/holmes-docs/deploys/6a3bcb51fa707c0008edf3a8
😎 Deploy Preview https://deploy-preview-2222--holmes-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@coderabbitai

coderabbitai Bot commented Jun 23, 2026 •

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 2a063588-6739-40d8-8fbc-8b8a06e8ad3d

📥 Commits

Reviewing files that changed from the base of the PR and between b046938 and f96a3a0.

📒 Files selected for processing (2)
  • helm/holmes/templates/toolset-config.yaml
  • helm/holmes/values.yaml

Walkthrough

The AWS MCP addon toolset configuration in the Helm chart is updated to append /mcp to the server URL and set mode: streamable-http. Both aws-api-mcp-server and multi-aws-api-mcp-server container image tags are bumped from 2.0.1 to 2.1.0.

Changes

AWS MCP Server Update

Layer / File(s) Summary
AWS MCP endpoint, transport mode, and image bumps
helm/holmes/templates/toolset-config.yaml, helm/holmes/values.yaml
Server url is updated to include the /mcp path and mode is set to streamable-http; both aws-api-mcp-server and multi-aws-api-mcp-server image tags are bumped from 2.0.1 to 2.1.0.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

  • HolmesGPT/holmesgpt#1318: Adds Azure MCP using the same /mcp + streamable-http pattern and refactors the AWS MCP block to include url/mode fields — directly related to the same config structure being modified here.
  • HolmesGPT/holmesgpt#1541: Also bumps aws-api-mcp-server image versions in helm/holmes/values.yaml, overlapping with the same Helm values image declarations.
  • HolmesGPT/holmesgpt#1335: Modifies the same AWS MCP server config in helm/holmes/templates/toolset-config.yaml, specifically removing the streamable-http mode — the inverse of what this PR does.

Suggested reviewers

  • RoiGlinik
  • Avi-Robusta
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately summarizes the main change: updating the AWS MCP addon Helm configuration to support native Streamable HTTP transport in version 2.1.0, which directly matches the changes made to the toolset config URL/mode and image tag updates.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@github-actions

github-actions Bot commented Jun 23, 2026 •

Copy link
Copy Markdown
Contributor

✅ Docker images ready for e8a762e29 (built in 6m 29s)

⚠️ Warning: does not support ARM (ARM images are built on release only - not on every PR)

Use these tags to pull the images for testing.

📋 Copy commands

⚠️ Temporary images are deleted after 30 days. Copy to a permanent registry before using them:

gcloud auth configure-docker us-central1-docker.pkg.dev
docker pull us-central1-docker.pkg.dev/robusta-development/temporary-builds/holmes:e8a762e29
docker tag us-central1-docker.pkg.dev/robusta-development/temporary-builds/holmes:e8a762e29 me-west1-docker.pkg.dev/robusta-development/development/holmes-dev:e8a762e29
docker push me-west1-docker.pkg.dev/robusta-development/development/holmes-dev:e8a762e29
docker pull us-central1-docker.pkg.dev/robusta-development/temporary-builds/holmes-operator:e8a762e29
docker tag us-central1-docker.pkg.dev/robusta-development/temporary-builds/holmes-operator:e8a762e29 me-west1-docker.pkg.dev/robusta-development/development/holmes-operator-dev:e8a762e29
docker push me-west1-docker.pkg.dev/robusta-development/development/holmes-operator-dev:e8a762e29

Patch Helm values in one line (choose the chart you use):

HolmesGPT chart:

helm upgrade --install holmesgpt ./helm/holmes \
  --set registry=me-west1-docker.pkg.dev/robusta-development/development \
  --set image=holmes-dev:e8a762e29 \
  --set operator.registry=me-west1-docker.pkg.dev/robusta-development/development \
  --set operator.image=holmes-operator-dev:e8a762e29

Robusta wrapper chart:

helm upgrade --install robusta robusta/robusta \
  --reuse-values \
  --set holmes.registry=me-west1-docker.pkg.dev/robusta-development/development \
  --set holmes.image=holmes-dev:e8a762e29 \
  --set holmes.operator.registry=me-west1-docker.pkg.dev/robusta-development/development \
  --set holmes.operator.image=holmes-operator-dev:e8a762e29

@moshemorad
moshemorad enabled auto-merge (squash) June 24, 2026 12:19
@moshemorad
moshemorad merged commit b79f2dd into master Jun 24, 2026
14 of 17 checks passed
@moshemorad
moshemorad deleted the claude/holmesgpt-aws-mcp-2-1-0-ilz001 branch June 24, 2026 12:23
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