fix netbox-community#1335

ihumster · Oct 14, 2024 · b663c7e · b663c7e
1 parent 432b8fe
commit b663c7e
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 10 deletions.
diff --git a/changelogs/fragments/1335-api-status-page-not-found.yml b/changelogs/fragments/1335-api-status-page-not-found.yml
@@ -0,0 +1,2 @@
+bugfixes:
+  - fix call /api/status/ instead /api/status in nb_inventory plugin. (https://github.com/netbox-community/ansible_modules/issues/1335).
diff --git a/plugins/inventory/nb_inventory.py b/plugins/inventory/nb_inventory.py
@@ -1348,9 +1348,9 @@ def refresh_services(self):
             service_id = service["id"]
 
             if service.get("device"):
-                self.device_services_lookup[service["device"]["id"]][
-                    service_id
-                ] = service
+                self.device_services_lookup[service["device"]["id"]][service_id] = (
+                    service
+                )
 
             if service.get("virtual_machine"):
                 self.vm_services_lookup[service["virtual_machine"]["id"]][
@@ -1486,14 +1486,14 @@ def refresh_ipaddresses(self):
 
                 if ipaddress["assigned_object_type"] == "virtualization.vminterface":
                     self.vm_ipaddresses_lookup[ip_id] = ipaddress_copy
-                    self.vm_ipaddresses_intf_lookup[interface_id][
-                        ip_id
-                    ] = ipaddress_copy
+                    self.vm_ipaddresses_intf_lookup[interface_id][ip_id] = (
+                        ipaddress_copy
+                    )
                 else:
                     self.device_ipaddresses_lookup[ip_id] = ipaddress_copy
-                    self.device_ipaddresses_intf_lookup[interface_id][
-                        ip_id
-                    ] = ipaddress_copy  # Remove "assigned_object_X" attributes, as that's redundant when ipaddress is added to an interface
+                    self.device_ipaddresses_intf_lookup[interface_id][ip_id] = (
+                        ipaddress_copy  # Remove "assigned_object_X" attributes, as that's redundant when ipaddress is added to an interface
+                    )
 
                 del ipaddress_copy["assigned_object_id"]
                 del ipaddress_copy["assigned_object_type"]
@@ -1607,7 +1607,7 @@ def fetch_api_docs(self):
             cached_api_version = None
             cache = None
 
-        status = self._fetch_information(self.api_endpoint + "/api/status")
+        status = self._fetch_information(self.api_endpoint + "/api/status/")
         netbox_api_version = ".".join(status["netbox-version"].split(".")[:2])
 
         if version.parse(netbox_api_version) >= version.parse("3.5.0"):