727488 - [cb-seamonkey-win32-0x, sea-win32-0x] Intermittent/permanent purple: "No such file or directory: '/e/builds/slave/test/minidumps'"

Status: RESOLVED WORKSFORME

(SeaMonkey :: Release Engineering, defect, P2)

Description

[Serge Gautherie (:sgautherie)]

I had noticed this recently: confirming.

http://tinderbox.mozilla.org/showlog.cgi?log=SeaMonkey/1328972889.1328974569.19617.gz
WINNT 5.2 comm-central-trunk debug test mochitest-other on 2012/02/11 07:08:09
s: cb-seamonkey-win32-03
+
http://tinderbox.mozilla.org/showlog.cgi?log=SeaMonkey/1329149723.1329151362.1676.gz
WINNT 5.2 comm-central-trunk debug test mochitest-other on 2012/02/13 08:15:23
s: cb-seamonkey-win32-01
+
http://tinderbox.mozilla.org/showlog.cgi?log=SeaMonkey/1329252901.1329254569.6196.gz&fulltext=1
WINNT 5.2 comm-central-trunk debug test mochitest-other on 2012/02/14 12:55:01
s: cb-seamonkey-win32-02
{
INFO | runtests.py | Received unexpected exception while running application '[Errno 2] No such file or directory: '/e/builds/slave/test/minidumps''
}

Comment 1

[Serge Gautherie (:sgautherie)]

http://tinderbox.mozilla.org/showlog.cgi?log=SeaMonkey/1341379736.1341382775.27840.gz
WINNT 5.2 comm-central-trunk debug test mochitest-other on 2012/07/03 22:28:56
s: sea-win32-01
{
PROCESS-CRASH | chrome://mochitests/content/browser/suite/common/places/tests/browser_0_library_left_pane_migration.js | application crashed (minidump found)
Crash dump filename: c:\docume~1\seabld\locals~1\temp\tmpglor4c\minidumps\c11ae747-81e1-4557-b9b2-755835bca9fe.dmp
...
INFO | runtests.py | Received unexpected exception while running application
Traceback (most recent call last):
  File "mochitest/runtests.py", line 697, in runTests
    timeout = timeout)
  File "e:\builds\slave\test\build\mochitest\automation.py", line 908, in runApp
    self.checkForCrashes(profileDir, symbolsPath)
  File "e:\builds\slave\test\build\mochitest\automation.py", line 840, in checkForCrashes
    automationutils.checkForCrashes(os.path.join(profileDir, "minidumps"), symbolsPath, self.lastTestSeen)
  File "e:\builds\slave\test\build\mochitest\automationutils.py", line 195, in checkForCrashes
    shutil.move(d, dumpSavePath)
  File "d:\mozilla-build\python25\lib\shutil.py", line 199, in move
    copy2(src,dst)
  File "d:\mozilla-build\python25\lib\shutil.py", line 91, in copy2
    copyfile(src, dst)
  File "d:\mozilla-build\python25\lib\shutil.py", line 47, in copyfile
    fdst = open(dst, 'wb')
IOError: [Errno 2] No such file or directory: '/e/builds/slave/test/minidumps'
}

This looks like a path issue.

NB: Currently perma-purple, due to bug 770687.

Comment 3

[Edmund Wong (:ewong)]

Attachment: Use Windows style paths when building on Windows.

Comment 4

[Justin Wood (:Callek)]

Comment on attachment 651097 [details] [diff] [review]
Use Windows style paths when building on Windows.

Pretty sure thats more complex than i feel comfortable with. Would also love either bhearsum or catlee to look at next iteration or speak their thoughts

Comment 5

[Edmund Wong (:ewong)]

Attachment: Use Windows style paths when building on Windows. (v2)

Comment 6

[[e-mail] (:bhearsum)]

Comment on attachment 651246 [details] [diff] [review]
Use Windows style paths when building on Windows. (v2)

Review of attachment 651246 [details] [diff] [review]:
-----------------------------------------------------------------

This isn't an issue for our masters, so please don't land it anywhere except the seamonkey branch - we recently ejected most of the SeaMonkey-specific code, and aren't looking to add more back in.

::: process/factory.py
@@ +431,5 @@
>          ))
>          self.addInitialSteps()
>  
>      def addInitialSteps(self):
> +        if self.platform and self.platform.startswith('WIN'):

Our platforms are lower cased, so this definitely won't work.

Comment 7

[Edmund Wong (:ewong)]

(In reply to Ben Hearsum [:bhearsum] from comment #6)
> Comment on attachment 651246 [details] [diff] [review]
> Use Windows style paths when building on Windows. (v2)
> 
> Review of attachment 651246 [details] [diff] [review]:
> -----------------------------------------------------------------
> 
> This isn't an issue for our masters, so please don't land it anywhere except
> the seamonkey branch - we recently ejected most of the SeaMonkey-specific
> code, and aren't looking to add more back in.

Sorry.  I forgot to mention in my feedback request that I was looking
more on the coding merit rather than where it's landing.  When ok,
the patch will only be landing on the seamonkey-production side of things.

Thanks for the feedback.

Comment 8

[Serge Gautherie (:sgautherie)]

(In reply to Ben Hearsum [:bhearsum] from comment #6)
> This isn't an issue for our masters

Edmund (or Ben), I wonder what the "underlying" cause of the SeaMonkey symptom is (compared to MoCo masters)...

Comment 9

[Edmund Wong (:ewong)]

Attachment: Use Windows style paths when building on Windows. (v3)

This is for seamonkey-production branch.

Comment 10

[Justin Wood (:Callek)]

Comment on attachment 653172 [details] [diff] [review]
Use Windows style paths when building on Windows. (v3)

Review of attachment 653172 [details] [diff] [review]:
-----------------------------------------------------------------

I'm nervous about switching this to a win-style, so I'm going to push this a bit lower on my patch-priority-list

Comment 11

[Edmund Wong (:ewong)]

Comment on attachment 653172 [details] [diff] [review]
Use Windows style paths when building on Windows. (v3)

patch is severely bitrotted.

Comment 12

[Edmund Wong (:ewong)]

Closing due to being irrelevant as the machine is now gone.