fix: return sorted alias_ips in server_network module (#458)

##### SUMMARY

- test: use testing framework for server_network integration tests
- fix: return sorted alias_ips in server_network module

##### ISSUE TYPE

- Bugfix Pull Request


##### COMPONENT NAME

server_network

.ansible-lint

@@ -15,7 +15,6 @@ exclude_paths:
   - tests/integration/targets/primary_ip
   - tests/integration/targets/route
   - tests/integration/targets/server
-  - tests/integration/targets/server_network
   - tests/integration/targets/ssh_key
   - tests/integration/targets/volume
 

changelogs/fragments/sort-alias-ips-in-server-network.yml

@@ -0,0 +1,2 @@
+bugfixes:
+  - server_network - The returned `alias_ips` list is now sorted.

plugins/modules/server_network.py

@@ -131,7 +131,7 @@ class AnsibleHCloudServerNetwork(AnsibleHCloud):
             "network": to_native(self.hcloud_network.name),
             "server": to_native(self.hcloud_server.name),
             "ip": to_native(self.hcloud_server_network.ip),
-            "alias_ips": self.hcloud_server_network.alias_ips,
+            "alias_ips": [to_native(ip) for ip in sorted(self.hcloud_server_network.alias_ips)],
         }
 
     def _get_server_and_network(self):

tests/integration/targets/server_network/aliases

@@ -1,3 +1,2 @@
 cloud/hcloud
 azp/group2
-disabled

tests/integration/targets/server_network/tasks/cleanup.yml

@@ -0,0 +1,10 @@
+---
+- name: Cleanup test_server
+  hetzner.hcloud.server:
+    name: "{{ hcloud_server_name }}"
+    state: absent
+
+- name: Cleanup test_network
+  hetzner.hcloud.network:
+    name: "{{ hcloud_network_name }}"
+    state: absent

tests/integration/targets/server_network/tasks/prepare.yml

@@ -0,0 +1,24 @@
+---
+- name: Create test_network
+  hetzner.hcloud.network:
+    name: "{{ hcloud_network_name }}"
+    ip_range: 10.0.0.0/16
+    labels:
+      key: value
+  register: test_network
+
+- name: Create test_subnetwork
+  hetzner.hcloud.subnetwork:
+    network: "{{ hcloud_network_name }}"
+    type: server
+    network_zone: eu-central
+    ip_range: 10.0.1.0/24
+  register: test_subnetwork
+
+- name: Create test_server
+  hetzner.hcloud.server:
+    name: "{{ hcloud_server_name }}"
+    server_type: cx11
+    image: ubuntu-22.04
+    state: stopped
+  register: test_server

tests/integration/targets/server_network/tasks/test.yml

@@ -1,222 +1,152 @@
 # Copyright: (c) 2019, Hetzner Cloud GmbH <info@hetzner-cloud.de>
 # GNU General Public License v3.0+ (see COPYING or https://www.gnu.org/licenses/gpl-3.0.txt)
 ---
-- name: setup network
+- name: Test missing required parameters
-  hetzner.hcloud.network:
-    name: "{{ hcloud_network_name }}"
-    ip_range: "10.0.0.0/8"
-    state: present
-  register: network
-- name: verify setup network
-  assert:
-    that:
-      - network is success
-
-- name: setup subnetwork
-  hetzner.hcloud.subnetwork:
-    network: "{{ hcloud_network_name }}"
-    ip_range: "10.0.0.0/16"
-    type: "server"
-    network_zone: "eu-central"
-    state: present
-  register: subnetwork
-- name: verify subnetwork
-  assert:
-    that:
-      - subnetwork is success
-
-- name: setup server
-  hetzner.hcloud.server:
-    name: "{{hcloud_server_name}}"
-    server_type: cx11
-    image: ubuntu-22.04
-    state: started
-    location: "fsn1"
-  register: server
-- name: verify setup server
-  assert:
-    that:
-      - server is success
-
-- name: test missing required parameters on create server network
   hetzner.hcloud.server_network:
     state: present
-  register: result
   ignore_errors: true
-- name: verify fail test missing required parameters on create server network
+  register: result
-  assert:
+- name: Verify missing required parameters
+  ansible.builtin.assert:
     that:
       - result is failed
       - 'result.msg == "missing required arguments: network, server"'
 
-- name: test create server network with checkmode
+- name: Test create with checkmode
   hetzner.hcloud.server_network:
     network: "{{ hcloud_network_name }}"
-    server: "{{hcloud_server_name}}"
+    server: "{{ hcloud_server_name }}"
     state: present
-  register: result
   check_mode: true
-- name: verify test create server network with checkmode
+  register: result
-  assert:
+- name: Verify create with checkmode
+  ansible.builtin.assert:
     that:
       - result is changed
 
-- name: test create server network
+- name: Test create
   hetzner.hcloud.server_network:
     network: "{{ hcloud_network_name }}"
-    server: "{{hcloud_server_name}}"
+    server: "{{ hcloud_server_name }}"
     state: present
-  register: serverNetwork
+  register: result
-- name: verify create server network
+- name: Verify create
-  assert:
+  ansible.builtin.assert:
     that:
-      - serverNetwork is changed
+      - result is changed
-      - serverNetwork.hcloud_server_network.network == hcloud_network_name
+      - result.hcloud_server_network.network == hcloud_network_name
-      - serverNetwork.hcloud_server_network.server == hcloud_server_name
+      - result.hcloud_server_network.server == hcloud_server_name
 
-- name: test create server network idempotency
+- name: Test create idempotency
   hetzner.hcloud.server_network:
     network: "{{ hcloud_network_name }}"
-    server: "{{hcloud_server_name}}"
+    server: "{{ hcloud_server_name }}"
     state: present
-  register: serverNetwork
+  register: result
-- name: verify create server network idempotency
+- name: Verify create idempotency
-  assert:
+  ansible.builtin.assert:
     that:
-      - serverNetwork is not changed
+      - result is not changed
 
-- name: test absent server network
+- name: Test delete
   hetzner.hcloud.server_network:
     network: "{{ hcloud_network_name }}"
-    server: "{{hcloud_server_name}}"
+    server: "{{ hcloud_server_name }}"
     state: absent
   register: result
-- name: verify test absent server network
+- name: Verify delete
-  assert:
+  ansible.builtin.assert:
     that:
       - result is changed
 
-- name: test create server network with specified ip
+- name: Test create with ip
   hetzner.hcloud.server_network:
     network: "{{ hcloud_network_name }}"
-    server: "{{hcloud_server_name}}"
+    server: "{{ hcloud_server_name }}"
-    ip: "10.0.0.2"
+    ip: "10.0.1.2"
     state: present
-  register: serverNetwork
+  register: result
-- name: verify create server network with specified ip
+- name: Verify create with ip
-  assert:
+  ansible.builtin.assert:
     that:
-      - serverNetwork is changed
+      - result is changed
-      - serverNetwork.hcloud_server_network.network == hcloud_network_name
+      - result.hcloud_server_network.network == hcloud_network_name
-      - serverNetwork.hcloud_server_network.server == hcloud_server_name
+      - result.hcloud_server_network.server == hcloud_server_name
-      - serverNetwork.hcloud_server_network.ip == "10.0.0.2"
+      - result.hcloud_server_network.ip == "10.0.1.2"
 
-- name: cleanup create server network with specified ip
+- name: Test delete with ip
   hetzner.hcloud.server_network:
     network: "{{ hcloud_network_name }}"
-    server: "{{hcloud_server_name}}"
+    server: "{{ hcloud_server_name }}"
     state: absent
   register: result
-- name: verify cleanup create server network with specified ip
+- name: Verify delete with ip
-  assert:
+  ansible.builtin.assert:
     that:
       - result is changed
 
-- name: test create server network with alias ips
+- name: Test create with alias ips
   hetzner.hcloud.server_network:
     network: "{{ hcloud_network_name }}"
-    server: "{{hcloud_server_name}}"
+    server: "{{ hcloud_server_name }}"
-    ip: "10.0.0.2"
+    ip: "10.0.1.2"
     alias_ips:
-      - "10.0.1.2"
+      - "10.0.1.10"
-      - "10.0.2.3"
+      - "10.0.1.11"
     state: present
-  register: serverNetwork
-- name: verify create server network with alias ips
-  assert:
-    that:
-      - serverNetwork is changed
-      - serverNetwork.hcloud_server_network.network == hcloud_network_name
-      - serverNetwork.hcloud_server_network.server == hcloud_server_name
-      - serverNetwork.hcloud_server_network.ip == "10.0.0.2"
-      - 'serverNetwork.hcloud_server_network.alias_ips[0] == "10.0.2.3"'
-      - 'serverNetwork.hcloud_server_network.alias_ips[1] == "10.0.1.2"'
-
-- name: test update server network with alias ips
-  hetzner.hcloud.server_network:
-    network: "{{ hcloud_network_name }}"
-    server: "{{hcloud_server_name}}"
-    ip: "10.0.0.2"
-    alias_ips:
-      - "10.0.2.3"
-      - "10.0.3.1"
-    state: present
-  register: serverNetwork
-- name: verify create server network with alias ips
-  assert:
-    that:
-      - serverNetwork is changed
-      - serverNetwork.hcloud_server_network.network == hcloud_network_name
-      - serverNetwork.hcloud_server_network.server == hcloud_server_name
-      - serverNetwork.hcloud_server_network.ip == "10.0.0.2"
-      - 'serverNetwork.hcloud_server_network.alias_ips[0] == "10.0.2.3"'
-      - 'serverNetwork.hcloud_server_network.alias_ips[1] == "10.0.3.1"'
-
-- name: test update server network with alias ips idempotency
-  hetzner.hcloud.server_network:
-    network: "{{ hcloud_network_name }}"
-    server: "{{hcloud_server_name}}"
-    ip: "10.0.0.2"
-    alias_ips:
-      - "10.0.2.3"
-      - "10.0.3.1"
-    state: present
-  register: serverNetwork
-- name: verify create server network with alias ips idempotency
-  assert:
-    that:
-      - serverNetwork is not changed
-
-- name: cleanup create server network with alias ips
-  hetzner.hcloud.server_network:
-    network: "{{ hcloud_network_name }}"
-    server: "{{hcloud_server_name}}"
-    state: absent
   register: result
-- name: verify cleanup create server network with alias ips
+- name: Verify create with alias ips
-  assert:
+  ansible.builtin.assert:
     that:
       - result is changed
+      - result.hcloud_server_network.network == hcloud_network_name
+      - result.hcloud_server_network.server == hcloud_server_name
+      - result.hcloud_server_network.ip == "10.0.1.2"
+      - result.hcloud_server_network.alias_ips[0] == "10.0.1.10"
+      - result.hcloud_server_network.alias_ips[1] == "10.0.1.11"
 
-- name: cleanup server
+- name: Test update with alias ips
-  hetzner.hcloud.server:
+  hetzner.hcloud.server_network:
-    name: "{{ hcloud_server_name }}"
-    state: absent
-  register: result
-- name: verify cleanup server
-  assert:
-    that:
-      - result is success
-
-- name: cleanup subnetwork
-  hetzner.hcloud.subnetwork:
     network: "{{ hcloud_network_name }}"
-    ip_range: "10.0.0.0/16"
+    server: "{{ hcloud_server_name }}"
-    type: "server"
+    ip: "10.0.1.2"
-    network_zone: "eu-central"
+    alias_ips:
-    state: absent
+      - "10.0.1.10"
+      - "10.0.1.20"
+    state: present
   register: result
-- name: verify cleanup subnetwork
+- name: Verify update with alias ips
-  assert:
+  ansible.builtin.assert:
     that:
       - result is changed
+      - result.hcloud_server_network.network == hcloud_network_name
+      - result.hcloud_server_network.server == hcloud_server_name
+      - result.hcloud_server_network.ip == "10.0.1.2"
+      - result.hcloud_server_network.alias_ips[0] == "10.0.1.10"
+      - result.hcloud_server_network.alias_ips[1] == "10.0.1.20"
 
-- name: cleanup
+- name: Test update with alias ips idempotency
-  hetzner.hcloud.network:
+  hetzner.hcloud.server_network:
-    name: "{{hcloud_network_name}}"
+    network: "{{ hcloud_network_name }}"
+    server: "{{ hcloud_server_name }}"
+    ip: "10.0.1.2"
+    alias_ips:
+      - "10.0.1.10"
+      - "10.0.1.20"
+    state: present
+  register: result
+- name: Verify update with alias ips idempotency
+  ansible.builtin.assert:
+    that:
+      - result is not changed
+
+- name: Test delete with alias ips
+  hetzner.hcloud.server_network:
+    network: "{{ hcloud_network_name }}"
+    server: "{{ hcloud_server_name }}"
     state: absent
   register: result
-- name: verify cleanup
+- name: Verify delete with alias ips
-  assert:
+  ansible.builtin.assert:
     that:
-      - result is success
+      - result is changed