Hi Stephen,
I don't recall exact details, but I've seen the rb-site upgrade command
fail on occasion. I think in my case the sequence was:
* yum update [Included a reviewboard update]
* reboot
* While httpd service was starting rb-site upgrade was invoked.
* rb-site upgrade failed
2.0.8 fixes this for us. Thanks so much!
On Tuesday, September 23, 2014 6:32:55 PM UTC-5, Christian Hammond wrote:
Hi Sara,
David has a repro case for this now, so we're looking into a fix.
There will be a 2.0.8 release tonight.
Christian
--
Christian Hammond -
Hi Chris,
thanks a lot, the --console flag helped me in sorting out my installation
issue. Now, I've got a ReviewBoard site running correclty, I think.
Going a bit further, though, I'm a bit stuck in my workflow with RB and
Mercurial and maybe you could help again.
On the RB website, I've
Paul,
This is not the issue for me. I am demonstrating below that (a) the
'restart httpd' succeeds when the hook is *not* in place, and (b) that even
if mariadb is running, the 'restart httpd' hangs when the hook is in place.
The (0) at the beginning of my prompt is the return code ($?).
By the way, I can do the 'rb-site upgrade --all-sites' as a stand-alone
command, whether or not the hook is in place. So, the problem has
something to do with the rb-site being called by the httpd restart (or
start).
On Wednesday, September 24, 2014 9:59:59 AM UTC-4, David Carson wrote:
In my attempts to get the command used, the developer informed me that they
can see the diffs now.
I'm going to assume I fat-fingered the password to the repo in the setup
and fixed it by reentering it yesterday.
Christian, your help is amazing, and I feel somewhat silly for having
wasted
It's 8:10 pm and I have been fighting with reviewboard installation for
last SIX hours.
I tried with apache2 and I ran into 000-default site problem.. from
https://www.reviewboard.org/docs/manual/dev/admin/installation/creating-sites/#creating-sites:
Some Apache installations (such as the
Hi everyone,
Early this morning, we put out a release of Review Board 2.0.8. This fixes some
of the major issues recently brought up on this list, including several issues
in the New Review Request page, installation problems due to a new release of
Python-Markdown, and more.
Announcement:
Hi,
I want to ask help with an issue with rbt patch + perforce.
When I try to apply a patch from review board by using rbt patch, the tool
fails with a can't find the file to patch error.
$ rbt patch --server https://reviewboard.eng.mydomain.com/ --debug
--username hgontijo --repository
Hi Neel,
lighttpd is not well-supported, which is why it's considered an advanced
option. Very few people seem to use it.
I'd *strongly* suggest using Apache + mod_wsgi (daemon mode works well these
days), if at all possible. You'll have the best support that way.
FastCGI is a bad option.
Hi Pierre,
Can you try using RBTools to post a change?
Christian
--
Christian Hammond - christ...@beanbaginc.com
Review Board - https://www.reviewboard.org
Beanbag, Inc. - https://www.beanbaginc.com
On September 24, 2014 at 6:29:41 AM, Pierre Leroy (pierre.lero...@gmail.com)
wrote:
Hi
Glad it works, Sara! Thanks for your patience and feedback on this!
Christian
--
Christian Hammond - christ...@beanbaginc.com
Review Board - https://www.reviewboard.org
Beanbag, Inc. - https://www.beanbaginc.com
On September 24, 2014 at 6:10:36 AM, Sara Rayburn (sararayb...@gmail.com) wrote:
Hi Neel,
You definitely want to remove the 000-default file.
If the site isn't showing, that's often an indicator that Apache either isn't
seeing the Review Board config file, or the VirtualHost setup is wrong.
Can you provide the reviewboard.conf file along with Apache's httpd.conf?
Can you
Okay, I fixed the problem. here are the couple of changes I did:
sudo apt-get install apache2
sudo apt-get install libapache2-mod-wsgi
sudo a2enmod *wsgi*
sudo apt-get install mysql-server
sudo apt-get install python-mysqldb
sudo apt-get install memcached
sudo apt-get install memcached-dev
sudo
Hi Christian,
Just upgraded to 2.0.8:
[[[
bash-4.1# rb-site manage /var/www.reviewboard reset-issue-counts
--recalculate --verbosity=2 2050
Usage: rb-site command [options] path
rb-site: error: no such option: --recalculate
bash-4.1# rb-site --version
rb-site 2.0.8
]]]
BTW, release notes for
Thanks, I'll fix the release notes.
Make sure to put a -- before any --arguments, like:
rb-site manage /var/www/reviewboard reset-issue-counts -- --verbosity=2
--recalculate
Not sure what you mean by the help output being wrong though?
Christian
--
Christian Hammond -
Figured it out, needs -- to have --recalculate argument applied to
'reset-issue-counts' rather than rb-site itself.
Here is the output:
DEBUG:root:Calculated issue counts for review request ID 2050 across 5
review(s): Resulting counts = {u'R': 0, u'D': 0, u'O': 1}; DB values =
Re help: the order of the 'manage' and path arguments is swapped.
Correct order: rb-site manage path
Help message: rb-site path manage
Regards,
Alexey.
On Wed, Sep 24, 2014 at 12:41 PM, Christian Hammond
christ...@beanbaginc.com wrote:
Thanks, I'll fix the release notes.
Make sure to put a
Thanks for the debug info!
Yeah, as I mentioned in an earlier reply, a reset command will *not* fix the
problem, because the calculation is broken. Best I could do is to get this
command in to reset and provide debug information, so that I can hopefully
figure out the cause. I'll look into
Re release notes: you've fixed only one instance of the two where 'manage'
was missing:
A new rb-site path reset-issue-counts management command has been
added...
On Wed, Sep 24, 2014 at 12:41 PM, Christian Hammond
christ...@beanbaginc.com wrote:
Thanks, I'll fix the release notes.
Make
Can you tell it's been a very long few days?
Fixed. New docs are building now.
Christian
--
Christian Hammond - christ...@beanbaginc.com
Review Board - https://www.reviewboard.org
Beanbag, Inc. - https://www.beanbaginc.com
On September 24, 2014 at 12:47:39 PM, Alexey Neyman
So according to that debug output, there is one comment that has an issue open.
I want to do some probing into this.
Can you run this and provide the full output, commands included?
$ rb-site manage /path/to/site shell
from reviewboard.reviews.models import Comment, Review
comment
On 09/24/2014 01:05 PM, Christian Hammond wrote:
Hi everyone,
Early this morning, we put out a release of Review Board 2.0.8. This
fixes some of the major issues recently brought up on this list,
including several issues in the New Review Request page, installation
problems due to a new
Here it is:
from reviewboard.reviews.models import Comment, Review
comment = Comment.objects.get(pk=8199)
print comment.reply_to_id
None
print comment.issue_opened
False
print comment.issue_status
O
review = comment.review.get()
print review.pk
6722
print review.public
True
print
Hi Alexey,
Thanks, that helps a lot. I think I know what's going on then. I'll send a
patch your way for testing once I have a fix.
Christian
--
Christian Hammond - christ...@beanbaginc.com
Review Board - https://www.reviewboard.org
Beanbag, Inc. - https://www.beanbaginc.com
On September
Upgrading Pygments to 1.6 fixed the problem.
--Steve
On Thursday, August 28, 2014 2:25:51 PM UTC-7, Steve wrote:
Knowing it should work helps. The .pl files I'm looking at are smaller
than the .pm files that work, so I'm going to rule out file size as the
cause. I'll try to isolate the
Stephen,
Yep, removing the ...oneshot line fixed it for me. Thanks a lot.
~David
On Wednesday, September 24, 2014 3:39:34 PM UTC-4, Stephen Gallagher wrote:
On 09/24/2014 10:06 AM, David Carson wrote:
By the way, I can do the 'rb-site upgrade --all-sites' as a stand-alone
command,
Status: New
Owner:
Labels: Type-Defect Priority-Medium
New issue 3581 by whyisyou...@gmail.com: If a user exists in a local site
they will not get emails from the normal site if they are part of a reveiw
group in the normal site.
Comment #15 on issue 421 by da.s...@gmail.com: Enhancement request: nag
owners about stale review requests
https://code.google.com/p/reviewboard/issues/detail?id=421
This is tangential/related, I'd like to be able to abstain or remove myself
from a review request. I would do this regularly
Comment #4 on issue 3484 by john.sirois: `rbt patch -c` fails for patches
to public github repos with an RBCommons team when the patch if from a
non-team member
https://code.google.com/p/reviewboard/issues/detail?id=3484
Afaict there has been a regression.
I see currently for
Status: New
Owner:
Labels: Type-Defect Priority-Medium
New issue 3582 by john.sirois: `rbt patch -c` fails for patches to public
github repos with an RBCommons team when the patch if from a non-team member
https://code.google.com/p/reviewboard/issues/detail?id=3582
What version are you
Comment #5 on issue 3484 by john.sirois: `rbt patch -c` fails for patches
to public github repos with an RBCommons team when the patch if from a
non-team member
https://code.google.com/p/reviewboard/issues/detail?id=3484
Filed a new bug with the regression detail in case that works better
Updates:
Status: Fixed
Comment #4 on issue 3452 by trowb...@gmail.com: RR updated altough no
changes made / warn when no changes are made
https://code.google.com/p/reviewboard/issues/detail?id=3452
Fixed in release-2.0.x (f29c57f). Thanks!
--
You received this message because this
Status: New
Owner:
Labels: Type-Defect Priority-Medium
New issue 3583 by jared.gr...@gmail.com: RB2: No way to post patch with
parent-diff in initial web pages
https://code.google.com/p/reviewboard/issues/detail?id=3583
What version are you running?
ReviewBoard 2.0.6
What's the URL of
Comment #1 on issue 3583 by jared.gr...@gmail.com: RB2: No way to post
patch with parent-diff in initial web pages
https://code.google.com/p/reviewboard/issues/detail?id=3583
Even having a post no patch option, where I could then use the UI in the
review editor would be nice.
--
You
Updates:
Status: Fixed
Comment #2 on issue 3583 by trowb...@gmail.com: RB2: No way to post patch
with parent-diff in initial web pages
https://code.google.com/p/reviewboard/issues/detail?id=3583
This is already fixed in git, and will ship with the 2.1 release.
rbt is fine with ports
36 matches
Mail list logo