nautobot 2.3.0__py3-none-any.whl → 2.3.0b1__py3-none-any.whl

nautobot/extras/tests/test_jobs.py

@@ -55,10 +55,10 @@ class JobTest(TestCase):
 
         self.assertInHTML(
             """<tr><th><label for="id_var_int">Var int:</label></th><td>
-<input class="form-control" id="id_var_int" max="3600" name="var_int" placeholder="None" required type="number" value="0">
+<input class="form-control form-control" id="id_var_int" max="3600" name="var_int" placeholder="None" required type="number" value="0">
 <br><span class="helptext">Test default of 0 Falsey</span></td></tr>
 <tr><th><label for="id_var_int_no_default">Var int no default:</label></th><td>
-<input class="form-control" id="id_var_int_no_default" max="3600" name="var_int_no_default" placeholder="None" type="number">
+<input class="form-control form-control" id="id_var_int_no_default" max="3600" name="var_int_no_default" placeholder="None" type="number">
 <br><span class="helptext">Test default without default</span></td></tr>""",
             form.as_table(),
         )

nautobot/extras/tests/test_models.py

@@ -1352,7 +1352,7 @@ class ObjectMetadataTest(ModelTestCases.BaseModelTestCase):
                 value="Invalid assigned object type",
                 scoped_fields=["status"],
                 assigned_object_type=ContentType.objects.get_for_model(IPAddress),
-                assigned_object_id=Contact.objects.filter(associated_object_metadata__isnull=True).first().pk,
+                assigned_object_id=Contact.objects.first().pk,
             )
             obj_metadata.validated_save()
 
@@ -1362,29 +1362,29 @@ class ObjectMetadataTest(ModelTestCases.BaseModelTestCase):
         )
         type_contact_team.content_types.add(ContentType.objects.get_for_model(Contact))
         type_contact_team.content_types.add(ContentType.objects.get_for_model(Team))
-        instance1 = ObjectMetadata(
+        instance1 = ObjectMetadata.objects.create(
             metadata_type=type_contact_team,
             contact=Contact.objects.first(),
             team=Team.objects.first(),
             scoped_fields=["address"],
             assigned_object_type=ContentType.objects.get_for_model(Contact),
-            assigned_object_id=Contact.objects.filter(associated_object_metadata__isnull=True).first().pk,
+            assigned_object_id=Contact.objects.first().pk,
         )
-        instance2 = ObjectMetadata(
+        instance2 = ObjectMetadata.objects.create(
             metadata_type=type_contact_team,
             contact=None,
             team=None,
             scoped_fields=["phone"],
             assigned_object_type=ContentType.objects.get_for_model(Contact),
-            assigned_object_id=Contact.objects.filter(associated_object_metadata__isnull=True).last().pk,
+            assigned_object_id=Contact.objects.last().pk,
         )
-        instance3 = ObjectMetadata(
+        instance3 = ObjectMetadata.objects.create(
             metadata_type=type_contact_team,
             contact=Contact.objects.first(),
             team=None,
             scoped_fields=["email"],
             assigned_object_type=ContentType.objects.get_for_model(Team),
-            assigned_object_id=Team.objects.filter(associated_object_metadata__isnull=True).first().pk,
+            assigned_object_id=Team.objects.first().pk,
         )
         with self.assertRaises(ValidationError):
             instance1.validated_save()
@@ -1407,7 +1407,7 @@ class ObjectMetadataTest(ModelTestCases.BaseModelTestCase):
             value="Some text value",
             scoped_fields=["status", "parent"],
             assigned_object_type=obj_type,
-            assigned_object_id=Location.objects.filter(associated_object_metadata__isnull=True).first().pk,
+            assigned_object_id=Location.objects.first().pk,
         )
         obj_metadata.save()
 
@@ -1440,7 +1440,7 @@ class ObjectMetadataTest(ModelTestCases.BaseModelTestCase):
             value=15,
             scoped_fields=["status", "parent"],
             assigned_object_type=obj_type,
-            assigned_object_id=Location.objects.filter(associated_object_metadata__isnull=True).first().pk,
+            assigned_object_id=Location.objects.first().pk,
         )
         obj_metadata.validated_save()
 
@@ -1484,7 +1484,7 @@ class ObjectMetadataTest(ModelTestCases.BaseModelTestCase):
             value=15.245,
             scoped_fields=["status", "parent"],
             assigned_object_type=obj_type,
-            assigned_object_id=Location.objects.filter(associated_object_metadata__isnull=True).first().pk,
+            assigned_object_id=Location.objects.first().pk,
         )
         obj_metadata.validated_save()
 
@@ -1525,7 +1525,7 @@ class ObjectMetadataTest(ModelTestCases.BaseModelTestCase):
             value=False,
             scoped_fields=["status", "parent"],
             assigned_object_type=obj_type,
-            assigned_object_id=Location.objects.filter(associated_object_metadata__isnull=True).first().pk,
+            assigned_object_id=Location.objects.first().pk,
         )
         obj_metadata.validated_save()
 
@@ -1559,7 +1559,7 @@ class ObjectMetadataTest(ModelTestCases.BaseModelTestCase):
             value="1994-01-01",
             scoped_fields=["status", "parent"],
             assigned_object_type=obj_type,
-            assigned_object_id=Location.objects.filter(associated_object_metadata__isnull=True).first().pk,
+            assigned_object_id=Location.objects.first().pk,
         )
         obj_metadata.validated_save()
 
@@ -1596,7 +1596,7 @@ class ObjectMetadataTest(ModelTestCases.BaseModelTestCase):
             value="2024-06-27T17:58:47-0500",
             scoped_fields=["status", "parent"],
             assigned_object_type=obj_type,
-            assigned_object_id=Location.objects.filter(associated_object_metadata__isnull=True).first().pk,
+            assigned_object_id=Location.objects.first().pk,
         )
         obj_metadata.validated_save()
 
@@ -1663,7 +1663,7 @@ class ObjectMetadataTest(ModelTestCases.BaseModelTestCase):
             value="Option A",
             scoped_fields=["status", "parent"],
             assigned_object_type=obj_type,
-            assigned_object_id=Location.objects.filter(associated_object_metadata__isnull=True).first().pk,
+            assigned_object_id=Location.objects.first().pk,
         )
         obj_metadata.validated_save()
 
@@ -1689,7 +1689,7 @@ class ObjectMetadataTest(ModelTestCases.BaseModelTestCase):
             value=["Option A"],
             scoped_fields=["status", "parent"],
             assigned_object_type=obj_type,
-            assigned_object_id=Location.objects.filter(associated_object_metadata__isnull=True).first().pk,
+            assigned_object_id=Location.objects.first().pk,
         )
         obj_metadata.validated_save()
 
@@ -1700,22 +1700,20 @@ class ObjectMetadataTest(ModelTestCases.BaseModelTestCase):
         self.assertIn(f"Invalid choice(s) ({invalid_options})", str(context.exception))
 
     def test_no_scoped_fields_overlap(self):
-        """
-        Test that overlapping scoped_fields of ObjectMetadata with same metadata_type/assigned_object is not allowed.
-        """
+        """Test that overlapping between scoped_fields of ObjectMetadata with the same metadata_type and the same assigned_object is not allowed"""
         ObjectMetadata.objects.create(
             metadata_type=MetadataType.objects.first(),
             contact=Contact.objects.first(),
             scoped_fields=["host", "mask_length", "type", "role", "status"],
             assigned_object_type=ContentType.objects.get_for_model(IPAddress),
-            assigned_object_id=IPAddress.objects.filter(associated_object_metadata__isnull=True).first().pk,
+            assigned_object_id=IPAddress.objects.first().pk,
         )
         instance2 = ObjectMetadata.objects.create(
             metadata_type=MetadataType.objects.first(),
             contact=Contact.objects.first(),
             scoped_fields=[],
             assigned_object_type=ContentType.objects.get_for_model(IPAddress),
-            assigned_object_id=IPAddress.objects.filter(associated_object_metadata__isnull=True).first().pk,
+            assigned_object_id=IPAddress.objects.first().pk,
         )
         with self.assertRaises(ValidationError):
             # try scope all fields

nautobot/extras/tests/test_views.py

@@ -2594,7 +2594,7 @@ class JobTestCase(
 
         self.assertInHTML('<option value="uniquequeue" selected>', content)
         self.assertInHTML(
-            '<input type="text" name="var" value="456" class="form-control" required placeholder="None" id="id_var">',
+            '<input type="text" name="var" value="456" class="form-control form-control" required placeholder="None" id="id_var">',
             content,
         )
         self.assertInHTML('<input type="hidden" name="_profile" value="True" id="id__profile">', content)
@@ -2994,27 +2994,6 @@ class JobButtonRenderingTestCase(TestCase):
             )
 
 
-class JobCustomTemplateTestCase(TestCase):
-    @classmethod
-    def setUpTestData(cls):
-        # Job model objects are automatically created during database migrations
-
-        # But we do need to make sure the ones we're testing are flagged appropriately
-        cls.example_job = Job.objects.get(job_class_name="ExampleCustomFormJob")
-        cls.example_job.enabled = True
-        cls.example_job.save()
-
-        cls.run_url = reverse("extras:job_run", kwargs={"pk": cls.example_job.pk})
-
-    def test_rendering_custom_template(self):
-        obj_perm = ObjectPermission(name="Test permission", actions=["view", "run"])
-        obj_perm.save()
-        obj_perm.users.add(self.user)
-        obj_perm.object_types.add(ContentType.objects.get_for_model(Job))
-        with self.assertTemplateUsed("example_app/custom_job_form.html"):
-            self.client.get(self.run_url)
-
-
 # TODO: Convert to StandardTestCases.Views
 class ObjectChangeTestCase(TestCase):
     user_permissions = ("extras.view_objectchange",)
@@ -3050,7 +3029,8 @@ class ObjectChangeTestCase(TestCase):
 
 
 class ObjectMetadataTestCase(
-    ViewTestCases.GetObjectViewTestCase,
+    ViewTestCases.DeleteObjectViewTestCase,
+    ViewTestCases.BulkDeleteObjectsViewTestCase,
     ViewTestCases.GetObjectChangelogViewTestCase,
     ViewTestCases.ListObjectsViewTestCase,
 ):

nautobot/extras/utils.py

@@ -1,5 +1,4 @@
 import collections
-import contextlib
 import hashlib
 import hmac
 import logging
@@ -15,7 +14,6 @@ from django.db import transaction
 from django.db.models import Q
 from django.template.loader import get_template, TemplateDoesNotExist
 from django.utils.deconstruct import deconstructible
-import redis.exceptions
 
 from nautobot.core.choices import ColorChoices
 from nautobot.core.constants import CHARFIELD_MAX_LENGTH
@@ -111,17 +109,12 @@ class ChangeLoggedModelsQuery(FeaturedQueryMixin):
 def change_logged_models_queryset():
     """
     Cacheable function for cases where we need this queryset many times, such as when saving multiple objects.
-
-    Cache is cleared by post_migrate signal (nautobot.extras.signals.post_migrate_clear_content_type_caches).
     """
-    queryset = None
     cache_key = "nautobot.extras.utils.change_logged_models_queryset"
-    with contextlib.suppress(redis.exceptions.ConnectionError):
-        queryset = cache.get(cache_key)
+    queryset = cache.get(cache_key)
     if queryset is None:
         queryset = ChangeLoggedModelsQuery().as_queryset()
-        with contextlib.suppress(redis.exceptions.ConnectionError):
-            cache.set(cache_key, queryset)
+        cache.set(cache_key, queryset)
     return queryset
 
 
@@ -167,7 +160,7 @@ class FeatureQuery:
         """
         Given an extras feature, return a iterable of app_label: [models] for content type lookup.
 
-        Misnamed, as it returns an iterable of (key, value) (i.e. dict.items()) rather than an actual dict.
+        Mis-named, as it returns an iterable of (key, value) (i.e. dict.items()) rather than an actual dict.
 
         Raises a KeyError if the given feature doesn't exist.
         """
@@ -180,34 +173,12 @@ class FeatureQuery:
 
             >>> FeatureQuery('statuses').get_choices()
             [('dcim.device', 13), ('dcim.rack', 34)]
-
-        Cache is cleared by post_migrate signal (nautobot.extras.signals.post_migrate_clear_content_type_caches).
         """
-        choices = None
-        cache_key = f"nautobot.extras.utils.FeatureQuery.choices.{self.feature}"
-        with contextlib.suppress(redis.exceptions.ConnectionError):
-            choices = cache.get(cache_key)
-        if choices is None:
-            choices = [(f"{ct.app_label}.{ct.model}", ct.pk) for ct in ContentType.objects.filter(self.get_query())]
-            with contextlib.suppress(redis.exceptions.ConnectionError):
-                cache.set(cache_key, choices)
-        return choices
+        return [(f"{ct.app_label}.{ct.model}", ct.pk) for ct in ContentType.objects.filter(self.get_query())]
 
     def list_subclasses(self):
-        """
-        Return a list of model classes that declare this feature.
-
-        Cache is cleared by post_migrate signal (nautobot.extras.signals.post_migrate_clear_content_type_caches).
-        """
-        subclasses = None
-        cache_key = f"nautobot.extras.utils.FeatureQuery.subclasses.{self.feature}"
-        with contextlib.suppress(redis.exceptions.ConnectionError):
-            subclasses = cache.get(cache_key)
-        if subclasses is None:
-            subclasses = [ct.model_class() for ct in ContentType.objects.filter(self.get_query())]
-            with contextlib.suppress(redis.exceptions.ConnectionError):
-                cache.set(cache_key, subclasses)
-        return subclasses
+        """Return a list of model classes that declare this feature."""
+        return [ct.model_class() for ct in ContentType.objects.filter(self.get_query())]
 
 
 @deconstructible

nautobot/extras/views.py

@@ -1,3 +1,4 @@
+from datetime import timedelta
 import logging
 from urllib.parse import parse_qs
 
@@ -1387,25 +1388,55 @@ class JobRunView(ObjectPermissionRequiredMixin, View):
             schedule_type = schedule_form.cleaned_data["_schedule_type"]
 
             if (not dryrun and job_model.approval_required) or schedule_type in JobExecutionType.SCHEDULE_CHOICES:
-                scheduled_job = ScheduledJob.create_schedule(
-                    job_model,
-                    request.user,
-                    name=schedule_form.cleaned_data.get("_schedule_name"),
-                    start_time=schedule_form.cleaned_data.get("_schedule_start_time"),
+                crontab = ""
+
+                if schedule_type == JobExecutionType.TYPE_IMMEDIATELY:
+                    # The job must be approved.
+                    # If the schedule_type is immediate, we still create the task, but mark it for approval
+                    # as a once in the future task with the due date set to the current time. This means
+                    # when approval is granted, the task is immediately due for execution.
+                    schedule_type = JobExecutionType.TYPE_FUTURE
+                    schedule_datetime = timezone.now()
+                    schedule_name = f"{job_model} - {schedule_datetime}"
+
+                else:
+                    schedule_name = schedule_form.cleaned_data["_schedule_name"]
+
+                    if schedule_type == JobExecutionType.TYPE_CUSTOM:
+                        crontab = schedule_form.cleaned_data["_recurrence_custom_time"]
+                        # doing .get("key", "default") returns None instead of "default" here for some reason
+                        schedule_datetime = schedule_form.cleaned_data.get("_schedule_start_time")
+                        if schedule_datetime is None:
+                            # "_schedule_start_time" is checked against ScheduledJob.earliest_possible_time()
+                            # which returns timezone.now() + timedelta(seconds=15)
+                            schedule_datetime = timezone.now() + timedelta(seconds=20)
+                    else:
+                        schedule_datetime = schedule_form.cleaned_data["_schedule_start_time"]
+
+                celery_kwargs = {"nautobot_job_profile": profile, "queue": task_queue}
+                scheduled_job = ScheduledJob(
+                    name=schedule_name,
+                    task=job_model.class_path,
+                    job_model=job_model,
+                    start_time=schedule_datetime,
+                    description=f"Nautobot job {schedule_name} scheduled by {request.user} for {schedule_datetime}",
+                    kwargs=job_class.serialize_data(job_form.cleaned_data),
+                    celery_kwargs=celery_kwargs,
                     interval=schedule_type,
-                    crontab=schedule_form.cleaned_data.get("_recurrence_custom_time"),
+                    one_off=schedule_type == JobExecutionType.TYPE_FUTURE,
+                    queue=task_queue,
+                    user=request.user,
                     approval_required=job_model.approval_required,
-                    task_queue=task_queue,
-                    profile=profile,
-                    **job_class.serialize_data(job_form.cleaned_data),
+                    crontab=crontab,
                 )
+                scheduled_job.validated_save()
 
                 if job_model.approval_required:
-                    messages.success(request, f"Job {scheduled_job.name} successfully submitted for approval")
-                    return redirect(return_url or "extras:scheduledjob_approval_queue_list")
+                    messages.success(request, f"Job {schedule_name} successfully submitted for approval")
+                    return redirect(return_url if return_url else "extras:scheduledjob_approval_queue_list")
                 else:
-                    messages.success(request, f"Job {scheduled_job.name} successfully scheduled")
-                    return redirect(return_url or "extras:scheduledjob_list")
+                    messages.success(request, f"Job {schedule_name} successfully scheduled")
+                    return redirect(return_url if return_url else "extras:scheduledjob_list")
 
             else:
                 # Enqueue job for immediate execution
@@ -1691,12 +1722,7 @@ class SavedViewUIViewSet(
         view_name = new_global_default_view.view
         message = ""
         if new_global_default_view.is_global_default:
-            message = format_html(
-                '<br>The global default saved view for "{}" is set to <a href="{}">{}</a>',
-                view_name,
-                new_global_default_view.get_absolute_url(),
-                new_global_default_view.name,
-            )
+            message += f"<br>The global default saved View for '{view_name}' is set to <a href='{new_global_default_view.get_absolute_url()}'>{new_global_default_view.name}</a>."
         return message
 
     def list(self, request, *args, **kwargs):
@@ -2050,13 +2076,8 @@ class JobLogEntryTableView(generic.GenericView):
         else:
             queryset = instance.job_log_entries.all()
         log_table = tables.JobLogEntryTable(data=queryset, user=request.user)
-        paginate = {
-            "paginator_class": EnhancedPaginator,
-            "per_page": get_paginate_count(request),
-        }
-        RequestConfig(request, paginate).configure(log_table)
-        table = log_table.as_html(request)
-        return HttpResponse(table)
+        RequestConfig(request).configure(log_table)
+        return HttpResponse(log_table.as_html(request))
 
 
 #
@@ -2216,13 +2237,15 @@ class MetadataTypeUIViewSet(NautobotUIViewSet):
 
 
 class ObjectMetadataUIViewSet(
+    ObjectBulkDestroyViewMixin,
     ObjectChangeLogViewMixin,
+    ObjectDestroyViewMixin,
     ObjectDetailViewMixin,
     ObjectListViewMixin,
 ):
     filterset_class = filters.ObjectMetadataFilterSet
     filterset_form_class = forms.ObjectMetadataFilterForm
-    queryset = ObjectMetadata.objects.all().order_by("assigned_object_type", "assigned_object_id", "scoped_fields")
+    queryset = ObjectMetadata.objects.all().order_by("assigned_object_type", "scoped_fields")
     serializer_class = serializers.ObjectMetadataSerializer
     table_class = tables.ObjectMetadataTable
     action_buttons = ("export",)

nautobot/ipam/filters.py

@@ -498,7 +498,7 @@ class IPAddressFilterSet(
             return queryset.none()
         interface_ids = []
         for device in devices:
-            interface_ids.extend(device.vc_interfaces.values_list("id", flat=True))
+            interface_ids.extend(device.all_interfaces.values_list("id", flat=True))
         return queryset.filter(interfaces__in=interface_ids)
 
     def filter_virtual_machine(self, queryset, name, value):

nautobot/ipam/forms.py

@@ -899,7 +899,7 @@ class ServiceForm(NautobotModelForm):
         # Limit IP address choices to those assigned to interfaces of the parent device/VM
         if self.instance.device:
             self.fields["ip_addresses"].queryset = IPAddress.objects.filter(
-                interfaces__in=self.instance.device.vc_interfaces.values_list("id", flat=True)
+                interfaces__in=self.instance.device.all_interfaces.values_list("id", flat=True)
             )
         elif self.instance.virtual_machine:
             self.fields["ip_addresses"].queryset = IPAddress.objects.filter(

nautobot/ipam/models.py

@@ -49,6 +49,7 @@ logger = logging.getLogger(__name__)
 @extras_features(
     "custom_links",
     "custom_validators",
+    "dynamic_groups",
     "export_templates",
     "graphql",
     "locations",
@@ -398,6 +399,7 @@ class RIR(OrganizationalModel):
 @extras_features(
     "custom_links",
     "custom_validators",
+    "dynamic_groups",
     "export_templates",
     "graphql",
     "locations",
@@ -499,6 +501,11 @@ class Prefix(PrimaryModel):
         "type",
         "vlan",
     ]
+    """
+    dynamic_group_filter_fields = {
+        "vrf": "vrf_id",  # Duplicate filter fields that will be collapsed in 2.0
+    }
+    """
 
     class Meta:
         ordering = (
@@ -979,6 +986,7 @@ class PrefixLocationAssignment(BaseModel):
 @extras_features(
     "custom_links",
     "custom_validators",
+    "dynamic_groups",
     "export_templates",
     "graphql",
     "statuses",
@@ -1012,7 +1020,7 @@ class IPAddress(PrimaryModel):
     parent = models.ForeignKey(
         "ipam.Prefix",
         blank=True,
-        null=True,  # TODO remove this, it shouldn't be permitted for the database!
+        null=True,
         related_name="ip_addresses",  # `IPAddress` to use `related_name="ip_addresses"`
         on_delete=models.PROTECT,
         help_text="The parent Prefix of this IPAddress.",
@@ -1109,7 +1117,7 @@ class IPAddress(PrimaryModel):
             raise ValidationError({"namespace": "No suitable parent Prefix exists in this Namespace"}) from e
 
     def clean(self):
-        self.address = self.address  # not a no-op - forces re-calling of self._deconstruct_address()
+        super().clean()
 
         # Validate that host is not being modified
         if self.present_in_database:
@@ -1123,8 +1131,8 @@ class IPAddress(PrimaryModel):
 
         closest_parent = self._get_closest_parent()
         # Validate `parent` can be used as the parent for this ipaddress
-        if closest_parent is not None:
-            if self.parent is not None and self.parent != closest_parent:
+        if self.parent and closest_parent:
+            if self.parent != closest_parent:
                 raise ValidationError(
                     {
                         "parent": (
@@ -1136,20 +1144,23 @@ class IPAddress(PrimaryModel):
             self.parent = closest_parent
             self._namespace = None
 
+    def save(self, *args, **kwargs):
         # 3.0 TODO: uncomment the below to enforce this constraint
         # if self.parent.type != choices.PrefixTypeChoices.TYPE_NETWORK:
         #     err_msg = f"IP addresses cannot be created in {self.parent.type} prefixes. You must create a network prefix first."
         #     raise ValidationError({"address": err_msg})
 
+        self.address = self.address  # not a no-op - forces re-calling of self._deconstruct_address()
+
         # Force dns_name to lowercase
         if not self.dns_name.islower:
             self.dns_name = self.dns_name.lower()
 
-        super().clean()
-
-    def save(self, *args, **kwargs):
-        self.clean()  # MUST do data fixup as above
-
+        # Host and mask_length are required to get closest parent
+        closest_parent = self._get_closest_parent()
+        if closest_parent is not None:
+            self.parent = closest_parent
+            self._namespace = None
         super().save(*args, **kwargs)
 
     @property

nautobot/ipam/tables.py

@@ -355,9 +355,6 @@ class PrefixTable(StatusTableMixin, RoleTableMixin, BaseTable):
     location_count = LinkedCountColumn(
         viewname="dcim:location_list", url_params={"prefixes": "pk"}, verbose_name="Locations"
     )
-    cloud_networks_count = LinkedCountColumn(
-        viewname="cloud:cloudnetwork_list", url_params={"prefixes": "pk"}, verbose_name="Cloud Networks"
-    )
 
     class Meta(BaseTable.Meta):
         model = Prefix
@@ -371,7 +368,6 @@ class PrefixTable(StatusTableMixin, RoleTableMixin, BaseTable):
             "namespace",
             "tenant",
             "location_count",
-            "cloud_networks_count",
             "vlan",
             "role",
             "rir",

nautobot/ipam/tests/test_models.py

@@ -1003,9 +1003,8 @@ class TestIPAddress(ModelTestCases.BaseModelTestCase):
     def test_duplicate_global_unique(self):
         """Test that duplicate IPs in the same Namespace raises an error."""
         IPAddress.objects.create(address="192.0.2.1/24", status=self.status, namespace=self.namespace)
-        duplicate_ip = IPAddress(address="192.0.2.1/24", status=self.status, namespace=self.namespace)
-        with self.assertRaises(ValidationError):
-            duplicate_ip.full_clean()
+        with self.assertRaises(IntegrityError):
+            IPAddress.objects.create(address="192.0.2.1/24", status=self.status, namespace=self.namespace)
 
     def test_multiple_nat_outside_list(self):
         """

nautobot/ipam/views.py

@@ -15,7 +15,7 @@ from django.views.generic import View
 from django_tables2 import RequestConfig
 import netaddr
 
-from nautobot.cloud.tables import CloudNetworkTable
+from nautobot.cloud.tables import CloudNetworkPrefixAssignmentTable
 from nautobot.core.models.querysets import count_related
 from nautobot.core.utils.config import get_settings_or_config
 from nautobot.core.utils.permissions import get_permission_for_model
@@ -456,9 +456,8 @@ class PrefixView(generic.ObjectView):
         vrfs = instance.vrf_assignments.restrict(request.user, "view")
         vrf_table = tables.VRFPrefixAssignmentTable(vrfs, orderable=False)
 
-        cloud_networks = instance.cloud_networks.restrict(request.user, "view")
-        cloud_network_table = CloudNetworkTable(cloud_networks, orderable=False)
-        cloud_network_table.exclude = ("actions", "assigned_prefix_count", "circuit_count", "cloud_service_count")
+        cloud_networks = instance.cloud_network_assignments.restrict(request.user, "view")
+        cloud_network_table = CloudNetworkPrefixAssignmentTable(cloud_networks, orderable=False)
 
         return {
             "vrf_table": vrf_table,
@@ -935,8 +934,14 @@ class IPAddressAssignView(view_mixins.GetReturnURLMixin, generic.ObjectView):
             ip_addresses = IPAddress.objects.restrict(request.user, "view").filter(pk__in=pks)
             interface.ip_addresses.add(*ip_addresses)
             return redirect(self.get_return_url(request))
-        messages.error(request, "Please select at least one IP Address from the table.")
-        return redirect(request.get_full_path())
+
+        return render(
+            request,
+            "ipam/ipaddress_assign.html",
+            {
+                "return_url": self.get_return_url(request),
+            },
+        )
 
 
 class IPAddressMergeView(view_mixins.GetReturnURLMixin, view_mixins.ObjectPermissionRequiredMixin, View):

nautobot/project-static/css/base.css

@@ -300,7 +300,6 @@ table.report th a {
 }
 .panel table {
     margin-bottom: 0;
-    overflow: hidden;
 }
 .panel .table th {
     border-bottom-width: 1px;