Re: RB 1.7.7.1 Internal Server Error.

2013-05-24 Thread J Arrizza
Ok. We'll just skip the review on this one change. Thanks for your help! John On Fri, May 24, 2013 at 4:51 PM, Christian Hammond wrote: > Hi, > > I don't know. It's not our timeout. That's pretty much an unmanageably > long diff though, and you're going to hit other issues with it as well (not

Re: RB 1.7.7.1 Internal Server Error.

2013-05-24 Thread Christian Hammond
Hi, I don't know. It's not our timeout. That's pretty much an unmanageably long diff though, and you're going to hit other issues with it as well (not the least of which is review time). Christian -- Christian Hammond - chip...@chipx86.com Review Board - http://www.reviewboard.org Beanbag, In

Re: RB 1.7.7.1 Internal Server Error.

2013-05-24 Thread J Arrizza
Christian, Looks like it is a timeout. There are only 9 changesets in the repo (it's a new one) and rev 4 is in fact a big change. I used the used the incorrect command to do the diff (hg diff -r 4). The correct mercurial command is hg diff -c 4 | wc 136638 672689 5040973 (Sorry for th

Re: RB 1.7.7.1 Internal Server Error.

2013-05-24 Thread Christian Hammond
Okay. Well, the stack trace indicates it has to do with reading the uploaded file. There's an I/O error at some point reading that. I wouldn't be able to say why without some extensive debugging in there. I should point out that this is all inside Django code where this loading is happening, an

Re: RB 1.7.7.1 Internal Server Error.

2013-05-24 Thread J Arrizza
No, it's quite small: hg diff -r 4 | wc 106 5393737 On Fri, May 24, 2013 at 3:00 PM, Christian Hammond wrote: > How big is the diff you're trying to upload? Could it be just too large > and timing out? > > Christian > > -- > Christian Hammond - chip...@chipx86.com > Review Boar

Re: RB 1.7.7.1 Internal Server Error.

2013-05-24 Thread Christian Hammond
How big is the diff you're trying to upload? Could it be just too large and timing out? Christian -- Christian Hammond - chip...@chipx86.com Review Board - http://www.reviewboard.org Beanbag, Inc. - http://www.beanbaginc.com On Fri, May 24, 2013 at 2:40 PM, J Arrizza wrote: > Hi Christian, >

Re: RB 1.7.7.1 Internal Server Error.

2013-05-24 Thread J Arrizza
Hi Christian, I fixed the directory owner to be -R www-data:www-data (it was incorrect). I deleted the current reviewboard.log I pressed "Save" in the Logging Settings I redid the hg postreview Here is the complete log cm5@cm5:/fmcapps/log$ cat reviewboard.log 2013-05-24 21:36:58,071 - INFO - -

Re: RB 1.7.7.1 Internal Server Error.

2013-05-24 Thread Christian Hammond
Hi, Can you check to see what reviewboard.log says when doing this? Christian -- Christian Hammond - chip...@chipx86.com Review Board - http://www.reviewboard.org Beanbag, Inc. - http://www.beanbaginc.com On Fri, May 24, 2013 at 2:25 PM, J Arrizza wrote: > We recently upgraded from RB 1.7 B

RB 1.7.7.1 Internal Server Error.

2013-05-24 Thread J Arrizza
We recently upgraded from RB 1.7 Beta 1 to 1.7.7.1 and are now getting Internal Server Errors on one repository, see below for the traceback. - Posting a review creates an empty review, i.e. no diffs. - I can create reviews with diffs in other repositories. Only one repo (so far) that is failing.