Skip to content

Conversation

@timgrein
Copy link
Contributor

@timgrein timgrein commented Feb 3, 2025

Fixes #121294

Sometimes tests are run with -Dbuild.snapshot=false, which means the tests are run on a non-snapshot build, which implies that feature flags are disabled. As we inject NoopNodeLocalRateLimitCalculator as implementation for InferenceServiceRateLimitCalculator, when inference_cluster_aware_rate_limiting_feature_flag is disabled we should skip InferenceServiceNodeLocalRateLimitCalculatorTests as these tests should only test the real implementation (which is injected if the feature flag is enabled).

@timgrein timgrein added >test Issues or PRs that are addressing/adding tests auto-backport Automatically create backport pull requests when merged v9.0.0 v8.18.0 labels Feb 3, 2025
@elasticsearchmachine elasticsearchmachine added needs:triage Requires assignment of a team area label v9.1.0 labels Feb 3, 2025
@maxjakob
Copy link
Contributor

maxjakob commented Feb 3, 2025

LGTM as long as for Sometimes tests are run with -Dbuild.snapshot=false "sometimes" here means at least once during continuous integration.

@timgrein
Copy link
Contributor Author

timgrein commented Feb 3, 2025

LGTM as long as for Sometimes tests are run with -Dbuild.snapshot=false "sometimes" here means at least once during continuous integration.

Citing the mute reasons I think they run often enough in several CIs. I don't know, whether this means at least once on every PR:

[main] 7 consecutive failures in step release-tests
[main] 4 consecutive failures in step checkpart2_release-tests
[main] 18 failures in class org.elasticsearch.xpack.inference.common.InferenceServiceNodeLocalRateLimitCalculatorTests (3.2% fail rate in 570 executions)
[main] 8 failures in step release-tests (72.7% fail rate in 11 executions)
[main] 10 failures in step checkpart2_release-tests (66.7% fail rate in 15 executions)
[main] 8 failures in pipeline elasticsearch-periodic (72.7% fail rate in 11 executions)
[main] 9 failures in pipeline elasticsearch-pull-request (4.4% fail rate in 205 executions)

@timgrein timgrein added :ml Machine learning Team:ML Meta label for the ML team and removed needs:triage Requires assignment of a team area label labels Feb 3, 2025
@elasticsearchmachine
Copy link
Collaborator

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


public void setUp() throws Exception {
super.setUp();
assumeTrue(
Copy link
Contributor

Choose a reason for hiding this comment

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

TIL 👍

@timgrein timgrein merged commit ac34cad into elastic:main Feb 4, 2025
16 checks passed
@elasticsearchmachine
Copy link
Collaborator

💔 Backport failed

Status Branch Result
9.0 Commit could not be cherrypicked due to conflicts
8.18 Commit could not be cherrypicked due to conflicts

You can use sqren/backport to manually backport by running backport --upstream elastic/elasticsearch --pr 121527

@timgrein
Copy link
Contributor Author

timgrein commented Feb 4, 2025

💚 All backports created successfully

Status Branch Result
8.x
9.0
8.18

Questions ?

Please refer to the Backport tool documentation

fzowl pushed a commit to voyage-ai/elasticsearch that referenced this pull request Feb 4, 2025
timgrein added a commit that referenced this pull request Feb 5, 2025
timgrein added a commit that referenced this pull request Feb 5, 2025
timgrein added a commit that referenced this pull request Feb 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auto-backport Automatically create backport pull requests when merged backport pending :ml Machine learning Team:ML Meta label for the ML team >test Issues or PRs that are addressing/adding tests v8.18.0 v9.0.0 v9.1.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[CI] InferenceServiceNodeLocalRateLimitCalculatorTests class failing

4 participants