Hi Diederik,
Diederik de Haas wrote:
> According to man:/xl.cfg the default action of 'on_crash' is 'destroy',
> but /etc/xen-tools/xm.tmpl contains:
> on_crash = 'restart'
Yes, at that point xen-tools' defaults are different than Xen's. But
is that a bug? Even if xen-tools does it deliberately?
But I must admit, I'm unsure if I should really change xen-tools'
default here. I do see that not following Xen's default values might
break some users' expectations, but I consider the automatic reboot on
crash rather a feature than a bug. I see that setting like a watchdog.
Of course, if I start a DomU and it crashes immediately and over and
over again, I should fix that. But I should anyways check if a freshly
started DomU actually works. (And I consider crashing later and then
getting into a endless boot loop to be rather seldom, maybe happening
with rolling-release-style distributions.)
Additionally, if I want to not deviate from Xen's defaults I'd rather
remove that value completely. Because what that line does is
overwriting Xen's default. If I overwrite the default value with the
default value I ask for future deviation from a potentially changing
default value. So while I do see the problem, I don't consider your
submitted patch to be suitable as a fix.
Another potential way to fix this bug report is IMHO to make clear
in the documentation that xen-tools' defaults deviate from Xen's
defaults.
Regards, Axel
--
,''`. | Axel Beckert <[email protected]>, http://people.debian.org/~abe/
: :' : | Debian Developer, ftp.ch.debian.org Admin
`. `' | 4096R: 2517 B724 C5F6 CA99 5329 6E61 2FF9 CD59 6126 16B5
`- | 1024D: F067 EA27 26B9 C3FC 1486 202E C09E 1D89 9593 0EDE
