George Dunlap
2013-May-29 11:53 UTC
[PATCH] xl: Update example xl.conf with correct name for default script option
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
Roger Pau Monné
2013-May-29 14:41 UTC
Re: [PATCH] xl: Update example xl.conf with correct name for default script option
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
Ian Campbell
2013-May-30 08:58 UTC
Re: [PATCH] xl: Update example xl.conf with correct name for default script option
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
Ian Campbell
2013-May-30 09:10 UTC
Re: [PATCH] xl: Update example xl.conf with correct name for default script option
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