> On Dec. 17, 2015, 8:37 a.m., Kevin Klues wrote: > > 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.
Thanks Kevin, Had discarded 41189. Since the changes were on the same branch the dependency got created but persisted after discarding 41189. Have updated the dependency to 41187 - Avinash ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41380/#review110922 ----------------------------------------------------------- On Dec. 17, 2015, 9:33 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, 9:33 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 > >
