Porting replica count validation changes to release (#12289)

Author: shigupt202

Tasks/KubernetesManifestV0/Strings/resources.resjson/en-US/resources.resjson

@@ -88,7 +88,7 @@
   "loc.messages.UnableToCreateTrafficSplitManifestFile": "Unable to create TrafficSplit manifest file. %s",
   "loc.messages.StableSpecSelectorNotExist": "Resource %s not deployed using SMI canary deployment.",
   "loc.messages.InvalidPercentage": "Invalid value for percentage.",
-  "loc.messages.InvalidBaselineAndCanaryReplicas": "Invalid value for replica count. Enter a value greater than 0.",
+  "loc.messages.InvalidBaselineAndCanaryReplicas": "Invalid value for replica count.",
   "loc.messages.InvalidTimeoutValue": "Invalid value for timeout. Enter a valid number.",
   "loc.messages.RolloutStatusTimedout": "Rollout status check failed.",
   "loc.messages.EnvironmentLink": "For more information, go to %s"

Tasks/KubernetesManifestV0/src/models/TaskInputParameters.ts

@@ -43,7 +43,7 @@ export function validateCanaryPercentage() {
 }
 
 export function validateReplicaCount() {
-    if (deploymentStrategy.toUpperCase() === canaryDeploymentHelper.CANARY_DEPLOYMENT_STRATEGY && trafficSplitMethod.toUpperCase() === canaryDeploymentHelper.TRAFFIC_SPLIT_STRATEGY && !validateRegex("^[1-9]\\d*$", baselineAndCanaryReplicas)) {
+    if (deploymentStrategy.toUpperCase() === canaryDeploymentHelper.CANARY_DEPLOYMENT_STRATEGY && trafficSplitMethod.toUpperCase() === canaryDeploymentHelper.TRAFFIC_SPLIT_STRATEGY && !validateRegex("(^([0-9]|([1-9]\\d*))$)", baselineAndCanaryReplicas)) {
         throw new Error(tl.loc('InvalidBaselineAndCanaryReplicas'));
     }
 }

Tasks/KubernetesManifestV0/task.json

@@ -14,7 +14,7 @@
     "version": {
         "Major": 0,
         "Minor": 164,
-        "Patch": 2
+        "Patch": 3
     },
     "demands": [],
     "groups": [],
@@ -100,8 +100,8 @@
             "defaultValue": 1,
             "visibleRule": "strategy = Canary && action = deploy && trafficSplitMethod = SMI",
             "validation": {
-                "expression": "isMatch(value, '(^[1-9]\\d*$)','Multiline')",
-                "message": "Enter valid value greater than 0."
+                "expression": "isMatch(value, '(^([0-9]|([1-9]\\d*))$)','Multiline')",
+                "message": "Enter valid value for baseline and canary replica count."
             }
         },
         {
@@ -376,7 +376,7 @@
         "UnableToCreateTrafficSplitManifestFile": "Unable to create TrafficSplit manifest file. %s",
         "StableSpecSelectorNotExist": "Resource %s not deployed using SMI canary deployment.",
         "InvalidPercentage": "Invalid value for percentage.",
-        "InvalidBaselineAndCanaryReplicas": "Invalid value for replica count. Enter a value greater than 0.",
+        "InvalidBaselineAndCanaryReplicas": "Invalid value for replica count.",
         "InvalidTimeoutValue": "Invalid value for timeout. Enter a valid number.",
         "RolloutStatusTimedout": "Rollout status check failed.",
         "EnvironmentLink": "For more information, go to %s"

Tasks/KubernetesManifestV0/task.loc.json

@@ -14,7 +14,7 @@
   "version": {
     "Major": 0,
     "Minor": 164,
-    "Patch": 2
+    "Patch": 3
   },
   "demands": [],
   "groups": [],
@@ -101,8 +101,8 @@
       "defaultValue": 1,
       "visibleRule": "strategy = Canary && action = deploy && trafficSplitMethod = SMI",
       "validation": {
-        "expression": "isMatch(value, '(^[1-9]\\d*$)','Multiline')",
-        "message": "Enter valid value greater than 0."
+        "expression": "isMatch(value, '(^([0-9]|([1-9]\\d*))$)','Multiline')",
+        "message": "Enter valid value for baseline and canary replica count."
       }
     },
     {