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

[FEATURE] v2 volume supports auto salvage #8430

Closed
derekbit opened this issue Apr 24, 2024 · 3 comments
Closed

[FEATURE] v2 volume supports auto salvage #8430

derekbit opened this issue Apr 24, 2024 · 3 comments
Assignees
Labels
area/data-recovery Data recovery and rescue area/resilience System or volume resilience area/spdk SPDK upstream/downstream area/v2-data-engine v2 data engine (SPDK) highlight Important feature/issue to highlight kind/feature Feature request, new feature priority/0 Must be implement or fixed in this release (managed by PO) require/auto-e2e-test Require adding/updating auto e2e test cases if they can be automated require/doc Require updating the longhorn.io documentation require/lep Require adding/updating enhancement proposal require/manual-test-plan Require adding/updating manual test cases if they can't be automated
Milestone

Comments

@derekbit
Copy link
Member

derekbit commented Apr 24, 2024

Is your feature request related to a problem? Please describe (👍 if you like this request)

Describe the solution you'd like

Describe alternatives you've considered

Additional context

Some discussions
https://suse.slack.com/archives/C02DR3N5T24/p1713926457976349

@derekbit derekbit added kind/feature Feature request, new feature priority/0 Must be implement or fixed in this release (managed by PO) require/auto-e2e-test Require adding/updating auto e2e test cases if they can be automated require/doc Require updating the longhorn.io documentation require/lep Require adding/updating enhancement proposal require/manual-test-plan Require adding/updating manual test cases if they can't be automated area/v2-data-engine v2 data engine (SPDK) area/spdk SPDK upstream/downstream labels Apr 24, 2024
@derekbit derekbit added this to the v1.8.0 milestone Apr 24, 2024
@DamiaSan
Copy link
Contributor

See data consistency in #7031

@longhorn-io-github-bot
Copy link

longhorn-io-github-bot commented Sep 27, 2024

Pre Ready-For-Testing Checklist

@c3y1huang c3y1huang moved this from Review to Implement in Longhorn Sprint Nov 1, 2024
@c3y1huang c3y1huang moved this from Implement to Review in Longhorn Sprint Nov 1, 2024
@c3y1huang c3y1huang moved this from Review to Ready For Testing in Longhorn Sprint Nov 6, 2024
@c3y1huang c3y1huang moved this from Ready For Testing to Review in Longhorn Sprint Nov 6, 2024
@c3y1huang c3y1huang moved this from Review to Ready For Testing in Longhorn Sprint Nov 7, 2024
@chriscchien chriscchien self-assigned this Nov 8, 2024
@chriscchien
Copy link
Contributor

Verified pass on longhorn v1.8.0-dev(longhorn/longhorn-manager@a503949)

Test steps:

  1. Dynamic provision a v2 volume, having data in the mount point.
  2. Set auto-salvage to false
  3. Force delete all v2 volume instance-manager pod.
  4. V2 volume become faulted and remains in that state.
  5. Set auto-salvage to true
  6. The faulted v2 volume become healthy and data in the workload is correct.
  7. Force delete all v2 volume instance-manager pod again.
  8. V2 olume become faulted then healthy again.
  9. After volume healthy, data in workload correct.

In v1.7.2, set auto-salvage to true and force delete all v2 volume instance-manager pod, the v2 volume will loop from faulted -> attaching -> faulted.

@github-project-automation github-project-automation bot moved this from Testing to Closed in Longhorn Sprint Nov 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/data-recovery Data recovery and rescue area/resilience System or volume resilience area/spdk SPDK upstream/downstream area/v2-data-engine v2 data engine (SPDK) highlight Important feature/issue to highlight kind/feature Feature request, new feature priority/0 Must be implement or fixed in this release (managed by PO) require/auto-e2e-test Require adding/updating auto e2e test cases if they can be automated require/doc Require updating the longhorn.io documentation require/lep Require adding/updating enhancement proposal require/manual-test-plan Require adding/updating manual test cases if they can't be automated
Projects
Status: Closed
Development

No branches or pull requests

7 participants