Improve VPC info handling in DigitalOcean tasks

Replaced direct access to vpc_info.data with vpc_info.get('data', []) to prevent errors when 'data' is missing. Updated conditional checks and set_fact tasks for more robust handling of VPC information.

Author: vitabaks

automation/roles/cloud_resources/tasks/digitalocean.yml

@@ -143,15 +143,15 @@
       ansible.builtin.set_fact:
         default_ip_range: >-
           {{
-            vpc_info.data
+            vpc_info.get('data', [])
             | selectattr('region', 'equalto', server_location)
             | selectattr('default', 'equalto', true)
             | map(attribute='ip_range')
             | first
           }}
       when:
         - server_network | length < 1
-        - vpc_info.data | selectattr('region', 'equalto', server_location) | selectattr('default', 'equalto', true) | list | length > 0
+        - vpc_info.get('data', []) | selectattr('region', 'equalto', server_location) | selectattr('default', 'equalto', true) | list | length > 0
 
     # if server_network is not specified and there is no default VPC, create a network
     - name: "DigitalOcean: Create a VPC '{{ digital_ocean_vpc_name | default('network-' + server_location | default('')) }}'"
@@ -163,32 +163,41 @@
       register: digital_ocean_vpc
       when:
         - server_network | length < 1
-        - vpc_info.data | selectattr('region', 'equalto', server_location) | selectattr('default', 'equalto', true) | list | length == 0
+        - vpc_info.get('data', []) | selectattr('region', 'equalto', server_location) | selectattr('default', 'equalto', true) | list | length == 0
 
     - name: "Set variable: server_network"
       ansible.builtin.set_fact:
         server_network: "{{ digital_ocean_vpc_name | default('network-' + server_location) }}"
-      when: digital_ocean_vpc is changed
+      when:
+        - digital_ocean_vpc is defined
+        - digital_ocean_vpc is changed
 
     - name: "DigitalOcean: Gather information about VPC"
       community.digitalocean.digital_ocean_vpc_info:
         oauth_token: "{{ lookup('ansible.builtin.env', 'DO_API_TOKEN') }}"
       register: vpc_info
-      when: digital_ocean_vpc is changed
+      when:
+        - digital_ocean_vpc is defined
+        - digital_ocean_vpc is changed
 
     # if server_network is specified
     - name: "Fail if no VPC found in the specified region"
       ansible.builtin.fail:
         msg: "No VPC found with name '{{ server_network }}' in region '{{ server_location }}'"
       when:
         - server_network | length > 0
-        - vpc_info.data | selectattr('region', 'equalto', server_location) | selectattr('name', 'equalto', server_network) | list | length == 0
+        - (vpc_info.get('data', [])
+            | selectattr('region', 'equalto', server_location)
+            | selectattr('name', 'equalto', server_network)
+            | list
+            | length
+          ) == 0
 
     - name: Extract ip_range from VPC "{{ server_network | default('') }}"
       ansible.builtin.set_fact:
         vpc_ip_range: >-
           {{
-            vpc_info.data
+            vpc_info.get('data', [])
             | selectattr('region', 'equalto', server_location)
             | selectattr('name', 'equalto', server_network)
             | map(attribute='ip_range')
@@ -200,7 +209,7 @@
       ansible.builtin.set_fact:
         vpc_id: >-
           {{
-            vpc_info.data
+            vpc_info.get('data', [])
             | selectattr('region', 'equalto', server_location)
             | selectattr('name', 'equalto', server_network)
             | map(attribute='id')