Skip to content

chore: [DevOps] Fix e2e tests#948

Open
CharlesDuboisSAP wants to merge 5 commits into
mainfrom
e2e
Open

chore: [DevOps] Fix e2e tests#948
CharlesDuboisSAP wants to merge 5 commits into
mainfrom
e2e

Conversation

@CharlesDuboisSAP

Copy link
Copy Markdown
Contributor

Context

AI/ai-sdk-java-backlog#448.

Fix e2e tests

Feature scope:

  • Added OrchestrationAiModel.GEMINI_3_1_PRO_PREVIEW_EA
  • Fixed some OrchestrationClientException that were reported as OrchestrationFilterException.Input.
  • Updated error message and made them more resilient to changes

Definition of Done

  • Functionality scope stated & covered
  • Tests cover the scope above
  • Error handling created / updated & covered by the tests above
  • Aligned changes with the JavaScript SDK
  • Documentation updated
  • Release notes updated

@CharlesDuboisSAP CharlesDuboisSAP self-assigned this Jul 15, 2026
@CharlesDuboisSAP CharlesDuboisSAP added the please-review Request to review a pull-request label Jul 15, 2026
Jonas-Isr
Jonas-Isr previously approved these changes Jul 15, 2026

@Jonas-Isr Jonas-Isr left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Only very minor comments.

Comment thread docs/release_notes.md Outdated
vladimir-a-sap
vladimir-a-sap previously approved these changes Jul 15, 2026

@vladimir-a-sap vladimir-a-sap left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM; I wonder if we should agree on a structured exception format with AI-Core/Orchestration for the longer term to avoid heuristic-based error string resolution

@CharlesDuboisSAP

Copy link
Copy Markdown
Contributor Author

LGTM; I wonder if we should agree on a structured exception format with AI-Core/Orchestration for the longer term to avoid heuristic-based error string resolution

Unfortunately we already tried and the orchestration team doesn't do teamwork, they don't care about accomodating their users

@CharlesDuboisSAP CharlesDuboisSAP enabled auto-merge (squash) July 15, 2026 13:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

please-review Request to review a pull-request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants