Bug 86789
| Summary: | We should try to download the built product before other steps | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Lucas Forschler <lforschler> |
| Component: | Tools / Tests | Assignee: | Lucas Forschler <lforschler> |
| Status: | RESOLVED WONTFIX | ||
| Severity: | Normal | CC: | ossy, rniwa, slewis, tony |
| Priority: | P2 | ||
| Version: | 528+ (Nightly build) | ||
| Hardware: | Unspecified | ||
| OS: | Unspecified | ||
Lucas Forschler
Currently our test bots do an svn operation to sync up source, and then downloads the built product to test. Normally this is fine. However when bots get behind, the build product has been removed due to space constraints. In this scenario, when the self.addStep(DownloadBuiltProduct) runs, the file does not exist, and the rest of the of the steps are aborted.
We should put the DownloadBuildProduct step BEFORE the svn operation, such that we can short circuit the svn operation which takes a fair amount of time.
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
Ryosuke Niwa
(In reply to comment #0)
> Currently our test bots do an svn operation to sync up source, and then downloads the built product to test. Normally this is fine. However when bots get behind, the build product has been removed due to space constraints. In this scenario, when the self.addStep(DownloadBuiltProduct) runs, the file does not exist, and the rest of the of the steps are aborted.
>
> We should put the DownloadBuildProduct step BEFORE the svn operation, such that we can short circuit the svn operation which takes a fair amount of time.
Hm... there is a slight problem with this, which is that sometimes "svn up" and failing would remove the entire checkout. That may prune the build files as well. Or maybe that's rare enough event that we can just ignore it?
Stephanie Lewis
If he writes this correctly it should never even start the svn step. I don't think it would remove itself in that case.
Lucas Forschler
I think we'd need an extra factory. At first, I thought we could simply move the addStep(DownloadBuiltProduct) from TestFactory to Factory. However, then it would run every time the base Factory class was called. (Not just TestFactory).
Also, I do know we are getting more storage in the next month or so, which should alleviate this problem. However, we should be smart in software and not rely on the storage (or lack of) to 'do the right thing.'
Stephanie Lewis
Nevermind, I see what you're getting at. Maybe we should download to someplace that is not in the svn checkout?
Lucas Forschler
I'm not sure this is relevant any longer... svn operations are fairly quick, except for new checkouts... I think we should just close it.