Re: expected tuple with server trust credentials

2014-06-11 Thread David Trowbridge
Is the new server using pysvn or subvertpy? -David On Tue, Jun 10, 2014 at 10:24 PM, bhagwat mane manebhag...@gmail.com wrote: Hi All, I am getting this error while adding or updating SVN repository (which is in *HTTPS* mode) in reviewboard application. It works for *HTTP* but not for

Re: expected tuple with server trust credentials

2014-06-11 Thread bhagwat mane
Both On Wednesday, June 11, 2014 12:54:32 PM UTC+5:30, David Trowbridge wrote: Is the new server using pysvn or subvertpy? -David On Tue, Jun 10, 2014 at 10:24 PM, bhagwat mane maneb...@gmail.com javascript: wrote: Hi All, I am getting this error while adding or updating SVN

Re: expected tuple with server trust credentials

2014-06-11 Thread David Trowbridge
Can you try removing subvertpy and running with only the pysvn backend? -David On Wed, Jun 11, 2014 at 1:04 AM, bhagwat mane manebhag...@gmail.com wrote: Both On Wednesday, June 11, 2014 12:54:32 PM UTC+5:30, David Trowbridge wrote: Is the new server using pysvn or subvertpy? -David

Re: expected tuple with server trust credentials

2014-06-11 Thread bhagwat mane
Removed subvertpy, restart apache2 now getting error as *u'failures'*. - Bhagwat On Wednesday, June 11, 2014 1:40:38 PM UTC+5:30, David Trowbridge wrote: Can you try removing subvertpy and running with only the pysvn backend? -David On Wed, Jun 11, 2014 at 1:04 AM, bhagwat mane

Re: rbt post using Mercurial

2014-06-11 Thread Ali Ghorashi
Thanks Christian, See the attached file for the debug output. There is nothing special about the repo setup: I don't have any branches. Everything up to changeset 100:79c09fdfad23 is pushed into the main repo. I changed a file with changeset: 101:1170d2410ead and then did a rbt post. To say it

Issue buttons not working

2014-06-11 Thread Yugesh Kambham
Hello. We go into a strange situation where issue - drop, resolve buttons are not working consistently. We have a review with 10 images attached with over 160 issues(76 open), when I(owner of review) try to drop/resolve/add comment to an open issue nothing is happening, reviewboard is not

Re: Web API: uploading raw diffs

2014-06-11 Thread George Miroshnykov
Hi Christian, Thanks for your help! I'm trying to generate a squashed diff of multiple commits by specifying first and last commits like this: hg diff --rev 'f0c92cb17217::5db37ca696a4' Adding the --git flag doesn't change the output. Is there a way to create review requests based on such

Error creating review request: Unable to authenticate with the repository using the provided credentials. (HTTP 403, API Error 218)

2014-06-11 Thread Diego Velasquez
Hello, I set up review board in my Ubuntu VM with success I can see the review board administration from my local machine(Mac OS). We use perforce for our development, when I try to post a review trough a terminal but I got the following error: rbt post -o -d --p4-port $P4PORT CLNUMBER

ReviewBoard now charmed for Juju

2014-06-11 Thread adam.coll...@gmail.com
Hi, For the past couple of days I've been working on a Juju[1] Charm for Review Board, which I'm happy to announce has just been approved and released into the official Charm Store[2]. For those of you who aren't aware, Juju is a service orchestration technology which makes it really easy to

Re: ReviewBoard now charmed for Juju

2014-06-11 Thread Christian Hammond
Hey Adam, Very cool! Anything you need from us for this? Christian --  Christian Hammond - christ...@beanbaginc.com Review Board - http://www.reviewboard.org Beanbag, Inc. - http://www.beanbaginc.com On June 11, 2014 at 1:20:36 PM, adam.coll...@gmail.com (adam.coll...@gmail.com) wrote: Hi,

Re: Issue buttons not working

2014-06-11 Thread Christian Hammond
Hi, What version of Review Board is this? And what version of what browser? Can you check the browser’s JavaScript console log when you attempt to modify an issue and see if any errors appear? Christian --  Christian Hammond - christ...@beanbaginc.com Review Board - http://www.reviewboard.org

Re: ReviewBoard now charmed for Juju

2014-06-11 Thread Adam Collard
Hi Christian, It wouldn't hurt to get some eye balls on the code. Very much in its infancy but I have grand plans to add more awesomeness so want a solid starting point. I made the logo in SVG by hand copying the one on the site. Is that ok? Is there an official logo you'd like me to use

Re: ReviewBoard now charmed for Juju

2014-06-11 Thread Christian Hammond
Hey Adam, Review Board 2.1 will have a new repository selector that will make it very easy to configure repositories on supported hosting services. Right now, I’m targeting GitHub for development, but we’ll bring support to Bitbucket pretty soon too. This is all still very young though. (Also

Re: expected tuple with server trust credentials

2014-06-11 Thread Christian Hammond
This is a known problem with cert validation right now. We’re putting out a release by Friday at the latest (hoping to do later today or tomorrow) that will fix this. Christian --  Christian Hammond - christ...@beanbaginc.com Review Board - http://www.reviewboard.org Beanbag, Inc. - 

Re: Error creating review request: Unable to authenticate with the repository using the provided credentials. (HTTP 403, API Error 218)

2014-06-11 Thread Diego Velasquez
I got working by adding: os.environ['P4USER'] = to reviewboard.wsgi On Wednesday, June 11, 2014 3:47:30 PM UTC-4, Diego Velasquez wrote: Hello, I set up review board in my Ubuntu VM with success I can see the review board administration from my local machine(Mac OS). We use perforce

p4 custom tool

2014-06-11 Thread Diego Velasquez
Hi there, I am been trying to run rb post inside p4 by creating a custom tool which does the following: application = /usr/local/bin/rbt arguments = post -d --server=http://myserver --p4-client=$c --p4-port=$p --username=myuser --password=mypassword %C But for some reason the terminal

Re: Issue buttons not working

2014-06-11 Thread Yugesh Kambham
We are using v1.7.7.1. We tried IE10 and Chrome v35.0.1916.15 . Console log : POST https://xxx/reviewboard/api/review-requests/42036/reviews/67932/file-attachment-comments/ 400 (BAD REQUEST) jquery-1.8.2.min.cfa9051cc0b0.js:2 Uncaught TypeError: Cannot read property 'id' of undefined

Re: Issue buttons not working

2014-06-11 Thread Christian Hammond
Hi Yugesh, First thing I’d ask that you do is upgrade to something modern. 1.7.7.1 is from April of 2013, and we’ve made a lot of fixes in that time. Christian --  Christian Hammond - christ...@beanbaginc.com Review Board - http://www.reviewboard.org Beanbag, Inc. - http://www.beanbaginc.com

Re: Issue buttons not working

2014-06-11 Thread Yugesh Kambham
Upgrading would be big thing for us considering the other priorities, but we do have it in our plan. What do you think caused this? We have around 13 new files and 13 old files as of now on same review and over 70 issues open. Does size of review failing this? Reviews seem to be working fine

Re: Issue buttons not working

2014-06-11 Thread Christian Hammond
I can’t imagine what would have caused this aside from some issue on your server. Can you open the Network tab in Chrome’s developer tools, repeat a request, and show what the resulting payload looks like? Christian --  Christian Hammond - christ...@beanbaginc.com Review Board - 

Re: Issue buttons not working

2014-06-11 Thread Yugesh Kambham
Yes see below. [image: chrome-devtools://devtools/bundled/devtools.html?dockSide=undockedtoolbarColor=rgba(223,223,223,1)textColor=rgba(0,0,0,1)can_dock=trueupdateAppcache] file-attachment-comments/ /reviewboard/api/review-requests/42036/reviews/68007 POST 400 BAD REQUEST

Re: Issue buttons not working

2014-06-11 Thread Christian Hammond
I need to see the actual response payload. Is the failure only on file attachments? Do you ever see it for diffs? Christian --  Christian Hammond - christ...@beanbaginc.com Review Board - http://www.reviewboard.org Beanbag, Inc. - http://www.beanbaginc.com On June 11, 2014 at 3:05:51 PM,

Re: p4 custom tool

2014-06-11 Thread David Trowbridge
It looks like something is actually wrong with your execution path: it can't execute the 'p4' binary. -David On Wed, Jun 11, 2014 at 2:40 PM, Diego Velasquez dvelasquez1...@gmail.com wrote: Hi there, I am been trying to run rb post inside p4 by creating a custom tool which does the

Re: Issue 3312 in reviewboard: Diffs without embedded commit IDs can not be used

2014-06-11 Thread reviewboard
Comment #3 on issue 3312 by george.m...@gmail.com: Diffs without embedded commit IDs can not be used http://code.google.com/p/reviewboard/issues/detail?id=3312 Just tried to use rbt post which failed too, but maybe this is a separate issue:

Re: Issue 3376 in reviewboard: RBTools 0.6 - UnicodeEncodeError with ISO-8859-15

2014-06-11 Thread reviewboard
Comment #5 on issue 3376 by nicolas@gmail.com: RBTools 0.6 - UnicodeEncodeError with ISO-8859-15 http://code.google.com/p/reviewboard/issues/detail?id=3376 Hello, No, the issue is not causing only by this diff file. We have several others which generate the same error. As you can

Re: Issue 3329 in reviewboard: Show proper error message when attempting to publish review request with duplicate commit ID

2014-06-11 Thread reviewboard
Comment #6 on issue 3329 by thom.ger...@gmail.com: Show proper error message when attempting to publish review request with duplicate commit ID http://code.google.com/p/reviewboard/issues/detail?id=3329 Seeing this in deployed Reviewboard 2.0.1 with a MySQL db. user doing PUT

Re: Issue 3376 in reviewboard: RBTools 0.6 - UnicodeEncodeError with ISO-8859-15

2014-06-11 Thread reviewboard
Comment #6 on issue 3376 by trowb...@gmail.com: RBTools 0.6 - UnicodeEncodeError with ISO-8859-15 http://code.google.com/p/reviewboard/issues/detail?id=3376 Does this particular .diff file produce the exact same text in the Error data part of the traceback? -- You received this message

Re: Issue 3329 in reviewboard: Show proper error message when attempting to publish review request with duplicate commit ID

2014-06-11 Thread reviewboard
Comment #7 on issue 3329 by trowb...@gmail.com: Show proper error message when attempting to publish review request with duplicate commit ID http://code.google.com/p/reviewboard/issues/detail?id=3329 I think that yours is a separate bug. Would you mind filing it, and including the stack

Re: Issue 3410 in reviewboard: perforce post-submit review fails using rbt post when the submitted change set contains branch actions

2014-06-11 Thread reviewboard
Updates: Status: PendingReview Owner: trowb...@gmail.com Labels: Component-RBTools Comment #1 on issue 3410 by trowb...@gmail.com: perforce post-submit review fails using rbt post when the submitted change set contains branch actions

Re: Issue 3356 in reviewboard: Diff currently unavailable when reviewing change to file that is not pushed to remote repo yet

2014-06-11 Thread reviewboard
Comment #5 on issue 3356 by trowb...@gmail.com: Diff currently unavailable when reviewing change to file that is not pushed to remote repo yet http://code.google.com/p/reviewboard/issues/detail?id=3356 So the diffs there look correct--it's generating a diff between def456 and abc123 (the

Re: Issue 3354 in reviewboard: TypeError for New Review Request and View Diff after updade to Review Board 2.0

2014-06-11 Thread reviewboard
Updates: Status: ThirdParty Comment #8 on issue 3354 by trowb...@gmail.com: TypeError for New Review Request and View Diff after updade to Review Board 2.0 http://code.google.com/p/reviewboard/issues/detail?id=3354 This traceback is nonsense, and I can't even find ConfigItem used in

Re: Issue 2184 in reviewboard: Mercurial rename support: Pure file renames with *no actual changes* to the file fails with Diff did not apply cleanly

2014-06-11 Thread reviewboard
Comment #8 on issue 2184 by bruce.c...@gmail.com: Mercurial rename support: Pure file renames with *no actual changes* to the file fails with Diff did not apply cleanly http://code.google.com/p/reviewboard/issues/detail?id=2184 I came across this problem today: the problem is that patch

Re: Issue 2184 in reviewboard: Mercurial rename support: Pure file renames with *no actual changes* to the file fails with Diff did not apply cleanly

2014-06-11 Thread reviewboard
Updates: Status: ThirdParty Comment #9 on issue 2184 by trowb...@gmail.com: Mercurial rename support: Pure file renames with *no actual changes* to the file fails with Diff did not apply cleanly http://code.google.com/p/reviewboard/issues/detail?id=2184 That sounds like great news.

Re: Issue 3409 in reviewboard: List of which commits that are reviewed/not reviewed

2014-06-11 Thread reviewboard
Updates: Status: NeedInfo Comment #1 on issue 3409 by trowb...@gmail.com: List of which commits that are reviewed/not reviewed http://code.google.com/p/reviewboard/issues/detail?id=3409 If you're using the new post-commit UI in 2.0, you should see an icon next to all commits that