From Dan Kenigsberg <danken(a)redhat.com>:
Dan Kenigsberg has submitted this change and it was merged.
Change subject: net: reformat dhcp response data and always report it
......................................................................
net: reformat dhcp response data and always report it
DHCP response has been collected through the dhclient options handler.
The VDSM handler, located at /etc/dhcp/dhclient.d/sourceRoute.sh is
creating an unique file under /var/run/vdsm/sourceRoutes that contains
the IP information from the DHCP response.
This patch reformats the data representation and always reports it even
with partial data (e.g. even when the route is missing), leaving it up
to the event handler at the sourceroutethread.
This patch aims to allow (in future patches) to add actions beyond the
existing sourceroute one.
Change-Id: Ifa531e4697b0154849ef02c06100dde26c46b7eb
Signed-off-by: Edward Haas <edwardh(a)redhat.com>
---
M lib/vdsm/network/sourceroutethread.py
M vdsm/sourceRoute.sh
2 files changed, 73 insertions(+), 28 deletions(-)
Approvals:
Jenkins CI: Passed CI tests
Dan Kenigsberg: Looks good to me, approved
Edward Haas: Verified
--
To view, visit
https://gerrit.ovirt.org/69159
To unsubscribe, visit
https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ifa531e4697b0154849ef02c06100dde26c46b7eb
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Edward Haas <edwardh(a)redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: Edward Haas <edwardh(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Leon Goldberg <lgoldber(a)redhat.com>
Gerrit-Reviewer: Petr Horáček <phoracek(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>