Skip to content

added list of valid actions #127108

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

added list of valid actions #127108

wants to merge 1 commit into from

Conversation

AdalineL
Copy link

@AdalineL AdalineL commented Apr 21, 2025 •
edited
Loading

This commit improves current error messages by including a list of valid actions.

Previously, error messages only indicated that an action was not found. Now, users will be able to see the list of valid actions.

Fixes #75441

@cla-checker-service CLA Checker Service
Copy link

❌ Author of the following commits did not sign a Contributor Agreement:
1b9592e

Please, read and sign the above mentioned agreement if you want to contribute to this project

@elasticsearchmachine elasticsearchmachine added needs:triage Requires assignment of a team area label v9.1.0 external-contributor Pull request authored by a developer outside the Elasticsearch team labels Apr 21, 2025
@gbanasiak gbanasiak added the :Core/Infra/Core Core issues without another label label Apr 28, 2025
@elasticsearchmachine elasticsearchmachine added the Team:Core/Infra Meta label for core/infra team label Apr 28, 2025
@elasticsearchmachine
Copy link
Collaborator

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

@elasticsearchmachine elasticsearchmachine removed the needs:triage Requires assignment of a team area label label Apr 28, 2025
rjernst
rjernst reviewed Apr 30, 2025
View reviewed changes
Copy link
Member

@rjernst rjernst left a comment

Choose a reason for hiding this comment

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

The available actions are going to be enormous, several hundred long. It's not really going to be readable.

Did you experience an issue that it was difficult to figure out why an action was missing?

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

@rjernst rjernst rjernst left review comments

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
:Core/Infra/Core Core issues without another label external-contributor Pull request authored by a developer outside the Elasticsearch team Team:Core/Infra Meta label for core/infra team v9.1.0
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Provide better error messages on search request parse failures
4 participants
@AdalineL @elasticsearchmachine @rjernst @gbanasiak