Bug 1558507
Summary: | Gluster allows renaming of folders, which contain WORMed/Retain or WORMed files | ||
---|---|---|---|
Product: | [Community] GlusterFS | Reporter: | david.spisla |
Component: | core | Assignee: | Vishal Pandey <vpandey> |
Status: | CLOSED WONTFIX | QA Contact: | |
Severity: | medium | Docs Contact: | |
Priority: | medium | ||
Version: | mainline | CC: | atumball, bugs, ksubrahm, vpandey |
Target Milestone: | --- | ||
Target Release: | --- | ||
Hardware: | Unspecified | ||
OS: | Unspecified | ||
Whiteboard: | |||
Fixed In Version: | Doc Type: | If docs needed, set a value | |
Doc Text: | Story Points: | --- | |
Clone Of: | Environment: | ||
Last Closed: | 2019-08-27 12:39:46 UTC | Type: | Bug |
Regression: | --- | Mount Type: | --- |
Documentation: | --- | CRM: | |
Verified Versions: | Category: | --- | |
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
Cloudforms Team: | --- | Target Upstream Version: | |
Embargoed: |
Description
david.spisla
2018-03-20 11:18:04 UTC
Hi David, I know you sent couple of patches previously for WORM, and we did fix issues like delete etc. Is this still an issue in latest master? Hello Amar, yes, it is still an issue. If a folder (or a subfolder of this folder) contains a WORMed file, it shouldn't be allowed to rename the folder Had a discussion with Karthik and he is suspicious of why we need this change. Few reasons why this change might not be needed - (In reply to Vishal Pandey from comment #3) > Had a discussion with Karthik and he is suspicious of why we need this > change. Few reasons why this change might not be needed - 1- Rename will not change the contents or metadata of the files inside src directory 2- Rename will not change the xattrs specific to WORM feature on the files 3- The dir can contain other files as well which may not be WORM-Retained or WORMed yet 4- Also even if we do check a directory if it has worm files or no, it will take a long time in case when the number of files are too large. I have mentioned some of the reasons why this feature might not be needed afetr some discussions with Karthik. If there is anything else anyone would like to add or else can we close the issue ? I followed the discussion and I agree to the arguments. You can close the issue from my point of view Thanks for the ack David. Closing this bug as per comment #7. |