Today I discovered that the dispatcher is still not deleting all its temp files. The current hole is demonstrated by our kernel CI loop. Basically, we aren't cleaning up temp files for ubuntu builds that send a hwpack plus RFS combo.
To clarify, there are 3 types of deployments the dispatcher must support: 1) pre-built images 2) android images 3) hwpack + rfs
Version 0.15.1 only cleans up files for 1 and 2. I spent some time today looking at the core problem which turned out to be an inconsistent approach to managing files the dispatcher creates. I created a common approach, moved code to use it, and then tested to make sure 1,2,and 3 work properly.
I now have an MP:
https://code.launchpad.net/~doanac/lava-dispatcher/tempdir-madness/+merge/116141
That should fix all of this. I realize its the weekend, but I'd like to get this in to our 12.07 release which is targeted for Monday. Could someone take a sanity check? In the end, its better than what we have now
-andy
Hi,
I found another failure: https://bugs.launchpad.net/lava-dispatcher/+bug/1027906, in atexit.
On 22 July 2012 12:57, Andy Doan andy.doan@linaro.org wrote:
Today I discovered that the dispatcher is still not deleting all its temp files. The current hole is demonstrated by our kernel CI loop. Basically, we aren't cleaning up temp files for ubuntu builds that send a hwpack plus RFS combo.
To clarify, there are 3 types of deployments the dispatcher must support:
- pre-built images
- android images
- hwpack + rfs
Version 0.15.1 only cleans up files for 1 and 2. I spent some time today looking at the core problem which turned out to be an inconsistent approach to managing files the dispatcher creates. I created a common approach, moved code to use it, and then tested to make sure 1,2,and 3 work properly.
I now have an MP:
<https://code.launchpad.net/~**doanac/lava-dispatcher/** tempdir-madness/+merge/116141https://code.launchpad.net/%7Edoanac/lava-dispatcher/tempdir-madness/+merge/116141
That should fix all of this. I realize its the weekend, but I'd like to get this in to our 12.07 release which is targeted for Monday. Could someone take a sanity check? In the end, its better than what we have now
-andy
______________________________**_________________ linaro-validation mailing list linaro-validation@lists.**linaro.org linaro-validation@lists.linaro.org http://lists.linaro.org/**mailman/listinfo/linaro-**validationhttp://lists.linaro.org/mailman/listinfo/linaro-validation
linaro-validation@lists.linaro.org