[PATCH OSSTEST] cr-try-bisect-adhoc: Set laundered_testid so graph URL is correct

[PATCH OSSTEST] cr-try-bisect-adhoc: Set laundered_testid so graph URL is correct

[Ian Campbell]

Otherwise the testid is missing from the filename, resulting in e.g.
http://osstest.test-lab.xenproject.org/~osstest/pub/results-adhoc/bisect/xen-unstable/test-amd64-amd64-qemuu-nested-intel..svg

Instead of test-amd64-amd64-qemuu-nested-intel.debian-hvm-install-l1-l2.svg

Signed-off-by: Ian Campbell <ian.campbell@citrix.com>
---
Untested...
---
 cr-try-bisect-adhoc | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/cr-try-bisect-adhoc b/cr-try-bisect-adhoc
index f451324..6d5d7c4 100755
--- a/cr-try-bisect-adhoc
+++ b/cr-try-bisect-adhoc
@@ -36,6 +36,8 @@ mkdir -p tmp
 #   testid=
 #   bisect= list of args to cs-bisection-step, eg  '--fail-flight=<mumble>'
 
+laundered_testid=${testid//\//--}
+
 . ./cri-bisect
 . ./cri-args-hostlists
 select_branch
-- 
2.6.1

Re: [PATCH OSSTEST] cr-try-bisect-adhoc: Set laundered_testid so graph URL is correct

[Ian Jackson]

Ian Campbell writes ("[PATCH OSSTEST] cr-try-bisect-adhoc: Set laundered_testid so graph URL is correct"):
> Otherwise the testid is missing from the filename, resulting in e.g.
> http://osstest.test-lab.xenproject.org/~osstest/pub/results-adhoc/bisect/xen-unstable/test-amd64-amd64-qemuu-nested-intel..svg

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

Looks plausible.

Ian.

Re: [PATCH OSSTEST] cr-try-bisect-adhoc: Set laundered_testid so graph URL is correct

[Ian Campbell]

On Wed, 2015-12-02 at 16:41 +0000, Ian Jackson wrote:
> Ian Campbell writes ("[PATCH OSSTEST] cr-try-bisect-adhoc: Set
> laundered_testid so graph URL is correct"):
> > Otherwise the testid is missing from the filename, resulting in e.g.
> > http://osstest.test-lab.xenproject.org/~osstest/pub/results-adhoc/bisec
> > t/xen-unstable/test-amd64-amd64-qemuu-nested-intel..svg
> 
> Acked-by: Ian Jackson <ian.jackson@eu.citrix.com>
> 
> Looks plausible.

FYI I've cowboyed it into ~osstest/bisects/adhoc.git so we ought to find
out on the next step of the current adhoc bisect.

Ian.