Closed Bug 521396 Opened 15 years ago Closed 13 years ago

[exit] Last run_dir in mozmill-restart creates a new unnecessary profile

Categories

(Testing Graveyard :: Mozmill, defect)

Product:
Testing Graveyard
Component:
Mozmill
Type:
defect
Priority:
Not set
Severity:
major

Tracking

(Not tracked)

Status:
RESOLVED DUPLICATE of bug 584464

People

(Reporter: whimboo, Assigned: k0scist)

References

Details

(Whiteboard: [mozmill-2.0+]) 

Running any restart tests with Mozmill will create a new profile after the last execution of run_dir. That's not necessary and should only happen if another dir is still in the queue.

http://github.com/mikeal/mozmill/blob/c1eff61ffe060de5513fd38b5ae6664bfba352cc/mozmill/__init__.py#L284

That way the profile manager is flooded with temporary profiles. Mikeal, can you please fix it? I tried but all the stuff is a bit too complicated and I don't know where profile is used all over the place. Thanks.
Lets add bug 508643 just as reference because this change made it visible.
Blocks: 508643
An interim fix has been landed:
http://github.com/mikeal/mozmill/commit/0c41edaae2fb9a6d7d999b0651fa10f41967b21e

For a final fix we should only create a new profile when more tests are in the queue for the current run.
Whiteboard: [mozmill-2.0?]
Summary: Last run_dir in mozmill-restart creates a new unnecessary profile → [exit] Last run_dir in mozmill-restart creates a new unnecessary profile
Assignee: nobody → jhammel
See Also: → 617790
Depends on: 617790
This is going to be fixed by bug 584464

DUPE
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → DUPLICATE
Whiteboard: [mozmill-2.0?] → [mozmill-2.0+]
Product: Testing → Testing Graveyard
You need to log in before you can comment on or make changes to this bug.