OCPNODE-4078: Add DRAPartitionableDevices to TechPreviewNoUpgrade by harche · Pull Request #2694 · openshift/api

harche · 2026-02-05T14:58:54Z

Add DRAPartitionableDevices to TechPreviewNoUpgrade

openshift-ci-robot · 2026-02-05T14:58:57Z

Pipeline controller notification
This repo is configured to use the pipeline controller. Second-stage tests will be triggered either automatically or after lgtm label is added, depending on the repository configuration. The pipeline controller will automatically detect which contexts are required and will utilize /test Prow commands to trigger the second stage.

For optional jobs, comment /test ? to see a list of all defined jobs. To trigger manually all jobs from second stage use /pipeline required command.

This repository is configured in: LGTM mode

qodo-code-review · 2026-02-05T14:59:25Z

Review Summary by Qodo

Add DRAPartitionableDevices feature gate to TechPreviewNoUpgrade

✨ Enhancement

Walkthroughs

Description

• Add DRAPartitionableDevices feature gate to codebase
• Enable feature in TechPreviewNoUpgrade cluster profile
• Update all feature gate manifest files consistently
• Document feature in features.md table

Diagram

flowchart LR
A["DRAPartitionableDevices<br/>Feature Gate"] -->|"Defined in"| B["features.go"]
A -->|"Enabled in"| C["TechPreviewNoUpgrade<br/>Profile"]
B -->|"Manifests updated"| D["Feature Gate<br/>YAML Files"]
A -->|"Documented in"| E["features.md"]

File Changes

1. features/features.go ✨ Enhancement +8/-0

Define DRAPartitionableDevices feature gate
features/features.go

2. features.md 📝 Documentation +1/-0

Add DRAPartitionableDevices to feature table
features.md

3. payload-manifests/featuregates/featureGate-Hypershift-Default.yaml ⚙️ Configuration changes +3/-0

Add DRAPartitionableDevices to Default profile
payload-manifests/featuregates/featureGate-Hypershift-Default.yaml

View more (7)

4. payload-manifests/featuregates/featureGate-Hypershift-DevPreviewNoUpgrade.yaml ⚙️ Configuration changes +3/-0

Disable DRAPartitionableDevices in DevPreviewNoUpgrade
payload-manifests/featuregates/featureGate-Hypershift-DevPreviewNoUpgrade.yaml

5. payload-manifests/featuregates/featureGate-Hypershift-OKD.yaml ⚙️ Configuration changes +3/-0

Add DRAPartitionableDevices to OKD Default profile
payload-manifests/featuregates/featureGate-Hypershift-OKD.yaml

6. payload-manifests/featuregates/featureGate-Hypershift-TechPreviewNoUpgrade.yaml ⚙️ Configuration changes +3/-0

Add DRAPartitionableDevices to TechPreviewNoUpgrade
payload-manifests/featuregates/featureGate-Hypershift-TechPreviewNoUpgrade.yaml

7. payload-manifests/featuregates/featureGate-SelfManagedHA-Default.yaml ⚙️ Configuration changes +3/-0

Add DRAPartitionableDevices to Default profile
payload-manifests/featuregates/featureGate-SelfManagedHA-Default.yaml

8. payload-manifests/featuregates/featureGate-SelfManagedHA-DevPreviewNoUpgrade.yaml ⚙️ Configuration changes +3/-0

Disable DRAPartitionableDevices in DevPreviewNoUpgrade
payload-manifests/featuregates/featureGate-SelfManagedHA-DevPreviewNoUpgrade.yaml

9. payload-manifests/featuregates/featureGate-SelfManagedHA-OKD.yaml ⚙️ Configuration changes +3/-0

Add DRAPartitionableDevices to OKD Default profile
payload-manifests/featuregates/featureGate-SelfManagedHA-OKD.yaml

10. payload-manifests/featuregates/featureGate-SelfManagedHA-TechPreviewNoUpgrade.yaml ⚙️ Configuration changes +3/-0

Add DRAPartitionableDevices to TechPreviewNoUpgrade
payload-manifests/featuregates/featureGate-SelfManagedHA-TechPreviewNoUpgrade.yaml

openshift-ci · 2026-02-05T14:59:48Z

Hello @harche! Some important instructions when contributing to openshift/api:
API design plays an important part in the user experience of OpenShift and as such API PRs are subject to a high level of scrutiny to ensure they follow our best practices. If you haven't already done so, please review the OpenShift API Conventions and ensure that your proposed changes are compliant. Following these conventions will help expedite the api review process for your PR.

qodo-code-review · 2026-02-05T15:02:39Z

Code Review by Qodo

🐞 Bugs (1)📘 Rule violations (0)📎 Requirement gaps (0)

1. Jira component casing mismatch 🐞 Bug✧ Quality

Description

• The new feature gate uses reportProblemsToJiraComponent("Node") while nearby/related feature
gates use the lowercase "node" component string.
• If this field is used for ownership grouping or automated bug routing, inconsistent casing can
fragment reporting or route bugs to the wrong component.
• This is low risk today (no current validation), but it is a maintainability/operational hygiene
issue.

Code

features/features.go[R910-912]

+	FeatureGateDRAPartitionableDevices = newFeatureGate("DRAPartitionableDevices").+ reportProblemsToJiraComponent("Node").+ contactPerson("harche").

Evidence

The new gate declares Jira component as "Node" (capitalized) whereas other node-owned feature gates
in the same file consistently use "node" (lowercase), indicating a likely typo/inconsistency.

features/features.go[910-916]
features/features.go[111-136]

Agent prompt

The issue below was found during a code review. Follow the provided context and guidance below and implement a solution
### Issue description
The new feature gate DRAPartitionableDevices uses a Jira component string with different casing (`&quot;Node&quot;`) than other node-owned feature gates (`&quot;node&quot;`). This can lead to fragmented ownership/reporting.
### Issue Context
Other node-related feature gates in `features/features.go` use `reportProblemsToJiraComponent(&quot;node&quot;)`.
### Fix Focus Areas
- features/features.go[910-916]

ⓘ Copy this prompt and use it to remediate the issue with your preferred AI generation tools

ⓘ The new review experience is currently in Beta. Learn more

harche · 2026-02-05T15:05:19Z

/test

qodo-code-review · 2026-02-05T15:05:25Z

PR-Agent: Missing component name in test command: /test <component_name>.
Running automatically on the largest changed component.

✨ Test tool usage guide:

The test tool generate tests for a selected component, based on the PR code changes.
It can be invoked manually by commenting on any PR:

/test component_name

where 'component_name' is the name of a specific component in the PR. To get a list of the components that changed in the PR, use the analyze tool.
Language that are currently supported: Python, Java, C++, JavaScript, TypeScript.

Configuration options:

num_tests: number of tests to generate. Default is 3.
testing_framework: the testing framework to use. If not set, for Python it will use pytest, for Java it will use JUnit, for C++ it will use Catch2, and for JavaScript and TypeScript it will use jest.
avoid_mocks: if set to true, the tool will try to avoid using mocks in the generated tests. Note that even if this option is set to true, the tool might still use mocks if it cannot generate a test without them. Default is true.
extra_instructions: Optional extra instructions to the tool. For example: "use the following mock injection scheme: ...".
file: in case there are several components with the same name, you can specify the relevant file.
class_name: in case there are several components with the same name in the same file, you can specify the relevant class name.

See more information about the test tool in the docs.

JoelSpeed · 2026-02-05T15:19:00Z

+ contactPerson("harpatil").
+ productScope(kubernetes).
+ enhancementPR("https://github.com/kubernetes/enhancements/issues/4815").
+ enableIn(configv1.TechPreviewNoUpgrade).


Must also be in DevPreviewNoUpgrade if it's in TechPreviewNoUpgrade

Updated, thanks.

coderabbitai · 2026-02-05T15:26:17Z

📝 Walkthrough

Walkthrough

A new feature gate DRAPartitionableDevices was added. It is declared in features/features.go with Jira component "Node", contact person "harche", product scope kubernetes, enhancement PR URL https://github.com/kubernetes/enhancements/issues/4815, and enabled for DevPreviewNoUpgrade and TechPreviewNoUpgrade. An entry was added to features.md. The feature gate was added to multiple FeatureGate YAMLs for Hypershift and SelfManagedHA, marked enabled in DevPreviewNoUpgrade and TechPreviewNoUpgrade variants and disabled in default/OKD variants.

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description check	✅ Passed	The description is directly related to the changeset, matching the title and confirming the objective to add DRAPartitionableDevices to TechPreviewNoUpgrade.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Title check	✅ Passed	The title accurately describes the main change: adding DRAPartitionableDevices feature gate to TechPreviewNoUpgrade configuration.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing touches

📝 Generate docstrings

🧪 Generate unit tests (beta)

Create PR with unit tests
Post copyable unit tests in a comment

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.

❤️ Share

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

harche · 2026-02-05T15:39:45Z

/retest

Signed-off-by: Harshal Patil <12152047+harche@users.noreply.github.com>

harche · 2026-02-05T16:03:30Z

/retitle OCPNODE-4078: Add DRAPartitionableDevices to TechPreviewNoUpgrade

qodo-code-review · 2026-02-05T16:06:46Z

Review Summary by Qodo

Add DRAPartitionableDevices feature gate to preview profiles

✨ Enhancement

Walkthroughs

Description

• Add DRAPartitionableDevices feature gate definition
• Enable feature in DevPreviewNoUpgrade and TechPreviewNoUpgrade profiles
• Update feature gate manifests across all cluster profiles
• Update feature documentation with new gate entry

Diagram

flowchart LR
A["Feature Gate Definition"] -->|"enableIn DevPreview/TechPreview"| B["Feature Gate Configuration"]
B -->|"update manifests"| C["Hypershift Profiles"]
B -->|"update manifests"| D["SelfManagedHA Profiles"]
C -->|"document"| E["Feature Documentation"]
D -->|"document"| E

File Changes

1. features/features.go ✨ Enhancement +8/-0

Define DRAPartitionableDevices feature gate
features/features.go

2. features.md 📝 Documentation +1/-0

Add DRAPartitionableDevices to feature table
features.md

3. payload-manifests/featuregates/featureGate-Hypershift-Default.yaml ⚙️ Configuration changes +3/-0

Add DRAPartitionableDevices to Default profile
payload-manifests/featuregates/featureGate-Hypershift-Default.yaml

View more (7)

4. payload-manifests/featuregates/featureGate-Hypershift-DevPreviewNoUpgrade.yaml ⚙️ Configuration changes +3/-0

Add DRAPartitionableDevices to DevPreview profile
payload-manifests/featuregates/featureGate-Hypershift-DevPreviewNoUpgrade.yaml

5. payload-manifests/featuregates/featureGate-Hypershift-OKD.yaml ⚙️ Configuration changes +3/-0

Add DRAPartitionableDevices to OKD profile
payload-manifests/featuregates/featureGate-Hypershift-OKD.yaml

6. payload-manifests/featuregates/featureGate-Hypershift-TechPreviewNoUpgrade.yaml ⚙️ Configuration changes +3/-0

Add DRAPartitionableDevices to TechPreview profile
payload-manifests/featuregates/featureGate-Hypershift-TechPreviewNoUpgrade.yaml

7. payload-manifests/featuregates/featureGate-SelfManagedHA-Default.yaml ⚙️ Configuration changes +3/-0

Add DRAPartitionableDevices to Default profile
payload-manifests/featuregates/featureGate-SelfManagedHA-Default.yaml

8. payload-manifests/featuregates/featureGate-SelfManagedHA-DevPreviewNoUpgrade.yaml ⚙️ Configuration changes +3/-0

Add DRAPartitionableDevices to DevPreview profile
payload-manifests/featuregates/featureGate-SelfManagedHA-DevPreviewNoUpgrade.yaml

9. payload-manifests/featuregates/featureGate-SelfManagedHA-OKD.yaml ⚙️ Configuration changes +3/-0

Add DRAPartitionableDevices to OKD profile
payload-manifests/featuregates/featureGate-SelfManagedHA-OKD.yaml

10. payload-manifests/featuregates/featureGate-SelfManagedHA-TechPreviewNoUpgrade.yaml ⚙️ Configuration changes +3/-0

Add DRAPartitionableDevices to TechPreview profile
payload-manifests/featuregates/featureGate-SelfManagedHA-TechPreviewNoUpgrade.yaml

harche · 2026-02-05T16:11:01Z

/ok-to-test

qodo-code-review · 2026-02-05T16:11:01Z

Persistent review updated to latest commit 1fece6c

harche · 2026-02-05T16:12:19Z

/cc @haircommander

haircommander · 2026-02-05T16:31:54Z

/lgtm

openshift-ci-robot · 2026-02-05T16:34:16Z

Scheduling tests matching the pipeline_run_if_changed or not excluded by pipeline_skip_if_only_changed parameters:
/test e2e-aws-ovn
/test e2e-aws-ovn-hypershift
/test e2e-aws-ovn-hypershift-conformance
/test e2e-aws-ovn-techpreview
/test e2e-aws-serial-1of2
/test e2e-aws-serial-2of2
/test e2e-aws-serial-techpreview-1of2
/test e2e-aws-serial-techpreview-2of2
/test e2e-azure
/test e2e-gcp
/test e2e-upgrade
/test e2e-upgrade-out-of-change

harche · 2026-02-05T16:38:35Z

/test

qodo-code-review · 2026-02-05T16:38:40Z

PR-Agent: Missing component name in test command: /test <component_name>.
Running automatically on the largest changed component.

✨ Test tool usage guide:

The test tool generate tests for a selected component, based on the PR code changes.
It can be invoked manually by commenting on any PR:

/test component_name

where 'component_name' is the name of a specific component in the PR. To get a list of the components that changed in the PR, use the analyze tool.
Language that are currently supported: Python, Java, C++, JavaScript, TypeScript.

Configuration options:

num_tests: number of tests to generate. Default is 3.
testing_framework: the testing framework to use. If not set, for Python it will use pytest, for Java it will use JUnit, for C++ it will use Catch2, and for JavaScript and TypeScript it will use jest.
avoid_mocks: if set to true, the tool will try to avoid using mocks in the generated tests. Note that even if this option is set to true, the tool might still use mocks if it cannot generate a test without them. Default is true.
extra_instructions: Optional extra instructions to the tool. For example: "use the following mock injection scheme: ...".
file: in case there are several components with the same name, you can specify the relevant file.
class_name: in case there are several components with the same name in the same file, you can specify the relevant class name.

See more information about the test tool in the docs.

harche · 2026-02-05T16:39:33Z

/test e2e-aws-ovn-techpreview

qodo-code-review · 2026-02-05T16:39:39Z

PR-Agent: could not fine a component named e2e-aws-ovn-techpreview in a supported language in this PR.

harche · 2026-02-05T16:40:53Z

/lgtm

Thanks @haircommander

cc @JoelSpeed for approval.

JoelSpeed · 2026-02-05T17:24:54Z

/approve

openshift-ci · 2026-02-05T17:27:35Z

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: haircommander, JoelSpeed

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details

Needs approval from an approver in each of these files:

~~OWNERS~~ [JoelSpeed]

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

harche · 2026-02-05T17:36:14Z

/approve

Thanks @JoelSpeed

I am wondering why ci bot is not triggering the CI jobs and removing the do-not-merge/work-in-progress label.

JoelSpeed · 2026-02-05T17:36:50Z

/pipeline required

Likely a CI cluster issue

openshift-ci-robot · 2026-02-05T17:36:53Z

Scheduling tests matching the pipeline_run_if_changed or not excluded by pipeline_skip_if_only_changed parameters:
/test e2e-aws-ovn
/test e2e-aws-ovn-hypershift
/test e2e-aws-ovn-hypershift-conformance
/test e2e-aws-ovn-techpreview
/test e2e-aws-serial-1of2
/test e2e-aws-serial-2of2
/test e2e-aws-serial-techpreview-1of2
/test e2e-aws-serial-techpreview-2of2
/test e2e-azure
/test e2e-gcp
/test e2e-upgrade
/test e2e-upgrade-out-of-change

harche · 2026-02-05T18:34:20Z

/test unit

qodo-code-review · 2026-02-05T18:34:26Z

PR-Agent: could not fine a component named unit in a supported language in this PR.

harche · 2026-02-05T23:43:54Z

/retest-required

JoelSpeed · 2026-02-06T12:56:24Z

/verified by CI

Techpreview isn't blowing up so this looks good

/retest

openshift-ci-robot · 2026-02-06T12:56:38Z

@JoelSpeed: This PR has been marked as verified by CI.

Details

In response to this:

/verified by CI
Techpreview isn't blowing up so this looks good
/retest

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.

JoelSpeed · 2026-02-06T17:31:35Z

/test integration

qodo-code-review · 2026-02-06T17:31:41Z

PR-Agent: could not fine a component named integration in a supported language in this PR.

haircommander · 2026-02-06T19:09:21Z

/retest

harche · 2026-02-06T22:00:45Z

/test integration

qodo-code-review · 2026-02-06T22:00:50Z

PR-Agent: could not fine a component named integration in a supported language in this PR.

harche · 2026-02-06T22:02:28Z

PR-Agent: could not fine a component named integration in a supported language in this PR.

No you silly bot! the test is restarted with /test <name>

(I don't know why I am arguing with a robot on a Friday evening)

openshift-ci · 2026-02-06T22:35:14Z

@harche: all tests passed!

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.

openshift-ciBot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Feb 5, 2026

openshift-ciBot added the size/M Denotes a PR that changes 30-99 lines, ignoring generated files. label Feb 5, 2026

openshift-ciBot requested review from JoelSpeed and everettraven February 5, 2026 15:06

JoelSpeed reviewed Feb 5, 2026
View reviewed changes

harche force-pushed the DRAPartitionableDevices branch from 317f36c to bee641aCompare February 5, 2026 15:25

Add DRAPartitionableDevices to TechPreviewNoUpgrade
1fece6c
Signed-off-by: Harshal Patil <12152047+harche@users.noreply.github.com>

harche force-pushed the DRAPartitionableDevices branch from bee641a to 1fece6cCompare February 5, 2026 15:44

harche marked this pull request as draft February 5, 2026 16:06

harche marked this pull request as ready for review February 5, 2026 16:06

harche changed the title ~~WIP: Add DRAPartitionableDevices to TechPreviewNoUpgrade~~OCPNODE-4078: Add DRAPartitionableDevices to TechPreviewNoUpgradeFeb 5, 2026

openshift-ciBot assigned haircommander Feb 5, 2026

openshift-ciBot added the lgtm Indicates that a PR is ready to be merged. label Feb 5, 2026

openshift-ciBot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Feb 5, 2026

JoelSpeed removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Feb 5, 2026

openshift-ci-robot added the verified Signifies that the PR passed pre-merge verification criteria label Feb 6, 2026

openshift-merge-botBot merged commit 639b501 into openshift:masterFeb 6, 2026
28 checks passed

harche deleted the DRAPartitionableDevices branch February 6, 2026 22:52

coderabbitaiBot mentioned this pull request Jun 9, 2026
Implement E2Es for Partitionable Devices openshift/kueue-operator#1990
Merged

harche mentioned this pull request Aug 18, 2026
OCPNODE-4659: Register DRANodeAllocatableResources feature gate for TechPreview #2981
Open

Uh oh!

Conversation

harche commented Feb 5, 2026• edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

openshift-ci-robot commented Feb 5, 2026

Uh oh!

qodo-code-reviewBot commented Feb 5, 2026

Review Summary by Qodo

Walkthroughs

File Changes

Uh oh!

openshift-ciBot commented Feb 5, 2026

Uh oh!

qodo-code-reviewBot commented Feb 5, 2026• edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Code Review by Qodo

Uh oh!

harche commented Feb 5, 2026

Uh oh!

qodo-code-reviewBot commented Feb 5, 2026

Uh oh!

JoelSpeedFeb 5, 2026

Choose a reason for hiding this comment

Uh oh!

harcheFeb 5, 2026

Choose a reason for hiding this comment

Uh oh!

coderabbitaiBot commented Feb 5, 2026• edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Walkthrough

Uh oh!

harche commented Feb 5, 2026

Uh oh!

harche commented Feb 5, 2026

Uh oh!

qodo-code-reviewBot commented Feb 5, 2026

Review Summary by Qodo

Walkthroughs

File Changes

Uh oh!

harche commented Feb 5, 2026

Uh oh!

qodo-code-reviewBot commented Feb 5, 2026

Uh oh!

harche commented Feb 5, 2026

Uh oh!

haircommander commented Feb 5, 2026

Uh oh!

openshift-ci-robot commented Feb 5, 2026

Uh oh!

harche commented Feb 5, 2026

Uh oh!

qodo-code-reviewBot commented Feb 5, 2026

Uh oh!

harche commented Feb 5, 2026

Uh oh!

qodo-code-reviewBot commented Feb 5, 2026• edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

harche commented Feb 5, 2026

Uh oh!

JoelSpeed commented Feb 5, 2026

Uh oh!

openshift-ciBot commented Feb 5, 2026

Uh oh!

harche commented Feb 5, 2026

Uh oh!

JoelSpeed commented Feb 5, 2026

Uh oh!

openshift-ci-robot commented Feb 5, 2026

Uh oh!

harche commented Feb 5, 2026

Uh oh!

qodo-code-reviewBot commented Feb 5, 2026• edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

harche commented Feb 5, 2026

Uh oh!

JoelSpeed commented Feb 6, 2026

harche commented Feb 5, 2026•
edited
Loading

qodo-code-reviewBot commented Feb 5, 2026•
edited
Loading

coderabbitaiBot commented Feb 5, 2026•
edited
Loading

qodo-code-reviewBot commented Feb 5, 2026•
edited
Loading

qodo-code-reviewBot commented Feb 5, 2026•
edited
Loading

qodo-code-reviewBot commented Feb 6, 2026•
edited
Loading

qodo-code-reviewBot commented Feb 6, 2026•
edited
Loading