Closed
Bug 1434281
Opened 3 years ago
Closed 3 years ago
test_refresh_firefox.py doesn't clean-up cache folder of created profile
Categories
(Firefox :: Migration, defect, P3)
Tracking
()
RESOLVED
FIXED
Firefox 63
Tracking | Status | |
---|---|---|
firefox63 | --- | fixed |
People
(Reporter: gcp, Assigned: Gijs)
References
Details
Attachments
(1 file)
https://bugzilla.mozilla.org/show_bug.cgi?id=1386404#c104 During marionette testruns, it seems the .cache/mozilla directory is not properly cleaned up, and this causes that directory to accumulate a large number of profile-subdirectories, each (for example) containing SafeBrowsing and cache2 data. Cleaning out this directory makes some tests pass that are orange due to timeouts otherwise.
Reporter | ||
Updated•3 years ago
|
Comment 1•3 years ago
|
||
So by default the cache should be placed under the profile directory. That is what usually happens for mozprofile created profiles in use by Marionette. Is that different for the refresh firefox test, which is failing in your case? Also with XDG_CACHE_HOME set to a different folder, mozprofile would have to know the exact path of the cache. Gian-Carlo can you please check if the folder name matches the one from the profile itself?
Flags: needinfo?(gpascutto)
Updated•3 years ago
|
Summary: Profile dir in cachedir not cleaned up, accumulates during testruns → [mozprofile] Profile dir in cachedir not cleaned up, accumulates during testruns
Reporter | ||
Comment 2•3 years ago
|
||
>So by default the cache should be placed under the profile directory. The way this is worded is a bit suspect. There is a .cache/XDG_CACHE_HOME directory on every system. Inside this directory, we make a per-profile specific subdir for various caches, that has got nothing to do with what is regularly considered "the profile". >Is that different for the refresh firefox test, which is failing in your case? No. About the only specific thing that test does is have two profiles alive at once during the test. >Gian-Carlo can you please check if the folder name matches the one from the profile itself? I don't really understand what you're asking here.
Flags: needinfo?(gpascutto)
![]() |
||
Updated•3 years ago
|
Priority: -- → P3
Comment 3•3 years ago
|
||
(In reply to Gian-Carlo Pascutto [:gcp] from comment #2) > >So by default the cache should be placed under the profile directory. > > The way this is worded is a bit suspect. There is a .cache/XDG_CACHE_HOME > directory on every system. Inside this directory, we make a per-profile > specific subdir for various caches, that has got nothing to do with what is > regularly considered "the profile". Maybe this is not the case because the profile gets created at a temporary location and not withing app data? For each profile which is created via mozprofile the subfolders like `OfflineCache`, `Cache2` etc are always located under the created profile in %tmp%. I don't know where the decision is made which path to use. > >Gian-Carlo can you please check if the folder name matches the one from the profile itself? > > I don't really understand what you're asking here. Maybe the above explains a bit more, and might be the underlying reason why XDG_CACHE_HOME is not honored.
Flags: needinfo?(gpascutto)
Reporter | ||
Comment 4•3 years ago
|
||
>For each profile which is created via mozprofile the subfolders like `OfflineCache`, `Cache2` etc are always located under the created profile in %tmp% As explained above, this does not appear to be the case, see also: https://bugzilla.mozilla.org/show_bug.cgi?id=1386404#c104
Flags: needinfo?(gpascutto)
Comment 5•3 years ago
|
||
(In reply to Gian-Carlo Pascutto [:gcp] from comment #4) > As explained above, this does not appear to be the case, see also: > https://bugzilla.mozilla.org/show_bug.cgi?id=1386404#c104 This is a temporary profile which gets created by Firefox but only for this specific test. In all other cases we use mozprofile to create the user profile, which then have the cache files located in itself. So the test test_refresh_firefox.py should then also clean-up the cache folder during teardown. Gian-Carlo, can you please check that no other test beside the above leaves traces behind? If that is the case we should move this bug to Firefox / Migration.
Flags: needinfo?(gpascutto)
Reporter | ||
Comment 6•3 years ago
|
||
>Gian-Carlo, can you please check that no other test beside the above leaves traces behind?
I only found this case because it was causing a test to go orange on a (mostly) not related patch, and I stumbled upon this bug while inspecting a loaner. I have no idea how to do a generic check here?
Flags: needinfo?(gpascutto)
Comment 7•3 years ago
|
||
Ok, so this is clearly a problem with `test_refresh_firefox.py` and that it creates the new to migrate to profile on its own. Beside the profile folder it clearly should remove the cache folder too. Gijs could you or someone else please have a look at this? Thanks
Component: Mozbase → Migration
Flags: needinfo?(gijskruitbosch+bugs)
Product: Testing → Firefox
Summary: [mozprofile] Profile dir in cachedir not cleaned up, accumulates during testruns → test_refresh_firefox.py doesn't clean-up cache folder of created profile
Version: Version 3 → unspecified
Assignee | ||
Updated•3 years ago
|
Flags: needinfo?(gijskruitbosch+bugs)
Assignee | ||
Updated•3 years ago
|
Assignee: nobody → gijskruitbosch+bugs
Status: NEW → ASSIGNED
Assignee | ||
Comment 8•3 years ago
|
||
Ensure local profile directory is also deleted in test teardown.
Comment 9•3 years ago
|
||
Comment on attachment 8997086 [details] Bug 1434281 - also delete local/cache profile directory from refresh test, r?whimboo Henrik Skupin (:whimboo) has approved the revision. https://phabricator.services.mozilla.com/D2671
Attachment #8997086 -
Flags: review+
Comment 10•3 years ago
|
||
Pushed by gijskruitbosch@gmail.com: https://hg.mozilla.org/integration/autoland/rev/1a0c506d69a8 also delete local/cache profile directory from refresh test, r=whimboo
Comment 11•3 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/1a0c506d69a8
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
status-firefox63:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 63
You need to log in
before you can comment on or make changes to this bug.
Description
•