fix: remove e2e flakiness by making the loop deterministic

Signed-off-by: AhmedGrati <ahmedgrati1999@gmail.com>

pkg/transformer/kubernetes/k8sutils.go

@@ -686,14 +686,15 @@ func getServiceGroupID(service kobject.ServiceConfig, mode string) string {
 //     A warn/info message should be printed to let the user know.
 func KomposeObjectToServiceConfigGroupMapping(komposeObject *kobject.KomposeObject, opt kobject.ConvertOptions) map[string]kobject.ServiceConfigGroup {
 	serviceConfigGroup := make(map[string]kobject.ServiceConfigGroup)
-
+	sortedServiceConfigs := SortedKeys(komposeObject.ServiceConfigs)
-	for name, service := range komposeObject.ServiceConfigs {
+	for _, service := range sortedServiceConfigs {
-		groupID := getServiceGroupID(service, opt.ServiceGroupMode)
+		serviceConfig := komposeObject.ServiceConfigs[service]
+		groupID := getServiceGroupID(serviceConfig, opt.ServiceGroupMode)
 		if groupID != "" {
-			service.Name = name
+			serviceConfig.Name = service
-			service.InGroup = true
+			serviceConfig.InGroup = true
-			serviceConfigGroup[groupID] = append(serviceConfigGroup[groupID], service)
+			serviceConfigGroup[groupID] = append(serviceConfigGroup[groupID], serviceConfig)
-			komposeObject.ServiceConfigs[name] = service
+			komposeObject.ServiceConfigs[service] = serviceConfig
 		}
 	}
 

script/test/fixtures/service-group/output-k8s.yaml

@@ -39,6 +39,14 @@ spec:
         io.kompose.service: librenms-dispatcher
     spec:
       containers:
+        - env:
+            - name: TZ
+          image: librenms/dispatcher:latest
+          name: dispatcher
+          resources: {}
+          volumeMounts:
+            - mountPath: /data
+              name: librenms-dispatcher-claim0
         - env:
             - name: TZ
           image: librenms/librenms:latest
@@ -51,15 +59,7 @@ spec:
           volumeMounts:
             - mountPath: /data
               name: librenms-dispatcher-claim0
-        - env:
+      hostname: librenms
-            - name: TZ
-          image: librenms/dispatcher:latest
-          name: dispatcher
-          resources: {}
-          volumeMounts:
-            - mountPath: /data
-              name: librenms-dispatcher-claim0
-      hostname: dispatcher
       restartPolicy: Always
       volumes:
         - name: librenms-dispatcher-claim0