Re: Review Board 3.0.14 is out

[Daniel]

Stephen, thank you for helping me to a better understanding.

Since yesterday I read up about EPEL usage with RHEL and now i think i have 
a better grasp of this. 

If i got it right, to do a "yum install reviewboard" with more up to date 
RB-versions it would be feasible to wait a bit until i can get my hands on 
an RHEL8 machine, get the EPEL configured (epel-release-latest-8 
<https://dl.fedoraproject.org/pub/epel/epel-release-latest-8.noarch.rpm>), 
regularly check for good news concerning RB via EPEL and upon arrival be 
happily installing via yum (not that i am not happy already, having 
realized once more what an amount of time you pour into this!)

So thank you once more and now i am back in lurkmode :)

Am Dienstag, 20. August 2019 17:03:42 UTC+2 schrieb Stephen Gallagher:
>
> On Tue, Aug 20, 2019 at 11:00 AM Daniel  > wrote: 
> > 
> > Hi Christian and Stephen, 
> > 
> > I just scanned the forums for possible updates concerning the 
> version-compatilities/ version-availabilites and found this post from 
> March. 
> > 
> > Some time ago when i was even more unexperienced in the ways of how the 
> system needs to be to be able to run reviewboard i asked about EPELs and 
> RHELs for revieboard 3.x (
> https://groups.google.com/d/msg/reviewboard/mSQPKhkKmMw/zFkNNf3gBgAJ) 
> > 
> > Since that time we are using RB in the Version 2.5.17 (the Version 
> available via yum like Paul mentioned). 
> > 
> > If i understand it correctly, than an EPEL can be configured for the 
> RHEL7 and afterwards i can use yum to install something out of that EPEL7 
> repo (e.g. reviewboard) 
> > 
> > Stephen was hoping that RB4 might be able to be delivered that way and 
> Christian was reducing that hope as there are still other hindering factors 
> so that the earliest maybe possible version of reviewboard available via an 
> EPEL again might be RB5. But this still is just a possibility and not 
> certain. 
> > 
> > We are working with RHEL7 Machines and i personally do not have much 
> legroom to customize that install ... so i was curious what your last 
> messages in this thread might mean for me and my usecase. 
> > 
> > I would be very happy if you would maybe find the time to describe the 
> way (if there is any) in that i might be able to install a "more modern 
> than 2.5.17"-version of RB into a RHEL7 system without manually installing 
> too many dependencies. 
> > 
>
> Short version: I probably can't provide RB 3.0+ on EPEL 7 any time 
> soon. However, I do expect to be able to deliver 3.0 and later on EPEL 
> 8 within the next six months. We have support for 
> Modularity/AppStreams in the works there and I will be able to package 
> the newer versions of Review Board for RHEL/CentOS 8.1+ 
>

-- 
Supercharge your Review Board with Power Pack: 
https://www.reviewboard.org/powerpack/
Want us to host Review Board for you? Check out RBCommons: 
https://rbcommons.com/
Happy user? Let us know! https://www.reviewboard.org/users/
--- 
You received this message because you are subscribed to the Google Groups 
"Review Board Community" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard+unsubscr...@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/reviewboard/17f1309a-8576-4bbd-b3fe-5d081e392771%40googlegroups.com.

Re: Review Board 3.0.14 is out

[Daniel]

Hi Christian and Stephen,

I just scanned the forums for possible updates concerning the 
version-compatilities/ version-availabilites and found this post from March.

Some time ago when i was even more unexperienced in the ways of how the 
system needs to be to be able to run reviewboard i asked about EPELs and 
RHELs for revieboard 3.x 
(https://groups.google.com/d/msg/reviewboard/mSQPKhkKmMw/zFkNNf3gBgAJ)

Since that time we are using RB in the Version 2.5.17 (the Version 
available via yum like Paul mentioned).

If i understand it correctly, than an EPEL can be configured for the RHEL7 
and afterwards i can use yum to install something out of that EPEL7 repo 
(e.g. reviewboard)

Stephen was hoping that RB4 might be able to be delivered that way and 
Christian was reducing that hope as there are still other hindering factors 
so that the earliest maybe possible version of reviewboard available via an 
EPEL again might be RB5. But this still is just a possibility and not 
certain.

We are working with RHEL7 Machines and i personally do not have much 
legroom to customize that install ... so i was curious what your last 
messages in this thread might mean for me and my usecase. 

I would be very happy if you would maybe find the time to describe the way 
(if there is any) in that i might be able to install a "more modern than 
2.5.17"-version of RB into a RHEL7 system without manually installing too 
many dependencies.

TIA and cheers
Daniel

>

-- 
Supercharge your Review Board with Power Pack: 
https://www.reviewboard.org/powerpack/
Want us to host Review Board for you? Check out RBCommons: 
https://rbcommons.com/
Happy user? Let us know! https://www.reviewboard.org/users/
--- 
You received this message because you are subscribed to the Google Groups 
"Review Board Community" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard+unsubscr...@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/reviewboard/51e99052-d01c-4658-ae84-a1431e193b9d%40googlegroups.com.

Re: Reviewboard 3.0 RPM availability

[daniel . forumsmail]

Thank you for all your efforts and also that you try to keep on! You have 
my sympathy and understanding for not mentioning anything about something 
concerning the concept of a timeframe!


Am Mittwoch, 18. Juli 2018 19:23:09 UTC+2 schrieb Stephen Gallagher:
>
>
>
> On Wed, Jul 18, 2018 at 12:18 PM > 
> wrote:
>
>> Hi Stephen,
>>
>> would you maybe have had the possibility to look into a rpm for EPEL7? 
>> (not sure about either grammar or EPEL naming ... i think i am asking for 
>> something i can install via yum :)[...]
>>
> Getting this working on RHEL/CentOS 7 is something I very much want to do, 
> but please understand that I do this in my spare time. I'm not paid to work 
> on ReviewBoard at all. I have ReviewBoard 3.0 now available in Fedora 28 
> and I am looking into getting it working on EPEL 7, but I won't commit to 
> any particular timeframe, sorry.
>

-- 
Supercharge your Review Board with Power Pack: 
https://www.reviewboard.org/powerpack/
Want us to host Review Board for you? Check out RBCommons: 
https://rbcommons.com/
Happy user? Let us know! https://www.reviewboard.org/users/
--- 
You received this message because you are subscribed to the Google Groups 
"Review Board Community" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

Re: Reviewboard 3.0 RPM availability

[daniel . forumsmail]

Hi Stephen,

would you maybe have had the possibility to look into a rpm for EPEL7? (not 
sure about either grammar or EPEL naming ... i think i am asking for 
something i can install via yum :)

I was just asking something in a different thread which got a nice and 
comprehensive reply by Christian Hammond ... also concerning a RB 3.x for 
yum ... quoting from 
(https://groups.google.com/d/msg/reviewboard-dev/2MuigbF_-X8/wU-aPDPKBgAJ)
 

> For c), I'm not sure of the current status there. This is not a Review 
> Board issue exactly, but rather a packaging issue, which I think is being 
> solved now? Basically, we need some older versions of some modules for very 
> good reasons I don't want to go into at the moment (various 
> compatibility... things...) and this makes packaging a bit harder. I think 
> that's being addressed through a new modular packaging system in Fedora. 
> Stephen can go into details more.
>

This quote is about helping Reviewboard 3.x being made available for Redhat 
rpm yum stuff ... as he mentioned that you might comment on that, i'd like 
to take the opportunity to ask you about it :)

Sadly, i am just a consumer and probably not versed enough to be helpful 
concerning your plea for help in supporting this kind of backporting stuff 
that is needed to make RB 3.x available via EPEL.

I tried to install RB in a newer Version  by following the instructions 
using pip but that lead to just a bunch of errors that i could not 
comprehend ... after that unsuccessful attempt i had the machine get 
reverted to previous "untouched" state and afterwards installed the rpm for 
2.5.17 ... and now, like stated, i am thumbing my nose at all those nice 
stuff that is coming in with the 3.x releases :o)

cheers 
Daniel


Am Freitag, 20. April 2018 18:57:52 UTC+2 schrieb Stephen Gallagher:
>
>
>
> On Fri, Apr 20, 2018 at 5:10 AM > wrote:
>
>> For Reviewboard 2.x there's EPEL RPM packages available, I'm just 
>> wondering if Reviewboard is planning to make RPM packages available for 
>> version 3.0 as well.
>>
>>
> I've been working on it, but it's difficult because EPEL 7 has a lot of 
> packages that are very old and hard to update. (Also, EPEL has a policy 
> against replacing any package shipped by RHEL/CentOS).
>
> However, I *do* now have packages working against Fedora 28:  
> https://bodhi.fedoraproject.org/updates/FEDORA-MODULAR-2018-d31893afa3
>
> It can be installed on F28 Beta by running the command:
> `dnf install @reviewboard:3.0`
>
> I'm hoping to get an update out to EPEL 7 at some point, but it's slow 
> going and my dayjob is coming down hard on me at the moment. For the time 
> being, I'm only committing to maintaining updates on the 2.5.x line.
>
> As always, anyone who would like to help me maintain these in Fedora/EPEL 
> should speak up.
>

-- 
Supercharge your Review Board with Power Pack: 
https://www.reviewboard.org/powerpack/
Want us to host Review Board for you? Check out RBCommons: 
https://rbcommons.com/
Happy user? Let us know! https://www.reviewboard.org/users/
--- 
You received this message because you are subscribed to the Google Groups 
"Review Board Community" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

jQuery 1.8 vulnerability

[Daniel]

Hello,

My corp security department prevents me using the ReviewBoard because there 
are publicly known vulnerabilities in the one of RB's components 
(particularly jQuery 1.8).

Would it possible for the community to mitigate those issues ?


CVE-2015-9251 

CVE-2012-6708 

 

https://www.cvedetails.com/vulnerability-list/vendor_id-6538/product_id-11031/version_id-235567/opxss-1/Jquery-Jquery-1.8.1.html
 


I saw that master branch in GitHub already has jQuery 3.3, hopefully RB 4.0 
will met security requirements. Would it possible to backport jQuery3.3 to 
2.0 and/or 3.0 ?


Thanks in advance.

-- 
Supercharge your Review Board with Power Pack: 
https://www.reviewboard.org/powerpack/
Want us to host Review Board for you? Check out RBCommons: 
https://rbcommons.com/
Happy user? Let us know! https://www.reviewboard.org/users/
--- 
You received this message because you are subscribed to the Google Groups 
"Review Board Community" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

Re: Issue installing reviewbot-worker on latest RB

[Daniel CABRERA]

I have done all that already, with same results:-(

On Fri, 16 Mar. 2018, 11:37 am David Trowbridge,  wrote:

> As I suggested, disable and enable the extension while keeping an eye on
> the log files. And double check the permissions on the directory to verify
> that the web server can write to it.
> On Thu, Mar 15, 2018 at 5:14 PM MoonWalker 
> wrote:
>
>> David,
>>
>> Not sure what else to try 😞
>>
>> --
>> Supercharge your Review Board with Power Pack:
>> https://www.reviewboard.org/powerpack/
>> Want us to host Review Board for you? Check out RBCommons:
>> https://rbcommons.com/
>> Happy user? Let us know! https://www.reviewboard.org/users/
>> ---
>> You received this message because you are subscribed to the Google Groups
>> "Review Board Community" group.
>> To unsubscribe from this group and stop receiving emails from it, send an
>> email to reviewboard+unsubscr...@googlegroups.com.
>> For more options, visit https://groups.google.com/d/optout.
>>
> --
> Supercharge your Review Board with Power Pack:
> https://www.reviewboard.org/powerpack/
> Want us to host Review Board for you? Check out RBCommons:
> https://rbcommons.com/
> Happy user? Let us know! https://www.reviewboard.org/users/
> ---
> You received this message because you are subscribed to a topic in the
> Google Groups "Review Board Community" group.
> To unsubscribe from this topic, visit
> https://groups.google.com/d/topic/reviewboard/VIwOcAmKC-k/unsubscribe.
> To unsubscribe from this group and all its topics, send an email to
> reviewboard+unsubscr...@googlegroups.com.
> For more options, visit https://groups.google.com/d/optout.
>

-- 
Supercharge your Review Board with Power Pack: 
https://www.reviewboard.org/powerpack/
Want us to host Review Board for you? Check out RBCommons: 
https://rbcommons.com/
Happy user? Let us know! https://www.reviewboard.org/users/
--- 
You received this message because you are subscribed to the Google Groups 
"Review Board Community" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

Re: rb-extension-pack Compability with Review Board 3.0

[Daniel Morgan]

Christian,

Thank you for the reply.
I will wait for the patch to be applied and check again.

-- 
Supercharge your Review Board with Power Pack: 
https://www.reviewboard.org/powerpack/
Want us to host Review Board for you? Check out RBCommons: 
https://rbcommons.com/
Happy user? Let us know! https://www.reviewboard.org/users/
--- 
You received this message because you are subscribed to the Google Groups 
"reviewboard" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

Re: rb-extension-pack Compability with Review Board 3.0

[Daniel Morgan]

David,

Thank you for the reply.
I will check again once the Extensions get updated.

-- 
Supercharge your Review Board with Power Pack: 
https://www.reviewboard.org/powerpack/
Want us to host Review Board for you? Check out RBCommons: 
https://rbcommons.com/
Happy user? Let us know! https://www.reviewboard.org/users/
--- 
You received this message because you are subscribed to the Google Groups 
"reviewboard" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

rb-extension-pack Compability with Review Board 3.0

[Daniel Morgan]

Are there any plans to check the compatibility for the Extensions contained 
in rb-extension-pack with Review Board 3.0?
I am using a few of the Extensions there and have encountered errors using 
Review Board 3.0 RC1.

For example, when configuring the message for the "Message of the Day" 
Extension, I get the following error.
The log output is as follows (sensitive details edited):

None - User1 - /admin/extensions/rbmotd.extension.MotdExtension/config/ - 
djblets.log.middleware - Exception thrown for user User1 at 
http://x.x.x.x/admin/extensions/rbmotd.extension.MotdExtension/config/

'MotdSettingsForm' object has no attribute 'siteconfig'
Traceback (most recent call last):
  File 
"/usr/local/lib/python2.7/dist-packages/django/core/handlers/base.py", line 
112, in get_response
response = wrapped_callback(request, *callback_args, **callback_kwargs)
  File "/usr/local/lib/python2.7/dist-packages/django/utils/decorators.py", 
line 99, in _wrapped_view
response = view_func(request, *args, **kwargs)
  File 
"/usr/local/lib/python2.7/dist-packages/reviewboard/extensions/views.py", 
line 15, in configure_extension
template_name)
  File "/usr/local/lib/python2.7/dist-packages/django/utils/decorators.py", 
line 99, in _wrapped_view
response = view_func(request, *args, **kwargs)
  File 
"/usr/local/lib/python2.7/dist-packages/django/contrib/admin/views/decorators.py",
 
line 17, in _checklogin
return view_func(request, *args, **kwargs)
  File 
"/usr/local/lib/python2.7/dist-packages/djblets/extensions/views.py", line 
64, in configure_extension
form.save()
  File 
"/usr/local/lib/python2.7/dist-packages/rbmotd-1.0-py2.7.egg/rbmotd/forms.py", 
line 22, in save
self.siteconfig.set(
AttributeError: 'MotdSettingsForm' object has no attribute 'siteconfig'

-- 
Supercharge your Review Board with Power Pack: 
https://www.reviewboard.org/powerpack/
Want us to host Review Board for you? Check out RBCommons: 
https://rbcommons.com/
Happy user? Let us know! https://www.reviewboard.org/users/
--- 
You received this message because you are subscribed to the Google Groups 
"reviewboard" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

Re: Error opening existing review after upgrading to Review Board 3.0 RC1

[Daniel Morgan]

Thank you for the prompt reply.

Yes, you were spot on.
I didn't copy the SECRET_KEY from the production server over to the test 
server.
Once I did, the errors stopped occurring.

-- 
Supercharge your Review Board with Power Pack: 
https://www.reviewboard.org/powerpack/
Want us to host Review Board for you? Check out RBCommons: 
https://rbcommons.com/
Happy user? Let us know! https://www.reviewboard.org/users/
--- 
You received this message because you are subscribed to the Google Groups 
"reviewboard" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

Error opening existing review after upgrading to Review Board 3.0 RC1

[Daniel Morgan]

I just completed upgrading Review Board 2.5.16 to 3.0 RC1 on a test server.
Logged in without any issues and tried opening an existing review which 
resulted in a "Something broke! (Error 500)".

The log output is as follows (sensitive details edited):

None - user1 - /r/605/ - djblets.log.middleware - Exception thrown for user 
user1 at http://x.x.x.x/r/605/

'utf8' codec can't decode byte 0x87 in position 1: invalid start byte
Traceback (most recent call last):
  File 
"/usr/local/lib/python2.7/dist-packages/django/core/handlers/base.py", line 
112, in get_response
response = wrapped_callback(request, *callback_args, **callback_kwargs)
  File 
"/usr/local/lib/python2.7/dist-packages/django/views/generic/base.py", line 
69, in view
return self.dispatch(request, *args, **kwargs)
  File 
"/usr/local/lib/python2.7/dist-packages/djblets/views/generic/base.py", 
line 46, in dispatch
request, *args, **kwargs)
  File "/usr/local/lib/python2.7/dist-packages/django/utils/decorators.py", 
line 29, in _wrapper
return bound_func(*args, **kwargs)
  File 
"/usr/local/lib/python2.7/dist-packages/reviewboard/accounts/decorators.py", 
line 23, in _check
return login_required(view_func)(*args, **kwargs)
  File 
"/usr/local/lib/python2.7/dist-packages/django/contrib/auth/decorators.py", 
line 22, in _wrapped_view
return view_func(request, *args, **kwargs)
  File "/usr/local/lib/python2.7/dist-packages/django/utils/decorators.py", 
line 25, in bound_func
return func(self, *args2, **kwargs2)
  File 
"/usr/local/lib/python2.7/dist-packages/reviewboard/accounts/mixins.py", 
line 38, in dispatch
*args, **kwargs)
  File "/usr/local/lib/python2.7/dist-packages/django/utils/decorators.py", 
line 29, in _wrapper
return bound_func(*args, **kwargs)
  File 
"/usr/local/lib/python2.7/dist-packages/reviewboard/site/decorators.py", 
line 36, in _check
return view_func(request, local_site=local_site, *args, **kwargs)
  File "/usr/local/lib/python2.7/dist-packages/django/utils/decorators.py", 
line 25, in bound_func
return func(self, *args2, **kwargs2)
  File "/usr/local/lib/python2.7/dist-packages/reviewboard/site/mixins.py", 
line 53, in dispatch
request, *args, **kwargs)
  File 
"/usr/local/lib/python2.7/dist-packages/djblets/views/generic/base.py", 
line 91, in dispatch
super(PrePostDispatchViewMixin, self).dispatch(*args, **kwargs)
  File 
"/usr/local/lib/python2.7/dist-packages/djblets/views/generic/etag.py", 
line 50, in dispatch
**kwargs)
  File 
"/usr/local/lib/python2.7/dist-packages/django/views/generic/base.py", line 
87, in dispatch
return handler(request, *args, **kwargs)
  File 
"/usr/local/lib/python2.7/dist-packages/django/views/generic/base.py", line 
154, in get
context = self.get_context_data(**kwargs)
  File 
"/usr/local/lib/python2.7/dist-packages/reviewboard/reviews/views.py", line 
663, in get_context_data
context.update(make_review_request_context(request, review_request))
  File 
"/usr/local/lib/python2.7/dist-packages/reviewboard/reviews/context.py", 
line 105, in make_review_request_context
upload_diff_form = UploadDiffForm(review_request, request=request)
  File 
"/usr/local/lib/python2.7/dist-packages/reviewboard/reviews/forms.py", line 
129, in __init__
data, *args, **kwargs)
  File 
"/usr/local/lib/python2.7/dist-packages/reviewboard/diffviewer/forms.py", 
line 41, in __init__
if self.repository.get_scmtool().diffs_use_absolute_paths:
  File 
"/usr/local/lib/python2.7/dist-packages/reviewboard/scmtools/models.py", 
line 244, in get_scmtool
return self.scmtool_class(self)
  File 
"/usr/local/lib/python2.7/dist-packages/reviewboard/scmtools/svn/__init__.py", 
line 69, in __init__
credentials = repository.get_credentials()
  File 
"/usr/local/lib/python2.7/dist-packages/reviewboard/scmtools/models.py", 
line 326, in get_credentials
password = self.password
  File 
"/usr/local/lib/python2.7/dist-packages/reviewboard/scmtools/models.py", 
line 220, in _get_password
password = decrypt_password(password).decode('utf-8')
  File "/usr/lib/python2.7/encodings/utf_8.py", line 16, in decode
return codecs.utf_8_decode(input, errors, True)
UnicodeDecodeError: 'utf8' codec can't decode byte 0x87 in position 1: 
invalid start byte


The same review can be opened without any errors using Review Board 2.5.16 
so I am assuming there is some kind of Unicode Decoding issue with 3.0 RC1.
I'm not sure if this information will help, but all existing reviews result 
in the same error when opened.
All reviews contain a mixture of English and Japanese characters.

-- 
Supercharge your Review Board with Power Pack: 
https://www.reviewboard.org/powerpack/
Want us to host Review Board for you? Check out RBCommons: 
https://rbcommons.com/
Happy user? Let us know! https://www.reviewboard.org/users/
--- 
You received this message because you are subscribed to the Google Groups 
"reviewboard" group.
To unsubscribe from this group and stop recei

Re: cipher AES in CFB8 mode is not supported by this backend

[Daniel Kuecker]

Fixed it ... cryptography==2.0 needed to be upgraded to cryptography==2.0.3.

WHEW that was stressful!

On Monday, August 28, 2017 at 12:55:39 PM UTC-5, Daniel Kuecker wrote:
>
> I'm having an issue updating the user and passowrd for one of our 
> repositories via the admin page.  When I enter the info and hit save, I get 
> the following error:
>
> 2017-08-28 17:49:57,696 - ERROR - None - 888 - 
> /admin/db/scmtools/repository/1/ - djblets.log.middleware - Exception 
> thrown for user 88 at http:///admin/db/scmtools/repository/1/
>
> cipher AES in CFB8 mode is not supported by this backend.
> Traceback (most recent call last):
>   File "/usr/lib/python2.7/site-packages/django/core/handlers/base.py", 
> line 112, in get_response
> response = wrapped_callback(request, *callback_args, **callback_kwargs)
>   File "/usr/lib/python2.7/site-packages/django/contrib/admin/options.py", 
> line 465, in wrapper
> return self.admin_site.admin_view(view)(*args, **kwargs)
>   File "/usr/lib/python2.7/site-packages/django/utils/decorators.py", line 
> 99, in _wrapped_view
> response = view_func(request, *args, **kwargs)
>   File 
> "/usr/lib/python2.7/site-packages/django/views/decorators/cache.py", line 
> 52, in _wrapped_view_func
> response = view_func(request, *args, **kwargs)
>   File "/usr/lib/python2.7/site-packages/django/contrib/admin/sites.py", 
> line 198, in inner
> return view(request, *args, **kwargs)
>   File "/usr/lib/python2.7/site-packages/django/utils/decorators.py", line 
> 29, in _wrapper
> return bound_func(*args, **kwargs)
>   File "/usr/lib/python2.7/site-packages/django/utils/decorators.py", line 
> 99, in _wrapped_view
> response = view_func(request, *args, **kwargs)
>   File "/usr/lib/python2.7/site-packages/django/utils/decorators.py", line 
> 25, in bound_func
> return func(self, *args2, **kwargs2)
>   File "/usr/lib/python2.7/site-packages/django/db/transaction.py", line 
> 371, in inner
> return func(*args, **kwargs)
>   File "/usr/lib/python2.7/site-packages/django/contrib/admin/options.py", 
> line 1246, in change_view
> new_object = self.save_form(request, form, change=True)
>   File "/usr/lib/python2.7/site-packages/django/contrib/admin/options.py", 
> line 887, in save_form
> return form.save(commit=False)
>   File "/usr/lib/python2.7/site-packages/reviewboard/scmtools/forms.py", 
> line 1181, in save
> repository.password = self.cleaned_data['password'] or ''
>   File "/usr/lib/python2.7/site-packages/reviewboard/scmtools/models.py", 
> line 192, in _set_password
> encrypt_password(value.encode('utf-8')))
>   File 
> "/usr/lib/python2.7/site-packages/reviewboard/scmtools/crypto_utils.py", 
> line 150, in encrypt_password
> return base64.b64encode(aes_encrypt(password, key=key))
>   File 
> "/usr/lib/python2.7/site-packages/reviewboard/scmtools/crypto_utils.py", 
> line 88, in aes_encrypt
> encryptor = cipher.encryptor()
>   File 
> "/usr/lib64/python2.7/site-packages/cryptography/hazmat/primitives/ciphers/base.py",
>  
> line 104, in encryptor
> self.algorithm, self.mode
>   File 
> "/usr/lib64/python2.7/site-packages/cryptography/hazmat/backends/multibackend.py",
>  
> line 53, in create_symmetric_encryption_ctx
> _Reasons.UNSUPPORTED_CIPHER
> UnsupportedAlgorithm: cipher AES in CFB8 mode is not supported by this 
> backend.
>
>
> I'm running on RHEL 7.
>

-- 
Supercharge your Review Board with Power Pack: 
https://www.reviewboard.org/powerpack/
Want us to host Review Board for you? Check out RBCommons: 
https://rbcommons.com/
Happy user? Let us know! https://www.reviewboard.org/users/
--- 
You received this message because you are subscribed to the Google Groups 
"reviewboard" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

Re: cipher AES in CFB8 mode is not supported by this backend

[Daniel Kuecker]

My Pip freeze contents:

appdirs==1.4.2
asn1crypto==0.22.0
backports.ssl-match-hostname==3.4.0.2
bcrypt==3.1.3
cffi==1.10.0
chardet==2.2.1
configobj==4.7.2
configshell-fb==1.1.18
cryptography==2.0
decorator==3.4.0
Django==1.6.11.6
django-evolution==0.7.7
django-haystack==2.4.1
django-multiselectfield==0.1.7
django-pipeline==1.3.27
Djblets==0.9.8
dnspython==1.15.0
docutils==0.13.1
ecdsa==0.13
enum34==1.1.6
ethtool==0.8
feedparser==5.2.1
flup==1.0.2
futures==3.1.1
idna==2.5
iniparse==0.4
iotop==0.6
ipaddress==1.0.18
jabber.py==0.3.post1
javapackages==1.0.0
jsonpointer==1.9
kerberos==1.1
kitchen==1.1.1
kmod==0.1
langtable==0.0.31
lxml==3.2.1
M2Crypto==0.21.1
Markdown==2.4.1
mechanize==0.2.5
mercurial==2.6.2
mimeparse==0.1.3
MySQL-python==1.2.3
netaddr==0.7.5
nose==1.3.0
olefile==0.44
p4python==2017.1.1526044
packaging==16.8
paramiko==2.2.1
pciutils==1.7.3
pcp==1.0
perf==0.1
Pillow==4.2.1
pillowfight==0.3
ply==3.4
psutil==2.2.1
psycopg2==2.5.1
publicsuffix==1.1.0
pyasn1==0.2.3
pycparser==2.18
pycrypto==2.6.1
pycurl==7.19.0
Pygments==2.2.0
pygobject==3.14.0
pygpgme==0.3
pyliblzma==0.5.3
PyNaCl==1.1.2
pyOpenSSL==0.13.1
pyparsing==2.1.10
python-augeas==0.5.0
python-dateutil==2.6.1
python-dmidecode==3.10.13
python-ldap==2.4.15
python-memcached==1.58
python-mimeparse==1.6.0
python-nss==0.16.0
python-ntlm==1.1.0
python-yubico==1.2.3
pytz==2017.2
pyudev==0.15
pyusb==1.0.0b1
pyxattr==0.5.1
qrcode==5.0.1
rbmotd==1.0
rbnotefield==1.0.1
rbseverity==0.5.1
rbstopwatch==1.0.1
recaptcha-client==1.0.6
ReviewBoard==2.5.13.1
ReviewBoardPowerPack==1.4.2
rhnlib==2.5.77
rtslib-fb==2.1.57
simplejson==3.10.0
six==1.10.0
slip==0.4.0
slip.dbus==0.4.0
SSSDConfig==1.13.0
targetcli-fb===2.1.fb41
templateconfig==3.7
urlgrabber==3.10
urwid==1.1.1
Whoosh==2.7.4
yum-langpacks==0.4.2
yum-metadata-parser==1.1.4


On Monday, August 28, 2017 at 12:55:39 PM UTC-5, Daniel Kuecker wrote:
>
> I'm having an issue updating the user and passowrd for one of our 
> repositories via the admin page.  When I enter the info and hit save, I get 
> the following error:
>
> 2017-08-28 17:49:57,696 - ERROR - None - 888 - 
> /admin/db/scmtools/repository/1/ - djblets.log.middleware - Exception 
> thrown for user 88 at http:///admin/db/scmtools/repository/1/
>
> cipher AES in CFB8 mode is not supported by this backend.
> Traceback (most recent call last):
>   File "/usr/lib/python2.7/site-packages/django/core/handlers/base.py", 
> line 112, in get_response
> response = wrapped_callback(request, *callback_args, **callback_kwargs)
>   File "/usr/lib/python2.7/site-packages/django/contrib/admin/options.py", 
> line 465, in wrapper
> return self.admin_site.admin_view(view)(*args, **kwargs)
>   File "/usr/lib/python2.7/site-packages/django/utils/decorators.py", line 
> 99, in _wrapped_view
> response = view_func(request, *args, **kwargs)
>   File 
> "/usr/lib/python2.7/site-packages/django/views/decorators/cache.py", line 
> 52, in _wrapped_view_func
> response = view_func(request, *args, **kwargs)
>   File "/usr/lib/python2.7/site-packages/django/contrib/admin/sites.py", 
> line 198, in inner
> return view(request, *args, **kwargs)
>   File "/usr/lib/python2.7/site-packages/django/utils/decorators.py", line 
> 29, in _wrapper
> return bound_func(*args, **kwargs)
>   File "/usr/lib/python2.7/site-packages/django/utils/decorators.py", line 
> 99, in _wrapped_view
> response = view_func(request, *args, **kwargs)
>   File "/usr/lib/python2.7/site-packages/django/utils/decorators.py", line 
> 25, in bound_func
> return func(self, *args2, **kwargs2)
>   File "/usr/lib/python2.7/site-packages/django/db/transaction.py", line 
> 371, in inner
> return func(*args, **kwargs)
>   File "/usr/lib/python2.7/site-packages/django/contrib/admin/options.py", 
> line 1246, in change_view
> new_object = self.save_form(request, form, change=True)
>   File "/usr/lib/python2.7/site-packages/django/contrib/admin/options.py", 
> line 887, in save_form
> return form.save(commit=False)
>   File "/usr/lib/python2.7/site-packages/reviewboard/scmtools/forms.py", 
> line 1181, in save
> repository.password = self.cleaned_data['password'] or ''
>   File "/usr/lib/python2.7/site-packages/reviewboard/scmtools/models.py", 
> line 192, in _set_password
> encrypt_password(value.encode('utf-8')))
>   File 
> "/usr/lib/python2.7/site-packages/reviewboard/scmtools/crypto_utils.py", 
> line 150, in encrypt_password
> return base64.b64encode(aes_encrypt(password, key=key))
>   File 
> "/usr/lib/python2.7/site-packages/reviewboard/scmtools/crypto_utils.py", 
> line

cipher AES in CFB8 mode is not supported by this backend

[Daniel Kuecker]

I'm having an issue updating the user and passowrd for one of our 
repositories via the admin page.  When I enter the info and hit save, I get 
the following error:

2017-08-28 17:49:57,696 - ERROR - None - 888 - 
/admin/db/scmtools/repository/1/ - djblets.log.middleware - Exception 
thrown for user 88 at http:///admin/db/scmtools/repository/1/

cipher AES in CFB8 mode is not supported by this backend.
Traceback (most recent call last):
  File "/usr/lib/python2.7/site-packages/django/core/handlers/base.py", 
line 112, in get_response
response = wrapped_callback(request, *callback_args, **callback_kwargs)
  File "/usr/lib/python2.7/site-packages/django/contrib/admin/options.py", 
line 465, in wrapper
return self.admin_site.admin_view(view)(*args, **kwargs)
  File "/usr/lib/python2.7/site-packages/django/utils/decorators.py", line 
99, in _wrapped_view
response = view_func(request, *args, **kwargs)
  File "/usr/lib/python2.7/site-packages/django/views/decorators/cache.py", 
line 52, in _wrapped_view_func
response = view_func(request, *args, **kwargs)
  File "/usr/lib/python2.7/site-packages/django/contrib/admin/sites.py", 
line 198, in inner
return view(request, *args, **kwargs)
  File "/usr/lib/python2.7/site-packages/django/utils/decorators.py", line 
29, in _wrapper
return bound_func(*args, **kwargs)
  File "/usr/lib/python2.7/site-packages/django/utils/decorators.py", line 
99, in _wrapped_view
response = view_func(request, *args, **kwargs)
  File "/usr/lib/python2.7/site-packages/django/utils/decorators.py", line 
25, in bound_func
return func(self, *args2, **kwargs2)
  File "/usr/lib/python2.7/site-packages/django/db/transaction.py", line 
371, in inner
return func(*args, **kwargs)
  File "/usr/lib/python2.7/site-packages/django/contrib/admin/options.py", 
line 1246, in change_view
new_object = self.save_form(request, form, change=True)
  File "/usr/lib/python2.7/site-packages/django/contrib/admin/options.py", 
line 887, in save_form
return form.save(commit=False)
  File "/usr/lib/python2.7/site-packages/reviewboard/scmtools/forms.py", 
line 1181, in save
repository.password = self.cleaned_data['password'] or ''
  File "/usr/lib/python2.7/site-packages/reviewboard/scmtools/models.py", 
line 192, in _set_password
encrypt_password(value.encode('utf-8')))
  File 
"/usr/lib/python2.7/site-packages/reviewboard/scmtools/crypto_utils.py", 
line 150, in encrypt_password
return base64.b64encode(aes_encrypt(password, key=key))
  File 
"/usr/lib/python2.7/site-packages/reviewboard/scmtools/crypto_utils.py", 
line 88, in aes_encrypt
encryptor = cipher.encryptor()
  File 
"/usr/lib64/python2.7/site-packages/cryptography/hazmat/primitives/ciphers/base.py",
 
line 104, in encryptor
self.algorithm, self.mode
  File 
"/usr/lib64/python2.7/site-packages/cryptography/hazmat/backends/multibackend.py",
 
line 53, in create_symmetric_encryption_ctx
_Reasons.UNSUPPORTED_CIPHER
UnsupportedAlgorithm: cipher AES in CFB8 mode is not supported by this 
backend.


I'm running on RHEL 7.

-- 
Supercharge your Review Board with Power Pack: 
https://www.reviewboard.org/powerpack/
Want us to host Review Board for you? Check out RBCommons: 
https://rbcommons.com/
Happy user? Let us know! https://www.reviewboard.org/users/
--- 
You received this message because you are subscribed to the Google Groups 
"reviewboard" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

Re: Error: too many trys to get lock

[Daniel Kuecker]

Thanks for the quick reply Christian!  I see apache did not have write
access to the data folder.  I enabled it and will watch for future errors.

Thanks!

--DK

On Tue, May 23, 2017 at 3:17 PM, Christian Hammond  wrote:

> Hi Daniel,
>
> I haven't, but the first thing I'd check is whether that directory and its
> contents are writable by the web server.
>
> There's going to be a new Review Board release out by the end of the week
> that, amongst other things, makes some large improvements (and has better
> error handling) for Perforce authentication. I don't know if it would help
> much in this situation, but possibly.
>
> Christian
>
> --
> Christian Hammond
> President/CEO of Beanbag <https://www.beanbaginc.com/>
> Makers of Review Board <https://www.reviewboard.org/>
>
> On Tue, May 23, 2017 at 1:16 PM, Daniel Kuecker 
> wrote:
>
>> I'm getting this error on our production RB server.  We are using P4 with
>> LDAP authentication.
>>
>> SCMError: [P4#run] Errors during command execution( "p4 login" )
>>
>>
>>  [Error]: "Fatal client error; disconnecting!\nOperation
>> 'client-SetPassword' failed.\nToo many trys to get lock
>> /opt/apps/reviewboard/rbinstall/review-board..com/
>> data/.p4tickets.lck."
>>
>> Has anyone seen a similar error?
>>
>> --DK
>>
>> --
>> Supercharge your Review Board with Power Pack:
>> https://www.reviewboard.org/powerpack/
>> Want us to host Review Board for you? Check out RBCommons:
>> https://rbcommons.com/
>> Happy user? Let us know! https://www.reviewboard.org/users/
>> ---
>> You received this message because you are subscribed to the Google Groups
>> "reviewboard" group.
>> To unsubscribe from this group and stop receiving emails from it, send an
>> email to reviewboard+unsubscr...@googlegroups.com.
>> For more options, visit https://groups.google.com/d/optout.
>>
>
> --
> Supercharge your Review Board with Power Pack:
> https://www.reviewboard.org/powerpack/
> Want us to host Review Board for you? Check out RBCommons:
> https://rbcommons.com/
> Happy user? Let us know! https://www.reviewboard.org/users/
> ---
> You received this message because you are subscribed to a topic in the
> Google Groups "reviewboard" group.
> To unsubscribe from this topic, visit https://groups.google.com/d/
> topic/reviewboard/UjzdTSfwh34/unsubscribe.
> To unsubscribe from this group and all its topics, send an email to
> reviewboard+unsubscr...@googlegroups.com.
> For more options, visit https://groups.google.com/d/optout.
>

-- 
Supercharge your Review Board with Power Pack: 
https://www.reviewboard.org/powerpack/
Want us to host Review Board for you? Check out RBCommons: 
https://rbcommons.com/
Happy user? Let us know! https://www.reviewboard.org/users/
--- 
You received this message because you are subscribed to the Google Groups 
"reviewboard" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

Error: too many trys to get lock

[Daniel Kuecker]

I'm getting this error on our production RB server.  We are using P4 with 
LDAP authentication.

SCMError: [P4#run] Errors during command execution( "p4 login" )


 [Error]: "Fatal client error; disconnecting!\nOperation 
'client-SetPassword' failed.\nToo many trys to get lock 
/opt/apps/reviewboard/rbinstall/review-board..com/data/.p4tickets.lck."

Has anyone seen a similar error?

--DK

-- 
Supercharge your Review Board with Power Pack: 
https://www.reviewboard.org/powerpack/
Want us to host Review Board for you? Check out RBCommons: 
https://rbcommons.com/
Happy user? Let us know! https://www.reviewboard.org/users/
--- 
You received this message because you are subscribed to the Google Groups 
"reviewboard" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

Repository type is empty for GitHub Enterprise

[Pablo Daniel Dionisi]

I did the default installation (on windows with bitnami distribution) and 
I'm trying to add a GitHub enterprise repository buy the Repository 
Type comboBox is empty.

Could be an installation error?

Regards,
Pablo.

-- 
Supercharge your Review Board with Power Pack: 
https://www.reviewboard.org/powerpack/
Want us to host Review Board for you? Check out RBCommons: 
https://rbcommons.com/
Happy user? Let us know! https://www.reviewboard.org/users/
--- 
You received this message because you are subscribed to the Google Groups 
"reviewboard" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

Re: update_index always indexes the users class

[Daniel Kuecker]

Sure thing. 

Python: 2.7.5.  
I used easy_install to install RB.

On Monday, May 1, 2017 at 4:28:14 PM UTC-5, Christian Hammond wrote:
>
> Actually, rather than the Elasticsearch patch, I'd like you to try a 
> couple patches to the search indexing code. Turns out it was more 
> inefficient than I had thought. Alternatively, I could send you a build. 
> Just need to know the Python version and whether you used easy_install or 
> pip to install Review Board (or some other method).
>
> Christian
>
> -- 
> Christian Hammond
> President/CEO of Beanbag <https://www.beanbaginc.com/>
> Makers of Review Board <https://www.reviewboard.org/>
>
> On Mon, May 1, 2017 at 1:28 PM, Christian Hammond  > wrote:
>
>> Sure thing. I realized though that out of the box, this won't work 
>> without a patch to Review Board (the proper Elasticsearch support has only 
>> been thoroughly tested in 3.0). How comfortable do you feel applying 
>> patches?
>>
>> Christian
>>
>>
>> -- 
>> Christian Hammond
>> President/CEO of Beanbag <https://www.beanbaginc.com/>
>> Makers of Review Board <https://www.reviewboard.org/>
>>
>> On Mon, May 1, 2017 at 1:17 PM, Daniel Kuecker > > wrote:
>>
>>> Thanks for the quick reply Christian!
>>>
>>> I would like the info on configuring Elasticsearch if you don't mind.  
>>>
>>> --DK
>>>
>>> On Monday, May 1, 2017 at 3:12:06 PM UTC-5, Christian Hammond wrote:
>>>>
>>>> Hi Daniel,
>>>>
>>>> I believe this happens because unlike review requests, we have no way 
>>>> of determining with the search indexing stuff we use if a user's 
>>>> information is "new" to the index. Basically, we have no "last updated" 
>>>> for 
>>>> the users.
>>>>
>>>> I do see an area in the search index for users that isn't as optimal, 
>>>> so we can probably shave down the number of queries a fair amount, which 
>>>> will help.
>>>>
>>>> I assume you're using the default search backend of Whoosh? If so, at 
>>>> your scale, it might be time to install Elasticsearch. We can give you 
>>>> some 
>>>> info on how to configure that in settings_local.py (the upcoming Review 
>>>> Board 3.0 makes this very easy -- just select it as your search backend 
>>>> and 
>>>> fill in fields). Indexing with Elasticsearch is more efficient for large 
>>>> amounts of data.
>>>>
>>>> Christian
>>>>
>>>> -- 
>>>> Christian Hammond
>>>> President/CEO of Beanbag <https://www.beanbaginc.com/>
>>>> Makers of Review Board <https://www.reviewboard.org/>
>>>>
>>>> On Mon, May 1, 2017 at 12:59 PM, Daniel Kuecker  
>>>> wrote:
>>>>
>>>>> I have a fresh 2.5.9 install with a db taken from a backup of another 
>>>>> 2.5.9 install (basically a backup of my production instance).
>>>>>
>>>>> On the production instance, I am experiencing 100% cpu usage every 
>>>>> time the update_index cron job runs causing my users to complain about 
>>>>> responsiveness of the site.
>>>>>
>>>>> On my production system, the cron job shows it is indexing all the 
>>>>> records in the user table while only the new records in the review 
>>>>> requests 
>>>>> table.
>>>>>
>>>>> For debugging, I restored the production db backup to my dev machine 
>>>>> and set up a 2.5.9 instance pointing to it.  When update_index runs on 
>>>>> the 
>>>>> dev box (given the "-v 3" arg), I get the following warning: 
>>>>>
>>>>>>
>>>>>> WARNING:py.warnings:/usr/lib/python2.7/site-packages/haystack/indexes.py:167:
>>>>>>  
>>>>>> UserWarning: No updated date field found for 'User' - not restricting by 
>>>>>> age.
>>>>>>   warnings.warn(update_field_msg)
>>>>>> Indexing 1971 users
>>>>>>   indexed 1 - 1000 of 1971 (by 16819).
>>>>>>   indexed 1001 - 1971 of 1971 (by 16819).
>>>>>
>>>>>
>>>>> It then proceeds to index the entire user table and only the changed 
>>>>> review requests just as my production instance does.
>>>>>
&

Re: update_index always indexes the users class

[Daniel Kuecker]

Thanks for the quick reply Christian!

I would like the info on configuring Elasticsearch if you don't mind.  

--DK

On Monday, May 1, 2017 at 3:12:06 PM UTC-5, Christian Hammond wrote:
>
> Hi Daniel,
>
> I believe this happens because unlike review requests, we have no way of 
> determining with the search indexing stuff we use if a user's information 
> is "new" to the index. Basically, we have no "last updated" for the users.
>
> I do see an area in the search index for users that isn't as optimal, so 
> we can probably shave down the number of queries a fair amount, which will 
> help.
>
> I assume you're using the default search backend of Whoosh? If so, at your 
> scale, it might be time to install Elasticsearch. We can give you some info 
> on how to configure that in settings_local.py (the upcoming Review Board 
> 3.0 makes this very easy -- just select it as your search backend and fill 
> in fields). Indexing with Elasticsearch is more efficient for large amounts 
> of data.
>
> Christian
>
> -- 
> Christian Hammond
> President/CEO of Beanbag <https://www.beanbaginc.com/>
> Makers of Review Board <https://www.reviewboard.org/>
>
> On Mon, May 1, 2017 at 12:59 PM, Daniel Kuecker  > wrote:
>
>> I have a fresh 2.5.9 install with a db taken from a backup of another 
>> 2.5.9 install (basically a backup of my production instance).
>>
>> On the production instance, I am experiencing 100% cpu usage every time 
>> the update_index cron job runs causing my users to complain about 
>> responsiveness of the site.
>>
>> On my production system, the cron job shows it is indexing all the 
>> records in the user table while only the new records in the review requests 
>> table.
>>
>> For debugging, I restored the production db backup to my dev machine and 
>> set up a 2.5.9 instance pointing to it.  When update_index runs on the dev 
>> box (given the "-v 3" arg), I get the following warning: 
>>
>>>
>>> WARNING:py.warnings:/usr/lib/python2.7/site-packages/haystack/indexes.py:167:
>>>  
>>> UserWarning: No updated date field found for 'User' - not restricting by 
>>> age.
>>>   warnings.warn(update_field_msg)
>>> Indexing 1971 users
>>>   indexed 1 - 1000 of 1971 (by 16819).
>>>   indexed 1001 - 1971 of 1971 (by 16819).
>>
>>
>> It then proceeds to index the entire user table and only the changed 
>> review requests just as my production instance does.
>>
>> I have run the rebuild_index command on both systems.
>>
>> Am I missing something?  I have the cron job set to run ever 1/2 hour and 
>> am about to have to turn search indexing off to ensure the site is 
>> responsive.
>>
>> --DK
>>
>>
>>
>> -- 
>> Supercharge your Review Board with Power Pack: 
>> https://www.reviewboard.org/powerpack/
>> Want us to host Review Board for you? Check out RBCommons: 
>> https://rbcommons.com/
>> Happy user? Let us know! https://www.reviewboard.org/users/
>> --- 
>> You received this message because you are subscribed to the Google Groups 
>> "reviewboard" group.
>> To unsubscribe from this group and stop receiving emails from it, send an 
>> email to reviewboard...@googlegroups.com .
>> For more options, visit https://groups.google.com/d/optout.
>>
>
>

-- 
Supercharge your Review Board with Power Pack: 
https://www.reviewboard.org/powerpack/
Want us to host Review Board for you? Check out RBCommons: 
https://rbcommons.com/
Happy user? Let us know! https://www.reviewboard.org/users/
--- 
You received this message because you are subscribed to the Google Groups 
"reviewboard" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

update_index always indexes the users class

[Daniel Kuecker]

I have a fresh 2.5.9 install with a db taken from a backup of another 2.5.9 
install (basically a backup of my production instance).

On the production instance, I am experiencing 100% cpu usage every time the 
update_index cron job runs causing my users to complain about 
responsiveness of the site.

On my production system, the cron job shows it is indexing all the records 
in the user table while only the new records in the review requests table.

For debugging, I restored the production db backup to my dev machine and 
set up a 2.5.9 instance pointing to it.  When update_index runs on the dev 
box (given the "-v 3" arg), I get the following warning: 

>
> WARNING:py.warnings:/usr/lib/python2.7/site-packages/haystack/indexes.py:167: 
> UserWarning: No updated date field found for 'User' - not restricting by 
> age.
>   warnings.warn(update_field_msg)
> Indexing 1971 users
>   indexed 1 - 1000 of 1971 (by 16819).
>   indexed 1001 - 1971 of 1971 (by 16819).


It then proceeds to index the entire user table and only the changed review 
requests just as my production instance does.

I have run the rebuild_index command on both systems.

Am I missing something?  I have the cron job set to run ever 1/2 hour and 
am about to have to turn search indexing off to ensure the site is 
responsive.

--DK



-- 
Supercharge your Review Board with Power Pack: 
https://www.reviewboard.org/powerpack/
Want us to host Review Board for you? Check out RBCommons: 
https://rbcommons.com/
Happy user? Let us know! https://www.reviewboard.org/users/
--- 
You received this message because you are subscribed to the Google Groups 
"reviewboard" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

diffviewer_filediffdata table after 1.7.27 to 2.5.9 upgrade

[Daniel Kuecker]

I recently moved our RB 1.7.27 instance to a 2.5.9 instance.  I restored a 
backup of the 1.7.27 db to the 2.5.9 instance and ran rb-site install and 
waited and waited.  Everything converted successfully and we are now 
running 2.5.9 (our users love it! yay!)

NOTE:  Before the restore, to avoid the InnoDB conversion evolution, I 
manually edited the sql dump, changed the table engines to InnoDB and the 
charset to UTF8 for all tables. 

I noticed I still have a diffviewer_filediffdata table that is the same 
size as the original but has zero records.  Is that table needed anymore? 
 I'd like to get rid of it as it is 28GB in size.

Thanks!

--DK

-- 
Supercharge your Review Board with Power Pack: 
https://www.reviewboard.org/powerpack/
Want us to host Review Board for you? Check out RBCommons: 
https://rbcommons.com/
Happy user? Let us know! https://www.reviewboard.org/users/
--- 
You received this message because you are subscribed to the Google Groups 
"reviewboard" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

How big is your Review Board site?

[Daniel Kuecker]

With Review Board's recent 10th anniversary, I thought it would be 
interesting to compare the size of everyone's Review Board site.

We are sitting at 125,000 reviews, 2032 users and a 26GB database size on 
disk.

What's yours at?

-- 
Supercharge your Review Board with Power Pack: 
https://www.reviewboard.org/powerpack/
Want us to host Review Board for you? Check out RBCommons: 
https://rbcommons.com/
Happy user? Let us know! https://www.reviewboard.org/users/
--- 
You received this message because you are subscribed to the Google Groups 
"reviewboard" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

Re: Building RBTools 0.7.6 into a single-file exe for Windows

[Daniel Kuecker]

YES!  Thats a wonderful idea (the single shared install)!  Thanks for the
idea!  I think it will work for us!

--Daniel

On Fri, Sep 23, 2016 at 8:26 AM, Christian Hammond  wrote:

> Sorry, I don't have any pointers here. Tools like py2exe drastically
> changes the way Python projects execute, which can end up causing a large
> number of tricky to diagnose issues. Having done a lot of
> py2exe/pyinstaller work in the past, it's not a route I'd want to go again.
>
> While I'd accept reasonable patches for things RBTools is doing, it's
> important to note that some things simply will not work, and those are
> things we must ship with. Examples would be any commands that reference a
> shipped data file, or otherwise need to deal with something relative to a
> module path.
>
> I understand the desire to have a single .exe you can drop on development
> machines, but am not sold on this being the way to go. What a lot of
> companies do is have a single shared install of RBTools in some common
> directory on a file server, and have that directory added to the path. Some
> use a wrapper script to set the PYTHON_PATH and such for that. This gives
> you central management of RBTools, usage of the RBTools API, and wouldn't
> require the kinds of work that would be needed to get things in shape for
> tools like py2exe to work.
>
> Christian
>
> --
> Christian Hammond
> President/CEO of Beanbag <https://www.beanbaginc.com/>
> Makers of Review Board <https://www.reviewboard.org/>
>
> On Fri, Sep 23, 2016 at 6:22 AM, Daniel Kuecker 
> wrote:
>
>> Any pointers?
>>
>> On Thu, Sep 22, 2016 at 10:54 PM, john levin 
>> wrote:
>>
>>> I did.. and i was successful..
>>>
>>> On Sep 23, 2016 12:30 AM, "Daniel Kuecker"  wrote:
>>>
>>>> Has anyone been successful building RBTools 0.7.6 into a single-file
>>>> exe for distribution on windows?
>>>>
>>>> I'm trying to wrap rbt post functionality into an exe file and both
>>>> py2exe and pyinstaller built exes are having issues when pkg_resources
>>>> tries to resolve the rbtools package.
>>>>
>>>> The exe builds fine, but when I run it, I get a
>>>> "pkg_resources.distributionnotfound: the 'rbtools' distribution was
>>>> not found and is required by the application" error.
>>>>
>>>> I've tried listing rbtools as hidden imports and importing an egg
>>>> (although the egg import was never successful), to no avail.
>>>>
>>>> Thanks in advance!
>>>>
>>>> --Daniel
>>>>
>>>>
>>>>
>>>>
>>>> --
>>>> Supercharge your Review Board with Power Pack:
>>>> https://www.reviewboard.org/powerpack/
>>>> Want us to host Review Board for you? Check out RBCommons:
>>>> https://rbcommons.com/
>>>> Happy user? Let us know! https://www.reviewboard.org/users/
>>>> ---
>>>> You received this message because you are subscribed to the Google
>>>> Groups "reviewboard" group.
>>>> To unsubscribe from this group and stop receiving emails from it, send
>>>> an email to reviewboard+unsubscr...@googlegroups.com.
>>>> For more options, visit https://groups.google.com/d/optout.
>>>>
>>> --
>>> Supercharge your Review Board with Power Pack:
>>> https://www.reviewboard.org/powerpack/
>>> Want us to host Review Board for you? Check out RBCommons:
>>> https://rbcommons.com/
>>> Happy user? Let us know! https://www.reviewboard.org/users/
>>> ---
>>> You received this message because you are subscribed to the Google
>>> Groups "reviewboard" group.
>>> To unsubscribe from this group and stop receiving emails from it, send
>>> an email to reviewboard+unsubscr...@googlegroups.com.
>>> For more options, visit https://groups.google.com/d/optout.
>>>
>>
>> --
>> Supercharge your Review Board with Power Pack:
>> https://www.reviewboard.org/powerpack/
>> Want us to host Review Board for you? Check out RBCommons:
>> https://rbcommons.com/
>> Happy user? Let us know! https://www.reviewboard.org/users/
>> ---
>> You received this message because you are subscribed to the Google Groups
>> "reviewboard" group.
>> To unsubscribe from this group and stop receiving emails from it, send an
>> email to reviewboard+unsubscr...@googlegroups.com.
>> For more options, visit https://groups.go

Re: Building RBTools 0.7.6 into a single-file exe for Windows

[Daniel Kuecker]

Hey Christian,

Our clients would rather not install an app on their development machines.
We have been using an older version of RBTools, that we made a custom
post-review for, and built an exe.  The exe is dropped on the client
machine (among many other things) as a tool via a scripted setup script we
run and is called from other build-related scripts.

I only really need post and patch functionality and we hard-code the RB url.

Thanks for the response!!!

--Daniel


On Fri, Sep 23, 2016 at 12:05 AM, Christian Hammond <
christ...@beanbaginc.com> wrote:

> Out of curiosity, why not use the RBTools installer we already provide?
>
> The single .exe route will have limitations. I've done a lot of
> py2exe/pyinstaller work in the past for other projects, and it's not a
> route I'd recommend if you can avoid it, at least for more complex projects.
>
> Christian
>
> --
> Christian Hammond
> President/CEO of Beanbag <https://www.beanbaginc.com/>
> Makers of Review Board <https://www.reviewboard.org/>
>
> On Thu, Sep 22, 2016 at 11:59 AM, Daniel Kuecker 
> wrote:
>
>> Has anyone been successful building RBTools 0.7.6 into a single-file exe
>> for distribution on windows?
>>
>> I'm trying to wrap rbt post functionality into an exe file and both
>> py2exe and pyinstaller built exes are having issues when pkg_resources
>> tries to resolve the rbtools package.
>>
>> The exe builds fine, but when I run it, I get a
>> "pkg_resources.distributionnotfound: the 'rbtools' distribution was not
>> found and is required by the application" error.
>>
>> I've tried listing rbtools as hidden imports and importing an egg
>> (although the egg import was never successful), to no avail.
>>
>> Thanks in advance!
>>
>> --Daniel
>>
>>
>>
>>
>> --
>> Supercharge your Review Board with Power Pack:
>> https://www.reviewboard.org/powerpack/
>> Want us to host Review Board for you? Check out RBCommons:
>> https://rbcommons.com/
>> Happy user? Let us know! https://www.reviewboard.org/users/
>> ---
>> You received this message because you are subscribed to the Google Groups
>> "reviewboard" group.
>> To unsubscribe from this group and stop receiving emails from it, send an
>> email to reviewboard+unsubscr...@googlegroups.com.
>> For more options, visit https://groups.google.com/d/optout.
>>
>
> --
> Supercharge your Review Board with Power Pack:
> https://www.reviewboard.org/powerpack/
> Want us to host Review Board for you? Check out RBCommons:
> https://rbcommons.com/
> Happy user? Let us know! https://www.reviewboard.org/users/
> ---
> You received this message because you are subscribed to the Google Groups
> "reviewboard" group.
> To unsubscribe from this group and stop receiving emails from it, send an
> email to reviewboard+unsubscr...@googlegroups.com.
> For more options, visit https://groups.google.com/d/optout.
>

-- 
Supercharge your Review Board with Power Pack: 
https://www.reviewboard.org/powerpack/
Want us to host Review Board for you? Check out RBCommons: 
https://rbcommons.com/
Happy user? Let us know! https://www.reviewboard.org/users/
--- 
You received this message because you are subscribed to the Google Groups 
"reviewboard" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

Re: Building RBTools 0.7.6 into a single-file exe for Windows

[Daniel Kuecker]

Any pointers?

On Thu, Sep 22, 2016 at 10:54 PM, john levin  wrote:

> I did.. and i was successful..
>
> On Sep 23, 2016 12:30 AM, "Daniel Kuecker"  wrote:
>
>> Has anyone been successful building RBTools 0.7.6 into a single-file exe
>> for distribution on windows?
>>
>> I'm trying to wrap rbt post functionality into an exe file and both
>> py2exe and pyinstaller built exes are having issues when pkg_resources
>> tries to resolve the rbtools package.
>>
>> The exe builds fine, but when I run it, I get a
>> "pkg_resources.distributionnotfound: the 'rbtools' distribution was not
>> found and is required by the application" error.
>>
>> I've tried listing rbtools as hidden imports and importing an egg
>> (although the egg import was never successful), to no avail.
>>
>> Thanks in advance!
>>
>> --Daniel
>>
>>
>>
>>
>> --
>> Supercharge your Review Board with Power Pack:
>> https://www.reviewboard.org/powerpack/
>> Want us to host Review Board for you? Check out RBCommons:
>> https://rbcommons.com/
>> Happy user? Let us know! https://www.reviewboard.org/users/
>> ---
>> You received this message because you are subscribed to the Google Groups
>> "reviewboard" group.
>> To unsubscribe from this group and stop receiving emails from it, send an
>> email to reviewboard+unsubscr...@googlegroups.com.
>> For more options, visit https://groups.google.com/d/optout.
>>
> --
> Supercharge your Review Board with Power Pack:
> https://www.reviewboard.org/powerpack/
> Want us to host Review Board for you? Check out RBCommons:
> https://rbcommons.com/
> Happy user? Let us know! https://www.reviewboard.org/users/
> ---
> You received this message because you are subscribed to the Google Groups
> "reviewboard" group.
> To unsubscribe from this group and stop receiving emails from it, send an
> email to reviewboard+unsubscr...@googlegroups.com.
> For more options, visit https://groups.google.com/d/optout.
>

-- 
Supercharge your Review Board with Power Pack: 
https://www.reviewboard.org/powerpack/
Want us to host Review Board for you? Check out RBCommons: 
https://rbcommons.com/
Happy user? Let us know! https://www.reviewboard.org/users/
--- 
You received this message because you are subscribed to the Google Groups 
"reviewboard" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

Re: rbt patch "file not found"

[Daniel Kuecker]

Perforce repo and the diff was generated with and older version of RBTools
(post-review era.).

On Fri, Sep 23, 2016 at 12:06 AM Christian Hammond 
wrote:

> What type of repository is this, and how was the diff generated/uploaded?
>
> Christian
>
> --
> Christian Hammond
> President/CEO of Beanbag <https://www.beanbaginc.com/>
> Makers of Review Board <https://www.reviewboard.org/>
>
> On Thu, Sep 22, 2016 at 6:55 AM, Daniel Kuecker 
> wrote:
>
>> Oops, forgot to mention I am running this on Windows 10.
>>
>> RBTools:  0.7.6
>> RB: tried on versions 1.7.27 and 2.5.6.1
>>
>>
>> On Thursday, September 22, 2016 at 8:50:04 AM UTC-5, Daniel Kuecker wrote:
>>>
>>> Running "rbt patch " command results in a file not found
>>> error for every file in the request.  I verified the files are synced to my
>>> local drive.  Any ideas?
>>>
>>> Command line info (path info removed for anonymity):
>>> C:\P4>rbt patch 127325
>>> Patch is being applied from request 127325 with diff revision 1.
>>> Failed to execute command: [u'patch', u'-i',
>>> u'c:\\users\\***\\appdata\\local\\temp\\tmpjq4zbu']
>>> can't find file to patch at input line 3
>>> Perhaps you should have used the -p or --strip option?
>>> The text leading up to this was:
>>> --
>>> |---
>>> ///***/setupscripts/trunk/16.0/source//_common_impl_details.rb
>>>
>>> ///***/setupscripts/trunk/16.0/source//_common_impl_details.rb#6
>>> |+++
>>> ///***/setupscripts/trunk/16.0/source//_common_impl_details.rb
>>>   2016-02-18 09:21:42
>>> --
>>> File to patch:
>>> Skip this patch? [y]
>>> Skipping patch.
>>> 1 out of 1 hunk ignored
>>> missing header for unified diff at line 17 of patch
>>> can't find file to patch at input line 17
>>> Perhaps you should have used the -p or --strip option?
>>> The text leading up to this was:
>>> --
>>> |/setupscripts/trunk/16.0/source//_setup_common_impl_details.rb
>>>
>>> ///***/setupscripts/trunk/16.0/source//_setup_common_impl_details.rb#2
>>> |+++
>>> ///***/setupscripts/trunk/16.0/source//_setup_common_impl_details.rb
>>> 2016-02-18 09:21:43
>>> --
>>> File to patch:
>>> Skip this patch? [y]
>>> Skipping patch.
>>> 1 out of 1 hunk ignored
>>> missing header for unified diff at line 27 of patch
>>> can't find file to patch at input line 27
>>> Perhaps you should have used the -p or --strip option?
>>> The text leading up to this was:
>>> --
>>> |/***/setupscripts/trunk/16.0/source//_setup_impl_details.rb
>>>
>>>  
>>> ///***/setupscripts/trunk/16.0/source//_setup_impl_details.rb#2
>>> |+++
>>> ///***/setupscripts/trunk/16.0/source//_setup_impl_details.rb
>>>2016-02-18 09:21:43
>>> --
>>> File to patch:
>>> Skip this patch? [y]
>>> Skipping patch.
>>> 1 out of 1 hunk ignored
>>>
>>>
>>> --
>> Supercharge your Review Board with Power Pack:
>> https://www.reviewboard.org/powerpack/
>> Want us to host Review Board for you? Check out RBCommons:
>> https://rbcommons.com/
>> Happy user? Let us know! https://www.reviewboard.org/users/
>> ---
>> You received this message because you are subscribed to the Google Groups
>> "reviewboard" group.
>> To unsubscribe from this group and stop receiving emails from it, send an
>> email to reviewboard+unsubscr...@googlegroups.com.
>> For more options, visit https://groups.google.com/d/optout.
>>
>
> --
> Supercharge your Review Board with Power Pack:
> https://www.reviewboard.org/powerpack/
> Want us to host Review Board for you? Check out RBCommons:
> https://rbcommons.com/
> Happy user? Let us know! https://www.reviewboard.org/users/
> ---
> You received this message because you are subscribed to the Google Groups
> "reviewboard" group.
> To unsubscribe from this group and stop receiving emails from it, send an
> email to reviewboard+unsubscr...@googlegroups.com.
> For more options, visit https://groups.google.com/d/optout.
>

-- 
Supercharge your Review Board with Power Pack: 
https://www.reviewboard.org/powerpack/
Want us to host Review Board for you? Check out RBCommons: 
https://rbcommons.com/
Happy user? Let us know! https://www.reviewboard.org/users/
--- 
You received this message because you are subscribed to the Google Groups 
"reviewboard" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

Re: How to access the review board site from other system via browser?

[Daniel Kuecker]

Are you getting an error?

Did you copy your apache-wsgi.conf to /etc/httpd/conf.d/?

If you did that, you might need to enable httpd port passthru in the 
firewall.

On Thursday, September 22, 2016 at 9:19:22 AM UTC-5, P P Gobind Chandra 
Praharaj wrote:
>
> I have set-up all prerequisites and all the installation required for 
> local reviewboard along with rb-site, server is apache httpd with mod_wsi. 
>
>
> Can any one help me on how to access the review bpoard which I have 
> installed locally from different systems via web browser.
>
>
> Below are the steps detailed:
>
>
>
>1. yum -y install zlib zlib-devel 
>2. yum -y install openssl-devel 
>3. yum -y install memcached 
>4. yum -y install python-devel
>5.  yum -y install python-setuptools 
>6. yum -y install patch easy_install
>7.  mysql-python yum -y install git-core
>8. easy_install ReviewBoard 
>9. *rb-site install /var/www/ReviewBoard*
>
>
> *Steps followed during Installing rb-site*
>
>
>-  Domain name localhost:80
>-  Root Path : reviews
>- And other configurations are default.
>
>
> All steps are executed successfully.
>
>
> But I do not know how can I access the review board over browser.
>
> Please help me?
>

-- 
Supercharge your Review Board with Power Pack: 
https://www.reviewboard.org/powerpack/
Want us to host Review Board for you? Check out RBCommons: 
https://rbcommons.com/
Happy user? Let us know! https://www.reviewboard.org/users/
--- 
You received this message because you are subscribed to the Google Groups 
"reviewboard" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

Building RBTools 0.7.6 into a single-file exe for Windows

[Daniel Kuecker]

Has anyone been successful building RBTools 0.7.6 into a single-file exe 
for distribution on windows? 

I'm trying to wrap rbt post functionality into an exe file and both py2exe 
and pyinstaller built exes are having issues when pkg_resources tries to 
resolve the rbtools package.

The exe builds fine, but when I run it, I get a 
"pkg_resources.distributionnotfound: the 'rbtools' distribution was not 
found and is required by the application" error.

I've tried listing rbtools as hidden imports and importing an egg (although 
the egg import was never successful), to no avail.

Thanks in advance!

--Daniel




-- 
Supercharge your Review Board with Power Pack: 
https://www.reviewboard.org/powerpack/
Want us to host Review Board for you? Check out RBCommons: 
https://rbcommons.com/
Happy user? Let us know! https://www.reviewboard.org/users/
--- 
You received this message because you are subscribed to the Google Groups 
"reviewboard" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

Re: rbt patch "file not found"

[Daniel Kuecker]

Oops, forgot to mention I am running this on Windows 10.

RBTools:  0.7.6
RB: tried on versions 1.7.27 and 2.5.6.1

On Thursday, September 22, 2016 at 8:50:04 AM UTC-5, Daniel Kuecker wrote:
>
> Running "rbt patch " command results in a file not found error 
> for every file in the request.  I verified the files are synced to my local 
> drive.  Any ideas?
>
> Command line info (path info removed for anonymity): 
> C:\P4>rbt patch 127325
> Patch is being applied from request 127325 with diff revision 1.
> Failed to execute command: [u'patch', u'-i', 
> u'c:\\users\\***\\appdata\\local\\temp\\tmpjq4zbu']
> can't find file to patch at input line 3
> Perhaps you should have used the -p or --strip option?
> The text leading up to this was:
> --
> |--- 
> ///***/setupscripts/trunk/16.0/source//_common_impl_details.rb
>  
>   
> ///***/setupscripts/trunk/16.0/source//_common_impl_details.rb#6
> |+++ 
> ///***/setupscripts/trunk/16.0/source//_common_impl_details.rb
>  
>   2016-02-18 09:21:42
> --
> File to patch:
> Skip this patch? [y]
> Skipping patch.
> 1 out of 1 hunk ignored
> missing header for unified diff at line 17 of patch
> can't find file to patch at input line 17
> Perhaps you should have used the -p or --strip option?
> The text leading up to this was:
> --
> |/setupscripts/trunk/16.0/source//_setup_common_impl_details.rb 
>   
> ///***/setupscripts/trunk/16.0/source//_setup_common_impl_details.rb#2
> |+++ 
> ///***/setupscripts/trunk/16.0/source//_setup_common_impl_details.rb
>  
> 2016-02-18 09:21:43
> --
> File to patch:
> Skip this patch? [y]
> Skipping patch.
> 1 out of 1 hunk ignored
> missing header for unified diff at line 27 of patch
> can't find file to patch at input line 27
> Perhaps you should have used the -p or --strip option?
> The text leading up to this was:
> --
> |/***/setupscripts/trunk/16.0/source//_setup_impl_details.rb   
>   
>  
> ///***/setupscripts/trunk/16.0/source//_setup_impl_details.rb#2
> |+++ 
> ///***/setupscripts/trunk/16.0/source//_setup_impl_details.rb 
>2016-02-18 09:21:43
> --
> File to patch:
> Skip this patch? [y]
> Skipping patch.
> 1 out of 1 hunk ignored
>
>
>

-- 
Supercharge your Review Board with Power Pack: 
https://www.reviewboard.org/powerpack/
Want us to host Review Board for you? Check out RBCommons: 
https://rbcommons.com/
Happy user? Let us know! https://www.reviewboard.org/users/
--- 
You received this message because you are subscribed to the Google Groups 
"reviewboard" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

rbt patch "file not found"

[Daniel Kuecker]

Running "rbt patch " command results in a file not found error 
for every file in the request.  I verified the files are synced to my local 
drive.  Any ideas?

Command line info (path info removed for anonymity): 
C:\P4>rbt patch 127325
Patch is being applied from request 127325 with diff revision 1.
Failed to execute command: [u'patch', u'-i', 
u'c:\\users\\***\\appdata\\local\\temp\\tmpjq4zbu']
can't find file to patch at input line 3
Perhaps you should have used the -p or --strip option?
The text leading up to this was:
--
|--- 
///***/setupscripts/trunk/16.0/source//_common_impl_details.rb 
  
///***/setupscripts/trunk/16.0/source//_common_impl_details.rb#6
|+++ 
///***/setupscripts/trunk/16.0/source//_common_impl_details.rb 
  2016-02-18 09:21:42
--
File to patch:
Skip this patch? [y]
Skipping patch.
1 out of 1 hunk ignored
missing header for unified diff at line 17 of patch
can't find file to patch at input line 17
Perhaps you should have used the -p or --strip option?
The text leading up to this was:
--
|/setupscripts/trunk/16.0/source//_setup_common_impl_details.rb 
  
///***/setupscripts/trunk/16.0/source//_setup_common_impl_details.rb#2
|+++ 
///***/setupscripts/trunk/16.0/source//_setup_common_impl_details.rb
 
2016-02-18 09:21:43
--
File to patch:
Skip this patch? [y]
Skipping patch.
1 out of 1 hunk ignored
missing header for unified diff at line 27 of patch
can't find file to patch at input line 27
Perhaps you should have used the -p or --strip option?
The text leading up to this was:
--
|/***/setupscripts/trunk/16.0/source//_setup_impl_details.rb   
  
 ///***/setupscripts/trunk/16.0/source//_setup_impl_details.rb#2
|+++ 
///***/setupscripts/trunk/16.0/source//_setup_impl_details.rb 
   2016-02-18 09:21:43
--
File to patch:
Skip this patch? [y]
Skipping patch.
1 out of 1 hunk ignored


-- 
Supercharge your Review Board with Power Pack: 
https://www.reviewboard.org/powerpack/
Want us to host Review Board for you? Check out RBCommons: 
https://rbcommons.com/
Happy user? Let us know! https://www.reviewboard.org/users/
--- 
You received this message because you are subscribed to the Google Groups 
"reviewboard" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

Re: rbt patch on Windows troubles

[Daniel Kuecker]

I'm also having issues, but on Windows 10.

I was able to get patch.exe to run via the link Igor posted above 
(http://math.nist.gov/oommf/software-patchsets/patch_on_Windows7.html).

When I manually run patch with the temporary file "rbt patch" creates, it 
crashes.  An inspection of the temporary patch file shows windows line 
endings (LF) instead of unix line endings (CR/LF).  Adding the "--binary" 
option to the patch.exe call 
(per http://gnuwin32.sourceforge.net/packages/patch.htm) works when 
manually run.

Is there any way to get rbt patch to create a temporary patch file with 
proper line endings?

FWIW:  I ran this from a linux machine and it works fine.

Thanks!

--Daniel

On Tuesday, April 8, 2014 at 3:07:23 PM UTC-5, Igor Berger wrote:
>
> Hello,
>
> This is not a Review Board problem. I'm just wondering whether anyone is 
> using "rbt patch" on Windows 7.
>
> I figured out that in order for "rbt patch" to work, patch itself has to 
> be installed separately.
> I've found these precompiled binaries for Windows: 
> http://gnuwin32.sourceforge.net/packages/patch.htm
>
> However, patch crashes even if I use workarounds listed here:
> http://math.nist.gov/oommf/software-patchsets/patch_on_Windows7.html
>
> Any ideas?
>
> Thank you,
> Igor.
>
>

-- 
Supercharge your Review Board with Power Pack: 
https://www.reviewboard.org/powerpack/
Want us to host Review Board for you? Check out RBCommons: 
https://rbcommons.com/
Happy user? Let us know! https://www.reviewboard.org/users/
--- 
You received this message because you are subscribed to the Google Groups 
"reviewboard" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

Re: Reviewboard upgrade failure

['Daniel Laird' via reviewboard]

All,

I am trying to upgrade from 2.0.20 -> 2.5.3.
I do this by taking a backup of the 2.0.20 database then importing into 
MySQL and then running the rb-site upgrade.

However I get the following output:

Updating database. This may take a while.

The log output below, including warnings and errors,
can be ignored unless upgrade fails.

--  --
Creating tables ...
There are unapplied evolutions for accounts.
There are unapplied evolutions for attachments.
There are unapplied evolutions for diffviewer.
There are unapplied evolutions for reviews.
There are unapplied evolutions for webapi.
Project signature has changed - an evolution is required
Installing custom SQL ...
Installing indexes ...
Installed 0 object(s) from 0 fixture(s)
/usr/local/lib/python2.7/dist-packages/ReviewBoardPowerPack-1.4-py2.7.egg/rbpowerpack/scmtools/tfs.py:9:
 
DeprecationWarning: django.utils.simplejson is deprecated; use json instead.

CommandError: Error applying evolution: (1060, "Duplicate column name 
'visibility'")
...

Any ideas on why I am getting this error or how I can get more debug to 
help?
Much appreciated
Dan

On Friday, 15 January 2016 19:44:43 UTC, Ben Cooksley wrote:
>
> On Sat, Jan 16, 2016 at 6:09 AM, Christian Hammond  > wrote: 
> > Hi Ben, 
> > 
> > You'll have a much easier time restoring from a backup. It's hard to say 
> how 
> > far it went through the evolution process, and unfortunately today it 
> > doesn't keep track of how far it got and what it'd have to do to 
> recover. 
> > You'd have a lot of trial and error to fix it manually. You can try it, 
> > though. 
> > 
> > Basically, you'll need to dump the SQL that the evolutions want to 
> apply, 
> > and go through and hand-undo each thing it did until you get back to the 
> > point of where it was. You'd definitely want to do a backup first, 
> though. 
>
> Unfortunately people had started using it already so this was the 
> easiest approach :( 
>
> Would it be possible to get a copy of a normally, safely upgraded 
> schema so I can double check I haven't clobbered anything? 
>
> The queries I ended up having to run to revert things to a state where 
> the upgrade process would work was: 
>
> 160115 19:34:12 2197763 Query   ALTER TABLE 
> accounts_reviewrequestvisit DROP COLUMN visibility 
> 160115 19:34:24 2197763 Query   DROP INDEX 
> `accounts_reviewrequestvisit_05ee5d21` ON 
> `accounts_reviewrequestvisit` 
> 160115 19:34:45 2197763 Query   ALTER TABLE attachments_fileattachment 
> DROP COLUMN attachment_revision, DROP COLUMN attachment_history_id 
> 160115 19:34:58 2197763 Query   ALTER TABLE diffviewer_filediff DROP 
> COLUMN raw_diff_hash_id, DROP COLUMN raw_parent_diff_hash_id 
> 160115 19:36:18 2197763 Query   ALTER TABLE `reviews_group` DROP 
> COLUMN `email_list_only`, DROP COLUMN is_default_group 
> 160115 19:36:25 2197763 Query   DROP TABLE 
> reviews_reviewrequest_file_attachment_histories 
>
> Note that I observed that the Reviewboard process tries to reverse 
> it's failed upgrade by doing a rollback. It is noted in the case of 
> InnoDB that schema changes cannot be rolled back (see 
> http://www.sitepoint.com/mysql-transaction-gotchas-good-parts/) 
>
> > 
> > Christian 
>
> Cheers, 
> Ben 
>
> > 
> > -- 
> > Christian Hammond - chi...@chipx86.com  
> > Review Board - https://www.reviewboard.org 
> > Beanbag, Inc. - https://www.beanbaginc.com 
> > 
> > On Fri, Jan 15, 2016 at 2:26 AM, Ben Cooksley  > wrote: 
> >> 
> >> On Fri, Jan 15, 2016 at 11:09 PM, Ben Cooksley  > wrote: 
> >> > On Fri, Jan 15, 2016 at 11:04 PM, Christian Hammond 
> >> > > wrote: 
> >> >> Hi Ben, 
> >> >> 
> >> >> This is due to a mismatch between MySQL table types. The existing 
> >> >> tables are 
> >> >> likely MyISAM, with MySQL now defaulting to InnoDB for new ones. 
> You'll 
> >> >> need 
> >> >> to either migrate all the existing tables, or tell MySQL to use the 
> >> >> existing 
> >> >> type for new tables. 
> >> >> 
> >> >> (It's a pretty terrible error, but unfortunately, beyond our 
> control. I 
> >> >> just 
> >> >> recognize this sort of problem.) 
> >> > 
> >> > Argh. Our systems usually have InnoDB as default, guess that isn't 
> the 
> >> > case when we originally had Reviewboard provisioned. 
> >> > I shouldn't see any issues migrating all tables into InnoDB correct? 
> >> 
> >> Seems it is safe. 
> >> Unfortunately it looks like one of the evolutions got part way through 
> >> the process. 
> >> 
> >> CommandError: Error applying evolution: (1060, "Duplicate column name 
> >> 'visibility'") 
> >> 
> >> Any suggestions (I could restore from backups, but if I can avoid 
> it...)? 
> >> 
> >> > 
> >> >> 
> >> >> Christian 
> >> > 
> >> > Cheers, 
> >> > Ben 
> >> 
> >> Thanks, 
> >> Ben 
> >> 
> >> > 
> >> >> 
> >> >> -- 
> >> >> Christian Hammond - chri...@beanbaginc.com  
> >> >> Review Board - https://www.reviewboard.org 
> >> >> Beanbag, Inc. - https://www.beanbaginc.com 
> >> >> 
> >> >> On Thu, Jan 14, 

Re: Reviewboard upgrade failure

['Daniel Laird' via reviewboard]

I too had this issue (MySQL default changing from MyISAM to InnoDB).
Although I understand that this is caused by MySQL version being updated at 
the same time as a Reviewboard Update (and migrate).
But would there be any way for the 'upgrade' logic of Reviewboard to post a 
more useful error message (DB Table formats dont match) or something like 
that.
As I am sure it will save a lot of time for some people (but I dont have 
the skills to make these changes sadly)

Dan

On Friday, 15 January 2016 07:27:08 UTC, Ben Cooksley wrote:
>
> Hi everyone, 
>
> While upgrading from Reviewboard 2.0.17 to 2.5.2 i've encountered a 
> few SQL errors. 
> Reviewboard itself appears to boot and run fine though, based on the 
> nature of the errors I suspect it's a case of index name collisions. 
>
> Output from the upgrade process is below: 
>
> (virtualenv)reviewboard@mimi:~$ rb-site upgrade 
> /srv/www/reviewboard/git.reviewboard.kde.org/ 
> Rebuilding directory structure 
> Updating database. This may take a while. 
>
> The log output below, including warnings and errors, 
> can be ignored unless upgrade fails. 
>
> --  -- 
> Creating tables ... 
> Creating table accounts_trophy 
>
> [!] There was an error synchronizing the database. Make sure the 
> database is created and has the appropriate permissions, and then 
> continue. 
> [!] Details: (1005, "Can't create table 'reviewboard_git.#sql- 
> 331d_214215' (errno: 150)") 
>
> Press Enter to continue 
> Creating tables ... 
> Creating table attachments_fileattachmenthistory 
> Creating table diffviewer_rawfilediffdata 
> Creating table notifications_webhooktarget_repositories 
>
> [!] There was an error synchronizing the database. Make sure the 
> database is created and has the appropriate permissions, and then 
> continue. 
> [!] Details: (1005, "Can't create table 'reviewboard_git.#sql- 
> 331d_214215' (errno: 150)") 
>
> Press Enter to continue 
> Creating tables ... 
> Creating table notifications_webhooktarget 
>
> [!] There was an error synchronizing the database. Make sure the 
> database is created and has the appropriate permissions, and then 
> continue. 
> [!] Details: (1005, "Can't create table 'reviewboard_git.#sql- 
> 331d_214215' (errno: 150)") 
>
> Press Enter to continue 
> Creating tables ... 
> Creating table webapi_webapitoken 
>
> [!] There was an error synchronizing the database. Make sure the 
> database is created and has the appropriate permissions, and then 
> continue. 
> [!] Details: (1005, "Can't create table 'reviewboard_git.#sql- 
> 331d_214215' (errno: 150)") 
>
> Press Enter to continue 
> Creating tables ... 
> Upgrading Review Board from 2.0.17 to 2.5.2 
> There are unapplied evolutions for accounts. 
> There are unapplied evolutions for attachments. 
> There are unapplied evolutions for diffviewer. 
> There are unapplied evolutions for notifications. 
> There are unapplied evolutions for reviews. 
> There are unapplied evolutions for webapi. 
> Adding baseline version for new models 
> Evolutions in notifications baseline: webhooktarget_extra_state, 
> webhooktarget_extra_data_null 
> Project signature has changed - an evolution is required 
> Installing custom SQL ... 
> Installing indexes ... 
> Installed 0 object(s) from 0 fixture(s) 
> CommandError: Error applying evolution: (1005, "Can't create table 
> 'reviewboard_git.#sql-331d_214238' (errno: 150)") 
>
> Any pointers? 
>
> Cheers, 
> Ben Cooksley 
> KDE Sysadmin 
>

-- 
Supercharge your Review Board with Power Pack: 
https://www.reviewboard.org/powerpack/
Want us to host Review Board for you? Check out RBCommons: 
https://rbcommons.com/
Happy user? Let us know! https://www.reviewboard.org/users/
--- 
You received this message because you are subscribed to the Google Groups 
"reviewboard" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

Depends-On, Blocks and Circular Reference

[Daniel Laird]

WE are running 2.0.17 and have managed to create a situation and dont know 
how to resolve it - or whether its a defect

Someone created 2 reviews with code changes that were dependent.
They marked Review 2 as Depends On Review 1 -  this then updated the 
'Blocks' field.
They then marked Review 1 as Depends On Review 2 - as they felt that both 
reviews had to be committed together.

However the server then starts to give us lots of errors:
Traceback (most recent call last):

  File 
"/usr/local/lib/python2.7/dist-packages/Django-1.6.11-py2.7.egg/django/core/handlers/base.py",
 
line 199, in get_response
response = middleware_method(request, response)

  File 
"/usr/local/lib/python2.7/dist-packages/Django-1.6.11-py2.7.egg/django/middleware/http.py",
 
line 14, in process_response
response['Content-Length'] = str(len(response.content))

  File 
"/usr/local/lib/python2.7/dist-packages/Djblets-0.8.20-py2.7.egg/djblets/webapi/responses.py",
 
line 109, in _get_content
**self.encoder_kwargs)

  File 
"/usr/local/lib/python2.7/dist-packages/Djblets-0.8.20-py2.7.egg/djblets/webapi/encoders.py",
 
line 110, in encode
return super(JSONEncoderAdapter, self).encode(o)

  File "/usr/lib/python2.7/json/encoder.py", line 201, in encode
chunks = self.iterencode(o, _one_shot=True)

  File "/usr/lib/python2.7/json/encoder.py", line 264, in iterencode
return _iterencode(o, 0)

ValueError: Circular reference detected

It maybe that the double linking is incorrect usage - however having been 
able to get into that situation.  If we try to remove one of the 'Blocks' 
fields we just get a spinning 'Loading' icon and are unable to move past 
this to remove the Circular Reference.

Not sure if you think this is user error, a defect or a combination of both 
- but thought I would share it.

Cheers
Dan

-- 
Supercharge your Review Board with Power Pack: 
https://www.reviewboard.org/powerpack/
Want us to host Review Board for you? Check out RBCommons: 
https://rbcommons.com/
Happy user? Let us know! https://www.reviewboard.org/users/
--- 
You received this message because you are subscribed to the Google Groups 
"reviewboard" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

Re: Problems upgrading to 2.0.13 and 2.0.15

[Daniel Dominguez]

That fixed it.  Many Thanks!

On Friday, April 17, 2015 at 2:05:03 PM UTC-7, Christian Hammond wrote:
>
> Hi Daniel,
>
> Sorry, I've been pretty swamped.
>
> First, make a backup of your database. Then, try:
>
> DELETE FROM django_project_version WHERE id=4;
>
> Then perform the upgrade again.
>
> Christian
>
> -- 
> Christian Hammond - chi...@chipx86.com 
> Review Board - http://www.reviewboard.org
> Beanbag, Inc. - http://www.beanbaginc.com
>
> On Fri, Apr 17, 2015 at 10:29 AM, Daniel Dominguez  > wrote:
>
>> Any update here?
>>
>>
>> On Friday, March 20, 2015 at 1:29:07 PM UTC-7, Daniel Dominguez wrote:
>>>
>>> Running --hint --evolve had not been done before.  It was just my 
>>> experimenting after seeing this issue (with a current backup of the db 
>>> first of course).
>>>
>>> Anyhow, I've attached the sql dump you requested.
>>>
>>> Thanks!
>>>
>>> On Friday, March 20, 2015 at 8:28:36 AM UTC-7, Daniel Dominguez wrote:
>>>>
>>>> When I performed the upgrade form 2.0.12 to 2.0.13, the site-upgrade 
>>>> failed.  It didn't seem to break our reviewboard instance, so I just 
>>>> ignored it at the time.  I recently upgraded to 2.0.15, and the result is 
>>>> the same.  Again, our reviewboard server seems to be fine, but this is 
>>>> starting to make me nervous.
>>>>
>>>>
>>>> --  --
>>>> Creating tables ...
>>>> Project signature has changed - an evolution is required
>>>> Installing custom SQL ...
>>>> Installing indexes ...
>>>> Installed 0 object(s) from 0 fixture(s)
>>>> The stored evolutions do not completely resolve all model changes.
>>>>
>>>> Run `./manage.py evolve --hint` to see a suggestion for the changes 
>>>> required.
>>>>
>>>> The following are the changes that could not be resolved:
>>>> In model reviews.ReviewRequest:
>>>> Field 'description_rich_text' has been added
>>>> Field 'testing_done_rich_text' has been added
>>>> In model reviews.Review:
>>>> Field 'body_top_rich_text' has been added
>>>> Field 'body_bottom_rich_text' has been added
>>>> In model reviews.ReviewRequestDraft:
>>>> Field 'description_rich_text' has been added
>>>> Field 'testing_done_rich_text' has been added
>>>> In model accounts.Profile:
>>>> Field 'default_use_rich_text' has been added
>>>> CommandError: Your models contain changes that Django Evolution cannot 
>>>> resolve automatically.
>>>>
>>>> 
>>>> I've trying running evolve -- --hint --execute, but that fails with the 
>>>> following
>>>>
>>>> CommandError: Error applying evolution: (1060, "Duplicate column name 
>>>> 'default_use_rich_text'")
>>>>
>>>>
>>>>
>>>> I've checked that I have the latest django_evolution installed for my 
>>>> platform (0.7.4).  This is running on a MySql database.
>>>>
>>>> It seems like the evolution is trying to add columns that already 
>>>> exist.  Maybe a version table didn't get incremented in the prior 
>>>> evolution?  Any hints on how I can troubleshoot this?
>>>>
>>>>
>>>>
>>>>  -- 
>> Supercharge your Review Board with Power Pack: 
>> https://www.reviewboard.org/powerpack/
>> Want us to host Review Board for you? Check out RBCommons: 
>> https://rbcommons.com/
>> Happy user? Let us know! https://www.reviewboard.org/users/
>> --- 
>> You received this message because you are subscribed to the Google Groups 
>> "reviewboard" group.
>> To unsubscribe from this group and stop receiving emails from it, send an 
>> email to reviewboard...@googlegroups.com .
>> For more options, visit https://groups.google.com/d/optout.
>>
>
>

-- 
Supercharge your Review Board with Power Pack: 
https://www.reviewboard.org/powerpack/
Want us to host Review Board for you? Check out RBCommons: 
https://rbcommons.com/
Happy user? Let us know! https://www.reviewboard.org/users/
--- 
You received this message because you are subscribed to the Google Groups 
"reviewboard" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

Re: Problems upgrading to 2.0.13 and 2.0.15

[Daniel Dominguez]

Any update here?

On Friday, March 20, 2015 at 1:29:07 PM UTC-7, Daniel Dominguez wrote:
>
> Running --hint --evolve had not been done before.  It was just my 
> experimenting after seeing this issue (with a current backup of the db 
> first of course).
>
> Anyhow, I've attached the sql dump you requested.
>
> Thanks!
>
> On Friday, March 20, 2015 at 8:28:36 AM UTC-7, Daniel Dominguez wrote:
>>
>> When I performed the upgrade form 2.0.12 to 2.0.13, the site-upgrade 
>> failed.  It didn't seem to break our reviewboard instance, so I just 
>> ignored it at the time.  I recently upgraded to 2.0.15, and the result is 
>> the same.  Again, our reviewboard server seems to be fine, but this is 
>> starting to make me nervous.
>>
>>
>> --  --
>> Creating tables ...
>> Project signature has changed - an evolution is required
>> Installing custom SQL ...
>> Installing indexes ...
>> Installed 0 object(s) from 0 fixture(s)
>> The stored evolutions do not completely resolve all model changes.
>>
>> Run `./manage.py evolve --hint` to see a suggestion for the changes 
>> required.
>>
>> The following are the changes that could not be resolved:
>> In model reviews.ReviewRequest:
>> Field 'description_rich_text' has been added
>> Field 'testing_done_rich_text' has been added
>> In model reviews.Review:
>> Field 'body_top_rich_text' has been added
>> Field 'body_bottom_rich_text' has been added
>> In model reviews.ReviewRequestDraft:
>> Field 'description_rich_text' has been added
>> Field 'testing_done_rich_text' has been added
>> In model accounts.Profile:
>> Field 'default_use_rich_text' has been added
>> CommandError: Your models contain changes that Django Evolution cannot 
>> resolve automatically.
>>
>> 
>> I've trying running evolve -- --hint --execute, but that fails with the 
>> following
>>
>> CommandError: Error applying evolution: (1060, "Duplicate column name 
>> 'default_use_rich_text'")
>>
>>
>>
>> I've checked that I have the latest django_evolution installed for my 
>> platform (0.7.4).  This is running on a MySql database.
>>
>> It seems like the evolution is trying to add columns that already exist. 
>>  Maybe a version table didn't get incremented in the prior evolution?  Any 
>> hints on how I can troubleshoot this?
>>
>>
>>
>>

-- 
Supercharge your Review Board with Power Pack: 
https://www.reviewboard.org/powerpack/
Want us to host Review Board for you? Check out RBCommons: 
https://rbcommons.com/
Happy user? Let us know! https://www.reviewboard.org/users/
--- 
You received this message because you are subscribed to the Google Groups 
"reviewboard" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

Re: RBtools 0.7 - issue posting a review

[Daniel Laird]

Ok - Will post the raw contents and other stuff I think is useful to you at 
the email address.
Much appreciated
Dan

On Tuesday, 7 April 2015 23:53:24 UTC+1, Christian Hammond wrote:
>
> Hey Dan,
>
> Would you be able to provide us with the raw contents of that svn diff 
> command? We need some test data to help us repro the problem.
>
> Assuming this is confidential data, you can send that attachment along to 
> sup...@beanbaginc.com  (though we'll still discuss the issue 
> here).
>
> Thanks!
>
> - Christian
>
> -- 
> Christian Hammond - chi...@chipx86.com 
> Review Board - http://www.reviewboard.org
> Beanbag, Inc. - http://www.beanbaginc.com
>
> On Tue, Apr 7, 2015 at 8:59 AM, Daniel Laird  > wrote:
>
>> On a new Ubuntu 14.04 installation I am seeing something similar:
>> (We are testing 14.04 before wider rollout.)
>> >>> Running: svn info --non-interactive
>> >>> Running: diff --version
>> >>> Running: svn --version -q
>> >>> repository info: Path: http://REVIEWBOARD_URL/NAME, Base path: PATH 
>> TO CHANGES, Supports changesets: False
>> >>> Running: svn status -q --ignore-externals
>> >>> Running: svn diff --diff-cmd=diff --notice-ancestry -r BASE
>> Traceback (most recent call last):
>>   File "/usr/local/bin/rbt", line 9, in 
>> load_entry_point('RBTools==0.7.2', 'console_scripts', 'rbt')()
>>   File 
>> "/usr/local/lib/python2.7/dist-packages/RBTools-0.7.2-py2.7.egg/rbtools/commands/main.py",
>>  
>> line 133, in main
>> command.run_from_argv([RB_MAIN, command_name] + args)
>>   File 
>> "/usr/local/lib/python2.7/dist-packages/RBTools-0.7.2-py2.7.egg/rbtools/commands/__init__.py",
>>  
>> line 555, in run_from_argv
>> exit_code = self.main(*args) or 0
>>   File 
>> "/usr/local/lib/python2.7/dist-packages/RBTools-0.7.2-py2.7.egg/rbtools/commands/post.py",
>>  
>> line 639, in main
>> extra_args=extra_args)
>>   File 
>> "/usr/local/lib/python2.7/dist-packages/RBTools-0.7.2-py2.7.egg/rbtools/clients/svn.py",
>>  
>> line 319, in diff
>> diff = self._run_svn(diff_cmd, split_lines=True)
>>   File 
>> "/usr/local/lib/python2.7/dist-packages/RBTools-0.7.2-py2.7.egg/rbtools/clients/svn.py",
>>  
>> line 707, in _run_svn
>> return execute(cmdline, *args, **kwargs)
>>   File 
>> "/usr/local/lib/python2.7/dist-packages/RBTools-0.7.2-py2.7.egg/rbtools/utils/process.py",
>>  
>> line 104, in execute
>> data = [line.decode('utf-8') for line in data]
>>   File "/usr/lib/python2.7/encodings/utf_8.py", line 16, in decode
>> return codecs.utf_8_decode(input, errors, True)
>> UnicodeDecodeError: 'utf8' codec can't decode byte 0xe4 in position 17: 
>> invalid continuation byte
>>
>> I appear to have all the correct locates installed - but no luck.
>> It is running SVN 1.8.8 and Ubuntu 14.04 and the latest RBT- any ideas 
>> what to debug - it does not appear to be log related this time
>> This command:
>> svn diff --diff-cmd=diff --notice-ancestry -r BASE
>> Runs fine on the commandline standalone
>>
>> Dan
>>
>> On Sunday, 15 February 2015 21:20:00 UTC, Daniel Laird wrote:
>>>
>>> After some investigation we found the following:
>>>
>>> # TODO: This can break on systems that don't have the en_US local
>>>
>>> # installed (which isn't very many). Ideally in this case, we could
>>>
>>> # put something in the config file, but that's not plumbed through to 
>>> here.
>>>
>>> env['LC_ALL'] = 'en_US.UTF-8'
>>>
>>> env['LANGUAGE'] = 'en_US.UTF-8'
>>>
>>> We can confirm that it does break things!
>>>
>>> With the 'en_US.UTF-8' locale installed the problem goes away - is there 
>>> a way to check this locale is installed and error nicely rather than 
>>> breaking?
>>> Cheers
>>> Dan
>>>
>>> On Saturday, 31 January 2015 18:49:02 UTC, David Trowbridge wrote:
>>>>
>>>> Hi Daniel,
>>>>
>>>> Do you think you could add some code to print out the contents of 'log' 
>>>> inside _convert_symbolic_revision before we try to construct the etree?
>>>>
>>>> Thanks,
>>>> -David
>>>>
>>>>
>>>> On Fri 

Re: RBtools 0.7 - issue posting a review

[Daniel Laird]

On a new Ubuntu 14.04 installation I am seeing something similar:
(We are testing 14.04 before wider rollout.)
>>> Running: svn info --non-interactive
>>> Running: diff --version
>>> Running: svn --version -q
>>> repository info: Path: http://REVIEWBOARD_URL/NAME, Base path: PATH TO 
CHANGES, Supports changesets: False
>>> Running: svn status -q --ignore-externals
>>> Running: svn diff --diff-cmd=diff --notice-ancestry -r BASE
Traceback (most recent call last):
  File "/usr/local/bin/rbt", line 9, in 
load_entry_point('RBTools==0.7.2', 'console_scripts', 'rbt')()
  File 
"/usr/local/lib/python2.7/dist-packages/RBTools-0.7.2-py2.7.egg/rbtools/commands/main.py",
 
line 133, in main
command.run_from_argv([RB_MAIN, command_name] + args)
  File 
"/usr/local/lib/python2.7/dist-packages/RBTools-0.7.2-py2.7.egg/rbtools/commands/__init__.py",
 
line 555, in run_from_argv
exit_code = self.main(*args) or 0
  File 
"/usr/local/lib/python2.7/dist-packages/RBTools-0.7.2-py2.7.egg/rbtools/commands/post.py",
 
line 639, in main
extra_args=extra_args)
  File 
"/usr/local/lib/python2.7/dist-packages/RBTools-0.7.2-py2.7.egg/rbtools/clients/svn.py",
 
line 319, in diff
diff = self._run_svn(diff_cmd, split_lines=True)
  File 
"/usr/local/lib/python2.7/dist-packages/RBTools-0.7.2-py2.7.egg/rbtools/clients/svn.py",
 
line 707, in _run_svn
return execute(cmdline, *args, **kwargs)
  File 
"/usr/local/lib/python2.7/dist-packages/RBTools-0.7.2-py2.7.egg/rbtools/utils/process.py",
 
line 104, in execute
data = [line.decode('utf-8') for line in data]
  File "/usr/lib/python2.7/encodings/utf_8.py", line 16, in decode
return codecs.utf_8_decode(input, errors, True)
UnicodeDecodeError: 'utf8' codec can't decode byte 0xe4 in position 17: 
invalid continuation byte

I appear to have all the correct locates installed - but no luck.
It is running SVN 1.8.8 and Ubuntu 14.04 and the latest RBT- any ideas what 
to debug - it does not appear to be log related this time
This command:
svn diff --diff-cmd=diff --notice-ancestry -r BASE
Runs fine on the commandline standalone

Dan

On Sunday, 15 February 2015 21:20:00 UTC, Daniel Laird wrote:
>
> After some investigation we found the following:
>
> # TODO: This can break on systems that don't have the en_US local
>
> # installed (which isn't very many). Ideally in this case, we could
>
> # put something in the config file, but that's not plumbed through to here.
>
> env['LC_ALL'] = 'en_US.UTF-8'
>
> env['LANGUAGE'] = 'en_US.UTF-8'
>
> We can confirm that it does break things!
>
> With the 'en_US.UTF-8' locale installed the problem goes away - is there a 
> way to check this locale is installed and error nicely rather than breaking?
> Cheers
> Dan
>
> On Saturday, 31 January 2015 18:49:02 UTC, David Trowbridge wrote:
>>
>> Hi Daniel,
>>
>> Do you think you could add some code to print out the contents of 'log' 
>> inside _convert_symbolic_revision before we try to construct the etree?
>>
>> Thanks,
>> -David
>>
>>
>> On Fri Jan 30 2015 at 4:48:37 AM Daniel Laird  
>> wrote:
>>
>>> We have a shared Debian server and we have updated RBtools to 0.7.
>>>
>>> When we try to post a review we get the following:
>>>
>>> SERVER_NAME> rbt post -d NUM
>>> >>> RBTools 0.7
>>> >>> Python 2.7.3 (default, Mar 13 2014, 11:03:55) 
>>> [GCC 4.7.2]
>>> >>> Running on Linux-3.2.0-4-amd64-x86_64-with-debian-7.8
>>> >>> Home = /v/home/rad/USERNAME
>>> >>> Current directory = /v/space/SERVER_NAME/USERNAME/
>>> wc/memory_corruption
>>> >>> Checking for a Subversion repository...
>>> >>> Running: svn info --non-interactive
>>> >>> Running: diff --version
>>> >>> repository info: Path: https://SVN_SERVER/svn-restricted/PROJNAME, 
>>> Base path: /branches/users/bob/memory_corruption, Supports changesets: 
>>> False
>>> >>> Making HTTP GET request to http://REVIEWBOARD_SERVER/api/
>>> >>> HTTP GET request to http://REVIEWBOARD_SERVER/api/ cannot be cached
>>> >>> Making HTTP GET request to http://REVIEWBOARD_SERVER/api/
>>> repositories/
>>> >>> HTTP GET request to http://REVIEWBOARD_SERVER/api/repositories/ 
>>> cannot be cached
>>> >>> Making HTTP GET request to http://REVIEWBOARD_SERVER/api/
>>> repositories/?max-results=25

Re: Problems upgrading to 2.0.13 and 2.0.15

[Daniel Dominguez]

Running --hint --evolve had not been done before.  It was just my 
experimenting after seeing this issue (with a current backup of the db 
first of course).

Anyhow, I've attached the sql dump you requested.

Thanks!

On Friday, March 20, 2015 at 8:28:36 AM UTC-7, Daniel Dominguez wrote:
>
> When I performed the upgrade form 2.0.12 to 2.0.13, the site-upgrade 
> failed.  It didn't seem to break our reviewboard instance, so I just 
> ignored it at the time.  I recently upgraded to 2.0.15, and the result is 
> the same.  Again, our reviewboard server seems to be fine, but this is 
> starting to make me nervous.
>
>
> --  --
> Creating tables ...
> Project signature has changed - an evolution is required
> Installing custom SQL ...
> Installing indexes ...
> Installed 0 object(s) from 0 fixture(s)
> The stored evolutions do not completely resolve all model changes.
>
> Run `./manage.py evolve --hint` to see a suggestion for the changes 
> required.
>
> The following are the changes that could not be resolved:
> In model reviews.ReviewRequest:
> Field 'description_rich_text' has been added
> Field 'testing_done_rich_text' has been added
> In model reviews.Review:
> Field 'body_top_rich_text' has been added
> Field 'body_bottom_rich_text' has been added
> In model reviews.ReviewRequestDraft:
> Field 'description_rich_text' has been added
> Field 'testing_done_rich_text' has been added
> In model accounts.Profile:
> Field 'default_use_rich_text' has been added
> CommandError: Your models contain changes that Django Evolution cannot 
> resolve automatically.
>
> 
> I've trying running evolve -- --hint --execute, but that fails with the 
> following
>
> CommandError: Error applying evolution: (1060, "Duplicate column name 
> 'default_use_rich_text'")
>
>
>
> I've checked that I have the latest django_evolution installed for my 
> platform (0.7.4).  This is running on a MySql database.
>
> It seems like the evolution is trying to add columns that already exist. 
>  Maybe a version table didn't get incremented in the prior evolution?  Any 
> hints on how I can troubleshoot this?
>
>
>
>

-- 
Supercharge your Review Board with Power Pack: 
https://www.reviewboard.org/powerpack/
Want us to host Review Board for you? Check out RBCommons: 
https://rbcommons.com/
Happy user? Let us know! https://www.reviewboard.org/users/
--- 
You received this message because you are subscribed to the Google Groups 
"reviewboard" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.
-- MySQL dump 10.13  Distrib 5.5.41, for debian-linux-gnu (x86_64)
--
-- Host: localhostDatabase: reviewboard
-- --
-- Server version	5.5.41-0+wheezy1

/*!40101 SET @OLD_CHARACTER_SET_CLIENT=@@CHARACTER_SET_CLIENT */;
/*!40101 SET @OLD_CHARACTER_SET_RESULTS=@@CHARACTER_SET_RESULTS */;
/*!40101 SET @OLD_COLLATION_CONNECTION=@@COLLATION_CONNECTION */;
/*!40101 SET NAMES utf8 */;
/*!40103 SET @OLD_TIME_ZONE=@@TIME_ZONE */;
/*!40103 SET TIME_ZONE='+00:00' */;
/*!40014 SET @OLD_UNIQUE_CHECKS=@@UNIQUE_CHECKS, UNIQUE_CHECKS=0 */;
/*!40014 SET @OLD_FOREIGN_KEY_CHECKS=@@FOREIGN_KEY_CHECKS, FOREIGN_KEY_CHECKS=0 */;
/*!40101 SET @OLD_SQL_MODE=@@SQL_MODE, SQL_MODE='NO_AUTO_VALUE_ON_ZERO' */;
/*!40111 SET @OLD_SQL_NOTES=@@SQL_NOTES, SQL_NOTES=0 */;

--
-- Table structure for table `django_evolution`
--

DROP TABLE IF EXISTS `django_evolution`;
/*!40101 SET @saved_cs_client = @@character_set_client */;
/*!40101 SET character_set_client = utf8 */;
CREATE TABLE `django_evolution` (
  `id` int(11) NOT NULL AUTO_INCREMENT,
  `version_id` int(11) NOT NULL,
  `app_label` varchar(200) NOT NULL,
  `label` varchar(100) NOT NULL,
  PRIMARY KEY (`id`),
  KEY `django_evolution_f516c2b3` (`version_id`),
  CONSTRAINT `version_id_refs_id_946f97c5` FOREIGN KEY (`version_id`) REFERENCES `django_project_version` (`id`)
) ENGINE=InnoDB AUTO_INCREMENT=77 DEFAULT CHARSET=latin1;
/*!40101 SET character_set_client = @saved_cs_client */;

--
-- Dumping data for table `django_evolution`
--

LOCK TABLES `django_evolution` WRITE;
/*!4 ALTER TABLE `django_evolution` DISABLE KEYS */;
INSERT INTO `django_evolution` VALUES (1,1,'auth','auth_delete_message'),(2,1,'auth','auth_unique_together_baseline'),(3,1,'contenttypes','contenttypes_unique_together_baseline'),(4,1,'sessions','session_e

Problems upgrading to 2.0.13 and 2.0.15

[Daniel Dominguez]

When I performed the upgrade form 2.0.12 to 2.0.13, the site-upgrade 
failed.  It didn't seem to break our reviewboard instance, so I just 
ignored it at the time.  I recently upgraded to 2.0.15, and the result is 
the same.  Again, our reviewboard server seems to be fine, but this is 
starting to make me nervous.


--  --
Creating tables ...
Project signature has changed - an evolution is required
Installing custom SQL ...
Installing indexes ...
Installed 0 object(s) from 0 fixture(s)
The stored evolutions do not completely resolve all model changes.

Run `./manage.py evolve --hint` to see a suggestion for the changes 
required.

The following are the changes that could not be resolved:
In model reviews.ReviewRequest:
Field 'description_rich_text' has been added
Field 'testing_done_rich_text' has been added
In model reviews.Review:
Field 'body_top_rich_text' has been added
Field 'body_bottom_rich_text' has been added
In model reviews.ReviewRequestDraft:
Field 'description_rich_text' has been added
Field 'testing_done_rich_text' has been added
In model accounts.Profile:
Field 'default_use_rich_text' has been added
CommandError: Your models contain changes that Django Evolution cannot 
resolve automatically.


I've trying running evolve -- --hint --execute, but that fails with the 
following

CommandError: Error applying evolution: (1060, "Duplicate column name 
'default_use_rich_text'")



I've checked that I have the latest django_evolution installed for my 
platform (0.7.4).  This is running on a MySql database.

It seems like the evolution is trying to add columns that already exist. 
 Maybe a version table didn't get incremented in the prior evolution?  Any 
hints on how I can troubleshoot this?



-- 
Supercharge your Review Board with Power Pack: 
https://www.reviewboard.org/powerpack/
Want us to host Review Board for you? Check out RBCommons: 
https://rbcommons.com/
Happy user? Let us know! https://www.reviewboard.org/users/
--- 
You received this message because you are subscribed to the Google Groups 
"reviewboard" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

Re: Review Board version mismatch

[Daniel Bansch]

I used easy_install.

On Friday, February 20, 2015 at 10:45:53 AM UTC-6, Daniel Bansch wrote:
>
>I'm trying to install review board on redhat with apache, mod_wsgi and 
> mysql.  I get this error when I go to my review board site:
>
>
> ---
>
> Manual server updates required
>
> A recent upgrade requires manual updates to be made on this server. After 
> these changes are made, you should restart your web server. 
>
> If you have any questions or problems, please contact us on our mailing 
> list <http://groups.google.com/group/reviewboard>. 
> Review Board version mismatch 
>  
> The version of Review Board running does not match the version the site 
> was last upgraded to. You are running *2.0.13* and the site was last 
> upgraded to *2.0.12*. 
>
> Please upgrade your site to fix this by running: 
>
> $ rb-site upgrade /var/www/mysite
> ---
>
> I've run the "rb-site upgrade /var/www/mysite" and restarted everything, but 
> the same error pops up.
>
> I have ReviewBoard-2.0.12-py2.7.egg and ReviewBoard-2.0.13-py2.7.egg in my 
> python site-packages directory.
>
> Does anyone have any suggestions?
>
> Thanks,
> Daniel
>
>

-- 
Supercharge your Review Board with Power Pack: 
https://www.reviewboard.org/powerpack/
Want us to host Review Board for you? Check out RBCommons: 
https://rbcommons.com/
Happy user? Let us know! https://www.reviewboard.org/users/
--- 
You received this message because you are subscribed to the Google Groups 
"reviewboard" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

Re: Review Board version mismatch

[Daniel Bansch]

I was able to get it to run by changing the directory name of 
ReviewBoard-2.0.13-py2.7.egg 
to ReviewBoard-2.0.12-py2.7.egg.

On Friday, February 20, 2015 at 10:45:53 AM UTC-6, Daniel Bansch wrote:
>
>I'm trying to install review board on redhat with apache, mod_wsgi and 
> mysql.  I get this error when I go to my review board site:
>
>
> ---
>
> Manual server updates required
>
> A recent upgrade requires manual updates to be made on this server. After 
> these changes are made, you should restart your web server. 
>
> If you have any questions or problems, please contact us on our mailing 
> list <http://groups.google.com/group/reviewboard>. 
> Review Board version mismatch 
>  
> The version of Review Board running does not match the version the site 
> was last upgraded to. You are running *2.0.13* and the site was last 
> upgraded to *2.0.12*. 
>
> Please upgrade your site to fix this by running: 
>
> $ rb-site upgrade /var/www/mysite
> ---
>
> I've run the "rb-site upgrade /var/www/mysite" and restarted everything, but 
> the same error pops up.
>
> I have ReviewBoard-2.0.12-py2.7.egg and ReviewBoard-2.0.13-py2.7.egg in my 
> python site-packages directory.
>
> Does anyone have any suggestions?
>
> Thanks,
> Daniel
>
>

-- 
Supercharge your Review Board with Power Pack: 
https://www.reviewboard.org/powerpack/
Want us to host Review Board for you? Check out RBCommons: 
https://rbcommons.com/
Happy user? Let us know! https://www.reviewboard.org/users/
--- 
You received this message because you are subscribed to the Google Groups 
"reviewboard" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

Review Board version mismatch

[Daniel Bansch]

   I'm trying to install review board on redhat with apache, mod_wsgi and 
mysql.  I get this error when I go to my review board site:

---

Manual server updates required

A recent upgrade requires manual updates to be made on this server. After 
these changes are made, you should restart your web server. 

If you have any questions or problems, please contact us on our mailing list 
<http://groups.google.com/group/reviewboard>. 
Review Board version mismatch 
 
The version of Review Board running does not match the version the site was 
last upgraded to. You are running *2.0.13* and the site was last upgraded 
to *2.0.12*. 

Please upgrade your site to fix this by running: 

$ rb-site upgrade /var/www/mysite
---

I've run the "rb-site upgrade /var/www/mysite" and restarted everything, but 
the same error pops up.

I have ReviewBoard-2.0.12-py2.7.egg and ReviewBoard-2.0.13-py2.7.egg in my 
python site-packages directory.

Does anyone have any suggestions?

Thanks,
Daniel

   

-- 
Supercharge your Review Board with Power Pack: 
https://www.reviewboard.org/powerpack/
Want us to host Review Board for you? Check out RBCommons: 
https://rbcommons.com/
Happy user? Let us know! https://www.reviewboard.org/users/
--- 
You received this message because you are subscribed to the Google Groups 
"reviewboard" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

Re: RBtools 0.7 - issue posting a review

[Daniel Laird]

After some investigation we found the following:

# TODO: This can break on systems that don't have the en_US local

# installed (which isn't very many). Ideally in this case, we could

# put something in the config file, but that's not plumbed through to here.

env['LC_ALL'] = 'en_US.UTF-8'

env['LANGUAGE'] = 'en_US.UTF-8'

We can confirm that it does break things!

With the 'en_US.UTF-8' locale installed the problem goes away - is there a 
way to check this locale is installed and error nicely rather than breaking?
Cheers
Dan

On Saturday, 31 January 2015 18:49:02 UTC, David Trowbridge wrote:
>
> Hi Daniel,
>
> Do you think you could add some code to print out the contents of 'log' 
> inside _convert_symbolic_revision before we try to construct the etree?
>
> Thanks,
> -David
>
>
> On Fri Jan 30 2015 at 4:48:37 AM Daniel Laird  > wrote:
>
>> We have a shared Debian server and we have updated RBtools to 0.7.
>>
>> When we try to post a review we get the following:
>>
>> SERVER_NAME> rbt post -d NUM
>> >>> RBTools 0.7
>> >>> Python 2.7.3 (default, Mar 13 2014, 11:03:55) 
>> [GCC 4.7.2]
>> >>> Running on Linux-3.2.0-4-amd64-x86_64-with-debian-7.8
>> >>> Home = /v/home/rad/USERNAME
>> >>> Current directory = /v/space/SERVER_NAME/USERNAME/
>> wc/memory_corruption
>> >>> Checking for a Subversion repository...
>> >>> Running: svn info --non-interactive
>> >>> Running: diff --version
>> >>> repository info: Path: https://SVN_SERVER/svn-restricted/PROJNAME, 
>> Base path: /branches/users/bob/memory_corruption, Supports changesets: 
>> False
>> >>> Making HTTP GET request to http://REVIEWBOARD_SERVER/api/
>> >>> HTTP GET request to http://REVIEWBOARD_SERVER/api/ cannot be cached
>> >>> Making HTTP GET request to http://REVIEWBOARD_SERVER/api/
>> repositories/
>> >>> HTTP GET request to http://REVIEWBOARD_SERVER/api/repositories/ 
>> cannot be cached
>> >>> Making HTTP GET request to http://REVIEWBOARD_SERVER/api/
>> repositories/?max-results=25&start=25
>> >>> HTTP GET request to http://REVIEWBOARD_SERVER/api/
>> repositories/?max-results=25&start=25 cannot be cached
>> >>> Making HTTP GET request to http://REVIEWBOARD_SERVER/api/
>> repositories/1/info/
>> >>> HTTP GET request to http://REVIEWBOARD_SERVER/api/
>> repositories/1/info/ cannot be cached
>> >>> Making HTTP GET request to http://REVIEWBOARD_SERVER/api/
>> repositories/2/info/
>> >>> HTTP GET request to http://REVIEWBOARD_SERVER/api/
>> repositories/2/info/ cannot be cached
>> >>> Making HTTP GET request to http://REVIEWBOARD_SERVER/api/
>> repositories/3/info/
>> >>> HTTP GET request to http://REVIEWBOARD_SERVER/api/
>> repositories/3/info/ cannot be cached
>> >>> Making HTTP GET request to http://REVIEWBOARD_SERVER/api/
>> repositories/4/info/
>> >>> HTTP GET request to http://REVIEWBOARD_SERVER/api/
>> repositories/4/info/ cannot be cached
>> >>> Making HTTP GET request to http://REVIEWBOARD_SERVER/api/
>> repositories/5/info/
>> >>> HTTP GET request to http://REVIEWBOARD_SERVER/api/
>> repositories/5/info/ cannot be cached
>> >>> Making HTTP GET request to http://REVIEWBOARD_SERVER/api/
>> repositories/6/info/
>> >>> HTTP GET request to http://REVIEWBOARD_SERVER/api/
>> repositories/6/info/ cannot be cached
>> >>> Making HTTP GET request to http://REVIEWBOARD_SERVER/api/
>> repositories/7/info/
>> >>> HTTP GET request to http://REVIEWBOARD_SERVER/api/
>> repositories/7/info/ cannot be cached
>> >>> Making HTTP GET request to http://REVIEWBOARD_SERVER/api/
>> repositories/8/info/
>> >>> HTTP GET request to http://REVIEWBOARD_SERVER/api/
>> repositories/8/info/ cannot be cached
>> >>> Making HTTP GET request to http://REVIEWBOARD_SERVER/api/
>> repositories/9/info/
>> >>> HTTP GET request to http://REVIEWBOARD_SERVER/api/
>> repositories/9/info/ cannot be cached
>> >>> Making HTTP GET request to http://REVIEWBOARD_SERVER/api/
>> repositories/10/info/
>> >>> HTTP GET request to http://REVIEWBOARD_SERVER/api/
>> repositories/10/info/ cannot be cached
>> >>> Making HTTP GET request to http://REVIEWBOARD_SERVER/api/
>> repositories/11/info/
>> >>> HTTP GET request to http://REVIEWB

RBtools 0.7 - issue posting a review

[Daniel Laird]

We have a shared Debian server and we have updated RBtools to 0.7.

When we try to post a review we get the following:

SERVER_NAME> rbt post -d NUM
>>> RBTools 0.7
>>> Python 2.7.3 (default, Mar 13 2014, 11:03:55) 
[GCC 4.7.2]
>>> Running on Linux-3.2.0-4-amd64-x86_64-with-debian-7.8
>>> Home = /v/home/rad/USERNAME
>>> Current directory = /v/space/SERVER_NAME/USERNAME/wc/memory_corruption
>>> Checking for a Subversion repository...
>>> Running: svn info --non-interactive
>>> Running: diff --version
>>> repository info: Path: https://SVN_SERVER/svn-restricted/PROJNAME, Base 
path: /branches/users/bob/memory_corruption, Supports changesets: False
>>> Making HTTP GET request to http://REVIEWBOARD_SERVER/api/
>>> HTTP GET request to http://REVIEWBOARD_SERVER/api/ cannot be cached
>>> Making HTTP GET request to http://REVIEWBOARD_SERVER/api/repositories/
>>> HTTP GET request to http://REVIEWBOARD_SERVER/api/repositories/ cannot 
be cached
>>> Making HTTP GET request to 
http://REVIEWBOARD_SERVER/api/repositories/?max-results=25&start=25
>>> HTTP GET request to 
http://REVIEWBOARD_SERVER/api/repositories/?max-results=25&start=25 cannot 
be cached
>>> Making HTTP GET request to 
http://REVIEWBOARD_SERVER/api/repositories/1/info/
>>> HTTP GET request to http://REVIEWBOARD_SERVER/api/repositories/1/info/ 
cannot be cached
>>> Making HTTP GET request to 
http://REVIEWBOARD_SERVER/api/repositories/2/info/
>>> HTTP GET request to http://REVIEWBOARD_SERVER/api/repositories/2/info/ 
cannot be cached
>>> Making HTTP GET request to 
http://REVIEWBOARD_SERVER/api/repositories/3/info/
>>> HTTP GET request to http://REVIEWBOARD_SERVER/api/repositories/3/info/ 
cannot be cached
>>> Making HTTP GET request to 
http://REVIEWBOARD_SERVER/api/repositories/4/info/
>>> HTTP GET request to http://REVIEWBOARD_SERVER/api/repositories/4/info/ 
cannot be cached
>>> Making HTTP GET request to 
http://REVIEWBOARD_SERVER/api/repositories/5/info/
>>> HTTP GET request to http://REVIEWBOARD_SERVER/api/repositories/5/info/ 
cannot be cached
>>> Making HTTP GET request to 
http://REVIEWBOARD_SERVER/api/repositories/6/info/
>>> HTTP GET request to http://REVIEWBOARD_SERVER/api/repositories/6/info/ 
cannot be cached
>>> Making HTTP GET request to 
http://REVIEWBOARD_SERVER/api/repositories/7/info/
>>> HTTP GET request to http://REVIEWBOARD_SERVER/api/repositories/7/info/ 
cannot be cached
>>> Making HTTP GET request to 
http://REVIEWBOARD_SERVER/api/repositories/8/info/
>>> HTTP GET request to http://REVIEWBOARD_SERVER/api/repositories/8/info/ 
cannot be cached
>>> Making HTTP GET request to 
http://REVIEWBOARD_SERVER/api/repositories/9/info/
>>> HTTP GET request to http://REVIEWBOARD_SERVER/api/repositories/9/info/ 
cannot be cached
>>> Making HTTP GET request to 
http://REVIEWBOARD_SERVER/api/repositories/10/info/
>>> HTTP GET request to http://REVIEWBOARD_SERVER/api/repositories/10/info/ 
cannot be cached
>>> Making HTTP GET request to 
http://REVIEWBOARD_SERVER/api/repositories/11/info/
>>> HTTP GET request to http://REVIEWBOARD_SERVER/api/repositories/11/info/ 
cannot be cached
>>> Making HTTP GET request to 
http://REVIEWBOARD_SERVER/api/repositories/12/info/
>>> HTTP GET request to http://REVIEWBOARD_SERVER/api/repositories/12/info/ 
cannot be cached
>>> Making HTTP GET request to 
http://REVIEWBOARD_SERVER/api/repositories/13/info/
>>> HTTP GET request to http://REVIEWBOARD_SERVER/api/repositories/13/info/ 
cannot be cached
>>> Making HTTP GET request to 
http://REVIEWBOARD_SERVER/api/repositories/14/info/
>>> HTTP GET request to http://REVIEWBOARD_SERVER/api/repositories/14/info/ 
cannot be cached
>>> repository info: Path: https://SVN_SERVER/svn-restricted/PROJNAME, Base 
path: /branches/users/bob/memory_corruption, Supports changesets: False
>>> Running: svn log -r REVNUM -l 1 --xml
Traceback (most recent call last):
  File "/v/home/rad/USERNAME/.local/bin/rbt", line 9, in 
load_entry_point('RBTools==0.7', 'console_scripts', 'rbt')()
  File 
"/v/home/rad/USERNAME/.local/lib/python2.7/site-packages/RBTools-0.7-py2.7.egg/rbtools/commands/main.py",
 
line 133, in main
command.run_from_argv([RB_MAIN, command_name] + args)
  File 
"/v/home/rad/USERNAME/.local/lib/python2.7/site-packages/RBTools-0.7-py2.7.egg/rbtools/commands/__init__.py",
 
line 538, in run_from_argv
exit_code = self.main(*args) or 0
  File 
"/v/home/rad/USERNAME/.local/lib/python2.7/site-packages/RBTools-0.7-py2.7.egg/rbtools/commands/post.py",
 
line 618, in main
self.revisions = get_revisions(self.tool, self.cmd_args)
  File 
"/v/home/rad/USERNAME/.local/lib/python2.7/site-packages/RBTools-0.7-py2.7.egg/rbtools/utils/review_request.py",
 
line 73, in get_revisions
revisions = tool.parse_revision_spec(cmd_args)
  File 
"/v/home/rad/USERNAME/.local/lib/python2.7/site-packages/RBTools-0.7-py2.7.egg/rbtools/clients/svn.py",
 
line 128, in parse_revision_spec
revision = self._convert_symbolic_revision(revision)
  File 
"/v/home/rad/USERNAME/.local/lib

Adding a hyperlink to the navbar

[Daniel Cappuccio]

Hopefully this is a simple question with a simple answer as I am very new 
and unexperienced with the code for this web page.

My company uses review board for code reviews and also has a company wiki 
page with the procedures used for code reviews.  I would like to have a 
link to that wiki page appear in the navbar of review board.  (The light 
blue region at the top of the page with other internal links (MyDashboard, 
New Review Request, etc)).  

I found the source html file on the git hub page, but, i cannot find that 
file on the installed and running review board server.  

Is this change even possible?  Is there a process to do this posted 
somewhere else?

Thanks for your patience, understanding, and responses,
Dan

-- 
Supercharge your Review Board with Power Pack: 
https://www.reviewboard.org/powerpack/
Want us to host Review Board for you? Check out RBCommons: 
https://rbcommons.com/
Happy user? Let us know! https://www.reviewboard.org/users/
--- 
You received this message because you are subscribed to the Google Groups 
"reviewboard" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

Email all users......

[Daniel Laird]

All,

Before I try and write some python that queries all users (ideally active) 
and gets their email addresses.  Has anyone else done this and would you be 
willing to share?
I want to be able to announce things like RBtools 0.7 to all active users 
and cant think of another way (other than the MOTD banner).

If you have something I could use/tweak let me know

Dan

-- 
Get the Review Board Power Pack at http://www.reviewboard.org/powerpack/
---
Sign up for Review Board hosting at RBCommons: https://rbcommons.com/
---
Happy user? Let us know at http://www.reviewboard.org/users/
--- 
You received this message because you are subscribed to the Google Groups 
"reviewboard" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

RBTools, SVN and a strange issue

[Daniel Laird]

I am using RBtools 0.6.3 and am trying to post a review to ReviewBoard.
We use SVN as our version control system.

We are not using the .reviewboardrc files but the reviewboard:url property 
as this means less files in the workspace.
For most of our repos this works fine and rbt post creates a review request.

However for 1 of our repos (may be more - not tested all of them) I got the 
following error:
>>> Error data: {u'stat': u'fail', u'repository': u'SVN_SERVER/REPONAME/', 
u'err': {u'msg': u'The repository path specified is not in the list of 
known repositories.', u'code': 206}}
Traceback (most recent call last):
  File "/usr/local/bin/rbt", line 9, in 
load_entry_point('RBTools==0.6.3', 'console_scripts', 'rbt')()
  File 
"/usr/local/lib/python2.7/dist-packages/RBTools-0.6.3-py2.7.egg/rbtools/commands/main.py",
 
line 134, in main
command.run_from_argv([RB_MAIN, command_name] + args)
  File 
"/usr/local/lib/python2.7/dist-packages/RBTools-0.6.3-py2.7.egg/rbtools/commands/__init__.py",
 
line 416, in run_from_argv
exit_code = self.main(*args) or 0
  File 
"/usr/local/lib/python2.7/dist-packages/RBTools-0.6.3-py2.7.egg/rbtools/commands/post.py",
 
line 791, in main
submit_as=self.options.submit_as)
  File 
"/usr/local/lib/python2.7/dist-packages/RBTools-0.6.3-py2.7.egg/rbtools/commands/post.py",
 
line 507, in post_request
raise CommandError("Error creating review request: %s" % e)
rbtools.commands.CommandError: Error creating review request: The 
repository path specified is not in the list of known repositories. (HTTP 
400, API Error 206)

After some investigation I checked the Repo setup in Reviewboard.
I found that for most of my repositories I had:
http://SVN_SERVER/svn/REPONAME

However for the failing one in question I had:
http://SVN_SERVER/svn/REPONAME/

It seems the addition of the trailing '/' stopped rbtools from posting to 
the server.

With the -d option it shows that it finds the Repo in Reviewboard, only 
when it attempts to POST the diff did it fail.
This might be intentional, however if not.  Perhaps it can be made a little 
more forgiving - maybe don't allow trailing '/' when a repo is configured.

Cheers
Dan

-- 
Get the Review Board Power Pack at http://www.reviewboard.org/powerpack/
---
Sign up for Review Board hosting at RBCommons: https://rbcommons.com/
---
Happy user? Let us know at http://www.reviewboard.org/users/
--- 
You received this message because you are subscribed to the Google Groups 
"reviewboard" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

Backup and Restore of Reviewboard 2.0 failing (mysqldump issues)

[Daniel Laird]

All,

Since RB 1.5 I have backed up the RB installation everynight be by using
mysqldump user=x password=y reviewboard > backup.sql

I have also backed up all attachments etc.
I could then restore the data easily with another script.

I used this method to upgrade from 1.7.25 -> 2.0.11 and with the additional 
rb-site upgrade etc, it all worked great.
However, after 2 days I noticed an error on my setup and re-created the 
site and re-imported the data.

At this point it was a RB 2.0.11 backup being restored to a RB2.0.11 
database.
It all appeared to go ok, but when I clicked on any review i got
Error 500, something broke, I also got the following email:

Traceback (most recent call last):

  File 
"/usr/local/lib/python2.7/dist-packages/Django-1.6.8-py2.7.egg/django/core/handlers/base.py",
 
line 112, in get_response
response = wrapped_callback(request, *callback_args, **callback_kwargs)

  File 
"/usr/local/lib/python2.7/dist-packages/ReviewBoard-2.0.11-py2.7.egg/reviewboard/accounts/decorators.py",
 
line 23, in _check
return view_func(*args, **kwargs)

  File 
"/usr/local/lib/python2.7/dist-packages/ReviewBoard-2.0.11-py2.7.egg/reviewboard/site/decorators.py",
 
line 35, in _check
return view_func(request, local_site=local_site, *args, **kwargs)

  File 
"/usr/local/lib/python2.7/dist-packages/ReviewBoard-2.0.11-py2.7.egg/reviewboard/reviews/views.py",
 
line 710, in review_detail
'screenshots': screenshots,

  File 
"/usr/local/lib/python2.7/dist-packages/ReviewBoard-2.0.11-py2.7.egg/reviewboard/reviews/context.py",
 
line 92, in make_review_request_context
upload_diff_form = UploadDiffForm(review_request, request=request)

  File 
"/usr/local/lib/python2.7/dist-packages/ReviewBoard-2.0.11-py2.7.egg/reviewboard/reviews/forms.py",
 
line 95, in __init__
data, *args, **kwargs)

  File 
"/usr/local/lib/python2.7/dist-packages/ReviewBoard-2.0.11-py2.7.egg/reviewboard/diffviewer/forms.py",
 
line 41, in __init__
if self.repository.get_scmtool().get_diffs_use_absolute_paths():

  File 
"/usr/local/lib/python2.7/dist-packages/ReviewBoard-2.0.11-py2.7.egg/reviewboard/scmtools/models.py",
 
line 230, in get_scmtool
return cls(self)

  File 
"/usr/local/lib/python2.7/dist-packages/ReviewBoard-2.0.11-py2.7.egg/reviewboard/scmtools/svn/__init__.py",
 
line 73, in __init__
local_site_name)

  File 
"/usr/local/lib/python2.7/dist-packages/ReviewBoard-2.0.11-py2.7.egg/reviewboard/scmtools/svn/__init__.py",
 
line 415, in build_client
client = Client(config_dir, repopath, username, password)

  File 
"/usr/local/lib/python2.7/dist-packages/ReviewBoard-2.0.11-py2.7.egg/reviewboard/scmtools/svn/pysvn.py",
 
line 41, in __init__
self.client.set_default_password(six.text_type(password))

UnicodeDecodeError: 'ascii' codec can't decode byte 0xd5 in position 0: 
ordinal not in range(128)

Has something changed in the internal data structures?
Should I be using some options like "--default-character-set=utf8" when 
dumping/importing the database?

Hope you can help
Dan

-- 
Get the Review Board Power Pack at http://www.reviewboard.org/powerpack/
---
Sign up for Review Board hosting at RBCommons: https://rbcommons.com/
---
Happy user? Let us know at http://www.reviewboard.org/users/
--- 
You received this message because you are subscribed to the Google Groups 
"reviewboard" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

Re: Reviewboard 1.7 -> Reviewboard 2.0.x update

[Daniel Laird]

HI Christian,

Many thanks for the response, it was as you describe, our custom script 
(which is for following externals) - is meant to strip the URL.  But the 
wildcard match failed sometimes, leaving the full URL in the diff file. 
 Which RB then processed.
On moving the data from old server -> new server (with 2.0.11) - I ran some 
sed scripts to ensure the SVN paths were consistent. 
Now because the BaseUrl matches the file path RB displays it all fine.

I have another issue, but will mail the list about this.

Cheers
Dan

On Friday, 19 December 2014 22:38:42 UTC, Christian Hammond wrote:
>
> Hi Daniel, 
>
> We never intended the base directory to contain a URL. It sounds like the 
> custom script is probably responsible for this. 
>
> Can you confirm whether the base directory's URL exactly matches the 
> configured URL for the repository? 
>
> In the case that it does match, we take the URL as-is, but if it does not 
> match, we normalize and start building a path (by combining the repository 
> path, base directory, and file path in the diff). This was not intended to 
> handle URLs in the path, but it may be the cause of your problems. 
>
> Christian 
>
> --   
> Christian Hammond - chi...@chipx86.com  
> Review Board - https://www.reviewboard.org 
> Beanbag, Inc. - https://www.beanbaginc.com 
>
> -Original Message- 
> From: Daniel Laird > 
> Reply: revie...@googlegroups.com   >> 
> Date: December 19, 2014 at 3:29:19 AM 
> To: revie...@googlegroups.com   >> 
> Cc: chi...@chipx86.com  >> 
> Subject:  Re: Reviewboard 1.7 -> Reviewboard 2.0.x update 
>
> > Christian 
> >   
> > Many thanks for getting back to me. 
> > I think I know why it is breaking but don't know how to fix it. 
> >   
> > The diff looks as I would expect (with all files being something like 
> this): 
> > Index: libs/A/Source/File.cpp 
> > === 
> > --- libs/A/Source/File.cpp (revision 143446) 
> > +++ libs/A/Source/File.cpp (working copy) 
> >   
> > I used the webapi to query the Review and spotted the following on the 
> diff 
> > resource associated with the review request: 
> > http://SVN_SERVER/svn/REPO_NAME/branches/features/software   
> >   
> > I suspect that maybe in the old GUI (Reviewboard 1.7) it used to combine 
> > this basedir and the file path in the diff to draw the differences. 
> > I think that maybe we have a problem where it appends the Repo Path + 
> > BaseDir + FilePath 
> > Maybe there is a difference in RB2.0.x? 
> >   
> > Our databases on legacy SW used a lot of SVN externals, so someone wrote 
> a 
> > script to generate a single diff file relative to the SVN_SERVER root. 
> > It allowed us a single review which was better, but it probably set the 
> > Basedir to be the complete http path. 
> > Maybe it was not supposed to work that way, but we just got away with 
> it. 
> >   
> > Interested to see what you think and whether there is a regression or a 
> > misuse by us that has been cleared up in RB2.0, the only problem now is 
> we 
> > have 100s of requests that are like this 
> >   
> > Hope you can help, and happy to try patches. 
> > Cheers 
> > Dan 
> >   
> >   
> > On Thursday, 18 December 2014 20:40:16 UTC, Christian Hammond wrote: 
> > > 
> > > Hi Dan, 
> > > 
> > > Would you be able to provide some of the data from that diff 
> > > (specifically, the 'Index:' line, '---', and '+++' lines in the diff)? 
> > > 
> > > A lot of work was done on Subversion since 1.7.x, so something may 
> have 
> > > regressed. I don't recall being able to handle file paths containing 
> > > 'http:' in it, though. 
> > > 
> > > Christian 
> > > 
> > > -- 
> > > Christian Hammond - chi...@chipx86.com   
> > > Review Board - http://www.reviewboard.org 
> > > Beanbag, Inc. - http://www.beanbaginc.com 
> > > 
> > > On Thu, Dec 18, 2014 at 4:32 AM, Daniel Laird > > > wrote: 
> > >> 
> > >> To follow up the error message we get from the diff viewer is: 
> > >> The file '/svn/Common_PythonTests/ 
> > >> http://SVN_SERVER_NAME/svn/Common_PythonTests/path/to/file/base.py' 
> > >> (r1838) could not be found in the repository: 
> > >> 
> '/svn/Common_PythonTests/!svn/rvr/1838/http:/SVN_SERVER/svn/Common_PythonTests/path/to/file/base.py'
>  
>   
> > >> path no

Re: Reviewboard 1.7 -> Reviewboard 2.0.x update

[Daniel Laird]

Christian

Many thanks for getting back to me.
I think I know why it is breaking but don't know how to fix it.

The diff looks as I would expect (with all files being something like this):
Index: libs/A/Source/File.cpp
===
--- libs/A/Source/File.cpp (revision 143446)
+++ libs/A/Source/File.cpp (working copy)

I used the webapi to query the Review and spotted the following on the diff 
resource associated with the review request:
http://SVN_SERVER/svn/REPO_NAME/branches/features/software

I suspect that maybe in the old GUI (Reviewboard 1.7) it used to combine 
this basedir and the file path in the diff to draw the differences.
I think that maybe we have a problem where it appends the Repo Path + 
BaseDir + FilePath
Maybe there is a difference in RB2.0.x?

Our databases on legacy SW used a lot of SVN externals, so someone wrote a 
script to generate a single diff file relative to the SVN_SERVER root.
It allowed us a single review which was better, but it probably set the 
Basedir to be the complete http path.
Maybe it was not supposed to work that way, but we just got away with it.

Interested to see what you think and whether there is a regression or a 
misuse by us that has been cleared up in RB2.0, the only problem now is we 
have 100s of requests that are like this

Hope you can help, and happy to try patches.
Cheers
Dan


On Thursday, 18 December 2014 20:40:16 UTC, Christian Hammond wrote:
>
> Hi Dan,
>
> Would you be able to provide some of the data from that diff 
> (specifically, the 'Index:' line, '---', and '+++' lines in the diff)?
>
> A lot of work was done on Subversion since 1.7.x, so something may have 
> regressed. I don't recall being able to handle file paths containing 
> 'http:' in it, though.
>
> Christian
>
> -- 
> Christian Hammond - chi...@chipx86.com 
> Review Board - http://www.reviewboard.org
> Beanbag, Inc. - http://www.beanbaginc.com
>
> On Thu, Dec 18, 2014 at 4:32 AM, Daniel Laird  > wrote:
>>
>> To follow up the error message we get from the diff viewer is:
>> The file '/svn/Common_PythonTests/
>> http://SVN_SERVER_NAME/svn/Common_PythonTests/path/to/file/base.py' 
>> (r1838) could not be found in the repository: 
>> '/svn/Common_PythonTests/!svn/rvr/1838/http:/SVN_SERVER/svn/Common_PythonTests/path/to/file/base.py'
>>  
>> path not found
>>
>> The review was created before the migration (1.7.25) - we dumped the 
>> database, re-imported and off we went.
>> New requests seem to be ok and many of the old requests are fine.  This 
>> was probably created via the GUI in 1.7.25 and uploaded a diff file 
>> produced by SVN diff.
>>
>> Cheers
>> Dan 
>>
>> On Thursday, 18 December 2014 11:29:18 UTC, Daniel Laird wrote:
>>>
>>> All,
>>>
>>> We were running Reviewboard 1.7.25 and people used a number of methods 
>>> to create review requests:
>>> svn diff - manually upload
>>> post-review
>>> rbt
>>>
>>> I have just migrated us to Reviewboard 2.0.11 and seem to have hit an 
>>> issue.
>>>
>>> Some of our reviews no longer display it suggests that a file cant be 
>>> found.
>>> And seems to have prepended some sub part of the path before the full 
>>> file path.
>>> So I get something like
>>>
>>> /svn/REPONAME/http://SVN_SERVER/svn/REPONAME/path/to/file.cpp cant be 
>>> found.
>>>
>>> Have we got to change something to make these work?
>>>
>>> I can provide logs offline if that helps?
>>>
>>> Dan
>>>
>>>  -- 
>> Get the Review Board Power Pack at http://www.reviewboard.org/powerpack/
>> ---
>> Sign up for Review Board hosting at RBCommons: https://rbcommons.com/
>> ---
>> Happy user? Let us know at http://www.reviewboard.org/users/
>> --- 
>> You received this message because you are subscribed to the Google Groups 
>> "reviewboard" group.
>> To unsubscribe from this group and stop receiving emails from it, send an 
>> email to reviewboard...@googlegroups.com .
>> For more options, visit https://groups.google.com/d/optout.
>>
>

-- 
Get the Review Board Power Pack at http://www.reviewboard.org/powerpack/
---
Sign up for Review Board hosting at RBCommons: https://rbcommons.com/
---
Happy user? Let us know at http://www.reviewboard.org/users/
--- 
You received this message because you are subscribed to the Google Groups 
"reviewboard" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

Re: Reviewboard 1.7 -> Reviewboard 2.0.x update

[Daniel Laird]

To follow up the error message we get from the diff viewer is:
The file 
'/svn/Common_PythonTests/http://SVN_SERVER_NAME/svn/Common_PythonTests/path/to/file/base.py'
 
(r1838) could not be found in the repository: 
'/svn/Common_PythonTests/!svn/rvr/1838/http:/SVN_SERVER/svn/Common_PythonTests/path/to/file/base.py'
 
path not found

The review was created before the migration (1.7.25) - we dumped the 
database, re-imported and off we went.
New requests seem to be ok and many of the old requests are fine.  This was 
probably created via the GUI in 1.7.25 and uploaded a diff file produced by 
SVN diff.

Cheers
Dan 

On Thursday, 18 December 2014 11:29:18 UTC, Daniel Laird wrote:
>
> All,
>
> We were running Reviewboard 1.7.25 and people used a number of methods to 
> create review requests:
> svn diff - manually upload
> post-review
> rbt
>
> I have just migrated us to Reviewboard 2.0.11 and seem to have hit an 
> issue.
>
> Some of our reviews no longer display it suggests that a file cant be 
> found.
> And seems to have prepended some sub part of the path before the full file 
> path.
> So I get something like
>
> /svn/REPONAME/http://SVN_SERVER/svn/REPONAME/path/to/file.cpp cant be 
> found.
>
> Have we got to change something to make these work?
>
> I can provide logs offline if that helps?
>
> Dan
>
>

-- 
Get the Review Board Power Pack at http://www.reviewboard.org/powerpack/
---
Sign up for Review Board hosting at RBCommons: https://rbcommons.com/
---
Happy user? Let us know at http://www.reviewboard.org/users/
--- 
You received this message because you are subscribed to the Google Groups 
"reviewboard" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

Reviewboard 1.7 -> Reviewboard 2.0.x update

[Daniel Laird]

All,

We were running Reviewboard 1.7.25 and people used a number of methods to 
create review requests:
svn diff - manually upload
post-review
rbt

I have just migrated us to Reviewboard 2.0.11 and seem to have hit an issue.

Some of our reviews no longer display it suggests that a file cant be found.
And seems to have prepended some sub part of the path before the full file 
path.
So I get something like

/svn/REPONAME/http://SVN_SERVER/svn/REPONAME/path/to/file.cpp cant be found.

Have we got to change something to make these work?

I can provide logs offline if that helps?

Dan

-- 
Get the Review Board Power Pack at http://www.reviewboard.org/powerpack/
---
Sign up for Review Board hosting at RBCommons: https://rbcommons.com/
---
Happy user? Let us know at http://www.reviewboard.org/users/
--- 
You received this message because you are subscribed to the Google Groups 
"reviewboard" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

Re: Subversion not updating revisions

[Daniel Dominguez]

Awesome.  Thanks David!

On Wednesday, August 13, 2014 7:19:58 AM UTC-7, Daniel Dominguez wrote:
>
> Hi,
>
> I have a new reviewboard installation (2.0.5), and we've setup a 
> subversion repository.  However, the repository is never synchronized with 
> the server after it is first added.  I can re-add the repo and am able to 
> see the most recent revisions, but after that, the view into the repository 
> becomes stale.  By the way, this is when we use the "New Review Request" 
> link on the web UI.  Not sure yet if this is an issue or not with RBTools. 
>  We are able to successfully use RBTools for pre-commit reviews.
>
> I've enabled logging and inspected both the reviewboard and the apache log 
> files, but nothing interesting shows up.
>
> What is the mechanism to trigger a repository refresh?  Where else can I 
> look for useful troubleshooting information?  I'd love to get my 
> distributed team to adopt reviewboard, but this is a bit of a blocker at 
> the moment.
>
> Any help is appreciated.
>
>
> OS: Debian 7 64-bit
> Python 2.7.3
> Using python-svn (since subvertpy was unable to connect to our subversion 
> repo)
>

-- 
Get the Review Board Power Pack at http://www.reviewboard.org/powerpack/
---
Sign up for Review Board hosting at RBCommons: https://rbcommons.com/
---
Happy user? Let us know at http://www.reviewboard.org/users/
--- 
You received this message because you are subscribed to the Google Groups 
"reviewboard" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

Subversion not updating revisions

[Daniel Dominguez]

Hi,

I have a new reviewboard installation (2.0.5), and we've setup a subversion 
repository.  However, the repository is never synchronized with the server 
after it is first added.  I can re-add the repo and am able to see the most 
recent revisions, but after that, the view into the repository becomes 
stale.  By the way, this is when we use the "New Review Request" link on 
the web UI.  Not sure yet if this is an issue or not with RBTools.  We are 
able to successfully use RBTools for pre-commit reviews.

I've enabled logging and inspected both the reviewboard and the apache log 
files, but nothing interesting shows up.

What is the mechanism to trigger a repository refresh?  Where else can I 
look for useful troubleshooting information?  I'd love to get my 
distributed team to adopt reviewboard, but this is a bit of a blocker at 
the moment.

Any help is appreciated.


OS: Debian 7 64-bit
Python 2.7.3
Using python-svn (since subvertpy was unable to connect to our subversion 
repo)

-- 
Get the Review Board Power Pack at http://www.reviewboard.org/powerpack/
---
Sign up for Review Board hosting at RBCommons: https://rbcommons.com/
---
Happy user? Let us know at http://www.reviewboard.org/users/
--- 
You received this message because you are subscribed to the Google Groups 
"reviewboard" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

TeamForge and Reviewboard

[Daniel Laird]

All,

My company have a TeamForge service running and are using GIT as the 
default SCM.
I have a local Reviewboard server and would like to point it at the GIT 
repo - instead of using the default code review tool of gerrit

I am using Reviewboard 1.7.25 currently
The TeamForge project says that the clone URL is:
git clone ssh://MY_USERNAME@gerritforge.DOMAIN:PORT/dashboard

I have tried to setup a repo using my MY_USERNAME/PASSWORD, I have alos 
tried without.
But I am not having much luck.

Has anyone done this, or got it working or any ideas how I could get this 
working?

Many thanks
Dan

-- 
Get the Review Board Power Pack at http://www.reviewboard.org/powerpack/
---
Sign up for Review Board hosting at RBCommons: https://rbcommons.com/
---
Happy user? Let us know at http://www.reviewboard.org/users/
--- 
You received this message because you are subscribed to the Google Groups 
"reviewboard" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

Error using post-review and Reviewboard 1.7.25

[Daniel Laird]

All,

We are currently using post-review and Reviewboard 1.7.25 (RBtools 0.5.1).
We added a new Repo to Reviewboard and its listed as repository 26.
However when we try to post a new review to Reviewboard we get:

DEBUG:root:repository info: Path: http://server/svn/repo_name, Base path: 
/trunk, Supports changesets: False
 >>> Attempting to create review request on http://server/svn/repo_name for 
None  >>> HTTP POSTing to
http://reviewboard-server/api/review-requests/: 
{'repository': u'http://server/svn/repo_name'}  >>> Got API Error 206 (HTTP 
code 400): The repository path specified is not in the list of known 
repositories  >>> Error data: {u'stat': u'fail', u'repository': 
u'http://server/svn/repo_name', u'err': {u'msg': u'The repository path 
specified is not in the list of known repositories', u'code': 206}}

Any ideas why it sees the repo in the list, generates the diff and then 
when posting it falls over?

Cheers
Dan

-- 
Get the Review Board Power Pack at http://www.reviewboard.org/powerpack/
---
Sign up for Review Board hosting at RBCommons: https://rbcommons.com/
---
Happy user? Let us know at http://www.reviewboard.org/users/
--- 
You received this message because you are subscribed to the Google Groups 
"reviewboard" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

Re: Renaming Repositories

[Daniel Laird]

That sorted it - thank you.
Cheers
Dan

On Tuesday, 13 May 2014 21:49:26 UTC+1, David Trowbridge wrote:
>
> The fix itself is in djblets 1.7.26. If you upgrade to Review Board 1.7.25 
> you'll get the latest djblets along with it.
>
> -David
>
>
> On Tue, May 13, 2014 at 1:01 PM, Daniel Laird 
> 
> > wrote:
>
>> I am running 1.7.18 and having this problem.
>> I could upgrade to 1.7.25 if you think that will fix it?
>>
>> Dan
>>
>> --
>> Get the Review Board Power Pack at http://www.reviewboard.org/powerpack/
>> ---
>> Sign up for Review Board hosting at RBCommons: https://rbcommons.com/
>> ---
>> Happy user? Let us know at http://www.reviewboard.org/users/
>> ---
>> You received this message because you are subscribed to the Google Groups 
>> "reviewboard" group.
>> To unsubscribe from this group and stop receiving emails from it, send an 
>> email to reviewboard...@googlegroups.com .
>> For more options, visit https://groups.google.com/d/optout.
>>
>
>

-- 
Get the Review Board Power Pack at http://www.reviewboard.org/powerpack/
---
Sign up for Review Board hosting at RBCommons: https://rbcommons.com/
---
Happy user? Let us know at http://www.reviewboard.org/users/
--- 
You received this message because you are subscribed to the Google Groups 
"reviewboard" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

Re: Renaming Repositories

[Daniel Laird]

I am running 1.7.18 and having this problem.
I could upgrade to 1.7.25 if you think that will fix it?

Dan

-- 
Get the Review Board Power Pack at http://www.reviewboard.org/powerpack/
---
Sign up for Review Board hosting at RBCommons: https://rbcommons.com/
---
Happy user? Let us know at http://www.reviewboard.org/users/
--- 
You received this message because you are subscribed to the Google Groups 
"reviewboard" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

Re: Renaming Repositories

[Daniel Laird]

We are running 0.7.9
/usr/local/lib/python2.6/dist-packages/Djblets-0.7.9-py2.6.egg

Cheers
Dan

On Tuesday, 13 May 2014 10:07:55 UTC+1, David Trowbridge wrote:
>
> Dan,
>
> What version of djblets is installed?
>
> -David
>
>
> On Tue, May 13, 2014 at 1:28 AM, Daniel Laird 
> 
> > wrote:
>
>> All,
>>
>> I wish to change the name of a repository in Reviewboard (am using it in 
>> a RB extension).
>> However when i change the field and enter the authentication details I 
>> just get a box that says 
>> 'Please correct the error below'
>>
>> However nothing in shown.
>> I have check the logs and nothing is showing here either.
>>
>> Are you able to change a repository name (the name RB gives it)?
>> I am running RB 1.7.18 on Ubuntu Linux
>>
>> Cheers
>> Dan 
>>  
>> -- 
>> Get the Review Board Power Pack at http://www.reviewboard.org/powerpack/
>> ---
>> Sign up for Review Board hosting at RBCommons: https://rbcommons.com/
>> ---
>> Happy user? Let us know at http://www.reviewboard.org/users/
>> --- 
>> You received this message because you are subscribed to the Google Groups 
>> "reviewboard" group.
>> To unsubscribe from this group and stop receiving emails from it, send an 
>> email to reviewboard...@googlegroups.com .
>> For more options, visit https://groups.google.com/d/optout.
>>
>
>

-- 
Get the Review Board Power Pack at http://www.reviewboard.org/powerpack/
---
Sign up for Review Board hosting at RBCommons: https://rbcommons.com/
---
Happy user? Let us know at http://www.reviewboard.org/users/
--- 
You received this message because you are subscribed to the Google Groups 
"reviewboard" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

Renaming Repositories

[Daniel Laird]

All,

I wish to change the name of a repository in Reviewboard (am using it in a 
RB extension).
However when i change the field and enter the authentication details I just 
get a box that says 
'Please correct the error below'

However nothing in shown.
I have check the logs and nothing is showing here either.

Are you able to change a repository name (the name RB gives it)?
I am running RB 1.7.18 on Ubuntu Linux

Cheers
Dan 

-- 
Get the Review Board Power Pack at http://www.reviewboard.org/powerpack/
---
Sign up for Review Board hosting at RBCommons: https://rbcommons.com/
---
Happy user? Let us know at http://www.reviewboard.org/users/
--- 
You received this message because you are subscribed to the Google Groups 
"reviewboard" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

Re: Downgrading Reviewboard 2.0RC2 -> 1.7.25

[Daniel Laird]

My original email was not very clear.
On my backup server I want to remove the 2.0RC2 installation and get a 
1.7.x installation running.
However I removed all the RB2.0RC2 files and then dropped the reviewboard 
table from mysql database.
I then did a clean installation of 1.7.25.
I then imported the data from my old server (which was running 1.7.18) - at 
this point I would have thought I would have a 'clean' installation.
However the rb-site upgrade fails and when I visit the homepage I get the 
errors below.

It is almost as if there are some settings files that are stored someone 
not in my installation directory and not in the mysql database.
Might there be some files related to django evolutions that I have to 
delete?

Dan



On Friday, 9 May 2014 20:27:23 UTC+1, Christian Hammond wrote:
>
> Hi Dan,
>
> There are many reasons why a 2.0 database will not cleanly turn into a 
> 1.7.x database. We stick certain data in new tables/columns that don't 
> exist in 1.7. We've migrated data from those older columns. 
> Django-Evolution has no concept of a downgrade and will likely just break 
> every future attempt to ever upgrade your database again.
>
> You're looking to keep the 2.0 data in 1.7.x? Any chance you can just 
> upgrade to 2.0 RC 3 and stick with that until 2.0 is out (very very soon)?
>
> Christian
>
> -- 
> Christian Hammond - chi...@chipx86.com 
> Review Board - http://www.reviewboard.org
> Beanbag, Inc. - http://www.beanbaginc.com
>
>
> On Fri, May 9, 2014 at 8:04 AM, Daniel Laird 
> 
> > wrote:
>
>> All,
>>
>> Due to various issue my development server has to be re-purposed as the 
>> main server.
>> This has meant I have to revert from 2.0RC2 -> 1.7.25 (the latest stable 
>> version).
>>
>> At this point I usually import the database and media files and all works 
>> ok.
>> However I get the following:
>> Traceback (most recent call last):
>>
>>   File 
>> "/usr/local/lib/python2.6/dist-packages/Django-1.4.12-py2.6.egg/django/core/handlers/base.py",
>>  
>> line 89, in get_response
>> response = middleware_method(request)
>>
>>   File 
>> "/usr/local/lib/python2.6/dist-packages/ReviewBoard-1.7.25-py2.6.egg/reviewboard/accounts/middleware.py",
>>  
>> line 12, in process_request
>> user = Profile.objects.get(user=request.user)
>>
>>   File 
>> "/usr/local/lib/python2.6/dist-packages/Django-1.4.12-py2.6.egg/django/db/models/manager.py",
>>  
>> line 131, in get
>> return self.get_query_set().get(*args, **kwargs)
>>
>>   File 
>> "/usr/local/lib/python2.6/dist-packages/Django-1.4.12-py2.6.egg/django/db/models/query.py",
>>  
>> line 361, in get
>> num = len(clone)
>>
>>   File 
>> "/usr/local/lib/python2.6/dist-packages/Django-1.4.12-py2.6.egg/django/db/models/query.py",
>>  
>> line 85, in __len__
>> self._result_cache = list(self.iterator())
>>
>>   File 
>> "/usr/local/lib/python2.6/dist-packages/Django-1.4.12-py2.6.egg/django/db/models/query.py",
>>  
>> line 291, in iterator
>> for row in compiler.results_iter():
>>
>>   File 
>> "/usr/local/lib/python2.6/dist-packages/Django-1.4.12-py2.6.egg/django/db/models/sql/compiler.py",
>>  
>> line 763, in results_iter
>> for rows in self.execute_sql(MULTI):
>>
>>   File 
>> "/usr/local/lib/python2.6/dist-packages/Django-1.4.12-py2.6.egg/django/db/models/sql/compiler.py",
>>  
>> line 818, in execute_sql
>> cursor.execute(sql, params)
>>
>>   File 
>> "/usr/local/lib/python2.6/dist-packages/Django-1.4.12-py2.6.egg/django/db/backends/mysql/base.py",
>>  
>> line 114, in execute
>> return self.cursor.execute(query, args)
>>
>>   File 
>> "/usr/local/lib/python2.6/dist-packages/MySQL_python-1.2.3-py2.6-linux-x86_64.egg/MySQLdb/cursors.py",
>>  
>> line 174, in execute
>> self.errorhandler(self, exc, value)
>>
>>   File 
>> "/usr/local/lib/python2.6/dist-packages/MySQL_python-1.2.3-py2.6-linux-x86_64.egg/MySQLdb/connections.py",
>>  
>> line 36, in defaulterrorhandler
>> raise errorclass, errorvalue
>>
>> DatabaseError: (1054, "Unknown column 'accounts_profile.show_submitted' 
>> in 'field list'")
>>
>> I thought this could be a left over column (from the 2.0 installation) - 
>> so I dropped the tables and re-installed from scratch.
>> I then checked at my 'live&#

Downgrading Reviewboard 2.0RC2 -> 1.7.25

[Daniel Laird]

All,

Due to various issue my development server has to be re-purposed as the 
main server.
This has meant I have to revert from 2.0RC2 -> 1.7.25 (the latest stable 
version).

At this point I usually import the database and media files and all works 
ok.
However I get the following:
Traceback (most recent call last):

  File 
"/usr/local/lib/python2.6/dist-packages/Django-1.4.12-py2.6.egg/django/core/handlers/base.py",
 
line 89, in get_response
response = middleware_method(request)

  File 
"/usr/local/lib/python2.6/dist-packages/ReviewBoard-1.7.25-py2.6.egg/reviewboard/accounts/middleware.py",
 
line 12, in process_request
user = Profile.objects.get(user=request.user)

  File 
"/usr/local/lib/python2.6/dist-packages/Django-1.4.12-py2.6.egg/django/db/models/manager.py",
 
line 131, in get
return self.get_query_set().get(*args, **kwargs)

  File 
"/usr/local/lib/python2.6/dist-packages/Django-1.4.12-py2.6.egg/django/db/models/query.py",
 
line 361, in get
num = len(clone)

  File 
"/usr/local/lib/python2.6/dist-packages/Django-1.4.12-py2.6.egg/django/db/models/query.py",
 
line 85, in __len__
self._result_cache = list(self.iterator())

  File 
"/usr/local/lib/python2.6/dist-packages/Django-1.4.12-py2.6.egg/django/db/models/query.py",
 
line 291, in iterator
for row in compiler.results_iter():

  File 
"/usr/local/lib/python2.6/dist-packages/Django-1.4.12-py2.6.egg/django/db/models/sql/compiler.py",
 
line 763, in results_iter
for rows in self.execute_sql(MULTI):

  File 
"/usr/local/lib/python2.6/dist-packages/Django-1.4.12-py2.6.egg/django/db/models/sql/compiler.py",
 
line 818, in execute_sql
cursor.execute(sql, params)

  File 
"/usr/local/lib/python2.6/dist-packages/Django-1.4.12-py2.6.egg/django/db/backends/mysql/base.py",
 
line 114, in execute
return self.cursor.execute(query, args)

  File 
"/usr/local/lib/python2.6/dist-packages/MySQL_python-1.2.3-py2.6-linux-x86_64.egg/MySQLdb/cursors.py",
 
line 174, in execute
self.errorhandler(self, exc, value)

  File 
"/usr/local/lib/python2.6/dist-packages/MySQL_python-1.2.3-py2.6-linux-x86_64.egg/MySQLdb/connections.py",
 
line 36, in defaulterrorhandler
raise errorclass, errorvalue

DatabaseError: (1054, "Unknown column 'accounts_profile.show_submitted' in 
'field list'")

I thought this could be a left over column (from the 2.0 installation) - so 
I dropped the tables and re-installed from scratch.
I then checked at my 'live' server is running 1.7.18 whereas this is 
running 1.7.25.
So I then tried an rb-site upgrade and get:
Rebuilding directory structure
Updating database. This may take a while.

The log output below, including warnings and errors,
can be ignored unless upgrade fails.

--  --
Creating tables ...

[!] Unable to execute the manager command syncdb: No module named
db.fields

[!] Unable to execute the manager command evolve: No module named
db.fields
---  ---

Resetting in-database caches.

Upgrade complete!

But still no luck.
I know I am doing something stupid - but can anyone help me spot it
Any help will be much appreciated

Dan

-- 
Get the Review Board Power Pack at http://www.reviewboard.org/powerpack/
---
Sign up for Review Board hosting at RBCommons: https://rbcommons.com/
---
Happy user? Let us know at http://www.reviewboard.org/users/
--- 
You received this message because you are subscribed to the Google Groups 
"reviewboard" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

Re: Using --revision-range with single revision in subversion

[Daniel F]

Perfect - thanks!
I'll try to test when I have some free time.

On Thursday, January 30, 2014 12:05:29 PM UTC-5, Daniel F wrote:
>
> I haven't been able to successfully use *rbt post 
> --revision-range=SINGLE_REVISION*.
> It works fine if I use the syntax *rbt post --revision-range=*
> *OLD_REV:NEW_REV*
> Am I doing something wrong?
>
>
> My steps:
>
>1. Commit some files to svn revision 123.  Note: there are still 
>modified files in my working copy which I do NOT want to review.
>2. I want to review just the files committed in revision 123, so I 
>type:
>*rbt post --revision-range=123*
>This creates a review request with a diff of the *changes in my 
>working copy* instead of showing me a diff of what was committed.  It 
>was as if I posted a pre-commit review by typing: *rbt post* .
>3. Suddenly it works if I type: *rbt post --revision-range=122:123*
>
>
> Using:
> RBTools 0.5.5, ReviewBoard 1.7.21, subversion 1.7.8, python 2.7.5
>

-- 
Get the Review Board Power Pack at http://www.reviewboard.org/powerpack/
---
Sign up for Review Board hosting at RBCommons: https://rbcommons.com/
---
Happy user? Let us know at http://www.reviewboard.org/users/
--- 
You received this message because you are subscribed to the Google Groups 
"reviewboard" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/groups/opt_out.

Using --revision-range with single revision in subversion

[Daniel F]

I haven't been able to successfully use *rbt post 
--revision-range=SINGLE_REVISION*.
It works fine if I use the syntax *rbt post --revision-range=*
*OLD_REV:NEW_REV*
Am I doing something wrong?


My steps:

   1. Commit some files to svn revision 123.  Note: there are still 
   modified files in my working copy which I do NOT want to review.
   2. I want to review just the files committed in revision 123, so I type:
   *rbt post --revision-range=123*
   This creates a review request with a diff of the *changes in my working 
   copy* instead of showing me a diff of what was committed.  It was as if 
   I posted a pre-commit review by typing: *rbt post* .
   3. Suddenly it works if I type: *rbt post --revision-range=122:123*


Using:
RBTools 0.5.5, ReviewBoard 1.7.21, subversion 1.7.8, python 2.7.5

-- 
Get the Review Board Power Pack at http://www.reviewboard.org/powerpack/
---
Sign up for Review Board hosting at RBCommons: https://rbcommons.com/
---
Happy user? Let us know at http://www.reviewboard.org/users/
--- 
You received this message because you are subscribed to the Google Groups 
"reviewboard" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/groups/opt_out.

Word-break making it difficult to read numbers of reviews in dashboard navbar

[Daniel Kan]

In the dashboard navbar, one of my users noticed that if a review group is 
sufficiently long and the group has >=100 reviews, the UI wraps both the 
group name and the number of reviews. Wrapping the group name isn't too big 
a deal. However, wrapping the number of reviews makes it a bit hard to 
read, especially when a user is a member of many review groups. I changed 
word-break from 'break-all' to 'normal' in common.min.css which seemed to 
keep the number from wrapping. By doing this though, am I ignoring any 
design decision that led to 'word-break break-all' from being used 
explicitly?

I'm using 1.7.18 btw... Thanks!!!

-- 
Get the Review Board Power Pack at http://www.reviewboard.org/powerpack/
---
Sign up for Review Board hosting at RBCommons: https://rbcommons.com/
---
Happy user? Let us know at http://www.reviewboard.org/users/
--- 
You received this message because you are subscribed to the Google Groups 
"reviewboard" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/groups/opt_out.

Re: Change in LDAP authentication behavior in 1.7.14 breaks environments that don't allow anonymous searches

[Daniel Kan]

David,
  Sorry for the massive delay! I've finally posted it at 
https://reviews.reviewboard.org/r/4953/.

On Friday, October 18, 2013 1:51:59 AM UTC-7, David Trowbridge wrote:
>
> Daniel,
>
> I'd suggest making a quick clone of the reviewboard repository, making 
> your change, and then posting your diff against the reviewboard repository 
> by either creating a diff manually (git diff --full-index) or using rbt 
> post.
>
> -David
>
>
> On Tue, Oct 15, 2013 at 4:57 PM, Daniel Kan 
> > wrote:
>
>> David,
>>   Sure, although I'm not sure how to post a bare diff there without 
>> specifying a repository. Is that possible? Thanks!
>>
>>
>> On Thursday, September 26, 2013 2:43:43 PM UTC-7, David Trowbridge wrote:
>>
>>> Would you mind submitting your change to http://reviews.reviewboard.org/?
>>>
>>> Thanks!
>>> -David
>>>
>>>
>>> On Tue, Sep 24, 2013 at 7:32 PM, Daniel Kan  wrote:
>>>
>>>> I am using RB 1.7.x with an LDAP authentication backend and it's been 
>>>> working fine up through 1.7.13. With 1.7.14, the authentication mechanism 
>>>> no longer works in my environment. After some digging, it appears that 
>>>> there was a change in reviewboard/accounts/backends.py to search 
>>>> anonymously for the user before attempting to bind. The LDAP server in my 
>>>> environment is set up to not return any results to anonymous queries and 
>>>> so 
>>>> the authentication attempt fails. Is there a reason this was changed? 
>>>> Thanks!!!
>>>>
>>>> In an attempt to preserve the existing behavior of doing an anonymous 
>>>> search but then attempting a bind as a specific user if the anonymous 
>>>> search returns nothing, I made the following change which works for me:
>>>> --- reviewboard/accounts/backends.py.orig   2013-09-24 
>>>> 16:18:47.0 -0700
>>>> +++ reviewboard/accounts/backends.py2013-09-24 16:20:01.0 
>>>> -0700
>>>> @@ -210,7 +210,10 @@
>>>>   search = ldapo.search_s(settings.LDAP_BASE_DN,
>>>>  ldap.SCOPE_SUBTREE,
>>>>  uid)
>>>> -userbinding = search[0][0]
>>>> +   if (len(search) > 0):
>>>> +   userbinding = search[0][0]
>>>> +   else:
>>>> +   userbinding=','.join([uid,
>>>> settings.LDAP_BASE_DN])
>>>>  ldapo.bind_s(userbinding, password)
>>>>  
>>>>  return self.get_or_create_user(username, None, ldapo)
>>>>
>>>> -- 
>>>> Get the Review Board Power Pack at http://www.reviewboard.org/
>>>> powerpack/
>>>> ---
>>>> Sign up for Review Board hosting at RBCommons: https://rbcommons.com/
>>>> ---
>>>> Happy user? Let us know at http://www.reviewboard.org/users/
>>>> --- 
>>>> You received this message because you are subscribed to the Google 
>>>> Groups "reviewboard" group.
>>>> To unsubscribe from this group and stop receiving emails from it, send 
>>>> an email to reviewboard...@googlegroups.com.
>>>>
>>>> For more options, visit https://groups.google.com/groups/opt_out.
>>>>
>>>
>>>  -- 
>> Get the Review Board Power Pack at http://www.reviewboard.org/powerpack/
>> ---
>> Sign up for Review Board hosting at RBCommons: https://rbcommons.com/
>> ---
>> Happy user? Let us know at http://www.reviewboard.org/users/
>> --- 
>> You received this message because you are subscribed to the Google Groups 
>> "reviewboard" group.
>> To unsubscribe from this group and stop receiving emails from it, send an 
>> email to reviewboard...@googlegroups.com .
>> For more options, visit https://groups.google.com/groups/opt_out.
>>
>
>

-- 
Get the Review Board Power Pack at http://www.reviewboard.org/powerpack/
---
Sign up for Review Board hosting at RBCommons: https://rbcommons.com/
---
Happy user? Let us know at http://www.reviewboard.org/users/
--- 
You received this message because you are subscribed to the Google Groups 
"reviewboard" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/groups/opt_out.

Re: Change in LDAP authentication behavior in 1.7.14 breaks environments that don't allow anonymous searches

[Daniel Kan]

David,
  Sure, although I'm not sure how to post a bare diff there without 
specifying a repository. Is that possible? Thanks!

On Thursday, September 26, 2013 2:43:43 PM UTC-7, David Trowbridge wrote:
>
> Would you mind submitting your change to http://reviews.reviewboard.org/ ?
>
> Thanks!
> -David
>
>
> On Tue, Sep 24, 2013 at 7:32 PM, Daniel Kan 
> > wrote:
>
>> I am using RB 1.7.x with an LDAP authentication backend and it's been 
>> working fine up through 1.7.13. With 1.7.14, the authentication mechanism 
>> no longer works in my environment. After some digging, it appears that 
>> there was a change in reviewboard/accounts/backends.py to search 
>> anonymously for the user before attempting to bind. The LDAP server in my 
>> environment is set up to not return any results to anonymous queries and so 
>> the authentication attempt fails. Is there a reason this was changed? 
>> Thanks!!!
>>
>> In an attempt to preserve the existing behavior of doing an anonymous 
>> search but then attempting a bind as a specific user if the anonymous 
>> search returns nothing, I made the following change which works for me:
>> --- reviewboard/accounts/backends.py.orig   2013-09-24 
>> 16:18:47.0 -0700
>> +++ reviewboard/accounts/backends.py2013-09-24 16:20:01.0 
>> -0700
>> @@ -210,7 +210,10 @@
>>  search = ldapo.search_s(settings.LDAP_BASE_DN,
>>  ldap.SCOPE_SUBTREE,
>>  uid)
>> -userbinding = search[0][0]
>> +   if (len(search) > 0):
>> +   userbinding = search[0][0]
>> +   else:
>> +   userbinding=','.join([uid,settings.LDAP_BASE_DN])
>>  ldapo.bind_s(userbinding, password)
>>  
>>  return self.get_or_create_user(username, None, ldapo)
>>
>> -- 
>> Get the Review Board Power Pack at http://www.reviewboard.org/powerpack/
>> ---
>> Sign up for Review Board hosting at RBCommons: https://rbcommons.com/
>> ---
>> Happy user? Let us know at http://www.reviewboard.org/users/
>> --- 
>> You received this message because you are subscribed to the Google Groups 
>> "reviewboard" group.
>> To unsubscribe from this group and stop receiving emails from it, send an 
>> email to reviewboard...@googlegroups.com .
>> For more options, visit https://groups.google.com/groups/opt_out.
>>
>
>

-- 
Get the Review Board Power Pack at http://www.reviewboard.org/powerpack/
---
Sign up for Review Board hosting at RBCommons: https://rbcommons.com/
---
Happy user? Let us know at http://www.reviewboard.org/users/
--- 
You received this message because you are subscribed to the Google Groups 
"reviewboard" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/groups/opt_out.

Change in LDAP authentication behavior in 1.7.14 breaks environments that don't allow anonymous searches

[Daniel Kan]

I am using RB 1.7.x with an LDAP authentication backend and it's been 
working fine up through 1.7.13. With 1.7.14, the authentication mechanism 
no longer works in my environment. After some digging, it appears that 
there was a change in reviewboard/accounts/backends.py to search 
anonymously for the user before attempting to bind. The LDAP server in my 
environment is set up to not return any results to anonymous queries and so 
the authentication attempt fails. Is there a reason this was changed? 
Thanks!!!

In an attempt to preserve the existing behavior of doing an anonymous 
search but then attempting a bind as a specific user if the anonymous 
search returns nothing, I made the following change which works for me:
--- reviewboard/accounts/backends.py.orig   2013-09-24 
16:18:47.0 -0700
+++ reviewboard/accounts/backends.py2013-09-24 16:20:01.0 -0700
@@ -210,7 +210,10 @@
 search = ldapo.search_s(settings.LDAP_BASE_DN,
 ldap.SCOPE_SUBTREE,
 uid)
-userbinding = search[0][0]
+   if (len(search) > 0):
+   userbinding = search[0][0]
+   else:
+   userbinding=','.join([uid,settings.LDAP_BASE_DN])
 ldapo.bind_s(userbinding, password)
 
 return self.get_or_create_user(username, None, ldapo)

-- 
Get the Review Board Power Pack at http://www.reviewboard.org/powerpack/
---
Sign up for Review Board hosting at RBCommons: https://rbcommons.com/
---
Happy user? Let us know at http://www.reviewboard.org/users/
--- 
You received this message because you are subscribed to the Google Groups 
"reviewboard" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/groups/opt_out.

Error with home directory path expansion when using rbt?

[Daniel Kan]

New user to ReviewBoard here. Great tool with an intuitive interface, 
thanks for your efforts! Recently, one of my users was using 'rbt' from 
RBTools 0.5.1 and noticed that it would not accept '~' in the value for 
--diff-filename. My guess was that rbt does not run os.path.expanduser on 
the value, so I modified rbtools/commands/post.py with the following:
--- orig/rbtools/commands/post.py   2013-05-30 00:36:44.0 -0700
+++ RBTools-0.5.1.1-py2.7.egg/rbtools/commands/post.py  2013-06-11 
15:33:05.449908000 -0700
@@ -455,8 +455,11 @@
 diff = sys.stdin.read()
 else:
 try:
-diff_path = os.path.join(origcwd,
- self.options.diff_filename)
+if (self.options.diff_filename.startswith('~')):
+   diff_path = 
os.path.expanduser(self.options.diff_filename)
+else:
+diff_path = os.path.join(origcwd,
+ 
self.options.diff_filename)
 fp = open(diff_path, 'r')
 diff = fp.read()
 fp.close()

My question is, is this sufficient to fix the problem or are there more 
places where I need to run expanduser? Also, although debug output 
indicates that it is unable to open the file, running rbt in normal mode is 
silent about not being able to find the difffile. 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
--- 
You received this message because you are subscribed to the Google Groups 
"reviewboard" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/groups/opt_out.

Re: How can I remove an extension?

[Daniel Kelly]

No that's not it.  I've rebooted this box (  a VM ) multiple times since
the problem started happening.


On Mon, Jun 10, 2013 at 8:12 PM, Christian Hammond wrote:

> On Jun 10, 2013, at 1:07 PM, Daniel Kelly 
> wrote:
>
>
>> Extensions should be disabled from within the API before uninstalling
>> from the filesystem. We should make this more bullet-proof though, maybe
>> disable if it's not found.
>>
>> Are you still able to use RB?
>>
>
> RB works fine
>
>
>>  Is it just spewing logs?
>>
>
> Yes
>
>
>> If so, go into the admin UI -> Database -> Extensions, find the ascii art
>> extension, and disable it from there.
>>
>
> It's not listed.
> According to that, there are no extensions registered
>
>
> Try reloading Apache. There may just be other threads that still think it
> should be there.
>
> --
> Christian Hammond - chip...@chipx86.com
> Review Board - http://www.reviewboard.org
> Beanbag, Inc. - http://www.beanbaginc.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
> ---
> You received this message because you are subscribed to the Google Groups
> "reviewboard" group.
> To unsubscribe from this group and stop receiving emails from it, send an
> email to reviewboard+unsubscr...@googlegroups.com.
> For more options, visit https://groups.google.com/groups/opt_out.
>
>
>

-- 
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
--- 
You received this message because you are subscribed to the Google Groups 
"reviewboard" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/groups/opt_out.

Re: How can I remove an extension?

[Daniel Kelly]

>
>
> Extensions should be disabled from within the API before uninstalling from
> the filesystem. We should make this more bullet-proof though, maybe disable
> if it's not found.
>
> Are you still able to use RB?
>

RB works fine


> Is it just spewing logs?
>

Yes


> If so, go into the admin UI -> Database -> Extensions, find the ascii art
> extension, and disable it from there.
>

It's not listed.
According to that, there are no extensions registered


On Jun 10, 2013, at 11:50 AM, Daniel Kelly 
> wrote:
>
> I tried and failed to install the ascii art shipit extension.
>
> 2013-06-10 18:41:31,649 - ERROR -  - Error loading extension
> shipit_ascii_art_extension: No module named
> shipit_ascii_art_extension.extension
> 2013-06-10 18:41:36,053 - ERROR -  - Error loading extension
> shipit_ascii_art_extension: No module named
> shipit_ascii_art_extension.extension
> 2013-06-10 18:42:51,401 - ERROR -  - Error loading extension
> shipit_ascii_art_extension: No module named
> shipit_ascii_art_extension.extension
> 2013-06-10 18:43:53,995 - ERROR -  - Error loading extension
> shipit_ascii_art_extension: No module named
> shipit_ascii_art_extension.extension
> 2013-06-10 18:44:15,601 - ERROR -  - Error loading extension
> shipit_ascii_art_extension: No module named
> shipit_ascii_art_extension.extension
> 2013-06-10 18:46:30,115 - ERROR -  - Error loading extension
> shipit_ascii_art_extension: No module named
> shipit_ascii_art_extension.extension
> 2013-06-10 18:46:31,594 - ERROR -  - Error loading extension
> shipit_ascii_art_extension: No module named
> shipit_ascii_art_extension.extension
> 2013-06-10 18:46:36,524 - ERROR -  - Error loading extension
> shipit_ascii_art_extension: No module named
> shipit_ascii_art_extension.extension
> 2013-06-10 18:47:52,215 - ERROR -  - Error loading extension
> shipit_ascii_art_extension: No module named
> shipit_ascii_art_extension.extension
> 2013-06-10 18:48:54,037 - ERROR -  - Error loading extension
> shipit_ascii_art_extension: No module named
> shipit_ascii_art_extension.extension
>
> any idea how to remove it?
>
>
> --
> 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
> ---
> You received this message because you are subscribed to the Google Groups
> "reviewboard" group.
> To unsubscribe from this group and stop receiving emails from it, send an
> email to reviewboard+unsubscr...@googlegroups.com.
> For more options, visit https://groups.google.com/groups/opt_out.
>
>
>
>
>  --
> 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
> ---
> You received this message because you are subscribed to the Google Groups
> "reviewboard" group.
> To unsubscribe from this group and stop receiving emails from it, send an
> email to reviewboard+unsubscr...@googlegroups.com.
> For more options, visit https://groups.google.com/groups/opt_out.
>
>
>

-- 
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
--- 
You received this message because you are subscribed to the Google Groups 
"reviewboard" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/groups/opt_out.

How can I remove an extension?

[Daniel Kelly]

I tried and failed to install the ascii art shipit extension.

2013-06-10 18:41:31,649 - ERROR -  - Error loading extension
shipit_ascii_art_extension: No module named
shipit_ascii_art_extension.extension
2013-06-10 18:41:36,053 - ERROR -  - Error loading extension
shipit_ascii_art_extension: No module named
shipit_ascii_art_extension.extension
2013-06-10 18:42:51,401 - ERROR -  - Error loading extension
shipit_ascii_art_extension: No module named
shipit_ascii_art_extension.extension
2013-06-10 18:43:53,995 - ERROR -  - Error loading extension
shipit_ascii_art_extension: No module named
shipit_ascii_art_extension.extension
2013-06-10 18:44:15,601 - ERROR -  - Error loading extension
shipit_ascii_art_extension: No module named
shipit_ascii_art_extension.extension
2013-06-10 18:46:30,115 - ERROR -  - Error loading extension
shipit_ascii_art_extension: No module named
shipit_ascii_art_extension.extension
2013-06-10 18:46:31,594 - ERROR -  - Error loading extension
shipit_ascii_art_extension: No module named
shipit_ascii_art_extension.extension
2013-06-10 18:46:36,524 - ERROR -  - Error loading extension
shipit_ascii_art_extension: No module named
shipit_ascii_art_extension.extension
2013-06-10 18:47:52,215 - ERROR -  - Error loading extension
shipit_ascii_art_extension: No module named
shipit_ascii_art_extension.extension
2013-06-10 18:48:54,037 - ERROR -  - Error loading extension
shipit_ascii_art_extension: No module named
shipit_ascii_art_extension.extension

any idea how to remove it?

-- 
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
--- 
You received this message because you are subscribed to the Google Groups 
"reviewboard" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/groups/opt_out.

myisam or innodb ?

[Daniel Kelly]

This is probably a Django issue, but if I'm using mysql, should my tables 
be in innodb format or myisam?

-- 
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
--- 
You received this message because you are subscribed to the Google Groups 
"reviewboard" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/groups/opt_out.

Re: Chrome Extension: Review Board Notifier

[Daniel Laird]

I am logged into Reviewboard via Chome but still getting the spinning icon 
- is there a way to find out which cookie file its using?

Dan

On Monday, April 22, 2013 9:17:00 PM UTC+1, Kirollos Risk wrote:
>
> Ah, once installed and opened, it uses the the appropriate Review Board 
> cookie.
>
> If it spins indefinitely, it may mean that you haven't logged in to Review 
> Board, thus the authentication cookie isn't set.  
>
> Currently, you *cannot* log in via the extension itself (I am working on 
> that).
>
> - Kiro
>
>
> On Monday, 22 April 2013 08:56:14 UTC-7, Daniel Laird wrote:
>>
>> I have downloaded and installed it - I have then pointed it at my company 
>> server.
>> But it just spins - with the busy icon.
>>
>> Does it require anonymous access or anything as it does not seem to be 
>> asking for username/password?
>>
>> Dan
>>
>> On Saturday, April 20, 2013 12:15:31 AM UTC+1, Kirollos Risk wrote:
>>>
>>> Hey everyone!
>>>
>>> Here's a Chrome extension, *Review Board Notifie*r:
>>>
>>> *Features:*
>>>
>>>- Quick glance on an review board requests assigned to you, whether 
>>>they are waiting for *ship it*, or have been given *ship it*
>>>- Click on an item to quickly get to the details
>>>- Notifies you when there are new RB requests assigned to you
>>>- Configurable via the settings page
>>>
>>>
>>> *Screenshots:*
>>>
>>>
>>>
>>> <https://lh5.googleusercontent.com/-JIBwdna5Pso/UXHMoITr7eI/G0o/3QNhzGqN1po/s1600/Screen+Shot+2013-04-19+at+3.54.03+PM.png>
>>>
>>>
>>>
>>>
>>>
>>>
>>>
>>>
>>>
>>>
>>>
>>>
>>>
>>>
>>>
>>>
>>>
>>>
>>>
>>>
>>>
>>>
>>>
>>>
>>>
>>>
>>>
>>>
>>>
>>>
>>>
>>>
>>>
>>>
>>>
>>>
>>>
>>>
>>>
>>>
>>>
>>>
>>>
>>>
>>>
>>>
>>> <https://lh4.googleusercontent.com/-Lz-NmhJCzPA/UXHNSGlCA7I/G04/fB8AWAAchRU/s1600/Screen+Shot+2013-04-19+at+4.03.15+PM.png>
>>>
>>>
>>>
>>>
>>>
>>>
>>>
>>>
>>>
>>>
>>>
>>>
>>>
>>>
>>>
>>>
>>>
>>>
>>>
>>>
>>>
>>>
>>>
>>>
>>>
>>>
>>>
>>>
>>>
>>>
>>>
>>>
>>>
>>>
>>>
>>>
>>>
>>>
>>>
>>>
>>>
>>>
>>> You can install it for free, via https://github.com/krisk/rbn
>>>
>>> - Kiro
>>>
>>>

-- 
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
--- 
You received this message because you are subscribed to the Google Groups 
"reviewboard" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/groups/opt_out.

Re: Chrome Extension: Review Board Notifier

[Daniel Laird]

I have downloaded and installed it - I have then pointed it at my company 
server.
But it just spins - with the busy icon.

Does it require anonymous access or anything as it does not seem to be 
asking for username/password?

Dan

On Saturday, April 20, 2013 12:15:31 AM UTC+1, Kirollos Risk wrote:
>
> Hey everyone!
>
> Here's a Chrome extension, *Review Board Notifie*r:
>
> *Features:*
>
>- Quick glance on an review board requests assigned to you, whether 
>they are waiting for *ship it*, or have been given *ship it*
>- Click on an item to quickly get to the details
>- Notifies you when there are new RB requests assigned to you
>- Configurable via the settings page
>
>
> *Screenshots:*
>
>
>
> 
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
> 
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
> You can install it for free, via https://github.com/krisk/rbn
>
> - Kiro
>
>

-- 
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
--- 
You received this message because you are subscribed to the Google Groups 
"reviewboard" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/groups/opt_out.

rb-extension-pack - installation and usage?

[Daniel Laird]

I just found this pack of extensions and saw the XML Review Extension.
However there is little explaining what this does or how it enhances the 
process of reviewing XML.
Can anyone tell me what it adds? I thought I had installed it properly and 
then looked at a review with XML files and noted nothing different.

Cheers
Dan


-- 
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: Review Board 1.7 RC 1 is released!

[Daniel Laird]

Very exciting - I look forward to getting this installed.
One question I have is about RBTools?

I am using ReviewBot and that depends on the API branch of RBtools.
Do you know if the API branch is going to be merged / released at the same 
time?

Also I understand there was a plan for a Python API for 1.7 (as well as the 
RESTful API) - is that still planned or has that slipped to 1.8 (or later)

Cheers
Dan

On Thursday, November 29, 2012 10:08:07 PM UTC, Christian Hammond wrote:
>
> Review Board "Better Late Than Never" 1.7 RC 1 is out. We had hoped to get 
> this out earlier, but hit some problems and annoyances during some 
> production use that we felt should be fixed first. So trust me, it was 
> worth the wait :)
>
> There's a lot of fixes, enhancements, and some small extension interface 
> changes in this release.
>
> See the news and release notes for more information.
>
> http://www.reviewboard.org/news/2012/11/29/review-board-1-7-rc-1-released/
>
> We'd like to ask those who plan to upgrade to 1.7 to please give it a try. 
> Aside from major fixes, we are hoping there won't be many changes made 
> until the final 1.7, so now's the time to report any regressions you hit.
>
> Christian
>
> -- 
> Christian Hammond - chi...@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

SVN 1.7 support

[Daniel Laird]

Has anyone managed to patch or modify reviewboard so that diffs generated 
under SVN 1.7 actually work with post-review and ReviewBoard?

The upgrade to SVN is about to hit my department and it will be a shame to 
stop reviewing code as a result of this change

Dan

-- 
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: Review Board 1.7 beta 1 released

[Daniel Laird]

I checked permissions - it turned out that ext had permissions of 755 and 
infact needs 775 as the user is in group www-data but the www-data needs w 
access to ext.
Perhaps that could be added to the debug message to help other people who 
make the same mistake as me.

Cheers
Dan

On Friday, August 3, 2012 7:49:01 PM UTC+1, Christian Hammond wrote:
>
> Hi Daniel,
>
> The CACHE_* warnings you get are harmless. Just new APIs in Django we have 
> to migrate to.
>
> We can't detect the config file location. The user may be using our 
> auto-generated ones, or they may have copied/pasted into their Apache 
> configuration. There's no way for us to know.
>
> So I wasn't completely sure, but did you create the ext directory and set 
> the ownership on it? Can you show me a ls -ld of that directory? That 
> should be the last thing you'd need to configure.
>
> Christian
>
> -- 
> Christian Hammond - chip...@chipx86.com
> Review Board - http://www.reviewboard.org
> VMware, Inc. - http://www.vmware.com
>
>
> On Fri, Aug 3, 2012 at 6:39 AM, Daniel Laird <
> daniel.j.la...@googlemail.com> wrote:
>
>> I have tried to install this release.
>> I had to manually installed Djblets 0.7 and then I could install by 
>> specifying -f to easy_install.
>> Onc done I then did 
>> rb-site upgrade /path/my/site
>>
>> This took a while and generated the following:
>> Upgrading Review Board from 1.6.11 to 1.7 beta 1
>> There are unapplied evolutions for auth.
>> There are unapplied evolutions for accounts.
>> There are unapplied evolutions for diffviewer.
>> Adding baseline version for new models
>> Project signature has changed - an evolution is required
>> Installing custom SQL ...
>> Installing indexes ...
>> Installed 0 object(s) from 0 fixture(s)
>>
>> Evolution successful.
>> Resetting in-database caches.
>> Upgrade complete.
>>
>> The location of static media files (CSS, JavaScript, images)
>> has changed. You will need to make manual changes to 
>> your web server configuration.
>>
>> For Apache, you will need to add:
>>
>> Alias /static "/var/www/reviewboard-satv.tandbergtv.lan/htdocs/static"
>>
>> For lighttpd, add the following to alias.url:
>>
>> "/static" => "/var/www/reviewboard-satv.tandbergtv.lan/htdocs/static"
>>
>> Once you have made these changes, type the following 
>> to resolve this:
>>
>> $ rb-site manage /var/www/reviewboard-satv.tandbergtv.lan 
>> resolve-check static-media
>>
>> I think the help could tell you which conf file to update with the Alias. 
>>  
>> However did that and then ran the command specified.
>>
>> I then restarted memcached and apache.
>> However I now get a splash screen that says:
>>
>> *Manual server updates required
>>> **A recent upgrade requires manual updates to be made on this server. 
>>> After these changes are made, you should restart your server.
>>> **If you have any questions or problems, please contact us on our 
>>> mailing list.*
>>
>>
>> But I have done this:
>> It also says:
>>
>> *Your site's extensions media directory isn't properly set up. This 
>>> directory is where Review Board will store various extension media files.
>>> **Your extensions media directory is currently at: 
>>> /var/www/reviewboard-satv.tandbergtv.lan/htdocs/media/ext
>>> **Permission problems
>>> **The directory must be writable by the web server. On Linux/Unix/Mac, 
>>> you can fix this by typing:
>>> **$ sudo chown -R www-data 
>>> "/var/www/reviewboard-satv.tandbergtv.lan/htdocs/media/ext"
>>> **On Windows, right-click the data directory and change the ownership 
>>> to www-data.*
>>
>>  
>> But I have also checked this.
>> Have you got any logging I can try and turn on to debug further?
>> Finally I am also getting a lot of these when running any rb-site 
>> commands:
>>
>>> */usr/local/lib/python2.6/dist-packages/Django-1.4.1-py2.6.egg/django/core/cache/__init__.py:82:
>>>  
>>> DeprecationWarning: settings.CACHE_* is deprecated; use settings.CACHES 
>>> instead.
>>> **  DeprecationWarning*
>>
>>
>> I am running on Ubuntu 10.04 64 bit Server.
>>
>> Cheers
>> Dan
>>
>>
>> On Friday, August 3, 2012 9:49:58 AM UTC+1, Christian Hammond wrote:
>>>
>>> Hey everyone,
>>>
>>> I'm happy to announce that we've pu

Re: Review Board 1.7 beta 1 released

[Daniel Laird]

I have tried to install this release.
I had to manually installed Djblets 0.7 and then I could install by 
specifying -f to easy_install.
Onc done I then did 
rb-site upgrade /path/my/site

This took a while and generated the following:
Upgrading Review Board from 1.6.11 to 1.7 beta 1
There are unapplied evolutions for auth.
There are unapplied evolutions for accounts.
There are unapplied evolutions for diffviewer.
Adding baseline version for new models
Project signature has changed - an evolution is required
Installing custom SQL ...
Installing indexes ...
Installed 0 object(s) from 0 fixture(s)

Evolution successful.
Resetting in-database caches.
Upgrade complete.

The location of static media files (CSS, JavaScript, images)
has changed. You will need to make manual changes to 
your web server configuration.

For Apache, you will need to add:

Alias /static "/var/www/reviewboard-satv.tandbergtv.lan/htdocs/static"

For lighttpd, add the following to alias.url:

"/static" => "/var/www/reviewboard-satv.tandbergtv.lan/htdocs/static"

Once you have made these changes, type the following 
to resolve this:

$ rb-site manage /var/www/reviewboard-satv.tandbergtv.lan resolve-check 
static-media

I think the help could tell you which conf file to update with the Alias.  
However did that and then ran the command specified.

I then restarted memcached and apache.
However I now get a splash screen that says:

*Manual server updates required
> **A recent upgrade requires manual updates to be made on this server. 
> After these changes are made, you should restart your server.
> **If you have any questions or problems, please contact us on our mailing 
> list.*


But I have done this:
It also says:

*Your site's extensions media directory isn't properly set up. This 
> directory is where Review Board will store various extension media files.
> **Your extensions media directory is currently at: 
> /var/www/reviewboard-satv.tandbergtv.lan/htdocs/media/ext
> **Permission problems
> **The directory must be writable by the web server. On Linux/Unix/Mac, 
> you can fix this by typing:
> **$ sudo chown -R www-data 
> "/var/www/reviewboard-satv.tandbergtv.lan/htdocs/media/ext"
> **On Windows, right-click the data directory and change the ownership to 
> www-data.*

 
But I have also checked this.
Have you got any logging I can try and turn on to debug further?
Finally I am also getting a lot of these when running any rb-site commands:

> */usr/local/lib/python2.6/dist-packages/Django-1.4.1-py2.6.egg/django/core/cache/__init__.py:82:
>  
> DeprecationWarning: settings.CACHE_* is deprecated; use settings.CACHES 
> instead.
> **  DeprecationWarning*


I am running on Ubuntu 10.04 64 bit Server.

Cheers
Dan

On Friday, August 3, 2012 9:49:58 AM UTC+1, Christian Hammond wrote:
>
> Hey everyone,
>
> I'm happy to announce that we've pushed out the release of Review Board 
> 1.7 beta 1. There's many new and exciting things in this release, but 
> rather than repeat myself, I'll just link you to the news post.
>
> http://www.reviewboard.org/news/2012/08/03/review-board-17-beta-1-released/
>
> Please let me know if you hit any problems. And remember, it's a beta, so 
> use your best judgment when deciding where to deploy it. Any testing would 
> definitely be helpful, though, as that will help ensure a higher-quality 
> release.
>
> Christian
>
> -- 
> 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

Post-Review and Non publically accessible repositories

[Daniel Laird]

I am using post-review to generate my diff requests and post them to my 
reviewboard instance.
One of my SVN servers has restricted access, as such I mark it as non 
publically accessible and add all the Users /Groups that can access it.
The problem is when I run post-review it fails to find the repo and I get 
an error that the repo is not accessible.

However if I make it publically accessible it all works fine.
It seems to be a problem where post-review does not do any authentication 
until after it has tried to locate the repo - but this wont work.

Has anyone else seen the same issue and if so did you solve it OR is this a 
bigger problem in post-review that will take some hacking or should I not 
be doing this.

Cheers
Dan

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

rb-site help

[Daniel Laird]

Hi,

I am going to use rb-site to dump the data and then import onto a new 
webserver.
Whilst trying to check out the options I noticed that when I do rb-site -h 
there is no help regarding the manage options.
I am going to check the source code - but is this intentional or just an 
oversight?

Cheers
Dan

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

Phantom Review Closer

[Daniel]

We've been experiencing something weird within ReviewBoard. Reviews are 
being closed and either A) users aren't remembering that they're doing it 
(happening too often, so I don't believe this to be the case) B) Something 
else is closing the reviews.

The main problem is we can't figure out what is closing them as there is no 
record of who closed them, and I can't find any evidence in the logs to 
show when something is closed and an indication to who did it.

Any help will be much appreciated.

Thanks,
Daniel

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

Can Connect SVN Repository thru proxy

[Daniel Etchen]

Hi all,

We're using Review Board 1.6.4.1.
We access a Subversion server thru a proxy which is configured in /
root/.subversion/servers.
The 'svn info http://svn.foo.com:8080/repos/3845' command return a
correct response.

But when we try to create a new repository in reviewboard, the server
kicks back an error:
2012-06-20 09:34:08,240 - ERROR - SVN: Failed to get repository
information for http://svn.foo.com:8080/repos/3845: OPTIONS of 'http://
svn.foo.com:8080/repos/3845': Could not resolve hostname
`svn.foo.com': Host not found (http://svn.foo.com:8080)

Where is the problem ? It seems that the proxy configuration
configured in /root/.subversion/servers is not read by reviewboard

Thank you for your help.

-- 
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: Problem uploading screenshots

[Daniel]

Attached is the screenshot of the console you asked for.


Thanks,

Daniel

<https://lh6.googleusercontent.com/-zI4uTyoe4ys/T6QParQvlfI/AFQ/jEq2pTO8Q1E/s1600/ScreenShot003.jpg>


On Thursday, April 26, 2012 2:48:54 PM UTC-6, Christian Hammond wrote:
>
> What version of Review Board?
>
> Can you use Chrome or Firefox's developer tools (javascript console and 
> such) to see what the network transfer looks like for that request?
>
> Christian
>
> -- 
> Christian Hammond - chip...@chipx86.com
> Review Board - http://www.reviewboard.org
> VMware, Inc. - http://www.vmware.com
>
>
> On Thu, Apr 26, 2012 at 1:04 PM, Daniel  wrote:
>
>> We are having a problem where we cannot upload screenshots. Whenever we 
>> try we have an error that looks like the following:
>>
>> *Error Code:* 0
>> *Error Text:* n/a
>> *Request URL:* 
>> http://reviewboard.ks.quantum.com/api/review-requests/7424/screenshots/
>> *Request Data:* (none)
>> *Response Data:*
>> There may be useful error details below. The following error page may be 
>> useful to your system administrator or when reporting a 
>> bug<https://mail.quantum.com/owa/redir.aspx?C=01f5dced1e83444a9406c2539e8b4474&URL=http%3a%2f%2fwww.reviewboard.org%2fbugs%2fnew%2f>.
>>  
>> To save the page, right-click the error below and choose "Save Page As," if 
>> available, or "View Source" and save the result as a .html file.
>> *Warning:* Be sure to remove any sensitive material that may exist in 
>> the error page before reporting a bug!
>>
>> We've already installed PIL, and have already double checked the 
>> permissions of the media folder. I can't find information in the apache or 
>> rb logs. Any help appreciated.
>>
>> Thanks,
>> Daniel
>>
>> -- 
>> 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

Adding LDAP after the fact

[Daniel]

I'm looking at adding LDAP authentication to ReviewBoard. However I want to 
know what it will do regarding the users that currently already exist and 
the reviews that are under them. Especially if their usernames don't 
currently match their LDAP alternatives.

Thanks,
Daniel

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

Problem uploading screenshots

[Daniel]

We are having a problem where we cannot upload screenshots. Whenever we try 
we have an error that looks like the following:

*Error Code:* 0
*Error Text:* n/a
*Request 
URL:*http://reviewboard.ks.quantum.com/api/review-requests/7424/screenshots/
*Request Data:* (none)
*Response Data:*
There may be useful error details below. The following error page may be 
useful to your system administrator or when reporting a 
bug<https://mail.quantum.com/owa/redir.aspx?C=01f5dced1e83444a9406c2539e8b4474&URL=http%3a%2f%2fwww.reviewboard.org%2fbugs%2fnew%2f>.
 
To save the page, right-click the error below and choose "Save Page As," if 
available, or "View Source" and save the result as a .html file.
*Warning:* Be sure to remove any sensitive material that may exist in the 
error page before reporting a bug!

We've already installed PIL, and have already double checked the 
permissions of the media folder. I can't find information in the apache or 
rb logs. Any help appreciated.

Thanks,
Daniel

-- 
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: Password reset email said to come from example.com despite correct server name set in General Settings

[Daniel]

Yep, that did it. Thanks.

Daniel

On Monday, March 26, 2012 11:56:45 PM UTC-6, brajesh wrote:
>
> Hi Daniel,
>
> You need to make changes under Database -> Sites
>
> Thanks
> Brajesh
>
> On Tue, Mar 27, 2012 at 6:06 AM, Daniel  wrote:
>
>> One of my Review Board users attempted to reset his password. He got it 
>> correctly, however it didn't look right. He got the following:
>>
>> Subject: Password reset on *example.com*
>>
>>  You're receiving this e-mail because you requested a password reset for 
>> your account at *example.com*.
>>
>>  Please go to the following page and choose a new password:
>> 
>>
>>  In case you've forgotten, your username is John.Doe
>>
>>  Thanks for using Review Board!
>>
>> Does anyone know why example.com is showing and where I can fix it. It's 
>> set correctly in the admin site under General Settings.
>>
>> Thanks,
>> Daniel
>>
>> --
>>
>
>

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

Password reset email said to come from example.com despite correct server name set in General Settings

[Daniel]

One of my Review Board users attempted to reset his password. He got it 
correctly, however it didn't look right. He got the following:

Subject: Password reset on *example.com*

 You're receiving this e-mail because you requested a password reset for 
your account at *example.com*.

 Please go to the following page and choose a new password:


 In case you've forgotten, your username is John.Doe

 Thanks for using Review Board!

Does anyone know why example.com is showing and where I can fix it. It's 
set correctly in the admin site under General Settings.

Thanks,
Daniel

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

Paid Support

[Daniel]

Is there anybody out there who does or is willing to give us support
for a fee?

We've come across several problems with SQLite, and we are trying to
get moved off of it, however Django and SQLite stop us every step of
the way, and I'm at my wits end. We need to migrate while preserving
data. I've tried asking for help on this discussion and haven't gotten
anything, and in the meantime our Reviewboard is continually plagued
with database locked errors that we can't get away from. Especially
when django comes back and gives me a process killed with no
explanation. And as far as I can tell, that was with verbose turned
on.

Any and all help and offers are appreciated.

Thanks,
Daniel

-- 
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: Trying to move from sqlite to mysql running into disk image is malformed problems

[Daniel]

I found out what I needed to do to get our data free from the database
corruption. I ran an dump on the database, then simply reimported into
a new database file. The bad data was affected, however I was able to
get the 99.999% good data out.Now the database doesn't complain about
a malformed disk image. Instructions are here:

http://techblog.dorogin.com/2011/05/sqliteexception-database-disk-image-is.html

Then simply make sure to remove the rollback command at the end of the
script file, as well as make sure to run the commit command before
exiting the sqlite command line. If you miss these two, you end up
with an empty database and a zero byte file.

On Jan 9, 4:14 pm, Daniel  wrote:
> I went into the database via sqlite and ran a PRAGMA Integrity_Check:
>
> sqlite> PRAGMA integrity_check;
> *** in database main ***
> On tree page 244136 cell 0: invalid page number 218103808
> On tree page 241765 cell 63: 2nd reference to page 244130
> On tree page 241765 cell 63: Child page depth differs
> On tree page 241765 cell 64: Child page depth differs
> Page 244129 is never used
> Page 244131 is never used
> Page 244132 is never used
> Page 244133 is never used
> Page 244134 is never used
> Page 244135 is never used
> Page 398364 is never used
> Page 398365 is never used
>
> I've looked around and can't find a way to get these cleared. Does
> anybody know of a way? We really need this to get fixed so we can make
> the migration out of SQLite with data intact.
>
> Thanks,Daniel
>
> On Dec 17 2011, 10:16 am,Daniel wrote:
>
>
>
>
>
>
>
> > I'm in the midst of trying to move from SQLite to MySQL. I know that
> > we shouldn't be running SQLite on a production server, that's the
> > reason for the migration. However whenever I try to dump the data from
> > the database using:
> > rb-site manage . dumpdata > sqlite.json
> > I get the following error:
>
> > Traceback (most recent call last):
> >   File "/usr/local/bin/rb-site", line 8, in 
> >     load_entry_point('ReviewBoard==1.5.4', 'console_scripts', 'rb-
> > site')()
> >   File "/usr/local/lib/python2.6/dist-packages/ReviewBoard-1.5.4-
> > py2.6.egg/reviewboard/cmdline/rbsite.py", line 1611, in main
> >     command.run()
> >   File "/usr/local/lib/python2.6/dist-packages/ReviewBoard-1.5.4-
> > py2.6.egg/reviewboard/cmdline/rbsite.py", line 1545, in run
> >     site.run_manage_command(args[0], args[1:])
> >   File "/usr/local/lib/python2.6/dist-packages/ReviewBoard-1.5.4-
> > py2.6.egg/reviewboard/cmdline/rbsite.py", line 397, in
> > run_manage_command
> >     execute_manager(reviewboard.settings, [__file__, cmd] + params)
> >   File "/usr/local/lib/python2.6/dist-packages/Django-1.2.5-py2.6.egg/
> > django/core/management/__init__.py", line 438, in execute_manager
> >     utility.execute()
> >   File "/usr/local/lib/python2.6/dist-packages/Django-1.2.5-py2.6.egg/
> > django/core/management/__init__.py", line 379, in execute
> >     self.fetch_command(subcommand).run_from_argv(self.argv)
> >   File "/usr/local/lib/python2.6/dist-packages/Django-1.2.5-py2.6.egg/
> > django/core/management/base.py", line 191, in run_from_argv
> >     self.execute(*args, **options.__dict__)
> >   File "/usr/local/lib/python2.6/dist-packages/Django-1.2.5-py2.6.egg/
> > django/core/management/base.py", line 220, in execute
> >     output = self.handle(*args, **options)
> >   File "/usr/local/lib/python2.6/dist-packages/Django-1.2.5-py2.6.egg/
> > django/core/management/commands/dumpdata.py", line 84, in handle
> >     objects.extend(model._default_manager.using(using).all())
> >   File "/usr/local/lib/python2.6/dist-packages/Django-1.2.5-py2.6.egg/
> > django/db/models/query.py", line 82, in __len__
> >     self._result_cache.extend(list(self._iter))
> >   File "/usr/local/lib/python2.6/dist-packages/Django-1.2.5-py2.6.egg/
> > django/db/models/query.py", line 271, in iterator
> >     for row in compiler.results_iter():
> >   File "/usr/local/lib/python2.6/dist-packages/Django-1.2.5-py2.6.egg/
> > django/db/models/sql/compiler.py", line 677, in results_iter
> >     for rows in self.execute_sql(MULTI):
> >   File "/usr/local/lib/python2.6/dist-packages/Django-1.2.5-py2.6.egg/
> > django/db/models/sql/compiler.py", line 752, in execute_sql
> >     return list(result)
> >   File "/usr/local/lib/python2.6/dist-packages/Django-1.2.5-py2.6.egg/
> > django/db/models/sql/compiler.py", line 746, in

Re: Trying to move from sqlite to mysql running into disk image is malformed problems

[Daniel]

I went into the database via sqlite and ran a PRAGMA Integrity_Check:

sqlite> PRAGMA integrity_check;
*** in database main ***
On tree page 244136 cell 0: invalid page number 218103808
On tree page 241765 cell 63: 2nd reference to page 244130
On tree page 241765 cell 63: Child page depth differs
On tree page 241765 cell 64: Child page depth differs
Page 244129 is never used
Page 244131 is never used
Page 244132 is never used
Page 244133 is never used
Page 244134 is never used
Page 244135 is never used
Page 398364 is never used
Page 398365 is never used

I've looked around and can't find a way to get these cleared. Does
anybody know of a way? We really need this to get fixed so we can make
the migration out of SQLite with data intact.

Thanks,
Daniel

On Dec 17 2011, 10:16 am, Daniel  wrote:
> I'm in the midst of trying to move from SQLite to MySQL. I know that
> we shouldn't be running SQLite on a production server, that's the
> reason for the migration. However whenever I try to dump the data from
> the database using:
> rb-site manage . dumpdata > sqlite.json
> I get the following error:
>
> Traceback (most recent call last):
>   File "/usr/local/bin/rb-site", line 8, in 
>     load_entry_point('ReviewBoard==1.5.4', 'console_scripts', 'rb-
> site')()
>   File "/usr/local/lib/python2.6/dist-packages/ReviewBoard-1.5.4-
> py2.6.egg/reviewboard/cmdline/rbsite.py", line 1611, in main
>     command.run()
>   File "/usr/local/lib/python2.6/dist-packages/ReviewBoard-1.5.4-
> py2.6.egg/reviewboard/cmdline/rbsite.py", line 1545, in run
>     site.run_manage_command(args[0], args[1:])
>   File "/usr/local/lib/python2.6/dist-packages/ReviewBoard-1.5.4-
> py2.6.egg/reviewboard/cmdline/rbsite.py", line 397, in
> run_manage_command
>     execute_manager(reviewboard.settings, [__file__, cmd] + params)
>   File "/usr/local/lib/python2.6/dist-packages/Django-1.2.5-py2.6.egg/
> django/core/management/__init__.py", line 438, in execute_manager
>     utility.execute()
>   File "/usr/local/lib/python2.6/dist-packages/Django-1.2.5-py2.6.egg/
> django/core/management/__init__.py", line 379, in execute
>     self.fetch_command(subcommand).run_from_argv(self.argv)
>   File "/usr/local/lib/python2.6/dist-packages/Django-1.2.5-py2.6.egg/
> django/core/management/base.py", line 191, in run_from_argv
>     self.execute(*args, **options.__dict__)
>   File "/usr/local/lib/python2.6/dist-packages/Django-1.2.5-py2.6.egg/
> django/core/management/base.py", line 220, in execute
>     output = self.handle(*args, **options)
>   File "/usr/local/lib/python2.6/dist-packages/Django-1.2.5-py2.6.egg/
> django/core/management/commands/dumpdata.py", line 84, in handle
>     objects.extend(model._default_manager.using(using).all())
>   File "/usr/local/lib/python2.6/dist-packages/Django-1.2.5-py2.6.egg/
> django/db/models/query.py", line 82, in __len__
>     self._result_cache.extend(list(self._iter))
>   File "/usr/local/lib/python2.6/dist-packages/Django-1.2.5-py2.6.egg/
> django/db/models/query.py", line 271, in iterator
>     for row in compiler.results_iter():
>   File "/usr/local/lib/python2.6/dist-packages/Django-1.2.5-py2.6.egg/
> django/db/models/sql/compiler.py", line 677, in results_iter
>     for rows in self.execute_sql(MULTI):
>   File "/usr/local/lib/python2.6/dist-packages/Django-1.2.5-py2.6.egg/
> django/db/models/sql/compiler.py", line 752, in execute_sql
>     return list(result)
>   File "/usr/local/lib/python2.6/dist-packages/Django-1.2.5-py2.6.egg/
> django/db/models/sql/compiler.py", line 746, in 
>     result = iter((lambda: cursor.fetchmany(GET_ITERATOR_CHUNK_SIZE)),
> sqlite3.DatabaseError: databasediskimageismalformed
>
> I need some help getting this resolved. This data cannot be lost. Our
> actual RB site appears to do just fine, except that we get database
> locked errors because of high traffic. Another reason we are trying to
> migrate to MySQL. Any help would be greatly appreciated.
>
> Thanks,
> Daniel

-- 
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: Improvement for "Add File" feature

[Daniel]

This sounds great!

On Dec 20 2011, 5:16 pm, Christian Hammond 
wrote:
> On Tue, Dec 20, 2011 at 5:05 PM, Corinna  wrote:
> > I'm glad to see the "Add File" feature added to ReviewBoard.
>
> > It would be even cleaner to remove the "Add Screenshot" button and
> > have ReviewBoard automatically treat files with image extensions (PNG,
> > etc.) as screenshots.
>
> This is actually part of the plan. Step 1 was to get file attachments in
> there. We'll be a bit closer in 1.7, if not finished. The plan is to make
> file attachment review a thing, and to make it flexible enough where we can
> provide review UI for images but also allow for extensions to provide their
> own for other file types. Depending on how much we end up doing for 1.7, we
> may fully get there for images, and when we have that, I plan to remove
> screenshot uploading.
>
>
>
> > Would it be possible to annotate regions of a non-image file, such as
> > a PDF? If so, we would not need to differentiate between screenshots
> > and other file types at all.
>
> I would love something like PDF review, but it's unlikely we'll provide it.
> That's a considerable amount of work. We'd need a PDF-to-canvas/image/HTML
> renderer and a review UI for it. I'm hoping once we have proper extension
> support, others may write such things. If we provide that feature, it'll
> likely be a for-sale add-on.
>
> Christian
>
> --
> 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

Trying to move from sqlite to mysql running into disk image is malformed problems

[Daniel]

I'm in the midst of trying to move from SQLite to MySQL. I know that
we shouldn't be running SQLite on a production server, that's the
reason for the migration. However whenever I try to dump the data from
the database using:
rb-site manage . dumpdata > sqlite.json
I get the following error:

Traceback (most recent call last):
  File "/usr/local/bin/rb-site", line 8, in 
load_entry_point('ReviewBoard==1.5.4', 'console_scripts', 'rb-
site')()
  File "/usr/local/lib/python2.6/dist-packages/ReviewBoard-1.5.4-
py2.6.egg/reviewboard/cmdline/rbsite.py", line 1611, in main
command.run()
  File "/usr/local/lib/python2.6/dist-packages/ReviewBoard-1.5.4-
py2.6.egg/reviewboard/cmdline/rbsite.py", line 1545, in run
site.run_manage_command(args[0], args[1:])
  File "/usr/local/lib/python2.6/dist-packages/ReviewBoard-1.5.4-
py2.6.egg/reviewboard/cmdline/rbsite.py", line 397, in
run_manage_command
execute_manager(reviewboard.settings, [__file__, cmd] + params)
  File "/usr/local/lib/python2.6/dist-packages/Django-1.2.5-py2.6.egg/
django/core/management/__init__.py", line 438, in execute_manager
utility.execute()
  File "/usr/local/lib/python2.6/dist-packages/Django-1.2.5-py2.6.egg/
django/core/management/__init__.py", line 379, in execute
self.fetch_command(subcommand).run_from_argv(self.argv)
  File "/usr/local/lib/python2.6/dist-packages/Django-1.2.5-py2.6.egg/
django/core/management/base.py", line 191, in run_from_argv
self.execute(*args, **options.__dict__)
  File "/usr/local/lib/python2.6/dist-packages/Django-1.2.5-py2.6.egg/
django/core/management/base.py", line 220, in execute
output = self.handle(*args, **options)
  File "/usr/local/lib/python2.6/dist-packages/Django-1.2.5-py2.6.egg/
django/core/management/commands/dumpdata.py", line 84, in handle
objects.extend(model._default_manager.using(using).all())
  File "/usr/local/lib/python2.6/dist-packages/Django-1.2.5-py2.6.egg/
django/db/models/query.py", line 82, in __len__
self._result_cache.extend(list(self._iter))
  File "/usr/local/lib/python2.6/dist-packages/Django-1.2.5-py2.6.egg/
django/db/models/query.py", line 271, in iterator
for row in compiler.results_iter():
  File "/usr/local/lib/python2.6/dist-packages/Django-1.2.5-py2.6.egg/
django/db/models/sql/compiler.py", line 677, in results_iter
for rows in self.execute_sql(MULTI):
  File "/usr/local/lib/python2.6/dist-packages/Django-1.2.5-py2.6.egg/
django/db/models/sql/compiler.py", line 752, in execute_sql
return list(result)
  File "/usr/local/lib/python2.6/dist-packages/Django-1.2.5-py2.6.egg/
django/db/models/sql/compiler.py", line 746, in 
result = iter((lambda: cursor.fetchmany(GET_ITERATOR_CHUNK_SIZE)),
sqlite3.DatabaseError: database disk image is malformed

I need some help getting this resolved. This data cannot be lost. Our
actual RB site appears to do just fine, except that we get database
locked errors because of high traffic. Another reason we are trying to
migrate to MySQL. Any help would be greatly appreciated.

Thanks,
Daniel

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

post-receive hook for Reviewboard?

[Daniel Dieckmann]

Hey,

I installed ReviewBoard a couple of days ago and now I am wondering
if there is any possibility to make a post-receive hook in a git repo,
that creates a new review request.

So I want to do something like this:
1. Someone writes some code an does a commit. Now he pushes this
one to the server where ReviewBoard is running and which also has
a clone of the original git repo.
2. After the push there is some post-receive hook creating a new
review
request using post-review.

But none of my solutions worked, have zou any idea how to solve this
problem?

Thanks in advance,
Daniel

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

[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: Bzr support in post-review

[Daniel Benamy]

Hi Christian,

I can spend a little time working on the patch and testing. I'm not
bzr expert, but I use it daily. Would that be ok?

Who needs to do the full review?

Do you know when the freeze should be done? Ie, when should I poke at
this issue again?

Thanks!
Dan

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

Bzr support in post-review

[Daniel Benamy]

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/888a492465ded9/b42a2ec7bcc7e6c8?lnk=gst&q=dbenamy#b42a2ec7bcc7e6c8

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 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: Review Board accepted to Google Summer of Code 2011 - Looking for mentors/ideas

[Daniel Benamy]

I'll suggest finishing bazaar support in post-review although there
may not be that much work left there.

Dan

On Mar 21, 4:19 pm, Christian Hammond  wrote:
> Hey everyone,
>
> We've been accepted again this year in Google's Summer of Code, a yearly
> event where students from around the world participate in open source
> projects, developing work experience and getting paid for it.
>
> We're expecting about 4 or 5 students this year, but we can grow that number
> with more mentors.
>
> I'd like to see if there are any past contributors to Review Board who know
> the codebase well enough and would like to mentor a student or two this
> year. It's a great way to benefit the project.
>
> A mentor's job is to assist the student in his development, answer questions
> (forwarding on to us when needed), and to help collect status and run the
> weekly meetings. They're also expected to grade the students half-way
> through the Summer and at the end.
>
> We're also looking for ideas for projects, so if you have an idea for
> something awesome you'd like to see, just reply :)
>
> Thanks!
>
> Christian
>
> --
> 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