fix(core): Always use clusters parameter for ECS server group URLs (#10171) (#10175)

* fix(core): Always use clusters parameter for ECS server group URLs

* fix(core): Extend server group URL fix to handle both ECS and AWS providers

(cherry picked from commit 3f2418e)

Co-authored-by: Shlomo Daari <104773977+shlomodaari@users.noreply.github.com>

Author: mergify[bot]

packages/core/src/navigation/UrlBuilder.ts

@@ -244,6 +244,21 @@ class ServerGroupsUrlBuilder implements IUrlBuilder {
       { inherit: false },
     );
 
+    if (
+      input.provider === 'ecs' ||
+      (input.provider && input.provider.includes('ecs')) ||
+      input.provider === 'aws' ||
+      (input.provider && input.provider.includes('aws'))
+    ) {
+      const serverGroupParts = input.serverGroup.split('-');
+      let clusterName = input.serverGroup;
+      if (serverGroupParts.length > 1 && serverGroupParts[serverGroupParts.length - 1].match(/^v\d+$/)) {
+        clusterName = serverGroupParts.slice(0, -1).join('-');
+      }
+
+      return UrlBuilderUtils.buildUrl(href, { clusters: `${input.account}:${clusterName}` });
+    }
+
     return UrlBuilderUtils.buildUrl(href, { q: input.serverGroup, acct: input.account, reg: input.region });
   }
 }