[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [OSSTEST PATCH] i18n/l10n: Make Timezone configurable and change the default

Ian Campbell writes ("Re: [OSSTEST PATCH] i18n/l10n: Make Timezone configurable 
and change the default"):
> On Fri, 2015-05-15 at 11:44 +0100, Ian Jackson wrote:
> > Signed-off-by: Ian Jackson <Ian.Jackson@xxxxxxxxxxxxx>
> Acked-by: Ian Campbell <ian.campbell@xxxxxxxxxx>


I've made a new version which also documents the variable in README;
I guess you're happy for me to keep your ack.


commit 3c425a513e603184b8dd128b28398ceec2aea4b4
Author: Ian Jackson <ian.jackson@xxxxxxxxxxxxx>
Date:   Fri May 15 11:31:35 2015 +0100

    i18n/l10n: Make Timezone configurable and change the default
    * Introduce a new config option Timezone
    * Replace hardcoded Europe/London everywhere with $c{Timezone}
    * The default is UTC
    * But in production-config-cambridge set it to Europe/London
    The overall effect is:
    * No change in Cambridge
    * Default timezone changes to UTC but can now be overridden
    * Production instance timezone changes to UTC
    (It appears that there is no reasonable way to find out the Olson TZ
    name of the controller host's default timezone.  If there were, or we
    discover one, we should arrange that the default is set
    Signed-off-by: Ian Jackson <Ian.Jackson@xxxxxxxxxxxxx>

diff --git a/Osstest.pm b/Osstest.pm
index 7f72bc6..e8bd77b 100644
--- a/Osstest.pm
+++ b/Osstest.pm
@@ -76,6 +76,8 @@ our %c = qw(
     DebianNonfreeFirmware firmware-bnx2
     HostnameSortSwapWords 0
+    Timezone UTC
 $c{$_}='' foreach qw(
diff --git a/Osstest/Debian.pm b/Osstest/Debian.pm
index a577d1d..3adc84f 100644
--- a/Osstest/Debian.pm
+++ b/Osstest/Debian.pm
@@ -567,7 +567,7 @@ d-i mirror/country string manual
 d-i mirror/http/proxy string
 d-i clock-setup/utc boolean true
-d-i time/zone string Europe/London
+d-i time/zone string $c{Timezone}
 d-i clock-setup/ntp boolean true
 d-i partman-md/device_remove_md boolean true
diff --git a/README b/README
index 0aaba2c..44e2989 100644
--- a/README
+++ b/README
@@ -476,6 +476,9 @@ TftpPxeTemplatesReal
     Template filename which mg-hosts mkpxedir should make be a symlink to
     the TftpPxeTemplates.  Not used otherwise.
+    Olson TZ name, used by host and guest installers
 Host-specific config settigs
diff --git a/production-config-cambridge b/production-config-cambridge
index 62215d0..f32cd84 100644
--- a/production-config-cambridge
+++ b/production-config-cambridge
@@ -30,6 +30,8 @@ HostnameSortSwapWords 1
 DnsDomain xs.citrite.net
+Timezone Europe/London
 Stash /home/xc_osstest/logs
 Images /home/xc_osstest/images
 Logs /home/xc_osstest/logs
diff --git a/ts-freebsd-install b/ts-freebsd-install
index 6e8fada..0d6eb0c 100755
--- a/ts-freebsd-install
+++ b/ts-freebsd-install
@@ -82,7 +82,7 @@ END
             sed -i '/^ttyu0/s/off/on/' $mnt/etc/ttys
-            cp $mnt/usr/share/zoneinfo/Europe/London $mnt/etc/localtime
+            cp $mnt/usr/share/zoneinfo/$c{Timezone} $mnt/etc/localtime
             cat <<'ENDKEYS' >$mnt/boot/loader.conf
diff --git a/ts-redhat-install b/ts-redhat-install
index 22c3061..10da559 100755
--- a/ts-redhat-install
+++ b/ts-redhat-install
@@ -59,7 +59,7 @@ selinux --disable
 sshpw --username=root xenroot --plaintext
 sshpw --username=iwj xenroot --plaintext
-timezone --utc Europe/London
+timezone --utc $c{Timezone}
 bootloader --location=mbr --driveorder=sda --append="crashkernel=auto rhgb 
 # The following is the partition information you requested

Xen-devel mailing list



Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.