Allow setting deployment strategy (#43)

* Allow 'restarting' (recreating) in addition to rollout

* Fix tests, allowing a configurable strategy

Author: naanselmo

action.yml

@@ -9,6 +9,14 @@ inputs:
     description: "The tag used for re-deploy"
     required: false
     default: "develop"
+  strategy:
+    description: "The strategy to use for this deployment"
+    required: false
+    default: "rollout"
+    type: choice
+    options:
+      - rollout
+      - restart
   url:
     description: "The re-deployment service URL"
     required: true

autodeploy.js

@@ -23,17 +23,17 @@ class AutoDeployApi {
     });
   }
 
-  redeployUrl(type, targets) {
+  redeployUrl(type, strategy, targets) {
     const encodedTargets = targets
       .split(",")
       .map((target) => encodeURIComponent(target.trim()))
       .join(",");
-    return `${this.url}/${type}/${encodedTargets}/rollout`;
+    return `${this.url}/${type}/${encodedTargets}/${strategy}`;
   }
 
-  async redeploy(type, targets, tag) {
+  async redeploy(type, strategy, targets, tag) {
     const response = await this.client.postJson(
-      this.redeployUrl(type, targets),
+      this.redeployUrl(type, strategy, targets),
       { push_data: { tag } }
     );
 

dist/index.js

@@ -6,19 +6,20 @@ async function run() {
   try {
     const pods = core.getInput("pods");
     const images = core.getInput("images");
+    const strategy = core.getInput("strategy")
     const tag = core.getInput("tag") || "develop";
     const url = core.getInput("url", { required: true });
     const token = core.getInput("token", { required: true });
     const timeout = parseInt(core.getInput("timeout") || 60000);
 
-    core.info(`Re-deploying pods ${pods} for tag ${tag}`);
+    core.info(`Re-deploying pods ${pods} for tag ${tag} with a ${strategy} strategy`);
     const api = new AutoDeployApi(url, token, timeout);
 
     if (pods) {
-      status = await api.redeploy("services", pods, tag);
+      status = await api.redeploy("services", strategy, pods, tag);
     }
     if (images) {
-      status = await api.redeploy("images", images, tag);
+      status = await api.redeploy("images", strategy, images, tag);
     }
     if(!pods && !images) {
       throw new Error("Specify either the pods or container image for which you want to trigger a redeploy.");

dist/index.js.map

@@ -13,14 +13,14 @@ test("parses authorization token", () => {
 
 test("computes correct redeploy URL", () => {
   const api = new AutoDeployApi("https://foo.bar/", "u:p");
-  expect(api.redeployUrl("services", "foo, bar baz")).toBe(
+  expect(api.redeployUrl("services", "rollout", "foo, bar baz")).toBe(
     "https://foo.bar/services/foo,bar%20baz/rollout",
   );
 });
 
 test("properly escapes image names", () => {
   const api = new AutoDeployApi("https://foo.bar/", "u:p");
-  expect(api.redeployUrl("image", "foo/bar/baz:baw")).toBe(
+  expect(api.redeployUrl("image", "rollout", "foo/bar/baz:baw")).toBe(
     "https://foo.bar/image/foo%2Fbar%2Fbaz%3Abaw/rollout",
   );
 });
@@ -33,13 +33,13 @@ function dummyApi(url) {
 
 test("returns successfully on 2xx status codes", async () => {
   const api = dummyApi("https://httpbin.org/post");
-  const status = await api.redeploy("foo", "bar");
+  const status = await api.redeploy("foo", "rollout", "bar");
   expect(status).toBe(200);
 });
 
 test("throws on error codes", async () => {
   const api = dummyApi("https://httpbin.org/status/404");
-  await expect(api.redeploy("foo", "bar")).rejects.toThrow(
+  await expect(api.redeploy("foo", "rollout", "bar")).rejects.toThrow(
     "HTTP error 404",
   );
 });

index.js

@@ -1,6 +1,6 @@
 {
   "name": "autodeploy-action",
-  "version": "1.0.0",
+  "version": "1.1.0",
   "description": "An action to automatically trigger a re-deployment of Kubernetes pods",
   "main": "index.js",
   "scripts": {