Bug 43062

Summary: [chromium] cleanup temp files left by Linux DRT
Product: WebKit Reporter: Tony Chang <tony>
Component: New BugsAssignee: Tony Chang <tony>
Status: RESOLVED FIXED    
Severity: Normal CC: dglazkov, levin, wsiegrist
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Other   
OS: OS X 10.5   
Attachments:
Description Flags
Patch
none
Patch
none
Patch for landing
none
Patch for landing none

Description Tony Chang 2010-07-27 11:10:12 PDT
[chromium] cleanup temp files left by Linux DRT
Comment 1 Tony Chang 2010-07-27 11:10:34 PDT
Created attachment 62713 [details]
Patch
Comment 2 Tony Chang 2010-07-27 11:11:58 PDT
There doesn't seem to be a way to tell the Chromium OS, so I used perl.  Also, I'm not sure how to test this.  Specifically, I'm not sure if I need to escape some parameters (when I run it from the command line, I need to escape the forward ticks and the $ in $OSNAME).
Comment 3 Tony Chang 2010-07-27 11:12:58 PDT
Oh, and for reference, these temp files are only left around by DRT processes that crash.  Except right now, we have lots of crashing DRT processes.
Comment 4 David Levin 2010-07-27 16:58:21 PDT
(In reply to comment #2)
> There doesn't seem to be a way to tell the Chromium OS, so I used perl.  

Do any of the properties in ConfigureBuild (like platform, fullPlatform, etc.) give you this information?
Comment 5 Tony Chang 2010-07-27 17:00:33 PDT
(In reply to comment #4)
> (In reply to comment #2)
> > There doesn't seem to be a way to tell the Chromium OS, so I used perl.  
> 
> Do any of the properties in ConfigureBuild (like platform, fullPlatform, etc.) give you this information?

They don't currently since we name all of our machines "chromium".  We could rename them to chromium-win, chromium-mac and chromium-linux.  Perhaps that's a bit cleaner.  I will upload a new patch with that.
Comment 6 Tony Chang 2010-07-27 17:31:12 PDT
Created attachment 62777 [details]
Patch
Comment 7 Kent Tamura 2010-07-27 20:15:44 PDT
Comment on attachment 62777 [details]
Patch

We need to update WebKitTools/BuildSlaveSupport/test-result-archive too though it doesn't work for chromium at this moment.
Comment 8 Tony Chang 2010-07-28 09:38:31 PDT
(In reply to comment #7)
> (From update of attachment 62777 [details])
> We need to update WebKitTools/BuildSlaveSupport/test-result-archive too though it doesn't work for chromium at this moment.

I'm not sure, but I think it'll be ok.  The %(platform) property should be "chromium" (see the split on line 29 of master.cfg).  build-product-archive doesn't seem to special case things like mac-tiger or mac-leopard.

We might be able to use properties in some other places in master.cfg, but it's hard for me to tell which classes I can and which classes I can't use them.
Comment 9 Kent Tamura 2010-07-28 09:53:25 PDT
Comment on attachment 62777 [details]
Patch

ok.
Comment 10 Tony Chang 2010-07-28 09:55:56 PDT
wms: Is there a way for me to run a test buildbot master+slave to test this change before landing it?
Comment 11 William Siegrist 2010-07-28 09:59:09 PDT
Yes, use the master.cfg from the svn repo and  v0.7.12 of buildbot. You'll need to come up with a passwords file, but the master.cfg and config.json should make it obvious what to do with it.
Comment 12 Tony Chang 2010-07-28 12:10:25 PDT
Created attachment 62857 [details]
Patch for landing
Comment 13 Tony Chang 2010-07-28 12:19:59 PDT
Created attachment 62858 [details]
Patch for landing
Comment 14 Tony Chang 2010-07-28 12:24:05 PDT
Committed r64219: <http://trac.webkit.org/changeset/64219>
Comment 15 Tony Chang 2010-07-28 13:26:28 PDT
I was able to test on my machine and this should be working now. wms, can you schedule a master restart?
Comment 16 William Siegrist 2010-07-29 08:42:39 PDT
Graceful restart process has been begun.
Comment 17 Tony Chang 2010-07-30 09:20:13 PDT
(In reply to comment #16)
> Graceful restart process has been begun.

Did the restart happen?  I was expecting to see a change to Chromium Linux Release (Tests) but I don't see it right now.
Comment 18 Tony Chang 2010-07-30 09:21:24 PDT
(In reply to comment #17)
> (In reply to comment #16)
> > Graceful restart process has been begun.
> 
> Did the restart happen?  I was expecting to see a change to Chromium Linux Release (Tests) but I don't see it right now.

Ah, crap, the diff that was landed is missing a change.  I probably screwed up a merge.
Comment 19 William Siegrist 2010-07-30 09:23:20 PDT
The restart hasn't happened yet. It is still waiting for all of the slaves to be idle.
Comment 20 Tony Chang 2010-07-30 09:25:25 PDT
(In reply to comment #19)
> The restart hasn't happened yet. It is still waiting for all of the slaves to be idle.

Ah, ok. Thanks for the update.  It'll probably get a chance this weekend to restart.  Turns out I was mistaken, the change includes everything that I tested.