Re: [Xen-devel] [PATCH RFC OSSTEST v1 10/12] Qualify TftpDiVersion with the suite.

2016-01-12 Thread Ian Jackson
Ian Campbell writes ("[PATCH RFC OSSTEST v1 10/12] Qualify TftpDiVersion with 
the suite."):
> This allows the version to differ e.g. between Wheezy and Jessie.
> 
> Update production-config* to set TftpDiVersion_jessie instead of just
> TftpDiVersion, also add TftpDiVersion_wheezy using the version
> replaced in commit f610ea162836 "Switch to Debian 8.0 (jessie) as OS
> for test hosts".
> 
> In mfi-common we need to check for TftpDiVersion_$suite (_$guestsuite)
> and TftpDiVersion manually since getconfig In that context will not
> see any DebianSuite override in the environment.
> 
> This ensures that when a non-default suite is configured a
> corresponding useful version of DI is selected.

Acked-by: Ian Jackson 

provided the identifier `diversion' is changed (see my previous
mails).

Ian.

___
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel


[Xen-devel] [PATCH RFC OSSTEST v1 10/12] Qualify TftpDiVersion with the suite.

2016-01-08 Thread Ian Campbell
This allows the version to differ e.g. between Wheezy and Jessie.

Update production-config* to set TftpDiVersion_jessie instead of just
TftpDiVersion, also add TftpDiVersion_wheezy using the version
replaced in commit f610ea162836 "Switch to Debian 8.0 (jessie) as OS
for test hosts".

In mfi-common we need to check for TftpDiVersion_$suite (_$guestsuite)
and TftpDiVersion manually since getconfig In that context will not
see any DebianSuite override in the environment.

This ensures that when a non-default suite is configured a
corresponding useful version of DI is selected.

Signed-off-by: Ian Campbell 
---
 Osstest.pm  | 2 +-
 mfi-common  | 6 ++
 production-config   | 3 ++-
 production-config-cambridge | 3 ++-
 4 files changed, 11 insertions(+), 3 deletions(-)

diff --git a/Osstest.pm b/Osstest.pm
index 9ff86ab..814be28 100644
--- a/Osstest.pm
+++ b/Osstest.pm
@@ -224,7 +224,7 @@ sub readglobalconfig () {
 $c{TftpTmpDir} ||= "$c{TftpPlayDir}tmp/";
 
 $c{TftpDiBase} ||= "$c{TftpPlayDir}debian-installer";
-$c{TftpDiVersion} ||= 'current';
+$c{TftpDiVersion} ||= $c{ "TftpDiVersion_$c{DebianSuite}" } // 'current';
 
 $c{TftpGrubBase} ||= "$c{TftpPlayDir}grub";
 $c{TftpGrubVersion} ||= 'current';
diff --git a/mfi-common b/mfi-common
index c1c4ec5..1daa6dd 100644
--- a/mfi-common
+++ b/mfi-common
@@ -124,6 +124,9 @@ create_build_jobs () {
 fi
 
 if [ -z "$diversion" ] ; then
+local diversion=`getconfig TftpDiVersion_$suite`
+fi
+if [ -z "$diversion" ] ; then
 local diversion=`getconfig TftpDiVersion`
 fi
 
@@ -413,6 +416,9 @@ test_matrix_iterate () {
 fi
 
 if [ -z "$diversion" ] ; then
+local diversion=`getconfig TftpDiVersion_$guestsuite`
+fi
+if [ -z "$diversion" ] ; then
 local diversion=`getconfig TftpDiVersion`
 fi
 
diff --git a/production-config b/production-config
index b21153b..42948c8 100644
--- a/production-config
+++ b/production-config
@@ -87,7 +87,8 @@ TftpPxeTemplatesReal pxelinux.cfg/%ipaddrhex%
 
 TftpPxeGroup osstest
 # Update with ./mg-debian-installer-update(-all)
-TftpDiVersion 2015-09-18
+TftpDiVersion_wheezy 2015-09-07
+TftpDiVersion_jessie 2015-09-18
 
 # These should normally be the same.
 # Update with ./mg-cpu-microcode-update
diff --git a/production-config-cambridge b/production-config-cambridge
index b0b45ac..78d606b 100644
--- a/production-config-cambridge
+++ b/production-config-cambridge
@@ -69,7 +69,8 @@ TftpPxeTemplates %name%/pxelinux.cfg
 TftpPxeTemplatesReal pxelinux.cfg/%ipaddrhex%
 
 TftpPxeGroup osstest
-TftpDiVersion 2015-09-18
+TftpDiVersion_wheezy 2015-09-07
+TftpDiVersion_jessie 2015-09-18
 
 # These should normally be the same.
 MicrocodeUpdateAmd64 microcode.x86.2015-06-12.cpio
-- 
2.6.1


___
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel