NO-JIRA: Fix baremetal misconfiguration for KMS jobs#80283
Conversation
|
@ardaguclu: This pull request explicitly references no jira issue. 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. |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Repository YAML (base), Central YAML (inherited) Review profile: CHILL Plan: Enterprise Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (1)
WalkthroughThis PR updates three periodic test configurations for metal-based encryption-kms e2e scenarios in the cluster-kube-apiserver-operator test suite. It standardizes the cluster profile from equinix-ocp-metal-qe to equinix-ocp-metal, replaces QE-specific configuration fields with DEVSCRIPTS_CONFIG to manage networking, and updates test refs and workflow definitions across base, IPv6, and dualstack variants. ChangesEncryption-KMS periodic test configurations
Estimated code review effort🎯 2 (Simple) | ⏱️ ~8 minutes Suggested labels
🚥 Pre-merge checks | ✅ 15✅ Passed checks (15 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
|
[REHEARSALNOTIFIER]
Prior to this PR being merged, you will need to either run and acknowledge or opt to skip these rehearsals. Interacting with pj-rehearseComment: Once you are satisfied with the results of the rehearsals, comment: |
|
/cc @gangwgr |
|
/pj-rehearse |
|
@gangwgr: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel. |
|
/hold |
|
/lgtm |
|
/hold |
|
waiting for ci to passed |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: ardaguclu, gangwgr 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 |
|
@ardaguclu: 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. |
This PR fixes the misconfigurations of the baremetal jobs of KMS feature.
Summary by CodeRabbit
This PR fixes misconfiguration in the baremetal e2e test jobs for KMS encryption testing in the OpenShift cluster-kube-apiserver-operator CI pipeline.
What Changed:
The periodic test definitions for KMS encryption e2e scenarios on baremetal are being modernized:
equinix-ocp-metal-qesetup with explicit capability declarations and host sizing to the standardequinix-ocp-metalprofilecapabilities,AUX_HOST,RESERVE_BOOTSTRAP,architecture,masters,workers)DEVSCRIPTS_CONFIGenvironment variable to drive networking stack configuration, making it easier to manage IPv4, IPv6, and dual-stack test variantsopenshift-e2e-testandbaremetal-lab-*patterns to the newerbaremetalds-e2e-testand correspondingbaremetalds-e2e*workflowsImpact:
This aligns the KMS encryption test infrastructure with modern baremetal testing patterns and improves maintainability by reducing configuration duplication and using a standardized profile approach. The three test variants (standard OVN, IPv6, and dual-stack) now use a consistent configuration structure.