This is an automated email from the ASF dual-hosted git repository.
arvindsh pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/fluo-muchos.git
The following commit(s) were added to refs/heads/master by this push:
new 5ddf275 Update deprecated _facts Azure modules to _info (#323)
5ddf275 is described below
commit 5ddf27508f7a5515a20d70cf4e2902783ff7558b
Author: Arvind Shyamsundar <[email protected]>
AuthorDate: Tue Feb 18 10:28:09 2020 -0800
Update deprecated _facts Azure modules to _info (#323)
This addresses some deprecation warnings with older versions of Ansible
for Azure. An Azure ADLS Gen2 specific version check is also added.
---
ansible/roles/azure/tasks/create_adlsgen2.yml | 18 +++++++++---------
ansible/roles/azure/tasks/create_vmss.yml | 5 ++++-
ansible/roles/azure/tasks/main.yml | 9 ++++++++-
3 files changed, 21 insertions(+), 11 deletions(-)
diff --git a/ansible/roles/azure/tasks/create_adlsgen2.yml
b/ansible/roles/azure/tasks/create_adlsgen2.yml
index e074599..042903e 100644
--- a/ansible/roles/azure/tasks/create_adlsgen2.yml
+++ b/ansible/roles/azure/tasks/create_adlsgen2.yml
@@ -110,7 +110,7 @@
# Retrieving facts about User Assigned Identity
- name: Get facts for User Assigned Identity
- azure_rm_resource_facts:
+ azure_rm_resource_info:
resource_group: "{{ resource_group }}"
provider: ManagedIdentity
resource_type: userAssignedIdentities
@@ -134,7 +134,7 @@
# Retrieve facts about role assignment
- name: Get role definition id for "Storage Blob Data Owner"
- azure_rm_resource_facts:
+ azure_rm_resource_info:
resource_group: "{{ resource_group }}"
provider: Authorization
resource_type: roleDefinitions
@@ -143,25 +143,25 @@
register: RoleDefinitionInfo
# Retrieve storage acount informationn.
-- name: Check if the storage accounts is visible
- azure_rm_storageaccount_facts:
+- name: Check if the storage accounts are visible
+ azure_rm_storageaccount_info:
resource_group: "{{ resource_group }}"
name: "{{ item.split('@')[1].split('.')[0] }}"
register: StorageAccountsInfo
retries: 20
delay: 15
- until:
StorageAccountsInfo.storageaccounts|sum(start=[])|map(attribute='id')|join('')
is defined
+ until: StorageAccountsInfo.storageaccounts | map(attribute='id') | join('')
| length > 0
loop:
"{{ InstanceVolumes }}"
# Retrieve storage accounts id creeated -- Used for account assignments
- name: Get the id of storage accounts created
set_fact:
- StorageAccountsId:
"{{StorageAccountsInfo.results|map(attribute='ansible_facts')|map(attribute='azure_storageaccounts')|sum(start=[])|map(attribute='id')|list|unique
}}"
+ StorageAccountsId: "{{StorageAccountsInfo.results |
sum(attribute='storageaccounts', start=[]) | map(attribute='id') | list |
unique }}"
# Adding this module since role aassignment fails if it already exists.
- name: Get facts about role assignment
- azure_rm_roleassignment_facts:
+ azure_rm_roleassignment_info:
scope: "{{ item }}"
assignee: "{{
UserAssignedIdentityInfo.response|map(attribute='properties')|map(attribute='principalId')|list|join('')
}}"
role_definition_id: "{{
RoleDefinitionInfo.response|map(attribute='id')|list|join('') }}"
@@ -200,8 +200,8 @@
resource_group: "{{ resource_group }}"
storage_account_name: "{{ item.split('@')[1].split('.')[0] }}"
container: "{{ item.split('@')[0].split('://')[1] }}"
- retries: 30
- delay: 15
+ retries: 20
+ delay: 30
register: createfsresult
until: createfsresult is succeeded and (createfsresult.changed == False or
(createfsresult.changed == True and createfsresult.container|length > 0))
loop:
diff --git a/ansible/roles/azure/tasks/create_vmss.yml
b/ansible/roles/azure/tasks/create_vmss.yml
index c47fcb5..abc5116 100644
--- a/ansible/roles/azure/tasks/create_vmss.yml
+++ b/ansible/roles/azure/tasks/create_vmss.yml
@@ -78,7 +78,10 @@
name: "{{ azure_proxy_host }}-nic"
virtual_network: "{{ vnet }}"
subnet: "{{ subnet }}"
- public_ip_name: "{{ azure_proxy_host }}-ip"
+ ip_configurations:
+ - name: default
+ public_ip_address_name: "{{ azure_proxy_host }}-ip"
+ primary: True
security_group: "{{ azure_proxy_host }}-nsg"
when: azure_proxy_host is defined and azure_proxy_host != '' and
azure_proxy_host != None
diff --git a/ansible/roles/azure/tasks/main.yml
b/ansible/roles/azure/tasks/main.yml
index 37df069..e45dadf 100644
--- a/ansible/roles/azure/tasks/main.yml
+++ b/ansible/roles/azure/tasks/main.yml
@@ -17,7 +17,14 @@
# limitations under the License.
#
-# tasks file for azure
+- name: ensure Ansible version is >= 2.9 if ADLS Gen2 is to be used
+ assert:
+ quiet: yes
+ that: ansible_version.full is version('2.9.0', '>=')
+ fail_msg: "Ansible 2.9 or above is required for using Azure ADLS Gen2 in
Muchos.
+ Update it using pip install --upgrade ansible[azure]"
+ when: use_adlsg2
+
- import_tasks: create_vmss.yml
- import_tasks: create_adlsgen2.yml
when: use_adlsg2
