Skip to content

Commit

Permalink
Update purge workflow_dispatch logic (radius-project#7646)
Browse files Browse the repository at this point in the history
# Description

The check for creating issues was creating issues even on successes.
Tested this with manual runs and they're not creating issues

## Type of change

<!--

Please select **one** of the following options that describes your
change and delete the others. Clearly identifying the type of change you
are making will help us review your PR faster, and is used in authoring
release notes.

If you are making a bug fix or functionality change to Radius and do not
have an associated issue link please create one now.

-->

- This pull request is a minor refactor, code cleanup, test improvement,
or other maintenance task and doesn't change the functionality of Radius
(issue link optional).

<!--

Please update the following to link the associated issue. This is
required for some kinds of changes (see above).

-->

Fixes: radius-project#7645

---------

Signed-off-by: sk593 <shruthikumar@microsoft.com>
  • Loading branch information
sk593 committed May 30, 2024
1 parent 0cd0c8a commit de7033d
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion .github/workflows/purge-test-resources.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -105,7 +105,7 @@ jobs:
name: Create issue for failing purge test resources run
needs: [purge_ghcr_dev, purge_azure_resources]
runs-on: ubuntu-latest
if: always() && ${{ github.event_name != 'workflow_dispatch' }} && (needs.purge_ghcr_dev.result == 'failure' || needs.purge_azure_resources.result == 'failure')
if: failure() && ${{ github.event_name != 'workflow_dispatch' }}
steps:
- uses: actions/github-script@v7
with:
Expand Down

0 comments on commit de7033d

Please sign in to comment.