netbox_site - Add site_group (#545)

Author: andybro19

plugins/module_utils/netbox_utils.py

@@ -342,6 +342,7 @@
     "route_target": set(["name"]),
     "services": set(["device", "virtual_machine", "name", "port", "protocol"]),
     "site": set(["slug"]),
+    "site_group": set(["slug"]),
     "tags": set(["slug"]),
     "tagged_vlans": set(["group", "name", "site", "vid", "vlan_group", "tenant"]),
     "tenant": set(["slug"]),
@@ -420,6 +421,7 @@
     "rack_role": "role",
     "rear_port_template": "rear_port",
     "rear_port_template_position": "rear_port_position",
+    "site_group": "group",
     "tenant_group": "group",
     "termination_a": "termination_a_id",
     "termination_b": "termination_b_id",

plugins/modules/netbox_site.py

@@ -64,6 +64,11 @@
           - The region that the site should be associated with
         required: false
         type: raw
+      site_group:
+        description: 
+          - The site group the site will be associated with (NetBox 2.11+)
+        required: false
+        type: raw
       tenant:
         description:
           - The tenant the site will be assigned to
@@ -199,6 +204,7 @@
           name: Test - California
           status: Planned
           region: Test Region
+          site_group: Test Site Group
           tenant: Test Tenant
           facility: EquinoxCA7
           asn: 65001
@@ -252,6 +258,7 @@ def main():
                     name=dict(required=True, type="str"),
                     status=dict(required=False, type="raw"),
                     region=dict(required=False, type="raw"),
+                    site_group=dict(required=False, type="raw"),
                     tenant=dict(required=False, type="raw"),
                     facility=dict(required=False, type="str"),
                     asn=dict(required=False, type="int"),

tests/integration/targets/v2.11/tasks/netbox_site.yml

@@ -71,6 +71,7 @@
       name: Test - California
       status: Planned
       region: Test Region
+      site_group: Test Site Group
       tenant: Test Tenant
       facility: EquinoxCA7
       asn: 65001
@@ -98,6 +99,7 @@
       - test_four['msg'] == "site Test - California created"
       - test_four['site']['status'] == "planned"
       - test_four['site']['region'] == 1
+      - test_four['site']['group'] == 1
       - test_four['site']['tenant'] == 1
       - test_four['site']['facility'] == "EquinoxCA7"
       - test_four['site']['asn'] == 65001
@@ -121,6 +123,7 @@
       name: Test - California
       status: Planned
       region: Test Region
+      site_group: Test Site Group
       tenant: Test Tenant
       facility: EquinoxCA7
       asn: 65001
@@ -146,6 +149,7 @@
       - test_results['msg'] == "site Test - California already exists"
       - test_results['site']['status'] == "planned"
       - test_results['site']['region'] == 1
+      - test_results['site']['group'] == 1
       - test_results['site']['tenant'] == 1
       - test_results['site']['facility'] == "EquinoxCA7"
       - test_results['site']['asn'] == 65001

tests/integration/targets/v3.0/tasks/netbox_site.yml

@@ -71,6 +71,7 @@
       name: Test - California
       status: Planned
       region: Test Region
+      site_group: Test Site Group
       tenant: Test Tenant
       facility: EquinoxCA7
       asn: 65001
@@ -98,6 +99,7 @@
       - test_four['msg'] == "site Test - California created"
       - test_four['site']['status'] == "planned"
       - test_four['site']['region'] == 1
+      - test_four['site']['group'] == 1
       - test_four['site']['tenant'] == 1
       - test_four['site']['facility'] == "EquinoxCA7"
       - test_four['site']['asn'] == 65001
@@ -121,6 +123,7 @@
       name: Test - California
       status: Planned
       region: Test Region
+      site_group: Test Site Group
       tenant: Test Tenant
       facility: EquinoxCA7
       asn: 65001
@@ -146,6 +149,7 @@
       - test_results['msg'] == "site Test - California already exists"
       - test_results['site']['status'] == "planned"
       - test_results['site']['region'] == 1
+      - test_results['site']['group'] == 1
       - test_results['site']['tenant'] == 1
       - test_results['site']['facility'] == "EquinoxCA7"
       - test_results['site']['asn'] == 65001