post-review error
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
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
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
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
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
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
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
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
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
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
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
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