Change in vdsm[master]: vdsClient: getRoute verb added

danken at redhat.com danken at redhat.com
Thu Sep 4 23:26:33 UTC 2014


Dan Kenigsberg has posted comments on this change.

Change subject: vdsClient: getRoute verb added
......................................................................


Patch Set 1: Code-Review-1

Toni, I thought you wanted to extend the output in the future to include more information about the route,

Petr, please include a functional test for the new verb. The test should verify that it works well for ipv6 addresses, as well as ipv4 ones.

-- 
To view, visit http://gerrit.ovirt.org/32115
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Ia060ced7e9b13b1c461898f24170c49feab4382b
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Petr Horáček <phoracek at redhat.com>
Gerrit-Reviewer: Antoni Segura Puimedon <asegurap at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No


More information about the vdsm-patches mailing list