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

[Transform] TransformIT.testStartTransform_GivenTimeout_Returns408 may interfere with _transform/_node_stats API #109101

Open
przemekwitek opened this issue May 28, 2024 · 1 comment
Labels
>bug :ml/Transform Transform Team:ML Meta label for the ML team

Comments

@przemekwitek
Copy link
Contributor

Elasticsearch Version

main

Installed Plugins

No response

Java Version

bundled

OS Version

n/a

Problem Description

While working on the _transform/_node_stats API I've noticed start-transform-timeout transform created by the TransformIT.testStartTransform_GivenTimeout_Returns408 test case interfering with the assertions on the _transform/_node_stats API response.
When we expect no transform tasks in the scheduler, there is one task (start-transform-timeout) existing.

Steps to Reproduce

This came out in CI testing of the PR introducing _transform/_node_stats API (i.e. #107279).

Logs (if relevant)

java.lang.AssertionError: Failure at iteration 0 (sleepAfterStart=1807ms,force=true): Node stats were: {total={scheduler={registered_transform_count=1}}, j88QlbTQR6O0R7ACnDtuiw={scheduler={registered_transform_count=1, peek_transform=start-transform-timeout}}, IHzZKV2DTLCuGx4Tc7sH3w={scheduler={registered_transform_count=0, peek_transform=null}}, Mfx-FOPWTCOir7xQ9bsZyg={scheduler={registered_transform_count=0, peek_transform=null}}}	

Notice the peek_transform being start-transform-timeout even though these are two separate test cases.

@przemekwitek przemekwitek added >bug needs:triage Requires assignment of a team area label labels May 28, 2024
@przemekwitek przemekwitek self-assigned this May 28, 2024
@przemekwitek przemekwitek added :ml/Transform Transform and removed needs:triage Requires assignment of a team area label labels May 28, 2024
@elasticsearchmachine elasticsearchmachine added the Team:ML Meta label for the ML team label May 28, 2024
@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/ml-core (Team:ML)

@przemekwitek przemekwitek removed their assignment Jun 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
>bug :ml/Transform Transform Team:ML Meta label for the ML team
Projects
None yet
Development

No branches or pull requests

2 participants