forked from CloudStack-extras/reviewboard-tools
-
Notifications
You must be signed in to change notification settings - Fork 0
/
reviewboard_testpatch.py
213 lines (185 loc) · 8.95 KB
/
reviewboard_testpatch.py
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
#!/usr/bin/python
#
# Licensed to the Apache Software Foundation (ASF) under one or more
# contributor license agreements. See the NOTICE file distributed with
# this work for additional information regarding copyright ownership.
# The ASF licenses this file to You under the Apache License, Version 2.0
# (the "License"); you may not use this file except in compliance with
# the License. You may obtain a copy of the License at
#
# http://www.apache.org/licenses/LICENSE-2.0
#
# Unless required by applicable law or agreed to in writing, software
# distributed under the License is distributed on an "AS IS" BASIS,
# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
# See the License for the specific language governing permissions and
# limitations under the License.
#
import json
import requests
import sys
import time
import ConfigParser
import StringIO
review_url = "https://reviews.apache.org"
review_user = "XXXXXXXX"
review_pass = "XXXXXXXX"
jenkins_url = "http://jenkins.cloudstack.org"
jenkins_user = "XXXXXXXX"
jenkins_pass = "XXXXXXXX"
jenkins_job = "cloudstack-master-with-patch"
max_reviews_per_run = 1
def retrieve_object(url, params):
r = requests.get(url, params=params)
response = json.loads(r.text)
if response['stat'] != "ok":
pretty_print(response)
raise Exception("Exception while retrieving object")
return response
def get_repository_id_for_name(name):
params = { 'max-results' : '1000' }
result = retrieve_object(review_url+"/api/repositories", params)
for repository in result['repositories']:
if name == repository['name']:
return repository['id']
print "not found"
raise Exception
def pretty_print(json_object):
print json.dumps(json_object, sort_keys=True, indent=4, separators=(',', ': '))
def wait_for_job_completion(build_details):
next_build = build_details['nextBuildNumber']
job_str = "[ " + jenkins_job + "#" + str(next_build )+ " ]"
while build_details['lastBuild']['number'] < next_build:
print job_str + " is pending"
time.sleep(15) # TODO Implement some sort of timeout
r = requests.get(jenkins_url + "/job/" + jenkins_job + "/api/json")
build_details = json.loads(r.text)
print job_str + " is running"
r = requests.get(jenkins_url + "/job/" + jenkins_job + "/" + str(next_build) + "/api/json")
build_status = json.loads(r.text)
while build_status['building'] == True :
print job_str + " is running"
time.sleep(15) # TODO Implement some sort of timeout
r = requests.get(jenkins_url + "/job/" + jenkins_job + "/" + str(next_build) + "/api/json")
build_status = json.loads(r.text)
print job_str + " completed with status " + build_status['result']
return build_status
def trigger_jenkins(review_id, branch, patch_file):
r = requests.get(jenkins_url + "/job/" + jenkins_job + "/api/json")
build_details = json.loads(r.text)
print "Triggering jenkins jobs " + jenkins_job + "#" + str(build_details['nextBuildNumber']) + " for review_request " + str(review_id) + " on branch " + branch
# parameters to pass to jenkins
# patch.diff: the patch file
# review_id : the id of the review
# branch : the branch to use (defaults to master)
files = {'patch.diff': ('patch.diff', patch_file)}
r=requests.post(jenkins_url + "/job/" + jenkins_job + "/buildWithParameters", auth=(jenkins_user,jenkins_pass), files=files)
if (r.status_code == 404):
raise Exception("Job " + jenkins_job + " not found on " + jenkins_url)
if (r.status_code != 200):
raise Exception("Failed to trigger job " + jenkins_job + " on " + jenkins_url + ", status code " + str(r.status_code))
return build_details
def update_review(reviewrequest_id, message, ship_it):
params = { "body_top" : message, "public" : True, "ship_it" : ship_it }
auth = (review_user,review_pass)
r = requests.post(review_url + "/api/review-requests/" + str(reviewrequest_id) + "/reviews/", data=params, auth=auth)
response = json.loads(r.text)
if response['stat'] != "ok":
pretty_print(response)
raise Exception("Exception while retrieving object")
print "review " + str(reviewrequest_id) + " updated"
# needs_review
# get the date of the last review by 'review_user'
# get the date of the last upload of 'theDiff'
# decide by the dates whether review is neccessary
def needs_review(review_request):
reviews = retrieve_object(review_url + "/api/review-requests/" + str(review_request['id']) + "/reviews/", {'max-results':1000})
if not review_request['links'].has_key('diffs'):
print "No diff files found for this review, nothing to test"
return False
diffs = retrieve_object(review_request['links']['diffs']['href'], None)
revision = diffs['total_results']
# seems redundant but it fixes some strange bug; investigate?
if revision < 1:
print "no diff found!?!"
return False
latest_diff = diffs['diffs'][revision-1]
latest_upload_date = latest_diff['timestamp']
print "time of latest upload: " + latest_upload_date
for review in reviews['reviews']:
reviewer = review['links']['user']['title']
if (reviewer == review_user):
latest_review_date = review['timestamp']
# print reviewer + " reviewed at " + latest_review_date
if latest_review_date > latest_upload_date:
return False
if review['ship_it'] == True :
# Already shipped, skip it
return False
# print "pending review since " + latest_upload_date
return True
# check_reviews
# This function will connect to reviewboard and check if then jenkins user has posted any reviews.
# if it didn't it will trigger a jenkins build with the review id and branch for that review
def check_reviews():
repoid = get_repository_id_for_name('cloudstack-git')
params = { 'repository' : repoid, 'max-results': max_reviews_per_run}
review_requests = retrieve_object('http://reviews.apache.org/api/review-requests', params)
for review_request in review_requests['review_requests']:
print "[ review " + str(review_request['id']) + " ] " + review_request['summary'] + " (" + review_request['status'] + ") "
if not review_request['status'] == "pending":
print "Review has state " + review_request['status'] + " skipping jenkins build"
continue
branch = review_request['branch']
review_id = review_request['id']
# TODO Add support for multiple branches, for now assume everything is for master
#if branch=="" or branch.contains("master"):
branch = "master"
if needs_review(review_request):
diffs = retrieve_object(review_request['links']['diffs']['href'], None)
revision = diffs['total_results']
latest_diff = diffs['diffs'][revision-1]
headers = { "Accept" : "text/x-patch" }
r = requests.get(latest_diff['links']['self']['href'], headers=headers)
buf = StringIO.StringIO(r.text)
line = buf.readline()
patch_file = ''
if line.startswith('diff'):
# Regular patch file
patch_file = r.text
elif line.startswith('From '):
# Git format-patch, strip header and footer
while not line.startswith('diff'):
line = buf.readline()
while not (line.rstrip() == '--'):
patch_file += line
line = buf.readline()
build_details = trigger_jenkins(review_request['id'], branch, patch_file)
build_status = wait_for_job_completion(build_details)
# TODO Check if this is my build by comparing review id
message = ""
shipit = False
if build_status['result'] == "SUCCESS" :
message += "Review " + str(review_id) + " PASSED the build test\n"
shipit = True
else:
message += "Review " + str(review_id) + " failed the build test : " + build_status['result'] + "\n"
message += "The url of build " + build_status['fullDisplayName'] + " is : " + build_status['url']
print "Updating review with comment : " + message
update_review(review_id, message, shipit)
else:
print "Already reviewed"
try:
config = ConfigParser.ConfigParser()
config.readfp(open("reviewboard_testpatch.ini"))
review_url = config.get("reviewboard", "url")
review_user = config.get("reviewboard", "username")
review_pass = config.get("reviewboard", "password")
jenkins_url = config.get("jenkins", "url")
jenkins_user = config.get("jenkins", "username")
jenkins_pass = config.get("jenkins", "password")
jenkins_job = config.get("jenkins", "job")
max_reviews_per_run = config.get("general", "maxreviews")
except:
raise Exception("Unable to read configuration file")
check_reviews()