Sounds good, I just wanted to be sure that it was taken into account. Thanks,
Darren. On Fri Sep 23 16:40:22 2011, Ethan Quach wrote: > > > On 09/23/11 08:22, Darren Kenny wrote: >> On 23/09/2011 16:07, Ethan Quach wrote: >>> >>> On 09/23/11 01:38, Darren Kenny wrote: >>>> Hi Ethan, >>>> >>>> In a recent bugfix (7045369) Mary noticed that the reason for the warning >>>> from >>>> wget was that the hostname, not the IP address was included in the URLs for >>>> downloading, e.g: >>>> >>>> http://labtech-mpk-ai1:5555/export/aiserver/164-sparc//solaris.zlib >>>> >>>> This occurred only where the AI server was multi-homed. >>>> >>>> It is possible that this could also occur in this case? I'm not sure what >>>> the >>>> contents of the wanboot.conf file was in this case, but I'd like to >>>> double-check. >>> When the AI server is setup to serve multiple networks, the contents of >>> wanboot.conf will have something like: >>> >>> root_server=http://narn:5555/cgi-bin/wanboot-cgi >>> root_file=/export/home/images/re_sparc_nightly/boot/platform/sun4v/boot_archive >>> boot_file=/export/home/images/re_sparc_nightly/platform/sun4v/wanboot >>> system_conf=system.conf >>> encryption_type= >>> signature_type= >>> server_authentication=no >>> client_authentication=no >>> >>> >>> Does the fix for 7045369 not work if a hostname is in the wanboot.conf file? >> 7045369 is fine, it's this fix that I'm wondering about since the changes you >> have proposed - if wanboot.conf contains the hostname instead of the IP >> address >> - then your fix here would set IPADDR to be narn not an IP address. >> >> I don't know if this is an issue, or not, I just want to confirm it with you. > > Ok, I get your question now. If it were a hostname, it is not an issue > because that hostname will be resolvable locally because of the > additional entry that will have already been put into the local > /etc/hosts file earlier by net-assembly. This is noted in the Eval of > the bug, but perhaps I could add a note into the block comment here. > Would the following NOTE help: > > # SPARC clients extract the address of the AI > server > # from the root_server option in wanboot.conf, > which > # is in the following format: > # > # > root_server=http://<ai_server>:<port>/<path_to_wanboot-cgi> > # > # NOTE: the <ai_server> portion might be an IP > address > # or a hostname. For the cases where it is a > hostname, > # the local /etc/hosts file would have already been > # augmented with an entry to resolve that hostname. > IPADDR=$($GREP "^root_server" $WANBOOT_CONF | \ > $CUT -d '=' -f2 | $CUT -d'/' -f3 | \ > $CUT -d':' -f1) > > > > As far as the variable name goes, I kept it as IPADDR because for the > x86 case, it's always an IP address and so didn't want to bother with > changing it. > > > -ethan > >> >> Thanks, >> >> Darren. >> >> >>> >>> -ethan >>> >>> >>>> Thanks, >>>> >>>> Darren. >>>> >>>> On 22/09/2011 23:53, Ethan Quach wrote: >>>>> Team, >>>>> >>>>> Can I get a couple of reviewers for the following bugfix: >>>>> >>>>> >>>>> Webrev: >>>>> ---------- >>>>> https://cr.opensolaris.org/action/browse/caiman/equach/7093471/webrev/ >>>>> >>>>> BugID: >>>>> -------- >>>>> 7093471<http://monaco.us.oracle.com/detail.jsf?cr=7093471> wanboot >>>>> without DHCP >>>>> not working >>>>> >>>>> >>>>> Cases tested: >>>>> ----------------- >>>>> 1. Sparc, wanboot with DHCP, wanboot.conf has IP address in 'root_server' >>>>> 2. Sparc, wanboot with DHCP, wanboot.conf has hostname in 'root_server' >>>>> 3. Sparc, wanboot without DHCP, wanboot.conf has IP address in >>>>> 'root_server' >>>>> 4. Sparc, wanboot without DHCP, wanboot.conf has hostname in >>>>> 'root_server' >>>>> >>>>> >>>>> thanks, >>>>> -ethan >>>>> >>>>> >>>>> >>>>> _______________________________________________ >>>>> 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

