Skip to content

Commit

Permalink
bug fixing in azure_rm_aks module(#170)
Browse files Browse the repository at this point in the history
* bug fixing in azure_rm_aks module
  • Loading branch information
Fred-sun authored Jun 24, 2020
1 parent 9791422 commit 56d53e4
Show file tree
Hide file tree
Showing 4 changed files with 12 additions and 9 deletions.
2 changes: 2 additions & 0 deletions plugins/modules/azure_rm_aks.py
Original file line number Diff line number Diff line change
Expand Up @@ -377,6 +377,8 @@ def create_addon_dict(addon):
addon = addon or dict()
for key in addon.keys():
result[key] = addon[key].config
if result[key] is None:
result[key] = {}
result[key]['enabled'] = addon[key].enabled
return result

Expand Down
2 changes: 1 addition & 1 deletion requirements-azure.txt
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ azure-mgmt-cdn==3.0.0
azure-mgmt-compute==10.0.0
azure-mgmt-containerinstance==1.4.0
azure-mgmt-containerregistry==2.0.0
azure-mgmt-containerservice==4.4.0
azure-mgmt-containerservice==9.1.0
azure-mgmt-dns==2.1.0
azure-mgmt-keyvault==1.1.0
azure-mgmt-marketplaceordering==0.1.0
Expand Down
2 changes: 1 addition & 1 deletion tests/integration/targets/azure_rm_aks/aliases
Original file line number Diff line number Diff line change
@@ -1,3 +1,3 @@
cloud/azure
unsupported
destructive
shippable/azure/group6
15 changes: 8 additions & 7 deletions tests/integration/targets/azure_rm_aks/tasks/main.yml
Original file line number Diff line number Diff line change
@@ -1,5 +1,6 @@
- set_fact:
rpfx: "{{ resource_group | hash('md5') | truncate(8, True, '') }}"
noderpfx: "{{ resource_group | hash('md5') | truncate(4, True, '') }}"

- name: Find available k8s version
azure_rm_aksversion_info:
Expand Down Expand Up @@ -27,7 +28,7 @@
enable_auto_scaling: True
max_count: 3
min_count: 1
node_resource_group: nodetest
node_resource_group: "node{{ noderpfx }}"
enable_rbac: yes
check_mode: yes

Expand Down Expand Up @@ -63,7 +64,7 @@
enable_auto_scaling: True
max_count: 3
min_count: 1
node_resource_group: nodetest
node_resource_group: "node{{ noderpfx }}"
enable_rbac: yes
register: output

Expand Down Expand Up @@ -105,7 +106,7 @@
enable_auto_scaling: True
max_count: 3
min_count: 1
node_resource_group: nodetest
node_resource_group: "node{{ noderpfx }}"
enable_rbac: yes
register: output

Expand Down Expand Up @@ -137,7 +138,7 @@
count: 1
vm_size: Standard_DS1_v2
type: VirtualMachineScaleSets
node_resource_group: nodetest
node_resource_group: "node{{ noderpfx }}"
addon:
http_application_routing: {}
network_profile:
Expand Down Expand Up @@ -171,7 +172,7 @@
count: 1
vm_size: Standard_DS1_v2
type: VirtualMachineScaleSets
node_resource_group: nodetest
node_resource_group: "node{{ noderpfx }}"
addon:
http_application_routing: {}
network_profile:
Expand Down Expand Up @@ -204,7 +205,7 @@
enable_auto_scaling: True
max_count: 6
min_count: 3
node_resource_group: nodetest
node_resource_group: "node{{ noderpfx }}"
enable_rbac: yes
register: output

Expand Down Expand Up @@ -235,7 +236,7 @@
enable_auto_scaling: True
max_count: 6
min_count: 3
node_resource_group: nodetest
node_resource_group: "node{{ noderpfx }}"
enable_rbac: yes
register: output

Expand Down

0 comments on commit 56d53e4

Please sign in to comment.