From Martin Sivák <msivak(a)redhat.com>:
Martin Sivák has submitted this change and it was merged. (
https://gerrit.ovirt.org/81011
)
Change subject: broker: Broker should read storage info from the config.
......................................................................
broker: Broker should read storage info from the config.
As we do not support multiple storages now, it makes no sense
to accept storage info from the agent. Therefore, broker must be
able to get storage info (and construct relevant objects) on it's own,
using a config file. This also makes set-storage RPC call useless,
so it is removed.
This also removes support for different storage backends in broker,
so FilesystemBackend is not supported anymore.
https://github.com/oVirt/ovirt-site/pull/1157
Change-Id: I7dec5f52689d5051af4e2ec0e56401fd65fa1d6b
Signed-off-by: Denis Chaplygin <dchaplyg(a)redhat.com>
---
M ovirt_hosted_engine_ha/agent/hosted_engine.py
M ovirt_hosted_engine_ha/broker/listener.py
M ovirt_hosted_engine_ha/broker/storage_broker.py
M ovirt_hosted_engine_ha/client/client.py
M ovirt_hosted_engine_ha/env/constants.py.in
M ovirt_hosted_engine_ha/lib/brokerlink.py
6 files changed, 37 insertions(+), 147 deletions(-)
Approvals:
Denis Chaplygin: Verified
Martin Sivák: Looks good to me, approved
Jenkins CI: Passed CI tests
--
To view, visit
https://gerrit.ovirt.org/81011
To unsubscribe, visit
https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I7dec5f52689d5051af4e2ec0e56401fd65fa1d6b
Gerrit-PatchSet: 8
Gerrit-Project: ovirt-hosted-engine-ha
Gerrit-Branch: master
Gerrit-Owner: Denis Chaplygin <dchaplyg(a)redhat.com>
Gerrit-Reviewer: Andrej Krejcir <akrejcir(a)redhat.com>
Gerrit-Reviewer: Denis Chaplygin <dchaplyg(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Jenny Tokar <jtokar(a)redhat.com>
Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com>
Gerrit-Reviewer: Phillip Bailey <phbailey(a)redhat.com>
Gerrit-Reviewer: Roman Mohr <rmohr(a)redhat.com>
Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com>
Gerrit-Reviewer: Yanir Quinn <yquinn(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>