Fail safe wsgi logging

Bug #1054501 reported by Arathi
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Cinder
Invalid
Undecided
Unassigned
OpenStack Compute (nova)
Won't Fix
Low
Unassigned

Bug Description

If there are any uncaught exceptions from api controllers, they are handled in nova/api/openstack/__init__.py:FaultWrapper, logged the handled exception and HTTP 500 response is sent to the client.

There could be error while logging these uncaught exceptions and we missout them in the nova logs. In such a case, these exceptions should be logged into syslog for further investigation.

Tags: api
Changed in openstack-common:
assignee: nobody → Arathi (arathi-darshanam)
Changed in openstack-common:
status: New → In Progress
Changed in openstack-common:
status: In Progress → New
affects: openstack-common → nova
Revision history for this message
Arathi (arathi-darshanam) wrote :

For reference changes are at: https://review.openstack.org/#/c/13567/

Changed in nova:
status: New → In Progress
Changed in nova:
assignee: Arathi (arathi-darshanam) → nobody
Mathew Odden (locke105)
Changed in nova:
status: In Progress → New
tags: added: api
Changed in nova:
status: New → Confirmed
importance: Undecided → Low
Changed in nova:
assignee: nobody → Sergio Cazzolato (sergio-j-cazzolato)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to nova (master)

Fix proposed to branch: master
Review: https://review.openstack.org/60892

Changed in nova:
status: Confirmed → In Progress
Changed in cinder:
assignee: nobody → Sergio Cazzolato (sergio-j-cazzolato)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to cinder (master)

Fix proposed to branch: master
Review: https://review.openstack.org/61180

Changed in cinder:
status: New → In Progress
Changed in cinder:
assignee: Sergio Cazzolato (sergio-j-cazzolato) → nobody
Changed in nova:
assignee: Sergio Cazzolato (sergio-j-cazzolato) → nobody
Revision history for this message
Christopher Yeoh (cyeoh-0) wrote :

The first thing we do is log the exception. This happens before we attempt to return any information to the client or do any other processing. So I don't think we need any other fallback. Unless you have a testcase which demonstrates this is necessary.

Changed in nova:
status: In Progress → Won't Fix
Revision history for this message
Tom Fifield (fifieldt) wrote :

Cinder patch was abandoned

Changed in cinder:
status: In Progress → Incomplete
Changed in cinder:
status: Incomplete → Invalid
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.