Skip to content

/ok-to-test re-runs all the pipelines #1959

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
pierDipi opened this issue Feb 25, 2025 · 3 comments · May be fixed by #2048
Open

/ok-to-test re-runs all the pipelines #1959

pierDipi opened this issue Feb 25, 2025 · 3 comments · May be fixed by #2048
Assignees
@waveywaves
Labels
bug Something isn't working good first issue Good for newcomers

Comments

@pierDipi
Copy link

pierDipi commented Feb 25, 2025 •
edited
Loading

/ok-to-test is a command used to start OpenShift CI pipelines (Prow) for PRs from untrusted users.

When I commented here openshift-knative/eventing-kafka-broker#1467 (comment) /ok-to-test, Konflux restarted all the pipelines even though they were green.

This is kinda related to #1811.

The fact that OpenShift CI / Prow use the same commands with different behaviors doesn't help either.
@chmouel
Copy link
Member

chmouel commented Feb 25, 2025

yeah that's a bug we need to fix... it should only start the non already ran PipelineRun

@chmouel chmouel added bug Something isn't working good first issue Good for newcomers labels Feb 25, 2025
@chmouel chmouel mentioned this issue Mar 5, 2025
Closed
@chmouel chmouel marked this as a duplicate of #1811 Mar 5, 2025
@chmouel
Copy link
Member

chmouel commented Mar 5, 2025

jira issue https://issues.redhat.com/browse/SRVKP-7236

@waveywaves
Copy link
Contributor

waveywaves/tekton-pac-man#2 (comment) can confirm this issue, tested here. /ok-to-test reruns the already passed PipelineRun

@waveywaves waveywaves linked a pull request Apr 17, 2025 that will close this issue
Open
13 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@waveywaves waveywaves

Labels
bug Something isn't working good first issue Good for newcomers
Milestone
No milestone
3 participants
@chmouel @waveywaves @pierDipi