Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add support for "High with Hyperthreading" option for latency_sensitivity #2169

Open
4 tasks done
Guerckan opened this issue Apr 16, 2024 · 3 comments
Open
4 tasks done
Labels
area/vm Area: Virtual Machines enhancement Type: Enhancement upstream/govmomi Upstream: vmware/govmomi vsphere/v8 vSphere 8.0
Milestone

Comments

@Guerckan
Copy link

Guerckan commented Apr 16, 2024

Community Guidelines

  • I have read and agree to the HashiCorp Community Guidelines .
  • Vote on this issue by adding a 馃憤 reaction to the original issue initial description to help the maintainers prioritize.
  • Do not leave "+1" or other comments that do not add relevant information or questions.
  • If you are interested in working on this issue or have submitted a pull request, please leave a comment.

Description

As introduced by vSphere 8.0, the "latency sensitivity" virtual machine setting now has a new possible value named "High with Hyperthreading".
See VMWare documentation supporting this statement : https://docs.vmware.com/en/VMware-vSphere/8.0/vsphere-vm-administration/GUID-F5DAC793-7B13-418F-949E-0CD80CEC5D1E.html

Current provider implementation (2.7.0 at the time of writing) of the resource vsphere_virtual_machine does not support this new value as specified here : https://registry.terraform.io/providers/hashicorp/vsphere/latest/docs/resources/virtual_machine#latency_sensitivity

Use Case(s)

This setting is particularly interesting for network appliances virtual machines and the new functionality of "high with hyperthreading" offers a way to reduce the computing power footprint of a latency sensitivity "high" setting on the host without sacrificing much of its advantages.

Potential Terraform Provider Configuration

No response

References

No response

@Guerckan Guerckan added the enhancement Type: Enhancement label Apr 16, 2024
Copy link

Hello, Guerckan! 馃枑

Thank you for submitting an issue for this provider. The issue will now enter into the issue lifecycle.

If you want to contribute to this project, please review the contributing guidelines and information on submitting pull requests.

@tenthirtyam tenthirtyam added the area/vm Area: Virtual Machines label Jun 11, 2024
@tenthirtyam
Copy link
Collaborator

This item has an upstream dependency on [vmware/govmomi](https://github.com/vmware/govmomi/blob/bf28314d9e3287ad1674f80d8a80d2245c0159ae/vim25/types/enum.go#L4846-L4881) which would need to support values other than Low, Normal, Medium, and High. Once available and the dependency is updated, the virtualMachineLatencySensitivityAllowedValues could be updated to support the new value.

For example:

var virtualMachineLatencySensitivityAllowedValues = []string{
string(types.LatencySensitivitySensitivityLevelLow),
string(types.LatencySensitivitySensitivityLevelNormal),
string(types.LatencySensitivitySensitivityLevelMedium),
string(types.LatencySensitivitySensitivityLevelHigh),
}

to...

var virtualMachineLatencySensitivityAllowedValues = []string{
	string(types.LatencySensitivitySensitivityLevelLow),
	string(types.LatencySensitivitySensitivityLevelNormal),
	string(types.LatencySensitivitySensitivityLevelMedium),
	string(types.LatencySensitivitySensitivityLevelHigh),
        string(types.LatencySensitivitySensitivityLevelHighHyperthreading), # Example
}

Please consider opening an upstream enhancement in vmware/govmomi.

Ryan Johnson
Distinguished Engineer, VMware by Broadcom

@tenthirtyam tenthirtyam added the upstream/govmomi Upstream: vmware/govmomi label Jun 12, 2024
@tenthirtyam tenthirtyam changed the title Add support for VM latency sensitivity "High with Hyperthreading" Add support for "High with Hyperthreading" in VM latency_sensitivity Jun 12, 2024
@tenthirtyam tenthirtyam added the vsphere/v8 vSphere 8.0 label Jun 12, 2024
@tenthirtyam tenthirtyam changed the title Add support for "High with Hyperthreading" in VM latency_sensitivity Add support for "High with Hyperthreading" option for latency_sensitivity Jun 12, 2024
@tenthirtyam tenthirtyam added this to the Backlog milestone Jun 12, 2024
@Guerckan
Copy link
Author

Thank you for your help @tenthirtyam !
Upstream enhancement request has been opened as requested.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/vm Area: Virtual Machines enhancement Type: Enhancement upstream/govmomi Upstream: vmware/govmomi vsphere/v8 vSphere 8.0
Projects
None yet
Development

No branches or pull requests

2 participants