All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] xl: Update example xl.conf with correct name for default script option
@ 2013-05-29 11:53 George Dunlap
  2013-05-29 14:41 ` Roger Pau Monné
  2013-05-30  8:58 ` Ian Campbell
  0 siblings, 2 replies; 4+ messages in thread
From: George Dunlap @ 2013-05-29 11:53 UTC (permalink / raw)
  To: xen-devel; +Cc: George Dunlap, Ian Jackson, Ian Campbell, Roger Pau Monné

We've changed the config option from "vifscript" to "vif.default.script".  This
was changed in the manpage but not in the example xl.conf.

Also move the option down to be with the other vif options.

Signed-off-by: George Dunlap <george.dunlap@eu.citrix.com>
CC: Roger Pau Monné <roger.pau@citrix.com>
CC: Ian Campbell <ian.campbell@citrix.com>
CC: Ian Jackson <ian.jackson@citrix.com>
---
 tools/examples/xl.conf |    6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/tools/examples/xl.conf b/tools/examples/xl.conf
index 9402c3f..9c037a6 100644
--- a/tools/examples/xl.conf
+++ b/tools/examples/xl.conf
@@ -8,9 +8,6 @@
 # full path of the lockfile used by xl during domain creation
 #lockfile="/var/lock/xl"
 
-# default vif script 
-#vifscript="vif-bridge"
-
 # default output format used by "xl list -l"
 #output_format="json"
 
@@ -25,6 +22,9 @@
 # default gateway device to use with vif-route hotplug script
 #vif.default.gatewaydev="eth0"
 
+# default vif script to use if none is specified in the guest config
+#vif.default.script="vif-bridge"
+
 # default bridge device to use with vif-bridge hotplug scripts
 #vif.default.bridge="xenbr0"
 
-- 
1.7.9.5


_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel

^ permalink raw reply related	[flat|nested] 4+ messages in thread

* Re: [PATCH] xl: Update example xl.conf with correct name for default script option
  2013-05-29 11:53 [PATCH] xl: Update example xl.conf with correct name for default script option George Dunlap
@ 2013-05-29 14:41 ` Roger Pau Monné
  2013-05-30  9:10   ` Ian Campbell
  2013-05-30  8:58 ` Ian Campbell
  1 sibling, 1 reply; 4+ messages in thread
From: Roger Pau Monné @ 2013-05-29 14:41 UTC (permalink / raw)
  To: George Dunlap; +Cc: Ian Jackson, Ian Campbell, xen-devel

On 29/05/13 13:53, George Dunlap wrote:
> We've changed the config option from "vifscript" to "vif.default.script".  This
> was changed in the manpage but not in the example xl.conf.
> 
> Also move the option down to be with the other vif options.
> 
> Signed-off-by: George Dunlap <george.dunlap@eu.citrix.com>
> CC: Roger Pau Monné <roger.pau@citrix.com>
> CC: Ian Campbell <ian.campbell@citrix.com>
> CC: Ian Jackson <ian.jackson@citrix.com>

Acked-by: Roger Pau Monné <roger.pau@citrix.com>

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] xl: Update example xl.conf with correct name for default script option
  2013-05-29 11:53 [PATCH] xl: Update example xl.conf with correct name for default script option George Dunlap
  2013-05-29 14:41 ` Roger Pau Monné
@ 2013-05-30  8:58 ` Ian Campbell
  1 sibling, 0 replies; 4+ messages in thread
From: Ian Campbell @ 2013-05-30  8:58 UTC (permalink / raw)
  To: George Dunlap; +Cc: Ian Jackson, Roger Pau Monné, xen-devel

On Wed, 2013-05-29 at 12:53 +0100, George Dunlap wrote:
> We've changed the config option from "vifscript" to "vif.default.script".  This
> was changed in the manpage but not in the example xl.conf.
> 
> Also move the option down to be with the other vif options.
> 
> Signed-off-by: George Dunlap <george.dunlap@eu.citrix.com>
> CC: Roger Pau Monné <roger.pau@citrix.com>
> CC: Ian Campbell <ian.campbell@citrix.com>

Acked + applied ,thanks.



_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] xl: Update example xl.conf with correct name for default script option
  2013-05-29 14:41 ` Roger Pau Monné
@ 2013-05-30  9:10   ` Ian Campbell
  0 siblings, 0 replies; 4+ messages in thread
From: Ian Campbell @ 2013-05-30  9:10 UTC (permalink / raw)
  To: Roger Pau Monné; +Cc: George Dunlap, Ian Jackson, xen-devel

On Wed, 2013-05-29 at 16:41 +0200, Roger Pau Monné wrote:
> On 29/05/13 13:53, George Dunlap wrote:
> > We've changed the config option from "vifscript" to "vif.default.script".  This
> > was changed in the manpage but not in the example xl.conf.
> > 
> > Also move the option down to be with the other vif options.
> > 
> > Signed-off-by: George Dunlap <george.dunlap@eu.citrix.com>
> > CC: Roger Pau Monné <roger.pau@citrix.com>
> > CC: Ian Campbell <ian.campbell@citrix.com>
> > CC: Ian Jackson <ian.jackson@citrix.com>
> 
> Acked-by: Roger Pau Monné <roger.pau@citrix.com>

Oops, I just applied this without noticing your Ack, sorry about that.



_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2013-05-30  9:10 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2013-05-29 11:53 [PATCH] xl: Update example xl.conf with correct name for default script option George Dunlap
2013-05-29 14:41 ` Roger Pau Monné
2013-05-30  9:10   ` Ian Campbell
2013-05-30  8:58 ` Ian Campbell

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.