[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]

Re: [libvirt] [PATCH] "static" ip address



Hi,

Back from a not-so-funny moment :(

I'm impressed by the work done once I was away ! Thanx !

I've tried to test it : 

- uninstalled debian package libvirt-bin (running ubuntu server 8.04 LTS)
- checked-out CVS version
- ./autobuild.sh --with-xen=no
- make
- sudo make install

Running virsh tells it's impossible to connect to hypervisor. Even if I force -c qemu:///
If I try to run /usr/local/sbin/libvirtd, I get a : 

odeckmyn everest:/usr/local/bin$ sudo /usr/local/sbin/libvirtd 
/usr/local/sbin/libvirtd: symbol lookup error: /usr/local/sbin/libvirtd: undefined symbol: __virAlloc

Did I do something wrong or is CVS version unstable today ?

May be I should wait for the .deb version ?

Thanx,

Olivier.

On Tue, Aug 19, 2008 at 12:08 PM, Daniel Veillard <veillard redhat com> wrote:
On Fri, Aug 15, 2008 at 10:43:15AM +0100, Daniel P. Berrange wrote:
> On Tue, Aug 12, 2008 at 05:24:01PM -0400, Daniel Veillard wrote:
> >   New patch with just a pair of minor fixes which seems to work just fine for
> > me. The syntax is nearly the old one, just use host instead of statichost:
> >
> >   <ip address="192.168.122.1" netmask="255.255.255.0">
> >     <dhcp>
> >       <range start="192.168.122.4" end="192.168.122.254" />
> >       <host mac="00:16:3E:XX:XX:XX" host="XXX"
>
> Seems a little odd to have 'host' as the element name and attribute
> name at once. Perhaps the atribute should just be name='XXXX' ?

 Agreed, I noticed that but it's not a serious problem, still using
name feels cleaner I will change that before commiting !

> > @@ -1103,6 +1103,8 @@
> >          2 + /* --listen-address 10.0.0.1 */
> >          1 + /* --dhcp-leasefile=path */
> >          (2 * network->def->nranges) + /* --dhcp-range 10.0.0.2,10.0.0.254 */
> > +        /*  --dhcp-host 01:23:45:67:89:0a,hostname,10.0.0.3 */
> > +        (2 * network->def->nhosts) +
>
> Using the --dhcp-host option means we can't easily update on the fly, but
> this patch doesn't support on the fly updates anyway, so not a huge deal.
> In the future we can switch to --dhcp-host-file instead.
>
>
> ACK to including this.

 okay, will push later today,

   thanks !

Daniel

--
Daniel Veillard      | libxml Gnome XML XSLT toolkit  http://xmlsoft.org/
daniel veillard com  | Rpmfind RPM search engine http://rpmfind.net/
http://veillard.com/ | virtualization library  http://libvirt.org/


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]