Skip to content

Commit dbea483

Browse files
Fix SLMSnapshotBlockingIntegTests.testSnapshotInProgress (#59218)
Waiting `INIT` here is dead code in newer versions that don't use `INIT` any longer and leads to nothing being written to the repository in older versions if the snapshot is cancelled at the `INIT` step which then breaks repo consistency checks. Since we have other tests ensuring that snapshot abort works properly we can just remove the wait for `INIT` here and backport this down to 7.8 to fix tests. relates #59140
1 parent 865b6b5 commit dbea483

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

x-pack/plugin/ilm/src/test/java/org/elasticsearch/xpack/slm/SLMSnapshotBlockingIntegTests.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -123,8 +123,8 @@ public void testSnapshotInProgress() throws Exception {
123123
SnapshotLifecyclePolicyItem.SnapshotInProgress inProgress = item.getSnapshotInProgress();
124124
assertThat(inProgress.getSnapshotId().getName(), equalTo(snapshotName));
125125
assertThat(inProgress.getStartTime(), greaterThan(0L));
126-
assertThat(inProgress.getState(), anyOf(equalTo(SnapshotsInProgress.State.INIT), equalTo(SnapshotsInProgress.State.STARTED),
127-
equalTo(SnapshotsInProgress.State.SUCCESS)));
126+
assertThat(inProgress.getState(),
127+
anyOf(equalTo(SnapshotsInProgress.State.STARTED), equalTo(SnapshotsInProgress.State.SUCCESS)));
128128
assertNull(inProgress.getFailure());
129129
});
130130

0 commit comments

Comments
 (0)