[PATCH OSSTEST v2] No longer export $OSSTEST_CONFIG

[PATCH OSSTEST v2] No longer export $OSSTEST_CONFIG

[Ian Campbell]

>From cri-args-hostlists or invoke-daemon.

All sites now have a suitable $HOME/.xen-osstest/settings in place
which does this.

Signed-off-by: Ian Campbell <ian.campbell@citrix.com>
---
This was waiting to be applied once " allow instance specific
settings" passed the Cambridge push gate, which happened ages ago.
---
 cri-args-hostlists | 1 -
 invoke-daemon      | 2 --
 2 files changed, 3 deletions(-)

diff --git a/cri-args-hostlists b/cri-args-hostlists
index a4e57b3..0dd2ef3 100644
--- a/cri-args-hostlists
+++ b/cri-args-hostlists
@@ -19,7 +19,6 @@
 if [ -e $HOME/.xen-osstest/settings ]; then
      source $HOME/.xen-osstest/settings
 fi
-export OSSTEST_CONFIG=${OSSTEST_CONFIG:-production-config}
 
 check_stop_core () {
 	if [ "x$OSSTEST_IGNORE_STOP" = xy ]; then return; fi
diff --git a/invoke-daemon b/invoke-daemon
index 6006798..5fab1da 100755
--- a/invoke-daemon
+++ b/invoke-daemon
@@ -19,8 +19,6 @@
 if [ -e $HOME/.xen-osstest/settings ]; then
      source $HOME/.xen-osstest/settings
 fi
-export OSSTEST_CONFIG=${OSSTEST_CONFIG:-production-config}
-
 
 cd "${0%/*}"
 if [ "x$2" != x ]; then sleep $2; fi
-- 
2.1.4

Re: [PATCH OSSTEST v2] No longer export $OSSTEST_CONFIG

[Ian Jackson]

Ian Campbell writes ("[PATCH OSSTEST v2] No longer export $OSSTEST_CONFIG"):
> >From cri-args-hostlists or invoke-daemon.
> 
> All sites now have a suitable $HOME/.xen-osstest/settings in place
> which does this.
> 
> Signed-off-by: Ian Campbell <ian.campbell@citrix.com>
> ---
> This was waiting to be applied once " allow instance specific
> settings" passed the Cambridge push gate, which happened ages ago.

Verified that, and

Acked-by: Ian Jackson <ian.jackson@eu.citrix.com>

Re: [PATCH OSSTEST v2] No longer export $OSSTEST_CONFIG

[Ian Campbell]

On Fri, 2015-07-24 at 18:03 +0100, Ian Jackson wrote:
> Ian Campbell writes ("[PATCH OSSTEST v2] No longer export 
> $OSSTEST_CONFIG"):
> > > From cri-args-hostlists or invoke-daemon.
> > 
> > All sites now have a suitable $HOME/.xen-osstest/settings in place
> > which does this.
> > 
> > Signed-off-by: Ian Campbell <ian.campbell@citrix.com>
> > ---
> > This was waiting to be applied once " allow instance specific
> > settings" passed the Cambridge push gate, which happened ages ago.
> 
> Verified that,

I think you meant "passed the Cambridge push gate" and not "in place which
does this". I've just reconfirmed the latter too.

>  and
> 
> Acked-by: Ian Jackson <ian.jackson@eu.citrix.com>

Ta