nautobot 2.0.0a3__py3-none-any.whl → 2.0.0b1__py3-none-any.whl

nautobot/dcim/migrations/0024_alter_device_and_rack_role_add_new_role.py

@@ -7,7 +7,7 @@ import nautobot.extras.models.roles
 
 class Migration(migrations.Migration):
     dependencies = [
-        ("extras", "0061_collect_roles_from_related_apps_roles"),
+        ("extras", "0062_collect_roles_from_related_apps_roles"),
         ("dcim", "0023_alter_interface_mac_address"),
     ]
 
@@ -39,6 +39,7 @@ class Migration(migrations.Migration):
                 on_delete=django.db.models.deletion.PROTECT,
                 related_name="dcim_device_related",
                 to="extras.role",
+                null=True,
             ),
         ),
         # ##########

nautobot/dcim/migrations/0025_device_and_rack_roles_data_migrations.py

@@ -5,26 +5,32 @@ from nautobot.extras.utils import migrate_role_data
 
 
 def migrate_data_from_legacy_role_to_new_role(apps, schema):
-    """Copy record from role to temp_role"""
-
-    role_model = apps.get_model("extras", "Role")
+    """Transfer data from legacy_role to new_role."""
+    to_role_model = apps.get_model("extras", "Role")
     for model_name in ("Device", "Rack"):
         model = apps.get_model("dcim", model_name)
-        migrate_role_data(model=model, role_model=role_model)
+        from_role_model = apps.get_model("dcim", f"{model_name}Role")
+        migrate_role_data(
+            model_to_migrate=model,
+            from_role_field_name="legacy_role",
+            from_role_model=from_role_model,
+            to_role_field_name="new_role",
+            to_role_model=to_role_model,
+        )
 
 
 def reverse_role_data_migrate(apps, schema):
-    """Reverse changes made to new_role"""
-
-    model_role_map = {"Device": "DeviceRole", "Rack": "RackRole"}
-    for model_name, role_name in model_role_map.items():
+    """Transfer data from new_role to legacy_role."""
+    from_role_model = apps.get_model("extras", "Role")
+    for model_name in ("Device", "Rack"):
         model = apps.get_model("dcim", model_name)
-        role_model = apps.get_model("dcim", role_name)
+        to_role_model = apps.get_model("dcim", f"{model_name}Role")
         migrate_role_data(
-            model=model,
-            role_model=role_model,
-            legacy_role="new_role",
-            new_role="legacy_role",
+            model_to_migrate=model,
+            from_role_field_name="new_role",
+            from_role_model=from_role_model,
+            to_role_field_name="legacy_role",
+            to_role_model=to_role_model,
         )
 
 

nautobot/dcim/migrations/0027_remove_device_role_and_rack_role.py

@@ -8,7 +8,7 @@ class Migration(migrations.Migration):
         ("dcim", "0026_rename_device_and_rack_role"),
         ("ipam", "0012_rename_ipam_roles"),
         ("virtualization", "0014_rename_virtualmachine_roles"),
-        ("extras", "0065_rename_configcontext_role"),
+        ("extras", "0066_rename_configcontext_role"),
     ]
 
     operations = [

nautobot/dcim/migrations/0028_rename_foreignkey_fields.py

@@ -8,7 +8,7 @@ import nautobot.extras.utils
 
 class Migration(migrations.Migration):
     dependencies = [
-        ("extras", "0065_rename_configcontext_role"),
+        ("extras", "0066_rename_configcontext_role"),
         ("contenttypes", "0002_remove_content_type_name"),
         ("dcim", "0027_remove_device_role_and_rack_role"),
     ]

nautobot/dcim/migrations/0030_migrate_region_and_site_data_to_locations.py

@@ -6,7 +6,7 @@ import uuid
 from django.db import migrations
 from django.db.utils import IntegrityError
 
-from nautobot.core.models.generics import _NautobotTaggableManager
+from nautobot.core.models.managers import TagsManager
 from nautobot.core.models.utils import serialize_object
 from nautobot.extras import choices as extras_choices
 from nautobot.extras import models as extras_models
@@ -381,7 +381,7 @@ def reassign_model_instances_to_locations(apps, model):
     # https://github.com/jazzband/django-taggit/issues/101
     # https://github.com/jazzband/django-taggit/issues/454
     for model_loc in model_locs:
-        model_loc.tags = _NautobotTaggableManager(
+        model_loc.tags = TagsManager(
             through=extras_models.TaggedItem, model=Location, instance=model_loc, prefetch_cache_name="tags"
         )
 

nautobot/dcim/migrations/0035_related_name_changes.py

@@ -11,7 +11,7 @@ import nautobot.extras.utils
 class Migration(migrations.Migration):
     dependencies = [
         ("contenttypes", "0002_remove_content_type_name"),
-        ("extras", "0067_created_datetime"),
+        ("extras", "0068_created_datetime"),
         ("dcim", "0034_fixup_fks_and_related_names"),
     ]
 

nautobot/dcim/migrations/0036_remove_region_and_site.py

@@ -8,7 +8,7 @@ class Migration(migrations.Migration):
         ("circuits", "0011_remove_site_foreign_key_from_circuit_termination_class"),
         ("dcim", "0035_related_name_changes"),
         ("django_celery_results", "0006_taskresult_date_created"),
-        ("extras", "0068_remove_site_and_region_attributes_from_config_context"),
+        ("extras", "0069_remove_site_and_region_attributes_from_config_context"),
         ("ipam", "0018_remove_site_foreign_key_from_ipam_models"),
         ("virtualization", "0016_remove_site_foreign_key_from_cluster_class"),
     ]

nautobot/dcim/migrations/0040_tagsfield.py

@@ -0,0 +1,109 @@
+# Generated by Django 3.2.18 on 2023-04-21 13:59
+
+from django.db import migrations
+import nautobot.core.models.fields
+
+
+class Migration(migrations.Migration):
+    dependencies = [
+        ("extras", "0079_tagsfield"),
+        ("dcim", "0039_remove_slug"),
+    ]
+
+    operations = [
+        migrations.AlterField(
+            model_name="cable",
+            name="tags",
+            field=nautobot.core.models.fields.TagsField(through="extras.TaggedItem", to="extras.Tag"),
+        ),
+        migrations.AlterField(
+            model_name="consoleport",
+            name="tags",
+            field=nautobot.core.models.fields.TagsField(through="extras.TaggedItem", to="extras.Tag"),
+        ),
+        migrations.AlterField(
+            model_name="consoleserverport",
+            name="tags",
+            field=nautobot.core.models.fields.TagsField(through="extras.TaggedItem", to="extras.Tag"),
+        ),
+        migrations.AlterField(
+            model_name="device",
+            name="tags",
+            field=nautobot.core.models.fields.TagsField(through="extras.TaggedItem", to="extras.Tag"),
+        ),
+        migrations.AlterField(
+            model_name="devicebay",
+            name="tags",
+            field=nautobot.core.models.fields.TagsField(through="extras.TaggedItem", to="extras.Tag"),
+        ),
+        migrations.AlterField(
+            model_name="deviceredundancygroup",
+            name="tags",
+            field=nautobot.core.models.fields.TagsField(through="extras.TaggedItem", to="extras.Tag"),
+        ),
+        migrations.AlterField(
+            model_name="devicetype",
+            name="tags",
+            field=nautobot.core.models.fields.TagsField(through="extras.TaggedItem", to="extras.Tag"),
+        ),
+        migrations.AlterField(
+            model_name="frontport",
+            name="tags",
+            field=nautobot.core.models.fields.TagsField(through="extras.TaggedItem", to="extras.Tag"),
+        ),
+        migrations.AlterField(
+            model_name="interface",
+            name="tags",
+            field=nautobot.core.models.fields.TagsField(through="extras.TaggedItem", to="extras.Tag"),
+        ),
+        migrations.AlterField(
+            model_name="inventoryitem",
+            name="tags",
+            field=nautobot.core.models.fields.TagsField(through="extras.TaggedItem", to="extras.Tag"),
+        ),
+        migrations.AlterField(
+            model_name="location",
+            name="tags",
+            field=nautobot.core.models.fields.TagsField(through="extras.TaggedItem", to="extras.Tag"),
+        ),
+        migrations.AlterField(
+            model_name="powerfeed",
+            name="tags",
+            field=nautobot.core.models.fields.TagsField(through="extras.TaggedItem", to="extras.Tag"),
+        ),
+        migrations.AlterField(
+            model_name="poweroutlet",
+            name="tags",
+            field=nautobot.core.models.fields.TagsField(through="extras.TaggedItem", to="extras.Tag"),
+        ),
+        migrations.AlterField(
+            model_name="powerpanel",
+            name="tags",
+            field=nautobot.core.models.fields.TagsField(through="extras.TaggedItem", to="extras.Tag"),
+        ),
+        migrations.AlterField(
+            model_name="powerport",
+            name="tags",
+            field=nautobot.core.models.fields.TagsField(through="extras.TaggedItem", to="extras.Tag"),
+        ),
+        migrations.AlterField(
+            model_name="rack",
+            name="tags",
+            field=nautobot.core.models.fields.TagsField(through="extras.TaggedItem", to="extras.Tag"),
+        ),
+        migrations.AlterField(
+            model_name="rackreservation",
+            name="tags",
+            field=nautobot.core.models.fields.TagsField(through="extras.TaggedItem", to="extras.Tag"),
+        ),
+        migrations.AlterField(
+            model_name="rearport",
+            name="tags",
+            field=nautobot.core.models.fields.TagsField(through="extras.TaggedItem", to="extras.Tag"),
+        ),
+        migrations.AlterField(
+            model_name="virtualchassis",
+            name="tags",
+            field=nautobot.core.models.fields.TagsField(through="extras.TaggedItem", to="extras.Tag"),
+        ),
+    ]

nautobot/dcim/migrations/{0040_ipam__namespaces.py → 0041_ipam__namespaces.py}

@@ -7,7 +7,7 @@ import django.db.models.deletion
 class Migration(migrations.Migration):
     dependencies = [
         ("ipam", "0026_ipaddress_remove_assigned_object"),
-        ("dcim", "0039_remove_slug"),
+        ("dcim", "0040_tagsfield"),
     ]
 
     operations = [

nautobot/dcim/migrations/0042_fixup_null_statuses.py

@@ -0,0 +1,51 @@
+from django.db import migrations
+
+from nautobot.core.choices import ColorChoices
+from nautobot.extras.utils import fixup_null_statuses
+
+
+def fixup_null_roles(apps, schema):
+    ContentType = apps.get_model("contenttypes", "ContentType")
+    Device = apps.get_model("dcim", "Device")
+    device_ct = ContentType.objects.get_for_model(Device)
+    Role = apps.get_model("extras", "Role")
+    instances_to_fixup = Device.objects.filter(role__isnull=True)
+    if instances_to_fixup.exists():
+        null_role, _ = Role.objects.get_or_create(
+            name="NULL",
+            defaults={
+                "color": ColorChoices.COLOR_BLACK,
+                "description": "Created by Nautobot to replace invalid null references",
+            },
+        )
+        null_role.content_types.add(device_ct)
+        updated_count = instances_to_fixup.update(role=null_role)
+        print(f"    Found and fixed {updated_count} instances of Device that had null 'role' field.")
+
+
+def migrate_null_statuses(apps, schema):
+    status_model = apps.get_model("extras", "Status")
+    ContentType = apps.get_model("contenttypes", "ContentType")
+    for model_name in (
+        "Cable",
+        "Device",
+        "DeviceRedundancyGroup",
+        "Interface",
+        "Location",
+        "PowerFeed",
+        "Rack",
+    ):
+        model = apps.get_model("dcim", model_name)
+        model_ct = ContentType.objects.get_for_model(model)
+        fixup_null_statuses(model=model, model_contenttype=model_ct, status_model=status_model)
+
+
+class Migration(migrations.Migration):
+    dependencies = [
+        ("dcim", "0041_ipam__namespaces"),
+    ]
+
+    operations = [
+        migrations.RunPython(migrate_null_statuses, migrations.RunPython.noop),
+        migrations.RunPython(fixup_null_roles, migrations.RunPython.noop),
+    ]

nautobot/dcim/migrations/0043_status_nonnullable.py

@@ -0,0 +1,72 @@
+# Generated by Django 3.2.18 on 2023-05-19 18:05
+
+from django.db import migrations
+import django.db.models.deletion
+import nautobot.extras.models.roles
+import nautobot.extras.models.statuses
+
+
+class Migration(migrations.Migration):
+    dependencies = [
+        ("extras", "0083_rename_computed_field_slug_to_key"),
+        ("dcim", "0042_fixup_null_statuses"),
+    ]
+
+    operations = [
+        migrations.AlterField(
+            model_name="cable",
+            name="status",
+            field=nautobot.extras.models.statuses.StatusField(
+                on_delete=django.db.models.deletion.PROTECT, related_name="cables", to="extras.status"
+            ),
+        ),
+        migrations.AlterField(
+            model_name="device",
+            name="role",
+            field=nautobot.extras.models.roles.RoleField(
+                on_delete=django.db.models.deletion.PROTECT, related_name="devices", to="extras.role"
+            ),
+        ),
+        migrations.AlterField(
+            model_name="device",
+            name="status",
+            field=nautobot.extras.models.statuses.StatusField(
+                on_delete=django.db.models.deletion.PROTECT, related_name="devices", to="extras.status"
+            ),
+        ),
+        migrations.AlterField(
+            model_name="deviceredundancygroup",
+            name="status",
+            field=nautobot.extras.models.statuses.StatusField(
+                on_delete=django.db.models.deletion.PROTECT, related_name="device_redundancy_groups", to="extras.status"
+            ),
+        ),
+        migrations.AlterField(
+            model_name="interface",
+            name="status",
+            field=nautobot.extras.models.statuses.StatusField(
+                on_delete=django.db.models.deletion.PROTECT, related_name="interfaces", to="extras.status"
+            ),
+        ),
+        migrations.AlterField(
+            model_name="location",
+            name="status",
+            field=nautobot.extras.models.statuses.StatusField(
+                on_delete=django.db.models.deletion.PROTECT, related_name="locations", to="extras.status"
+            ),
+        ),
+        migrations.AlterField(
+            model_name="powerfeed",
+            name="status",
+            field=nautobot.extras.models.statuses.StatusField(
+                on_delete=django.db.models.deletion.PROTECT, related_name="power_feeds", to="extras.status"
+            ),
+        ),
+        migrations.AlterField(
+            model_name="rack",
+            name="status",
+            field=nautobot.extras.models.statuses.StatusField(
+                on_delete=django.db.models.deletion.PROTECT, related_name="racks", to="extras.status"
+            ),
+        ),
+    ]

nautobot/dcim/models/cables.py

@@ -6,7 +6,6 @@ from django.contrib.contenttypes.models import ContentType
 from django.core.exceptions import ObjectDoesNotExist, ValidationError
 from django.db import models
 from django.db.models import Sum
-from django.urls import reverse
 from django.utils.functional import classproperty
 
 from nautobot.core.models.fields import ColorField
@@ -19,7 +18,7 @@ from nautobot.dcim.utils import (
     object_to_path_node,
     path_node_to_object,
 )
-from nautobot.extras.models import Status, StatusModel
+from nautobot.extras.models import Status, StatusField
 from nautobot.extras.utils import extras_features
 
 # TODO: There's an ugly circular-import pattern where if we move this import "up" to above, we get into an import loop
@@ -52,7 +51,7 @@ logger = logging.getLogger(__name__)
     "statuses",
     "webhooks",
 )
-class Cable(PrimaryModel, StatusModel):
+class Cable(PrimaryModel):
     """
     A physical connection between two endpoints.
     """
@@ -74,6 +73,7 @@ class Cable(PrimaryModel, StatusModel):
     termination_b_id = models.UUIDField()
     termination_b = GenericForeignKey(ct_field="termination_b_type", fk_field="termination_b_id")
     type = models.CharField(max_length=50, choices=CableTypeChoices, blank=True)
+    status = StatusField(blank=False, null=False)
     label = models.CharField(max_length=100, blank=True)
     color = ColorField(blank=True)
     length = models.PositiveSmallIntegerField(blank=True, null=True)
@@ -93,19 +93,6 @@ class Cable(PrimaryModel, StatusModel):
         to=Device, on_delete=models.CASCADE, related_name="+", blank=True, null=True
     )
 
-    csv_headers = [
-        "termination_a_type",
-        "termination_a_id",
-        "termination_b_type",
-        "termination_b_id",
-        "type",
-        "status",
-        "label",
-        "color",
-        "length",
-        "length_unit",
-    ]
-
     class Meta:
         ordering = [
             "termination_a_type",
@@ -134,9 +121,6 @@ class Cable(PrimaryModel, StatusModel):
         pk = self.pk or self._pk
         return self.label or f"#{pk}"
 
-    def get_absolute_url(self):
-        return reverse("dcim:cable", args=[self.pk])
-
     @classproperty  # https://github.com/PyCQA/pylint-django/issues/240
     def STATUS_CONNECTED(cls):  # pylint: disable=no-self-argument
         """Return a cached "connected" `Status` object for later reference."""
@@ -276,20 +260,6 @@ class Cable(PrimaryModel, StatusModel):
         # Update the private pk used in __str__ in case this is a new object (i.e. just got its pk)
         self._pk = self.pk
 
-    def to_csv(self):
-        return (
-            f"{self.termination_a_type.app_label}.{self.termination_a_type.model}",
-            self.termination_a_id,
-            f"{self.termination_b_type.app_label}.{self.termination_b_type.model}",
-            self.termination_b_id,
-            self.get_type_display(),
-            self.get_status_display(),
-            self.label,
-            self.color,
-            self.length,
-            self.length_unit,
-        )
-
     def get_compatible_types(self):
         """
         Return all termination types compatible with termination A.

nautobot/dcim/models/device_component_templates.py

@@ -75,6 +75,12 @@ class ComponentTemplateModel(BaseModel, ChangeLoggedModel, CustomFieldModel, Rel
 
         return super().to_objectchange(action, related_object=device_type, **kwargs)
 
+    def get_absolute_url(self, api=False):
+        # TODO: in the new UI, this should be able to link directly to the object, instead of the device-type.
+        if not api:
+            return self.device_type.get_absolute_url(api=api)
+        return super().get_absolute_url(api=api)
+
     def instantiate_model(self, model, device, **kwargs):
         """
         Helper method to self.instantiate().