feat: allow autoscaling configuration for node pools

Copilot · fredleger · fredleger · commit 013567385dc1 · 2026-03-06T12:15:27.000+01:00
Co-authored-by: fredleger &lt;2778741+fredleger@users.noreply.github.com&gt;
diff --git a/action.yml b/action.yml
@@ -46,8 +46,22 @@ inputs:
     description: "The ID of the OVH MKS nodepool"
   number-of-nodes:
     required: true
-    description: "The number of nodes to scale to"
+    description: "The desired number of nodes to scale to"
     default: "1"
+  autoscale:
+    required: false
+    description: "Whether to enable autoscaling for the nodepool"
+    default: "true"
+  min-nodes:
+    required: false
+    description: |
+      The minimum number of nodes for autoscaling.
+      Defaults to the value of number-of-nodes if not provided.
+  max-nodes:
+    required: false
+    description: |
+      The maximum number of nodes for autoscaling.
+      Defaults to the value of number-of-nodes if not provided.
 
 outputs:
   response:
diff --git a/package.json b/package.json
@@ -85,6 +85,6 @@
     ]
   },
   "tsDevTools": {
-    "version": "20250623095500-add-prettier-oxc"
+    "version": "20250623095600-remove-prettier-oxc"
   }
-}
+}
diff --git a/src/index-runner.test.ts b/src/index-runner.test.ts
@@ -36,6 +36,9 @@ describe("run", () => {
       clusterId: "cluster-id",
       nodepoolId: "nodepool-id",
       numberOfNodes: 3,
+      autoscale: true,
+      minNodes: null,
+      maxNodes: null,
     }));
 
     scaleNodepoolMock.mockResolvedValueOnce({} as NodepoolUpdateResponse);
@@ -46,7 +49,7 @@ describe("run", () => {
     // Assert
     expect(debugMock).toHaveBeenNthCalledWith(
       1,
-      'inputs: {"endpoint":"ovh-eu","appKey":"app-key","appSecret":"app-secret","consumerKey":"consumer-key","clientId":null,"clientSecret":null,"projectId":"project-id","clusterId":"cluster-id","nodepoolId":"nodepool-id","numberOfNodes":3}'
+      'inputs: {"endpoint":"ovh-eu","appKey":"app-key","appSecret":"app-secret","consumerKey":"consumer-key","clientId":null,"clientSecret":null,"projectId":"project-id","clusterId":"cluster-id","nodepoolId":"nodepool-id","numberOfNodes":3,"autoscale":true,"minNodes":null,"maxNodes":null}'
     );
 
     expect(infoMock).toHaveBeenNthCalledWith(
@@ -59,6 +62,9 @@ describe("run", () => {
       clusterId: "cluster-id",
       nodepoolId: "nodepool-id",
       numberOfNodes: 3,
+      autoscale: true,
+      minNodes: null,
+      maxNodes: null,
     });
 
     expect(setFailedMock).not.toHaveBeenCalled();
@@ -82,6 +88,9 @@ describe("run", () => {
       clusterId: "cluster-id",
       nodepoolId: "nodepool-id",
       numberOfNodes: 3,
+      autoscale: true,
+      minNodes: null,
+      maxNodes: null,
     }));
 
     // Act
@@ -107,6 +116,9 @@ describe("run", () => {
       clusterId: "cluster-id",
       nodepoolId: "nodepool-id",
       numberOfNodes: 3,
+      autoscale: true,
+      minNodes: null,
+      maxNodes: null,
     }));
 
     // Act
diff --git a/src/index-runner.ts b/src/index-runner.ts
@@ -32,6 +32,9 @@ export async function run(): Promise<void> {
       clusterId: inputs.clusterId,
       nodepoolId: inputs.nodepoolId,
       numberOfNodes: inputs.numberOfNodes,
+      autoscale: inputs.autoscale,
+      minNodes: inputs.minNodes,
+      maxNodes: inputs.maxNodes,
     });
     loggerService.info("Nodepool scaling completed successfully.");
     setOutput("result", JSON.stringify(result));
diff --git a/src/services/input.service.test.ts b/src/services/input.service.test.ts
@@ -482,5 +482,168 @@ describe("InputService", () => {
         expect(inputs.numberOfNodes).toBeNaN();
       });
     });
+
+    describe("autoscale", () => {
+      it("should return true when autoscale input is not provided", () => {
+        getInputMock.mockImplementation((inputName) => {
+          switch (inputName) {
+            case InputNames.ProjectId:
+              return "project-123";
+            case InputNames.ClusterId:
+              return "cluster-123";
+            case InputNames.NodepoolId:
+              return "nodepool-456";
+            case InputNames.NumberOfNodes:
+              return "3";
+            case InputNames.Autoscale:
+              return "";
+            default:
+              return "";
+          }
+        });
+
+        const inputs = service.getInputs();
+
+        expect(inputs.autoscale).toBe(true);
+      });
+
+      it("should return true when autoscale input is 'true'", () => {
+        getInputMock.mockImplementation((inputName) => {
+          switch (inputName) {
+            case InputNames.ProjectId:
+              return "project-123";
+            case InputNames.ClusterId:
+              return "cluster-123";
+            case InputNames.NodepoolId:
+              return "nodepool-456";
+            case InputNames.NumberOfNodes:
+              return "3";
+            case InputNames.Autoscale:
+              return "true";
+            default:
+              return "";
+          }
+        });
+
+        const inputs = service.getInputs();
+
+        expect(inputs.autoscale).toBe(true);
+      });
+
+      it("should return false when autoscale input is 'false'", () => {
+        getInputMock.mockImplementation((inputName) => {
+          switch (inputName) {
+            case InputNames.ProjectId:
+              return "project-123";
+            case InputNames.ClusterId:
+              return "cluster-123";
+            case InputNames.NodepoolId:
+              return "nodepool-456";
+            case InputNames.NumberOfNodes:
+              return "3";
+            case InputNames.Autoscale:
+              return "false";
+            default:
+              return "";
+          }
+        });
+
+        const inputs = service.getInputs();
+
+        expect(inputs.autoscale).toBe(false);
+      });
+    });
+
+    describe("min-nodes", () => {
+      it("should return null when min-nodes input is not provided", () => {
+        getInputMock.mockImplementation((inputName) => {
+          switch (inputName) {
+            case InputNames.ProjectId:
+              return "project-123";
+            case InputNames.ClusterId:
+              return "cluster-123";
+            case InputNames.NodepoolId:
+              return "nodepool-456";
+            case InputNames.NumberOfNodes:
+              return "3";
+            default:
+              return "";
+          }
+        });
+
+        const inputs = service.getInputs();
+
+        expect(inputs.minNodes).toBeNull();
+      });
+
+      it("should return given min-nodes input as number", () => {
+        getInputMock.mockImplementation((inputName) => {
+          switch (inputName) {
+            case InputNames.ProjectId:
+              return "project-123";
+            case InputNames.ClusterId:
+              return "cluster-123";
+            case InputNames.NodepoolId:
+              return "nodepool-456";
+            case InputNames.NumberOfNodes:
+              return "3";
+            case InputNames.MinNodes:
+              return "1";
+            default:
+              return "";
+          }
+        });
+
+        const inputs = service.getInputs();
+
+        expect(inputs.minNodes).toEqual(1);
+      });
+    });
+
+    describe("max-nodes", () => {
+      it("should return null when max-nodes input is not provided", () => {
+        getInputMock.mockImplementation((inputName) => {
+          switch (inputName) {
+            case InputNames.ProjectId:
+              return "project-123";
+            case InputNames.ClusterId:
+              return "cluster-123";
+            case InputNames.NodepoolId:
+              return "nodepool-456";
+            case InputNames.NumberOfNodes:
+              return "3";
+            default:
+              return "";
+          }
+        });
+
+        const inputs = service.getInputs();
+
+        expect(inputs.maxNodes).toBeNull();
+      });
+
+      it("should return given max-nodes input as number", () => {
+        getInputMock.mockImplementation((inputName) => {
+          switch (inputName) {
+            case InputNames.ProjectId:
+              return "project-123";
+            case InputNames.ClusterId:
+              return "cluster-123";
+            case InputNames.NodepoolId:
+              return "nodepool-456";
+            case InputNames.NumberOfNodes:
+              return "3";
+            case InputNames.MaxNodes:
+              return "10";
+            default:
+              return "";
+          }
+        });
+
+        const inputs = service.getInputs();
+
+        expect(inputs.maxNodes).toEqual(10);
+      });
+    });
   });
 });
diff --git a/src/services/input.service.ts b/src/services/input.service.ts
@@ -16,6 +16,9 @@ export type Inputs = {
   clusterId: string;
   nodepoolId: string;
   numberOfNodes: number;
+  autoscale: boolean;
+  minNodes: number | null;
+  maxNodes: number | null;
 };
 
 export enum InputNames {
@@ -35,6 +38,9 @@ export enum InputNames {
   ClusterId = "cluster-id",
   NodepoolId = "nodepool-id",
   NumberOfNodes = "number-of-nodes",
+  Autoscale = "autoscale",
+  MinNodes = "min-nodes",
+  MaxNodes = "max-nodes",
 }
 
 export class InputService {
@@ -53,6 +59,9 @@ export class InputService {
       clusterId: this.getClusterId(),
       nodepoolId: this.getNodepoolId(),
       numberOfNodes: this.getNumberOfNodes(),
+      autoscale: this.getAutoscale(),
+      minNodes: this.getMinNodes(),
+      maxNodes: this.getMaxNodes(),
     };
   }
 
@@ -95,4 +104,21 @@ export class InputService {
   private getNumberOfNodes(): number {
     return parseInt(getInput(InputNames.NumberOfNodes, { required: true }), 10);
   }
+
+  private getAutoscale(): boolean {
+    const value = getInput(InputNames.Autoscale, { required: false });
+    return value === "" || value.toLowerCase() !== "false";
+  }
+
+  private getMinNodes(): number | null {
+    const value = getInput(InputNames.MinNodes, { required: false });
+    if (!value) return null;
+    return parseInt(value, 10);
+  }
+
+  private getMaxNodes(): number | null {
+    const value = getInput(InputNames.MaxNodes, { required: false });
+    if (!value) return null;
+    return parseInt(value, 10);
+  }
 }
diff --git a/src/services/ovh.service.test.ts b/src/services/ovh.service.test.ts
diff --git a/src/services/ovh.service.ts b/src/services/ovh.service.ts

Original file line number	Diff line number	Diff line change
`@@ -85,6 +85,6 @@`
`85`	`85`	`]`
`86`	`86`	`},`
`87`	`87`	`"tsDevTools": {`
`88`		`- "version": "20250623095500-add-prettier-oxc"`
	`88`	`+ "version": "20250623095600-remove-prettier-oxc"`
`89`	`89`	`}`
`90`		`-}`
	`90`	`+}`