[ovs-dev] [PATCH 01/12] xenserver: Fix comment in interface-reconfigure.
blp at nicira.com
Tue Feb 23 10:02:03 PST 2010
On Tue, Feb 23, 2010 at 10:20:10AM +0000, Ian Campbell wrote:
> I made some minor comments on individual patches but generally all looks
> fine to me (take with a pinch of salt given my lack of knowledge of the
> next branch).
I only saw one comment, on patch 3/4. Were there others?
Anyway, I pushed these out, so I'll have to fix them up.
> The unit test stuff looks nice, if only I could steal it without
> learning m4 ;-) I've attached the unit test I use plus a script which
> will setup a XenServer host appropriately. They might be a little
> Ian-specific and are completely devoid of comments but feel free to ask
> if you are interested in using or cribbing from them and something isn't
Thanks. Are these the scripts that are (were?) in the XAPI tree? I
remember seeing a few of them related to testing interface-reconfigure,
but I never got around to trying them out.
I haven't decided yet whether m4 is a godsend or a horrible curse, and
I've been working with it for a long time now.
> I assume these will these end up with a Signed-off-by etc when they
> committed? At some point in the future, as part of moving XCP and
> XenServer over to the next branch, I'll want to cherry-pick these
> changes and upstream them to the XCP project which will require a
Thanks for pointing out the need for "Signed-off-by" lines. We have not
been using them for OVS (I guess our lawyers have not told us that we
need them?), but I'm perfectly willing to add them anyway, and so I will
stick them on all of these commits before I push them.
More information about the dev