Sue,

Thanks for talking to me about this on IRC.  If /etc/netboot/<IP> is a file
then the os.dirlist() will throw an exception.  Thus the try/except version
of this fix is what needs to be implemented.

Thanks for catching that Sue.

John

On 02/23/11 03:49 PM, Susan Sohn wrote:
On 02/23/11 03:04 PM, John Fischer wrote:
All,

Sue was working on the new webserver and found another bug within installadm list
related to the bug fix for CR 7013975. The new CR is:

http://monaco.sfbay.sun.com/detail.jsf?cr=7021903

And will eventually be available for viewing at:

http://bugs.opensolaris.org/bugdatabase/view_bug.do?bug_id=7021903

The webrev is available at:

http://cr.opensolaris.org/~johnfisc/7021903-list-sparc-clients/

The issue occurs because the server does not have any older sparc clients. Thus when the code attempts to access the compatibility directory the exception is caught and the function returns with a blank dictionary. The solution is to rearrange the try/except block so that it catches the blank directory and continues
to the next path within the list instead of exiting.

Thanks,

John
_______________________________________________
caiman-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/caiman-discuss

Hi John,

Rather than doing the try/except, might it be better to continue if path doesn't exist and only get the dirlist if the path does exist?

Sue

_______________________________________________
caiman-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/caiman-discuss

Reply via email to