Skip to content

Commit

Permalink
[Profiling] Fix spurious test failures
Browse files Browse the repository at this point in the history
With this commit we force-merge indices to workaround
elastic#106657. We also revert a test mute that was
applied to the wrong test case so all integration tests are executed
again.

Relates elastic#106309
Relates elastic#106657
Closes elastic#106308
  • Loading branch information
danielmitterdorfer committed Mar 22, 2024
1 parent 19cffde commit 7ee6413
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -42,7 +42,6 @@ public void testGetStackTracesUnfiltered() throws Exception {
assertEquals("vmlinux", response.getExecutables().get("lHp5_WAgpLy2alrUVab6HA"));
}

@AwaitsFix(bugUrl = "https://github.com/elastic/elasticsearch/issues/106308")
public void testGetStackTracesFromAPMWithMatchNoDownsampling() throws Exception {
BoolQueryBuilder query = QueryBuilders.boolQuery();
query.must().add(QueryBuilders.termQuery("transaction.name", "encodeSha1"));
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -143,6 +143,9 @@ protected final void doSetupData() throws Exception {
bulkIndex("data/apm-legacy-test.ndjson");

refresh();

// temporary workaround for #106657, see also #106308.
forceMerge();
}

@After
Expand Down

0 comments on commit 7ee6413

Please sign in to comment.