Release 5.17.3

CHANGELOG.md

@@ -1,6 +1,11 @@
 cookbook-rb-manager CHANGELOG
 ===============
 
+## 5.17.3
+
+  - nilsver
+    - [1e5a1ab] remove dependency on serf
+
 ## 5.17.2
 
   - Pablo Pérez

resources/libraries/set_internal_vip.rb

@@ -1,20 +1,8 @@
 module RbManager
   module Helpers
-    def find_vip_from_serf
-      postgresql_master_node = node['redborder']['managers_per_services']['postgresql'].first
-      serf_output = `serf members`
-
-      master_ip = serf_output.lines.find do |line|
-        next unless line.include?('alive')
-
-        node_name = line.split[0]
-        node_name == postgresql_master_node
-      end
-
-      return unless master_ip
-
-      ip_part = master_ip.split[1]
-      ip_part&.split(':')&.first
+    def find_ip_of_master_node
+      postgres_ips = node['redborder']['cluster_info'].select { |m, _| node['redborder']['managers_per_services']['postgresql'].include?(m) }.map { |_, v| v['ipaddress_sync'] }
+      postgres_ips.first
     end
 
     def update_hosts_file(hosts_file, master_vip, service, vip)
@@ -39,7 +27,7 @@ def update_hosts_file(hosts_file, master_vip, service, vip)
 
     def set_internal_vip(vip, service)
       etc_hosts_updated = false
-      master_vip = vip || find_vip_from_serf
+      master_vip = vip || find_ip_of_master_node
       etc_hosts_updated = update_hosts_file('/etc/hosts', master_vip, service, vip) if master_vip
       etc_hosts_updated
     end

resources/metadata.rb

@@ -5,7 +5,7 @@
 maintainer_email '[email protected]'
 license          'AGPL-3.0'
 description      'Installs/Configures redborder manager'
-version          '5.17.2'
+version          '5.17.3'
 
 depends 'rb-common'
 depends 'chef-server'

resources/recipes/configure.rb

@@ -127,7 +127,6 @@
   if manager_services['postgresql'] && external_services&.dig('postgresql') == 'onpremise'
     cdomain node['redborder']['cdomain']
     ipaddress node['ipaddress_sync']
-    postgresql_hosts node['redborder']['managers_per_services']['postgresql']
     action [:add, :register]
   elsif !external_services.nil?
     action [:remove, :deregister]