WARNING - OLD ARCHIVES

This is an archived copy of the Xen.org mailing list, which we have preserved to ensure that existing links to archives are not broken. The live archive, which contains the latest emails, can be found at http://lists.xen.org/
   
 
 
Xen 
 
Home Products Support Community News
 
   
 

xen-devel

Re: [Xen-devel] [PATCH] [XEND] fix vif IP specification broken in 12633:

To: xen-devel@xxxxxxxxxxxxxxxxxxx
Subject: Re: [Xen-devel] [PATCH] [XEND] fix vif IP specification broken in 12633:5bed7bc05c8a
From: Ewan Mellor <ewan@xxxxxxxxxxxxx>
Date: Tue, 5 Dec 2006 23:21:47 +0000
Delivery-date: Tue, 05 Dec 2006 15:21:50 -0800
Envelope-to: www-data@xxxxxxxxxxxxxxxxxx
In-reply-to: <20061205205118.GA13614@xxxxxxxxxxxxxxxxx>
List-help: <mailto:xen-devel-request@lists.xensource.com?subject=help>
List-id: Xen developer discussion <xen-devel.lists.xensource.com>
List-post: <mailto:xen-devel@lists.xensource.com>
List-subscribe: <http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe>
References: <013e995e2a6b0c5b237b.1165276770@xxxxxxxxxxxxxxxxx> <20061205205118.GA13614@xxxxxxxxxxxxxxxxx>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
User-agent: Mutt/1.5.9i
On Tue, Dec 05, 2006 at 12:51:19PM -0800, Brendan Cully wrote:

> On Monday, 04 December 2006 at 15:59, Brendan Cully wrote:
> > # HG changeset patch
> > # User Brendan Cully <brendan@xxxxxxxxx>
> > # Date 1165276659 28800
> > # Node ID 013e995e2a6b0c5b237b29211e697b7757e5f94f
> > # Parent  fd28a1b139dea91b8bfcf06dd233dbdda8f51ff1
> > [XEND] fix vif IP specification broken in 12633:5bed7bc05c8a
> > 
> > Apparently at some point one could attach multiple IP addresses
> > to a single VIF in the config file. This doesn't address that
> > feature.
> > 
> > Signed-off-by: Brendan Cully <brendan@xxxxxxxxx>
> 
> Sorry, I should have included a better description of the problem:
> 
> vif = ['ip=xxx.xxx.xxx.xxx'] hasn't worked since 12633 (at least with
> Xen's nat scripts). The problem is that pre-12633, the IP field
> returned from the config parser was a list, but now it is a string. So
> ' '.join(ip) ends up sending 'x x x . x x x . x x x . x x x' to the
> scripts, which were expecting less whitespace.

So does your fix mean that multiple IP addresses work again, or can we
only have one address this way?  I'd prefer to fix the config parser if
that's what's needed to support multiple IP addresses, since it
certainly used to work.

Ewan.

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel