Project

General

Profile

Feature #26157

Add warning that setting the minio directory changes ownership

Added by Phillip Smith almost 3 years ago. Updated about 2 years ago.

Status:
Done
Priority:
No priority
Assignee:
Erin Clark
Category:
GUI (new)
Target version:
11.2-BETA2
Estimated time:
Severity:
Med High
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

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?

tank-perms-2018-07-11.png (21.1 KB) tank-perms-2018-07-11.png Michael Reynolds, 07/11/2018 07:29 AM
minio-DatasetWarning2018-07-17.mkv (5.55 MB) minio-DatasetWarning2018-07-17.mkv Michael Reynolds, 07/17/2018 01:06 PM
20926

Associated revisions

Revision e16e7d18 (diff)
Added by Erin Clark about 2 years ago

Warn users when they try to pick a root dataset for S3 minio Ticket: #26157

Revision 852da635 (diff)
Added by Erin Clark about 2 years ago

Fix warning for first time setup of minio dataset Ticket: #26157

History

#1 Updated by Dru Lavigne almost 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 almost 3 years ago

  • Status changed from Unscreened to Screened
  • Target version changed from 11.1 to 11.2-BETA1

#3 Avatar?id=14398&size=24x24 Updated by Kris Moore almost 3 years ago

  • Target version changed from 11.2-BETA1 to 11.3

#4 Avatar?id=14398&size=24x24 Updated by Kris Moore over 2 years ago

  • Status changed from Screened to Not Started

#5 Avatar?id=13649&size=24x24 Updated by Ben Gadd over 2 years ago

  • Target version changed from 11.3 to Backlog

#6 Updated by Marcelo Araujo over 2 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 over 2 years ago

  • Category changed from GUI (new) to Middleware
  • Assignee changed from Release Council to William Grzybowski

#8 Updated by William Grzybowski over 2 years ago

  • Category changed from Middleware to GUI (new)
  • Status changed from Not Started to Unscreened
  • Assignee changed from William Grzybowski to Erin Clark

Erin, can we get a warning added to new UI, please?

#9 Updated by Erin Clark over 2 years ago

  • Status changed from Unscreened to Screened

#10 Updated by Erin Clark over 2 years ago

  • Target version changed from Backlog to 11.2-RC2

#11 Updated by Erin Clark over 2 years ago

  • Severity changed from New to Med High

#13 Avatar?id=13649&size=24x24 Updated by Ben Gadd over 2 years ago

  • Needs Automation changed from No to Yes

#14 Updated by Warren Block over 2 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 Erin Clark about 2 years ago

  • Status changed from Screened to In Progress
  • Assignee changed from Erin Clark to Vaibhav Chauhan

#16 Updated by Dru Lavigne about 2 years ago

  • Assignee changed from Vaibhav Chauhan to Erin Clark
  • Target version changed from 11.2-RC2 to 11.2-BETA2

#17 Updated by Erin Clark about 2 years ago

  • Status changed from In Progress to Ready for Testing

#18 Updated by Dru Lavigne about 2 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 about 2 years ago

20926

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 about 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 Erin Clark about 2 years ago

  • Status changed from Failed Testing to In Progress

#23 Updated by Erin Clark about 2 years ago

  • Status changed from In Progress to Ready for Testing

#24 Updated by Michael Reynolds about 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 about 2 years ago

  • Needs Doc changed from Yes to No

#26 Updated by Dru Lavigne about 2 years ago

  • Status changed from Passed Testing to Done

Also available in: Atom PDF