Feature #25821
Remove unused idmap backends
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
Associated revisions
History
#1
Updated by Timur Bakeyev over 3 years ago
- Related to Bug #25500: Remove unused AD backends added
#2
Updated by Timur Bakeyev over 3 years 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 over 3 years ago
- Subject changed from Make a list of available Idmap backends dynamic to Remove unused idmap backends
#4
Updated by John Hixson over 3 years 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 over 3 years ago
- Status changed from Reviewed by Developer to 46
- Assignee changed from Release Council to Timur Bakeyev
#6
Updated by Timur Bakeyev over 3 years 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 over 3 years ago
- Assignee changed from Release Council to William Grzybowski
#8
Updated by Dru Lavigne over 3 years ago
- Assignee changed from William Grzybowski to John Hixson
#9
Updated by John Hixson over 3 years 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 over 3 years ago
- Assignee changed from Release Council to Timur Bakeyev
#11
Updated by Dru Lavigne over 3 years 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 over 3 years ago
- Status changed from Ready For Release to Resolved