mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Kevin Klues" <klue...@gmail.com>
Subject Re: Review Request 41380: Added repeated vip field to DiscoveryInfo and an instance_port field to Port
Date Thu, 17 Dec 2015 08:37:42 GMT

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


This review depends on 41189, but 41189 is somehow marked as "Invite-only", so it is not viewable
by the public.  This is causing the jenkins review-bot to fail with:

Verifying review 41381
Dependent review: https://reviews.apache.org/api/review-requests/41380/
Dependent review: https://reviews.apache.org/api/review-requests/41189/
Error handling URL https://reviews.apache.org/api/review-requests/41189/: FORBIDDEN

>From my reading of https://www.reviewboard.org/docs/manual/2.5/admin/configuration/access-control/
My guess is all you need to do is make sure that 41189 has the 'mesos' group added to it.
If no group is there, it will likely be treated as "Invite-only" since no public groups are
accessible.

- Kevin Klues


On Dec. 17, 2015, 3:31 a.m., Avinash sridharan wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/41380/
> -----------------------------------------------------------
> 
> (Updated Dec. 17, 2015, 3:31 a.m.)
> 
> 
> Review request for mesos, Adam B and Anand Mazumdar.
> 
> 
> Bugs: MESOS-4114
>     https://issues.apache.org/jira/browse/MESOS-4114
> 
> 
> Repository: mesos
> 
> 
> Description
> -------
> 
> Added repeated vip field to DiscoveryInfo and an instance_port field to Port
> 
> 
> Diffs
> -----
> 
>   include/mesos/mesos.proto 8ca213062c480f0266ffc51a621eb4a118140c77 
>   include/mesos/v1/mesos.proto 8f357b0fb778098ec66ac85d174bdd7e387954b5 
> 
> Diff: https://reviews.apache.org/r/41380/diff/
> 
> 
> Testing
> -------
> 
> make check, and make
> 
> 
> Thanks,
> 
> Avinash sridharan
> 
>


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