Re: [galaxy-dev] report server failing - lib/galaxy/webapps/reports/config.py does not have attribute sentry_dsn

2013-02-11 Thread Dave Bouvier

Jim,

Thanks for spotting that, I've pushed the fix in 8816:e5dcefc328bb.

   --Dave B.

On 2/11/13 20:14:13.000, Jim Johnson wrote:

File
'/home/galaxy/gx/prod/galaxy/database/compiled_templates/reports/base/base_panels.mako.py',
line 68 in render_body
   __M_writer(unicode(self.javascripts()))
File
'/home/galaxy/gx/prod/galaxy/database/compiled_templates/reports/webapps/reports/index.mako.py',
line 104 in render_javascripts
   __M_writer(unicode(parent.javascripts()))
File
'/home/galaxy/gx/prod/galaxy/database/compiled_templates/reports/base/base_panels.mako.py',
line 301 in render_javascripts
   if app.config.sentry_dsn:
AttributeError: 'Configuration' object has no attribute 'sentry_dsn'

I patched in this as a temporary work around:

$ hg diff -r 8182 lib/galaxy/webapps/reports/config.py
diff -r ec51a727a497 lib/galaxy/webapps/reports/config.py
--- a/lib/galaxy/webapps/reports/config.py  Thu Nov 01 23:25:39 2012
-0700
+++ b/lib/galaxy/webapps/reports/config.py  Mon Feb 11 19:06:30 2013
-0600
@@ -45,6 +45,8 @@
  global_conf_parser = ConfigParser.ConfigParser()
  if global_conf and "__file__" in global_conf:
  global_conf_parser.read(global_conf['__file__'])
+# Error logging with sentry
+self.sentry_dsn = kwargs.get( 'sentry_dsn', None )
  def get( self, key, default ):
  return self.config_dict.get( key, default )
  def check( self ):


___
Please keep all replies on the list by using "reply all"
in your mail client.  To manage your subscriptions to this
and other Galaxy lists, please use the interface at:

  http://lists.bx.psu.edu/

___
Please keep all replies on the list by using "reply all"
in your mail client.  To manage your subscriptions to this
and other Galaxy lists, please use the interface at:

 http://lists.bx.psu.edu/


[galaxy-dev] report server failing - lib/galaxy/webapps/reports/config.py does not have attribute sentry_dsn

2013-02-11 Thread Jim Johnson

File 
'/home/galaxy/gx/prod/galaxy/database/compiled_templates/reports/base/base_panels.mako.py',
 line 68 in render_body
  __M_writer(unicode(self.javascripts()))
File 
'/home/galaxy/gx/prod/galaxy/database/compiled_templates/reports/webapps/reports/index.mako.py',
 line 104 in render_javascripts
  __M_writer(unicode(parent.javascripts()))
File 
'/home/galaxy/gx/prod/galaxy/database/compiled_templates/reports/base/base_panels.mako.py',
 line 301 in render_javascripts
  if app.config.sentry_dsn:
AttributeError: 'Configuration' object has no attribute 'sentry_dsn'

I patched in this as a temporary work around:

$ hg diff -r 8182 lib/galaxy/webapps/reports/config.py
diff -r ec51a727a497 lib/galaxy/webapps/reports/config.py
--- a/lib/galaxy/webapps/reports/config.py  Thu Nov 01 23:25:39 2012 -0700
+++ b/lib/galaxy/webapps/reports/config.py  Mon Feb 11 19:06:30 2013 -0600
@@ -45,6 +45,8 @@
 global_conf_parser = ConfigParser.ConfigParser()
 if global_conf and "__file__" in global_conf:
 global_conf_parser.read(global_conf['__file__'])
+# Error logging with sentry
+self.sentry_dsn = kwargs.get( 'sentry_dsn', None )
 def get( self, key, default ):
 return self.config_dict.get( key, default )
 def check( self ):


___
Please keep all replies on the list by using "reply all"
in your mail client.  To manage your subscriptions to this
and other Galaxy lists, please use the interface at:

 http://lists.bx.psu.edu/