feat(workflows): add skip option for workflow steps

Closes #1927

docs/reference/commands.md

@@ -1223,12 +1223,18 @@ workflowConfigs:
         # A description of the workflow step.
         description:
 
-        # A bash script to run. Note that the host running the workflow must have bash installed and on path. It is
-        # considered to have run successfully if it returns an exit code of 0. Any other exit code signals an error,
+        # A bash script to run. Note that the host running the workflow must have bash installed and on path.
+        # It is considered to have run successfully if it returns an exit code of 0. Any other exit code signals an
+        # error,
         # and the remainder of the workflow is aborted.
+        #
         # The script may include template strings, including references to previous steps.
         script:
 
+        # Set to true to skip this step. Use this with template conditionals to skip steps for certain environments or
+        # scenarios.
+        skip:
+
     # A list of triggers that determine when the workflow should be run, and which environment should be used (Garden
     # Enterprise only).
     triggers:

docs/reference/config.md

@@ -970,12 +970,17 @@ steps:
     # A description of the workflow step.
     description:
 
-    # A bash script to run. Note that the host running the workflow must have bash installed and on path. It is
-    # considered to have run successfully if it returns an exit code of 0. Any other exit code signals an error, and
-    # the remainder of the workflow is aborted.
+    # A bash script to run. Note that the host running the workflow must have bash installed and on path.
+    # It is considered to have run successfully if it returns an exit code of 0. Any other exit code signals an error,
+    # and the remainder of the workflow is aborted.
+    #
     # The script may include template strings, including references to previous steps.
     script:
 
+    # Set to true to skip this step. Use this with template conditionals to skip steps for certain environments or
+    # scenarios.
+    skip: false
+
 # A list of triggers that determine when the workflow should be run, and which environment should be used (Garden
 # Enterprise only).
 triggers:
@@ -1218,13 +1223,33 @@ A description of the workflow step.
 
 [steps](#steps) > script
 
-A bash script to run. Note that the host running the workflow must have bash installed and on path. It is considered to have run successfully if it returns an exit code of 0. Any other exit code signals an error, and the remainder of the workflow is aborted.
+A bash script to run. Note that the host running the workflow must have bash installed and on path.
+It is considered to have run successfully if it returns an exit code of 0. Any other exit code signals an error,
+and the remainder of the workflow is aborted.
+
 The script may include template strings, including references to previous steps.
 
 | Type     | Required |
 | -------- | -------- |
 | `string` | No       |
 
+### `steps[].skip`
+
+[steps](#steps) > skip
+
+Set to true to skip this step. Use this with template conditionals to skip steps for certain environments or scenarios.
+
+| Type      | Default | Required |
+| --------- | ------- | -------- |
+| `boolean` | `false` | No       |
+
+Example:
+
+```yaml
+steps:
+  - skip: "${environment.name != 'prod'}"
+```
+
 ### `triggers[]`
 
 A list of triggers that determine when the workflow should be run, and which environment should be used (Garden Enterprise only).

garden-service/src/commands/run/workflow.ts

@@ -97,6 +97,17 @@ export class RunWorkflowCommand extends Command<Args, {}> {
       const stepBodyLog = outerLog.placeholder({ indent: 1, metadata })
       const stepFooterLog = outerLog.placeholder({ indent: 1, metadata })
       garden.log.setState({ metadata })
+
+      if (step.skip) {
+        stepBodyLog.setState(chalk.yellow(`Skipping`))
+        result.steps[stepName] = {
+          number: index + 1,
+          outputs: {},
+          log: "",
+        }
+        continue
+      }
+
       let stepResult: CommandResult
       const inheritedOpts = cloneDeep(opts)
 

garden-service/src/config/workflow.ts

@@ -136,6 +136,7 @@ export interface WorkflowStepSpec {
   command?: string[]
   description?: string
   script?: string
+  skip?: boolean
 }
 
 export const workflowStepSchema = () => {
@@ -175,14 +176,21 @@ export const workflowStepSchema = () => {
         .example(["run", "task", "my-task"]),
       description: joi.string().description("A description of the workflow step."),
       script: joi.string().description(
-        deline`
+        dedent`
         A bash script to run. Note that the host running the workflow must have bash installed and on path.
         It is considered to have run successfully if it returns an exit code of 0. Any other exit code signals an error,
         and the remainder of the workflow is aborted.
 
         The script may include template strings, including references to previous steps.
         `
       ),
+      skip: joi
+        .boolean()
+        .default(false)
+        .description(
+          `Set to true to skip this step. Use this with template conditionals to skip steps for certain environments or scenarios.`
+        )
+        .example("${environment.name != 'prod'}"),
     })
     .xor("command", "script")
     .description("A workflow step. Must specify either `command` or `script` (but not both).")

garden-service/test/unit/src/commands/run/workflow.ts

@@ -473,6 +473,28 @@ describe("RunWorkflowCommand", () => {
     expect(result?.steps["step-1"].log).to.equal(garden.projectRoot)
   })
 
+  it("should skip disabled steps", async () => {
+    garden.setWorkflowConfigs([
+      {
+        apiVersion: DEFAULT_API_VERSION,
+        name: "workflow-a",
+        kind: "Workflow",
+        path: garden.projectRoot,
+        files: [],
+        steps: [{ script: "pwd" }, { script: "echo fail!; exit 1", skip: true }],
+      },
+    ])
+
+    await cmd.action({ ...defaultParams, args: { workflow: "workflow-a" } })
+
+    const { result, errors } = await cmd.action({ ...defaultParams, args: { workflow: "workflow-a" } })
+
+    expect(result).to.exist
+    expect(errors).to.not.exist
+    expect(result?.steps["step-2"].outputs).to.eql({})
+    expect(result?.steps["step-2"].log).to.equal("")
+  })
+
   it("should collect log outputs, including stderr, from a script step", async () => {
     garden.setWorkflowConfigs([
       {

garden-service/test/unit/src/config/workflow.ts

@@ -41,7 +41,10 @@ describe("resolveWorkflowConfig", () => {
       name: "workflow-a",
       path: "/tmp/foo",
       description: "Sample workflow",
-      steps: [{ description: "Deploy the stack", command: ["deploy"] }, { command: ["test"] }],
+      steps: [
+        { description: "Deploy the stack", command: ["deploy"], skip: false },
+        { command: ["test"], skip: false },
+      ],
       triggers: [
         {
           environment: "local",
@@ -68,7 +71,10 @@ describe("resolveWorkflowConfig", () => {
       name: "workflow-a",
       path: "/tmp/foo",
       description: "Secret: ${secrets.foo}, var: ${variables.foo}",
-      steps: [{ description: "Deploy the stack", command: ["deploy"] }, { command: ["test"] }],
+      steps: [
+        { description: "Deploy the stack", command: ["deploy"], skip: false },
+        { command: ["test"], skip: false },
+      ],
     }
 
     expect(resolveWorkflowConfig(garden, config)).to.eql({
@@ -87,7 +93,14 @@ describe("resolveWorkflowConfig", () => {
       steps: [{ description: "Deploy the stack", command: ["deploy"] }, { command: ["test"] }],
     }
 
-    expect(resolveWorkflowConfig(garden, config)).to.eql({ ...config, ...defaults })
+    expect(resolveWorkflowConfig(garden, config)).to.eql({
+      ...config,
+      ...defaults,
+      steps: [
+        { description: "Deploy the stack", command: ["deploy"], skip: false },
+        { command: ["test"], skip: false },
+      ],
+    })
   })
 
   it("should throw if a step uses an invalid/unsupported command", async () => {