performa/os-faults
Code Issues Proposed changes

Rename private_key to private_key_file for devstack driver

Browse Source 
It makes names consistent for fuel and devstack drivers.

Change-Id: I39357abfe836f0422d7597cc89487a81b00c53a9
This commit is contained in:
Anton Studenov 2016-10-12 12:21:24 +03:00
parent be1125cff0
commit 065bed3f65
2 changed files with 4 additions and 2 deletions
os_faults
drivers
devstack.py
tests/unit
test_os_faults.py

4
os_faults/drivers/devstack.py

@ -106,7 +106,7 @@ class DevStackManagement(cloud_management.CloudManagement):
'properties': {
'address': {'type': 'string'},
'username': {'type': 'string'},
'private_key': {'type': 'string'},
'private_key_file': {'type': 'string'},
},
'required': ['address', 'username'],
@ -118,7 +118,7 @@ class DevStackManagement(cloud_management.CloudManagement):
self.address = cloud_management_params['address']
self.username = cloud_management_params['username']
self.private_key_file = cloud_management_params.get('private_key')
self.private_key_file = cloud_management_params.get('private_key_file')
self.executor = executor.AnsibleRunner(
remote_user=self.username, private_key_file=self.private_key_file,

2
os_faults/tests/unit/test_os_faults.py

@ -35,6 +35,7 @@ class OSFaultsTestCase(test.TestCase):
'args': {
'address': '10.30.00.5',
'username': 'root',
'private_key_file': '/my/path/pk.key',
}
},
'power_management': {
@ -52,6 +53,7 @@ class OSFaultsTestCase(test.TestCase):
'args': {
'address': 'devstack.local',
'username': 'developer',
'private_key_file': '/my/path/pk.key',
}
}
}