Re: Error on 1.5 - 1.7.1 upgrade: Model 'reviews.FileAttachmentComment' already has a field named 'extra_data'

2013-02-27 Thread Christian Hammond
Hi again, I actually think I may have a fix for Django Evolution, but I will need volunteers to test. I'll be sending out an e-mail with instructions shortly. Christian -- Christian Hammond - chip...@chipx86.com Review Board - http://www.reviewboard.org VMware, Inc. - http://www.vmware.com

Database upgrade issues - Possible fix

2013-02-27 Thread Christian Hammond
Hi everyone, So some of you have hit database upgrade issues before when upgrading to 1.7 (or, in some cases, 1.6). We've managed to figure out solutions for some of you, but not all. I've spent a lot of time trying to figure out solutions, find the common cases, and figure out a fix. Today,

Re: Error on 1.5 - 1.7.1 upgrade: Model 'reviews.FileAttachmentComment' already has a field named 'extra_data'

2013-02-27 Thread Nachiketh G
Hi Christian, Cool! I'll do the testing, i'm right now running a dry run of the migration on a test setup before actually doing it on the actual review board server, so I can assist you in whatever testing you want me to do. Thanks Regards, Nachiketh On Wed, Feb 27, 2013 at 1:35 PM, Christian

Web API: How to get the review id from the perforce change list number?

2013-02-27 Thread Steve
We have a perforce trigger that looks at the submitted change list number and makes an API call to review board to see if there is a corresponding review. If found, the url for the review is added to the submitted change list. Using the Web API 1.0, we use this url

Re: Web API: How to get the review id from the perforce change list number?

2013-02-27 Thread Steve
For some reason my initial post got mangled. Here it is again in briefer form: I need to be able to query the Web API to get the RB ID given a perforce change list number. With the 1.0 API, this url works: api/json/reviewrequests/repository/$repo/changenum/$changelist With the 2.0 API, I've

Re: SCMError('Unsupported View Type') when adding Clearcase Repository

2013-02-27 Thread hitendra chauhan
I also faced the same issue.. and struggling with it since 1 week.. On Wednesday, 27 February 2013 02:26:49 UTC+5:30, Jay Lawrence wrote: I've tried my best to setup ReviewBoard with Clearcase, but when I try to add a Repository the logfile ends up with the message 'Unsupported View

Re: Web API: How to get the review id from the perforce change list number?

2013-02-27 Thread Raja
You can add a status=submitted to the query string, so something like api/review-requests?changenum=$changeliststatus=submitted should get you the Submitted Requests. Same holds for discarded status as well. The default is pending, which is why you dont see the other requests. I dont see a way to

Re: Web API: How to get the review id from the perforce change list number?

2013-02-27 Thread Steve
Perfect! Thank you Raja. On Wednesday, February 27, 2013 8:53:10 AM UTC-8, Raja wrote: You can add a status=submitted to the query string, so something like api/review-requests?changenum=$changeliststatus=submitted should get you the Submitted Requests. Same holds for discarded status as

Re: SCMError('Unsupported View Type') when adding Clearcase Repository

2013-02-27 Thread Jay Lawrence
I've done some debug today by hacking inside the scmtools/clearcase.py file. I have 30 years of experience in C/C++/Perl, but my python is weak. I did manage to sprinkle enough logging.debug statements around to determine that the ClearCaseTool._get_view_type method is getting no output from

Re: You are not logged in

2013-02-27 Thread Charlie Davis
For sure. Here you go! display-1:~ cdavis$ post-review --debug 30663 RBTools 0.4.3 Python 2.7.2 (default, Jun 20

Getting Parent Diff from REST API.

2013-02-27 Thread Benjamin Mahler
Hi, I've been looking through the documentation in the hopes of finding a way to get the parent diff of a review. I see that on the 'Diff List Resource', the POST method can take the parent diff, but the GET cannot return it?

500 error when adding github repository for 1.7.6

2013-02-27 Thread Lei Zheng
Hey, experts: I got the following exception when I am adding a repository from github. Anyone can give me a clue what could be wrong? Thanks! Lei 013-02-27 17:56:22,444 - DEBUG - Trying SSH key ef811d10d36b2395a858f9f68c355057 2013-02-27 17:56:22,668 - DEBUG - userauth is OK 2013-02-27

Internal Server Error: /admin/db/scmtools/repository/add/ when adding repo for release 1.7.6

2013-02-27 Thread Lei Zheng
Hey, I installed reviewboard yesterday with release 1.7.6. However, when I add repo from github, I always get the following errors. Any clue what could be wrong? Thanks! 013-02-27 17:56:22,444 - DEBUG - Trying SSH key ef811d10d36b2395a858f9f68c355057 2013-02-27 17:56:22,668 - DEBUG -

Re: 500 error when adding github repository for 1.7.6

2013-02-27 Thread Christian Hammond
This looks like a setup problem of some sort in your database. First of all, when you upgraded to this version of Review Board, did you use easy_install, or pip? What's happening is that it's not finding the database model for the SCMTool (so, for Git). Can you go into the Admin UI - Database

Use of subprocess.popen with sequence and shell=ture is incorrect on linux, breaks clearcase.py

2013-02-27 Thread Jay Lawrence
This is the root cause of the problem entitle SCMError(Unsupported View Type earlier on this board, but appears fundamental enough that I've started a new thread with a more appropriate title In the scmtools/clearcase.py file, the following idiom is used to invoke clearcase commands

Re: Use of subprocess.popen with sequence and shell=ture is incorrect on linux, breaks clearcase.py

2013-02-27 Thread Jay Lawrence
Note, I just did cd /usr/llib/python2.6/site-packages/ReviewBoard-1.7.6-py2.6.egg/reviewboard find . -name \*.py -exec grep -i popen -A 10 {} \; -print /tmp/popen The result of this is printing every occurence of popen, and the 10 lines after it, in any file with a .py suffix. The

reviewboard not being able to add bazaar repository

2013-02-27 Thread WW Li
I am using reviewboard 1.7.5. - I uploaded ssh key to review board - In the repository type I chose 'Bazaar' - In the path I filled bzr+ssh://username@bzr.coppertreeanalytics.com/bzr/PD/ projectname/trunk - In Add repository page I clicked Save I got this error Connection closed: Unexpected end

Re: reviewboard not being able to add bazaar repository

2013-02-27 Thread WW Li
Thanks I didn't find a reviewboard.log file somewhere . is it supposed in reviewboardinstall_dir/logs? Do i need to upload both public and private keys? On Wednesday, February 27, 2013 5:30:59 PM UTC-8, Christian Hammond wrote: Hi, Can you check the reviewboard.log file for any messages

Re: Reviewboard returns error 200 when publishing a review

2013-02-27 Thread amac Mac
Hi Christian, I have not gotten the payload info as yet. The one update is at least two of the people who reported issues were ok when they used a later version of firefox. In both cases they were using 3.6.17. Thanks, -amac On Mon, Feb 25, 2013 at 11:39 AM, Christian Hammond

Re: reviewboard not being able to add bazaar repository

2013-02-27 Thread WW Li
I see a Unable to import settings_local.py: No module named settings_local error in error log On Wednesday, February 27, 2013 5:42:28 PM UTC-8, WW Li wrote: Thanks I didn't find a reviewboard.log file somewhere . is it supposed in reviewboardinstall_dir/logs? Do i need to upload both

Re: reviewboard not being able to add bazaar repository

2013-02-27 Thread Christian Hammond
Try adding this to your $sitedir/htdocs/reviewboard.wsgi file: os.environ['PYTHONPATH'] = '/path/to/site/conf:' + os.environ.get('PYTHONPATH', '') (Make sure to replace the /path/to/site part.) See if that helps any, and let me know. Christian -- Christian Hammond - chip...@chipx86.com

Re: reviewboard not being able to add bazaar repository

2013-02-27 Thread Christian Hammond
Actually, first, please upgrade to 1.7.6 and see if that fixes it for you. There's a potential fix in there. If that doesn't work, please let me know that, and try the suggestion I made below. Christian -- Christian Hammond - chip...@chipx86.com Review Board - http://www.reviewboard.org

Re: reviewboard not being able to add bazaar repository

2013-02-27 Thread WW Li
It works! Thanks a lot :) On Wednesday, February 27, 2013 8:18:56 PM UTC-8, Christian Hammond wrote: Try adding this to your $sitedir/htdocs/reviewboard.wsgi file: os.environ['PYTHONPATH'] = '/path/to/site/conf:' + os.environ.get('PYTHONPATH', '') (Make sure to replace the /path/to/site

Issue 2914 in reviewboard: Allow publishing issue fixed comments in e-mails sent out

2013-02-27 Thread reviewboard
Status: New Owner: Labels: Type-Enhancement Priority-Medium New issue 2914 by sbress...@gmail.com: Allow publishing issue fixed comments in e-mails sent out http://code.google.com/p/reviewboard/issues/detail?id=2914 *** For customer support, please post to reviewbo...@googlegroups.com

Issue 2915 in reviewboard: Allow publishing multiple responses and a diff all at once

2013-02-27 Thread reviewboard
Status: New Owner: Labels: Type-Enhancement Priority-Medium New issue 2915 by sbress...@gmail.com: Allow publishing multiple responses and a diff all at once http://code.google.com/p/reviewboard/issues/detail?id=2915 *** For customer support, please post to reviewbo...@googlegroups.com