Change in vdsm[master]: Revert "getRoute verb added"

danken at redhat.com danken at redhat.com
Wed May 13 09:44:14 UTC 2015


Dan Kenigsberg has submitted this change and it was merged.

Change subject: Revert "getRoute verb added"
......................................................................


Revert "getRoute verb added"

getRoute verb was introduced due to our mistake. It is not useful
for its task, was never used by Engine, and was never part of a released
version of vdsm. Instead of using it, Engine is to perform DNS resolution
of the host name and compare it to the addresses found on the interfaces
reported by getVdsCapabilities.

This reverts commit 41a839fc1d7d2e30262b961e33bf25f733a8de02.

Change-Id: Ifd1fa1c9a0af5218035affe8b41e4e24fa8202e6
Signed-off-by: Petr Horáček <phoracek at redhat.com>
Reviewed-on: https://gerrit.ovirt.org/40749
Reviewed-by: Dan Kenigsberg <danken at redhat.com>
Continuous-Integration: Jenkins CI
---
M client/vdsClient.py
M tests/functional/networkTests.py
M tests/functional/utils.py
M vdsm/API.py
M vdsm/rpc/BindingXMLRPC.py
M vdsm/rpc/Bridge.py
M vdsm/rpc/vdsmapi-schema.json
7 files changed, 1 insertion(+), 61 deletions(-)

Approvals:
  Jenkins CI: Passed CI tests
  Petr Horáček: Verified
  Dan Kenigsberg: Looks good to me, approved



-- 
To view, visit https://gerrit.ovirt.org/40749
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Ifd1fa1c9a0af5218035affe8b41e4e24fa8202e6
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Petr Horáček <phoracek at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Eliraz Levi <elevi at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Petr Horáček <phoracek at redhat.com>
Gerrit-Reviewer: automation at ovirt.org


More information about the vdsm-patches mailing list