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

Longhorn Online Rebuilding #2995

Merged
merged 5 commits into from
Jul 24, 2024
Merged

Longhorn Online Rebuilding #2995

merged 5 commits into from
Jul 24, 2024

Conversation

shuo-wu
Copy link
Contributor

@shuo-wu shuo-wu commented Jul 23, 2024

Which issue(s) this PR fixes:

Issue longhorn/longhorn#7199

What this PR does / why we need it:

Special notes for your reviewer:

Additional documentation or context

Copy link
Member

@derekbit derekbit left a comment

Choose a reason for hiding this comment

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

LGTM. Only one comment for readability.

derekbit
derekbit previously approved these changes Jul 23, 2024
Copy link
Member

@derekbit derekbit left a comment

Choose a reason for hiding this comment

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

LGTM

controller/volume_controller.go Show resolved Hide resolved
scheduler/replica_scheduler.go Show resolved Hide resolved
scheduler/replica_scheduler.go Show resolved Hide resolved
Longhorn 7199

Signed-off-by: Shuo Wu <shuo.wu@suse.com>
Now the SPDK replica head will be blindly exposed as a NVMf bdev
no matter if it is a local one.

Longhorn 7199

Signed-off-by: Shuo Wu <shuo.wu@suse.com>
Longhorn 7199

Signed-off-by: Shuo Wu <shuo.wu@suse.com>
Longhorn 7199

Signed-off-by: Shuo Wu <shuo.wu@suse.com>
Longhorn 7199

Signed-off-by: Shuo Wu <shuo.wu@suse.com>
@shuo-wu shuo-wu marked this pull request as ready for review July 24, 2024 05:01
@derekbit derekbit merged commit 45bef51 into longhorn:master Jul 24, 2024
6 checks passed
@derekbit
Copy link
Member

@mergify backport v1.7.x

Copy link

mergify bot commented Jul 24, 2024

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