Skip to content

Conversation

@mdelapenya
Copy link
Member

@mdelapenya mdelapenya commented Jul 31, 2025

What does this PR do?

In the tests, this PR replaces the Brave MCP server with the DuckDuckGo one

Why is it important?

Brave server was killed with this error:
"Can't start brave: context deadline exceeded"

making the testable examples to fail, as the tools retrieved by the MCP client were not detected.

Follow-ups

We probably need to ivestigate why the Brave MCP server is killed 🤔

Brave server was killed with this error:
"Can't start brave: context deadline exceeded"
@mdelapenya mdelapenya requested a review from a team as a code owner July 31, 2025 11:16
@mdelapenya mdelapenya added the chore Changes that do not impact the existing functionality label Jul 31, 2025
@mdelapenya mdelapenya self-assigned this Jul 31, 2025
@mdelapenya mdelapenya added the chore Changes that do not impact the existing functionality label Jul 31, 2025
@netlify
Copy link

netlify bot commented Jul 31, 2025

Deploy Preview for testcontainers-go ready!

Name Link
🔨 Latest commit 72966a7
🔍 Latest deploy log https://app.netlify.com/projects/testcontainers-go/deploys/688b5083306d8c0008da034d
😎 Deploy Preview https://deploy-preview-3247--testcontainers-go.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.

@mdelapenya mdelapenya merged commit c100f88 into testcontainers:main Jul 31, 2025
16 checks passed
@mdelapenya mdelapenya deleted the mcp-fix branch July 31, 2025 11:32
mdelapenya added a commit that referenced this pull request Jul 31, 2025
…util/v4-4.25.6

* main:
  fix(dockermcpgateway): use duckduckgo instead of brave (#3247)
  feat: add Solace pubsub+ module (#3230)
  feat(options): add WithProvider (#3241)
  chore(deps): bump github/codeql-action from 3.29.2 to 3.29.3 (#3237)
  chore(deps): bump golang.org/x/oauth2 in /modules/weaviate (#3240)
  chore(deps): bump mkdocs-include-markdown-plugin from 7.1.5 to 7.1.6 (#3239)
  chore(deps): bump requests from 2.32.0 to 2.32.4 (#3204)
  feat(mcpgateway): add MCP gateway module (#3232)
  chore(deps): bump golang.org/x/oauth2 in /modules/pulsar (#3236)
  chore(deps): bump golang.org/x/oauth2 in /modules/gcloud (#3235)
  chore(deps): bump golang.org/x/oauth2 in /modules/k3s (#3234)
  chore: prepare for next minor development cycle (0.39.0)
  chore: use new version (v0.38.0) in modules and examples
  Update go.mod in azure module (#3231)
  fix: strip headers from logs using log stream specification  (#3226)
  chore: clarify image auth warning message for public images (#3228)
  chore(deps): bump github.com/go-viper/mapstructure/v2 (#3219)
  chore(deps): bump github/codeql-action from 3.28.16 to 3.29.2 (#3222)
  chore(deps): bump mkdocs-include-markdown-plugin from 7.1.5 to 7.1.6 (#3225)
mdelapenya added a commit to knqyf263/testcontainers-go that referenced this pull request Sep 5, 2025
* main: (33 commits)
  feat(registry): add helper functions to pull and tag images (testcontainers#3275)
  fix(reaper): remove termSignal override (testcontainers#3261)
  chore(deps): bump ryuk to v0.13.0, which uses scratch as base image (testcontainers#3274)
  chore(release): refine release script to update inter-module dependencies (testcontainers#3273)
  fix(registry): update `WithHtpasswd` to use `os.CreateTemp` instead of `os.Create` with `filepath.Join`. (testcontainers#3272)
  chore(deps): bump github.com/docker/docker from 28.2.2+incompatible to 28.3.3+incompatible (testcontainers#3270)
  chore(postgres): use require.NotNil instead of assert.NotNil (testcontainers#3252)
  fix(nats): use wait for listening port instead of wait for log (testcontainers#3256)
  chore(deps): bump github.com/go-viper/mapstructure/v2 (testcontainers#3267)
  fix(postgres): snapshot restore (testcontainers#3264)
  fix(dockermcpgateway): use duckduckgo instead of brave (testcontainers#3247)
  feat: add Solace pubsub+ module (testcontainers#3230)
  feat(options): add WithProvider (testcontainers#3241)
  chore(deps): bump github/codeql-action from 3.29.2 to 3.29.3 (testcontainers#3237)
  chore(deps): bump golang.org/x/oauth2 in /modules/weaviate (testcontainers#3240)
  chore(deps): bump mkdocs-include-markdown-plugin from 7.1.5 to 7.1.6 (testcontainers#3239)
  chore(deps): bump requests from 2.32.0 to 2.32.4 (testcontainers#3204)
  feat(mcpgateway): add MCP gateway module (testcontainers#3232)
  chore(deps): bump golang.org/x/oauth2 in /modules/pulsar (testcontainers#3236)
  chore(deps): bump golang.org/x/oauth2 in /modules/gcloud (testcontainers#3235)
  ...
mdelapenya added a commit to mdelapenya/testcontainers-go that referenced this pull request Sep 5, 2025
* main: (30 commits)
  fix: preserve unix socket schema in testcontainers host from properties (testcontainers#3213)
  feat(registry): add helper functions to pull and tag images (testcontainers#3275)
  fix(reaper): remove termSignal override (testcontainers#3261)
  chore(deps): bump ryuk to v0.13.0, which uses scratch as base image (testcontainers#3274)
  chore(release): refine release script to update inter-module dependencies (testcontainers#3273)
  fix(registry): update `WithHtpasswd` to use `os.CreateTemp` instead of `os.Create` with `filepath.Join`. (testcontainers#3272)
  chore(deps): bump github.com/docker/docker from 28.2.2+incompatible to 28.3.3+incompatible (testcontainers#3270)
  chore(postgres): use require.NotNil instead of assert.NotNil (testcontainers#3252)
  fix(nats): use wait for listening port instead of wait for log (testcontainers#3256)
  chore(deps): bump github.com/go-viper/mapstructure/v2 (testcontainers#3267)
  fix(postgres): snapshot restore (testcontainers#3264)
  fix(dockermcpgateway): use duckduckgo instead of brave (testcontainers#3247)
  feat: add Solace pubsub+ module (testcontainers#3230)
  feat(options): add WithProvider (testcontainers#3241)
  chore(deps): bump github/codeql-action from 3.29.2 to 3.29.3 (testcontainers#3237)
  chore(deps): bump golang.org/x/oauth2 in /modules/weaviate (testcontainers#3240)
  chore(deps): bump mkdocs-include-markdown-plugin from 7.1.5 to 7.1.6 (testcontainers#3239)
  chore(deps): bump requests from 2.32.0 to 2.32.4 (testcontainers#3204)
  feat(mcpgateway): add MCP gateway module (testcontainers#3232)
  chore(deps): bump golang.org/x/oauth2 in /modules/pulsar (testcontainers#3236)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

chore Changes that do not impact the existing functionality

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant