From: Jeremy Fitzhardinge <jeremy@goop.org>
To: Ian Campbell <Ian.Campbell@citrix.com>
Cc: "xen-devel@lists.xensource.com" <xen-devel@lists.xensource.com>,
Stefano Stabellini <stefano.stabellini@eu.citrix.com>
Subject: Re: [PATCH] add xl init scripts
Date: Fri, 04 Jun 2010 17:12:09 -0700 [thread overview]
Message-ID: <4C099659.5040308@goop.org> (raw)
In-Reply-To: <1275665214.24218.35917.camel@zakaz.uk.xensource.com>
On 06/04/2010 08:26 AM, Ian Campbell wrote:
> Would it be better to create /etc/init.d/{xenstored,xenconsoled}
> and /etc/sysconfig/{xenstored,xenconsoled} and have those used
> consistently no matter which toolstack was in use? Or maybe a single
> xendaemons script+config would do. In either case /etc/init.d/xend could
> be run only on systems which want xend, similarly for /etc/init.d/xl (if
> such a thing is still necessary after my comments below).
>
Perhaps. xl doesn't need any setup per-se, except for "xl rename 0
Domain-0" to avoid an ugly (null).
> Is a network script really required for xl? I think it would be more in
> keeping with how xl works to require that the distro configuration be
> setup correctly to create bridges etc e.g. correctly
> setup /etc/network/interfaces under Debian
> or /etc/sysconfig/network-scripts/ifcfg-* under RH derived distros etc.
>
Yes, yes, yes. Its easy to get Redhat/Fedora to set up a bridge as part
of its normal network setup. There's no need for us to duplicate it.
> Similarly I think xl should assume that the platform has been correctly
> configured with a vif hotplug script with the correct behaviour? It's
> not clear to me why tools/hotplug/Linux/xen-backend.rules needs to use a
> different scheme for vif devices to all the other backend types. The
> script there could be a symlink to one of several variants if necessary,
> there's no reason for it to be a config item in xl (or xend for that
> matter)
>
> It seems like we are now replicating some of the worst bits of xend
> poking around behind the back of the distro config files in xl.
>
Agree, unless as Stefano mentions, there's a need for some kind of
per-domain network setup (bridges, vlans, firewall rules, etc?).
J
prev parent reply other threads:[~2010-06-05 0:12 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-06-04 11:37 [PATCH] add xl init scripts Stefano Stabellini
2010-06-04 15:14 ` Ian Jackson
2010-06-04 15:26 ` Ian Campbell
2010-06-04 16:22 ` Stefano Stabellini
2010-06-04 16:55 ` Stefano Stabellini
2010-06-05 0:12 ` Jeremy Fitzhardinge [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=4C099659.5040308@goop.org \
--to=jeremy@goop.org \
--cc=Ian.Campbell@citrix.com \
--cc=stefano.stabellini@eu.citrix.com \
--cc=xen-devel@lists.xensource.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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.