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

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 VMware

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 Revie

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 > /logs? > > Do i need to upload both public and private ke

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 wrote: > Okay, I s

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 /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 from when you > per

Re: reviewboard not being able to add bazaar repository

2013-02-27 Thread Christian Hammond
Hi, Can you check the reviewboard.log file for any messages from when you performed this action? Hopefully there's something in there that would help. Can you also go to your SSH settings and verify that there's both an SSH key there, and it's the one you expect? Christian -- Christian Hammo

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://@bzr.coppertreeanalytics.com/bzr/PD/ /trunk" - In "Add repository" page I clicked "Save" I got this error "Connection closed: Unexpected end of message.

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 onl

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 cm

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

2013-02-27 Thread Jay Lawrence
I have debugged this problem and it is a fundamental Python issue with the use of subprocess.popen with cmdline as a sequence and shell=true. Because this appears fundamental and may attract more attention, I'm going to start a new thread on the topic. Jay On Tuesday, February 26, 2013 3:56:49

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

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? http://www.reviewboard.org/docs/manual/dev/webapi/2.0/resources/diff-l

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 17:56:2

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

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, Ju

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 the

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=$changelist&status=submitted > > should get you the Submitted Requests. Same holds for "discarded" statu

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=$changelist&status=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 wa

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

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 t

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 api/json/reviewrequests/r

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

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, I

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 On