Bug 1191962 - [RFE] Add dbus backend for configuration backends
Summary: [RFE] Add dbus backend for configuration backends
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: ovirt-node
Classification: oVirt
Component: RFEs
Version: ---
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ovirt-3.6.0-rc
: ---
Assignee: Ryan Barry
QA Contact: bugs@ovirt.org
URL:
Whiteboard: node
Depends On:
Blocks: node-cockpit
TreeView+ depends on / blocked
 
Reported: 2015-02-12 10:01 UTC by Fabian Deutsch
Modified: 2016-02-10 19:17 UTC (History)
15 users (show)

Fixed In Version:
Doc Type: Enhancement
Doc Text:
Clone Of:
Environment:
Last Closed: 2015-10-19 07:13:17 UTC
oVirt Team: Node
Embargoed:
ylavi: ovirt-3.6.0?
ylavi: planning_ack+
rule-engine: devel_ack+
ylavi: testing_ack?


Attachments (Terms of Use)

Description Fabian Deutsch 2015-02-12 10:01:27 UTC
Description of problem:
Currently the configuration backends can only be used through python by using the appropriate classes.
A dbus backend should be introduced which exports the classes through a dbus api.

Comment 1 Fabian Deutsch 2015-02-12 11:56:42 UTC
To not interfere with the existing code, a safe approach is to create a standalone daemon.
The daemon can take any class, introspects it, instanciates and publishes it.
In our case we can feed the config classes to the daemon, to get them published.

Besides exporting the required update() and retrieve() methods, the daemon should also export the sometimes available configure_* methods, which allow a more high level configuration.

Besides taking care of exporting the necessary methods, it must also be investigated if some kind of authorization needs to be done by the existing dbus authorization mechanisms.

Comment 2 Ryan Barry 2015-03-05 17:18:33 UTC
Waiting on an upstream repo to put this in.

Sources and testing instructions are available at:

http://github.com/evol262/node-cockpit

I tested by copying the ovirt-node python bits to a fresh VM, installed the dependencies (listed in TESTING -- you should be able to import ovirt.node.config.defaults without any errors), then installed the ovirt-node-dbus-backend RPM.

I'll add an external tracker when the repo is up

Comment 3 Fabian Deutsch 2015-03-12 11:43:00 UTC
Please move this bug to MODIFIED once the repository on gerrit is available and the patches merged.

Comment 4 Ryan Barry 2015-03-12 15:51:42 UTC
That was the plan, and normal workflow.


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