Re: Issue 489 in reviewboard: [ ] DON'T SHIP IT button

2015-05-06 Thread reviewboard
Comment #43 on issue 489 by csipak.a...@ardinsys.eu: [ ] DON'T SHIP IT button https://code.google.com/p/reviewboard/issues/detail?id=489 @pvgodd: looks like this issue got forgotten. Last time it was touched by a project member, it was bound to the 1.8 milestone (after it was repeatedly

Re: Issue 489 in reviewboard: [ ] DON'T SHIP IT button

2015-05-06 Thread reviewboard
Comment #44 on issue 489 by trowb...@gmail.com: [ ] DON'T SHIP IT button https://code.google.com/p/reviewboard/issues/detail?id=489 It's not forgotten, just that this is a relatively tricky thing (literally everyone has an opinion about how ship-its should be improved/changed and whatever

Issue 3863 in reviewboard: Web API should provide an id field for a review request's every depends_on entry

2015-05-06 Thread reviewboard
Status: New Owner: Labels: Type-Enhancement Priority-Medium New issue 3863 by csipak.a...@ardinsys.eu: Web API should provide an id field for a review request's every depends_on entry https://code.google.com/p/reviewboard/issues/detail?id=3863 What version are you running? 2.0.15

Re: Issue after ReviewBoard upgrade

2015-05-06 Thread Papaswede
Ok, I tried that and that got rid of one of the error messages (the one that was referencing an invalid path), but the other two are still there, and those two messages do reference a valid path.

Re: Case of the strange disappearing diff

2015-05-06 Thread Christian Hammond
Hey Andrew, By any chance, this file that went missing.. was it a modification of a newly-introduced file not yet pushed upstream? So, something like: o -- MYFILE is modified | o -- MYFILE is newly created | o master | Christian -- Christian Hammond - chip...@chipx86.com Review Board -

Re: Issue after ReviewBoard upgrade

2015-05-06 Thread Papaswede
The webserver is actually running as the apache user. We are using Oracle Enterprise Linux 6, and SELinux is enabled. I had to get our IT department to make some changes to SELinux while installing MySQL and Apache. Are there any known things that I should check with regards to SELinux that I

Case of the strange disappearing diff

2015-05-06 Thread Andrew MacLeod
I have one user who posts a diff from a git workspace. The diff appears on RB for a few minutes and then disappears just showing the This is an empty file display. Reviews in other workspaces work as expected (ie. don't go missing) I am unable to reproduce using the downloaded diff in my own

Re: Issue after ReviewBoard upgrade

2015-05-06 Thread Papaswede
Ah, your last message caused me to dig some more, and I was able to find the issue. Apache was running the child processes as the wrong user and they did therefore not have access to the directories the way they should. So, it was a misconfiguration in my httpd.conf. Thanks! On Wednesday, May

Re: Issue after ReviewBoard upgrade

2015-05-06 Thread Christian Hammond
Looks like your web server is running as root, which is bad. It really should be using its own dedicated user. That's not the source of the problem, but it's something to be aware of. What Linux distro are you using? Any chance SELinux is enabled? If so, that can cause this. Christian --

Issue 3865 in reviewboard: rbt post fails when random repository is misconfigured in reviewboard

2015-05-06 Thread reviewboard
Status: New Owner: Labels: Type-Defect Priority-Medium New issue 3865 by jonathan...@gmail.com: rbt post fails when random repository is misconfigured in reviewboard https://code.google.com/p/reviewboard/issues/detail?id=3865 What version are you running? rbtools 0.7.2 reviewboard 2.0.6

Jenkins-ReviewBot

2015-05-06 Thread Sri Ram Kannan
Hi, I found a plugin that triggers pre-commit test through the diff files submitted in the ReviewBoard. The link of the plugin is : https://wiki.jenkins-ci.org/display/JENKINS/Jenkins-Reviewbot Has anyone in this group tried this with newer review board instance ? I request to guide me on how

Re: Issue after ReviewBoard upgrade

2015-05-06 Thread Papaswede
Ah, well maybe that is the issue then. Ok, so let me ask if I need to go about this a different way. We currently have reviewboard running on an old, antiquated box. I had them set up a new VM for me that I am migrating reviewboard to. I can't install reviewboard in the same location on this

Re: Installing ReviewBoard using pip

2015-05-06 Thread Sri Ram Kannan
Hi chris, Thanks for your update. I will go with easy_install. On Wednesday, May 6, 2015 at 2:37:57 PM UTC-4, Christian Hammond wrote: Hi Sri, For now, you need to use easy_install, for a number of reasons: 1. Pip doesn't support eggs (we'll be releasing Wheels in the next release of

Re: Issue 3862 in reviewboard: Loading Patch hangs due to french local

2015-05-06 Thread reviewboard
Comment #1 on issue 3862 by msu...@aurea.com: Loading Patch hangs due to french local https://code.google.com/p/reviewboard/issues/detail?id=3862 The patch file in question was generated on a windows machine configured with a french local using TurtoiseSVN. It looks like this changed the

Issue 3864 in reviewboard: rbt post //depot/path/...@r1,@r2 throws an exception in 0.7.2 and no review request gets created

2015-05-06 Thread reviewboard
Status: New Owner: Labels: Type-Defect Priority-Medium New issue 3864 by seide.al...@gmail.com: rbt post //depot/path/...@r1,@r2 throws an exception in 0.7.2 and no review request gets created https://code.google.com/p/reviewboard/issues/detail?id=3864 *** READ THIS BEFORE POSTING!

Installing ReviewBoard using pip

2015-05-06 Thread Sri Ram Kannan
Hi, I read few posts in this group that Review board cannot installed using pip. But I see reviewboard in PyPi page and when I tried to install using pip, I got the below error: Could not find any downloads that satisfy the requirement ReviewBoard Some externally hosted files were

Re: Issue 3863 in reviewboard: Web API should provide an id field for a review request's every depends_on entry

2015-05-06 Thread reviewboard
Updates: Status: Confirmed Labels: Component-API Comment #1 on issue 3863 by chip...@gmail.com: Web API should provide an id field for a review request's every depends_on entry https://code.google.com/p/reviewboard/issues/detail?id=3863 A worthwhile thing to add. What you

Re: RBTools 0.7.2: posting a review using a range of perforce change lists no longer works

2015-05-06 Thread Steve
Hi Chris, Just to close the loop on this topic - the problem with the diffs that I mentioned turned out to be a memcache issue. Turns out that using memcache on a test machine where you're continually deleting and recreating a perforce depot and rb site, thus reusing change list numbers, can

Customize Administrator Dashboard

2015-05-06 Thread Tanuj Sharma
Hello, First of all, I would like to appreciate your work. ReviewBoard is a very solution for online code reviews. I want to know if there some way I can customize administrator dashboard to show me more graphs like for a particular user how many comments were given on a review package? How