ROX-32966: Add disabling label for the konflux retests #81
Conversation
| if: github.event_name == 'pull_request' | ||
| if: | | ||
| github.event_name == 'pull_request' && | ||
| !contains(github.event.pull_request.labels.*.name, 'disable-konflux-auto-retest') |
There was a problem hiding this comment.
Different event types and because of that a bit different label extraction
| github.event.check_run.conclusion == 'failure' && | ||
| startsWith(github.event.check_run.name, 'Red Hat Konflux') | ||
| startsWith(github.event.check_run.name, 'Red Hat Konflux') && | ||
| !contains(github.event.check_run.pull_requests[0].labels.*.name, 'disable-konflux-auto-retest') |
There was a problem hiding this comment.
we know that there is a least one PR associated with the check (see check above). My understanding is that the Konflux check might have multiple PRs only when the same commit is used in those PRs. I assume it's quite rare and unique scenario so I believe [0] is correct instead of * (we don't want to disable if at least one PR has label)
msugakov
left a comment
msugakov
left a comment
There was a problem hiding this comment.
Regarding
No description provided.
Is there any way you can test this before merging to see if it really works?
I planned to test it with this PR, but looks like konflux is not starting for that repo ATM. https://github.com/stackrox/test-gh-actions/pull/236. Looks like this testing Konflux app was deleted. I will add a new one |
2 participants
Testing: https://github.com/stackrox/test-gh-actions/pull/240