nautobot 2.4.17__py3-none-any.whl → 2.4.19__py3-none-any.whl

nautobot/dcim/tests/test_views.py

@@ -7,7 +7,9 @@ from django.contrib.auth import get_user_model
 from django.contrib.contenttypes.models import ContentType
 from django.db.models import Q
 from django.test import override_settings
+from django.test.client import RequestFactory
 from django.urls import reverse
+from django.utils.html import strip_spaces_between_tags
 from netaddr import EUI
 import yaml
 
@@ -107,6 +109,7 @@ from nautobot.dcim.models import (
 from nautobot.dcim.views import (
     ConsoleConnectionsListView,
     InterfaceConnectionsListView,
+    LocationUIViewSet,
     PowerConnectionsListView,
 )
 from nautobot.extras.choices import CustomFieldTypeChoices, RelationshipTypeChoices
@@ -208,6 +211,8 @@ class LocationTypeTestCase(ViewTestCases.OrganizationalObjectViewTestCase, ViewT
 
 class LocationTestCase(ViewTestCases.PrimaryObjectViewTestCase):
     model = Location
+    # One query for the natural slug, one for `LocationViewSet.get_extra_context`
+    allowed_number_of_tree_queries_per_view_type = {"retrieve": 2}
 
     @classmethod
     def setUpTestData(cls):
@@ -230,7 +235,14 @@ class LocationTestCase(ViewTestCases.PrimaryObjectViewTestCase):
             status=status,
             description="Hi!",
         )
-        for loc in [loc1, loc2, loc3, loc4]:
+        loc5 = Location.objects.create(
+            name="Leaf 2",
+            location_type=lt3,
+            parent=loc3,
+            status=status,
+            description="Hi!",
+        )
+        for loc in [loc1, loc2, loc3, loc4, loc5]:
             loc.validated_save()
 
         cls.form_data = {
@@ -429,10 +441,28 @@ class LocationTestCase(ViewTestCases.PrimaryObjectViewTestCase):
         self.assertEqual(location.contact_phone, "")
         self.assertEqual(location.contact_email, "")
 
+    def test_get_extra_context(self):
+        view_set = LocationUIViewSet()
+        child_1, child_2 = Location.objects.filter(name__startswith="Leaf ")
+        parent_location = child_1.parent
+        child_1.location_type.content_types.add(ContentType.objects.get_for_model(Prefix))
+        status = Status.objects.get_for_model(Prefix).first()
+        prefix_1 = Prefix.objects.create(network="192.0.2.0", prefix_length=25, status=status)
+        prefix_2 = Prefix.objects.create(network="192.0.2.128", prefix_length=25, status=status)
+        prefix_1.locations.set([child_1, child_2])
+        prefix_2.locations.set([child_1, child_2])
+        request = RequestFactory().get(parent_location.get_absolute_url())
+        request.user = self.user
+        self.add_permissions("dcim.view_location")
+        self.add_permissions("ipam.view_prefix")
+        context = view_set.get_extra_context(request=request, instance=parent_location)
+        self.assertEqual(context["stats"]["prefix_count"], 2)
+
 
 class RackGroupTestCase(ViewTestCases.OrganizationalObjectViewTestCase, ViewTestCases.BulkEditObjectsViewTestCase):
     model = RackGroup
     sort_on_field = "name"
+    allowed_number_of_tree_queries_per_view_type = {"retrieve": 1}
 
     @classmethod
     def setUpTestData(cls):
@@ -460,6 +490,7 @@ class RackGroupTestCase(ViewTestCases.OrganizationalObjectViewTestCase, ViewTest
 
 class RackReservationTestCase(ViewTestCases.PrimaryObjectViewTestCase):
     model = RackReservation
+    allowed_number_of_tree_queries_per_view_type = {"retrieve": 1}
 
     @classmethod
     def setUpTestData(cls):
@@ -495,6 +526,7 @@ class RackReservationTestCase(ViewTestCases.PrimaryObjectViewTestCase):
 
 class RackTestCase(ViewTestCases.PrimaryObjectViewTestCase):
     model = Rack
+    allowed_number_of_tree_queries_per_view_type = {"retrieve": 1}
 
     @classmethod
     def setUpTestData(cls):
@@ -2081,6 +2113,7 @@ class PlatformTestCase(ViewTestCases.OrganizationalObjectViewTestCase, ViewTestC
 
 class DeviceTestCase(ViewTestCases.PrimaryObjectViewTestCase):
     model = Device
+    allowed_number_of_tree_queries_per_view_type = {"retrieve": 1}
 
     @classmethod
     def setUpTestData(cls):
@@ -2278,6 +2311,7 @@ class DeviceTestCase(ViewTestCases.PrimaryObjectViewTestCase):
     def test_vdc_panel_includes_add_vdc_btn(self):
         """Assert Add Virtual device Contexts button is in Device detail view: Issue from #6348"""
         device = Device.objects.first()
+        self.add_permissions("dcim.add_virtualdevicecontext")
         url = reverse("dcim:device", kwargs={"pk": device.pk})
         response = self.client.get(url)
         response_body = extract_page_body(response.content.decode(response.charset))
@@ -2292,6 +2326,63 @@ class DeviceTestCase(ViewTestCases.PrimaryObjectViewTestCase):
         self.assertInHTML(expected_add_vdc_button_html, response_body)
         self.assertIn("Add virtual device context", response_body)
 
+    @override_settings(EXEMPT_VIEW_PERMISSIONS=["*"])
+    def test_device_detail_with_parent_bay(self):
+        parent_device = Device.objects.first()
+        parent_device.subdevice_role = "parent"
+        parent_device.validated_save()
+        child_device = Device.objects.last()
+        child_device.location = parent_device.location
+        child_device.subdevice_role = "child"
+        child_device.validated_save()
+        device_bay = DeviceBay.objects.create(
+            name="test device bay", device=parent_device, installed_device=child_device
+        )
+
+        url = reverse("dcim:device", kwargs={"pk": child_device.pk})
+        response = self.client.get(url)
+        self.assertHttpStatus(response, 200)
+        response_body = extract_page_body(response.content.decode(response.charset))
+        self.assertInHTML(parent_device.display, response_body)
+        self.assertInHTML(str(device_bay), response_body)
+
+    @override_settings(EXEMPT_VIEW_PERMISSIONS=["*"])
+    def test_device_detail_with_rack(self):
+        device = Device.objects.filter(device_type__u_height__gt=0).first()
+        self.assertIsNotNone(device)
+        device.rack = Rack.objects.filter(u_height__gt=device.device_type.u_height).first()
+        self.assertIsNotNone(device.rack)
+        device.position = 1
+        device.face = "front"
+        device.validated_save()
+        url = reverse("dcim:device", kwargs={"pk": device.pk})
+        response = self.client.get(url)
+        self.assertHttpStatus(response, 200)
+        response_body = extract_page_body(response.content.decode(response.charset))
+        self.assertInHTML(f"U{device.position} / {device.get_face_display()}", response_body)
+
+    @override_settings(EXEMPT_VIEW_PERMISSIONS=["*"])
+    def test_device_modulebays(self):
+        device = Device.objects.filter(module_bays__isnull=True).first()
+        module = Module.objects.filter(parent_module_bay__isnull=True).first()
+
+        module_bays = (
+            ModuleBay.objects.create(parent_device=device, name="Test View Module Bay 1"),
+            ModuleBay.objects.create(parent_device=device, name="Test View Module Bay 2"),
+            ModuleBay.objects.create(parent_device=device, name="Test View Module Bay 3"),
+        )
+
+        module.location = None
+        module.parent_module_bay = module_bays[0]
+        module.validated_save()
+
+        url = reverse("dcim:device_modulebays", kwargs={"pk": device.pk})
+        response = self.client.get(url)
+        self.assertHttpStatus(response, 200)
+        # Custom badge - module count / module-bay count
+        response_body = extract_page_body(response.content.decode(response.charset))
+        self.assertInHTML("1/3", response_body)
+
     @override_settings(EXEMPT_VIEW_PERMISSIONS=["*"])
     def test_device_consoleports(self):
         device = Device.objects.first()
@@ -2491,6 +2582,32 @@ class DeviceTestCase(ViewTestCases.PrimaryObjectViewTestCase):
         url = reverse("dcim:device_inventory", kwargs={"pk": device.pk})
         self.assertHttpStatus(self.client.get(url), 200)
 
+    @override_settings(EXEMPT_VIEW_PERMISSIONS=["*"])
+    def test_device_wireless(self):
+        self.add_permissions("dcim.view_controllermanageddevicegroup")
+        device = Device.objects.first()
+        device.controller_managed_device_group = ControllerManagedDeviceGroup.objects.first()
+        device.validated_save()
+        device.controller_managed_device_group.capabilities = ["wireless"]
+        device.controller_managed_device_group.validated_save()
+
+        url = reverse("dcim:device_wireless", kwargs={"pk": device.pk})
+        response = self.client.get(url)
+        self.assertHttpStatus(response, 200)
+
+    @override_settings(EXEMPT_VIEW_PERMISSIONS=["*"])
+    def test_device_napalm_tabs(self):
+        self.add_permissions("dcim.napalm_read_device")
+        device = Device.objects.first()
+        # TODO set NAPALM args, mock NAPALM APIs?
+
+        url = reverse("dcim:device_status", kwargs={"pk": device.pk})
+        self.assertHttpStatus(self.client.get(url), 200)
+        url = reverse("dcim:device_lldp_neighbors", kwargs={"pk": device.pk})
+        self.assertHttpStatus(self.client.get(url), 200)
+        url = reverse("dcim:device_config", kwargs={"pk": device.pk})
+        self.assertHttpStatus(self.client.get(url), 200)
+
     @override_settings(EXEMPT_VIEW_PERMISSIONS=["*"])
     def test_device_primary_ips(self):
         """Test assigning a primary IP to a device."""
@@ -2543,6 +2660,13 @@ class DeviceTestCase(ViewTestCases.PrimaryObjectViewTestCase):
             schema.pk,
         )
 
+        url = reverse("dcim:device_configcontext", kwargs={"pk": self._get_queryset().get(name="Device X").pk})
+        response = self.client.get(url)
+        self.assertHttpStatus(response, 200)
+        response_body = extract_page_body(response.content.decode(response.charset))
+        self.assertIn("foo", response_body)
+        self.assertIn("bar", response_body)
+
     @override_settings(EXEMPT_VIEW_PERMISSIONS=["*"])
     def test_local_config_context_schema_validation_fails(self):
         """
@@ -2773,6 +2897,7 @@ class ModuleTestCase(ViewTestCases.PrimaryObjectViewTestCase):
 
 class ConsolePortTestCase(ViewTestCases.DeviceComponentViewTestCase):
     model = ConsolePort
+    allowed_number_of_tree_queries_per_view_type = {"retrieve": 1}
 
     @classmethod
     def setUpTestData(cls):
@@ -2822,6 +2947,7 @@ class ConsolePortTestCase(ViewTestCases.DeviceComponentViewTestCase):
 
 class ConsoleServerPortTestCase(ViewTestCases.DeviceComponentViewTestCase):
     model = ConsoleServerPort
+    allowed_number_of_tree_queries_per_view_type = {"retrieve": 1}
 
     @classmethod
     def setUpTestData(cls):
@@ -2870,6 +2996,7 @@ class ConsoleServerPortTestCase(ViewTestCases.DeviceComponentViewTestCase):
 
 class PowerPortTestCase(ViewTestCases.DeviceComponentViewTestCase):
     model = PowerPort
+    allowed_number_of_tree_queries_per_view_type = {"retrieve": 1}
 
     @classmethod
     def setUpTestData(cls):
@@ -2926,6 +3053,7 @@ class PowerPortTestCase(ViewTestCases.DeviceComponentViewTestCase):
 
 class PowerOutletTestCase(ViewTestCases.DeviceComponentViewTestCase):
     model = PowerOutlet
+    allowed_number_of_tree_queries_per_view_type = {"retrieve": 1}
 
     @classmethod
     def setUpTestData(cls):
@@ -2995,6 +3123,7 @@ class PowerOutletTestCase(ViewTestCases.DeviceComponentViewTestCase):
 
 class InterfaceTestCase(ViewTestCases.DeviceComponentViewTestCase):
     model = Interface
+    allowed_number_of_tree_queries_per_view_type = {"retrieve": 1}
 
     @classmethod
     def setUpTestData(cls):
@@ -3195,6 +3324,7 @@ class InterfaceTestCase(ViewTestCases.DeviceComponentViewTestCase):
 
 class FrontPortTestCase(ViewTestCases.DeviceComponentViewTestCase):
     model = FrontPort
+    allowed_number_of_tree_queries_per_view_type = {"retrieve": 1}
 
     @classmethod
     def setUpTestData(cls):
@@ -3310,6 +3440,7 @@ class FrontPortTestCase(ViewTestCases.DeviceComponentViewTestCase):
 
 class RearPortTestCase(ViewTestCases.DeviceComponentViewTestCase):
     model = RearPort
+    allowed_number_of_tree_queries_per_view_type = {"retrieve": 1}
 
     @classmethod
     def setUpTestData(cls):
@@ -3376,6 +3507,7 @@ class RearPortTestCase(ViewTestCases.DeviceComponentViewTestCase):
 
 class DeviceBayTestCase(ViewTestCases.DeviceComponentViewTestCase):
     model = DeviceBay
+    allowed_number_of_tree_queries_per_view_type = {"retrieve": 1}
 
     @classmethod
     def setUpTestData(cls):
@@ -4185,7 +4317,11 @@ class VirtualChassisTestCase(ViewTestCases.PrimaryObjectViewTestCase):
         Interface.objects.create(device=self.devices[0], name="eth0", status=interface_status)
         Interface.objects.create(device=self.devices[0], name="eth1", status=interface_status)
         response = self.client.get(reverse("dcim:device_interfaces", kwargs={"pk": self.devices[0].pk}))
-        self.assertBodyContains(response, "<th>Device</th>", html=True)
+        self.assertBodyContains(
+            response,
+            '<th class="orderable"><a data-column-name="device" href="?sort=device">Device</a></th>',
+            html=True,
+        )
 
     def test_device_column_not_visible(self):
         """
@@ -4198,13 +4334,60 @@ class VirtualChassisTestCase(ViewTestCases.PrimaryObjectViewTestCase):
         Interface.objects.create(device=self.devices[1], name="eth2", status=interface_status)
         Interface.objects.create(device=self.devices[1], name="eth3", status=interface_status)
         response = self.client.get(reverse("dcim:device_interfaces", kwargs={"pk": self.devices[1].pk}))
-        self.assertNotIn("<th >Device</th>", extract_page_body(response.content.decode(response.charset)))
+        self.assertNotIn(
+            '<th class="orderable"><a data-column-name="device" href="?sort=device">Device</a></th>',
+            strip_spaces_between_tags(extract_page_body(response.content.decode(response.charset))),
+        )
         # Sanity check:
-        self.assertBodyContains(response, "<th>Name</th>", html=True)
+        self.assertBodyContains(
+            response, '<th class="orderable"><a data-column-name="name" href="?sort=name">Name</a></th>', html=True
+        )
+
+    def test_set_master_after_adding_member(self):
+        """Ensure master can be set for a member that was added via the Add Member flow."""
+        self.add_permissions(
+            "dcim.view_device",
+            "dcim.view_virtualchassis",
+            "dcim.change_virtualchassis",
+            "dcim.change_device",
+        )
+
+        # Create VC
+        vc = VirtualChassis.objects.create(name="VC-test", domain="domain-test")
+
+        # Simulate adding a member via the separate "add-member" flow by creating the device with virtual_chassis
+        member = Device.objects.create(
+            device_type=self.devices[0].device_type,
+            role=self.devices[0].role,
+            status=self.devices[0].status,
+            name="separately-added-device",
+            location=self.devices[0].location,
+            virtual_chassis=vc,
+            vc_position=1,
+        )
+
+        # Now edit the VC and set the master to the existing member
+        payload_data = {
+            "name": vc.name,
+            "domain": vc.domain,
+            "master": str(member.pk),
+            # no members formset rows are required because the member already exists
+            "form-TOTAL_FORMS": "0",
+            "form-INITIAL_FORMS": "0",
+            "form-MIN_NUM_FORMS": "0",
+            "form-MAX_NUM_FORMS": "1000",
+        }
+
+        url = reverse("dcim:virtualchassis_edit", kwargs={"pk": vc.pk})
+        self.client.post(url, data=payload_data, follow=True)
+
+        vc.refresh_from_db()
+        self.assertEqual(vc.master, member)
 
 
 class PowerPanelTestCase(ViewTestCases.PrimaryObjectViewTestCase):
     model = PowerPanel
+    allowed_number_of_tree_queries_per_view_type = {"retrieve": 1}
 
     @classmethod
     def setUpTestData(cls):
@@ -4233,6 +4416,7 @@ class PowerPanelTestCase(ViewTestCases.PrimaryObjectViewTestCase):
 
 class PowerFeedTestCase(ViewTestCases.PrimaryObjectViewTestCase):
     model = PowerFeed
+    allowed_number_of_tree_queries_per_view_type = {"retrieve": 1}
 
     @classmethod
     def setUpTestData(cls):
@@ -4672,6 +4856,7 @@ class ControllerManagedDeviceGroupTestCase(ViewTestCases.PrimaryObjectViewTestCa
 class VirtualDeviceContextTestCase(ViewTestCases.PrimaryObjectViewTestCase):
     model = VirtualDeviceContext
     filterset = VirtualDeviceContextFilterSet
+    allowed_number_of_tree_queries_per_view_type = {"retrieve": 1}
 
     @classmethod
     def setUpTestData(cls):

nautobot/dcim/ui.py

@@ -0,0 +1,29 @@
+from django.template import Context
+
+from nautobot.core.ui.breadcrumbs import (
+    AncestorsBreadcrumbs,
+    InstanceBreadcrumbItem,
+    InstanceParentBreadcrumbItem,
+    ModelBreadcrumbItem,
+)
+from nautobot.core.views.utils import get_obj_from_context
+from nautobot.dcim.models import Rack
+
+
+class RackBreadcrumbs(AncestorsBreadcrumbs):
+    def get_detail_items(self, context: Context):
+        rack: Rack = get_obj_from_context(context)
+        base_items = [
+            ModelBreadcrumbItem(),
+            InstanceParentBreadcrumbItem(
+                parent_key="location",
+            ),
+        ]
+        detail_item = InstanceBreadcrumbItem(
+            label=self.detail_item_label,
+        )
+        if rack.rack_group:
+            ancestors_items = self.get_ancestors_items(rack.rack_group)
+            return [*base_items, *ancestors_items, InstanceBreadcrumbItem(instance=rack.rack_group), detail_item]
+
+        return [*base_items, detail_item]

nautobot/dcim/urls.py

@@ -31,6 +31,7 @@ router.register("controllers", views.ControllerUIViewSet)
 router.register("device-families", views.DeviceFamilyUIViewSet)
 router.register("device-redundancy-groups", views.DeviceRedundancyGroupUIViewSet)
 router.register("device-types", views.DeviceTypeUIViewSet)
+router.register("devices", views.DeviceUIViewSet)
 router.register("interface-redundancy-groups", views.InterfaceRedundancyGroupUIViewSet)
 router.register("interface-redundancy-groups-associations", views.InterfaceRedundancyGroupAssociationUIViewSet)
 router.register("locations", views.LocationUIViewSet)
@@ -332,27 +333,6 @@ urlpatterns = [
         name="devicebaytemplate_delete",
     ),
     # Devices
-    path("devices/", views.DeviceListView.as_view(), name="device_list"),
-    path("devices/add/", views.DeviceEditView.as_view(), name="device_add"),
-    path("devices/import/", views.DeviceBulkImportView.as_view(), name="device_import"),  # 3.0 TODO: remove, unused
-    path("devices/edit/", views.DeviceBulkEditView.as_view(), name="device_bulk_edit"),
-    path(
-        "devices/delete/",
-        views.DeviceBulkDeleteView.as_view(),
-        name="device_bulk_delete",
-    ),
-    path("devices/<uuid:pk>/", views.DeviceView.as_view(), name="device"),
-    path("devices/<uuid:pk>/edit/", views.DeviceEditView.as_view(), name="device_edit"),
-    path(
-        "devices/<uuid:pk>/delete/",
-        views.DeviceDeleteView.as_view(),
-        name="device_delete",
-    ),
-    path(
-        "devices/<uuid:pk>/console-ports/",
-        views.DeviceConsolePortsView.as_view(),
-        name="device_consoleports",
-    ),
     path(
         "devices/<uuid:pk>/console-ports/add/",
         RedirectView.as_view(
@@ -360,11 +340,6 @@ urlpatterns = [
         ),
         name="device_consoleports_add",
     ),
-    path(
-        "devices/<uuid:pk>/console-server-ports/",
-        views.DeviceConsoleServerPortsView.as_view(),
-        name="device_consoleserverports",
-    ),
     path(
         "devices/<uuid:pk>/console-server-ports/add/",
         RedirectView.as_view(
@@ -372,21 +347,11 @@ urlpatterns = [
         ),
         name="device_consoleserverports_add",
     ),
-    path(
-        "devices/<uuid:pk>/power-ports/",
-        views.DevicePowerPortsView.as_view(),
-        name="device_powerports",
-    ),
     path(
         "devices/<uuid:pk>/power-ports/add/",
         RedirectView.as_view(url="/dcim/power-ports/add/?device=%(pk)s&return_url=/dcim/devices/%(pk)s/power-ports/"),
         name="device_powerports_add",
     ),
-    path(
-        "devices/<uuid:pk>/power-outlets/",
-        views.DevicePowerOutletsView.as_view(),
-        name="device_poweroutlets",
-    ),
     path(
         "devices/<uuid:pk>/power-outlets/add/",
         RedirectView.as_view(
@@ -394,51 +359,26 @@ urlpatterns = [
         ),
         name="device_poweroutlets_add",
     ),
-    path(
-        "devices/<uuid:pk>/interfaces/",
-        views.DeviceInterfacesView.as_view(),
-        name="device_interfaces",
-    ),
     path(
         "devices/<uuid:pk>/interfaces/add/",
         RedirectView.as_view(url="/dcim/interfaces/add/?device=%(pk)s&return_url=/dcim/devices/%(pk)s/interfaces/"),
         name="device_interfaces_add",
     ),
-    path(
-        "devices/<uuid:pk>/front-ports/",
-        views.DeviceFrontPortsView.as_view(),
-        name="device_frontports",
-    ),
     path(
         "devices/<uuid:pk>/front-ports/add/",
         RedirectView.as_view(url="/dcim/front-ports/add/?device=%(pk)s&return_url=/dcim/devices/%(pk)s/front-ports/"),
         name="device_frontports_add",
     ),
-    path(
-        "devices/<uuid:pk>/rear-ports/",
-        views.DeviceRearPortsView.as_view(),
-        name="device_rearports",
-    ),
     path(
         "devices/<uuid:pk>/rear-ports/add/",
         RedirectView.as_view(url="/dcim/rear-ports/add/?device=%(pk)s&return_url=/dcim/devices/%(pk)s/rear-ports/"),
         name="device_rearports_add",
     ),
-    path(
-        "devices/<uuid:pk>/device-bays/",
-        views.DeviceDeviceBaysView.as_view(),
-        name="device_devicebays",
-    ),
     path(
         "devices/<uuid:pk>/device-bays/add/",
         RedirectView.as_view(url="/dcim/device-bays/add/?device=%(pk)s&return_url=/dcim/devices/%(pk)s/device-bays/"),
         name="device_devicebays_add",
     ),
-    path(
-        "devices/<uuid:pk>/module-bays/",
-        views.DeviceModuleBaysView.as_view(),
-        name="device_modulebays",
-    ),
     path(
         "devices/<uuid:pk>/module-bays/add/",
         RedirectView.as_view(
@@ -446,49 +386,6 @@ urlpatterns = [
         ),
         name="device_modulebays_add",
     ),
-    path(
-        "devices/<uuid:pk>/inventory/",
-        views.DeviceInventoryView.as_view(),
-        name="device_inventory",
-    ),
-    path(
-        "devices/<uuid:pk>/config-context/",
-        views.DeviceConfigContextView.as_view(),
-        name="device_configcontext",
-    ),
-    path(
-        "devices/<uuid:pk>/changelog/",
-        views.DeviceChangeLogView.as_view(),
-        name="device_changelog",
-        kwargs={"model": Device},
-    ),
-    path(
-        "devices/<uuid:pk>/notes/",
-        ObjectNotesView.as_view(),
-        name="device_notes",
-        kwargs={"model": Device},
-    ),
-    path(
-        "devices/<uuid:pk>/dynamic-groups/",
-        views.DeviceDynamicGroupsView.as_view(),
-        name="device_dynamicgroups",
-        kwargs={"model": Device},
-    ),
-    path(
-        "devices/<uuid:pk>/status/",
-        views.DeviceStatusView.as_view(),
-        name="device_status",
-    ),
-    path(
-        "devices/<uuid:pk>/lldp-neighbors/",
-        views.DeviceLLDPNeighborsView.as_view(),
-        name="device_lldp_neighbors",
-    ),
-    path(
-        "devices/<uuid:pk>/config/",
-        views.DeviceConfigView.as_view(),
-        name="device_config",
-    ),
     path(
         "devices/<uuid:device>/services/assign/",
         ServiceEditView.as_view(),
@@ -500,11 +397,6 @@ urlpatterns = [
         name="device_add_image",
         kwargs={"model": Device},
     ),
-    path(
-        "devices/<uuid:pk>/wireless/",
-        views.DeviceWirelessView.as_view(),
-        name="device_wireless",
-    ),
     # Console ports
     path("console-ports/", views.ConsolePortListView.as_view(), name="consoleport_list"),
     path(

nautobot/dcim/utils.py

@@ -3,9 +3,11 @@ import uuid
 
 from django.contrib.contenttypes.models import ContentType
 from django.core.exceptions import ValidationError
+from django.utils.html import format_html, format_html_join
 from netutils.lib_mapper import NAME_TO_ALL_LIB_MAPPER, NAME_TO_LIB_MAPPER_REVERSE
 
 from nautobot.core.choices import ColorChoices
+from nautobot.core.templatetags.helpers import hyperlinked_object
 from nautobot.core.utils.config import get_settings_or_config
 from nautobot.dcim.choices import InterfaceModeChoices
 
@@ -141,3 +143,31 @@ def convert_watts_to_va(watts, power_factor):
     if not watts:
         return 0
     return int(watts / power_factor)
+
+
+def render_software_version_and_image_files(instance, software_version, context):
+    display = hyperlinked_object(software_version)
+    overridden_software_image_files = instance.software_image_files.all()
+    if software_version is not None:
+        display += format_html(
+            '<ul class="software-image-hierarchy">{}</ul>',
+            format_html_join(
+                "\n",
+                "<li>{}{}</li>",
+                [
+                    [
+                        hyperlinked_object(img, "image_file_name"),
+                        " (overridden)" if overridden_software_image_files.exists() else "",
+                    ]
+                    for img in software_version.software_image_files.restrict(context["request"].user, "view")
+                ],
+            ),
+        )
+    if overridden_software_image_files.exists():
+        display += format_html(
+            "<br><strong>Software Image Files Overridden:</strong>\n<ul>{}</ul>",
+            format_html_join(
+                "\n", "<li>{}</li>", [[hyperlinked_object(img)] for img in overridden_software_image_files.all()]
+            ),
+        )
+    return display