Skip to content

NO-ISSUE: add 85889 automation #603

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
bandrade wants to merge 1 commit into
base: main
Choose a base branch
from
Open

NO-ISSUE: add 85889 automation #603

bandrade wants to merge 1 commit into from
+339 −10

Conversation

@bandrade
Copy link
Contributor

@bandrade bandrade commented Jan 14, 2026

No description provided.

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Jan 14, 2026

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: bandrade
Once this PR has been reviewed and has the lgtm label, please assign oceanc80 for approval. For more information see the Code Review Process.

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

Details 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

kuiwang02
kuiwang02 reviewed Jan 14, 2026
View reviewed changes
@kuiwang02
Copy link
Contributor

kuiwang02 commented Jan 14, 2026

/payload-aggregate periodic-ci-openshift-operator-framework-operator-controller-release-4.22-periodics-e2e-gcp-ovn-fips-extended-f2 3

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Jan 14, 2026

@kuiwang02: trigger 1 job(s) for the /payload-(with-prs|job|aggregate|job-with-prs|aggregate-with-prs) command

  • periodic-ci-openshift-operator-framework-operator-controller-release-4.22-periodics-e2e-gcp-ovn-fips-extended-f2

See details on https://pr-payload-tests.ci.openshift.org/runs/ci/669f1200-f0f5-11f0-84d2-4d2ce215d1fe-0

kuiwang02
kuiwang02 reviewed Jan 14, 2026
View reviewed changes
@kuiwang02
Copy link
Contributor

kuiwang02 commented Jan 14, 2026

@bandrade 85889 is failing. please refer to https://pr-payload-tests.ci.openshift.org/runs/ci/669f1200-f0f5-11f0-84d2-4d2ce215d1fe-0

@kuiwang02 kuiwang02 mentioned this pull request Jan 16, 2026
Merged
@bandrade bandrade force-pushed the OCP-85889 branch 2 times, most recently from f850ec5 to f7927a9 Compare January 20, 2026 23:34
@bandrade
Copy link
Contributor Author

bandrade commented Jan 21, 2026

/payload-aggregate periodic-ci-openshift-operator-framework-operator-controller-release-4.22-periodics-e2e-gcp-ovn-fips-extended-f2 3

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Jan 21, 2026

@bandrade: trigger 1 job(s) for the /payload-(with-prs|job|aggregate|job-with-prs|aggregate-with-prs) command

  • periodic-ci-openshift-operator-framework-operator-controller-release-4.22-periodics-e2e-gcp-ovn-fips-extended-f2

See details on https://pr-payload-tests.ci.openshift.org/runs/ci/17867860-f65f-11f0-8b10-0be95671d133-0

@kuiwang02
Copy link
Contributor

kuiwang02 commented Jan 21, 2026 •
edited
Loading

the case 85889 is failing. you could refer to https://pr-payload-tests.ci.openshift.org/runs/ci/17867860-f65f-11f0-8b10-0be95671d133-0

suggest you to run it locally before submitting the PR.

@openshift-merge-robot openshift-merge-robot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Jan 27, 2026
@openshift-merge-robot openshift-merge-robot removed the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Jan 27, 2026
@bandrade
Copy link
Contributor Author

bandrade commented Jan 27, 2026

/retitle NO-ISSUE: add 85889 automation

@openshift-ci openshift-ci bot changed the title UPSTREAM: <carry>: add 85889 automation NO-ISSUE: add 85889 automation Jan 27, 2026
@openshift-ci-robot openshift-ci-robot added the jira/valid-reference Indicates that this PR references a valid Jira ticket of any type. label Jan 27, 2026
@openshift-ci-robot
Copy link

openshift-ci-robot commented Jan 27, 2026

@bandrade: This pull request explicitly references no jira issue.

Details

In response to this:

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.

@kuiwang02
Copy link
Contributor

kuiwang02 commented Jan 28, 2026

/payload-aggregate periodic-ci-openshift-operator-framework-operator-controller-release-4.22-periodics-e2e-gcp-ovn-fips-extended-f2 3

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Jan 28, 2026

@kuiwang02: trigger 1 job(s) for the /payload-(with-prs|job|aggregate|job-with-prs|aggregate-with-prs) command

  • periodic-ci-openshift-operator-framework-operator-controller-release-4.22-periodics-e2e-gcp-ovn-fips-extended-f2

See details on https://pr-payload-tests.ci.openshift.org/runs/ci/3087dc80-fbe8-11f0-9338-8b322bf2a9c1-0

kuiwang02
kuiwang02 reviewed Jan 28, 2026
View reviewed changes
openshift/tests-extension/test/qe/specs/olmv1_cc.go Outdated
consoleLogs, err := getConsoleLogs()
o.Expect(err).NotTo(o.HaveOccurred())
consoleLogsLower := strings.ToLower(consoleLogs)
o.Expect(consoleLogsLower).To(o.ContainSubstring("reconcil"))
Copy link
Contributor

@kuiwang02 kuiwang02 Jan 28, 2026

Choose a reason for hiding this comment

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

@bandrade

consoleLogs is possible to be so big message. for the following code, once it fails, the consoleLogs will be error message. it is too long for error message possible.

o.Expect(consoleLogsLower).To(o.ContainSubstring("reconcil"))
		o.Expect(consoleLogsLower).To(o.ContainSubstring("clustercatalog"))
		o.Expect(consoleLogs).To(o.ContainSubstring(catalogName))
		o.Expect(consoleLogsLower).NotTo(o.ContainSubstring("panic"))
		o.Expect(consoleLogsLower).NotTo(o.ContainSubstring("stack trace"))

change

         o.Expect(strings.Contains(consoleLogsLower,"reconcil")).To(o.BeTrue(), )
         o.Expect(strings.Contains(consoleLogsLower,"clustercatalog")).To(o.BeTrue())
         o.Expect(strings.Contains(consoleLogs,catalogName)).To(o.BeTrue())
         o.Expect(strings.Contains(consoleLogsLower,"panic")).To(o.BeFalse())
         o.Expect(strings.Contains(consoleLogsLower,"stack trace")).To(o.BeFalse())

@kuiwang02
Copy link
Contributor

kuiwang02 commented Jan 28, 2026

/payload-aggregate periodic-ci-openshift-operator-framework-operator-controller-release-4.22-periodics-e2e-gcp-ovn-fips-extended-f2 3

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Jan 28, 2026

@kuiwang02: trigger 1 job(s) for the /payload-(with-prs|job|aggregate|job-with-prs|aggregate-with-prs) command

  • periodic-ci-openshift-operator-framework-operator-controller-release-4.22-periodics-e2e-gcp-ovn-fips-extended-f2

See details on https://pr-payload-tests.ci.openshift.org/runs/ci/6a7bd150-fc09-11f0-96c1-f7236a691700-0

@jianzhangbjz
Copy link
Member

jianzhangbjz commented Jan 30, 2026

Hi @bandrade any updates?

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Feb 2, 2026

@bandrade: 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.

@bandrade
Copy link
Contributor Author

bandrade commented Feb 3, 2026

/hold
Currently testing locally

@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 Feb 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@grokspawn grokspawn Awaiting requested review from grokspawn

@joelanford joelanford Awaiting requested review from joelanford

1 more reviewer

@kuiwang02 kuiwang02 kuiwang02 left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. jira/valid-reference Indicates that this PR references a valid Jira ticket of any type.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@bandrade @kuiwang02 @openshift-ci-robot @jianzhangbjz @openshift-merge-robot