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 TransportAction Integ Tests for paginated APIs. #16454

Open
gargharsh3134 opened this issue Oct 23, 2024 · 0 comments
Open

Add TransportAction Integ Tests for paginated APIs. #16454

gargharsh3134 opened this issue Oct 23, 2024 · 0 comments
Labels
Cluster Manager enhancement Enhancement or improvement to existing feature or request untriaged

Comments

@gargharsh3134
Copy link
Contributor

gargharsh3134 commented Oct 23, 2024

Is your feature request related to a problem? Please describe

_list/shards API makes use of TransportCatShardsAction to get the list of shards to be responded in a page. Currently, the support of Integration tests which assert the behaviour of TransportCatShardsAction response is missing. This issue tracks the addition of such tests, which would help avoid the unoptimizations like the one called out in (#16448).

Similar tests for _list/indices, would have to be added once the core logic gets transferred from Rest layer to a new CatIndices specific TransportAction

Describe the solution you'd like

Adding integ test support in TransportActions for paginated _list APIs.

Related component

Cluster Manager

Describe alternatives you've considered

No response

Additional context

No response

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Cluster Manager enhancement Enhancement or improvement to existing feature or request untriaged
Projects
Status: Now(This Quarter)
Development

No branches or pull requests

1 participant