[bookkeeper] branch master updated: [DEVTOOLS] Use ASF Jenkins API properly

[si...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

sijie pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/bookkeeper.git


The following commit(s) were added to refs/heads/master by this push:
     new 852367f  [DEVTOOLS] Use ASF Jenkins API properly
852367f is described below

commit 852367f053a3268885718257cbac55b696117670
Author: Sijie Guo <si...@apache.org>
AuthorDate: Thu Jun 7 18:45:30 2018 -0700

    [DEVTOOLS] Use ASF Jenkins API properly
    
    Descriptions of the changes in this PR:
    
    According to https://cwiki.apache.org/confluence/display/INFRA/Using+the+ASF+Jenkins+API,
    we need to include 'tree' or 'depth' when using ASF jenkins API.
    
    Author: Sijie Guo <si...@apache.org>
    
    Reviewers: Jia Zhai <None>
    
    This closes #1503 from sijie/access_jenkins_apis_should_include_depth
---
 dev/bk-merge-pr.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/dev/bk-merge-pr.py b/dev/bk-merge-pr.py
index f2ad7a3..c7d2adf 100755
--- a/dev/bk-merge-pr.py
+++ b/dev/bk-merge-pr.py
@@ -500,7 +500,7 @@ def is_jenkins_check(check):
     return check["context"].startswith("Jenkins:")
 
 def is_jenkins_passed(url):
-    jenkins_status = get_json("%sapi/json" % (url))
+    jenkins_status = get_json("%sapi/json?tree=result" % (url))
     return "SUCCESS" == jenkins_status['result'] 
 
 def is_integration_test_check(check):

-- 
To stop receiving notification emails like this one, please contact
sijie@apache.org.