fix(vrrp_instances) Do not duplicate the 'notify_master' script (#154)

* fix(vrrp_instances) Do not duplicate the 'notify_master' script

Author: john-scalingo

CHANGELOG.md

@@ -4,6 +4,8 @@ This file is used to list changes made in each version of the keepalived cookboo
 
 ## Unreleased
 
+- Fixed a bug that duplicated the `notify_master` script in a `vrrp_instance`
+
 ## 6.0.18 - *2024-05-03*
 
 ## 6.0.17 - *2024-05-03*

spec/resources/vrrp_instance_spec.rb

@@ -236,24 +236,35 @@ def vrrp_instance_file_name(name)
       end
 
       it('should render a config file with the notify_stop correctly') do
-        is_expected.to render_file(file_name).with_content(%r{notify_stop\s/path/to_stop\.sh})
+        is_expected.to render_file(file_name).with_content { |s|
+          expect(s.scan(%r{notify_stop\s/path/to_stop\.sh}).size).to eq(1)
+        }
       end
 
       it('should render a config file with the notify_master correctly') do
-        is_expected.to render_file(file_name).with_content(%r{notify_master\s/path/to_master\.sh})
+        is_expected.to render_file(file_name).with_content { |s|
+          expect(s.scan(%r{notify_master\s/path/to_master\.sh}).size).to eq(1)
+        }
       end
 
       it('should render a config file with the notify_backup correctly') do
-        is_expected.to render_file(file_name).with_content(%r{notify_backup\s/path/to_backup\.sh})
+        is_expected.to render_file(file_name).with_content { |s|
+          expect(s.scan(%r{notify_backup\s/path/to_backup\.sh}).size).to eq(1)
+        }
       end
 
       it('should render a config file with the notify_fault correctly') do
-        is_expected.to render_file(file_name).with_content(%r{notify_fault\s/path/fault\.sh})
+        is_expected.to render_file(file_name).with_content { |s|
+          expect(s.scan(%r{notify_fault\s/path/fault\.sh}).size).to eq(1)
+        }
       end
 
       it('should render a config file with the notify correctly') do
-        is_expected.to render_file(file_name).with_content(%r{notify\s/path/notify\.sh})
+        is_expected.to render_file(file_name).with_content { |s|
+          expect(s.scan(%r{notify\s/path/notify\.sh}).size).to eq(1)
+        }
       end
+
       it('should render a config file with the smtp_alert set to true') do
         is_expected.to render_file(file_name).with_content(/smtp_alert\strue/)
       end

templates/vrrp_instance.conf.erb

@@ -124,9 +124,6 @@ vrrp_instance <%= @instance_name %> {
 <% if @notify_master %>
   notify_master <%= @notify_master.to_s %>
 <% end %>
-<% if @notify_master %>
-  notify_master <%= @notify_master.to_s %>
-<% end %>
 <% if @notify_backup %>
   notify_backup <%= @notify_backup.to_s %>
 <% end %>