UPSTREAM: <carry>: migrate bandrade test cases #1193

bandrade · 2026-01-14T23:35:28Z

No description provided.

openshift-ci · 2026-01-14T23:35:57Z

[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 joelanford 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:

DOWNSTREAM_OWNERS

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

openshift-ci · 2026-01-14T23:57:12Z

@bandrade: The following tests 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/okd-scos-images	`3371fa7`	link	true	`/test okd-scos-images`
ci/prow/periodics-images	`3371fa7`	link	true	`/test periodics-images`
ci/prow/images	`3371fa7`	link	true	`/test images`
ci/prow/e2e-upgrade	`3371fa7`	link	true	`/test e2e-upgrade`
ci/prow/e2e-aws-olmv0-ext	`3371fa7`	link	true	`/test e2e-aws-olmv0-ext`
ci/prow/tests-extension	`3371fa7`	link	true	`/test tests-extension`
ci/prow/e2e-gcp-ovn	`3371fa7`	link	true	`/test e2e-gcp-ovn`

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.

Xia-Zhao-rh · 2026-01-15T01:37:31Z

tests-extension/test/qe/specs/olmv0_defaultoption.go

 	})

+	// Polarion ID: 43073
+	g.It("PolarionID:43073-[OTP]Indicate dependency class in resolution constraint text", g.Label("original-name:[sig-operator][Jira:OLM] OLMv0 optional should PolarionID:43073-[Skipped:Disconnected]Indicate dependency class in resolution constraint text"), func() {


43073 is a new case in this repo, no need to add original-name

Xia-Zhao-rh · 2026-01-15T01:37:37Z

tests-extension/.openshift-tests-extension/openshift_payload_olmv0.json

      "exclude": "topology==\"External\""
    }
+  },
+  {


not all cases are added in this json file
please run "make update-metadata"

Xia-Zhao-rh · 2026-01-15T01:38:06Z

tests-extension/test/qe/specs/olmv0_defaultoption.go

+	})
+
+	// Polarion ID: 47149
+	g.It("PolarionID:47149-[OTP]Conjunctive constraint of one package and one GVK", g.Label("original-name:[sig-operator][Jira:OLM] OLMv0 optional should PolarionID:47149-[Skipped:Disconnected]Conjunctive constraint of one package and one GVK"), func() {


no need to add original-name

Xia-Zhao-rh · 2026-01-15T01:38:13Z

tests-extension/test/qe/specs/olmv0_defaultoption.go

+	})
+
+	// Polarion ID: 47181
+	g.It("PolarionID:47181-[OTP]Disjunctive constraint of one package and one GVK", g.Label("original-name:[sig-operator][Jira:OLM] OLMv0 optional should PolarionID:47181-[Skipped:Disconnected]Disjunctive constraint of one package and one GVK"), func() {


no need to add original-name

Xia-Zhao-rh · 2026-01-15T01:38:18Z

tests-extension/test/qe/specs/olmv0_defaultoption.go

+	})
+
+	// Polarion ID: 47179
+	g.It("PolarionID:47179-[OTP]Disjunctive constraint of one package and one GVK", g.Label("original-name:[sig-operator][Jira:OLM] OLMv0 optional should PolarionID:47179-[Skipped:Disconnected]Disjunctive constraint of one package and one GVK"), func() {


no need to add original-name

Xia-Zhao-rh · 2026-01-15T01:40:34Z

tests-extension/test/qe/specs/olmv0_nonallns.go


 	})

+	g.It("PolarionID:41026-[LEVEL0]OCS should only one installplan generated when creating subscription", g.Label("original-name:[sig-operator][Jira:OLM] OLMv0 within a namespace PolarionID:41026-[LEVEL0]Critical-OCS should only one installplan generated when creating subscription"), func() {


should be [Level0] not [LEVEL0]
no need to add original-name

Xia-Zhao-rh · 2026-01-15T01:41:24Z

tests-extension/test/qe/specs/olmv0_nonallns.go

+		olmv0util.NewCheck("expect", exutil.AsAdmin, exutil.WithoutNamespace, exutil.Compare, "Complete", exutil.Ok, []string{"installplan", installPlan, "-n", ns, "-o=jsonpath={.status.phase}"}).Check(oc)
+	})
+
+	g.It("PolarionID:68521-Check failureThreshold of redhat-operators catalog", g.Label("NonHyperShiftHOST"), g.Label("original-name:[sig-operator][Jira:OLM] OLMv0 within a namespace PolarionID:68521-[Skipped:Disconnected]Check failureThreshold of redhat-operators catalog"), func() {


no need to add original-name

Xia-Zhao-rh · 2026-01-15T01:41:32Z

tests-extension/test/qe/specs/olmv0_nonallns.go

+		olmv0util.NewCheck("expect", exutil.AsAdmin, exutil.WithoutNamespace, exutil.Contain, "10", exutil.Ok, []string{"pods", "-n", "openshift-marketplace", "-l", "olm.catalogSource=redhat-operators", "-o=jsonpath={..spec.containers[0].startupProbe.failureThreshold}"}).Check(oc)
+	})
+
+	g.It("PolarionID:68901-Packageserver pod should not crash if updateStrategy is incorrect", g.Label("NonHyperShiftHOST"), g.Label("original-name:[sig-operator][Jira:OLM] OLMv0 within a namespace PolarionID:68901-[Skipped:Disconnected]Packageserver pod should not crash if updateStrategy is incorrect"), func() {


no need to add original-name

kuiwang02 · 2026-01-15T01:27:54Z

tests-extension/test/qe/specs/olmv0_common.go

 		dr.RmIr(itName)
 	})

+	g.It("PolarionID:OCP-24818-Checking OLM descriptors", func() {


@bandrade caseID format is not correct.

kuiwang02 · 2026-01-15T01:29:12Z

tests-extension/.openshift-tests-extension/openshift_payload_olmv0.json

    }
+  },
+  {
+    "name": "[sig-operator][Jira:OLM] OLMv0 webhook operator PolarionID:83979-[Skipped:Disconnected]ClusterExtension installs webhook-operator and webhooks work",


@bandrade the update is not full. you should miss some cases. please update them.

kuiwang02 · 2026-01-15T01:30:26Z

tests-extension/test/qe/specs/olmv0_defaultoption.go

 	})

+	// Polarion ID: 43073
+	g.It("PolarionID:43073-[OTP]Indicate dependency class in resolution constraint text", g.Label("original-name:[sig-operator][Jira:OLM] OLMv0 optional should PolarionID:43073-[Skipped:Disconnected]Indicate dependency class in resolution constraint text"), func() {


@bandrade no need g.Label because it is first added.

same comment for other new added cases.

kuiwang02 · 2026-01-15T01:31:16Z

tests-extension/test/qe/specs/olmv0_defaultoption.go

+
+		catsrc := olmv0util.CatalogSourceDescription{
+			Name:        "cs-43073",
+			Namespace:   "openshift-marketplace",


@bandrade better to create it in oc.Namespace.

kuiwang02 · 2026-01-15T01:38:32Z

tests-extension/test/qe/specs/olmv0_defaultoption.go

+			DisplayName: "ocp-47149",
+			Publisher:   "OLM QE",
+			SourceType:  "grpc",
+			Address:     "quay.io/olmqe/etcd-47149:1.0",


@bandrade if bundle image or operand iamge is not in quay.io/olmqe, quay.io/openshifttest, quay.io/openshift-qe-optional-operators, registry.redhat.io, brew.registry.redhat.io, registry.stage.redhat.io and registry-proxy.engineering.redhat.com (I am fear etcd image is not in them), please add [Skipped:Disconnected] because qe disc env only support to mirror these registry.

kuiwang02 · 2026-01-15T02:01:28Z

tests-extension/test/qe/specs/olmv0_nonallns.go

+		olmv0util.NewCheck("expect", exutil.AsAdmin, exutil.WithoutNamespace, exutil.Contain, "10", exutil.Ok, []string{"pods", "-n", "openshift-marketplace", "-l", "olm.catalogSource=redhat-operators", "-o=jsonpath={..spec.containers[0].startupProbe.failureThreshold}"}).Check(oc)
+	})
+
+	g.It("PolarionID:68901-Packageserver pod should not crash if updateStrategy is incorrect", g.Label("NonHyperShiftHOST"), g.Label("original-name:[sig-operator][Jira:OLM] OLMv0 within a namespace PolarionID:68901-[Skipped:Disconnected]Packageserver pod should not crash if updateStrategy is incorrect"), func() {


@bandrade
1, miss OTP
2, it was ConnectedOnly, so need to add [Skipped:Disconnected]

kuiwang02 · 2026-01-15T02:04:56Z

tests-extension/test/qe/specs/olmv0_defaultoption.go

+	})
+
+	// Polarion ID: 47179
+	g.It("PolarionID:47179-[OTP]Disjunctive constraint of one package and one GVK", g.Label("original-name:[sig-operator][Jira:OLM] OLMv0 optional should PolarionID:47179-[Skipped:Disconnected]Disjunctive constraint of one package and one GVK"), func() {


@bandrade it was ConnectedOnly, so need to add [Skipped:Disconnected]

kuiwang02 · 2026-01-15T02:06:06Z

tests-extension/test/qe/specs/olmv0_defaultoption.go

+	})
+
+	// Polarion ID: 47181
+	g.It("PolarionID:47181-[OTP]Disjunctive constraint of one package and one GVK", g.Label("original-name:[sig-operator][Jira:OLM] OLMv0 optional should PolarionID:47181-[Skipped:Disconnected]Disjunctive constraint of one package and one GVK"), func() {


@bandrade it was ConnectedOnly, so need to add [Skipped:Disconnected]

kuiwang02 · 2026-01-15T02:06:48Z

tests-extension/test/qe/specs/olmv0_defaultoption.go

+	})
+
+	// Polarion ID: 47149
+	g.It("PolarionID:47149-[OTP]Conjunctive constraint of one package and one GVK", g.Label("original-name:[sig-operator][Jira:OLM] OLMv0 optional should PolarionID:47149-[Skipped:Disconnected]Conjunctive constraint of one package and one GVK"), func() {


@bandrade it was ConnectedOnly, so need to add [Skipped:Disconnected]

kuiwang02 · 2026-01-15T02:07:27Z

tests-extension/test/qe/specs/olmv0_defaultoption.go

 	})

+	// Polarion ID: 43073
+	g.It("PolarionID:43073-[OTP]Indicate dependency class in resolution constraint text", g.Label("original-name:[sig-operator][Jira:OLM] OLMv0 optional should PolarionID:43073-[Skipped:Disconnected]Indicate dependency class in resolution constraint text"), func() {


@bandrade it was ConnectedOnly, so need to add [Skipped:Disconnected]

kuiwang02 · 2026-01-15T02:09:06Z

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

openshift-ci · 2026-01-15T02:09:09Z

@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/2c634da0-f1b7-11f0-858b-09190287355b-0

kuiwang02 · 2026-01-15T02:09:23Z

/payload-aggregate periodic-ci-openshift-operator-framework-operator-controller-release-4.22-periodics-e2e-gcp-ovn-techpreview-slow-extended-f1 3

openshift-ci · 2026-01-15T02:09:26Z

@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-techpreview-slow-extended-f1

See details on https://pr-payload-tests.ci.openshift.org/runs/ci/3631d4f0-f1b7-11f0-8329-1051d7132f07-0

kuiwang02 · 2026-01-15T02:10:53Z

/payload-aggregate periodic-ci-openshift-operator-framework-olm-release-4.22-periodics-e2e-gcp-ovn-ipi-disconnected-extended-f1 3

openshift-ci · 2026-01-15T02:10:56Z

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

periodic-ci-openshift-operator-framework-olm-release-4.22-periodics-e2e-gcp-ovn-ipi-disconnected-extended-f1

See details on https://pr-payload-tests.ci.openshift.org/runs/ci/6c2ec900-f1b7-11f0-903d-2300658efe38-0

UPSTREAM: <carry>: migrate bandrade test cases

3371fa7

openshift-ci bot requested review from camilamacedo86 and grokspawn January 14, 2026 23:35

UPSTREAM: <carry>: migrate bandrade test cases

cd6c72d

Xia-Zhao-rh reviewed Jan 15, 2026

View reviewed changes

kuiwang02 reviewed Jan 15, 2026

View reviewed changes


		})

		g.It("PolarionID:41026-[LEVEL0]OCS should only one installplan generated when creating subscription", g.Label("original-name:[sig-operator][Jira:OLM] OLMv0 within a namespace PolarionID:41026-[LEVEL0]Critical-OCS should only one installplan generated when creating subscription"), func() {

UPSTREAM: <carry>: migrate bandrade test cases #1193

Are you sure you want to change the base?

UPSTREAM: <carry>: migrate bandrade test cases #1193

Conversation

bandrade commented Jan 14, 2026

Uh oh!

openshift-ci bot commented Jan 14, 2026

Uh oh!

openshift-ci bot commented Jan 14, 2026

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

kuiwang02 commented Jan 15, 2026

Uh oh!

openshift-ci bot commented Jan 15, 2026

Uh oh!

kuiwang02 commented Jan 15, 2026

Uh oh!

openshift-ci bot commented Jan 15, 2026

Uh oh!

kuiwang02 commented Jan 15, 2026

Uh oh!

openshift-ci bot commented Jan 15, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants