Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

test: wait for exporter to close to prevent flaky test #10508

Merged

Conversation

lenaschoenburg
Copy link
Member

Here we await for the correct exporter phase before submitting resume/pause requests to the exporter director. This is to prevent cases where these jobs were submitted before the director had the chance to close the actor, resulting in an unexpected failure when the pause/resume jobs ran.

closes #10439

@github-actions
Copy link
Contributor

Test Results

   933 files  +    1     933 suites  +1   2h 2m 48s ⏱️ +51s
7 458 tests +110  7 447 ✔️ +109  10 💤 ±0  1 +1 
7 646 runs  +110  7 635 ✔️ +109  10 💤 ±0  1 +1 

For more details on these failures, see this check.

Results for commit dfd4ef4. ± Comparison against base commit f915b56.

@lenaschoenburg
Copy link
Member Author

bors r+

@zeebe-bors-camunda
Copy link
Contributor

Build succeeded:

@zeebe-bors-camunda zeebe-bors-camunda bot merged commit 8a2fc28 into main Sep 27, 2022
@zeebe-bors-camunda zeebe-bors-camunda bot deleted the 10439-fix-flaky-exporter-director-pause-test branch September 27, 2022 10:34
@backport-action
Copy link
Collaborator

Successfully created backport PR #10515 for release-8.1.0.

zeebe-bors-camunda bot added a commit that referenced this pull request Sep 27, 2022
10515: [Backport release-8.1.0] test: wait for exporter to close to prevent flaky test r=oleschoenburg a=backport-action

# Description
Backport of #10508 to `release-8.1.0`.

relates to #10439

Co-authored-by: Ole Schönburg <ole.schoenburg@gmail.com>
@Zelldon Zelldon added the version:8.1.0 Marks an issue as being completely or in parts released in 8.1.0 label Oct 4, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
version:8.1.0 Marks an issue as being completely or in parts released in 8.1.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

ExporterDirectorPauseTest flaky
3 participants