[xen-tools] Re: [PATCH] ubuntu hooks update

[Ward Vandewege]

Hi Steve,

Sorry for the delay; I've been a bit busy the past few days.

On Mon, Oct 16, 2006 at 01:43:14PM +0100, Steve Kemp wrote:
> > I've been testing and tweaking for my Ubuntu dapper installs. I've modified
> > several things in the ubuntu.d hooks directory - it seems some of these hooks
> > were tweaked for Sarge and not applicable to Dapper.
> 
>   I neither use nor test against Ubuntu ..

Sure, no problem. I can do that :)

> > In summary, here's what I've changed:
> > 
> > 1. removed 10-disable-tls. Not applicable on Dapper.
> 
>   Are you sure that there are no distributions which this applies?

Let's get something straight first; I tested on Dapper. Currently you have
dapper.d symlinked to ubuntu.d; I assume you will split those dirs up when
Edgy (the next Ubuntu release) comes out?

Basically I was assuming that any change I made that works on Dapper is fine,
because xen-tools doesn't support any other Ubuntu release right now. Not
much point in supporting older releases, and for newer ones I assumed we
could split the config because there will no doubt be some changes.

So; that being said; removing 10-disable-tls for Dapper makes sense because
the /lib/tls directory is simply not there by default. I'm a little confused
as to whether the glibc6 that ships with Dapper is actually xen-friendly (or
*exactly* what that constitutes, even; I only found some reference to being
compiled with the -mno-tls-direct-seg-refs CFLAG) or not. But in any case,
/lib/tls simply does not exist.

> > 2. added 21-fix-locale. This generates the default locale on Dapper,
> > otherwise there are lots of ugly warnings.
> 
>   That seems like a good thing to do, but I'm a little concerned
>  that you only generate the EN.us locale - is it possible to generate
>  the locales of the system instead?
>
>   I'd hate to see a bug report from somebody using, say, a Greek
>  locale ..

Well, here I have the same kind of problem; at this point we only have what
debootstrap created. And because debootstrap is automatic, it can not prompt
for a locale, so it just assumes the default (apparently). Except, there's a
bug in the sense that it does not generate the default locale (which seems to
be en_US.UTF-8). Even if you use a Greek locale, having the default locale
installed will not hurt - and it will still prevent all those ugly warnings
the first time you boot the domU, before you have installed your Greek locale.

> > 3. moved 15-disable-hwclock to 22-disable-hwclock (after the fix-locale
> > step), and fixed it; it now calls update-rc.d to remove the hwclock.sh
> > startup script, which is the only one that exists on Dapper.
> 
>   I think I tried this in the past and when the package is upgraded
>  the script comes back.  Still I'll accept that change right now.

Cool. Yes, it will probably come back when the package is upgraded. I wonder
what the Debian way is to deal with that.

> > 4. removed 55-create-dev. Dapper comes with udev by default. Removing this
> > hook saves a *lot* of time.
> 
>   OK .. Will remove that if you can confirm that nobody using Ubuntu
>  will have a static /dev.

That's not really the question though is it; the question is more like 'what
is debootstrap's default', or in other words what do you end up with the
first time you boot after running xen-create-image. No?

> > I've attached a tgz file with my modified ubuntu.d directory. I'd like to see
> > these changes merged; what do you think?
> 
>   I will take the hwclock update.  The locale one I'd like to see
>  improved before I take it.
> 
>   If you can confirm that the dev + tls fixes apply to *all* versions
>  of ubuntu I'll take them.  (I'm hazy on what releases are available
>  and what they offer.)

I think I'd prefer to say 'this is ok for Dapper's defaults'. And let's see
when Edgy comes out, and split the config then. And, let's ignore older
Ubuntu versions. Wht do you think about that?

>   Thanks for the updates, and for sharing :)

Hey no problem at all - thanks for this great tool!!

Thanks,
Ward.

-- 
Pong.be         -(   Economic advantage is not in and of itself a valid    )-
Virtual hosting -( purpose or justification for copyright or patent laws.  )-
http://pong.be  -(                                                         )-
GnuPG public key: http://gpg.dtype.org