Closed Bug 396095 Opened 13 years ago Closed 13 years ago
.zip generation script (from pageloader/ directory)
This script takes the contents of mozilla/layout/tools/pageloader and creates a tp-component.zip archive containing correctly formatted chrome/ and components/ directories. This can be used to have Talos boxes be able to auto-update the pageloader when changes are checked in. If we don't want it all zipped up the script could be altered to simply create chrome/ and components/ directories. I'm not sure if this belongs in the pageloader directory or the talos code, or possibly the buildbot scripts.
This is a slightly modified version of your script that generates the chrome/ and component/ directories in the current working directory. This is what will be used on the Talos slaves.
This version generates the pageloader properly. I also added a few comments.
If run twice in a row you run into this problem: Traceback (most recent call last): File "buildpl.py", line 45, in ? os.mkdir(os.path.join(working_dir, 'chrome')) OSError: [Errno 17] File exists: './chrome' This would be because the chrome/component directories aren't cleaned up between runs. Will this affect functionality with buildbot? Is buildbot handling the cleanup?
Buildbot is handling the clean-up, though that should probably be handled more gracefully. I'll put up a nicer version tomorrow.
This version works exactly like the last one but bails if chrome/ or components/ exists in the working_dir. It will also bail if the cvs checkout fails.
Attachment #281306 - Flags: review?(anodelman) → review+
I keep missing this when I go through my bugs so I'm going to assign it to me.
Assignee: nobody → bhearsum
Here's an updated version that uses cvs-mirror by default. I'm proposing that we put it in talos/page_load_test. I think it belongs with Talos as it will be very useful for stand-alone Talos. Not to mention it makes the Buildbot configuration a bit shorter.
Comment on attachment 281753 [details] [diff] [review] same as before, with cvs-mirror.m.o and patchification Thanks for th cvs-mirror change. It will make it simpler for other users to generate chrome/component directories.
Attachment #281753 - Flags: review?(anodelman) → review+
This was checked in as part of bug 395819. RCS file: /cvsroot/mozilla/tools/buildbot-configs/testing/talos/perfmaster/scripts/generate-tpcomponent.py,v done Checking in scripts/generate-tpcomponent.py; /cvsroot/mozilla/tools/buildbot-configs/testing/talos/perfmaster/scripts/generate-tpcomponent.py,v <-- generate-tpcomponent.py initial revision: 1.1
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Mass move of Core:Testing bugs to mozilla.org:ReleaseEngineering. Filter on RelEngMassMove to ignore.
Component: Testing → Release Engineering
Product: Core → mozilla.org
QA Contact: testing → release
Version: Trunk → other
Product: mozilla.org → Release Engineering
You need to log in before you can comment on or make changes to this bug.