From Dan Kenigsberg <danken(a)redhat.com>:
Dan Kenigsberg has submitted this change and it was merged. (
https://gerrit.ovirt.org/80805 )
Change subject: vdsm tool configurators: bond_defaults - do not depend on time
......................................................................
vdsm tool configurators: bond_defaults - do not depend on time
The bond defaults dump is considered updated if it exists (as it is
created per boot, and its content is dependent on the kernel).
The existing time check (verifying if the period elapsed from the file
update is no older then the time elapsed from the system boot) is no
longer needed, as the dump is placed under /var/run which is assured
to be cleaned on boot.
This patch removes the time checks which are not valid when the time is
updated during the boot.
Change-Id: Ia32455d36d600410a0b792f4e7045b97a08afc70
Bug-Url:
https://bugzilla.redhat.com/1482014
Signed-off-by: Edward Haas <edwardh(a)redhat.com>
---
M lib/vdsm/tool/configurators/bond_defaults.py
1 file changed, 1 insertion(+), 17 deletions(-)
Approvals:
Jenkins CI: Passed CI tests
Dan Kenigsberg: Looks good to me, approved
Leon Goldberg: Looks good to me, but someone else must approve
Edward Haas: Verified
--
To view, visit
https://gerrit.ovirt.org/80805
To unsubscribe, visit
https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ia32455d36d600410a0b792f4e7045b97a08afc70
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-4.1
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: gerrit-hooks <automation(a)ovirt.org>