From Dan Kenigsberg <dan...@redhat.com>:

Dan Kenigsberg has submitted this change and it was merged. ( 
https://gerrit.ovirt.org/80750 )

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 <edwa...@redhat.com>
---
M lib/vdsm/tool/configurators/bond_defaults.py
1 file changed, 1 insertion(+), 17 deletions(-)

Approvals:
  Nir Soffer: Looks good to me, but someone else must approve
  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/80750
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Ia32455d36d600410a0b792f4e7045b97a08afc70
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Edward Haas <edwa...@redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <dan...@redhat.com>
Gerrit-Reviewer: Edward Haas <edwa...@redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Leon Goldberg <lgold...@redhat.com>
Gerrit-Reviewer: Nir Soffer <nsof...@redhat.com>
Gerrit-Reviewer: Petr Horáček <phora...@redhat.com>
Gerrit-Reviewer: gerrit-hooks <automat...@ovirt.org>
_______________________________________________
vdsm-patches mailing list -- vdsm-patches@lists.fedorahosted.org
To unsubscribe send an email to vdsm-patches-le...@lists.fedorahosted.org

Reply via email to