Feature #26157
Add warning that setting the minio directory changes ownership
Description
When setting the "Disks" option for the S3 service (Services -> S3 -> Config -> Disks), the ownership of the selected path will be recursively changed to `minio:minio`
If you accidentally select your ZFS root, all your ownership is blatted and makes you sad.
Perhaps a warning that ownership will be changed would be helpful?
Associated revisions
History
#1
Updated by Dru Lavigne over 3 years ago
- Tracker changed from Bug to Feature
- Subject changed from Setting Minio Directory Changes Ownership to Add warning about Setting Minio Directory Changes Ownership
- Assignee changed from Release Council to Marcelo Araujo
- Target version set to 11.1
#2
Updated by Marcelo Araujo over 3 years ago
- Status changed from Unscreened to Screened
- Target version changed from 11.1 to 11.2-BETA1
#3
Updated by Kris Moore over 3 years ago
- Target version changed from 11.2-BETA1 to 11.3
#4
Updated by Kris Moore about 3 years ago
- Status changed from Screened to Not Started
#5
Updated by Ben Gadd almost 3 years ago
- Target version changed from 11.3 to Backlog
#6
Updated by Marcelo Araujo almost 3 years ago
- Assignee changed from Marcelo Araujo to Release Council
- Severity set to New
Could we send it to middlewared team?
#7
Updated by Dru Lavigne almost 3 years ago
- Category changed from GUI (new) to Middleware
- Assignee changed from Release Council to William Grzybowski
#8
Updated by William Grzybowski almost 3 years ago
- Category changed from Middleware to GUI (new)
- Status changed from Not Started to Unscreened
- Assignee changed from William Grzybowski to Anonymous
Erin, can we get a warning added to new UI, please?
#9 Updated by Anonymous almost 3 years ago
- Status changed from Unscreened to Screened
#10 Updated by Anonymous almost 3 years ago
- Target version changed from Backlog to 11.2-RC2
#11 Updated by Anonymous almost 3 years ago
- Severity changed from New to Med High
#13
Updated by Ben Gadd almost 3 years ago
- Needs Automation changed from No to Yes
#14
Updated by Warren Block almost 3 years ago
Off the top of my head:
WARNING: Assigning a directory to Minio changes the permissions of that directory and every directory in it to minio:minio, overriding any previous permissions. Creating a separate dataset just for Minio is strongly recommended.
#15 Updated by Anonymous almost 3 years ago
- Status changed from Screened to In Progress
- Assignee changed from Anonymous to Vaibhav Chauhan
#16
Updated by Dru Lavigne almost 3 years ago
- Assignee changed from Vaibhav Chauhan to Anonymous
- Target version changed from 11.2-RC2 to 11.2-BETA2
#17 Updated by Anonymous almost 3 years ago
- Status changed from In Progress to Ready for Testing
#18
Updated by Dru Lavigne almost 3 years ago
- Subject changed from Add warning about Setting Minio Directory Changes Ownership to Add warning that setting the minio directory changes ownership
- Needs Merging changed from Yes to No
#19
Updated by Michael Reynolds over 2 years ago
- File tank-perms-2018-07-11.png tank-perms-2018-07-11.png added
- Status changed from Ready for Testing to Failed Testing
I was able to select a root dataset for the minio share location.
Did not get the warning
I re-entered the minio configuration, then got the warning about not selecting a root dataset.
Permissions on /mnt/tank were set to minio:minio
#21
Updated by Michael Reynolds over 2 years ago
Confirmed the behaviour I reported here
was able to record the attached video (skip to about 25 seconds in)
#22 Updated by Anonymous over 2 years ago
- Status changed from Failed Testing to In Progress
#23 Updated by Anonymous over 2 years ago
- Status changed from In Progress to Ready for Testing
#24
Updated by Michael Reynolds over 2 years ago
- Status changed from Ready for Testing to Passed Testing
- Needs QA changed from Yes to No
With a layout of tank/dataset selecting tank gives the warning
#25
Updated by Timothy Moore II over 2 years ago
- Needs Doc changed from Yes to No
Docs PRs:
[master branch] https://github.com/freenas/freenas-docs/pull/241
[angulargui branch] https://github.com/freenas/freenas-docs/pull/240
#26
Updated by Dru Lavigne over 2 years ago
- Status changed from Passed Testing to Done