Project

General

Profile

Feature #25821

Remove unused idmap backends

Added by Timur Bakeyev over 1 year ago. Updated about 1 year ago.

Status:
Resolved
Priority:
Nice to have
Assignee:
Timur Bakeyev
Category:
GUI (new)
Target version:
Estimated time:
Severity:
New
Reason for Closing:
Reason for Blocked:
Needs QA:
No
Needs Doc:
Yes
Needs Merging:
Yes
Needs Automation:
No
Support Suite Ticket:
n/a
Hardware Configuration:

Description

Build list of the idmap backends based on the contend of the /usr/local/lib/shared-modules/idmap directory, so obsoleted plugins are not shown and any new ones at minimum offered as an option.


Related issues

Related to FreeNAS - Bug #25500: Remove unused AD backendsClosed: Duplicate2017-08-08

Associated revisions

Revision 3fbe66dd (diff)
Added by Dru Lavigne about 1 year ago

Update Idmap table and link to more recent man pages.
Ticket: #25821

History

#1 Updated by Timur Bakeyev over 1 year ago

  • Related to Bug #25500: Remove unused AD backends added

#2 Updated by Timur Bakeyev about 1 year ago

  • Status changed from Unscreened to Needs Developer Review
  • Assignee changed from Timur Bakeyev to John Hixson
  • Needs QA changed from Yes to No

Hi, John!

Can you, please, review https://github.com/freenas/freenas/pull/309

That's the cleaned up version of what we discussed in #25500, I've messed up the pull request and that was the easiest way to make things right.

#3 Updated by Dru Lavigne about 1 year ago

  • Subject changed from Make a list of available Idmap backends dynamic to Remove unused idmap backends

#4 Updated by John Hixson about 1 year ago

  • Status changed from Needs Developer Review to Reviewed by Developer
  • Assignee changed from John Hixson to Release Council

Timur Bakeyev wrote:

Hi, John!

Can you, please, review https://github.com/freenas/freenas/pull/309

That's the cleaned up version of what we discussed in #25500, I've messed up the pull request and that was the easiest way to make things right.

I've commented on the pull request. Some things need changing, other than that, everything looks good. I'm a little confused on the need for a "none" idmap type, since there should never be an empty idmap ever, so if you could explain the reasoning for that, that would be great ;-)

#5 Updated by Dru Lavigne about 1 year ago

  • Status changed from Reviewed by Developer to 46
  • Assignee changed from Release Council to Timur Bakeyev

#6 Updated by Timur Bakeyev about 1 year ago

  • Status changed from 46 to Needs Developer Review
  • Assignee changed from Timur Bakeyev to Release Council

John, I've made requested changes, please, take a look again.

#7 Updated by Dru Lavigne about 1 year ago

  • Assignee changed from Release Council to William Grzybowski

#8 Updated by Dru Lavigne about 1 year ago

  • Assignee changed from William Grzybowski to John Hixson

#9 Updated by John Hixson about 1 year ago

  • Status changed from Needs Developer Review to Reviewed by Developer
  • Assignee changed from John Hixson to Release Council

Looks good to me

#10 Updated by Dru Lavigne about 1 year ago

  • Assignee changed from Release Council to Timur Bakeyev

#11 Updated by Dru Lavigne about 1 year ago

  • Status changed from Reviewed by Developer to Ready For Release
  • Target version changed from 11.1 to 11.1-BETA1

#12 Updated by Dru Lavigne about 1 year ago

  • Status changed from Ready For Release to Resolved

Also available in: Atom PDF