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

Assertion Updates Needed due to esarchiver load action concurrency change - x-pack/test/monitoring_api_integration/apis/elasticsearch/indices.ts #171020

Open
wayneseymour opened this issue Nov 10, 2023 · 3 comments
Assignees

Comments

@wayneseymour
Copy link
Member

wayneseymour commented Nov 10, 2023

The es archiver load action is being updated to increase speed.

The usual culprit is the concurrency, which places data out of expected sort order.
We expect placing a sort order in the queries themselves is one solution.
Another could be asserting in such a fashion that does not require a sort order.

The tests were commented out in the index file.

This is issue is an attempt to ensure that no decreased coverage, goes overlooked.

@botelastic botelastic bot added the needs-team Issues missing a team label label Nov 10, 2023
@klacabane
Copy link
Contributor

Thanks for the issue @wayneseymour! Do we have a way to control concurrency from the client or the option can only be controlled internally ?

@jsanz jsanz added the Team:Infra Monitoring UI - DEPRECATED DEPRECATED - Label for the Infra Monitoring UI team. Use Team:obs-ux-infra_services label Nov 20, 2023
@elasticmachine
Copy link
Contributor

Pinging @elastic/infra-monitoring-ui (Team:Infra Monitoring UI)

@botelastic botelastic bot removed the needs-team Issues missing a team label label Nov 20, 2023
@smith smith added Team:Monitoring Stack Monitoring team Feature:Stack Monitoring and removed Team:Infra Monitoring UI - DEPRECATED DEPRECATED - Label for the Infra Monitoring UI team. Use Team:obs-ux-infra_services labels Dec 1, 2023
@wayneseymour
Copy link
Member Author

Thanks for the issue @wayneseymour! Do we have a way to control concurrency from the client or the option can only be controlled internally ?

No, we do not. Is that something you would like?

@wayneseymour wayneseymour changed the title Assertion Updates Needed due to esarchiver load action concurrency change Assertion Updates Needed due to esarchiver load action concurrency change - x-pack/test/monitoring_api_integration/apis/elasticsearch/indices.ts Dec 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants