-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/43747/
-----------------------------------------------------------
Review request for mesos, Joris Van Remoortere and Kevin Klues.
Bugs: MESOS-4713
https://issues.apache.org/jira/browse/MESOS-4713
Repository: mesos
Description
-------
Fixed ReviewBot to catch circular dependencies in review requests.
Diffs
-----
support/verify_reviews.py f0b9996b04e29e5d70ff57784c46d00e5ec4e45c
Diff: https://reviews.apache.org/r/43747/diff/
Testing
-------
Tested locally with a circular dep.
? mesos git:(vinod/reviewbot_recursive) BUILD_URL=foo
./support/verify_reviews.py kone vinod 1
"?to-groups=mesos&status=pending&start=2"
git rev-parse HEAD
Checking if review: 43418 needs verification
Skipping blocking review 43418
Checking if review: 43691 needs verification
Skipping blocking review 43691
Checking if review: 43689 needs verification
Skipping blocking review 43689
Checking if review: 43697 needs verification
Skipping blocking review 43697
Checking if review: 43698 needs verification
Skipping blocking review 43698
Checking if review: 43699 needs verification
Skipping blocking review 43699
Checking if review: 43692 needs verification
Skipping blocking review 43692
Checking if review: 43693 needs verification
Skipping blocking review 43693
Checking if review: 43694 needs verification
Skipping blocking review 43694
Checking if review: 43695 needs verification
Skipping blocking review 43695
Checking if review: 43271 needs verification
Skipping blocking review 43271
Checking if review: 43272 needs verification
Latest diff timestamp: 2016-02-18 19:19:48
Verifying review 43272
Dependent review: https://reviews.apache.org/api/review-requests/43271/
Dependent review: https://reviews.apache.org/api/review-requests/43261/
Dependent review: https://reviews.apache.org/api/review-requests/43271/
Posting review: Bad review!
Reviews applied: [43272, 43271, 43261]
Error:
Circular dependency detected for review 43271.Please fix the 'depends_on' field.
Error handling URL
https://reviews.apache.org/api/review-requests/43272/reviews/: UNAUTHORIZED
({"stat": "fail", "err": {"msg": "The username or password was not correct",
"code": 104}})
git clean -fd
git reset --hard fb779576521abc35d99f2b8834684f3a8f020895
Thanks,
Vinod Kone
