Skip to content

Add PodVMInfo type and method#3850

Merged
akutz merged 3 commits intovmware:mainfrom
rohith05:topic/rj/podVMInfo
Sep 10, 2025
Merged

Add PodVMInfo type and method#3850
akutz merged 3 commits intovmware:mainfrom
rohith05:topic/rj/podVMInfo

Conversation

@rohith05
Copy link
Copy Markdown

@rohith05 rohith05 commented Sep 2, 2025

Description

This patch adds support to PodVMInfo VMODL APIs under HostRuntimeInfo which includes hasPodVM and podVMOverheadInfo properties. In addition this patch also adds UpdatePodVMProperty method under HostSystem. The PodVMInfo property and method will be updated by ESXi host and will be consumed by DRS to determine PodVM placement. The change also includes a unit test that tests the setting of PodVMInfo property.

Note: PodVMOverheadInfo that is currently exposed as part of HostCapability will be removed
as a follow-up change, once existing consumers have moved away from it.

How Has This Been Tested?

  • make test:
    === RUN TestPodVMInfo
    --- PASS: TestPodVMInfo (0.69s)
  • Verified UpdatePodVMProperty() method from ESXi:
    --- PASS: TestSetOverheadVMODLProperty (0.20s)
    PASS

Copy link
Copy Markdown
Member

@akutz akutz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@rohith05 ,

I'm not sure we can support this change in the public repo as you've updated a generated type that will be wiped once the types are regenerated. You may need to make this change in the internal GoVmomi mirror.

Rohith Jagannathan added 3 commits September 9, 2025 21:02
This patch adds support to PodVMInfo VMODL APIs under HostRuntimeInfo which
includes hasPodVM and podVMOverheadInfo properties. In addition this patch
also adds UpdatePodVMProperty method under HostSystem. The PodVMInfo property
and method will be updated by ESXi host and will be consumed by DRS to
determine PodVM placement. The change also includes a unit test that tests
the setting of PodVMInfo property.

Testing:
- make test:
  === RUN   TestPodVMInfo
  --- PASS: TestPodVMInfo (0.69s)
- Verified UpdatePodVMProperty() method from ESXi:
  --- PASS: TestSetOverheadVMODLProperty (0.20s)
  PASS

Signed-off-by: Rohith Jagannathan <rohith.jagannathan@broadcom.com>
Signed-off-by: Rohith Jagannathan <rohith.jagannathan@broadcom.com>
Signed-off-by: Rohith Jagannathan <rohith.jagannathan@broadcom.com>
@akutz akutz merged commit 2575a1f into vmware:main Sep 10, 2025
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants