Eli Mesika has uploaded a new change for review.

Change subject: core: Report event on time drift ...(#848862)
......................................................................

core: Report event on time drift ...(#848862)

Report event on time drift between engine and vdsm(#848862)

This patch series adds time drift check per host.
New configuration values were added
EnableHostTimeDrift - indicates if time-drift check is ON (default
false)
HostTimeDriftInSec  - Max allowed time drift in seconds (default 300)
Each time that getVdsStsats is executed by the engine, it gets the host
date in GMT and compare it with local engine date, if the difference in
seconds exceeded HostTimeDriftInSec and EnableHostTimeDrift is ON, an
ALERT is written to the log.
The new configuration values are available for editing in engine-config
and the new event named VDS_TIME_DRIFT_ALERT can be managed to get
notifications on that event.

To prevent event-flood, this event is issued once an hour per Host.

Change-Id: I9544418dc49f8e9e75f6c8c7f091c96726b1f30d
Signed-off-by: Eli Mesika <emes...@redhat.com>
---
M backend/manager/dbscripts/upgrade/pre_upgrade/0000_config.sql
M 
backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/config/ConfigValues.java
2 files changed, 14 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/26/7826/1

diff --git a/backend/manager/dbscripts/upgrade/pre_upgrade/0000_config.sql 
b/backend/manager/dbscripts/upgrade/pre_upgrade/0000_config.sql
index b0a8a14..47fabdb 100644
--- a/backend/manager/dbscripts/upgrade/pre_upgrade/0000_config.sql
+++ b/backend/manager/dbscripts/upgrade/pre_upgrade/0000_config.sql
@@ -104,6 +104,8 @@
 select fn_db_add_config_value('EmulatedMachine','rhel6.0.0','3.0');
 select fn_db_add_config_value('EmulatedMachine','pc-0.14','3.1');
 select fn_db_add_config_value('EmulatedMachine','pc-0.14','3.2');
+-- Host time drift
+select fn_db_add_config_value('EnableHostTimeDrift','false','general');
 --Handling Enable Spice Root Certification Validation
 select 
fn_db_add_config_value('EnableSpiceRootCertificateValidation','true','general');
 select fn_db_add_config_value('EnableSwapCheck','true','general');
@@ -137,6 +139,7 @@
 select 
fn_db_add_config_value('GuestToolsSetupIsoPrefix','RHEV-toolsSetup_','general');
 select 
fn_db_add_config_value('HighUtilizationForEvenlyDistribute','75','general');
 select fn_db_add_config_value('HighUtilizationForPowerSave','75','general');
+select fn_db_add_config_value('HostTimeDriftInSec','300','general');
 select fn_db_add_config_value('HotPlugEnabled','false','2.2');
 select fn_db_add_config_value('HotPlugEnabled','false','3.0');
 select fn_db_add_config_value('HotPlugEnabled','true','3.1');
diff --git 
a/backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/config/ConfigValues.java
 
b/backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/config/ConfigValues.java
index 2059b36..da7b3fd 100644
--- 
a/backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/config/ConfigValues.java
+++ 
b/backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/config/ConfigValues.java
@@ -1463,6 +1463,17 @@
     @DefaultValueAttribute("true")
     EnableMACAntiSpoofingFilterRules(381),
 
+    // Host time drift
+    @Reloadable
+    @TypeConverterAttribute(Boolean.class)
+    @DefaultValueAttribute("false")
+    EnableHostTimeDrift(382),
+
+    @Reloadable
+    @TypeConverterAttribute(Integer.class)
+    @DefaultValueAttribute("300")
+    HostTimeDriftInSec(383),
+
     Invalid(65535);
 
     private int intValue;


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I9544418dc49f8e9e75f6c8c7f091c96726b1f30d
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Eli Mesika <emes...@redhat.com>
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to