Finish updating the role_prefix

The update to move to role_prefix was incomplete.
There was also an issue with the networking_stack that was fixed as
well.

Author: p3ck

README.md

@@ -69,10 +69,10 @@ Once installed, you can reference the cloud.azure_ops collection content by its
             ansible.builtin.include_role:
                 name: cloud.azure_ops.azure_load_balancer_with_public_ip
             vars:
-                operation: create
-                azure_resource_group: "{{ resource_group }}"
-                azure_load_balancer:
-                name: "{{ resource_group }}-lb"
+                azure_load_balancer_with_public_ip_operation: create
+                azure_load_balancer_with_public_ip_azure_resource_group: "{{ resource_group }}"
+                azure_load_balancer_with_public_ip_load_balancer:
+                    name: "{{ resource_group }}-lb"
 ```
 
 ### See Also

changelogs/fragments/20240307-update_prefixes.yml

@@ -0,0 +1,7 @@
+bugfixes:
+  - Update README.md with proper variable names in example
+  - Update playbooks/roles/scale_virtual_machine/tasks/main.yml to use correct operation variable
+  - Fix syntax in roles/azure_manage_networking_stack/README.md
+  - Use correct variables in roles/azure_manage_networking_stack/tasks/create.yml
+  - Update roles/azure_manage_security_group/tasks/main.yml to use correct operation variable
+  - Update roles/azure_virtual_machine_with_public_ip/tasks/main.yml to use correct prefix vars

playbooks/roles/scale_virtual_machine/tasks/main.yml

@@ -9,5 +9,5 @@
     msg: Azure region must be defined as scale_virtual_machine_region
   when: scale_virtual_machine_region is not defined
 
-- name: Include operation tasks
-  ansible.builtin.include_tasks: "{{ operation }}.yml"
+- name: Include scale_virtual_machine_operation tasks
+  ansible.builtin.include_tasks: "{{ scale_virtual_machine_operation }}.yml"

roles/azure_manage_networking_stack/README.md

@@ -43,7 +43,7 @@ Example Playbook
            azure_manage_networking_stack_vnet_address_prefixes_cidr:
             - "10.1.0.0/16"
             - "172.100.0.0/16"
-           azure_manage_networking_stack_subnet_address_prefixes_cidr
+           azure_manage_networking_stack_subnet_address_prefixes_cidr:
             - "172.100.0.0/8"
            azure_manage_networking_stack_tags:
              tag0: "tag0"
@@ -59,4 +59,4 @@ See [LICENCE](https://github.com/redhat-cop/cloud.azure_ops/blob/main/LICENSE) t
 Author Information
 ------------------
 
-- Ansible Cloud Content Team
+- Ansible Cloud Content Team

roles/azure_manage_networking_stack/tasks/create.yml

@@ -1,7 +1,7 @@
 ---
 - name: Check that resource group exists
   azure.azcollection.azure_rm_resourcegroup_info:
-    name: "{{ azure_resource_group }}"
+    name: "{{ azure_manage_networking_stack_resource_group }}"
   register: result
 
 - name: Create resource group when it does not exists
@@ -16,16 +16,16 @@
 
 - name: Create azure virtual network
   azure.azcollection.azure_rm_virtualnetwork:
-    location: "{{ azure_region }}"
-    name: "{{ azure_virtual_network }}"
-    address_prefixes_cidr: "{{ azure_vnet_address_prefixes_cidr }}"
-    resource_group: "{{ azure_resource_group }}"
-    tags: "{{ azure_tags | default(omit) }}"
+    location: "{{ azure_manage_networking_stack_region }}"
+    name: "{{ azure_manage_networking_stack_virtual_network }}"
+    address_prefixes_cidr: "{{ azure_manage_networking_stack_vnet_address_prefixes_cidr }}"
+    resource_group: "{{ azure_manage_networking_stack_resource_group }}"
+    tags: "{{ azure_manage_networking_stack_tags | default(omit) }}"
 
 - name: Create azure subnet
   azure.azcollection.azure_rm_subnet:
-    name: "{{ azure_subnet }}"
-    virtual_network: "{{ azure_virtual_network }}"
-    address_prefix_cidr: "{{ azure_subnet_address_prefixes_cidr }}"
-    resource_group: "{{ azure_resource_group }}"
-    security_group: "{{ azure_security_group | default(omit) }}"
+    name: "{{ azure_manage_networking_stack_subnet }}"
+    virtual_network: "{{ azure_manage_networking_stack_virtual_network }}"
+    address_prefixes_cidr: "{{ azure_manage_networking_stack_subnet_address_prefixes_cidr }}"
+    resource_group: "{{ azure_manage_networking_stack_resource_group }}"
+    security_group: "{{ azure_manage_networking_stack_security_group | default(omit) }}"

roles/azure_manage_security_group/tasks/main.yml

@@ -1,7 +1,7 @@
 ---
 - name: Check operation validation
   ansible.builtin.fail:
-    msg: Please provide operation as 'create' or 'delete'
+    msg: Please provide azure_manage_security_group_operation as 'create' or 'delete'
   when: azure_manage_security_group_operation not in ['create', 'delete']
 
 - name: Check resource group name

roles/azure_virtual_machine_with_public_ip/tasks/main.yml

@@ -11,12 +11,12 @@
 
 - name: Ensure vm name is defined
   ansible.builtin.fail:
-    msg: "Missing parameter: key 'name' not found in azure_virtual_machine_with_public_ip_operation_vm"
-  when: azure_virtual_machine_with_public_ip_operation_vm.name is not defined
+    msg: "Missing parameter: key 'name' not found in azure_virtual_machine_with_public_ip_vm"
+  when: azure_virtual_machine_with_public_ip_vm.name is not defined
 
 - name: Replace invalid chars in name
   ansible.builtin.set_fact:
-    vm_name: "{{ azure_virtual_machine_with_public_ip_operation_vm.name | regex_replace('[^a-zA-Z0-9]', '-') }}"
+    vm_name: "{{ azure_virtual_machine_with_public_ip_vm.name | regex_replace('[^a-zA-Z0-9]', '-') }}"
 
 - name: Get resource group info
   azure.azcollection.azure_rm_resourcegroup_info:
@@ -28,28 +28,28 @@
     resource_group: "{{ azure_virtual_machine_with_public_ip_resource_group }}"
     name: "{{ vm_name }}"
     started: false
-  when: operation == 'power_off'
+  when: azure_virtual_machine_with_public_ip_operation == 'power_off'
 
 - name: Deallocate VM
   azure.azcollection.azure_rm_virtualmachine:
     resource_group: "{{ azure_virtual_machine_with_public_ip_resource_group }}"
     name: "{{ vm_name }}"
     allocated: false
-  when: operation == 'deallocate'
+  when: azure_virtual_machine_with_public_ip_operation == 'deallocate'
 
 - name: Power On VM
   azure.azcollection.azure_rm_virtualmachine:
     resource_group: "{{ azure_virtual_machine_with_public_ip_resource_group }}"
     name: "{{ vm_name }}"
-  when: operation == 'power_on'
+  when: azure_virtual_machine_with_public_ip_operation == 'power_on'
 
 - name: Restart VM
   azure.azcollection.azure_rm_virtualmachine:
     resource_group: "{{ azure_virtual_machine_with_public_ip_resource_group }}"
     name: "{{ vm_name }}"
     restarted: true
-  when: operation == 'restart'
+  when: azure_virtual_machine_with_public_ip_operation == 'restart'
 
 - name: Create or delete VM
   ansible.builtin.include_tasks: "{{ azure_virtual_machine_with_public_ip_operation }}.yml"
-  when: operation in ['create', 'delete']
+  when: azure_virtual_machine_with_public_ip_operation in ['create', 'delete']