Re: Facing issues submit review requests using support/post_review.py

2015-03-29 Thread Alex Rukletsov
Aditi, Check your .reviewboardrc file. REVIEWBOARD_URL should point to https://reviews.apache.org/, REPOSITORY should be mesos. Hope this helps, Alex On 29/03/2015 3:56 pm, Aditi Dixit aditi96di...@gmail.com wrote: Hi, I'm trying to create a review request but running support/post_review.py

Re: Facing issues submit review requests using support/post_review.py

2015-03-29 Thread Alex Rukletsov
Also make sure your patch (branch) is on top of current master, rebase if necessary. You can also manually upload the diff from ReviewBoard UI. On 29/03/2015 6:02 pm, Alex Rukletsov a...@mesosphere.io wrote: Aditi, Check your .reviewboardrc file. REVIEWBOARD_URL should point to

Facing issues submit review requests using support/post_review.py

2015-03-29 Thread Aditi Dixit
Hi, I'm trying to create a review request but running support/post_review.py keeps failing with the following output : Running 'rbt post' across all of ... efe1191ea53347f193dddc962030047e5eb45e0b - (HEAD, MESOS-2564) [MESOS-2564] Removed useless forward declaration comments (40 minutes ago)

Re: Splitting reviews and build emails into their own mailing lists

2015-03-29 Thread Anirudha Jadhav
+1 please On Sun, Mar 29, 2015 at 8:55 AM, haosdent haosd...@gmail.com wrote: +1 On Sun, Mar 29, 2015 at 12:56 PM, Yan Xu y...@jxu.me wrote: +1 -- Jiang Yan Xu y...@jxu.me @xujyan http://twitter.com/xujyan On Fri, Mar 27, 2015 at 12:19 PM, Vinod Kone vinodk...@apache.org wrote:

Re: Splitting reviews and build emails into their own mailing lists

2015-03-29 Thread Kapil Arya
+1. On Sun, Mar 29, 2015 at 9:16 AM, Anirudha Jadhav aniru...@nyu.edu wrote: +1 please On Sun, Mar 29, 2015 at 8:55 AM, haosdent haosd...@gmail.com wrote: +1 On Sun, Mar 29, 2015 at 12:56 PM, Yan Xu y...@jxu.me wrote: +1 -- Jiang Yan Xu y...@jxu.me @xujyan

Re: Splitting reviews and build emails into their own mailing lists

2015-03-29 Thread haosdent
+1 On Sun, Mar 29, 2015 at 12:56 PM, Yan Xu y...@jxu.me wrote: +1 -- Jiang Yan Xu y...@jxu.me @xujyan http://twitter.com/xujyan On Fri, Mar 27, 2015 at 12:19 PM, Vinod Kone vinodk...@apache.org wrote: Hi, What do you guys think about moving the review and build (CI) emails to

Re: Facing issues submit review requests using support/post_review.py

2015-03-29 Thread Aditi Dixit
Thanks for the response Alex. I checked the .reviewboardrc file and its parameters were the same that you mentioned. I had also taken care to rebase the so that my commit was over the current master. While trying to submit a patch manually through the ReviewBoard UI, I was getting an error that