nautobot 2.2.0b1__py3-none-any.whl → 2.2.1__py3-none-any.whl

nautobot/dcim/management/commands/migrate_location_contacts.py

@@ -0,0 +1,218 @@
+from textwrap import dedent, indent
+
+from django.core.exceptions import ValidationError
+from django.core.management.base import BaseCommand
+from django.db import transaction
+
+from nautobot.dcim.models import Location
+from nautobot.extras.filters import ContactFilterSet, TeamFilterSet
+from nautobot.extras.models import Contact, ContactAssociation, Role, Status, Team
+
+
+class Command(BaseCommand):
+    help = "Migrate Location contact fields to Contact and Team objects."
+    verbose_help = """
+    This command will present a series of prompts to guide you through migrating Locations that
+    have data in the `contact_name`, `contact_phone`, or `contact_email` fields which are not
+    already associated to a Contact or Team. This command will give you the option to create new
+    Contacts or Teams or, if a similar Contact or Team already exists, to link the Location to the
+    existing Contact or Team. Note that when assigning a Location to an existing Contact or Team
+    that has a blank `phone` or `email` field, the value from the Location will be copied to the
+    Contact/Team. After a Location has been associated to a Contact or Team, the `contact_name`,
+    `contact_phone`, and `contact_email` fields will be cleared from the Location.
+    """
+
+    def handle(self, *args, **kwargs):
+        status_role_err_msg = "No {0} found for the ContactAssociation content type. Please ensure {0} are created before running this command."
+        if not Status.objects.get_for_model(ContactAssociation).exists():
+            self.stdout.write(self.style.ERROR(status_role_err_msg.format("statuses")))
+            return
+        if not Role.objects.get_for_model(ContactAssociation).exists():
+            self.stdout.write(self.style.ERROR(status_role_err_msg.format("roles")))
+            return
+
+        self.stdout.write(self.style.NOTICE(" ".join(dedent(self.verbose_help).split("\n"))))
+
+        try:
+            with transaction.atomic():
+                try:
+                    self.migrate_location_contacts()
+                except KeyboardInterrupt:
+                    while True:
+                        rollback = input("\nRoll back changes? [y/n]: ").strip().lower()
+                        if rollback == "y":
+                            raise
+                        elif rollback == "n":
+                            break
+        except KeyboardInterrupt:
+            self.stdout.write(self.style.ERROR("\nOperation cancelled, all changes rolled back."))
+        except:
+            self.stdout.write(self.style.ERROR("\nOperation failed, all changes rolled back."))
+            raise
+
+    def migrate_location_contacts(self):
+        """Iterate through Locations with contact information and try to match to existing Contact or Team."""
+        locations_with_contact_data = (
+            Location.objects.without_tree_fields()
+            .exclude(
+                contact_name="",
+                contact_phone="",
+                contact_email="",
+            )
+            .filter(associated_contacts__isnull=True)
+        )
+
+        for location in locations_with_contact_data:
+            self.stdout.write(f"Finding existing Contacts or Teams for location {location.display}...")
+            selected_contact = None
+            similar_contacts = list(ContactFilterSet(data={"similar_to_location_data": [location]}).qs)
+            similar_teams = list(TeamFilterSet(data={"similar_to_location_data": [location]}).qs)
+            similar_contacts_and_teams = similar_contacts + similar_teams
+
+            if not similar_contacts_and_teams:
+                self.stdout.write(
+                    self.style.WARNING(f"No similar Contacts or Teams found for location {location.display}.")
+                )
+
+            else:
+                # Found similar contacts or teams, prompt user for action
+                self.stdout.write("")
+                self.stdout.write(self.style.WARNING(f"Found similar contacts/teams for location {location.display}:"))
+                self.stdout.write(f"    current contact name: {location.contact_name!r}")
+                self.stdout.write(f"    current contact phone: {location.contact_phone!r}")
+                self.stdout.write(f"    current contact email: {location.contact_email!r}")
+                self.stdout.write("")
+
+                # Output menu of choices of valid contacts/teams
+                for i, contact in enumerate(similar_contacts_and_teams, start=1):
+                    self.stdout.write(f"{self.style.WARNING(i)}: {contact._meta.model_name.title()}: {contact.name}")
+                    self.print_contact_fields(contact, rjust=len(str(i)) + len(contact._meta.model_name) + 2)
+                    self.stdout.write("")
+
+            self.stdout.write(self.style.WARNING("c") + ": Create a new Contact")
+            self.stdout.write(self.style.WARNING("t") + ": Create a new Team")
+            self.stdout.write(self.style.WARNING("s") + ": Skip this location")
+
+            # Retrieve desired contact/team from user input
+            while True:
+                choice = input("Select a choice from the list of items: ")
+                if choice == "s":
+                    self.stdout.write(f"Skipping location {location.display}")
+                    break
+                elif choice == "c":
+                    selected_contact = self.create_new_contact_from_location(location, model=Contact)
+                    break
+                elif choice == "t":
+                    selected_contact = self.create_new_contact_from_location(location, model=Team)
+                    break
+                elif choice.lower() == "q":
+                    raise KeyboardInterrupt
+                elif choice.isdigit() and 0 < int(choice) <= len(similar_contacts_and_teams):
+                    selected_contact = similar_contacts_and_teams[int(choice) - 1]
+                    break
+
+            if selected_contact is not None:
+                self.associate_contact_to_location(selected_contact, location)
+
+    def associate_contact_to_location(self, contact, location):
+        role, status = self.prompt_for_role_and_status()
+
+        # If email or phone fields are present in the location but not the contact, update the contact fields
+        updated_fields = {}
+        if location.contact_phone and not contact.phone:
+            contact.phone = location.contact_phone
+            updated_fields["phone"] = location.contact_phone
+        if location.contact_email and not contact.email:
+            contact.email = location.contact_email
+            updated_fields["email"] = location.contact_email
+        if updated_fields:
+            try:
+                contact.validated_save()
+                self.stdout.write(
+                    self.style.SUCCESS(
+                        f"Updated {contact._meta.model_name.title()} {contact.name} field(s): {updated_fields!r}"
+                    )
+                )
+            except ValidationError as e:
+                contact.refresh_from_db()
+                self.stdout.write(
+                    self.style.ERROR(f"Attempted to update {contact!r} field(s) but failed: {updated_fields!r}: {e}")
+                )
+
+        try:
+            contact_association = ContactAssociation(
+                contact=contact if isinstance(contact, Contact) else None,
+                team=contact if isinstance(contact, Team) else None,
+                associated_object=location,
+                role=role,
+                status=status,
+            )
+            contact_association.validated_save()
+            location.contact_name = ""
+            location.contact_email = ""
+            location.contact_phone = ""
+            location.validated_save()
+        except Exception as e:
+            self.stdout.write(self.style.ERROR(f"Failed to create association: {e}"))
+        else:
+            self.stdout.write(self.style.SUCCESS(f"Associated {contact!r} to location {location.display}"))
+
+    def prompt_for_role_and_status(self):
+        # Prompt for role
+        self.stdout.write("\nValid roles for this association:")
+        valid_roles = list(Role.objects.get_for_model(ContactAssociation))
+        for i, role in enumerate(valid_roles, start=1):
+            self.stdout.write(self.style.WARNING(f"{i}") + f": {role}")
+        while True:
+            selected_role = input("Select a role for this association: ")
+            if selected_role.isdigit() and 0 < int(selected_role) <= len(valid_roles):
+                role = valid_roles[int(selected_role) - 1]
+                break
+
+        # Prompt for status
+        self.stdout.write("\nValid statuses for this association:")
+        valid_statuses = list(Status.objects.get_for_model(ContactAssociation))
+        for i, status in enumerate(valid_statuses, start=1):
+            self.stdout.write(self.style.WARNING(f"{i}") + f": {status}")
+        while True:
+            selected_status = input("Select a status for this association: ")
+            if selected_status.isdigit() and 0 < int(selected_status) <= len(valid_statuses):
+                status = valid_statuses[int(selected_status) - 1]
+                break
+
+        return role, status
+
+    def create_new_contact_from_location(self, location, model):
+        """Create a new Contact or Team from the location's contact data."""
+
+        self.stdout.write(f"Creating new {model._meta.model_name.title()} for location {location.display}...")
+        name = location.contact_name
+        phone = location.contact_phone
+        email = location.contact_email
+        self.stdout.write(f"    contact name: {name!r}")
+        self.stdout.write(f"    contact phone: {phone!r}")
+        self.stdout.write(f"    contact email: {email!r}")
+
+        while not name:
+            name = input(f"Name is required. Enter a name for the new {model._meta.model_name.title()}: ")
+
+        try:
+            contact = model(
+                name=name,
+                phone=phone,
+                email=email,
+            )
+            contact.validated_save()
+            return contact
+        except Exception as e:
+            self.stdout.write(self.style.ERROR(f"Failed to create {model._meta.model_name.title()}: {e}"))
+            return None
+
+    def print_contact_fields(self, contact, rjust=0):
+        for field_name in ["phone", "email"]:
+            if getattr(contact, field_name):
+                self.stdout.write(
+                    field_name.title().rjust(rjust)
+                    + ": "
+                    + indent(getattr(contact, field_name), " " * (rjust + 2)).lstrip()
+                )

nautobot/dcim/migrations/0057_controller_models.py

@@ -1,4 +1,4 @@
-# Generated by Django 3.2.24 on 2024-03-08 10:18
+# Generated by Django 3.2.24 on 2024-03-22 15:36
 
 import uuid
 
@@ -9,16 +9,13 @@ import django.db.models.deletion
 import nautobot.core.models.fields
 import nautobot.core.models.tree_queries
 import nautobot.extras.models.mixins
-import nautobot.extras.models.models
 import nautobot.extras.models.roles
 import nautobot.extras.models.statuses
-import nautobot.extras.utils
 
 
 class Migration(migrations.Migration):
     dependencies = [
         ("extras", "0106_populate_default_statuses_and_roles_for_contact_associations"),
-        ("contenttypes", "0002_remove_content_type_name"),
         ("tenancy", "0009_update_all_charfields_max_length_to_255"),
         ("dcim", "0056_update_all_charfields_max_length_to_255"),
     ]
@@ -39,29 +36,24 @@ class Migration(migrations.Migration):
                     "_custom_field_data",
                     models.JSONField(blank=True, default=dict, encoder=django.core.serializers.json.DjangoJSONEncoder),
                 ),
-                (
-                    "local_config_context_data",
-                    models.JSONField(blank=True, encoder=django.core.serializers.json.DjangoJSONEncoder, null=True),
-                ),
-                ("local_config_context_data_owner_object_id", models.UUIDField(blank=True, default=None, null=True)),
                 ("name", models.CharField(max_length=255, unique=True)),
                 ("description", models.CharField(blank=True, max_length=255)),
                 (
-                    "deployed_controller_device",
+                    "controller_device",
                     models.ForeignKey(
                         blank=True,
                         null=True,
-                        on_delete=django.db.models.deletion.SET_NULL,
+                        on_delete=django.db.models.deletion.PROTECT,
                         related_name="controllers",
                         to="dcim.device",
                     ),
                 ),
                 (
-                    "deployed_controller_group",
+                    "controller_device_redundancy_group",
                     models.ForeignKey(
                         blank=True,
                         null=True,
-                        on_delete=django.db.models.deletion.SET_NULL,
+                        on_delete=django.db.models.deletion.PROTECT,
                         related_name="controllers",
                         to="dcim.deviceredundancygroup",
                     ),
@@ -71,33 +63,11 @@ class Migration(migrations.Migration):
                     models.ForeignKey(
                         blank=True,
                         null=True,
-                        on_delete=django.db.models.deletion.SET_NULL,
+                        on_delete=django.db.models.deletion.PROTECT,
                         related_name="controllers",
                         to="extras.externalintegration",
                     ),
                 ),
-                (
-                    "local_config_context_data_owner_content_type",
-                    nautobot.core.models.fields.ForeignKeyWithAutoRelatedName(
-                        blank=True,
-                        default=None,
-                        limit_choices_to=nautobot.extras.utils.FeatureQuery("config_context_owners"),
-                        null=True,
-                        on_delete=django.db.models.deletion.CASCADE,
-                        related_name="controllers",
-                        to="contenttypes.contenttype",
-                    ),
-                ),
-                (
-                    "local_config_context_schema",
-                    nautobot.core.models.fields.ForeignKeyWithAutoRelatedName(
-                        blank=True,
-                        null=True,
-                        on_delete=django.db.models.deletion.SET_NULL,
-                        related_name="controllers",
-                        to="extras.configcontextschema",
-                    ),
-                ),
                 (
                     "location",
                     models.ForeignKey(
@@ -149,11 +119,10 @@ class Migration(migrations.Migration):
                 models.Model,
                 nautobot.extras.models.mixins.DynamicGroupMixin,
                 nautobot.extras.models.mixins.NotesMixin,
-                nautobot.extras.models.models.ConfigContextSchemaValidationMixin,
             ),
         ),
         migrations.CreateModel(
-            name="ControllerDeviceGroup",
+            name="ControllerManagedDeviceGroup",
             fields=[
                 (
                     "id",
@@ -167,43 +136,16 @@ class Migration(migrations.Migration):
                     "_custom_field_data",
                     models.JSONField(blank=True, default=dict, encoder=django.core.serializers.json.DjangoJSONEncoder),
                 ),
-                (
-                    "local_config_context_data",
-                    models.JSONField(blank=True, encoder=django.core.serializers.json.DjangoJSONEncoder, null=True),
-                ),
-                ("local_config_context_data_owner_object_id", models.UUIDField(blank=True, default=None, null=True)),
                 ("name", models.CharField(max_length=255, unique=True)),
                 ("weight", models.PositiveIntegerField(default=1000)),
                 (
                     "controller",
                     models.ForeignKey(
                         on_delete=django.db.models.deletion.CASCADE,
-                        related_name="controller_device_groups",
+                        related_name="controller_managed_device_groups",
                         to="dcim.controller",
                     ),
                 ),
-                (
-                    "local_config_context_data_owner_content_type",
-                    nautobot.core.models.fields.ForeignKeyWithAutoRelatedName(
-                        blank=True,
-                        default=None,
-                        limit_choices_to=nautobot.extras.utils.FeatureQuery("config_context_owners"),
-                        null=True,
-                        on_delete=django.db.models.deletion.CASCADE,
-                        related_name="controller_device_groups",
-                        to="contenttypes.contenttype",
-                    ),
-                ),
-                (
-                    "local_config_context_schema",
-                    nautobot.core.models.fields.ForeignKeyWithAutoRelatedName(
-                        blank=True,
-                        null=True,
-                        on_delete=django.db.models.deletion.SET_NULL,
-                        related_name="controller_device_groups",
-                        to="extras.configcontextschema",
-                    ),
-                ),
                 (
                     "parent",
                     models.ForeignKey(
@@ -211,7 +153,7 @@ class Migration(migrations.Migration):
                         null=True,
                         on_delete=django.db.models.deletion.CASCADE,
                         related_name="children",
-                        to="dcim.controllerdevicegroup",
+                        to="dcim.controllermanageddevicegroup",
                     ),
                 ),
                 ("tags", nautobot.core.models.fields.TagsField(through="extras.TaggedItem", to="extras.Tag")),
@@ -223,7 +165,6 @@ class Migration(migrations.Migration):
                 models.Model,
                 nautobot.extras.models.mixins.DynamicGroupMixin,
                 nautobot.extras.models.mixins.NotesMixin,
-                nautobot.extras.models.models.ConfigContextSchemaValidationMixin,
             ),
             managers=[
                 ("objects", nautobot.core.models.tree_queries.TreeManager()),
@@ -231,13 +172,13 @@ class Migration(migrations.Migration):
         ),
         migrations.AddField(
             model_name="device",
-            name="controller_device_group",
+            name="controller_managed_device_group",
             field=models.ForeignKey(
                 blank=True,
                 null=True,
                 on_delete=django.db.models.deletion.SET_NULL,
                 related_name="devices",
-                to="dcim.controllerdevicegroup",
+                to="dcim.controllermanageddevicegroup",
             ),
         ),
     ]

nautobot/dcim/models/__init__.py

@@ -27,7 +27,7 @@ from .device_components import (
 )
 from .devices import (
     Controller,
-    ControllerDeviceGroup,
+    ControllerManagedDeviceGroup,
     Device,
     DeviceFamily,
     DeviceRedundancyGroup,
@@ -53,7 +53,7 @@ __all__ = (
     "ConsoleServerPort",
     "ConsoleServerPortTemplate",
     "Controller",
-    "ControllerDeviceGroup",
+    "ControllerManagedDeviceGroup",
     "Device",
     "DeviceBay",
     "DeviceBayTemplate",

nautobot/dcim/models/devices.py

@@ -43,7 +43,7 @@ from .device_components import (
 
 __all__ = (
     "Controller",
-    "ControllerDeviceGroup",
+    "ControllerManagedDeviceGroup",
     "Device",
     "DeviceRedundancyGroup",
     "DeviceType",
@@ -592,8 +592,8 @@ class Device(PrimaryModel, ConfigContextModel):
         verbose_name="Software Image Files",
         help_text="Override the software image files associated with the software version for this device",
     )
-    controller_device_group = models.ForeignKey(
-        to="dcim.ControllerDeviceGroup",
+    controller_managed_device_group = models.ForeignKey(
+        to="dcim.ControllerManagedDeviceGroup",
         on_delete=models.SET_NULL,
         related_name="devices",
         blank=True,
@@ -1247,7 +1247,7 @@ class SoftwareVersion(PrimaryModel):
     "statuses",
     "webhooks",
 )
-class Controller(PrimaryModel, ConfigContextModel):
+class Controller(PrimaryModel):
     """Represents an entity that manages or controls one or more devices, acting as a central point of control.
 
     A Controller can be deployed to a single device or a group of devices represented by a DeviceRedundancyGroup.
@@ -1278,21 +1278,21 @@ class Controller(PrimaryModel, ConfigContextModel):
     )
     external_integration = models.ForeignKey(
         to="extras.ExternalIntegration",
-        on_delete=models.SET_NULL,
+        on_delete=models.PROTECT,
         related_name="controllers",
         blank=True,
         null=True,
     )
-    deployed_controller_device = models.ForeignKey(
+    controller_device = models.ForeignKey(
         to="dcim.Device",
-        on_delete=models.SET_NULL,
+        on_delete=models.PROTECT,
         related_name="controllers",
         blank=True,
         null=True,
     )
-    deployed_controller_group = models.ForeignKey(
+    controller_device_redundancy_group = models.ForeignKey(
         to="dcim.DeviceRedundancyGroup",
-        on_delete=models.SET_NULL,
+        on_delete=models.PROTECT,
         related_name="controllers",
         blank=True,
         null=True,
@@ -1307,12 +1307,10 @@ class Controller(PrimaryModel, ConfigContextModel):
     def clean(self):
         super().clean()
 
-        if self.deployed_controller_device and self.deployed_controller_group:
+        if self.controller_device and self.controller_device_redundancy_group:
             raise ValidationError(
                 {
-                    "deployed_controller_device": (
-                        "Cannot assign both a device and a device redundancy group to a controller."
-                    ),
+                    "controller_device": ("Cannot assign both a device and a device redundancy group to a controller."),
                 },
             )
 
@@ -1331,7 +1329,7 @@ class Controller(PrimaryModel, ConfigContextModel):
     "graphql",
     "webhooks",
 )
-class ControllerDeviceGroup(TreeModel, PrimaryModel, ConfigContextModel):
+class ControllerManagedDeviceGroup(TreeModel, PrimaryModel):
     """Represents a mapping of controlled devices to a specific controller.
 
     This model allows for the organization of controlled devices into hierarchical groups for structured representation.
@@ -1349,7 +1347,7 @@ class ControllerDeviceGroup(TreeModel, PrimaryModel, ConfigContextModel):
     controller = models.ForeignKey(
         to="dcim.Controller",
         on_delete=models.CASCADE,
-        related_name="controller_device_groups",
+        related_name="controller_managed_device_groups",
         blank=False,
         null=False,
         help_text="Controller that manages the devices in this group",
@@ -1373,7 +1371,7 @@ class ControllerDeviceGroup(TreeModel, PrimaryModel, ConfigContextModel):
         if self.controller == self._original_controller and self.parent == self._original_parent:
             return
 
-        if self.parent and self.controller and self.parent.controller and self.controller != self.parent.controller:
+        if self.parent and self.controller and self.controller != self.parent.controller:
             raise ValidationError(
                 {"controller": "Controller device group must have the same controller as the parent group."}
             )

nautobot/dcim/models/racks.py

@@ -1,5 +1,3 @@
-from collections import OrderedDict
-
 from django.conf import settings
 from django.contrib.contenttypes.fields import GenericRelation
 from django.contrib.contenttypes.models import ContentType
@@ -275,7 +273,7 @@ class Rack(PrimaryModel):
             contains a height attribute for the device
         """
 
-        elevation = OrderedDict()
+        elevation = {}
         for u in self.units:
             elevation[u] = {
                 "id": u,

nautobot/dcim/navigation.py

@@ -18,33 +18,33 @@ menu_items = (
                 weight=150,
                 items=(
                     NavMenuItem(
-                        link="dcim:locationtype_list",
-                        name="Location Types",
+                        link="dcim:location_list",
+                        name="Locations",
                         weight=100,
                         permissions=[
-                            "dcim.view_locationtype",
+                            "dcim.view_location",
                         ],
                         buttons=(
                             NavMenuAddButton(
-                                link="dcim:locationtype_add",
+                                link="dcim:location_add",
                                 permissions=[
-                                    "dcim.add_locationtype",
+                                    "dcim.add_location",
                                 ],
                             ),
                         ),
                     ),
                     NavMenuItem(
-                        link="dcim:location_list",
-                        name="Locations",
+                        link="dcim:locationtype_list",
+                        name="Location Types",
                         weight=200,
                         permissions=[
-                            "dcim.view_location",
+                            "dcim.view_locationtype",
                         ],
                         buttons=(
                             NavMenuAddButton(
-                                link="dcim:location_add",
+                                link="dcim:locationtype_add",
                                 permissions=[
-                                    "dcim.add_location",
+                                    "dcim.add_locationtype",
                                 ],
                             ),
                         ),
@@ -211,33 +211,33 @@ menu_items = (
                         ),
                     ),
                     NavMenuItem(
-                        link="dcim:manufacturer_list",
-                        name="Manufacturers",
+                        link="dcim:devicefamily_list",
+                        name="Device Families",
                         weight=200,
                         permissions=[
-                            "dcim.view_manufacturer",
+                            "dcim.view_devicefamily",
                         ],
                         buttons=(
                             NavMenuAddButton(
-                                link="dcim:manufacturer_add",
+                                link="dcim:devicefamily_add",
                                 permissions=[
-                                    "dcim.add_manufacturer",
+                                    "dcim.add_devicefamily",
                                 ],
                             ),
                         ),
                     ),
                     NavMenuItem(
-                        link="dcim:devicefamily_list",
-                        name="Device Families",
+                        link="dcim:manufacturer_list",
+                        name="Manufacturers",
                         weight=300,
                         permissions=[
-                            "dcim.view_devicefamily",
+                            "dcim.view_manufacturer",
                         ],
                         buttons=(
                             NavMenuAddButton(
-                                link="dcim:devicefamily_add",
+                                link="dcim:manufacturer_add",
                                 permissions=[
-                                    "dcim.add_devicefamily",
+                                    "dcim.add_manufacturer",
                                 ],
                             ),
                         ),
@@ -319,20 +319,12 @@ menu_items = (
                         ),
                     ),
                     NavMenuItem(
-                        link="dcim:controllerdevicegroup_list",
-                        name="Controller Device Groups",
+                        link="dcim:controllermanageddevicegroup_list",
+                        name="Managed Device Groups",
                         weight=200,
                         permissions=[
-                            "dcim.view_controllerdevicegroup",
+                            "dcim.view_controllermanageddevicegroup",
                         ],
-                        buttons=(
-                            NavMenuAddButton(
-                                link="dcim:controllerdevicegroup_add",
-                                permissions=[
-                                    "dcim.add_controllerdevicegroup",
-                                ],
-                            ),
-                        ),
                     ),
                 ),
             ),

nautobot/dcim/signals.py

@@ -11,7 +11,7 @@ from nautobot.core.signals import disable_for_loaddata
 from .models import (
     Cable,
     CablePath,
-    ControllerDeviceGroup,
+    ControllerManagedDeviceGroup,
     Device,
     DeviceRedundancyGroup,
     Interface,
@@ -290,17 +290,17 @@ def prevent_adding_tagged_vlans_with_incorrect_mode_or_site(sender, instance, ac
 
 
 #
-# ControllerDeviceGroup
+# ControllerManagedDeviceGroup
 #
 
 
-@receiver(post_save, sender=ControllerDeviceGroup)
-def handle_controller_device_group_controller_change(instance, raw=False, **_):
-    """Update descendants when the top level `ControllerDeviceGroup.controller` changes."""
+@receiver(post_save, sender=ControllerManagedDeviceGroup)
+def handle_controller_managed_device_group_controller_change(instance, raw=False, **_):
+    """Update descendants when the top level `ControllerManagedDeviceGroup.controller` changes."""
     if instance.parent or instance._original_controller == instance.controller:
         return
 
-    logger = logging.getLogger(__name__ + ".ControllerDeviceGroup")
+    logger = logging.getLogger(__name__ + ".ControllerManagedDeviceGroup")
 
     if raw:
         logger.debug("Skipping controller update for imported controller device group %s", instance)