Hi Ethan,
Thanks for pointing that out. I didn't notice the comments in suggested fix
field, but now that I see this, that 2nd error is very obvious in the
description.
I'll look into this and send out another webrev.
Thanks,
Martin
On 8/16/2011 10:58 AM, Ethan Quach wrote:
Hey Martin,
In the suggested fix, it notes that the other exception shown in the description
File "/usr/bin/auto-install", line 40, in <module>
sys.exit(ai.AI_EXIT_FAILURE)
NameError: name 'ai' is not defined
is coming from the auto-install.py file because of where it's trying to get the
AI_EXIT_FAILURE value from. Can you fix that bit with this as well?
thanks,
-ethan
On 08/15/11 23:46, Martin Widjaja wrote:
Hi all,
Can I request at least 2 reviewers for this quite simple fix for bug 7066541?
*7066541 installing several zones in parallel sometimes fail: [Errno 17] File exists:
'/var/tmp/install' <http://monaco.sfbay.sun.com/detail.jsf?cr=7066541>*
Webrev is here:
https://cr.opensolaris.org/action/browse/caiman/widjaja/7066541/webrev/
I don't have much luck trying to create the issue listed in the CR during
multiple zone installs in parallel. I have tried many methods, including the
use of a script to run 5 zone installs, and have executed this over and over,
but so far I have not been able to create the race condition (system too
slow?). At this point I am requesting the filer to help test since he seems to
be able to produce the race condition much more easily and consistently.
Good news is I've tested this multiple times, and no regression is seen so far.
Additionally, I've created a slightly different version for detecting the race
condition during testing by adding an else/print statement corresponding to the
if statement on line 258.
That version of the file can be copied from
10.132.145.50:/export/home/mwidjaja/logger.py.
If anyone has a better idea on producing a much more frequent and consistent
race condition for a parallel zone installs, please let me know.
Thanks,
Martin
_______________________________________________
caiman-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/caiman-discuss
_______________________________________________
caiman-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/caiman-discuss
_______________________________________________
caiman-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/caiman-discuss
