Skip to content

Conversation

l-trotta
Copy link
Contributor

Fixes #5286

@l-trotta l-trotta requested a review from a team as a code owner September 30, 2025 11:08
Copy link
Contributor

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

No changes detected.

You can validate these APIs yourself by using the make validate target.

@l-trotta l-trotta added the skip-backport This pull request should not be backported label Sep 30, 2025
@lcawl
Copy link
Contributor

lcawl commented Oct 1, 2025

I see content like this in the output for the privileges, which is great IMO:

        "x-req-auth": [
          "Index privileges: `create_index`,`index`,`manage`,`read`,`view_index_metadata`\n",
          "Cluster privileges: `manage_ml`\n"
        ],

For the path variations, I see:

        "x-variations": [
          "<div>\n                      <span class=\"operation-verb get\">GET</span>\n                      <span class=\"operation-path\">/_ml/datafeeds</span>\n                      </div>\n                    ",
          "<div>\n                      <span class=\"operation-verb get\">GET</span>\n                      <span class=\"operation-path\">/_ml/datafeeds/{datafeed_id}</span>\n                      </div>\n                    "
        ],

We can check how this renders, worst case might need to remove the div and span and just use simple backticks here too.

@l-trotta l-trotta requested a review from Anaethelion October 2, 2025 08:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
compiler skip-backport This pull request should not be backported
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Store method/path info and authorization info outside of the endpoint description field
2 participants