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

2015-04-01 Thread Adam Bordelon
We do accept Pull-Requests https://github.com/apache/mesos/pulls for small patches, especially (markdown) documentation patches; but for larger code changes we prefer the issue tracking, multi-line comments, and incremental diffs provided by ReviewBoard. On Tue, Mar 31, 2015 at 10:55 AM, haosdent

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

2015-03-31 Thread Alex Rukletsov
Aditi, for a single commit I use `git show sha patch.diff`. I would encourage you to use rbt tools though. Your github tree should look as follows before your run ./support/post-review.sh : O [your branch], active, checked out | ... | O [master] [upstream/master] | ... | O hope this

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

2015-03-31 Thread haosdent
If it is possible, I suggest use the workflow like https://github.com/apache/spark: Submit push request to github. On Wed, Apr 1, 2015 at 1:49 AM, Vinod Kone vinodk...@apache.org wrote: Are you still having issue with this? I've seen this issue before when my branch depends on a file that was

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

2015-03-31 Thread Vinod Kone
Are you still having issue with this? I've seen this issue before when my branch depends on a file that was recently committed. The mesos repo used by ReviewBoard is a mirror of the canonical mesos repo and there is sometimes lag in the commits from the latter reaching the former. But the issue

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: 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