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: print message when assertion failed #1889

Merged
merged 3 commits into from
May 8, 2024

Conversation

yangchiu
Copy link
Member

@yangchiu yangchiu commented May 8, 2024

Which issue(s) this PR fixes:

Issue longhorn/longhorn#8521

What this PR does / why we need it:

print message when assertion failed

Special notes for your reviewer:

Additional documentation or context

Signed-off-by: Yang Chiu <yang.chiu@suse.com>
@yangchiu yangchiu requested a review from a team May 8, 2024 02:56
@yangchiu yangchiu self-assigned this May 8, 2024
@khushboo-rancher khushboo-rancher enabled auto-merge (rebase) May 8, 2024 22:49
@khushboo-rancher khushboo-rancher merged commit 2e0b8c1 into longhorn:master May 8, 2024
1 of 2 checks passed
@roger-ryao
Copy link
Contributor

I encountered the same error on v1.5.x . Backport the PR to v1.5.x and v1.6.x.

@roger-ryao
Copy link
Contributor

@mergify backport v1.6.x v1.5.x

Copy link

mergify bot commented May 27, 2024

backport v1.6.x v1.5.x

✅ Backports have been created

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants