[release-4.19] OCPBUGS-104577: Backport node_e2e test migrations - #31484
[release-4.19] OCPBUGS-104577: Backport node_e2e test migrations#31484Chandan9112 wants to merge 1 commit into
Conversation
|
Skipping CI for Draft Pull Request. |
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Repository: openshift/coderabbit/.coderabbit.yaml Review profile: CHILL Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: Chandan9112 The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
|
@Chandan9112: This pull request references Jira Issue OCPBUGS-104577, which is valid. The bug has been moved to the POST state. 7 validation(s) were run on this bug
The bug has been updated to refer to the pull request using the external bug tracker. DetailsIn 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. |
|
@Chandan9112: This pull request references Jira Issue OCPBUGS-104577, which is valid. 7 validation(s) were run on this bug
DetailsIn 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. |
00e5958 to
5f1f6d1
Compare
|
/payload-job periodic-ci-openshift-release-main-nightly-4.19-e2e-aws-disruptive-longrunning |
|
@Chandan9112: trigger 2 job(s) for the /payload-(with-prs|job|aggregate|job-with-prs|aggregate-with-prs) command
See details on https://pr-payload-tests.ci.openshift.org/runs/ci/d3f7cb20-956a-11f1-9d20-5b7f8aa3d03e-0 |
|
/retest |
9f24026 to
91d445d
Compare
|
/retest |
91d445d to
8d7d271
Compare
Bulk backport of node_e2e test case migrations from main (OCP 5.0) to release-4.19. These tests were previously backported to release-4.22 (PR openshift#31306, merged), release-4.21 (PR openshift#31426, merged), and release-4.20 (PR openshift#31430, merged). Key adaptations for 4.19: - Created minimal imagepolicy_helpers.go with MCP helper functions since test/extended/imagepolicy/ directory does not exist in 4.19 - Removed ote.Informing() decorators since openshift-tests-extension package is not available in 4.19 - Moved k8s.io/kubelet from indirect to direct dependency Co-authored-by: Cursor <cursoragent@cursor.com>
8d7d271 to
c3d592d
Compare
|
@Chandan9112: The following test failed, say
Full PR test history. Your PR dashboard. DetailsInstructions 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. |
|
/payload-job periodic-ci-openshift-release-main-nightly-4.19-e2e-aws-disruptive-longrunning |
|
@Chandan9112: trigger 1 job(s) for the /payload-(with-prs|job|aggregate|job-with-prs|aggregate-with-prs) command
See details on https://pr-payload-tests.ci.openshift.org/runs/ci/b2006b30-96cd-11f1-9f54-63e4f4585316-0 |
|
/payload-job periodic-ci-openshift-release-main-nightly-4.22-e2e-aws-disruptive-longrunning-techpreview |
|
@Chandan9112: trigger 2 job(s) for the /payload-(with-prs|job|aggregate|job-with-prs|aggregate-with-prs) command
See details on https://pr-payload-tests.ci.openshift.org/runs/ci/e12582b0-96cd-11f1-9614-43ad23381ce3-0 |
|
/test e2e-aws-ovn-edge-zones |
|
/payload-job periodic-ci-openshift-release-main-nightly-4.19-e2e-aws-disruptive-longrunning-techpreview |
|
@Chandan9112: trigger 0 job(s) for the /payload-(with-prs|job|aggregate|job-with-prs|aggregate-with-prs) command |
|
/payload-job periodic-ci-openshift-release-main-ci-4.19-e2e-gcp-ovn-techpreview |
|
@Chandan9112: trigger 1 job(s) for the /payload-(with-prs|job|aggregate|job-with-prs|aggregate-with-prs) command
See details on https://pr-payload-tests.ci.openshift.org/runs/ci/f772a740-96fb-11f1-85fe-680467779a1a-0 |
Summary
Bulk backport of node_e2e test case migrations from
main(OCP 5.0) torelease-4.19.These tests were previously backported to
release-4.22via PR #31306 (merged ✅),release-4.21via PR #31426 (merged ✅), andrelease-4.20via PR #31430 (merged ✅). This PR performs the same backport torelease-4.19.Jira
OCPBUGS-104577
Related PRs
Changes (18 files)
New test files (
test/extended/node/node_e2e/)container_runtime_config.goimage_mirror_set.goimage_registry_config.goinitcontainer.gonetns_cleanup.gonode.gopdb_drain.goprobe_termination.goHelper files added
node_utils.go— Helper functions (ExecOnNodeWithChroot, WaitForMCP, etc.)node_mcp_helpers.go— Custom MCP creation/cleanup (NodeHasCustomRole)imagepolicy/imagepolicy_helpers.go— MCP config helper functions (GetMCPCurrentSpecConfigName, WaitForMCPConfigSpecChangeAndUpdated)Modified files
include.go— Addednode/node_e2eblank import for test registrationgo.mod— Movedk8s.io/kubeletfrom indirect to direct dependencyREADME.md— Documentation for running node testsbindata.go— Regenerated to include new testdatazz_generated.annotations.go— Regenerated test annotationsTestdata
testdata/node/node_e2e/pod-dev-fuse.yaml— Test fixture for devfuse test4.19-Specific Adaptations
Since
release-4.19has a different dependency baseline compared to 4.20+, the following adaptations were made:Created
imagepolicy/imagepolicy_helpers.go: Thetest/extended/imagepolicy/directory does not exist in 4.19. A minimal helper file was created with justGetMCPCurrentSpecConfigNameandWaitForMCPConfigSpecChangeAndUpdatedfunctions (used bycontainer_runtime_config.go,image_mirror_set.go, andimage_registry_config.go).Removed
ote.Informing()decorators: Theopenshift-tests-extensionpackage is not available in 4.19's dependency tree. Theote.Informing()test decorator was removed from 4 files (image_mirror_set.go,netns_cleanup.go,pdb_drain.go,probe_termination.go). The tests function identically without this decorator.Moved
k8s.io/kubeletto direct dependency:node_utils.godirectly imports kubelet config types.Feature Verification
All backported tests target features available in OCP 4.19:
[Suite:openshift/disruptive-longrunning]or standard[sig-node]suite tagsTesting
To validate, trigger the disruptive-longrunning payload job: