Derek Su
Derek Su
Merged. Let's finish the pre-check list and move the `ready for testing`. Thank you.
@DamiaSan PR is merged. If everything is ready, you can move it to `ready for testing`. Thank you.
@DamiaSan Do we already have an E2E test case for this scenario? If not, please add a skeleton E2E test case for this ticket. cc @roger-ryao Could you help guide...
> [@derekbit](https://github.com/derekbit) This development has: > > * enabled new xattr setting for snapshot in `SPDK` > * added the API to set xattr into `go-spdk-helper` > > Don't we...
> Could you point to me where is the functionality to add labels to an existing snapshot in `longhorn-instance-manager`? Because I find labels only in snapshot creation, which is a...
> > > Could you point to me where is the functionality to add labels to an existing snapshot in `longhorn-instance-manager`? Because I find labels only in snapshot creation, which...
Assigned the ticket to @mantissahz, so @mantissahz can dive into SPDK internals while working on this ticket. cc @innobead
@DrummyFloyd Would you like to submit a PR for the doc update?
cc @c3y1huang
We will do technical review soon. Thanks for the contribution.