Closed Bug 883526 Opened 12 years ago Closed 12 years ago

mozharness's rmtree_windows is spammy

Categories

(Release Engineering :: Applications: MozharnessCore, defect)

Product:
Release Engineering
Component:
Applications: MozharnessCore
Platform:
x86_64
Linux
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED FIXED

People

(Reporter: catlee, Unassigned)

Details

(Whiteboard: [mozharness])

Attachments

(1 file)

move rmtree_windows logging
1.54 KB, patch
mozilla
: review+
catlee
: checked-in+
Details | Diff | Splinter Review
The new implementation of rmtree_windows works, and is very vocal about it: https://tbpl.mozilla.org/php/getParsedLog.php?id=24189352&tree=Mozilla-Inbound 02:58:37 INFO - Using _rmtree_windows ... 02:58:37 INFO - Using _rmtree_windows ... 02:58:38 INFO - Using _rmtree_windows ... 02:58:38 INFO - Using _rmtree_windows ... 02:58:38 INFO - Using _rmtree_windows ... 02:58:38 INFO - Using _rmtree_windows ... 02:58:38 INFO - Using _rmtree_windows ... 02:58:38 INFO - Using _rmtree_windows ... 02:58:38 INFO - Using _rmtree_windows ... 02:58:38 INFO - Using _rmtree_windows ... 02:58:38 INFO - Using _rmtree_windows ... 02:58:38 INFO - Using _rmtree_windows ... 02:58:38 INFO - Using _rmtree_windows ... 02:58:38 INFO - Using _rmtree_windows ... 02:58:38 INFO - Using _rmtree_windows ... 02:58:38 INFO - Using _rmtree_windows ... 02:58:38 INFO - Using _rmtree_windows ... 02:58:38 INFO - Using _rmtree_windows ... 02:58:38 INFO - Using _rmtree_windows ... 02:58:39 INFO - Using _rmtree_windows ... 02:58:39 INFO - Using _rmtree_windows ... 02:58:39 INFO - Using _rmtree_windows ... 02:58:39 INFO - Using _rmtree_windows ... 02:58:39 INFO - Using _rmtree_windows ... We could use a flag here to only log this once per invocation? Or move the logging call up into rmtree() http://hg.mozilla.org/build/mozharness/file/ccaaa9e18f86/mozharness/base/script.py#l131
Attachment #763110 - Flags: review?(aki)
Attachment #763110 - Flags: review?(aki) → review+
Attachment #763110 - Flags: checked-in+
In production.
This looks to be fixed.
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Product: mozilla.org → Release Engineering
Component: General Automation → Mozharness
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: