add base_url to api endpoint to allow non-root context nextbox endpoints

fix linting

Author: lanefu

plugins/inventory/nb_inventory.py

@@ -406,6 +406,8 @@
 from ansible.module_utils.urls import open_url
 from ansible.module_utils.six.moves.urllib import error as urllib_error
 from ansible.module_utils.six.moves.urllib.parse import urlencode
+from ansible.module_utils.six.moves.urllib.parse import urlparse
+
 from ansible.module_utils.six import raise_from
 
 try:
@@ -1348,9 +1350,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 +1488,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"]
@@ -1629,26 +1631,32 @@ def fetch_api_docs(self):
         self.api_version = version.parse(netbox_api_version)
 
         if self.api_version >= version.parse("3.5.0"):
+            parsed_endpoint_url = urlparse(self.api_endpoint)
+            base_path = parsed_endpoint_url.path
             self.allowed_device_query_parameters = [
                 p["name"]
-                for p in openapi["paths"]["/api/dcim/devices/"]["get"]["parameters"]
+                for p in openapi["paths"][base_path + "/api/dcim/devices/"]["get"][
+                    "parameters"
+                ]
             ]
             self.allowed_vm_query_parameters = [
                 p["name"]
-                for p in openapi["paths"]["/api/virtualization/virtual-machines/"][
-                    "get"
-                ]["parameters"]
+                for p in openapi["paths"][
+                    base_path + "/api/virtualization/virtual-machines/"
+                ]["get"]["parameters"]
             ]
         else:
             self.allowed_device_query_parameters = [
                 p["name"]
-                for p in openapi["paths"]["/dcim/devices/"]["get"]["parameters"]
+                for p in openapi["paths"][base_path + "/dcim/devices/"]["get"][
+                    "parameters"
+                ]
             ]
             self.allowed_vm_query_parameters = [
                 p["name"]
-                for p in openapi["paths"]["/virtualization/virtual-machines/"]["get"][
-                    "parameters"
-                ]
+                for p in openapi["paths"][
+                    base_path + "/virtualization/virtual-machines/"
+                ]["get"]["parameters"]
             ]
 
     def validate_query_parameter(self, parameter, allowed_query_parameters):