OCPNODE-3751: Add automated tests for kubelet LimitedSwap drop-in configuration for CNV#30795
OCPNODE-3751: Add automated tests for kubelet LimitedSwap drop-in configuration for CNV#30795BhargaviGudi wants to merge 1 commit intoopenshift:mainfrom
Conversation
BhargaviGudi
changed the title
|
/ok-to-test |
|
/test all |
BhargaviGudi
changed the title
openshift-ci-robot
added
the
jira/valid-reference
|
@BhargaviGudi: This pull request references OCPNODE-3751 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.22.0" version, but no target version was set. 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. |
BhargaviGudi
force-pushed
the
bgudi-node-cnv-swap
branch
from
2c7a4a3 to
5283c24
Compare
|
/approve |
openshift-ci
bot
added
the
approved
BhargaviGudi
force-pushed
the
bgudi-node-cnv-swap
branch
from
5283c24 to
efa4d61
Compare
|
/lgtm |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: BhargaviGudi, cpmeadors, smg247 The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
|
Scheduling required tests: |
|
/verified by @BhargaviGudi |
openshift-ci-robot
added
the
verified
|
@BhargaviGudi: This PR has been marked as verified by 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. |
|
/test e2e-aws-csi |
|
/label acknowledge-critical-fixes-only |
openshift-ci
bot
added
the
acknowledge-critical-fixes-only
BhargaviGudi
force-pushed
the
bgudi-node-cnv-swap
branch
from
efa4d61 to
a6328e9
Compare
openshift-ci-robot
removed
the
verified
openshift-merge-robot
added
the
needs-rebase
|
New changes are detected. LGTM label has been removed. |
BhargaviGudi
force-pushed
the
bgudi-node-cnv-swap
branch
from
a6328e9 to
dd589ca
Compare
openshift-merge-robot
removed
the
needs-rebase
BhargaviGudi
force-pushed
the
bgudi-node-cnv-swap
branch
from
dd589ca to
5fb0a13
Compare
openshift-merge-robot
added
the
needs-rebase
BhargaviGudi
force-pushed
the
bgudi-node-cnv-swap
branch
from
5fb0a13 to
ab38179
Compare
openshift-merge-robot
removed
the
needs-rebase
…figuration for CNV Updated go.mod by runing go mod tidy and make verify-deps verify-deps in the PR was failing due to this issue Resolved conflicts Removed exstra spaces resolved gofmt issue
BhargaviGudi
force-pushed
the
bgudi-node-cnv-swap
branch
from
ab38179 to
49758de
Compare
|
Scheduling required tests: |
|
/verified by @BhargaviGudi |
openshift-ci-robot
added
the
verified
|
@BhargaviGudi: This PR has been marked as verified by 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. |
|
@BhargaviGudi: The following tests 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. |
openshift-merge-robot
added
the
needs-rebase
|
PR needs rebase. 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. |
|
/hold until #30807 is merged |
openshift-ci
bot
added
the
do-not-merge/hold
5 participants
This PR adds automated tests for the LimitedSwap feature using the /etc/openshift/kubelet.conf.d/ drop-in directory on CNV-enabled clusters.
Jira: https://issues.redhat.com/browse/OCPNODE-3751
Test Cases:
TC1: Verify silent creation and ownership of drop-in directory on CNV nodes
TC2: Verify kubelet starts normally with empty directory
TC3: Apply LimitedSwap configuration from drop-in file
TC4: Revert to NoSwap when drop-in file is removed
TC5: Verify control plane kubelets ignore drop-in config
TC6: Verify drop-in directory is auto-recreated after deletion
TC7: Validate security and permissions of drop-in directory
TC8: Verify cluster stability with LimitedSwap enabled
TC9: Verify non-CNV workers have no swap configuration
TC10: Apply correct precedence with multiple drop-in files
TC11: Maintain consistent configuration with checksum verification across CNV nodes
TC12: Handle LimitedSwap config gracefully when OS swap is disabled
TC13: Work correctly with various swap sizes
TC14: Expose swap metrics correctly via Prometheus
Changes
test/extended/node/node_swap_cnv.go - New test file with CNV swap tests
test/extended/node/node_utils.go - Helper functions for node operations and CNV lifecycle
test/extended/testdata/node/cnv-swap/ - Test data YAML files and README
pkg/testsuites/standard_suites.go - Added openshift/nodes/cnv suite definition
Running Tests
Run individual test
./openshift-tests run-test "[Jira:Node/Kubelet][sig-node][Feature:NodeSwap][Serial][Disruptive][Suite:openshift/nodes/cnv] Kubelet LimitedSwap Drop-in Configuration for CNV TC1: should verify silent creation and ownership of drop-in directory on CNV nodesRun entire suite
./openshift-tests run openshift/nodes/cnvAll the testcases are passed with 4.22.0-0.nightly-2026-01-19-085729
I ran each test case individually and captured the outputs here.
Ran test suite and kept the output here
CI job:
http://github.com/openshift/release/pull/73670 WIP