Use correct esxi host variable in vmware_cfg_backup

Fixed bug where hostname could not be found and minor documentation changes.

(cherry picked from commit 8f5320435f)
This commit is contained in:
Abhijeet Kasurde 2018-04-17 15:20:09 +05:30 committed by GitHub
parent d9436f18f5
commit 5f8758d6a3
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
2 changed files with 8 additions and 6 deletions

View file

@ -0,0 +1,2 @@
bugfixes:
- Changed hostname variable in order for the esxi host to be found when authentication against a vcenter was done.

View file

@ -31,7 +31,7 @@ requirements:
options:
esxi_hostname:
description:
- Name of ESXi server.
- Name of ESXi server. This is required only if authentication against a vCenter is done.
required: False
dest:
description:
@ -51,17 +51,17 @@ extends_documentation_fragment: vmware.documentation
'''
EXAMPLES = '''
# save the ESXi configuration locally
# save the ESXi configuration locally by authenticating directly against the ESXi host
- name: ESXI backup test
local_action:
module: vmware_cfg_backup
hostname: esxi_host
hostname: esxi_hostname
username: user
password: pass
state: saved
dest: /tmp/
# save the ESXi configuration locally for specific ESXi
# save the ESXi configuration locally by authenticating against the vCenter and selecting the ESXi host
- name: ESXI backup test
local_action:
module: vmware_cfg_backup
@ -133,7 +133,7 @@ class VMwareConfigurationBackup(PyVmomi):
self.module.fail_json(msg="Source file {} does not exist".format(self.src))
url = self.host.configManager.firmwareSystem.QueryFirmwareConfigUploadURL()
url = url.replace('*', self.hostname)
url = url.replace('*', self.host.name)
# find manually the url if there is a redirect because urllib2 -per RFC- doesn't do automatic redirects for PUT requests
try:
request = open_url(url=url, method='HEAD', validate_certs=self.validate_certs)
@ -169,7 +169,7 @@ class VMwareConfigurationBackup(PyVmomi):
def save_configuration(self):
url = self.host.configManager.firmwareSystem.BackupFirmwareConfiguration()
url = url.replace('*', self.hostname)
url = url.replace('*', self.host.name)
if os.path.isdir(self.dest):
filename = url.rsplit('/', 1)[1]
self.dest = os.path.join(self.dest, filename)