Bug 726401 - Systems page - needs better error behavior when tomcat6 is not running
Summary: Systems page - needs better error behavior when tomcat6 is not running
Keywords:
Status: CLOSED UPSTREAM
Alias: None
Product: Red Hat Satellite
Classification: Red Hat
Component: WebUI
Version: 6.0.1
Hardware: Unspecified
OS: Unspecified
unspecified
low vote
Target Milestone: Unspecified
Assignee: Bryan Kearney
QA Contact: Katello QA List
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2011-07-28 13:54 UTC by Brad Buckingham
Modified: 2019-09-26 13:18 UTC (History)
4 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2013-09-19 18:16:06 UTC
Target Upstream Version:


Attachments (Terms of Use)

Description Brad Buckingham 2011-07-28 13:54:55 UTC
Description of problem:

Attempting to access Systems->Registered when tomcat6 (for candlepin) is not running generates an ugly error.  See below.

Version-Release number of selected component (if applicable):
git master

How reproducible:


Steps to Reproduce:
1. sudo /sbin/service tomcat6 stop
2. sdript/rails s
3. got to Systems->Registered in UI
  
Actual results:

rrno::ECONNREFUSED in Systems#index

Showing /home/bbucking/git/katello/src/app/views/systems/_list_system.html.haml where line #9 raised:

Connection refused - connect(2)

Extracted source (around line #9):

6:       - elsif col.to_s == 'created'
7:         = convert_time(item.created)
8:       - elsif col.to_s == 'lastCheckin'
9:         = get_checkin(item)
10:       - else
11:         = item.send(col)
12:   %div

Trace of template inclusion: app/views/systems/_list_systems.html.haml, app/views/common/_panel.html.haml, app/views/systems/index.html.haml

Rails.root: /home/bbucking/git/katello/src
Application Trace | Framework Trace | Full Trace

lib/http_resource.rb:83:in `get'
lib/resources/candlepin.rb:110:in `get'
app/models/glue/candlepin/consumer.rb:25:in `included'
app/models/lazy_accessor.rb:125:in `instance_eval'
app/models/lazy_accessor.rb:125:in `run_initializer'
app/models/lazy_accessor.rb:71:in `lastCheckin'
app/helpers/systems_helper.rb:21:in `get_checkin'
app/views/systems/_list_system.html.haml:9:in `_app_views_systems__list_system_html_haml___2092465704_70238783180780_32924596'
app/views/systems/_list_system.html.haml:2:in `each'
app/views/systems/_list_system.html.haml:2:in `_app_views_systems__list_system_html_haml___2092465704_70238783180780_32924596'
app/views/systems/_list_systems.html.haml:2:in `_app_views_systems__list_systems_html_haml___2076420724_70238783388120_5925588'
app/views/systems/_list_systems.html.haml:1:in `_app_views_systems__list_systems_html_haml___2076420724_70238783388120_5925588'
app/helpers/systems_helper.rb:17:in `render_rows'
app/views/common/_panel.html.haml:46:in `_app_views_common__panel_html_haml__2691000_70238781909580_2069205172'
app/helpers/application_helper.rb:96:in `two_panel'
app/views/systems/index.html.haml:13:in `_app_views_systems_index_html_haml___1100083565_70238782267800_0'
lib/util/threadsession.rb:77:in `thread_locals'


Expected results:
Better error handling.  Perhaps the page comes up without data and displaying error message that service is unavailable.

Additional info:

Comment 1 Mike McCune 2011-09-02 19:37:02 UTC
We need a heartbeat.

Comment 2 Mike McCune 2012-01-26 19:39:24 UTC
mass move to CFSE product.

Comment 4 Miroslav Suchý 2012-12-18 11:57:36 UTC
I do not think that we need heartbeat. If service accidentally crash, system must restart it automatically. For this we have systemD. So on recent Fedoras this should not happen (unless you explicitely stop the service). And I'm proposing to WONTFIX on RHEL6. Brad and Shannon do you agree?

Comment 5 Payal Godhani 2013-05-31 19:50:50 UTC
https://github.com/Katello/katello/pull/2412

Comment 7 Mike McCune 2013-08-16 18:20:53 UTC
getting rid of 6.0.0 version since that doesn't exist

Comment 8 Mike McCune 2013-09-19 18:16:06 UTC
These bugs have been resolved in upstream projects for a period of months so I'm mass-closing them as CLOSED:UPSTREAM.  If this is a mistake feel free to re-open.


Note You need to log in before you can comment on or make changes to this bug.