Skip to content

Conversation

@Jan-Kazlouski-elastic
Copy link
Contributor

@Jan-Kazlouski-elastic Jan-Kazlouski-elastic commented Nov 12, 2025

This PR adds changes to specification caused by elastic/elasticsearch#136624

Corresponding PR for elasticsearch repo:
elastic/elasticsearch#138025

Additional actions

  • Signed the CLA
  • Executed make contrib

@github-actions
Copy link
Contributor

github-actions bot commented Nov 12, 2025

Following you can find the validation changes against the target branch for the API.

API Status Request Response
inference.put_openshift_ai ➕ ⚪ Missing test Missing test

You can validate this API yourself by using the make validate target.

@Jan-Kazlouski-elastic Jan-Kazlouski-elastic changed the title Openshift ai integration OpenShift AI integration Nov 12, 2025
@Jan-Kazlouski-elastic Jan-Kazlouski-elastic changed the title OpenShift AI integration Add OpenShift AI integration specifications Nov 12, 2025
inference-api-put-llama,https://www.elastic.co/docs/api/doc/elasticsearch/operation/operation-inference-put-llama,,
inference-api-put-mistral,https://www.elastic.co/docs/api/doc/elasticsearch/operation/operation-inference-put-mistral,https://www.elastic.co/guide/en/elasticsearch/reference/8.18/infer-service-mistral.html,
inference-api-put-openai,https://www.elastic.co/docs/api/doc/elasticsearch/operation/operation-inference-put-openai,https://www.elastic.co/guide/en/elasticsearch/reference/8.18/infer-service-openai.html,
inference-api-put-openshift-ai,https://www.elastic.co/docs/api/doc/elasticsearch/operation/operation-inference-put-openshift-ai,,
Copy link
Member

Choose a reason for hiding this comment

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

This URL does not exist yet, which will cause some clients' docs builds to fail. Will this page be available soon?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I believe this will be added by merging of elastic/elasticsearch#138025 PR.
And in order to merge - overall changes to specification must be reviewed. Any change to mentioned PR - will result in changes to this PR.

Please correct me if my understanding is not valid.

Copy link
Member

Choose a reason for hiding this comment

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

You're correct! Just making sure any broken URLs are temporary.

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

Labels

ml skip-backport This pull request should not be backported specification Team:ML

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants