[Fuego] [PATCH 1/3] chart: https protocol support in Jenkins path

Tim.Bird at sony.com Tim.Bird at sony.com
Wed Jul 11 00:35:12 UTC 2018


Sorry for the slow response. Just getting back from vacation and back
into the swing of things.

> -----Original Message-----
> From: Li Xiaoming
> Subject: [Fuego] [PATCH 1/3] chart: https protocol support in Jenkins path
> 
> From: Liu Wenlong <liuwl.fnst at cn.fujitsu.com>
> 
> Using flexible protocol to get the test results
> 
> Signed-off-by: Kyohei Oki <kyohhei.oki at denso-ten.com>
It's a little strange to have the e-mail From:, the patch
author From:,  and Signed-off-by: lines all have different people.

According to this, the patch was written by Liu Wenlong,
mailed by Li Xiaoming, but asserted to be legally OK by
Kyohei Oki.

In the future, if you have different people involved in the patch,
please have the person who e-mails the patch also
include a Signed-off-by: line for the patch.  This can be in addition
to the Signed-off-by: line(s) for other people involved in
the patch creation and review.
  
> ---
>  engine/scripts/mod.js | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/engine/scripts/mod.js b/engine/scripts/mod.js
> index c23dca4..d9074cc 100644
> --- a/engine/scripts/mod.js
> +++ b/engine/scripts/mod.js
> @@ -24,7 +24,7 @@ jQuery.noConflict();
>  jQuery(document).ready(function () {
> 
>  // Jenkins logs path
> -var jenkins_logs_path = 'http://'+location['host'] +
> '/fuego/userContent/fuego.logs/';
> +var jenkins_logs_path = location['protocol'] + '//' + location['host'] +
> '/fuego/userContent/fuego.logs/';
> 
>  // get the test name from the URL
>  var localurl = jQuery(location).attr('href').split("/");
> --
> 2.7.4

This is a great fix!  Thanks very much for the patch!
Applied and pushed in my master branch
 -- Tim



More information about the Fuego mailing list