Skip to content

Use correct provider name for OpenAI observations#5925

Merged
sdeleuze merged 1 commit intospring-projects:mainfrom
ThomasVitale:gh-5924
May 4, 2026
Merged

Use correct provider name for OpenAI observations#5925
sdeleuze merged 1 commit intospring-projects:mainfrom
ThomasVitale:gh-5924

Conversation

@ThomasVitale
Copy link
Copy Markdown
Contributor

Use 'openai' as the provider name for the Spring AI OpenAI observations instead of 'openai_sdk'.

Fixes gh-5924.

Use 'openai' as the provider name for the Spring AI OpenAI observations instead of 'openai_sdk'.

Fixes spring-projectsgh-5924.

Signed-off-by: Thomas Vitale <ThomasVitale@users.noreply.github.com>
/**
* AI system provided by Azure.
*/
AZURE_OPENAI("azure-openai"),
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Not used anymore after the unification under the Spring AI OpenAI module.

/**
* AI system provided by Oracle OCI.
*/
OCI_GENAI("oci_genai"),
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

The OCI support has been removed from the Spring AI project, so this is not used anymore.

/**
* AI system provided by the official OpenAI SDK.
*/
OPENAI_SDK("openai_sdk"),
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Not used anymore after the unification under the Spring AI OpenAI module.

@ThomasVitale
Copy link
Copy Markdown
Contributor Author

The failures in the MCP Integration Tests seem to be related to a flaky WebFluxStreamableIT.testRootsServerCloseWithActiveSubscription test. Not related to the code changes introduced in this PR.

@sdeleuze sdeleuze self-assigned this May 3, 2026
@sdeleuze sdeleuze added this to the 2.0.0-M6 milestone May 3, 2026
@sdeleuze sdeleuze merged commit bed76df into spring-projects:main May 4, 2026
5 of 6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Observations for Spring AI OpenAI are using the wrong provider attribute value

2 participants