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 rule_type_counts to QueryRulesetListResponse spec #3150

Merged
merged 2 commits into from
Nov 19, 2024

Conversation

kderusso
Copy link
Member

Adds the rule_type_counts added in elastic/elasticsearch#116357 to the specification

Copy link
Contributor

Following you can find the validation results for the API you have changed.

API Status Request Response
query_rules.list_rulesets 🟢 8/8 8/8

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

1 similar comment
Copy link
Contributor

Following you can find the validation results for the API you have changed.

API Status Request Response
query_rules.list_rulesets 🟢 8/8 8/8

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

@kderusso kderusso requested a review from pquentin November 19, 2024 13:53
Copy link
Member

@pquentin pquentin left a comment

Choose a reason for hiding this comment

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

Thank you! LGTM. Should we backport it to the 8.x or 8.16 branches?

@kderusso
Copy link
Member Author

@pquentin good catch - I added labels to backport to 8.16 and 8.x. Is that sufficient?

@kderusso kderusso merged commit b74dd77 into main Nov 19, 2024
9 checks passed
@kderusso kderusso deleted the kderusso/query-rule-type-counts branch November 19, 2024 14:44
github-actions bot pushed a commit that referenced this pull request Nov 19, 2024
* Add rule_type_counts to QueryRulesetListResponse spec

* Generate schema

(cherry picked from commit b74dd77)
github-actions bot pushed a commit that referenced this pull request Nov 19, 2024
* Add rule_type_counts to QueryRulesetListResponse spec

* Generate schema

(cherry picked from commit b74dd77)
@pquentin
Copy link
Member

@kderusso Yes, that's enough to open the pull request. I closed/reopened them, which is an optional trick I use to get CI to run, which confirmed this API was fixed. And then I approved, which gave me the green merge button.

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.

2 participants