post-review error

2011-08-18 Thread Deva
Hi,
Am getting below error while posting the review in windows. It says
'Unable to access http://x.x.x.x/api/json/accounts/login/. The host
path may be invalid
'

C:\workspace\BDBJE>python C:\rbtools\rbtools\postreview.py --debug
>>> svn info
>>> repository info: Path: http://x.x.x.x/wds, Base path: /projects/NavApp/
devBranches/1.8.X-BDB/BDBJE, Supports changesets: False
>>> svn diff --diff-cmd=diff
>>> svn info src/main/java/com/macys/bdb/cache/da/exception/DataAccessorExceptio
n.java
>>> svn info src/main/java/com/macys/bdb/cache/da/exception/DataAccessorExceptio
n.java
>>> svn info src/main/java/com/macys/bdb/cache/da/exception/DataAccessorExceptio
n.java
>>> svn info src/main/java/com/macys/bdb/cache/da/exception/DataAccessorExceptio
n.java
>>> Looking for 'x.x.x.x /' cookie in C:\Documents and Settings\devaraj_m\
Local Settings\Application Data\.post-review-cookies.txt
>>> Couldn't load cookie file: [Errno 2] No such file or directory: 'C:\\Documen
ts and Settings\\devaraj_m\\Local Settings\\Application Data\\.post-
review-cooki
es.txt'
==> Review Board Login Required
Enter username and password for Review Board at http://x.x.x.x/
Username: deva
Password:
>>> Logging in with username "deva"
>>> HTTP POSTing to http://x.x.x.x/api/json/accounts/login/: {'userna
me': 'deva', 'password': '**'}
Unable to access http://x.x.x.x/api/json/accounts/login/. The host pa
th may be invalid


Can anyone please help me ?

Regards,
Deva

-- 
Want to help the Review Board project? Donate today at 
http://www.reviewboard.org/donate/
Happy user? Let us know at http://www.reviewboard.org/users/
-~--~~~~--~~--~--~---
To unsubscribe from this group, send email to 
reviewboard+unsubscr...@googlegroups.com
For more options, visit this group at 
http://groups.google.com/group/reviewboard?hl=en


Re: Slow response after publishing review or comments

2011-08-18 Thread rmc
Hello Christian,

I confirm that email is the culprit.

The interface should not block because of it, shouldn't this be done in the 
background? 

R

-- 
Want to help the Review Board project? Donate today at 
http://www.reviewboard.org/donate/
Happy user? Let us know at http://www.reviewboard.org/users/
-~--~~~~--~~--~--~---
To unsubscribe from this group, send email to 
reviewboard+unsubscr...@googlegroups.com
For more options, visit this group at 
http://groups.google.com/group/reviewboard?hl=en

Re: post-review cannot find the SVN repository

2011-08-18 Thread Radha Srinivasan
Christian,
  Thanks for the pointer. The trailing / was the culprit. Removing that
fixed the problem.
Thanks for your help.
Radha

On Wed, Aug 17, 2011 at 7:46 PM, Christian Hammond wrote:

> Hi,
>
> The path must match exactly, character for character. Check that therea not
> a typo or trailing / inconsistency.
>
> Christian
>
>
>
> On Wednesday, August 17, 2011, Radha Srinivasan 
> wrote:
> > We have ReviewBoard working with one SVN Repository. I added a new
> > repository from a different SVN server. When we execute post-review
> > for the files in the newly added repository we get an error
> >
> > There was an error creating this review request.
> >
> > The repository path "http://devproj.internal.com/svn/cocoon"; is not in
> > the list of known repositories on the server.
> >
> > Ask the administrator to add this repository to the Review Board
> > server.
> > For information on adding repositories, please read
> >
> http://www.reviewboard.org/docs/manual/dev/admin/management/repositories/
> >
> >
> > I used the GUI to add the above repository to the ReviewBoard Server
> > and I can see that it is added. Any guidance on what the problem could
> > be?
> > Thanks,
> > Radha
> >
> > --
> > Want to help the Review Board project? Donate today at
> http://www.reviewboard.org/donate/
> > Happy user? Let us know at http://www.reviewboard.org/users/
> > -~--~~~~--~~--~--~---
> > To unsubscribe from this group, send email to
> reviewboard+unsubscr...@googlegroups.com
> > For more options, visit this group at
> http://groups.google.com/group/reviewboard?hl=en
>
> --
> --
> Christian Hammond - chip...@chipx86.com
> Review Board - http://www.reviewboard.org
> VMware, Inc. - http://www.vmware.com
>
> --
> Want to help the Review Board project? Donate today at
> http://www.reviewboard.org/donate/
> Happy user? Let us know at http://www.reviewboard.org/users/
> -~--~~~~--~~--~--~---
> To unsubscribe from this group, send email to
> reviewboard+unsubscr...@googlegroups.com
> For more options, visit this group at
> http://groups.google.com/group/reviewboard?hl=en

-- 
Want to help the Review Board project? Donate today at 
http://www.reviewboard.org/donate/
Happy user? Let us know at http://www.reviewboard.org/users/
-~--~~~~--~~--~--~---
To unsubscribe from this group, send email to 
reviewboard+unsubscr...@googlegroups.com
For more options, visit this group at 
http://groups.google.com/group/reviewboard?hl=en

Re: SVN user config

2011-08-18 Thread Justin Georgeson
This may be a silly question but can you restrict which repositories
each user can work with?

On Aug 17, 1:42 pm, Christian Hammond  wrote:
> Hi Justin,
>
> There's no way to do this in Review Board today. It would involve storing
> credentials in the database for remote access (lots of security issues
> there), or dealing with certificate verification, and it won't always be the
> owner who's triggering access to the repo so they may not be able to provide
> the necessary info.
>
> We generally recommend having some dedicated RB account or read-only access
> to the repo (though generally for very locked down or public servers).
>
> Christian
>
> --
> Christian Hammond - chip...@chipx86.com
> Review Board -http://www.reviewboard.org
> VMware, Inc. -http://www.vmware.com
>
> On Wed, Aug 17, 2011 at 8:33 AM, Justin Georgeson wrote:
>
>
>
>
>
>
>
> > I have a dev team using reviewboard for just their one project, hosted
> > in svn+ssh. We are now looking at deploying it out more globally. I
> > notice in the Admin docs there's a section for configuring an SVN
> > user. I'm wondering if reviewboard can be configured to use the
> > credentials of the user generating the review request rather than
> > using a single account per repo. This would benefit me for auditing
> > purposes to have the username of a particular person in the SVN logs.
>
> > --
> > Want to help the Review Board project? Donate today at
> >http://www.reviewboard.org/donate/
> > Happy user? Let us know athttp://www.reviewboard.org/users/
> > -~--~~~~--~~--~--~---
> > To unsubscribe from this group, send email to
> > reviewboard+unsubscr...@googlegroups.com
> > For more options, visit this group at
> >http://groups.google.com/group/reviewboard?hl=en

-- 
Want to help the Review Board project? Donate today at 
http://www.reviewboard.org/donate/
Happy user? Let us know at http://www.reviewboard.org/users/
-~--~~~~--~~--~--~---
To unsubscribe from this group, send email to 
reviewboard+unsubscr...@googlegroups.com
For more options, visit this group at 
http://groups.google.com/group/reviewboard?hl=en


Problems with inactive users

2011-08-18 Thread rmc
Hi,

I am having two problems when I set a user to be inactive:

1 - Related reviews no longer appear on the list. This is bad, because 
prevents active users from learning about things in older reviews.
2 - Because of 1 above, I usually keep users active. However, their 
mailboxes no longer exist and I get error replies all the time

Is there an elegant solution for this?

This is what I would expect:

* User active/inactive checkbox should only disallow login, posting reviews, 
and comments

or

* A new checkbox to disable email for the user

Thanks.

-- 
Want to help the Review Board project? Donate today at 
http://www.reviewboard.org/donate/
Happy user? Let us know at http://www.reviewboard.org/users/
-~--~~~~--~~--~--~---
To unsubscribe from this group, send email to 
reviewboard+unsubscr...@googlegroups.com
For more options, visit this group at 
http://groups.google.com/group/reviewboard?hl=en

Re: Problems with inactive users

2011-08-18 Thread rmc
Ok, I have found a way to disable email while keeping the user active by 
blanking the email field.

I thought email was a mandatory field, but it isn't. Not perfect, but serves 
my purpose.

Thanks.

-- 
Want to help the Review Board project? Donate today at 
http://www.reviewboard.org/donate/
Happy user? Let us know at http://www.reviewboard.org/users/
-~--~~~~--~~--~--~---
To unsubscribe from this group, send email to 
reviewboard+unsubscr...@googlegroups.com
For more options, visit this group at 
http://groups.google.com/group/reviewboard?hl=en

Re: Bzr support in post-review

2011-08-18 Thread Daniel Benamy
Now that 0.3.3 is out, would this work be merged if it was fixed up?

On Jun 26, 3:37 am, Christian Hammond  wrote:
> Hi Dan,
>
> We need to do a full review and make sure it's suitable. I don't know if the
> original author is still around (there weren't any updates made after the
> current reviews), but we'd need someone familiar with bzr to help finish up
> any changes and test them.
>
> Right now, we're freezing RBTools while some restructuring work finishes up
> in a branch. It's too tedious to keep things in sync right now. Once that
> work is in, we can start accepting patches again.
>
> Christian
>
> --
> Christian Hammond - chip...@chipx86.com
> Review Board -http://www.reviewboard.org
> VMware, Inc. -http://www.vmware.com
>
>
>
>
>
>
>
> On Sat, Jun 25, 2011 at 3:29 PM, Daniel Benamy  wrote:
> > Hi,
>
> > Adding bzr support to post-review seems to be in limbo. There's a
> > patch for it at
> >http://reviews.reviewboard.org/r/1739/
> > and a thread about it at
>
> >http://groups.google.com/group/reviewboard/browse_thread/thread/888a4...
>
> > There's a comment in the review that says that changing the regex to
> > BRANCH_REGEX = r'\w*(branch|branch root): (?P.+)$'
> > makes it work. That regex matches the output of my "bzr info" too and
> > I think I saw one or two comments on other lists of it working for
> > people but it was a while ago.
>
> > Is there anything holding up the merging of that patch + new regex?
>
> > Thanks!
> > Dan
>
> > --
> > Want to help the Review Board project? Donate today at
> >http://www.reviewboard.org/donate/
> > Happy user? Let us know athttp://www.reviewboard.org/users/
> > -~--~~~~--~~--~--~---
> > To unsubscribe from this group, send email to
> > reviewboard+unsubscr...@googlegroups.com
> > For more options, visit this group at
> >http://groups.google.com/group/reviewboard?hl=en

-- 
Want to help the Review Board project? Donate today at 
http://www.reviewboard.org/donate/
Happy user? Let us know at http://www.reviewboard.org/users/
-~--~~~~--~~--~--~---
To unsubscribe from this group, send email to 
reviewboard+unsubscr...@googlegroups.com
For more options, visit this group at 
http://groups.google.com/group/reviewboard?hl=en


Re: Problems with inactive users

2011-08-18 Thread Christian Hammond
This used to be how inactive users worked, but we got a lot of people
complaining that it results in stale reviews that never go away. That's a
real problem at some companies. So that's why we hide reviews.

I think the solution will be new filters on the dashboard to show such
review requests. I think if there's some big change that needed to go in,
it's best if the person leaving, or their manager, makes sure that someone's
aware of the review request so they can get the patch and take over.
Otherwise, you just have a review request there indefinitely.

Christian

-- 
Christian Hammond - chip...@chipx86.com
Review Board - http://www.reviewboard.org
VMware, Inc. - http://www.vmware.com


On Thu, Aug 18, 2011 at 8:17 AM, rmc  wrote:

> Hi,
>
> I am having two problems when I set a user to be inactive:
>
> 1 - Related reviews no longer appear on the list. This is bad, because
> prevents active users from learning about things in older reviews.
> 2 - Because of 1 above, I usually keep users active. However, their
> mailboxes no longer exist and I get error replies all the time
>
> Is there an elegant solution for this?
>
> This is what I would expect:
>
> * User active/inactive checkbox should only disallow login, posting
> reviews, and comments
>
> or
>
> * A new checkbox to disable email for the user
>
> Thanks.
>
> --
> Want to help the Review Board project? Donate today at
> http://www.reviewboard.org/donate/
> Happy user? Let us know at http://www.reviewboard.org/users/
> -~--~~~~--~~--~--~---
> To unsubscribe from this group, send email to
> reviewboard+unsubscr...@googlegroups.com
> For more options, visit this group at
> http://groups.google.com/group/reviewboard?hl=en

-- 
Want to help the Review Board project? Donate today at 
http://www.reviewboard.org/donate/
Happy user? Let us know at http://www.reviewboard.org/users/
-~--~~~~--~~--~--~---
To unsubscribe from this group, send email to 
reviewboard+unsubscr...@googlegroups.com
For more options, visit this group at 
http://groups.google.com/group/reviewboard?hl=en

Re: Bzr support in post-review

2011-08-18 Thread Christian Hammond
Hi Dan,

Sorry, your previous e-mail slipped past me.

You're more than welcome to work on getting that patch in shape. David and I
will likely do the code review on it (please poke us if we don't get to it
right away).

The freeze is over. The new BZR code lives in rbtools/clients/bzr.py. Code
will have to be ported to that location.

All work should be done on the master branch.

Thanks!

Christian

-- 
Christian Hammond - chip...@chipx86.com
Review Board - http://www.reviewboard.org
VMware, Inc. - http://www.vmware.com


On Thu, Aug 18, 2011 at 9:43 AM, Daniel Benamy  wrote:

> Now that 0.3.3 is out, would this work be merged if it was fixed up?
>
> On Jun 26, 3:37 am, Christian Hammond  wrote:
> > Hi Dan,
> >
> > We need to do a full review and make sure it's suitable. I don't know if
> the
> > original author is still around (there weren't any updates made after the
> > current reviews), but we'd need someone familiar with bzr to help finish
> up
> > any changes and test them.
> >
> > Right now, we're freezing RBTools while some restructuring work finishes
> up
> > in a branch. It's too tedious to keep things in sync right now. Once that
> > work is in, we can start accepting patches again.
> >
> > Christian
> >
> > --
> > Christian Hammond - chip...@chipx86.com
> > Review Board -http://www.reviewboard.org
> > VMware, Inc. -http://www.vmware.com
> >
> >
> >
> >
> >
> >
> >
> > On Sat, Jun 25, 2011 at 3:29 PM, Daniel Benamy 
> wrote:
> > > Hi,
> >
> > > Adding bzr support to post-review seems to be in limbo. There's a
> > > patch for it at
> > >http://reviews.reviewboard.org/r/1739/
> > > and a thread about it at
> >
> > >http://groups.google.com/group/reviewboard/browse_thread/thread/888a4.
> ..
> >
> > > There's a comment in the review that says that changing the regex to
> > > BRANCH_REGEX = r'\w*(branch|branch root): (?P.+)$'
> > > makes it work. That regex matches the output of my "bzr info" too and
> > > I think I saw one or two comments on other lists of it working for
> > > people but it was a while ago.
> >
> > > Is there anything holding up the merging of that patch + new regex?
> >
> > > Thanks!
> > > Dan
> >
> > > --
> > > Want to help the Review Board project? Donate today at
> > >http://www.reviewboard.org/donate/
> > > Happy user? Let us know athttp://www.reviewboard.org/users/
> > > -~--~~~~--~~--~--~---
> > > To unsubscribe from this group, send email to
> > > reviewboard+unsubscr...@googlegroups.com
> > > For more options, visit this group at
> > >http://groups.google.com/group/reviewboard?hl=en
>
> --
> Want to help the Review Board project? Donate today at
> http://www.reviewboard.org/donate/
> Happy user? Let us know at http://www.reviewboard.org/users/
> -~--~~~~--~~--~--~---
> To unsubscribe from this group, send email to
> reviewboard+unsubscr...@googlegroups.com
> For more options, visit this group at
> http://groups.google.com/group/reviewboard?hl=en
>

-- 
Want to help the Review Board project? Donate today at 
http://www.reviewboard.org/donate/
Happy user? Let us know at http://www.reviewboard.org/users/
-~--~~~~--~~--~--~---
To unsubscribe from this group, send email to 
reviewboard+unsubscr...@googlegroups.com
For more options, visit this group at 
http://groups.google.com/group/reviewboard?hl=en

Re: SVN user config

2011-08-18 Thread Christian Hammond
Hi Justin,

That's a feature in the upcoming 1.6 release. We're working on docs and some
regressions and will then release it. Hopefully soon.

Christian


On Thursday, August 18, 2011, Justin Georgeson  wrote:
> This may be a silly question but can you restrict which repositories
> each user can work with?
>
> On Aug 17, 1:42 pm, Christian Hammond  wrote:
>> Hi Justin,
>>
>> There's no way to do this in Review Board today. It would involve storing
>> credentials in the database for remote access (lots of security issues
>> there), or dealing with certificate verification, and it won't always be
the
>> owner who's triggering access to the repo so they may not be able to
provide
>> the necessary info.
>>
>> We generally recommend having some dedicated RB account or read-only
access
>> to the repo (though generally for very locked down or public servers).
>>
>> Christian
>>
>> --
>> Christian Hammond - chip...@chipx86.com
>> Review Board -http://www.reviewboard.org
>> VMware, Inc. -http://www.vmware.com
>>
>> On Wed, Aug 17, 2011 at 8:33 AM, Justin Georgeson wrote:
>>
>>
>>
>>
>>
>>
>>
>> > I have a dev team using reviewboard for just their one project, hosted
>> > in svn+ssh. We are now looking at deploying it out more globally. I
>> > notice in the Admin docs there's a section for configuring an SVN
>> > user. I'm wondering if reviewboard can be configured to use the
>> > credentials of the user generating the review request rather than
>> > using a single account per repo. This would benefit me for auditing
>> > purposes to have the username of a particular person in the SVN logs.
>>
>> > --
>> > Want to help the Review Board project? Donate today at
>> >http://www.reviewboard.org/donate/
>> > Happy user? Let us know athttp://www.reviewboard.org/users/
>> > -~--~~~~--~~--~--~---
>> > To unsubscribe from this group, send email to
>> > reviewboard+unsubscr...@googlegroups.com
>> > For more options, visit this group at
>> >http://groups.google.com/group/reviewboard?hl=en
>
> --
> Want to help the Review Board project? Donate today at
http://www.reviewboard.org/donate/
> Happy user? Let us know at http://www.reviewboard.org/users/
> -~--~~~~--~~--~--~---
> To unsubscribe from this group, send email to
reviewboard+unsubscr...@googlegroups.com
> For more options, visit this group at
http://groups.google.com/group/reviewboard?hl=en

-- 
-- 
Christian Hammond - chip...@chipx86.com
Review Board - http://www.reviewboard.org
VMware, Inc. - http://www.vmware.com

-- 
Want to help the Review Board project? Donate today at 
http://www.reviewboard.org/donate/
Happy user? Let us know at http://www.reviewboard.org/users/
-~--~~~~--~~--~--~---
To unsubscribe from this group, send email to 
reviewboard+unsubscr...@googlegroups.com
For more options, visit this group at 
http://groups.google.com/group/reviewboard?hl=en

Re: Slow response after publishing review or comments

2011-08-18 Thread Christian Hammond
We talk directly to the SMTP server (by way of Django). The response time is
taking too long, hence the blocking. One alternative is to fork or create a
thread and do it, but there are some compatibility issues to deal with. We
might have a solution down the road, but for now, it'd be worth figuring out
why the response time is so bad.

Christian


On Thursday, August 18, 2011, rmc  wrote:
> Hello Christian,
> I confirm that email is the culprit.
> The interface should not block because of it, shouldn't this be done in
the background?
> R
>
> --
> Want to help the Review Board project? Donate today at
http://www.reviewboard.org/donate/
> Happy user? Let us know at http://www.reviewboard.org/users/
> -~--~~~~--~~--~--~---
> To unsubscribe from this group, send email to
reviewboard+unsubscr...@googlegroups.com <
reviewboard%2bunsubscr...@googlegroups.com>
> For more options, visit this group at
http://groups.google.com/group/reviewboard?hl=en

-- 
-- 
Christian Hammond - chip...@chipx86.com
Review Board - http://www.reviewboard.org
VMware, Inc. - http://www.vmware.com

-- 
Want to help the Review Board project? Donate today at 
http://www.reviewboard.org/donate/
Happy user? Let us know at http://www.reviewboard.org/users/
-~--~~~~--~~--~--~---
To unsubscribe from this group, send email to 
reviewboard+unsubscr...@googlegroups.com
For more options, visit this group at 
http://groups.google.com/group/reviewboard?hl=en

Re: post-review error

2011-08-18 Thread Christian Hammond
Hi Deva,

Are you behind a proxy?

Christian


On Thursday, August 18, 2011, Deva  wrote:
> Hi,
> Am getting below error while posting the review in windows. It says
> 'Unable to access http://x.x.x.x/api/json/accounts/login/. The host
> path may be invalid
>  closed by the r
> emote host>'
>
> C:\workspace\BDBJE>python C:\rbtools\rbtools\postreview.py --debug
 svn info
 repository info: Path: http://x.x.x.x/wds, Base path: /projects/NavApp/
> devBranches/1.8.X-BDB/BDBJE, Supports changesets: False
 svn diff --diff-cmd=diff
 svn info
src/main/java/com/macys/bdb/cache/da/exception/DataAccessorExceptio
> n.java
 svn info
src/main/java/com/macys/bdb/cache/da/exception/DataAccessorExceptio
> n.java
 svn info
src/main/java/com/macys/bdb/cache/da/exception/DataAccessorExceptio
> n.java
 svn info
src/main/java/com/macys/bdb/cache/da/exception/DataAccessorExceptio
> n.java
 Looking for 'x.x.x.x /' cookie in C:\Documents and Settings\devaraj_m\
> Local Settings\Application Data\.post-review-cookies.txt
 Couldn't load cookie file: [Errno 2] No such file or directory:
'C:\\Documen
> ts and Settings\\devaraj_m\\Local Settings\\Application Data\\.post-
> review-cooki
> es.txt'
> ==> Review Board Login Required
> Enter username and password for Review Board at http://x.x.x.x/
> Username: deva
> Password:
 Logging in with username "deva"
 HTTP POSTing to http://x.x.x.x/api/json/accounts/login/: {'userna
> me': 'deva', 'password': '**'}
> Unable to access http://x.x.x.x/api/json/accounts/login/. The host pa
> th may be invalid
>  closed by the r
> emote host>
>
> Can anyone please help me ?
>
> Regards,
> Deva
>
> --
> Want to help the Review Board project? Donate today at
http://www.reviewboard.org/donate/
> Happy user? Let us know at http://www.reviewboard.org/users/
> -~--~~~~--~~--~--~---
> To unsubscribe from this group, send email to
reviewboard+unsubscr...@googlegroups.com
> For more options, visit this group at
http://groups.google.com/group/reviewboard?hl=en

-- 
-- 
Christian Hammond - chip...@chipx86.com
Review Board - http://www.reviewboard.org
VMware, Inc. - http://www.vmware.com

-- 
Want to help the Review Board project? Donate today at 
http://www.reviewboard.org/donate/
Happy user? Let us know at http://www.reviewboard.org/users/
-~--~~~~--~~--~--~---
To unsubscribe from this group, send email to 
reviewboard+unsubscr...@googlegroups.com
For more options, visit this group at 
http://groups.google.com/group/reviewboard?hl=en