Skip to content

Commit f236373

Browse files
authored
[ML] Use consistent assertBusy() maxWaitTime in datafeed test (#137981)
Other uses of assertBusy() when checking datafeed stats in DatafeedJobsIT.testDatafeedTimingStats_DatafeedRecreated() use a maxWaitTime of 30 seconds instead of the default of 10, which is necessary to account for possible delays in persisting stat values. Update the maxWaitTime on the first datafeed stats assert to 30 seconds Closes #137207
1 parent 5875915 commit f236373

File tree

2 files changed

+1
-4
lines changed

2 files changed

+1
-4
lines changed

muted-tests.yml

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -426,9 +426,6 @@ tests:
426426
- class: org.elasticsearch.xpack.esql.qa.multi_node.GenerativeIT
427427
method: test
428428
issue: https://github.com/elastic/elasticsearch/issues/137909
429-
- class: org.elasticsearch.xpack.ml.integration.DatafeedJobsIT
430-
method: testDatafeedTimingStats_DatafeedRecreated
431-
issue: https://github.com/elastic/elasticsearch/issues/137207
432429
- class: org.elasticsearch.xpack.inference.integration.AuthorizationTaskExecutorIT
433430
method: testCreatesChatCompletion_AndThenCreatesTextEmbedding
434431
issue: https://github.com/elastic/elasticsearch/issues/138012

x-pack/plugin/ml/qa/native-multi-node-tests/src/javaRestTest/java/org/elasticsearch/xpack/ml/integration/DatafeedJobsIT.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -250,7 +250,7 @@ public void testDatafeedTimingStats_DatafeedRecreated() throws Exception {
250250

251251
putDatafeed(datafeedConfig);
252252
// Datafeed did not do anything yet, hence search_count is equal to 0.
253-
assertBusy(() -> assertDatafeedStats(datafeedId, DatafeedState.STOPPED, job.getId(), equalTo(0L)));
253+
assertBusy(() -> assertDatafeedStats(datafeedId, DatafeedState.STOPPED, job.getId(), equalTo(0L)), 30, TimeUnit.SECONDS);
254254
startDatafeed(datafeedId, 0L, now.toEpochMilli());
255255

256256
// First, wait for data processing to complete

0 commit comments

Comments
 (0)