615298 - mware needs crashStorageClass defined in commonconfig.py.dist

Status: RESOLVED FIXED

(Socorro :: General, task)

Description

[Robert Helmer [:rhelmer]]

Attachment: set a default for crashStorageClass

We need this defined somewhere, in order to override it using environment variables in /etc/socorro/common.conf

On staging, the attached has been added to commonconfig.py manually. Not sure if CrashStorageSystemForHBase is a good default or not though.

Comment 1

[K Lars Lohn [:lars] [:klohn]]

Comment on attachment 493752 [details] [diff] [review]
set a default for crashStorageClass

the problem is actually just the opposite.  'crashStorageClass' is a deprecated config parameter.  Anything that is depending on it need to be fixed.  Which app was complaining that it needed it?

Comment 2

[Robert Helmer [:rhelmer]]

(In reply to comment #1)
> Comment on attachment 493752 [details] [diff] [review]
> set a default for crashStorageClass
> 
> the problem is actually just the opposite.  'crashStorageClass' is a deprecated
> config parameter.  Anything that is depending on it need to be fixed.  Which
> app was complaining that it needed it?

mware ( scripts/webservices.py ), running via WSGI:

mod_wsgi (pid=3551): Target WSGI script '/data/socorro/application/scripts/webservices.py' cannot be loaded as Python module.
mod_wsgi (pid=3551): Exception occurred processing WSGI script '/data/socorro/application/scripts/webservices.py'.
Traceback (most recent call last):
  File "/data/socorro/application/scripts/webservices.py", line 6, in <module>
    import config.webapiconfig as configModule
  File "/data/socorro/application/scripts/config/webapiconfig.py", line 9, in <module>
    from config.commonconfig import crashStorageClass
ImportError: cannot import name crashStorageClass

Comment 3

[K Lars Lohn [:lars] [:klohn]]

a fix has been checked into .../scripts/config/webapiconfig.py.dist  removing references to 'crashStorageClass'.  Please update the config in staging and try again...

Comment 4

[Robert Helmer [:rhelmer]]

(In reply to comment #3)
> a fix has been checked into .../scripts/config/webapiconfig.py.dist  removing
> references to 'crashStorageClass'.  Please update the config in staging and try
> again...

Almost.. had to remove this as well:

from config.commonconfig import crashStorageClass

Went ahead and checked in removal of that line, r2794

Thanks!