Bug 43062 – [chromium] cleanup temp files left by Linux DRT

• Status: RESOLVED
• Resolution: FIXED
• Component: New Bugs

[Tony Chang]

[chromium] cleanup temp files left by Linux DRT

[Tony Chang]

Created attachment 62713 [details]
Patch

[Tony Chang]

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).

[Tony Chang]

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.

[David Levin]

(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?

[Tony Chang]

(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.

[Tony Chang]

Created attachment 62777 [details]
Patch

[Kent Tamura]

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.

[Tony Chang]

(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.

[Kent Tamura]

Comment on attachment 62777 [details]
Patch

ok.

[Tony Chang]

wms: Is there a way for me to run a test buildbot master+slave to test this change before landing it?

[William Siegrist]

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.

[Tony Chang]

Created attachment 62857 [details]
Patch for landing

[Tony Chang]

Created attachment 62858 [details]
Patch for landing

[Tony Chang]

Committed r64219: <http://trac.webkit.org/changeset/64219>

[Tony Chang]

I was able to test on my machine and this should be working now. wms, can you schedule a master restart?

[William Siegrist]

Graceful restart process has been begun.

[Tony Chang]

(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.

[Tony Chang]

(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.

[William Siegrist]

The restart hasn't happened yet. It is still waiting for all of the slaves to be idle.

[Tony Chang]

(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.