Merge pull request #366 from gthiemonge/fix_ip_sort

Fix net.SortIPs calls
openstack-k8s-operators · Aug 14, 2024 · 7d738db · 7d738db
2 parents 3501057 + e8b9edd
commit 7d738db
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/controllers/amphoracontroller_controller.go b/controllers/amphoracontroller_controller.go
@@ -600,8 +600,8 @@ func (r *OctaviaAmphoraControllerReconciler) generateServiceConfigMaps(
 			rsyslogIPAddresses = append(rsyslogIPAddresses, fmt.Sprintf("%s:514", val))
 		}
 	}
-	net.SortIPs(ipAddresses)
-	net.SortIPs(rsyslogIPAddresses)
+	ipAddresses = net.SortIPs(ipAddresses)
+	rsyslogIPAddresses = net.SortIPs(rsyslogIPAddresses)
 	ipAddressString := strings.Join(ipAddresses, ",")
 	templateParameters["ControllerIPList"] = ipAddressString
 	templateParameters["AdminLogTargetList"] = strings.Join(rsyslogIPAddresses, ",")