Project

General

Profile

Feature #10851

Add 'Lock Volume' to API

Added by C Led about 4 years ago. Updated over 1 year ago.

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

Description

There is the ability to unlock a volume with the API but do not have the ability to lock a volume. Would be nice to have the lock functionally as well.

Associated revisions

Revision 1fd2762d (diff)
Added by William Grzybowski over 2 years ago

feat(api): add lock endpoint

Ticket: #10851

Revision 84c3afa1 (diff)
Added by William Grzybowski over 2 years ago

feat(api): document volume lock

Ticket: #10851

Revision bbfdc409 (diff)
Added by William Grzybowski over 2 years ago

fix(api): make sure volume is encrypted before lock

Ticket: #10851

Revision dc0299a3 (diff)
Added by William Grzybowski over 2 years ago

fix(gui): redundant if statement

Ticket: #10851

History

#1 Updated by Jakub Klama about 4 years ago

  • Status changed from Unscreened to Screened

#2 Updated by Jakub Klama almost 3 years ago

  • Status changed from Screened to Unscreened
  • Assignee changed from Jakub Klama to William Grzybowski
  • Target version set to 272

Sounds like a feature request for FreeNAS 10 restd

#3 Updated by William Grzybowski almost 3 years ago

  • Status changed from Unscreened to Screened

c Lederer wrote:

There is the ability to unlock a volume with the API but do not have the ability to lock a volume. Would be nice to have the lock functionally as well.

Ca you clarify what you mean? Are you talking about FreeNAS 10 REST API or FreeNAS 9.10 v1.0 API?

#4 Updated by William Grzybowski almost 3 years ago

  • Project changed from 9 to FreeNAS
  • Category changed from 165 to Middleware
  • Target version changed from 272 to 9.10.2

Well, its been an year ago, so I am sure its 9.10 :D

#5 Updated by William Grzybowski over 2 years ago

  • Status changed from Screened to Needs Developer Review

#6 Updated by William Grzybowski over 2 years ago

  • Assignee changed from William Grzybowski to Josh Paetzel

Can you review pls, josh?

#7 Updated by Josh Paetzel over 2 years ago

  • Status changed from Needs Developer Review to 15
  • Assignee changed from Josh Paetzel to William Grzybowski

Can midclt be used instead of instatiating the notifier?

If no, then this looks good to me.

#8 Updated by William Grzybowski over 2 years ago

  • Status changed from 15 to Reviewed

midclt would not make sense in this case since it is already within django.

#9 Updated by Vaibhav Chauhan over 2 years ago

  • Status changed from Reviewed to Ready For Release

#10 Updated by Dru Lavigne over 1 year ago

  • Status changed from Ready For Release to Resolved

Also available in: Atom PDF