Bug 973762 - [RFE] UI Plugins - Support callback for detecting runtime errors originating from plugin code
Summary: [RFE] UI Plugins - Support callback for detecting runtime errors originating ...
Keywords:
Status: CLOSED WONTFIX
Alias: None
Product: oVirt
Classification: Retired
Component: ovirt-engine-webadmin
Version: unspecified
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
: ---
Assignee: Vojtech Szocs
QA Contact: bugs@ovirt.org
URL:
Whiteboard: ux
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2013-06-12 15:59 UTC by Vojtech Szocs
Modified: 2014-06-16 15:28 UTC (History)
4 users (show)

Fixed In Version:
Doc Type: Enhancement
Doc Text:
Clone Of:
Environment:
Last Closed: 2014-06-16 15:28:20 UTC
oVirt Team: ---
Embargoed:


Attachments (Terms of Use)

Description Vojtech Szocs 2013-06-12 15:59:27 UTC
According to UI plugin lifecycle [1], if an uncaught exception escapes (originates) from plugin code due to the infrastructure being the initiator of the call (i.e. invoking specific plugin event handler function), this will be considered as internal error within the plugin code -- the infrastructure will remove such plugin from service (move plugin to FAILED state).

[1] http://www.ovirt.org/Features/UIPlugins#Plugin_lifecycle

This RFE aims to support error detection within the plugin code, i.e. introduce new event handler function that will be called by the infrastructure right before the plugin is removed from service due to uncaught exception.

For example:

api.register({
    UiInit: function() {
        // code might throw exception
    },
    PluginError: function(ex) {
        // process exception before being removed from service
        // this can be useful during plugin development
    }
});

We can also consider revisiting current approach of removing plugins out of service due to uncaught exception, i.e. swallow exception and let plugin stay in service.

Comment 1 Itamar Heim 2014-06-16 15:28:20 UTC
Closing old bugs. If this issue is still relevant/important in current version, please re-open the bug.


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