Skip to content

Commit

Permalink
Ensure that cleanup input is used on workflow_dispatch
Browse files Browse the repository at this point in the history
  • Loading branch information
jonathan-innis committed Oct 26, 2023
1 parent aaaf150 commit d7fc15b
Show file tree
Hide file tree
Showing 6 changed files with 8 additions and 31 deletions.
3 changes: 2 additions & 1 deletion .github/workflows/alpha-e2e-scale-trigger.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -38,6 +38,7 @@ jobs:
region: ${{ inputs.region || 'us-west-2' }}
enable_metrics: ${{ inputs.enable_metrics || true }}
workflow_trigger: "alpha-scale"
cleanup: ${{ inputs.cleanup || true }}
# Default to true unless using a workflow_dispatch
cleanup: ${{ github.event_name != 'workflow_dispatch' && true || inputs.cleanup }}
secrets:
SLACK_WEBHOOK_URL: ${{ secrets.SLACK_WEBHOOK_URL }}
3 changes: 2 additions & 1 deletion .github/workflows/e2e-conformance-trigger.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -37,6 +37,7 @@ jobs:
region: ${{ inputs.region || 'eu-west-1' }}
k8s_version: ${{ matrix.k8s_version }}
workflow_trigger: "conformance"
cleanup: ${{ inputs.cleanup || true }}
# Default to true unless using a workflow_dispatch
cleanup: ${{ github.event_name != 'workflow_dispatch' && true || inputs.cleanup }}
secrets:
SLACK_WEBHOOK_URL: ${{ secrets.SLACK_WEBHOOK_URL }}
3 changes: 2 additions & 1 deletion .github/workflows/e2e-matrix-trigger.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -34,6 +34,7 @@ jobs:
git_ref: ${{ needs.resolve.outputs.GIT_REF }}
region: ${{ inputs.region || 'us-east-2' }}
workflow_trigger: "matrix"
cleanup: ${{ inputs.cleanup || true }}
# Default to true unless using a workflow_dispatch
cleanup: ${{ github.event_name != 'workflow_dispatch' && true || inputs.cleanup }}
secrets:
SLACK_WEBHOOK_URL: ${{ secrets.SLACK_WEBHOOK_URL }}
3 changes: 2 additions & 1 deletion .github/workflows/e2e-scale-trigger.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -38,6 +38,7 @@ jobs:
region: ${{ inputs.region || 'us-west-2' }}
enable_metrics: ${{ inputs.enable_metrics || true }}
workflow_trigger: "scale"
cleanup: ${{ inputs.cleanup || true }}
# Default to true unless using a workflow_dispatch
cleanup: ${{ github.event_name != 'workflow_dispatch' && true || inputs.cleanup }}
secrets:
SLACK_WEBHOOK_URL: ${{ secrets.SLACK_WEBHOOK_URL }}
14 changes: 0 additions & 14 deletions .github/workflows/test-workflow-call.yaml

This file was deleted.

13 changes: 0 additions & 13 deletions .github/workflows/test.yaml

This file was deleted.

0 comments on commit d7fc15b

Please sign in to comment.