Change in otopi[master]: network: firewalld: query version from python client

[Alon Bar-Lev]

Alon Bar-Lev has submitted this change and it was merged.

Change subject: network: firewalld: query version from python client
......................................................................


network: firewalld: query version from python client

querying firewalld version is required as firewalld interfaces (both api
and cli) are unstable.

current implementation queries firewalld version using command-line,
however, in order to query the version the firewalld service must be
started.

starting firewalld in order to detect firewalld is bad requirement,
it has also some side effects, as once firewalld is started we actually
effect the host.

new implementation uses an undocumented firewalld python library to
query the client version, which should be sufficient to our purposes.

Change-Id: I3db43d6dc808261a1f687ea29fe23bd752ffade8
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=988075
Signed-off-by: Alon Bar-Lev <alonbl at redhat.com>
---
M src/plugins/otopi/network/firewalld.py
1 file changed, 23 insertions(+), 31 deletions(-)

Approvals:
  Alon Bar-Lev: Verified; Looks good to me, approved
  Sandro Bonazzola: Looks good to me, but someone else must approve


-- 
To view, visit http://gerrit.ovirt.org/17302
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I3db43d6dc808261a1f687ea29fe23bd752ffade8
Gerrit-PatchSet: 1
Gerrit-Project: otopi
Gerrit-Branch: master
Gerrit-Owner: Alon Bar-Lev <alonbl at redhat.com>
Gerrit-Reviewer: Alon Bar-Lev <alonbl at redhat.com>
Gerrit-Reviewer: Sandro Bonazzola <sbonazzo at redhat.com>