Skip to content

Conversation

@rhamilto
Copy link
Member

@rhamilto rhamilto commented Oct 2, 2025

Includes changes from #15492.

Note now that we have tables with horizontal scrolling, the limit of 3 additional printer columns has been removed and we display all (demonstrated in the SecurityContextConstraints list).

Screen.Recording.2025-10-02.at.2.31.05.PM.mov

@openshift-ci-robot openshift-ci-robot added the jira/valid-reference Indicates that this PR references a valid Jira ticket of any type. label Oct 2, 2025
@openshift-ci-robot
Copy link
Contributor

openshift-ci-robot commented Oct 2, 2025

@rhamilto: This pull request references CONSOLE-4615 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 story to target the "4.21.0" version, but no target version was set.

Details

In response to this:

Includes changes from https://github.com/openshift/console/pull/15492/files#diff-462b6630c494b9ed11ced9c76d39113638cbec98b8f148927b7838343b4836fe

Screen.Recording.2025-10-02.at.2.31.05.PM.mov

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 review from TheRealJon and jhadvig October 2, 2025 18:33
@openshift-ci openshift-ci bot added component/core Related to console core functionality component/shared Related to console-shared approved Indicates a PR has been approved by an approver from all required OWNERS files. kind/cypress Related to Cypress e2e integration testing kind/i18n Indicates issue or PR relates to internationalization or has content that needs to be translated labels Oct 2, 2025
@openshift-ci-robot
Copy link
Contributor

openshift-ci-robot commented Oct 2, 2025

@rhamilto: This pull request references CONSOLE-4615 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 story to target the "4.21.0" version, but no target version was set.

Details

In response to this:

Includes changes from https://github.com/openshift/console/pull/15492/files#diff-462b6630c494b9ed11ced9c76d39113638cbec98b8f148927b7838343b4836fe

Note now that we have tables with horizontal scrolling, the limit of 3 additional printer columns has been removed and we display all.

Screen.Recording.2025-10-02.at.2.31.05.PM.mov

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-robot
Copy link
Contributor

openshift-ci-robot commented Oct 2, 2025

@rhamilto: This pull request references CONSOLE-4615 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 story to target the "4.21.0" version, but no target version was set.

Details

In response to this:

Includes changes from https://github.com/openshift/console/pull/15492/files#diff-462b6630c494b9ed11ced9c76d39113638cbec98b8f148927b7838343b4836fe

Note now that we have tables with horizontal scrolling, the limit of 3 additional printer columns has been removed and we display all (demonstrated in the SecurityContextConstraints list).

Screen.Recording.2025-10-02.at.2.31.05.PM.mov

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.

@rhamilto
Copy link
Member Author

rhamilto commented Oct 3, 2025

/retest

1 similar comment
@rhamilto
Copy link
Member Author

rhamilto commented Oct 3, 2025

/retest

@rhamilto
Copy link
Member Author

rhamilto commented Oct 6, 2025

/retest

@rhamilto
Copy link
Member Author

rhamilto commented Oct 7, 2025

/retest

2 similar comments
@rhamilto
Copy link
Member Author

rhamilto commented Oct 7, 2025

/retest

@rhamilto
Copy link
Member Author

rhamilto commented Oct 7, 2025

/retest

@rhamilto
Copy link
Member Author

rhamilto commented Oct 8, 2025

/retest

@rhamilto
Copy link
Member Author

rhamilto commented Oct 8, 2025

/retest

@jseseCCS
Copy link

btw, @rhamilto, anywhere that i say "please confirm," i'm comfortable with you confirming for yourself and team and not to me, though i appreciate your thoroughness! whichever way you prefer!

@rhamilto
Copy link
Member Author

/hold for #15492 to merge first

@openshift-ci openshift-ci bot added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Oct 15, 2025
@rhamilto
Copy link
Member Author

/test frontend

@rhamilto
Copy link
Member Author

/hold cancel

Includes changes from #15492, which is struggling to get a clean run of integration tests (all appear to be flakes).

@openshift-ci openshift-ci bot removed the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Oct 15, 2025
@rhamilto rhamilto added the plugin-api-approved Indicates a PR with plugin API changes has been approved by an API reviewer label Oct 15, 2025
@openshift-ci-robot
Copy link
Contributor

openshift-ci-robot commented Oct 15, 2025

@rhamilto: This pull request references CONSOLE-4615 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 story to target the "4.21.0" version, but no target version was set.

Details

In response to this:

Includes changes from #15492.

Note now that we have tables with horizontal scrolling, the limit of 3 additional printer columns has been removed and we display all (demonstrated in the SecurityContextConstraints list).

Screen.Recording.2025-10-02.at.2.31.05.PM.mov

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.

@rhamilto
Copy link
Member Author

/test backend

@openshift-ci-robot
Copy link
Contributor

/retest-required

Remaining retests: 0 against base HEAD fd541b6 and 2 for PR HEAD a832846 in total

@openshift-ci-robot
Copy link
Contributor

/retest-required

Remaining retests: 0 against base HEAD 4290cf0 and 1 for PR HEAD a832846 in total

@jhadvig
Copy link
Member

jhadvig commented Oct 16, 2025

/retest

1 similar comment
@rhamilto
Copy link
Member Author

/retest

@rhamilto
Copy link
Member Author

/test e2e-gcp-console

@krishagarwal278
Copy link
Member

/label tide/merge-method-squash

@openshift-ci openshift-ci bot added the tide/merge-method-squash Denotes a PR that should be squashed by tide when it merges. label Oct 17, 2025
@rhamilto
Copy link
Member Author

/test e2e-gcp-console

@rhamilto rhamilto removed the tide/merge-method-squash Denotes a PR that should be squashed by tide when it merges. label Oct 17, 2025
@rhamilto
Copy link
Member Author

/override gcp-e2e-console

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Oct 17, 2025

@rhamilto: /override requires failed status contexts, check run or a prowjob name to operate on.
The following unknown contexts/checkruns were given:

  • gcp-e2e-console

Only the following failed contexts/checkruns were expected:

  • ci/prow/analyze
  • ci/prow/backend
  • ci/prow/e2e-gcp-console
  • ci/prow/frontend
  • ci/prow/images
  • ci/prow/okd-scos-e2e-aws-ovn
  • ci/prow/okd-scos-images
  • pull-ci-openshift-console-main-analyze
  • pull-ci-openshift-console-main-backend
  • pull-ci-openshift-console-main-e2e-gcp-console
  • pull-ci-openshift-console-main-frontend
  • pull-ci-openshift-console-main-images
  • pull-ci-openshift-console-main-okd-scos-e2e-aws-ovn
  • pull-ci-openshift-console-main-okd-scos-images
  • tide

If you are trying to override a checkrun that has a space in it, you must put a double quote on the context.

Details

In response to this:

/override gcp-e2e-console

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.

@rhamilto
Copy link
Member Author

/override e2e-gcp-console

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Oct 17, 2025

@rhamilto: /override requires failed status contexts, check run or a prowjob name to operate on.
The following unknown contexts/checkruns were given:

  • e2e-gcp-console

Only the following failed contexts/checkruns were expected:

  • ci/prow/analyze
  • ci/prow/backend
  • ci/prow/e2e-gcp-console
  • ci/prow/frontend
  • ci/prow/images
  • ci/prow/okd-scos-e2e-aws-ovn
  • ci/prow/okd-scos-images
  • pull-ci-openshift-console-main-analyze
  • pull-ci-openshift-console-main-backend
  • pull-ci-openshift-console-main-e2e-gcp-console
  • pull-ci-openshift-console-main-frontend
  • pull-ci-openshift-console-main-images
  • pull-ci-openshift-console-main-okd-scos-e2e-aws-ovn
  • pull-ci-openshift-console-main-okd-scos-images
  • tide

If you are trying to override a checkrun that has a space in it, you must put a double quote on the context.

Details

In response to this:

/override e2e-gcp-console

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.

@rhamilto
Copy link
Member Author

/override ci/prow/e2e-gcp-console

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Oct 17, 2025

@rhamilto: Overrode contexts on behalf of rhamilto: ci/prow/e2e-gcp-console

Details

In response to this:

/override ci/prow/e2e-gcp-console

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.

@rhamilto
Copy link
Member Author

/override ci/prow/frontend

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Oct 17, 2025

@rhamilto: Overrode contexts on behalf of rhamilto: ci/prow/frontend

Details

In response to this:

/override ci/prow/frontend

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.

@rhamilto rhamilto merged commit 0f28bc6 into openshift:main Oct 17, 2025
8 checks passed
@rhamilto rhamilto deleted the CONSOLE-4615 branch October 17, 2025 16:06
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Oct 17, 2025

@rhamilto: all tests passed!

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.

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

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. component/core Related to console core functionality component/dev-console Related to dev-console component/shared Related to console-shared docs-approved Signifies that Docs has signed off on this PR jira/valid-reference Indicates that this PR references a valid Jira ticket of any type. kind/cypress Related to Cypress e2e integration testing kind/i18n Indicates issue or PR relates to internationalization or has content that needs to be translated lgtm Indicates that a PR is ready to be merged. plugin-api-approved Indicates a PR with plugin API changes has been approved by an API reviewer px-approved Signifies that Product Support has signed off on this PR verified Signifies that the PR passed pre-merge verification criteria

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants