Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add missing parameter to xpack.info rest-api-spec #118954

Merged
merged 3 commits into from
Jan 8, 2025

Conversation

pquentin
Copy link
Member

I noticed this when reviewing elastic/elasticsearch-specification#3297. The parameter is read here:

// we piggyback verbosity on "human" output
boolean verbose = request.paramAsBoolean("human", true);

By the way, should accept_enterprise be removed in 9.0, as stated in this comment?

@pquentin pquentin added >bug :Core/Infra/REST API REST infrastructure and utilities auto-backport Automatically create backport pull requests when merged v9.0.0 v8.17.1 v8.16.3 labels Dec 18, 2024
@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/es-core-infra (Team:Core/Infra)

@elasticsearchmachine elasticsearchmachine added the Team:Core/Infra Meta label for core/infra team label Dec 18, 2024
@elasticsearchmachine
Copy link
Collaborator

Hi @pquentin, I've created a changelog YAML for you.

@mosche
Copy link
Contributor

mosche commented Jan 8, 2025

By the way, should accept_enterprise be removed in 9.0, as stated in this comment?

Pinging @elastic/es-security regarding accept_enterprise, it's license related...

@pquentin pquentin merged commit ee7848e into elastic:main Jan 8, 2025
16 checks passed
pquentin added a commit to pquentin/elasticsearch that referenced this pull request Jan 8, 2025
* Add missing parameter to xpack.info rest-api-spec

* Update docs/changelog/118954.yaml
pquentin added a commit to pquentin/elasticsearch that referenced this pull request Jan 8, 2025
* Add missing parameter to xpack.info rest-api-spec

* Update docs/changelog/118954.yaml
@elasticsearchmachine
Copy link
Collaborator

💚 Backport successful

Status Branch Result
8.x
8.17

elasticsearchmachine pushed a commit that referenced this pull request Jan 8, 2025
* Add missing parameter to xpack.info rest-api-spec

* Update docs/changelog/118954.yaml
elasticsearchmachine pushed a commit that referenced this pull request Jan 8, 2025
* Add missing parameter to xpack.info rest-api-spec

* Update docs/changelog/118954.yaml
@jakelandis
Copy link
Contributor

@mosche - thanks for the ping. We are tracking this internally with ES-9427 and ES-9748. cc @gwbrown

sarog pushed a commit to sarog/elasticsearch that referenced this pull request Jan 22, 2025
…lastic#119739)

* Add missing parameter to xpack.info rest-api-spec

* Update docs/changelog/118954.yaml
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
auto-backport Automatically create backport pull requests when merged >bug :Core/Infra/REST API REST infrastructure and utilities Team:Core/Infra Meta label for core/infra team v8.17.2 v8.18.0 v9.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants