fix: sort node names to get consistent event messages

Signed-off-by: Suleyman Akbas <sakbas@redhat.com>

internal/controllers/persistent-volume-claim/controller.go

@@ -4,6 +4,7 @@ import (
 	"context"
 	"fmt"
 	"math"
+	"sort"
 	"strings"
 	"time"
 
@@ -110,6 +111,9 @@ func (r *Reconciler) Reconcile(ctx context.Context, req ctrl.Request) (ctrl.Resu
 		nodeMessage = append(nodeMessage, fmt.Sprintf("node %s has %s free storage", node.Name, prettyByteSize(capacityQuantity.Value())))
 	}
 
+	// sort the message alphabetically so we always publish the same event
+	sort.Strings(nodeMessage)
+
 	// Publish an event if the requested storage is greater than the available capacity
 	if !found {
 		msg := fmt.Sprintf("Requested storage (%s) is greater than available capacity on any node (%s).", requestedStorage.String(), strings.Join(nodeMessage, ","))