Alon Bar-Lev has posted comments on this change. Change subject: First draft for engine-setup splitting ......................................................................
Patch Set 14: (1 comment) http://gerrit.ovirt.org/#/c/27647/14/packaging/setup/plugins/ovirt-engine-common/base/db/connection.py File packaging/setup/plugins/ovirt-engine-common/base/db/connection.py: Line 34: from ovirt_engine_setup import constants as osetupcons Line 35: from ovirt_engine_setup import engineconstants as oenginecons Line 36: # TODO: ES: fix it, now it's under ovirt-engine-common Line 37: # the engine plugin could not be here Line 38: # move it to the engine plugin? > Engine itself, DWH and reports need a connection to the Engine DB and so th websocket proxy and may be fence listener and future ca etc... are standalone, these should depend directly on base, while engine packages these that requires jboss, java, database etc, will benefit from this engine-common, this is why it was introduced. Line 39: from ovirt_engine_setup import database Line 40: Line 41: Line 42: @util.export -- To view, visit http://gerrit.ovirt.org/27647 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I4d465766f10a490bd213ca3510d02b3d6fdfa7c6 Gerrit-PatchSet: 14 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Simone Tiraboschi <[email protected]> Gerrit-Reviewer: Alon Bar-Lev <[email protected]> Gerrit-Reviewer: Sandro Bonazzola <[email protected]> Gerrit-Reviewer: Simone Tiraboschi <[email protected]> Gerrit-Reviewer: Yedidyah Bar David <[email protected]> Gerrit-Reviewer: [email protected] Gerrit-Reviewer: oVirt Jenkins CI Server Gerrit-HasComments: Yes _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
