Skip to content

Commit cc3c8be

Browse files
Fix SLMSnapshotBlockingIntegTests.testSnapshotInProgress (#59218) (#59239)
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 838f717 commit cc3c8be

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
@@ -144,8 +144,8 @@ public void testSnapshotInProgress() throws Exception {
144144
SnapshotLifecyclePolicyItem.SnapshotInProgress inProgress = item.getSnapshotInProgress();
145145
assertThat(inProgress.getSnapshotId().getName(), equalTo(snapshotName));
146146
assertThat(inProgress.getStartTime(), greaterThan(0L));
147-
assertThat(inProgress.getState(), anyOf(equalTo(SnapshotsInProgress.State.INIT), equalTo(SnapshotsInProgress.State.STARTED),
148-
equalTo(SnapshotsInProgress.State.SUCCESS)));
147+
assertThat(inProgress.getState(),
148+
anyOf(equalTo(SnapshotsInProgress.State.STARTED), equalTo(SnapshotsInProgress.State.SUCCESS)));
149149
assertNull(inProgress.getFailure());
150150
});
151151

0 commit comments

Comments
 (0)