Skip to content

SRVKP-12204: Chery-pick PR for fix from master to main - #1287

Merged
anwesha-palit-redhat merged 1 commit into
openshift-pipelines:mainfrom
ankrsinha:fix/SRVKP-12204-main
Aug 18, 2026
Merged

SRVKP-12204: Chery-pick PR for fix from master to main#1287
anwesha-palit-redhat merged 1 commit into
openshift-pipelines:mainfrom
ankrsinha:fix/SRVKP-12204-main

Conversation

@ankrsinha

Copy link
Copy Markdown
Contributor

This PR cherry-picks the SRVKP-12204 fix (PR #1275) from master to main.

Screen Recordings / Screenshot

Screencast.From.2026-08-17.14-59-00.mp4

@openshift-ci-robot

openshift-ci-robot commented Aug 17, 2026

Copy link
Copy Markdown
Collaborator

@ankrsinha: This pull request references SRVKP-12204 which is a valid jira issue.

Warning: The referenced jira issue has an invalid target version for the target branch this PR targets: expected the bug to target the "5.1.0" version, but no target version was set.

Details

In response to this:

This PR cherry-picks the SRVKP-12204 fix (PR #1275) from master to main.

Screen Recordings / Screenshot

Screencast.From.2026-08-17.14-59-00.mp4

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 openshift-eng/jira-lifecycle-plugin repository.

@openshift-ci
openshift-ci Bot requested a review from vdemeester August 17, 2026 09:48
@qodo-code-review

Copy link
Copy Markdown

Preparing PR labels...

@ankrsinha
ankrsinha removed the request for review from vdemeester August 17, 2026 10:06
@ankrsinha ankrsinha changed the title SRVKP-12204: Chery-pick PR for fix to main SRVKP-12204: Chery-pick PR for fix from master to main Aug 17, 2026
…eason

The sort was using status.conditions[0].reason (raw Tekton reason) while
the UI displays a mapped status via the filter reducer. Multiple raw
reasons map to the same label (e.g. PipelineRunTimeout → Failed),
causing items with the same visible status to scatter when sorted.

Replace with a custom sort function that uses the same reducer as the UI,
fixing both TaskRun and PipelineRun list pages.

Signed-off-by: Ankur Sinha <anksinha@redhat.com>
@ankrsinha
ankrsinha force-pushed the fix/SRVKP-12204-main branch from 71a55be to ea93943 Compare August 17, 2026 13:02
@anwesha-palit-redhat
anwesha-palit-redhat removed the request for review from arvindk-softwaredev August 17, 2026 14:01

@anwesha-palit-redhat anwesha-palit-redhat left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

/lgtm

@anwesha-palit-redhat

Copy link
Copy Markdown
Contributor

/retest

@anwesha-palit-redhat

Copy link
Copy Markdown
Contributor

verified, please excuse the poor resolution

Screen.Recording.2026-08-17.at.17.34.24.mov

@anwesha-palit-redhat

Copy link
Copy Markdown
Contributor

/retest

@anwesha-palit-redhat anwesha-palit-redhat added approved Label for Approved PRs lgtm Looks Good to Me Label labels Aug 18, 2026
@openshift-ci

openshift-ci Bot commented Aug 18, 2026

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: ankrsinha, anwesha-palit-redhat

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

The pull request process is described here

Details Needs approval from an approver in each of these files:
  • OWNERS [anwesha-palit-redhat]

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

@openshift-ci

openshift-ci Bot commented Aug 18, 2026

Copy link
Copy Markdown
Contributor

@ankrsinha: 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/e2e ea93943 link true /test e2e

Full PR test history. Your PR dashboard.

Details

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.

@anwesha-palit-redhat

Copy link
Copy Markdown
Contributor

/cherry-pick release-v1.22.x

@openshift-cherrypick-robot

Copy link
Copy Markdown
Contributor

@anwesha-palit-redhat: once the present PR merges, I will cherry-pick it on top of release-v1.22.x in a new PR and assign it to you.

Details

In response to this:

/cherry-pick release-v1.22.x

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.

@anwesha-palit-redhat

Copy link
Copy Markdown
Contributor

/test e2e

@qodo-code-review

qodo-code-review Bot commented Aug 18, 2026

Copy link
Copy Markdown

PR-Agent: could not find a component named e2e in a supported language in this PR.

@anwesha-palit-redhat
anwesha-palit-redhat merged commit b7270dd into openshift-pipelines:main Aug 18, 2026
4 of 7 checks passed
@openshift-cherrypick-robot

Copy link
Copy Markdown
Contributor

@anwesha-palit-redhat: new pull request created: #1289

Details

In response to this:

/cherry-pick release-v1.22.x

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.

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

Labels

approved Label for Approved PRs jira/valid-reference lgtm Looks Good to Me Label

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants