Skip to content

Fixing the Failure for make update for OCP-80983 on 4.20 #29883

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

asahay19
Copy link
Contributor

@asahay19 asahay19 commented Jun 3, 2025

Fix make update for this PR
#29883. Test case OCP-80983 is getting failed in make update with below error:

goroutine 1 [running]: runtime/debug.Stack() /usr/local/go/src/runtime/debug/stack.go:26 +0x5e github.com/openshift/origin/test/extended/util.FatalErr({0x6710e80, 0xc002325560}) /home/asahay/OCP-80983/origin/test/extended/util/client.go:1023 +0x25 github.com/openshift/origin/test/extended/util.(*CLI).AdminConfig(0x0?) /home/asahay/OCP-80983/origin/test/extended/util/client.go:824 +0x3b github.com/openshift/origin/test/extended/util.(*CLI).SetupProject(0xc0023eac80) /home/asahay/OCP-80983/origin/test/extended/util/client.go:314 +0x1c github.com/openshift/origin/test/extended/node.init.func1() /home/asahay/OCP-80983/origin/test/extended/node/cgroups.go:23 +0xe5 github.com/onsi/ginkgo/v2/internal.NewNode.func2({0xc0023ea780?, 0x0?}) /home/asahay/OCP-80983/origin/vendor/github.com/onsi/ginkgo/v2/internal/node.go:324 +0x13 github.com/onsi/ginkgo/v2/internal.(*Suite).PushNode.func1(0xc0014c2908?) /home/asahay/OCP-80983/origin/vendor/github.com/onsi/ginkgo/v2/internal/suite.go:209 +0x5b github.com/onsi/ginkgo/v2/internal.(*Suite).PushNode(_, {0x193, 0x2, {0x700cdee, 0x72}, 0xc000f9b120, {{0x8bf23cb, 0x3b}, 0x10, {0x0, ...}, ...}, ...}) /home/asahay/OCP-80983/origin/vendor/github.com/onsi/ginkgo/v2/internal/suite.go:211 +0x71e github.com/onsi/ginkgo/v2/internal.(*Suite).BuildTree(0xc000893508) /home/asahay/OCP-80983/origin/vendor/github.com/onsi/ginkgo/v2/internal/suite.go:101 +0xb8 k8s.io/kubernetes/openshift-hack/e2e/annotate.Run(0xa61c878?, 0x71be008) /home/asahay/OCP-80983/origin/vendor/k8s.io/kubernetes/openshift-hack/e2e/annotate/annotate.go:31 +0xd5 main.main() /home/asahay/OCP-80983/origin/test/extended/util/annotate/annotate.go:14 +0x25

So, fixing it by removing oc.SetupProject() .

cc @sairameshv

@openshift-ci openshift-ci bot requested review from JoelSpeed and sairameshv June 3, 2025 07:42
@dusk125
Copy link
Contributor

dusk125 commented Jun 3, 2025

/lgtm

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Jun 3, 2025
Copy link
Contributor

openshift-ci bot commented Jun 3, 2025

@asahay19: 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/e2e-aws-ovn-cgroupsv2 2114960 link false /test e2e-aws-ovn-cgroupsv2
ci/prow/okd-scos-e2e-aws-ovn 2114960 link false /test okd-scos-e2e-aws-ovn
ci/prow/e2e-aws-ovn-single-node 2114960 link false /test e2e-aws-ovn-single-node
ci/prow/e2e-vsphere-ovn-upi 2114960 link true /test e2e-vsphere-ovn-upi
ci/prow/e2e-azure-ovn-upgrade 2114960 link false /test e2e-azure-ovn-upgrade
ci/prow/e2e-vsphere-ovn-dualstack-primaryv6 2114960 link false /test e2e-vsphere-ovn-dualstack-primaryv6
ci/prow/e2e-metal-ipi-ovn-ipv6 2114960 link true /test e2e-metal-ipi-ovn-ipv6
ci/prow/e2e-aws-proxy 2114960 link false /test e2e-aws-proxy
ci/prow/e2e-aws-ovn 2114960 link false /test e2e-aws-ovn
ci/prow/e2e-metal-ipi-ovn-dualstack-local-gateway 2114960 link false /test e2e-metal-ipi-ovn-dualstack-local-gateway
ci/prow/e2e-vsphere-ovn 2114960 link true /test e2e-vsphere-ovn
ci/prow/e2e-gcp-fips-serial-2of2 2114960 link false /test e2e-gcp-fips-serial-2of2
ci/prow/e2e-metal-ipi-ovn-dualstack 2114960 link false /test e2e-metal-ipi-ovn-dualstack
ci/prow/e2e-aws-ovn-single-node-serial 2114960 link false /test e2e-aws-ovn-single-node-serial
ci/prow/4.12-upgrade-from-stable-4.11-e2e-aws-ovn-upgrade-rollback 2114960 link false /test 4.12-upgrade-from-stable-4.11-e2e-aws-ovn-upgrade-rollback
ci/prow/e2e-aws-disruptive 2114960 link false /test e2e-aws-disruptive
ci/prow/e2e-gcp-ovn-etcd-scaling 2114960 link false /test e2e-gcp-ovn-etcd-scaling
ci/prow/e2e-openstack-ovn 2114960 link false /test e2e-openstack-ovn
ci/prow/e2e-metal-ipi-ovn 2114960 link false /test e2e-metal-ipi-ovn
ci/prow/e2e-aws-ovn-edge-zones 2114960 link true /test e2e-aws-ovn-edge-zones
ci/prow/e2e-azure-ovn-etcd-scaling 2114960 link false /test e2e-azure-ovn-etcd-scaling
ci/prow/e2e-gcp-fips-serial-1of2 2114960 link false /test e2e-gcp-fips-serial-1of2
ci/prow/e2e-aws-ovn-single-node-upgrade 2114960 link false /test e2e-aws-ovn-single-node-upgrade
ci/prow/e2e-gcp-ovn 2114960 link true /test e2e-gcp-ovn
ci/prow/e2e-aws-ovn-fips 2114960 link true /test e2e-aws-ovn-fips
ci/prow/e2e-aws 2114960 link false /test e2e-aws
ci/prow/e2e-aws-ovn-etcd-scaling 2114960 link false /test e2e-aws-ovn-etcd-scaling
ci/prow/e2e-azure 2114960 link false /test e2e-azure
ci/prow/e2e-metal-ipi-virtualmedia 2114960 link false /test e2e-metal-ipi-virtualmedia
ci/prow/e2e-vsphere-ovn-etcd-scaling 2114960 link false /test e2e-vsphere-ovn-etcd-scaling
ci/prow/e2e-aws-ovn-serial-publicnet-1of2 2114960 link false /test e2e-aws-ovn-serial-publicnet-1of2
ci/prow/okd-e2e-gcp 2114960 link false /test okd-e2e-gcp
ci/prow/e2e-gcp-disruptive 2114960 link false /test e2e-gcp-disruptive

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.

Copy link

openshift-trt bot commented Jun 3, 2025

Risk analysis has seen new tests most likely introduced by this PR.
Please ensure that new tests meet guidelines for naming and stability.

New Test Risks for sha: 2114960

Job Name New Test Risk
pull-ci-openshift-origin-main-e2e-aws High - "[sig-node][Feature:Remove support to configure Cgroup v1 from OCP version >= 4.19][apigroup:operator.openshift.io] Should result in an error when cgroupMode is changed from v2 to v1 [Suite:openshift/conformance/parallel]" is a new test that failed 1 time(s) against the current commit
pull-ci-openshift-origin-main-e2e-aws-ovn High - "[sig-node][Feature:Remove support to configure Cgroup v1 from OCP version >= 4.19][apigroup:operator.openshift.io] Should result in an error when cgroupMode is changed from v2 to v1 [Suite:openshift/conformance/parallel]" is a new test that failed 1 time(s) against the current commit
pull-ci-openshift-origin-main-e2e-aws-ovn-cgroupsv2 High - "[sig-node][Feature:Remove support to configure Cgroup v1 from OCP version >= 4.19][apigroup:operator.openshift.io] Should result in an error when cgroupMode is changed from v2 to v1 [Suite:openshift/conformance/parallel]" is a new test that failed 1 time(s) against the current commit
pull-ci-openshift-origin-main-e2e-aws-ovn-edge-zones High - "[sig-node][Feature:Remove support to configure Cgroup v1 from OCP version >= 4.19][apigroup:operator.openshift.io] Should result in an error when cgroupMode is changed from v2 to v1 [Suite:openshift/conformance/parallel]" is a new test that failed 1 time(s) against the current commit
pull-ci-openshift-origin-main-e2e-aws-ovn-fips High - "[sig-node][Feature:Remove support to configure Cgroup v1 from OCP version >= 4.19][apigroup:operator.openshift.io] Should result in an error when cgroupMode is changed from v2 to v1 [Suite:openshift/conformance/parallel]" is a new test that failed 1 time(s) against the current commit
pull-ci-openshift-origin-main-e2e-aws-ovn-single-node High - "[sig-node][Feature:Remove support to configure Cgroup v1 from OCP version >= 4.19][apigroup:operator.openshift.io] Should result in an error when cgroupMode is changed from v2 to v1 [Suite:openshift/conformance/parallel]" is a new test that failed 1 time(s) against the current commit
pull-ci-openshift-origin-main-e2e-aws-ovn-single-node-upgrade High - "[sig-node][Feature:Remove support to configure Cgroup v1 from OCP version >= 4.19][apigroup:operator.openshift.io] Should result in an error when cgroupMode is changed from v2 to v1 [Suite:openshift/conformance/parallel]" is a new test that failed 1 time(s) against the current commit
pull-ci-openshift-origin-main-e2e-aws-proxy High - "[sig-node][Feature:Remove support to configure Cgroup v1 from OCP version >= 4.19][apigroup:operator.openshift.io] Should result in an error when cgroupMode is changed from v2 to v1 [Suite:openshift/conformance/parallel]" is a new test that failed 1 time(s) against the current commit
pull-ci-openshift-origin-main-e2e-azure High - "[sig-node][Feature:Remove support to configure Cgroup v1 from OCP version >= 4.19][apigroup:operator.openshift.io] Should result in an error when cgroupMode is changed from v2 to v1 [Suite:openshift/conformance/parallel]" is a new test that failed 1 time(s) against the current commit
pull-ci-openshift-origin-main-e2e-azure-ovn-upgrade High - "[sig-node][Feature:Remove support to configure Cgroup v1 from OCP version >= 4.19][apigroup:operator.openshift.io] Should result in an error when cgroupMode is changed from v2 to v1 [Suite:openshift/conformance/parallel]" is a new test that failed 1 time(s) against the current commit
pull-ci-openshift-origin-main-e2e-gcp-ovn High - "[sig-node][Feature:Remove support to configure Cgroup v1 from OCP version >= 4.19][apigroup:operator.openshift.io] Should result in an error when cgroupMode is changed from v2 to v1 [Suite:openshift/conformance/parallel]" is a new test that failed 1 time(s) against the current commit
pull-ci-openshift-origin-main-e2e-metal-ipi-ovn High - "[sig-node][Feature:Remove support to configure Cgroup v1 from OCP version >= 4.19][apigroup:operator.openshift.io] Should result in an error when cgroupMode is changed from v2 to v1 [Suite:openshift/conformance/parallel]" is a new test that failed 1 time(s) against the current commit
pull-ci-openshift-origin-main-e2e-metal-ipi-ovn-dualstack High - "[sig-node][Feature:Remove support to configure Cgroup v1 from OCP version >= 4.19][apigroup:operator.openshift.io] Should result in an error when cgroupMode is changed from v2 to v1 [Suite:openshift/conformance/parallel]" is a new test that failed 1 time(s) against the current commit
pull-ci-openshift-origin-main-e2e-metal-ipi-ovn-dualstack-local-gateway High - "[sig-node][Feature:Remove support to configure Cgroup v1 from OCP version >= 4.19][apigroup:operator.openshift.io] Should result in an error when cgroupMode is changed from v2 to v1 [Suite:openshift/conformance/parallel]" is a new test that failed 1 time(s) against the current commit
pull-ci-openshift-origin-main-e2e-metal-ipi-ovn-ipv6 High - "[sig-node][Feature:Remove support to configure Cgroup v1 from OCP version >= 4.19][apigroup:operator.openshift.io] Should result in an error when cgroupMode is changed from v2 to v1 [Suite:openshift/conformance/parallel]" is a new test that failed 1 time(s) against the current commit
pull-ci-openshift-origin-main-e2e-metal-ipi-virtualmedia High - "[sig-node][Feature:Remove support to configure Cgroup v1 from OCP version >= 4.19][apigroup:operator.openshift.io] Should result in an error when cgroupMode is changed from v2 to v1 [Suite:openshift/conformance/parallel]" is a new test that failed 1 time(s) against the current commit
pull-ci-openshift-origin-main-e2e-openstack-ovn High - "[sig-node][Feature:Remove support to configure Cgroup v1 from OCP version >= 4.19][apigroup:operator.openshift.io] Should result in an error when cgroupMode is changed from v2 to v1 [Suite:openshift/conformance/parallel]" is a new test that failed 1 time(s) against the current commit
pull-ci-openshift-origin-main-e2e-vsphere-ovn High - "[sig-node][Feature:Remove support to configure Cgroup v1 from OCP version >= 4.19][apigroup:operator.openshift.io] Should result in an error when cgroupMode is changed from v2 to v1 [Suite:openshift/conformance/parallel]" is a new test that failed 1 time(s) against the current commit
pull-ci-openshift-origin-main-e2e-vsphere-ovn-upi High - "[sig-node][Feature:Remove support to configure Cgroup v1 from OCP version >= 4.19][apigroup:operator.openshift.io] Should result in an error when cgroupMode is changed from v2 to v1 [Suite:openshift/conformance/parallel]" is a new test that failed 1 time(s) against the current commit
pull-ci-openshift-origin-main-okd-scos-e2e-aws-ovn High - "[sig-node][Feature:Remove support to configure Cgroup v1 from OCP version >= 4.19][apigroup:operator.openshift.io] Should result in an error when cgroupMode is changed from v2 to v1 [Suite:openshift/conformance/parallel]" is a new test that failed 1 time(s) against the current commit

New tests seen in this PR at sha: 2114960

  • "[sig-node][Feature:Remove support to configure Cgroup v1 from OCP version >= 4.19][apigroup:operator.openshift.io] Should result in an error when cgroupMode is changed from v2 to v1 [Suite:openshift/conformance/parallel]" [Total: 21, Pass: 1, Fail: 20, Flake: 0]

@dusk125
Copy link
Contributor

dusk125 commented Jun 3, 2025

Looks like the test is failing now
/remove-label lgtm

Copy link
Contributor

openshift-ci bot commented Jun 3, 2025

@dusk125: The label(s) /remove-label lgtm cannot be applied. These labels are supported: acknowledge-critical-fixes-only, platform/aws, platform/azure, platform/baremetal, platform/google, platform/libvirt, platform/openstack, ga, tide/merge-method-merge, tide/merge-method-rebase, tide/merge-method-squash, px-approved, docs-approved, qe-approved, ux-approved, no-qe, downstream-change-needed, rebase/manual, cluster-config-api-changed, approved, backport-risk-assessed, bugzilla/valid-bug, cherry-pick-approved, jira/valid-bug, stability-fix-approved, staff-eng-approved. Is this label configured under labels -> additional_labels or labels -> restricted_labels in plugin.yaml?

In response to this:

Looks like the test is failing now
/remove-label lgtm

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.

@dusk125
Copy link
Contributor

dusk125 commented Jun 3, 2025

/lgtm cancel

@openshift-ci openshift-ci bot removed the lgtm Indicates that a PR is ready to be merged. label Jun 3, 2025
Copy link
Contributor

openshift-ci bot commented Jun 3, 2025

[APPROVALNOTIFIER] This PR is NOT APPROVED

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

The full list of commands accepted by this bot can be found 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

@openshift-merge-robot openshift-merge-robot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Jun 6, 2025
@openshift-merge-robot
Copy link
Contributor

PR needs rebase.

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
needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants