Skip to content

Conversation

@Baoyuantop
Copy link
Contributor

@Baoyuantop Baoyuantop commented Dec 10, 2025

Description

Which issue(s) this PR fixes:

  1. Due to the issue mentioned in help request: mcp-bridge test failed to execute #12806, we have temporarily skipped the mcp-bridge test in this PR.
  2. The httpbin.org service became unstable, preventing tests from running. All httpbin requests in the tests were replaced with a locally deployed httpbin service.

Checklist

  • I have explained the need for this PR and the problem it solves
  • I have explained the changes or the new features added to this PR
  • I have added tests corresponding to this change
  • I have updated the documentation to reflect this change
  • I have verified that this change is backward compatible (If not, please discuss on the APISIX mailing list first)

@Baoyuantop Baoyuantop marked this pull request as ready for review December 12, 2025 03:07
@dosubot dosubot bot added size:M This PR changes 30-99 lines, ignoring generated files. bug Something isn't working labels Dec 12, 2025
@Baoyuantop Baoyuantop changed the title fix: temporarily skip mcp-bridge test fix: repairing failed CI Dec 12, 2025
- apisix

httpbin:
image: ghcr.io/mccutchen/go-httpbin
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why does kennethreitz/httpbin need to use different image versions?

Copy link
Contributor Author

@Baoyuantop Baoyuantop Dec 12, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This configuration serves the workflow at https://github.com/apache/apisix/blob/master/.github/workflows/push-dev-image-on-commit.yml

This workflow runs on both amd and arm platforms. kennethreitz/httpbin does not provide an arm image, causing it to fail. Therefore, I used this image here.

@Baoyuantop Baoyuantop merged commit 575f217 into apache:master Dec 12, 2025
31 checks passed
@Baoyuantop Baoyuantop deleted the test-skip-mcp branch December 12, 2025 03:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working size:M This PR changes 30-99 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants