ROX-32966: Set konflux retest command alias#18917
Conversation
|
Skipping CI for Draft Pull Request. |
There was a problem hiding this comment.
Hey - I've left some high level feedback:
- The
/konflux-retest <job>-on-pushpattern is duplicated across all Tekton pipeline manifests; consider centralizing this in a shared template or generator to avoid copy‑paste drift when job names change or new pipelines are added.
Prompt for AI Agents
Please address the comments from this code review:
## Overall Comments
- The `/konflux-retest <job>-on-push` pattern is duplicated across all Tekton pipeline manifests; consider centralizing this in a shared template or generator to avoid copy‑paste drift when job names change or new pipelines are added.Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.
|
Images are ready for the commit at 27fb4ba. To use with deploy scripts, first |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #18917 +/- ##
==========================================
- Coverage 49.52% 49.51% -0.01%
==========================================
Files 2666 2670 +4
Lines 201181 201549 +368
==========================================
+ Hits 99629 99801 +172
- Misses 94115 94294 +179
- Partials 7437 7454 +17
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
/retest scanner-v4-db-on-push |
|
/retest operator-bundle-on-push |
|
/retest central-db-on-push |
|
/retest main-on-push |
|
/retest operator-bundle-on-push |
|
/konflux-retest main-on-push |
|
/konflux-retest operator-bundle-on-push |
|
/konflux-retest create-custom-snapshot |
|
/konflux-retest operator-bundle-on-push |
Description
Set konflux retest command override to avoid getting spammed by openshift-ci when konflux retest command invoked. The
/retest <konflux-pipeline>will still work as before.Github workflow changes do not have effect until it's merged to the default branch. See testing PR below to see how these changes work.
User-facing documentation
Testing and quality
Automated testing
How I validated my change
See testing PR where this alias is merged to the default branch: stackrox/acs-fleet-manager#2573
Also tested that existing CEL conditions work as expected
I've used kubearchive for fetching pipelineruns and search was baed on commit + branch