Martin Sivák has submitted this change and it was merged. Change subject: Refactor config.py ...................................................................... Refactor config.py Break the methods for loading and refreshing configuration files on to several smaller methods, that can be easily reused for other purposes. Change-Id: I0beff8c15fe6cd186bb87034c6ebe8a094a95eee Bug-Url: https://bugzilla.redhat.com/1301681 Signed-off-by: Jenny Tokar <jtokar(a)redhat.com&gt; --- M ovirt_hosted_engine_ha/env/config.py 1 file changed, 156 insertions(+), 133 deletions(-) Approvals: Martin Sivák: Looks good to me, approved Jenkins CI: Passed CI tests Jenny Tokar: Verified -- To view, visit https://gerrit.ovirt.org/67924 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I0beff8c15fe6cd186bb87034c6ebe8a094a95eee Gerrit-PatchSet: 10 Gerrit-Project: ovirt-hosted-engine-ha Gerrit-Branch: master Gerrit-Owner: Jenny Tokar <jtokar(a)redhat.com&gt; Gerrit-Reviewer: Andrej Krejcir <akrejcir(a)redhat.com&gt; Gerrit-Reviewer: Denis Chaplygin <dchaplyg(a)redhat.com&gt; Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Jenny Tokar <jtokar(a)redhat.com&gt; Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com&gt; Gerrit-Reviewer: Phillip Bailey <phbailey(a)redhat.com&gt; Gerrit-Reviewer: Roman Mohr <rmohr(a)redhat.com&gt; Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com&gt; Gerrit-Reviewer: Yanir Quinn <yquinn(a)redhat.com&gt; Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org&gt;