moved lowercase check to beginning

Azure · Nov 27, 2024 · 06444fa · 06444fa
1 parent 19fb963
commit 06444fa
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 3 deletions.
diff --git a/src/actions/deploy.ts b/src/actions/deploy.ts
@@ -39,9 +39,10 @@ export async function deploy(
 
    // check manifest stability
    core.startGroup('Checking manifest stability')
-   const resourceType =
+   const resourceType = (
       core.getInput('resource-type') ||
       'Microsoft.ContainerService/managedClusters'
+   ).toLowerCase()
    const resourceTypes: Resource[] = getResources(
       deployedManifestFiles,
       models.DEPLOYMENT_TYPES.concat([

diff --git a/src/actions/promote.ts b/src/actions/promote.ts
@@ -166,9 +166,10 @@ async function promoteBlueGreen(kubectl: Kubectl, manifests: string[]) {
 
    // checking stability of newly created deployments
    core.startGroup('Checking manifest stability')
-   const resourceType =
+   const resourceType = (
       core.getInput('resource-type') ||
       'Microsoft.ContainerService/managedClusters'
+   ).toLowerCase()
    const deployedManifestFiles = deployResult.manifestFiles
    const resources: Resource[] = getResources(
       deployedManifestFiles,

diff --git a/src/utilities/manifestStabilityUtils.ts b/src/utilities/manifestStabilityUtils.ts
@@ -13,7 +13,7 @@ export async function checkManifestStability(
    resourceType: string
 ): Promise<void> {
    // Skip if resource type is microsoft.containerservice/fleets
-   if (resourceType.toLowerCase() === 'microsoft.containerservice/fleets') {
+   if (resourceType === 'microsoft.containerservice/fleets') {
       core.info(
          'Skipping checkManifestStability for microsoft.containerservice/fleets'
       )