From ed0da146529c5a38c32ac346335138baab428bac Mon Sep 17 00:00:00 2001
From: fett0 <fernando.gmaidana@gmail.com>
Date: Thu, 19 Jan 2023 18:24:14 +0000
Subject:  T4939: VRRP startup delay for bonding fix

---
 data/templates/high-availability/keepalived.conf.j2 |  5 +++++
 interface-definitions/high-availability.xml.in      | 19 +++++++++++++++++++
 smoketest/scripts/cli/test_ha_vrrp.py               |  6 ++++++
 3 files changed, 30 insertions(+)

diff --git a/data/templates/high-availability/keepalived.conf.j2 b/data/templates/high-availability/keepalived.conf.j2
index ebff52e1f..4465f8640 100644
--- a/data/templates/high-availability/keepalived.conf.j2
+++ b/data/templates/high-availability/keepalived.conf.j2
@@ -5,6 +5,11 @@
 global_defs {
     dynamic_interfaces
     script_user root
+{% if vrrp.global_parameters is vyos_defined %}
+{%     if vrrp.global_parameters.startup_delay is vyos_defined %}
+    vrrp_startup_delay {{ vrrp.global_parameters.startup_delay }}
+{%     endif %}
+{% endif %}
     notify_fifo /run/keepalived/keepalived_notify_fifo
     notify_fifo_script /usr/libexec/vyos/system/keepalived-fifo.py
 }
diff --git a/interface-definitions/high-availability.xml.in b/interface-definitions/high-availability.xml.in
index d67a142d1..09ebfdf39 100644
--- a/interface-definitions/high-availability.xml.in
+++ b/interface-definitions/high-availability.xml.in
@@ -11,6 +11,25 @@
           <help>Virtual Router Redundancy Protocol settings</help>
         </properties>
         <children>
+          <node name="global-parameters">
+            <properties>
+              <help>VRRP Globas parameters</help>
+            </properties>
+            <children>
+              <leafNode name="startup-delay">
+                <properties>
+                  <help>Time vrrp startup process (in seconds)</help>
+                  <valueHelp>
+                    <format>u32:1-600</format>
+                    <description>Interval in seconds</description>
+                  </valueHelp>
+                  <constraint>
+                    <validator name="numeric" argument="--range 1-600"/>
+                  </constraint>
+                </properties>
+              </leafNode>
+            </children>
+          </node>
           <tagNode name="group">
             <properties>
               <help>VRRP group</help>
diff --git a/smoketest/scripts/cli/test_ha_vrrp.py b/smoketest/scripts/cli/test_ha_vrrp.py
index 68905e447..fba1f4687 100755
--- a/smoketest/scripts/cli/test_ha_vrrp.py
+++ b/smoketest/scripts/cli/test_ha_vrrp.py
@@ -87,11 +87,13 @@ class TestVRRP(VyOSUnitTestSHIM.TestCase):
         advertise_interval = '77'
         priority = '123'
         preempt_delay = '400'
+        startup_delay = '120'
 
         for group in groups:
             vlan_id = group.lstrip('VLAN')
             vip = f'100.64.{vlan_id}.1/24'
             group_base = base_path + ['vrrp', 'group', group]
+            global_param_base = base_path + ['vrrp', 'global-parameters']
 
             self.cli_set(['interfaces', 'ethernet', vrrp_interface, 'vif', vlan_id, 'address', inc_ip(vip, 1) + '/' + vip.split('/')[-1]])
 
@@ -110,6 +112,10 @@ class TestVRRP(VyOSUnitTestSHIM.TestCase):
             self.cli_set(group_base + ['authentication', 'type', 'plaintext-password'])
             self.cli_set(group_base + ['authentication', 'password', f'{group}'])
 
+            # Global parameters
+            config = getConfig(f'global_defs')
+            self.cli_set(global_param_base + ['startup-delay', f'{startup_delay}'])
+
         # commit changes
         self.cli_commit()
 
-- 
cgit v1.2.3