Derek Su

Results 1074 comments of Derek Su

@bachmanity1 Thanks for the contribution. Thanks @hookak @shuo-wu @c3y1huang for review as well. All PRs are merged. We can move it to `Ready-For-Testing`.

Agree with the proposal as well. Thanks @bachmanity1 cc @davidcheng0922 @c3y1huang

@harshitasao Can you help resolve the comment from @innobead? Thanks.

Also found the issue often happens in longhorn-spdk-engine CI ``` [2025-09-01 09:37:29.111661] thread.c:2516:spdk_put_io_channel: *DEBUG*: Putting io_channel 0x195e1860 for io_device test-vol-4-replica-1-snap-rebuild-a1d9f84fn1 (0x19e7ee50) on thread app_thread refcnt 1 [2025-09-01 09:37:29.111666] thread.c:2516:spdk_put_io_channel: *DEBUG*:...

> Hi, team The patch has been merged upstream. I've opened a PR to cherry-pick it into Longhorn v25.05. When you have a moment, could you review it? Thanks [@derekbit](https://github.com/derekbit)...

@chriscchien Can you help check if the issue is valid? Thanks.

> I prefer to support strict-local volumes with the block-for-eviction-if-contains-last-replica setting. This gives users more control over strict-local volumes during node down/drain cases. This can wait until after https://github.com/longhorn/longhorn/issues/8769 is...

@nzhan126 Since you are working on the fix, could we assign the issue to you?

Hello @nzhan126 The PR is merged. Could you help finish the pre-check list https://github.com/longhorn/longhorn/issues/8537#issuecomment-2968052935? Thank you.