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

2015-12-02 Thread 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 
---
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='
 
+laundered_testid=${testid//\//--}
+
 . ./cri-bisect
 . ./cri-args-hostlists
 select_branch
-- 
2.6.1


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


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

2015-12-02 Thread 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 

Looks plausible.

Ian.

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


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

2015-12-02 Thread Ian Campbell
On Wed, 2015-12-02 at 16:41 +, 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 
> 
> 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.


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