Change in ovirt-hosted-engine-ha[master]: broker: Broker should read storage info from the config.

Code Review gerrit at ovirt.org
Wed Aug 30 15:26:54 UTC 2017


>From Martin Sivák <msivak at 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 at 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 at redhat.com>
Gerrit-Reviewer: Andrej Krejcir <akrejcir at redhat.com>
Gerrit-Reviewer: Denis Chaplygin <dchaplyg at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Jenny Tokar <jtokar at redhat.com>
Gerrit-Reviewer: Martin Sivák <msivak at redhat.com>
Gerrit-Reviewer: Phillip Bailey <phbailey at redhat.com>
Gerrit-Reviewer: Roman Mohr <rmohr at redhat.com>
Gerrit-Reviewer: Simone Tiraboschi <stirabos at redhat.com>
Gerrit-Reviewer: Yanir Quinn <yquinn at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>


More information about the Engine-commits mailing list