Skip to content

CI: disable fail-fast in test-suite#669

Merged
mergify[bot] merged 1 commit intomasterfrom
pr/ansiwen/ci-no-fail-fast
Apr 12, 2022
Merged

CI: disable fail-fast in test-suite#669
mergify[bot] merged 1 commit intomasterfrom
pr/ansiwen/ci-no-fail-fast

Conversation

@ansiwen
Copy link
Collaborator

@ansiwen ansiwen commented Apr 12, 2022

Because of the flaky tests sometimes the non-required test cancels the required one, which makes the flaky tests even more annoying.

@ansiwen ansiwen requested a review from phlogistonjohn April 12, 2022 11:33
@ansiwen ansiwen added small no-API This PR does not include any changes to the public API of a go-ceph package labels Apr 12, 2022
Copy link
Collaborator

@phlogistonjohn phlogistonjohn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cool. I didn't even know that existed. :-)

@phlogistonjohn
Copy link
Collaborator

@Mergifyio rebase

Because of the flaky tests sometimes the non-required test cancels the required one, which makes the flaky tests even more annoying.
@mergify
Copy link

mergify bot commented Apr 12, 2022

rebase

✅ Branch has been successfully rebased

@leseb leseb force-pushed the pr/ansiwen/ci-no-fail-fast branch from 3e255f4 to 57b4632 Compare April 12, 2022 13:32
@mergify mergify bot merged commit 8c28840 into master Apr 12, 2022
@mergify mergify bot deleted the pr/ansiwen/ci-no-fail-fast branch April 12, 2022 14:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no-API This PR does not include any changes to the public API of a go-ceph package small

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants