Note: This bug is displayed in read-only format because the product is no longer active in Red Hat Bugzilla.

Bug 1191962

Summary: [RFE] Add dbus backend for configuration backends
Product: [oVirt] ovirt-node Reporter: Fabian Deutsch <fdeutsch>
Component: RFEsAssignee: Ryan Barry <rbarry>
Status: CLOSED NEXTRELEASE QA Contact: bugs <bugs>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: ---CC: bugs, cshao, ecohen, fdeutsch, gklein, huiwa, leiwang, lsurette, mgoldboi, ovirt-bugs, rbalakri, rbarry, yaniwang, ycui, yeylon
Target Milestone: ovirt-3.6.0-rcKeywords: FutureFeature
Target Release: ---Flags: ylavi: ovirt-3.6.0?
ylavi: planning_ack+
rule-engine: devel_ack+
ylavi: testing_ack?
Hardware: Unspecified   
OS: Unspecified   
Whiteboard: node
Fixed In Version: Doc Type: Enhancement
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2015-10-19 07:13:17 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: Node RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 1190758    

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.