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’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support 'numCoresPerSocket' in vcsim #929

Closed
Akasurde opened this issue Nov 21, 2017 · 0 comments
Closed

Support 'numCoresPerSocket' in vcsim #929

Akasurde opened this issue Nov 21, 2017 · 0 comments

Comments

@Akasurde
Copy link
Contributor

Currently, vcsim does not support 'numCoresPerSocket', so Could you please support this feature ?

Ansible Playbook

---
- name: Create VM with 'numCoresPerSocket'
  hosts: localhost
  tasks:
    - name: create a vm
      vmware_guest:
        datacenter: DC0
        esxi_hostname: DC0_H0
        hostname: 127.0.0.1
        name: DC0_H0_VM4
        folder: /vm
        validate_certs: False
        guest_id: sample
        username: user
        password: pass
        state: poweredon
        disk:
          - size_gb: 1
            type: thin
            datastore: LocalDS_0
        hardware:
            memory_mb: 4096
            num_cpus: 4
            num_cpu_cores_per_socket: 2
            osid: centos64guest
            scsi: paravirtual

Actual PDB output :

(Pdb) vmobj.config.hardware
(vim.vm.VirtualHardware) {
   dynamicType = <unset>,
   dynamicProperty = (vmodl.DynamicProperty) [],
   numCPU = 4,
   numCoresPerSocket = <unset>,
   memoryMB = 4096,

Expected PDB output :

(Pdb) vmobj.config.hardware
(vim.vm.VirtualHardware) {
   dynamicType = <unset>,
   dynamicProperty = (vmodl.DynamicProperty) [],
   numCPU = 4,
   numCoresPerSocket = 2,
   memoryMB = 4096,
dougm added a commit to dougm/govmomi that referenced this issue Nov 21, 2017
We set the default to 1 in the spec, but did not apply to the config.

Fixes vmware#929
dougm added a commit that referenced this issue Nov 21, 2017
We set the default to 1 in the spec, but did not apply to the config.

Fixes #929
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants