From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Pang, LongtaoX" Subject: Re: [OSSTEST Nested PATCH v8 3/7] Edit some APIs in TestSupport.pm for nested test Date: Wed, 22 Apr 2015 08:25:59 +0000 Message-ID: <86C3224E41A7434B904EC364302132D80E49861C@SHSMSX101.ccr.corp.intel.com> References: <1428959990-24199-1-git-send-email-longtaox.pang@intel.com> <1428959990-24199-4-git-send-email-longtaox.pang@intel.com> <1429611542.4743.22.camel@citrix.com> <21814.17314.479011.970684@mariner.uk.xensource.com> <1429620780.4743.67.camel@citrix.com> <21814.20580.723370.301076@mariner.uk.xensource.com> <1429623708.4743.97.camel@citrix.com> <21814.24305.129814.960644@mariner.uk.xensource.com> <1429627421.4743.119.camel@citrix.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <1429627421.4743.119.camel@citrix.com> Content-Language: en-US List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xen.org Errors-To: xen-devel-bounces@lists.xen.org To: Ian Campbell , Ian Jackson Cc: "Hu, Robert" , "wei.liu2@citrix.com" , "xen-devel@lists.xen.org" List-Id: xen-devel@lists.xenproject.org > -----Original Message----- > From: Ian Campbell [mailto:ian.campbell@citrix.com] > Sent: Tuesday, April 21, 2015 10:44 PM > To: Ian Jackson > Cc: Pang, LongtaoX; xen-devel@lists.xen.org; wei.liu2@citrix.com; Hu, Robert > Subject: Re: [OSSTEST Nested PATCH v8 3/7] Edit some APIs in TestSupport.pm for > nested test > > On Tue, 2015-04-21 at 15:30 +0100, Ian Jackson wrote: > > Ian Campbell writes ("Re: [OSSTEST Nested PATCH v8 3/7] Edit some APIs in > TestSupport.pm for nested test"): > > > On Tue, 2015-04-21 at 14:28 +0100, Ian Jackson wrote: > > > > I'm only objecting to the involvement of the host property machinery. > > > > > > I'm afraid I'm not 100% sure I understand, but do you just mean that > > > longtao should assign to $ho->{Ip} directly within selecthost instead of > > > using $setprop to set the IpAddr hostprop and therefore indirectly > > > ->{Ip}? > > > > Yes. > > > > > IOW the existing code in selecthost which looks up the IpAddr host prop > > > into $ho->{IpStatic} and then into $ho->{Ip} should prefer the runvar to > > > the host db if it is set? > > > > Yes. > > Good. Longtao, do you understand what is needed here? > I think I get it, use ${ident} rather than ${name} and assign L1's IP to $ho->{Ip} directly. The code will be like: if ( $r{"${ident}_ip"} ) { $ho->{Ip}= $r{"${ident}_ip"}; } > > > BTW, what is the difference between ->{Ip} and ->{IpStatic} and should > > > the runvar be reflected in both or just in ->{Ip}? > > > > AFAICT the only difference is that only IpStatic is used for expanding > > `ipaddr' and `ipaddrhex' in the patterns for dhcp pxe configurations. > > > > There is an incipient problem here: our existing setup does not > > require that osstest has complete control of the pxe config file for > > every host on the network. And indeed the Citrix (Cambridge) instance > > lacks such control. > > > > Instead there are some symlinks etc. The mg-hosts mkpxedir subcommand > > arranges to delegate the DHCP from root to the osstest user (and > > group), by making an appropriate subdirectory with the right > > permissions, and a suitable symlink. > > > > It's not quite clear to me how this should work for nested HVM hosts. > > I think this is OK with the current design because the nested HVM host > is not PXE booted, it is installed as a guest but preseeded to look a > lot like a host (quite a lot of sharing in the preseed creation) and > then tailored into a more host-like thing by things like this IP addr > runvar. > > So I think from what you are saying that the ->{Ip} and ->{IpStatic} > distinction here is correct and a host which is really a guest should > not have nor use ->{IpStatic} (we want e.g. attempts to create a pxe dir > to fail). > > Longtao, the conclusion is that the runvar should only appear in ->{Ip}. > > ->{IpStatic} should be left untouched (i.e. not initialised). > > Ian.