cmueller/ansible
1
0
Fork 0
Code Issues Pull requests Projects Releases Wiki Activity

proper approach to query paged resources (#53500)

Browse source 
* proper approach to query paged resources

* fixed pep8

* removed unnecessary check
This commit is contained in:
Zim Kalinowski 2019-03-11 15:13:41 +08:00 committed by ansibot
parent a41028244d
commit 9413a6c3f1
1 changed files with 17 additions and 10 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

27
lib/ansible/modules/cloud/azure/azure_rm_resource_facts.py
View file

@ -183,18 +183,25 @@ class AzureRMResourceFacts(AzureRMModuleBase):
header_parameters = {}
header_parameters['Content-Type'] = 'application/json; charset=utf-8'
skiptoken = None
response = self.mgmt_client.query(self.url, "GET", query_parameters, header_parameters, None, [200, 404], 0, 0)
try:
response = json.loads(response.text)
if response is list:
self.results['response'] = response
else:
self.results['response'] = [response]
except Exception:
self.results['response'] = []
while True:
if skiptoken:
query_parameters['skiptoken'] = skiptoken
response = self.mgmt_client.query(self.url, "GET", query_parameters, header_parameters, None, [200, 404], 0, 0)
try:
response = json.loads(response.text)
if isinstance(response, dict):
if response.get('value'):
self.results['response'] = self.results['response'] + response['value']
skiptoken = response.get('nextLink')
else:
self.results['response'] = self.results['response'] + [response]
except Exception:
self.results['response'] = []
if not skiptoken:
break
return self.results