-
Notifications
You must be signed in to change notification settings - Fork 24.9k
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] XPackRestIT test {p0=ml/jobs_get_stats/Test get job stats after uploading data prompting the creation of some stats} failing #79518
Labels
Comments
benwtrent
added
:ml
Machine learning
>test-failure
Triaged test failures from CI
labels
Oct 19, 2021
Pinging @elastic/ml-core (Team:ML) |
benwtrent
added a commit
to benwtrent/elasticsearch
that referenced
this issue
Oct 19, 2021
benwtrent
added a commit
that referenced
this issue
Oct 19, 2021
weizijun
added a commit
to weizijun/elasticsearch
that referenced
this issue
Oct 20, 2021
* upstream/master: (24 commits) Implement framework for migrating system indices (elastic#78951) Improve transient settings deprecation message (elastic#79504) Remove getValue and getValues from Field (elastic#79516) Store Template's mappings as bytes for disk serialization (elastic#78746) [ML] Add queue_capacity setting to start deployment API (elastic#79433) [ML] muting rest compat test issue elastic#79518 (elastic#79519) Avoid redundant available indices check (elastic#76540) Re-enable BWC tests TEST Ensure password 14 chars length on Kerberos FIPS tests (elastic#79496) [DOCS] Temporarily remove APM links (elastic#79411) Fix CCSDuelIT for skipped shards (elastic#79490) Add other time accounting in HotThreads (elastic#79392) Add deprecation info API entries for deprecated monitoring settings (elastic#78799) Add note in breaking changes for nameid_format (elastic#77785) Use 'migration' instead of 'upgrade' in GET system feature migration status responses (elastic#79302) Upgrade lucene version 8b68bf60c98 (elastic#79461) Use Strings#EMPTY_ARRAY (elastic#79452) Quicker shared cache file preallocation (elastic#79447) [ML] Removing some code that's obsolete for 8.0 (elastic#79444) Ensure indexing_data CCR requests are compressed (elastic#79413) ...
droberts195
added a commit
to droberts195/elasticsearch
that referenced
this issue
Oct 21, 2021
The way we detect an ML node from node attributes is now to look for ml.machine_memory instead of ml.max_open_jobs. Relates elastic#79518
droberts195
added a commit
to droberts195/elasticsearch
that referenced
this issue
Oct 21, 2021
Where possible we should now detect ML nodes by looking for the ML node role rather than an attribute. Sadly this is not possible in the YAML tests, as the syntax doesn't support asserting that a list contains a particular item without asserting on the entire list contents. So in the YAML case we need to look for the ml.machine_memory node attribute, as this is reliably set on ML nodes but not other types of nodes. After elastic#79622 is merged this closes elastic#79518
droberts195
added a commit
that referenced
this issue
Oct 21, 2021
The way we detect an ML node from node attributes is now to look for ml.machine_memory instead of ml.max_open_jobs. Relates #79518
droberts195
added a commit
that referenced
this issue
Oct 21, 2021
Where possible we should now detect ML nodes by looking for the ML node role rather than an attribute. Sadly this is not possible in the YAML tests, as the syntax doesn't support asserting that a list contains a particular item without asserting on the entire list contents. So in the YAML case we need to look for the ml.machine_memory node attribute, as this is reliably set on ML nodes but not other types of nodes. After #79622 is merged this closes #79518
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
Build scan:
https://gradle-enterprise.elastic.co/s/xhcwxk6kvhqtc/tests/:x-pack:plugin:yamlRestTestV7CompatTest/org.elasticsearch.xpack.test.rest.XPackRestIT/test%20%7Bp0=ml%2Fjobs_get_stats%2FTest%20get%20job%20stats%20after%20uploading%20data%20prompting%20the%20creation%20of%20some%20stats%7D
Reproduction line:
./gradlew ':x-pack:plugin:yamlRestTestV7CompatTest' --tests "org.elasticsearch.xpack.test.rest.XPackRestIT.test {p0=ml/jobs_get_stats/Test get job stats after uploading data prompting the creation of some stats}" -Dtests.seed=EF079440456288AA -Dtests.locale=fr-CH -Dtests.timezone=Asia/Dushanbe -Druntime.java=11
Applicable branches:
master
Reproduces locally?:
Yes
Failure history:
https://gradle-enterprise.elastic.co/scans/tests?tests.container=org.elasticsearch.xpack.test.rest.XPackRestIT&tests.test=test%20%7Bp0%3Dml/jobs_get_stats/Test%20get%20job%20stats%20after%20uploading%20data%20prompting%20the%20creation%20of%20some%20stats%7D
Failure excerpt:
The text was updated successfully, but these errors were encountered: