Skip to content

[CI] ModelRegistryMetadataTests testAlreadyUpgraded failing #125585

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

Closed
elasticsearchmachine opened this issue Mar 25, 2025 · 4 comments · Fixed by #125756
Closed

[CI] ModelRegistryMetadataTests testAlreadyUpgraded failing #125585

elasticsearchmachine opened this issue Mar 25, 2025 · 4 comments · Fixed by #125756
Labels
:ml Machine learning needs:risk Requires assignment of a risk label (low, medium, blocker) Team:ML Meta label for the ML team >test-failure Triaged test failures from CI

Comments

@elasticsearchmachine
Copy link
Collaborator

elasticsearchmachine commented Mar 25, 2025

Build Scans:

Reproduction Line:

./gradlew ":x-pack:plugin:inference:test" --tests "org.elasticsearch.xpack.inference.registry.ModelRegistryMetadataTests.testAlreadyUpgraded" -Dtests.seed=68482702E60D912A -Dtests.locale=he -Dtests.timezone=US/Pacific -Druntime.java=24

Applicable branches:
8.x

Reproduces locally?:
N/A

Failure History:
See dashboard

Failure Message:

junit.framework.AssertionFailedError: Expected exception IllegalArgumentException but no exception was thrown

Issue Reasons:

  • [8.x] 5 failures in test testAlreadyUpgraded (1.1% fail rate in 445 executions)
  • [8.x] 2 failures in step part2 (3.7% fail rate in 54 executions)
  • [8.x] 2 failures in pipeline elasticsearch-intake (3.7% fail rate in 54 executions)
  • [8.x] 2 failures in pipeline elasticsearch-periodic-platform-support (22.2% fail rate in 9 executions)

Note:
This issue was created using new test triage automation. Please report issues or feedback to es-delivery.

@elasticsearchmachine elasticsearchmachine added :ml Machine learning >test-failure Triaged test failures from CI labels Mar 25, 2025
@elasticsearchmachine
Copy link
Collaborator Author

This has been muted on branch main

Mute Reasons:

  • [main] 6 failures in test testAlreadyUpgraded (1.6% fail rate in 374 executions)
  • [main] 5 failures in step part-2 (2.9% fail rate in 173 executions)
  • [main] 5 failures in pipeline elasticsearch-pull-request (3.0% fail rate in 169 executions)

Build Scans:

@elasticsearchmachine elasticsearchmachine added Team:ML Meta label for the ML team needs:risk Requires assignment of a risk label (low, medium, blocker) labels Mar 25, 2025
@elasticsearchmachine
Copy link
Collaborator Author

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

@elasticsearchmachine
Copy link
Collaborator Author

This has been muted on branch 8.x

Mute Reasons:

  • [8.x] 5 failures in test testAlreadyUpgraded (1.1% fail rate in 445 executions)
  • [8.x] 2 failures in step part2 (3.7% fail rate in 54 executions)
  • [8.x] 2 failures in pipeline elasticsearch-intake (3.7% fail rate in 54 executions)
  • [8.x] 2 failures in pipeline elasticsearch-periodic-platform-support (22.2% fail rate in 9 executions)

Build Scans:

omricohenn pushed a commit to omricohenn/elasticsearch that referenced this issue Mar 28, 2025
@Mikep86
Copy link
Contributor

Mikep86 commented Apr 1, 2025

Fixed by #126042

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:ml Machine learning needs:risk Requires assignment of a risk label (low, medium, blocker) Team:ML Meta label for the ML team >test-failure Triaged test failures from CI
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants