Change in ovirt-hosted-engine-setup[master]: networking: set VLAN on top level network object
Sandro Bonazzola has submitted this change and it was merged.
Change subject: networking: set VLAN on top level network object
......................................................................
networking: set VLAN on top level network object
The /cluster/{cluster:id}/networks/{network:id} resource
only supports updates of the cluster itself, which is weird.
For any other update, like the VLAN, you need to sue the top
level /networks/{network:id} resource which can be addressed
only by id.
Pooling to check network status and vlan id since
setting vlan id is async.
Detecting the interface name if the management bridge
was created in the past by a previous attempt.
Change-Id: Ifcff652ef28e6b912514df2dd2daac2b07eca61e
Bug-Url: https://bugzilla.redhat.com/1291083
Signed-off-by: Simone Tiraboschi <stirabos(a)redhat.com>
---
M src/plugins/ovirt-hosted-engine-setup/engine/add_host.py
M src/plugins/ovirt-hosted-engine-setup/network/bridge.py
2 files changed, 117 insertions(+), 2 deletions(-)
Approvals:
Sandro Bonazzola: Looks good to me, approved; Passed CI tests
Simone Tiraboschi: Verified
--
To view, visit https://gerrit.ovirt.org/53064
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ifcff652ef28e6b912514df2dd2daac2b07eca61e
Gerrit-PatchSet: 9
Gerrit-Project: ovirt-hosted-engine-setup
Gerrit-Branch: master
Gerrit-Owner: Simone Tiraboschi <stirabos(a)redhat.com>
Gerrit-Reviewer: Alona Kaplan <alkaplan(a)redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: Douglas Schilling Landgraf <dougsland(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com>
Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>
Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>