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

fix: attach volumes to eliminate duplicated replicas #2109

Conversation

ChanYiLin
Copy link
Contributor

ref: longhorn/longhorn#8573

longhorn/longhorn#8573 (comment)
When the engine image is not stable, there is a chance that the volume changes the owner and creates duplicated replicas.
This might fail this e2e test because it checks the number of replicas before the volume is attached.

We should attach the volume and let Longhorn deletes duplicated replicas before starting the test.

ref: longhorn/longhorn 8573

Signed-off-by: Jack Lin <jack.lin@suse.com>
@ChanYiLin ChanYiLin force-pushed the LH8573_fix_volume_creating_duplica_replicas_issues branch from e5175e7 to 3b2332a Compare September 20, 2024 11:50
@ChanYiLin
Copy link
Contributor Author

Copy link
Member

@yangchiu yangchiu left a comment

Choose a reason for hiding this comment

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

LGTM

@yangchiu yangchiu merged commit eb5c6a3 into longhorn:master Sep 23, 2024
5 of 6 checks passed
@yangchiu
Copy link
Member

yangchiu commented Oct 1, 2024

@mergify backport v1.7.x v1.6.x

Copy link

mergify bot commented Oct 1, 2024

backport v1.7.x v1.6.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.

2 participants