Harold,
Yes. That would be helpful.
John
On 07/ 1/11 03:11 PM, Harold Shaw wrote:
Thanks, John. Do you mean include the name/description of the macro
as part of the output so that the system admin knows where in the dhcp
config file to look? If so, no problem.
Harold
On 07/ 1/11 03:12 PM, John Fischer wrote:
Harold,
Looks good. My only comment is that it might be nice to show the macro
that failed each time the config_file.write() has an exception as
this will allow
the system administrator a better clue why the failure occurred and what
might need hand translation.
Thanks,
John
On 07/ 1/11 12:42 PM, Harold Shaw wrote:
Code Review can be found at:
https://cr.opensolaris.org/action/browse/caiman/hshaw/7061636
On 07/ 1/11 01:40 PM, Harold Shaw wrote:
Please do a code review for the bugs listed below:
7061636 <http://monaco.us.oracle.com/detail.jsf?cr=7061636>
installadm-convert dhcp conversion traceback if clients specified
7061635 <http://monaco.us.oracle.com/detail.jsf?cr=7061635>
installadm-convert dhcp conversion needs to be hardened
7061620 <http://monaco.us.oracle.com/detail.jsf?cr=7061620>
installadm-convert should only say it's moving install.conf if it actually exists
The fix for the first two bugs has been tested on several different dhcp
configurations (some illegal). The last bug was tested on
a pre-ISIM system and verified that the message was displayed only for those
systems that had install.conf (SPARC) and that the file
had indeed been moved.
_______________________________________________
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
