Skip to content

Comments

[CI] Skip Responses API#34990

Open
robertgshaw2-redhat wants to merge 1 commit intomainfrom
remove-responses-api-tests
Open

[CI] Skip Responses API#34990
robertgshaw2-redhat wants to merge 1 commit intomainfrom
remove-responses-api-tests

Conversation

@robertgshaw2-redhat
Copy link
Collaborator

@robertgshaw2-redhat robertgshaw2-redhat commented Feb 20, 2026

Purpose

  • skip responses API tests, which are flakly.
  • it needs further investigation but this will help to stabilize the CI

Signed-off-by: Robert Shaw <robshaw@redhat.com>
@dosubot
Copy link

dosubot bot commented Feb 20, 2026

Related Documentation

Checked 0 published document(s) in 1 knowledge base(s). No updates required.

How did I do? Any feedback?  Join Discord

Copy link
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

The pull request correctly addresses the flakiness in the CI by skipping the test_mcp_code_interpreter_streaming test in tests/entrypoints/openai/responses/test_harmony.py. The skip reason provided is clear and points to the need for further investigation into the code interpreter MCP implementation. This change is an appropriate short-term measure to ensure CI reliability while the underlying issues are being addressed.

Copy link
Collaborator

@LucasWilkinson LucasWilkinson left a comment

Choose a reason for hiding this comment

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

Thank you!!!!

@AndreasKaratzas
Copy link
Collaborator

@robertgshaw2-redhat @LucasWilkinson Can we merge #33949?
There is a v1 e2e failure but that is not correlated with that PR.

@mgoin mgoin added ready ONLY add when PR is ready to merge/full CI is needed ci-failure Issue about an unexpected test failure in CI labels Feb 20, 2026
@mgoin mgoin enabled auto-merge (squash) February 20, 2026 23:07
@AndreasKaratzas
Copy link
Collaborator

#33949 just got merged :) So I suggest that we give it a chance and if it does not solve our problems, revisit skipping these tests.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci-failure Issue about an unexpected test failure in CI ready ONLY add when PR is ready to merge/full CI is needed

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

6 participants