Skip to content

KONFLUX-134: Cancel in-progress PipelineRuns for updated pull requests (prod) #7355

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.

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

ralphbean
Copy link
Member

This should reduce some load on the cluster when pull requests are updated over and over again. Earlier unnecessary PipelinesRuns will be cancelled and impose less load.

This was deployed to staging a few weeks ago in #7079. This catches things up in prod.

Additionally, I re-ordered a few settings on different clusters to make them consistent, and updated the hack/new-cluster/templates/ to reflect this change.

…s (prod)

This should reduce some load on the cluster when pull requests are
updated over and over again. Earlier unnecessary PipelinesRuns will be
cancelled and impose less load.

This was deployed to staging a few weeks ago. This catches things up in
prod.

Additionally, I re-ordered a few settings on different clusters to make
them consistent, and updated the hack/new-cluster/templates/ to reflect
this change.

Signed-off-by: Ralph Bean <[email protected]>
@ralphbean ralphbean requested review from aThorp96 and manish-jangra and removed request for aThorp96 July 28, 2025 17:17
@openshift-ci openshift-ci bot requested review from enarha and Roming22 July 28, 2025 17:18
@ralphbean ralphbean requested review from aThorp96 and gbenhaim and removed request for enarha, Roming22 and manish-jangra July 28, 2025 17:18
@ralphbean ralphbean requested a review from manish-jangra July 28, 2025 17:18
Copy link
Contributor

@aThorp96 aThorp96 left a comment

Choose a reason for hiding this comment

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

/lgtm

Copy link

openshift-ci bot commented Jul 28, 2025

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: aThorp96, ralphbean

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@enkeefe00 enkeefe00 self-requested a review July 28, 2025 17:59
Copy link

openshift-ci bot commented Jul 28, 2025

@ralphbean: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
ci/prow/appstudio-e2e-tests 1f47421 link true /test appstudio-e2e-tests

Full PR test history. Your PR dashboard.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

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

Successfully merging this pull request may close these issues.

2 participants