Skip to content

ROX-32966: Add disabling label for the konflux retests#81

Open
kurlov wants to merge 1 commit intomainfrom
akurlov/ROX-32966-add-disabling-label
Open

ROX-32966: Add disabling label for the konflux retests#81
kurlov wants to merge 1 commit intomainfrom
akurlov/ROX-32966-add-disabling-label

Conversation

@kurlov
Copy link
Member

@kurlov kurlov commented Feb 4, 2026

@kurlov kurlov requested a review from a team as a code owner February 4, 2026 18:32
if: github.event_name == 'pull_request'
if: |
github.event_name == 'pull_request' &&
!contains(github.event.pull_request.labels.*.name, 'disable-konflux-auto-retest')
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Different event types and because of that a bit different label extraction

@kurlov kurlov requested a review from msugakov February 4, 2026 18:33
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')
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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)

Copy link
Contributor

@msugakov msugakov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Regarding

No description provided.

Is there any way you can test this before merging to see if it really works?

@kurlov
Copy link
Member Author

kurlov commented Feb 5, 2026

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

@msugakov
Copy link
Contributor

msugakov commented Feb 6, 2026

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants