Closed Bug 722202 Opened 11 years ago Closed 8 years ago

Stop doing a recursive chmod on directories every time we upload to a candidates dir


(Release Engineering :: Release Automation: Other, defect, P2)



(Not tracked)



(Reporter: nthomas, Assigned: nthomas)



(Whiteboard: [release][automation])


(2 files)

Bug 484972 added a recursive chmod on dirs, to make sure the permissions were set safely on all candidate sub-directories. But didn't think about performance at all in the face of a busy netapp. Now you upload a log (while a virus scan is running amongst multiple other things), and the post_upload bogs down doing chmod that it needn't.
Assignee: nobody → coop
Priority: -- → P3
Hardware: x86 → All
Lets get this done asap to try to dig stage out of it's hole.
Assignee: coop → nrthomas
Priority: P3 → P2
As far as we can tell this code isn't necessary, because dirs are created 755 and files set to 644, but we will find out when we do the next release when check_permissions runs.
Attachment #595621 - Flags: review?(catlee)
Attachment #595621 - Flags: review?(catlee) → review+
Comment on attachment 595621 [details] [diff] [review]
[tools] Disable chmod for release to candidates
Attachment #595621 - Flags: checked-in+
Deployed to stage.m.o.
I think we have a problem with this on bug 729998.
We don't know though why it would show up for beta4 rather than beta 2 & 3 as well.
Duplicate of this bug: 733726
Mass move of bugs to Release Automation component.
Component: Release Engineering → Release Engineering: Automation (Release Automation)
Flags: checked-in+
No longer blocks: hg-automation
Product: → Release Engineering
Looks like we still do this for mobile candidates directories:
Only 79 dirs in a mobile candidate dir vs 1500+ for desktop so not as big of a problem. Here's the patch for completeness anyway.
Attachment #8547413 - Flags: review?(bhearsum)
Attachment #8547413 - Flags: review?(bhearsum) → review+
Comment on attachment 8547413 [details] [diff] [review]
[tools] Remove dirs chmod for mobile candidates too

Canonical copy:

Deployed using IT puppet:
Committed revision 99145.
Closed: 8 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.