Simplify, no need for pointers

Signed-off-by: Kimmo Lehto <[email protected]>

phase/configure_k0s.go

@@ -270,14 +270,13 @@ func (p *ConfigureK0s) configureK0s(h *cluster.Host) error {
 	return nil
 }
 
-func addUnlessExist(slice *[]string, s string) bool {
-	for _, v := range *slice {
+func addUnlessExist(slice []string, s string) bool {
+	for _, v := range slice {
 		if v == s {
-			found = true
 			return false
 		}
 	}
-	*slice = append(*slice, s)
+	slice = append(slice, s)
 	return true
 }
 
@@ -313,22 +312,22 @@ func (p *ConfigureK0s) configFor(h *cluster.Host) (string, error) {
 	case []interface{}:
 		for _, v := range oldsans {
 			if s, ok := v.(string); ok {
-				_ = addUnlessExist(&sans, s)
+				_ = addUnlessExist(sans, s)
 			}
 		}
 	case []string:
 		sans = append(sans, oldsans...)
 	}
 
-	if addUnlessExist(&sans, addr) {
+	if addUnlessExist(sans, addr) {
 		log.Infof("%s: added %s to spec.api.sans", h, addr)
 	}
 
 	for i, c := range p.Config.Spec.Hosts.Controllers() {
-		if addUnlessExist(&sans, c.Address()) {
+		if addUnlessExist(sans, c.Address()) {
 			log.Infof("%s: added controller %d address %s to spec.api.sans", h, i+1, c.Address())
 		}
-		if c.PrivateAddress != "" && addUnlessExist(&sans, c.PrivateAddress) {
+		if c.PrivateAddress != "" && addUnlessExist(sans, c.PrivateAddress) {
 			log.Infof("%s: added controller %d private address %s to spec.api.sans", h, i+1, c.PrivateAddress)
 		}
 	}