Re: Unable to download attachments that have been uploaded.

[Not Saying]

I got it working.  THANKS

On Tuesday, October 17, 2017 at 10:01:17 AM UTC-7, Not Saying wrote:
>
> RHEL 7
> *Review Board*  2.5.15 
>
> *I click on the download button and it give me a page *This page can’t be 
> displayed
> The URL is 
> http://localhost/reviewboard/media/uploaded/files/2017/10/16/4654d4a1-10f2-45c5-ab3d-6e9f37b85624__Atsugi_UH_06012017.pdf
>
> I see the localhost in the URL where is that defined so I can change it.
>
> Thank you.
>
>

-- 
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: Unable to download attachments that have been uploaded.

[Not Saying]

On Tuesday, October 17, 2017 at 10:01:17 AM UTC-7, Not Saying wrote:
>
> RHEL 7
> *Review Board*  2.5.15 
>
> *I click on the download button and it give me a page *This page can’t be 
> displayed
> The URL is 
> http://localhost/reviewboard/media/uploaded/files/2017/10/16/4654d4a1-10f2-45c5-ab3d-6e9f37b85624__Atsugi_UH_06012017.pdf
>
> I see the localhost in the URL where is that defined so I can change it.
>
> Thank you.
>
>

-- 
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: Unable to download attachments that have been uploaded.

[Not Saying]

[root@linuxa2 reviewboard-2.0.15-0]# ./ctlscript.sh start
Warning: World-writable config file 
'/var/reviewboard-2.0.15-0/mysql/my.cnf' is ignored
Warning: World-writable config file 
'/var/reviewboard-2.0.15-0/mysql/my.cnf' is ignored
171018 16:20:59 mysqld_safe Logging to 
'/var/reviewboard-2.0.15-0/mysql/data/mysqld.log'.
171018 16:20:59 mysqld_safe Starting mysqld daemon with databases from 
/var/reviewboard-2.0.15-0/mysql/data
171018 16:21:01 mysqld_safe mysqld from pid file 
/var/reviewboard-2.0.15-0/mysql/data/mysqld.pid ended
/var/reviewboard-2.0.15-0/mysql/scripts/ctl.sh : mysql  could not be started
/var/reviewboard-2.0.15-0/memcached/scripts/ctl.sh : Memcached started
[root@linuxa2 reviewboard-2.0.15-0]#

Now from the local file for MySQL

171018 16:29:55 mysqld_safe Starting mysqld.bin daemon with databases from 
/var/reviewboard-2.0.15-0/mysql/data
171018 16:29:55 [Note] Plugin 'FEDERATED' is disabled.
^G/var/reviewboard-2.0.15-0/mysql/bin/mysqld.bin: Can't find file: 
'./mysql/plugin.frm' (errno: 13)
171018 16:29:55 [ERROR] Can't open the mysql.plugin table. Please run 
mysql_upgrade to create it.
171018 16:29:55 InnoDB: The InnoDB memory heap is disabled
171018 16:29:55 InnoDB: Mutexes and rw_locks use GCC atomic builtins
171018 16:29:55 InnoDB: Compressed tables use zlib 1.2.3
171018 16:29:55 InnoDB: Using Linux native AIO
^G/var/reviewboard-2.0.15-0/mysql/bin/mysqld.bin: Can't create/write to 
file '/var/reviewboard-2.0.15-0/mysql/tmp/ibCiB4PU' (Errcode: 13)
171018 16:29:55  InnoDB: Error: unable to create temporary file; errno: 13
171018 16:29:55 [ERROR] Plugin 'InnoDB' init function returned error.
171018 16:29:55 [ERROR] Plugin 'InnoDB' registration as a STORAGE ENGINE 
failed.
171018 16:29:55 [ERROR] Unknown/unsupported storage engine: InnoDB
171018 16:29:55 [ERROR] Aborting
171018 16:29:55 [Note] /var/reviewboard-2.0.15-0/mysql/bin/mysqld.bin: 
Shutdown complete
171018 16:29:55 mysqld_safe mysqld from pid file 
/var/reviewboard-2.0.15-0/mysql/data/mysqld.pid ended

???   Need HELP

On Tuesday, October 17, 2017 at 10:01:17 AM UTC-7, Not Saying wrote:

> RHEL 7
> *Review Board*  2.5.15 
>
> *I click on the download button and it give me a page *This page can’t be 
> displayed
> The URL is 
> http://localhost/reviewboard/media/uploaded/files/2017/10/16/4654d4a1-10f2-45c5-ab3d-6e9f37b85624__Atsugi_UH_06012017.pdf
>
> I see the localhost in the URL where is that defined so I can change it.
>
> Thank you.
>
>

-- 
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: Unable to download attachments that have been uploaded.

[Not Saying]

I made the changes and now MySQL won't start???   I will post back when I 
know more.



On Tuesday, October 17, 2017 at 10:01:17 AM UTC-7, Not Saying wrote:

> RHEL 7
> *Review Board*  2.5.15 
>
> *I click on the download button and it give me a page *This page can’t be 
> displayed
> The URL is 
> http://localhost/reviewboard/media/uploaded/files/2017/10/16/4654d4a1-10f2-45c5-ab3d-6e9f37b85624__Atsugi_UH_06012017.pdf
>
> I see the localhost in the URL where is that defined so I can change it.
>
> Thank you.
>
>

-- 
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: Unable to download attachments that have been uploaded.

[Not Saying]

I made the change suggested and restart the server.   Now MySQL is not 
working see below

On Tuesday, October 17, 2017 at 10:01:17 AM UTC-7, Not Saying wrote:
>
> RHEL 7
> *Review Board*  2.5.15 
>
> *I click on the download button and it give me a page *This page can’t be 
> displayed
> The URL is 
> http://localhost/reviewboard/media/uploaded/files/2017/10/16/4654d4a1-10f2-45c5-ab3d-6e9f37b85624__Atsugi_UH_06012017.pdf
>
> I see the localhost in the URL where is that defined so I can change it.
>
> Thank you.
>
>

-- 
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: Unable to download attachments that have been uploaded.

[Not Saying]

THANK YOU

On Wednesday, October 18, 2017 at 2:01:32 AM UTC-7, Christian Hammond wrote:
>
> Hi,
>
> Looks like your Media Path setting may be pointing to the wrong place, or 
> your Review Board server name setting isn't set to your domain name. Can 
> you check the Administration UI -> General Settings and see what those are 
> set to?
>
> Christian
>
> On Tue, Oct 17, 2017 at 10:06 AM, Not Saying  > wrote:
>
>>
>>
>> On Tuesday, October 17, 2017 at 10:01:17 AM UTC-7, Not Saying wrote:
>>>
>>> RHEL 7
>>> *Review Board*  2.5.15 
>>>
>>> *I click on the download button and it give me a page *This page can’t 
>>> be displayed
>>> The URL is 
>>> http://localhost/reviewboard/media/uploaded/files/2017/10/16/4654d4a1-10f2-45c5-ab3d-6e9f37b85624__Atsugi_UH_06012017.pdf
>>>
>>> I see the localhost in the URL where is that defined so I can change it.
>>> If I change the local host to linuxa2ng1.loc the download works.
>>>
>>
>>
>>
>>  
>>
>>> Thank you.
>>>
>>> -- 
>> 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.
>>
>
>
>
> -- 
> Christian Hammond
> President/CEO of Beanbag 
> Makers of Review Board 
>

-- 
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: MySQL/MariaDB utf8mb4

[Christian Hammond]

Hi Vince,

Right now, don't use utf8mb4 by itself. That *is* the right encoding to use
for proper Unicode support, yes, but additional settings need to be enabled
manually on your database (globally and per-table) in order for key lengths
to not be truncated (which will otherwise break things horribly). I could
provide this information if you wanted to be a guinea pig, but the reality
is that future upgrades won't be able to apply the settings on newer tables
just yet, so upgrades can still break the compatibility.

I've done a lot of experimentation with this, and I'm hoping that by the
Review Board 4.0 timeframe that I can have built-in support, documentation,
and a conversion tool for properly enabling utf8mb4 and all related
settings as part of Review Board. I know exactly what needs to be done,
it's just a matter of doing it and having a lot of testing in place, which
we weren't able to do for 3.0.

Christian

On Wed, Oct 18, 2017 at 9:57 AM, Vince Thyng  wrote:

> Is anyone using utf8mb4?  We are looking at moving from latin1 to utf8.
> This article makes the case for using it over utf8, https://mathiasbynens.
> be/notes/mysql-utf8mb4 and am hoping someone in the community already has
> experience with it.  I figure we will need to look at the size of every
> text field to be sure that losing 25% of the max will not cut off any data,
> or else increase the size before switching to mb4.
>
> Thanks!
>
> Vince
>
> --
> 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.
>



-- 
Christian Hammond
President/CEO of Beanbag 
Makers of Review Board 

-- 
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/MariaDB utf8mb4

[Vince Thyng]

Is anyone using utf8mb4?  We are looking at moving from latin1 to utf8.  
This article makes the case for using it over 
utf8, https://mathiasbynens.be/notes/mysql-utf8mb4 and am hoping someone in 
the community already has experience with it.  I figure we will need to 
look at the size of every text field to be sure that losing 25% of the max 
will not cut off any data, or else increase the size before switching to 
mb4.

Thanks!

Vince

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

MariaDB vs MySQL

[Vince Thyng]

Reviewboard currently recommends MySQL 5.6.  MariaDB branches to a 10.x 
numbering with what would have been MariaDB 5.6.  Is anyone using MariaDB 
10.x in place of MySQL.

Thanks!

Vince

-- 
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: Index failure after upgrade to ReviewBoard 2.5.16

['Rob Backhurst' via reviewboard]

Yep thanks, index looks to be running now.
Hopefully it will get to the end this time - if so I'll make the same 
change on our live system.

Cheers
Rob

On Wednesday, 18 October 2017 14:42:55 UTC+1, Erik Johansson wrote:
>
> I'm guessing the if statement should be "if not ..." (i.e. not is missing).
>
> // Erik
>
>
> On Oct 18, 2017 12:44, "'Rob Backhurst' via reviewboard" <
> revie...@googlegroups.com > wrote:
>
> Hi Chris,
>
> Ah yes sorry missed that...I have added it in but I get the same error 
> though.
>
>  81 # Check for `__` in the field for looking through the 
> relation.
>  82 attrs = self.model_attr.split('__')
>  83 current_object = obj
>  84
>  85 for attr in attrs:
>  86 hasattr(current_object, attr)
>  87
>  88 if hasattr(current_object, attr):
>  89 raise SearchFieldError("The model '%s' 
> does not have a model_attr '%s'." % (repr(current_object), attr))
>  90
>  91 current_object = getattr(current_object, attr, 
> None)
>  92
>  93 if current_object is None:
>  94 if self.has_default():
>  95 current_object = self._default
>  96 # Fall out of the loop, given any further 
> attempts at
>
>
>   File "/usr/lib/python2.7/site-packages/haystack/fields.py", line 89, in 
> prepare
> raise SearchFieldError("The model '%s' does not have a model_attr 
> '%s'." % (repr(current_object), attr))
> haystack.exceptions.SearchFieldError: The model '' does not 
> have a model_attr 'username'.
>
>
> Thanks
> Rob
>
>
>
> On Wednesday, 18 October 2017 10:02:54 UTC+1, Christian Hammond wrote:
>
>> Hi Rob,
>>
>> That doesn't include the code I mentioned in my previous e-mail. Note the 
>> standalone 'hasattr' call on the line preceding the if statement. The 
>> workaround is to call that in a standalone way to prime a cache and avoid 
>> the error.
>>
>> Christian
>>
>> On Tue, Oct 17, 2017 at 1:17 AM, 'Rob Backhurst' via reviewboard <
>> revie...@googlegroups.com> wrote:
>>
>>>  77 # Give priority to a template.
>>>  78 if self.use_template:
>>>  79 return self.prepare_template(obj)
>>>  80 elif self.model_attr is not None:
>>>  81 # Check for `__` in the field for looking through 
>>> the relation.
>>>  82 attrs = self.model_attr.split('__')
>>>  83 current_object = obj
>>>  84
>>>  85 for attr in attrs:
>>>  86 if hasattr(current_object, attr):
>>>  87 raise SearchFieldError("The model '%s' 
>>> does not have a model_attr '%s'." % (repr(current_object), attr))
>>>  88
>>>  89 current_object = getattr(current_object, attr, 
>>> None)
>>>  90
>>>  91 if current_object is None:
>>>  92 if self.has_default():
>>>  93 current_object = self._default
>>>  94 # Fall out of the loop, given any 
>>> further attempts at
>>>  95 # accesses will fail misreably.
>>>
>>>
>>> Thanks
>>> Rob
>>>
>>> On Tuesday, 17 October 2017 06:28:49 UTC+1, Christian Hammond wrote:

 Can you show me all the code within about 5 lines of your modification?

 Christian


 On Mon, Oct 16, 2017 at 18:01 'Rob Backhurst' via reviewboard <
 revie...@googlegroups.com> wrote:

> Hi Christian,
>
> It crashes straight away with this error...
>
> Removing all documents from your index because you said so.
> All documents removed.
> Indexing 558 users
> ERROR:root:Error updating auth using default
> Traceback (most recent call last):
>   File 
> "/usr/lib/python2.7/site-packages/haystack/management/commands/update_index.py",
>  
> line 188, in handle_label
> self.update_backend(label, using)
>   File 
> "/usr/lib/python2.7/site-packages/haystack/management/commands/update_index.py",
>  
> line 233, in update_backend
> do_update(backend, index, qs, start, end, total, 
> verbosity=self.verbosity, commit=self.commit)
>   File 
> "/usr/lib/python2.7/site-packages/haystack/management/commands/update_index.py",
>  
> line 96, in do_update
> backend.update(index, current_qs, commit=commit)
>   File 
> "/usr/lib/python2.7/site-packages/haystack/backends/whoosh_backend.py", 
> line 196, in update
> doc = index.full_prepare(obj)
>   File "/usr/lib/python2.7/site-packages/haystack/indexes.py", line 
> 212, in full_prepare
> self.prepared_data = self.prepare(obj)
>   File "/usr/lib/python2.7/site-packages/haystack/indexes.py", line 
> 203, in prepare
> self.prepared_data[field.inde

Re: Index failure after upgrade to ReviewBoard 2.5.16

[Erik Johansson]

I'm guessing the if statement should be "if not ..." (i.e. not is missing).

// Erik


On Oct 18, 2017 12:44, "'Rob Backhurst' via reviewboard" <
reviewboard@googlegroups.com> wrote:

Hi Chris,

Ah yes sorry missed that...I have added it in but I get the same error
though.

 81 # Check for `__` in the field for looking through the
relation.
 82 attrs = self.model_attr.split('__')
 83 current_object = obj
 84
 85 for attr in attrs:
 86 hasattr(current_object, attr)
 87
 88 if hasattr(current_object, attr):
 89 raise SearchFieldError("The model '%s' does
not have a model_attr '%s'." % (repr(current_object), attr))
 90
 91 current_object = getattr(current_object, attr, None)
 92
 93 if current_object is None:
 94 if self.has_default():
 95 current_object = self._default
 96 # Fall out of the loop, given any further
attempts at


  File "/usr/lib/python2.7/site-packages/haystack/fields.py", line 89, in
prepare
raise SearchFieldError("The model '%s' does not have a model_attr
'%s'." % (repr(current_object), attr))
haystack.exceptions.SearchFieldError: The model '' does not
have a model_attr 'username'.


Thanks
Rob



On Wednesday, 18 October 2017 10:02:54 UTC+1, Christian Hammond wrote:

> Hi Rob,
>
> That doesn't include the code I mentioned in my previous e-mail. Note the
> standalone 'hasattr' call on the line preceding the if statement. The
> workaround is to call that in a standalone way to prime a cache and avoid
> the error.
>
> Christian
>
> On Tue, Oct 17, 2017 at 1:17 AM, 'Rob Backhurst' via reviewboard <
> revie...@googlegroups.com> wrote:
>
>>  77 # Give priority to a template.
>>  78 if self.use_template:
>>  79 return self.prepare_template(obj)
>>  80 elif self.model_attr is not None:
>>  81 # Check for `__` in the field for looking through the
>> relation.
>>  82 attrs = self.model_attr.split('__')
>>  83 current_object = obj
>>  84
>>  85 for attr in attrs:
>>  86 if hasattr(current_object, attr):
>>  87 raise SearchFieldError("The model '%s'
>> does not have a model_attr '%s'." % (repr(current_object), attr))
>>  88
>>  89 current_object = getattr(current_object, attr,
>> None)
>>  90
>>  91 if current_object is None:
>>  92 if self.has_default():
>>  93 current_object = self._default
>>  94 # Fall out of the loop, given any further
>> attempts at
>>  95 # accesses will fail misreably.
>>
>>
>> Thanks
>> Rob
>>
>> On Tuesday, 17 October 2017 06:28:49 UTC+1, Christian Hammond wrote:
>>>
>>> Can you show me all the code within about 5 lines of your modification?
>>>
>>> Christian
>>>
>>>
>>> On Mon, Oct 16, 2017 at 18:01 'Rob Backhurst' via reviewboard <
>>> revie...@googlegroups.com> wrote:
>>>
 Hi Christian,

 It crashes straight away with this error...

 Removing all documents from your index because you said so.
 All documents removed.
 Indexing 558 users
 ERROR:root:Error updating auth using default
 Traceback (most recent call last):
   File 
 "/usr/lib/python2.7/site-packages/haystack/management/commands/update_index.py",
 line 188, in handle_label
 self.update_backend(label, using)
   File 
 "/usr/lib/python2.7/site-packages/haystack/management/commands/update_index.py",
 line 233, in update_backend
 do_update(backend, index, qs, start, end, total,
 verbosity=self.verbosity, commit=self.commit)
   File 
 "/usr/lib/python2.7/site-packages/haystack/management/commands/update_index.py",
 line 96, in do_update
 backend.update(index, current_qs, commit=commit)
   File 
 "/usr/lib/python2.7/site-packages/haystack/backends/whoosh_backend.py",
 line 196, in update
 doc = index.full_prepare(obj)
   File "/usr/lib/python2.7/site-packages/haystack/indexes.py", line
 212, in full_prepare
 self.prepared_data = self.prepare(obj)
   File "/usr/lib/python2.7/site-packages/haystack/indexes.py", line
 203, in prepare
 self.prepared_data[field.index_fieldname] = field.prepare(obj)
   File "/usr/lib/python2.7/site-packages/haystack/fields.py", line
 159, in prepare
 return self.convert(super(CharField, self).prepare(obj))
   File "/usr/lib/python2.7/site-packages/haystack/fields.py", line 87,
 in prepare
 raise SearchFieldError("The model '%s' does not have a model_attr
 '%s'." % (repr(current_object), attr))
 SearchFieldError: The mo

Re: Index failure after upgrade to ReviewBoard 2.5.16

['Rob Backhurst' via reviewboard]

Hi Chris,

Ah yes sorry missed that...I have added it in but I get the same error 
though.

 81 # Check for `__` in the field for looking through the 
relation.
 82 attrs = self.model_attr.split('__')
 83 current_object = obj
 84
 85 for attr in attrs:
 86 hasattr(current_object, attr)
 87
 88 if hasattr(current_object, attr):
 89 raise SearchFieldError("The model '%s' does 
not have a model_attr '%s'." % (repr(current_object), attr))
 90
 91 current_object = getattr(current_object, attr, None)
 92
 93 if current_object is None:
 94 if self.has_default():
 95 current_object = self._default
 96 # Fall out of the loop, given any further 
attempts at


  File "/usr/lib/python2.7/site-packages/haystack/fields.py", line 89, in 
prepare
raise SearchFieldError("The model '%s' does not have a model_attr 
'%s'." % (repr(current_object), attr))
haystack.exceptions.SearchFieldError: The model '' does not 
have a model_attr 'username'.


Thanks
Rob



On Wednesday, 18 October 2017 10:02:54 UTC+1, Christian Hammond wrote:
>
> Hi Rob,
>
> That doesn't include the code I mentioned in my previous e-mail. Note the 
> standalone 'hasattr' call on the line preceding the if statement. The 
> workaround is to call that in a standalone way to prime a cache and avoid 
> the error.
>
> Christian
>
> On Tue, Oct 17, 2017 at 1:17 AM, 'Rob Backhurst' via reviewboard <
> revie...@googlegroups.com > wrote:
>
>>  77 # Give priority to a template.
>>  78 if self.use_template:
>>  79 return self.prepare_template(obj)
>>  80 elif self.model_attr is not None:
>>  81 # Check for `__` in the field for looking through the 
>> relation.
>>  82 attrs = self.model_attr.split('__')
>>  83 current_object = obj
>>  84
>>  85 for attr in attrs:
>>  86 if hasattr(current_object, attr):
>>  87 raise SearchFieldError("The model '%s' 
>> does not have a model_attr '%s'." % (repr(current_object), attr))
>>  88
>>  89 current_object = getattr(current_object, attr, 
>> None)
>>  90
>>  91 if current_object is None:
>>  92 if self.has_default():
>>  93 current_object = self._default
>>  94 # Fall out of the loop, given any further 
>> attempts at
>>  95 # accesses will fail misreably.
>>
>>
>> Thanks
>> Rob
>>
>> On Tuesday, 17 October 2017 06:28:49 UTC+1, Christian Hammond wrote:
>>>
>>> Can you show me all the code within about 5 lines of your modification?
>>>
>>> Christian
>>>
>>>
>>> On Mon, Oct 16, 2017 at 18:01 'Rob Backhurst' via reviewboard <
>>> revie...@googlegroups.com> wrote:
>>>
 Hi Christian,

 It crashes straight away with this error...

 Removing all documents from your index because you said so.
 All documents removed.
 Indexing 558 users
 ERROR:root:Error updating auth using default
 Traceback (most recent call last):
   File 
 "/usr/lib/python2.7/site-packages/haystack/management/commands/update_index.py",
  
 line 188, in handle_label
 self.update_backend(label, using)
   File 
 "/usr/lib/python2.7/site-packages/haystack/management/commands/update_index.py",
  
 line 233, in update_backend
 do_update(backend, index, qs, start, end, total, 
 verbosity=self.verbosity, commit=self.commit)
   File 
 "/usr/lib/python2.7/site-packages/haystack/management/commands/update_index.py",
  
 line 96, in do_update
 backend.update(index, current_qs, commit=commit)
   File 
 "/usr/lib/python2.7/site-packages/haystack/backends/whoosh_backend.py", 
 line 196, in update
 doc = index.full_prepare(obj)
   File "/usr/lib/python2.7/site-packages/haystack/indexes.py", line 
 212, in full_prepare
 self.prepared_data = self.prepare(obj)
   File "/usr/lib/python2.7/site-packages/haystack/indexes.py", line 
 203, in prepare
 self.prepared_data[field.index_fieldname] = field.prepare(obj)
   File "/usr/lib/python2.7/site-packages/haystack/fields.py", line 159, 
 in prepare
 return self.convert(super(CharField, self).prepare(obj))
   File "/usr/lib/python2.7/site-packages/haystack/fields.py", line 87, 
 in prepare
 raise SearchFieldError("The model '%s' does not have a model_attr 
 '%s'." % (repr(current_object), attr))
 SearchFieldError: The model '' does not have a model_attr 
 'username'.
 - show quoted text -
   File "/usr/lib/python2.7/site-packages/haystack/fields.

Re: Index failure after upgrade to ReviewBoard 2.5.16

[Christian Hammond]

Hi Rob,

That doesn't include the code I mentioned in my previous e-mail. Note the
standalone 'hasattr' call on the line preceding the if statement. The
workaround is to call that in a standalone way to prime a cache and avoid
the error.

Christian

On Tue, Oct 17, 2017 at 1:17 AM, 'Rob Backhurst' via reviewboard <
reviewboard@googlegroups.com> wrote:

>  77 # Give priority to a template.
>  78 if self.use_template:
>  79 return self.prepare_template(obj)
>  80 elif self.model_attr is not None:
>  81 # Check for `__` in the field for looking through the
> relation.
>  82 attrs = self.model_attr.split('__')
>  83 current_object = obj
>  84
>  85 for attr in attrs:
>  86 if hasattr(current_object, attr):
>  87 raise SearchFieldError("The model '%s'
> does not have a model_attr '%s'." % (repr(current_object), attr))
>  88
>  89 current_object = getattr(current_object, attr,
> None)
>  90
>  91 if current_object is None:
>  92 if self.has_default():
>  93 current_object = self._default
>  94 # Fall out of the loop, given any further
> attempts at
>  95 # accesses will fail misreably.
>
>
> Thanks
> Rob
>
> On Tuesday, 17 October 2017 06:28:49 UTC+1, Christian Hammond wrote:
>>
>> Can you show me all the code within about 5 lines of your modification?
>>
>> Christian
>>
>>
>> On Mon, Oct 16, 2017 at 18:01 'Rob Backhurst' via reviewboard <
>> revie...@googlegroups.com> wrote:
>>
>>> Hi Christian,
>>>
>>> It crashes straight away with this error...
>>>
>>> Removing all documents from your index because you said so.
>>> All documents removed.
>>> Indexing 558 users
>>> ERROR:root:Error updating auth using default
>>> Traceback (most recent call last):
>>>   File 
>>> "/usr/lib/python2.7/site-packages/haystack/management/commands/update_index.py",
>>> line 188, in handle_label
>>> self.update_backend(label, using)
>>>   File 
>>> "/usr/lib/python2.7/site-packages/haystack/management/commands/update_index.py",
>>> line 233, in update_backend
>>> do_update(backend, index, qs, start, end, total,
>>> verbosity=self.verbosity, commit=self.commit)
>>>   File 
>>> "/usr/lib/python2.7/site-packages/haystack/management/commands/update_index.py",
>>> line 96, in do_update
>>> backend.update(index, current_qs, commit=commit)
>>>   File 
>>> "/usr/lib/python2.7/site-packages/haystack/backends/whoosh_backend.py",
>>> line 196, in update
>>> doc = index.full_prepare(obj)
>>>   File "/usr/lib/python2.7/site-packages/haystack/indexes.py", line
>>> 212, in full_prepare
>>> self.prepared_data = self.prepare(obj)
>>>   File "/usr/lib/python2.7/site-packages/haystack/indexes.py", line
>>> 203, in prepare
>>> self.prepared_data[field.index_fieldname] = field.prepare(obj)
>>>   File "/usr/lib/python2.7/site-packages/haystack/fields.py", line 159,
>>> in prepare
>>> return self.convert(super(CharField, self).prepare(obj))
>>>   File "/usr/lib/python2.7/site-packages/haystack/fields.py", line 87,
>>> in prepare
>>> raise SearchFieldError("The model '%s' does not have a model_attr
>>> '%s'." % (repr(current_object), attr))
>>> SearchFieldError: The model '' does not have a model_attr
>>> 'username'.
>>> - show quoted text -
>>>   File "/usr/lib/python2.7/site-packages/haystack/fields.py", line 159,
>>> in prepare
>>> return self.convert(super(CharField, self).prepare(obj))
>>>   File "/usr/lib/python2.7/site-packages/haystack/fields.py", line 87,
>>> in prepare
>>> raise SearchFieldError("The model '%s' does not have a model_attr
>>> '%s'." % (repr(current_object), attr))
>>> haystack.exceptions.SearchFieldError: The model '' does
>>> not have a model_attr 'username'.
>>>
>>> Thanks
>>> Rob
>>>
>>> On Monday, 16 October 2017 22:46:09 UTC+1, Christian Hammond wrote:
>>>
 Hi Rob,

 Actually, this should function as a workaround for now. You can do this
 in that same Haystack file. Change the entirety of that previous code to:

 hasattr(current_object, attr)

 if hasattr(current_object, attr):
 raise SearchFieldError("The model '%s' does not have a
 model_attr '%s'." % (repr(current_object), attr))

 What will happen is the initial hasattr will trigger the crash that's
 resulting in the failure, but we're discarding the result of the initial
 one. A lucky (in this case) side-effect is that only the first call on a
 given review request will fail, and the second will succeed (internal state
 caching stuff). This should allow a full index to proceed.

 It's a temporary fix until we get the next release out.

 Christian

>>> On Mon, Oct 16, 2017 at 11:48 AM, 'Rob Backhurst' via reviewboard 

Re: Unable to download attachments that have been uploaded.

[Christian Hammond]

Hi,

Looks like your Media Path setting may be pointing to the wrong place, or
your Review Board server name setting isn't set to your domain name. Can
you check the Administration UI -> General Settings and see what those are
set to?

Christian

On Tue, Oct 17, 2017 at 10:06 AM, Not Saying  wrote:

>
>
> On Tuesday, October 17, 2017 at 10:01:17 AM UTC-7, Not Saying wrote:
>>
>> RHEL 7
>> *Review Board*  2.5.15
>>
>> *I click on the download button and it give me a page *This page can’t
>> be displayed
>> The URL is http://localhost/reviewboard/media/uploaded/files/2017/10/16
>> /4654d4a1-10f2-45c5-ab3d-6e9f37b85624__Atsugi_UH_06012017.pdf
>>
>> I see the localhost in the URL where is that defined so I can change it.
>> If I change the local host to linuxa2ng1.loc the download works.
>>
>
>
>
>
>
>> Thank you.
>>
>> --
> 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.
>



-- 
Christian Hammond
President/CEO of Beanbag 
Makers of Review Board 

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