On Thu, 2014-11-20 at 16:53 -0700, Xing Lin wrote: > I git clone nova-juno from github and searched for 'pygrub'. Here is what i > get. > > > .//etc/nova/rootwrap.d/compute.filters:104:# nova/virt/xenapi/vm_utils.py: > 'pygrub', '-qn', dev_path > .//etc/nova/rootwrap.d/compute.filters:105:pygrub: CommandFilter, pygrub, root > .//nova/tests/unit/virt/xenapi/test_xenapi.py:667: > self.assertEqual(self.vm['PV_bootloader'], 'pygrub') > .//nova/virt/xenapi/vm_utils.py:298: rec['PV_bootloader'] = > 'pygrub' > > > It seems that nova does not specify absolute path for pygrub. > > > I checked libvirt source code and found the following definition. > > > .//src/libxl/libxl_conf.h:52:# define LIBXL_BOOTLOADER_PATH BINDIR "/pygrub"
Thanks for investigating. I think libvirt is wrong to specify an absolute path here, IMHO by default it should just specify "pygrub" and let libxl figure out the correct path. Jim, what do you think? Ian. _______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel
