Project

General

Profile

Feature #37590

Umbrella #25917: Move business logic from Django to middlewared and make websocket API feature complete

Convert Storage:Detach Volume to Middlewared

Added by William Grzybowski about 1 year ago. Updated 5 months ago.

Status:
Done
Priority:
Expected
Assignee:
William Grzybowski
Category:
Middleware
Target version:
Estimated time:
Severity:
Medium
Reason for Closing:
Reason for Blocked:
Needs QA:
No
Needs Doc:
No
Needs Merging:
No
Needs Automation:
Yes
Support Suite Ticket:
n/a
Hardware Configuration:

Description

We need to change legacy UI to detach a volume using middlewared calls (this is to make sure v1.0 API will continue working as expected)
We need to create associated middlewared API transfering logic from legacy UI code.

Risk
This is a medium/high risk task, we may introduce new bugs on detaching a volume.

Acceptance Criteria
When this is done we should be able to continue to detach a volume at full extent when using legacy UI.
1. Pool not encrypted, not makring disks as new
2. Pool not encrypted, marking disks as new (destroy)
3. Pool encrypted, not makring disks as new
4. Pool encrypted, marking disks as new (destroy)
5. Pool not encrypted, being used as a system dataset, marking disks as new (destroy)


Related issues

Related to FreeNAS - Bug #38779: Run zpool enclosure sync on pool detach/destroyReady for Testing

Associated revisions

Revision 7a75cc50 (diff)
Added by William Grzybowski about 1 year ago

feat(middlewared/pool): implement pool.export

Ticket: #37590

Revision 089f4cba (diff)
Added by William Grzybowski about 1 year ago

feat(gui): use middleware to detach/destroy a pool

Ticket: #37590

Revision 587c0045 (diff)
Added by William Grzybowski about 1 year ago

rm(notifier): one more unused method

Ticket: #37590

Revision 68e2fb9b (diff)
Added by William Grzybowski about 1 year ago

fix(middlewared/disk): log exception string

Ticket: #37590

Revision 5bab79ee (diff)
Added by William Grzybowski about 1 year ago

feat(middlewared/pool): implement pool.export

Ticket: #37590

Revision d9ddcad4 (diff)
Added by William Grzybowski about 1 year ago

feat(gui): use middleware to detach/destroy a pool

Ticket: #37590

Revision 8e800ce9 (diff)
Added by William Grzybowski about 1 year ago

rm(notifier): one more unused method

Ticket: #37590

Revision b7183465 (diff)
Added by William Grzybowski about 1 year ago

fix(middlewared/disk): log exception string

Ticket: #37590

Revision f3ceb8a0 (diff)
Added by William Grzybowski about 1 year ago

fix(gui): typo

Ticket: #37590

Revision 79a7b1a1 (diff)
Added by William Grzybowski about 1 year ago

fix(gui): typo

Ticket: #37590

History

#1 Updated by William Grzybowski about 1 year ago

  • Status changed from Unscreened to Screened

#2 Updated by Bug Clerk about 1 year ago

  • Status changed from Screened to In Progress

#3 Updated by William Grzybowski about 1 year ago

  • Description updated (diff)

#4 Updated by William Grzybowski about 1 year ago

  • Related to Bug #38779: Run zpool enclosure sync on pool detach/destroy added

#5 Updated by Bug Clerk about 1 year ago

  • Status changed from In Progress to Ready for Testing

#6 Updated by Bug Clerk about 1 year ago

  • Status changed from Ready for Testing to In Progress

#7 Updated by Bug Clerk about 1 year ago

  • Status changed from In Progress to Ready for Testing

#8 Updated by Dru Lavigne 7 months ago

  • Target version changed from 11.3 to 11.3-BETA1

#12 Updated by Dru Lavigne 6 months ago

  • Status changed from Passed Testing to Done

#13 Updated by Dru Lavigne 5 months ago

  • Target version changed from 11.3-BETA1 to 11.3-ALPHA1

Also available in: Atom PDF