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

[CI] {p0=mixed_cluster/30_ml_jobs_crud/Test get old cluster categorization job} fails #65893

Closed
droberts195 opened this issue Dec 4, 2020 · 2 comments
Labels
:ml Machine learning >test-failure Triaged test failures from CI

Comments

@droberts195
Copy link
Contributor

droberts195 commented Dec 4, 2020

Build scan:

Repro line:

./gradlew ':x-pack:qa:rolling-upgrade:v7.11.0#bwcTest'

Reproduces locally?:

No

Applicable branches:

master

Failure history:

Started failing yesterday: https://build-stats.elastic.co/app/kibana#/discover?_g=(refreshInterval:(pause:!t,value:0),time:(from:now-7d,mode:quick,to:now))&_a=(columns:!(_source),filters:!(),index:b646ed00-7efc-11e8-bf69-63c8ef516157,interval:auto,query:(language:lucene,query:'UpgradeClusterClientYamlTestSuiteIT%20AND%20%22Test%20get%20old%20cluster%20categorization%20job%22'),sort:!(process.time-start,desc))

However, looking more closely the failures yesterday are not the same problem. The [jobs.0.node] doesn't have a true value error started today.

Failure excerpt:

java.lang.AssertionError: Failure at [mixed_cluster/30_ml_jobs_crud:77]: field [jobs.0.node] doesn't have a true value
@droberts195 droberts195 added >test-failure Triaged test failures from CI :ml Machine learning labels Dec 4, 2020
@elasticmachine
Copy link
Collaborator

Pinging @elastic/ml-core (:ml)

droberts195 added a commit to droberts195/elasticsearch that referenced this issue Dec 4, 2020
droberts195 added a commit to droberts195/elasticsearch that referenced this issue Dec 8, 2020
Due to a backporting mistake, master and 7.x were
serializing the members of model_size_stats in
different orders.  This led to BWC test failures
on master.

This change makes the ordering in 7.x the same as
the ordering in master.  It will be tested when
the mute done in elastic#65895 is reverted on the master
branch.

Fixes elastic#65893
droberts195 added a commit that referenced this issue Dec 8, 2020
Due to a backporting mistake, master and 7.x were
serializing the members of model_size_stats in
different orders.  This led to BWC test failures
on master.

This change makes the ordering in 7.x the same as
the ordering in master.  It will be tested when
the mute done in #65895 is reverted on the master
branch.

Fixes #65893
droberts195 added a commit to droberts195/elasticsearch that referenced this issue Dec 8, 2020
This reverts commit 1cf0a6e.

The tests should work now that elastic#66015 is merged.

Relates elastic#65893
droberts195 added a commit that referenced this issue Dec 8, 2020
This reverts commit 1cf0a6e.

The tests should work now that #66015 is merged.

Relates #65893
@droberts195
Copy link
Contributor Author

Fixed by #66015

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:ml Machine learning >test-failure Triaged test failures from CI
Projects
None yet
Development

No branches or pull requests

2 participants