diff options
author | Abhijeet Kasurde <akasurde@redhat.com> | 2019-07-22 09:03:25 +0530 |
---|---|---|
committer | Toshio Kuratomi <a.badger@gmail.com> | 2019-07-22 16:37:02 -0700 |
commit | 302de41a314e5ce5e72373873c2dfe767e4342da (patch) | |
tree | 796fdfe5901ba5991fc56f63712c6c4a75a040aa | |
parent | e5f68cad5a9e46242d80c6189d2eabb5f8432608 (diff) | |
download | ansible-302de41a314e5ce5e72373873c2dfe767e4342da.tar.gz |
[2.8] VMware: Accept 0 as valid value for memory reservation
vmware_guest accepts 0MB as valid value for memory reservation in
virtual machine hardware configuration. This fixes the regression
introduced via 193f69064fb40a83e3e7d2112ef24868b45233b3.
Fixes: #59190
Signed-off-by: Abhijeet Kasurde <akasurde@redhat.com>
(cherry picked from commit 1f49abb51c208ac5e713740561952b8411c0761b)
Signed-off-by: Abhijeet Kasurde <akasurde@redhat.com>
4 files changed, 159 insertions, 2 deletions
diff --git a/changelogs/fragments/59190-vmware_guest-mem_reservation.yml b/changelogs/fragments/59190-vmware_guest-mem_reservation.yml new file mode 100644 index 0000000000..edb637caef --- /dev/null +++ b/changelogs/fragments/59190-vmware_guest-mem_reservation.yml @@ -0,0 +1,2 @@ +bugfixes: + - vmware_guest accepts 0 MB of memory reservation, fix regression introduced via 193f69064fb40a83e3e7d2112ef24868b45233b3 (https://github.com/ansible/ansible/issues/59190). diff --git a/lib/ansible/modules/cloud/vmware/vmware_guest.py b/lib/ansible/modules/cloud/vmware/vmware_guest.py index 0711ca1f5c..6fcb5541bd 100644 --- a/lib/ansible/modules/cloud/vmware/vmware_guest.py +++ b/lib/ansible/modules/cloud/vmware/vmware_guest.py @@ -897,9 +897,10 @@ class PyVmomiHelper(PyVmomi): memory_allocation.limit = mem_limit if vm_obj is None or memory_allocation.limit != vm_obj.config.memoryAllocation.limit: rai_change_detected = True - if 'mem_reservation' in self.params['hardware'] or 'memory_reservation' in self.params['hardware']: - mem_reservation = self.params['hardware'].get('mem_reservation') or self.params['hardware'].get('memory_reservation') or None + mem_reservation = self.params['hardware'].get('mem_reservation') + if mem_reservation is None: + mem_reservation = self.params['hardware'].get('memory_reservation') try: mem_reservation = int(mem_reservation) except ValueError: diff --git a/test/integration/targets/vmware_guest/tasks/main.yml b/test/integration/targets/vmware_guest/tasks/main.yml index ef230fac7e..e2e54f5963 100644 --- a/test/integration/targets/vmware_guest/tasks/main.yml +++ b/test/integration/targets/vmware_guest/tasks/main.yml @@ -29,3 +29,4 @@ - include: boot_firmware_d1_c1_f0.yml - include: clone_with_convert.yml #- include: max_connections.yml +- include: mem_reservation.yml
\ No newline at end of file diff --git a/test/integration/targets/vmware_guest/tasks/mem_reservation.yml b/test/integration/targets/vmware_guest/tasks/mem_reservation.yml new file mode 100644 index 0000000000..ff71ee3093 --- /dev/null +++ b/test/integration/targets/vmware_guest/tasks/mem_reservation.yml @@ -0,0 +1,153 @@ +# Test code for the vmware_guest module. +# Copyright: (c) 2019, Abhijeet Kasurde <akasurde@redhat.com> +# GNU General Public License v3.0+ (see COPYING or https://www.gnu.org/licenses/gpl-3.0.txt) + +- name: Wait for Flask controller to come up online + wait_for: + host: "{{ vcsim }}" + port: 5000 + state: started + +- name: kill vcsim + uri: + url: http://{{ vcsim }}:5000/killall +- name: start vcsim with no folders + uri: + url: http://{{ vcsim }}:5000/spawn?datacenter=1&cluster=1&folder=0 + register: vcsim_instance + +- name: Wait for Flask controller to come up online + wait_for: + host: "{{ vcsim }}" + port: 443 + state: started + +- name: get a list of VMS from vcsim + uri: + url: http://{{ vcsim }}:5000/govc_find?filter=VM + register: vmlist + +- debug: var=vcsim_instance +- debug: var=vmlist + +- &add_mem_reserve + name: Create new VMs with mem_reservation as 0 + vmware_guest: + validate_certs: False + hostname: "{{ vcsim }}" + username: "{{ vcsim_instance['json']['username'] }}" + password: "{{ vcsim_instance['json']['password'] }}" + name: "{{ 'vm_mem_res_' + item|basename }}" + guest_id: centos64Guest + datacenter: "{{ (item|basename).split('_')[0] }}" + hardware: + num_cpus: 4 + memory_mb: 512 + mem_reservation: 0 + disk: + - size: 1gb + type: thin + autoselect_datastore: True + state: present + folder: "{{ item|dirname }}" + with_items: "{{ vmlist['json'] }}" + register: mem_reserve_result_0001 + +- debug: var=mem_reserve_result_0001 + +- name: Assert that changes were made + assert: + that: + - "mem_reserve_result_0001.results|map(attribute='changed')|unique|list == [true]" + +- <<: *add_mem_reserve + name: Again create new VMs with mem_reservation as 0 + register: mem_reserve_result_0002 + +- debug: var=mem_reserve_result_0002 + +- name: Assert that changes were not made + assert: + that: + - "mem_reserve_result_0002.results|map(attribute='changed')|unique|list == [false]" + +- &add_memory_reserve + name: Create new VMs with memory_reservation as 0 + vmware_guest: + validate_certs: False + hostname: "{{ vcsim }}" + username: "{{ vcsim_instance['json']['username'] }}" + password: "{{ vcsim_instance['json']['password'] }}" + name: "{{ 'vm_memory_res_' + item|basename }}" + guest_id: centos64Guest + datacenter: "{{ (item|basename).split('_')[0] }}" + hardware: + num_cpus: 4 + memory_mb: 512 + memory_reservation: 0 + disk: + - size: 1gb + type: thin + autoselect_datastore: True + state: present + folder: "{{ item|dirname }}" + with_items: "{{ vmlist['json'] }}" + register: memory_reserve_result_0003 + +- debug: var=memory_reserve_result_0003 + +- name: Assert that changes were made + assert: + that: + - "memory_reserve_result_0003.results|map(attribute='changed')|unique|list == [true]" + +- <<: *add_memory_reserve + name: Again create new VMs with memory_reservation as 0 + register: memory_reserve_result_0004 + +- debug: var=memory_reserve_result_0004 + +- name: Assert that changes were not made + assert: + that: + - "memory_reserve_result_0004.results|map(attribute='changed')|unique|list == [false]" + +- &without_memory_reserve + name: Create new VMs without mem_reservation or memory_reservation + vmware_guest: + validate_certs: False + hostname: "{{ vcsim }}" + username: "{{ vcsim_instance['json']['username'] }}" + password: "{{ vcsim_instance['json']['password'] }}" + name: "{{ 'vm_no_memory_res_' + item|basename }}" + guest_id: centos64Guest + datacenter: "{{ (item|basename).split('_')[0] }}" + hardware: + num_cpus: 4 + memory_mb: 512 + disk: + - size: 1gb + type: thin + autoselect_datastore: True + state: present + folder: "{{ item|dirname }}" + with_items: "{{ vmlist['json'] }}" + register: no_memory_reserve_result_0004 + +- debug: var=no_memory_reserve_result_0004 + +- name: Assert that changes were made + assert: + that: + - "no_memory_reserve_result_0004.results|map(attribute='changed')|unique|list == [true]" + +- <<: *without_memory_reserve + name: Again create new VMs without mem_reservation or memory_reservation + register: no_memory_reserve_result_0005 + +- debug: var=no_memory_reserve_result_0005 + +- name: Assert that changes were not made + assert: + that: + - "no_memory_reserve_result_0005.results|map(attribute='changed')|unique|list == [false]" |