You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@mesos.apache.org by an...@apache.org on 2018/09/04 20:58:32 UTC
[mesos] branch master updated: Added --skip-verify flag to
verify-reviews.py to skip verification.
This is an automated email from the ASF dual-hosted git repository.
andschwa pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/mesos.git
The following commit(s) were added to refs/heads/master by this push:
new ac766c8 Added --skip-verify flag to verify-reviews.py to skip verification.
ac766c8 is described below
commit ac766c8bd0fba348399d8eac52f75f62e64db64e
Author: Dragos Schebesch <v-...@microsoft.com>
AuthorDate: Tue Sep 4 13:51:05 2018 -0700
Added --skip-verify flag to verify-reviews.py to skip verification.
This flag allows devs to skip verification and just print or write to
a file the review ids that need verification.
Review: https://reviews.apache.org/r/68497/
---
support/verify-reviews.py | 6 +++++-
1 file changed, 5 insertions(+), 1 deletion(-)
diff --git a/support/verify-reviews.py b/support/verify-reviews.py
index eb91eb9..ce9a359 100755
--- a/support/verify-reviews.py
+++ b/support/verify-reviews.py
@@ -63,6 +63,9 @@ def parse_parameters():
parser.add_argument("-r", "--reviews", type=int, required=False,
default=-1, help="The number of reviews to fetch,"
" that will need verification")
+ parser.add_argument("--skip-verify", action='store_true', required=False,
+ help="Skip the verification and just write the review"
+ " ids that need verification")
default_hours_behind = 8
datetime_before = (datetime.datetime.now() -
datetime.timedelta(hours=default_hours_behind))
@@ -288,7 +291,8 @@ def main():
continue
review_ids.append(str(review_request["id"]))
num_reviews += 1
- verify_review(review_request, handler)
+ if not parameters.skip_verify:
+ verify_review(review_request, handler)
verification_needed_write(review_ids, parameters)