mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Mesos ReviewBot <revi...@mesos.apache.org>
Subject Re: Review Request 43747: Fixed ReviewBot to catch circular dependencies in review requests.
Date Fri, 19 Feb 2016 05:08:16 GMT

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/43747/#review119814
-----------------------------------------------------------



Patch looks great!

Reviews applied: [43747]

Passed command: export OS='ubuntu:14.04' CONFIGURATION='--verbose' COMPILER='gcc' ENVIRONMENT='GLOG_v=1
MESOS_VERBOSE=1'; ./support/docker_build.sh

- Mesos ReviewBot


On Feb. 19, 2016, 12:38 a.m., Vinod Kone wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/43747/
> -----------------------------------------------------------
> 
> (Updated Feb. 19, 2016, 12:38 a.m.)
> 
> 
> 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
> 
>


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message