Skip to content

Commit 89e5e71

Browse files
Bugfix: check_mode for netbox_prefix (#232)
1 parent e31b9bf commit 89e5e71

File tree

2 files changed

+23
-0
lines changed

2 files changed

+23
-0
lines changed

plugins/module_utils/netbox_ipam.py

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -102,6 +102,11 @@ def _get_new_available_prefix(self, data, endpoint_name):
102102
self.result["changed"] = False
103103
self.result["msg"] = "Parent prefix does not exist - %s" % (data["parent"])
104104
elif self.nb_object.available_prefixes.list():
105+
if self.check_mode:
106+
self.result["changed"] = True
107+
self.result["msg"] = "New prefix created within %s" % (data["parent"])
108+
self.module.exit_json(**self.result)
109+
105110
self.nb_object, diff = self._create_netbox_object(
106111
self.nb_object.available_prefixes, data
107112
)

tests/integration/targets/regression/tasks/main.yml

Lines changed: 18 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -26,3 +26,21 @@
2626
that:
2727
- test_one is failed
2828
- test_one['msg'] == "Failed to establish connection to Netbox API"
29+
30+
- name: "2 - Check to see if netbox_prefix with parent defined will pass via check-mode"
31+
netbox.netbox.netbox_prefix:
32+
netbox_url: "http://localhost:32768"
33+
netbox_token: "0123456789abcdef0123456789abcdef01234567"
34+
data:
35+
parent: "10.10.0.0/16"
36+
prefix_length: 24
37+
first_available: yes
38+
state: present
39+
register: test_two
40+
check_mode: yes
41+
42+
- name: "2 - ASSERT"
43+
assert:
44+
that:
45+
- test_two is changed
46+
- test_two['msg'] == "New prefix created within 10.10.0.0/16"

0 commit comments

Comments
 (0)