Project

General

Profile

Bug #55923

Umbrella #55899: Prevent Cloud Sync Tasks from overflowing /mnt

Make Services -> S3 -> Disks a required field

Added by Caleb St. John 9 months ago. Updated 7 months ago.

Status:
Done
Priority:
No priority
Assignee:
Waqar Ahmed
Category:
Services
Target version:
Seen in:
Severity:
High
Reason for Closing:
Reason for Blocked:
Needs QA:
No
Needs Doc:
No
Needs Merging:
No
Needs Automation:
No
Support Suite Ticket:
n/a
Hardware Configuration:
ChangeLog Required:
No

Description

If you configure the S3 service in the webUI, and you choose the "/mnt/zpool" directory and nothing more, winacl is triggered and it will recursively reset user/group ownership on all files and directories to minio/minio.

Screen Shot 2018-12-11 at 2.51.55 PM.png (10.9 KB) Screen Shot 2018-12-11 at 2.51.55 PM.png Prevented me from setting /mnt and the path Bonnie Follweiler, 12/11/2018 11:53 AM
Screen Shot 2018-12-11 at 2.50.48 PM.png (30.4 KB) Screen Shot 2018-12-11 at 2.50.48 PM.png The cloudsysnc was successful to /mnt/tank Bonnie Follweiler, 12/11/2018 11:55 AM
Screen Shot 2018-12-11 at 2.51.08 PM.png (19.3 KB) Screen Shot 2018-12-11 at 2.51.08 PM.png I detatched /mnt/tank Bonnie Follweiler, 12/11/2018 11:56 AM
Screen Shot 2018-12-11 at 2.51.34 PM.png (32.9 KB) Screen Shot 2018-12-11 at 2.51.34 PM.png The Cloudsync task failed Bonnie Follweiler, 12/11/2018 11:57 AM
44883
44892
44901
44910

Related issues

Copied to FreeNAS - Bug #56112: Make Services -> S3 -> Disks a required fieldDone
Copied to FreeNAS - Bug #60462: Make Services -> S3 -> Disks a required fieldDone

Associated revisions

Revision e3e9ac8b (diff)
Added by Waqar Ahmed 9 months ago

Make s3 disks field required

This commit makes s3 disks field in s3 service required for editing it's configuration.
Ticket: #55923

Revision 19b654ad (diff)
Added by Waqar Ahmed 9 months ago

Make s3 disks field required

This commit makes s3 disks field in s3 service required for editing it's configuration and ensures that top level datasets aren't selectable by the end user.
Ticket: #55923

Revision e3b6d2bd (diff)
Added by Waqar Ahmed 8 months ago

Make s3 disks field required

This commit makes s3 disks field in s3 service required for editing it's configuration and ensures that top level datasets aren't selectable by the end user.
Ticket: #55923

History

#1 Updated by Dru Lavigne 9 months ago

  • Assignee changed from Release Council to William Grzybowski

#2 Updated by Caleb St. John 9 months ago

  • Subject changed from cloud sync will recursively wipe all permissions starting at /mnt directory to S3 service will recursively wipe all permissions starting at /mnt directory
  • Description updated (diff)

#3 Updated by William Grzybowski 9 months ago

  • Assignee changed from William Grzybowski to Waqar Ahmed

#4 Updated by Waqar Ahmed 9 months ago

  • Status changed from Unscreened to In Progress

#5 Updated by Caleb St. John 9 months ago

  • Description updated (diff)

Updating original description to be more accurate with reality.

#6 Updated by Waqar Ahmed 9 months ago

  • Copied to Bug #56112: Make Services -> S3 -> Disks a required field added

#7 Updated by Caleb St. John 9 months ago

  • Subject changed from S3 service will recursively wipe all permissions starting at /mnt directory to S3 service will recursively wipe all permissions starting at /mnt/zpool

#8 Updated by Bug Clerk 9 months ago

  • Status changed from In Progress to Ready for Testing

#9 Updated by Dru Lavigne 8 months ago

  • Subject changed from S3 service will recursively wipe all permissions starting at /mnt/zpool to Make Services -> S3 -> Disks a required field
  • Needs Doc changed from No to Yes
  • Needs Merging changed from Yes to No

#10 Updated by Dru Lavigne 8 months ago

  • Copied to Bug #60462: Make Services -> S3 -> Disks a required field added

#12 Updated by Bonnie Follweiler 8 months ago

  • Status changed from Ready for Testing to Passed Testing
  • Needs QA changed from Yes to No

Test Passed TrueNAS-11.1-U6.2-INTERNAL2

#13 Updated by Dru Lavigne 7 months ago

  • Status changed from Passed Testing to Done
  • Needs Doc changed from Yes to No

Also available in: Atom PDF