Skip to content

Commit

Permalink
merge: #9581
Browse files Browse the repository at this point in the history
9581: [Backport 8.0]: Do not restart broker on snapshot test r=npepinpe a=Zelldon

## Description

Backports #9573 
<!-- Please explain the changes you made here. -->

## Related issues

<!-- Which issues are closed by this PR or are related -->

closes #9517 



Co-authored-by: Christopher Zell <[email protected]>
  • Loading branch information
zeebe-bors-camunda[bot] and ChrisKujawa authored Jun 22, 2022
2 parents ec656da + a22ce30 commit 81f19d9
Showing 1 changed file with 0 additions and 6 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -49,12 +49,6 @@ void update(final String name, final UpdateTestCase testCase, final ContainerSta
final long key = testCase.runBefore(state);

// when
// it's necessary to restart without the debug exporter to allow snapshotting
state.close();
state.broker(LAST_VERSION).start(false);
// there's a slight chance that we'd processed everything before shutting down, so we send a
// dummy message to ensure we have processed something since we recovered and so take a snapshot
sendDummyMessageToEnforceSnapshot(state);
EitherAssert.assertThat(state.getPartitionsActuatorClient().takeSnapshot())
.as("expect successful response as right member")
.isRight();
Expand Down

0 comments on commit 81f19d9

Please sign in to comment.