Remove role-tracking feature no longer used

Author: AlanCoding

ansible_base/rbac/models/role.py

@@ -297,11 +297,6 @@ def give_or_remove_permission(self, actor, content_object, giving=True, sync_act
 
         update_after_assignment(update_teams, to_update)
 
-        if not sync_action and self.name in permission_registry._trackers:
-            tracker = permission_registry._trackers[self.name]
-            with tracker.sync_active():
-                tracker.sync_relationship(actor, content_object, giving=giving)
-
         return assignment
 
     @classmethod

ansible_base/rbac/permission_registry.py

@@ -32,8 +32,6 @@ def __init__(self):
         self._parent_fields = dict()
         self._managed_roles = dict()  # code-defined role definitions, managed=True
         self.apps_ready = False
-        self._tracked_relationships = set()
-        self._trackers = dict()
 
     def register(self, *args: Type[Model], parent_field_name: Optional[str] = 'organization'):
         if self.apps_ready:
@@ -50,9 +48,6 @@ def register(self, *args: Type[Model], parent_field_name: Optional[str] = 'organ
             else:
                 logger.debug(f'Model {cls._meta.model_name} registered to permission registry more than once')
 
-    def track_relationship(self, cls, relationship, role_name):
-        self._tracked_relationships.add((cls, relationship, role_name))
-
     def get_parent_model(self, model) -> Optional[type]:
         model = self._name_to_model[model._meta.model_name]
         parent_field_name = self.get_parent_fd_name(model)
@@ -158,14 +153,6 @@ def call_when_apps_ready(self, apps, app_config) -> None:
             triggers.connect_rbac_signals(cls)
             connect_rbac_methods(cls)
 
-        for cls, relationship, role_name in self._tracked_relationships:
-            if role_name in self._trackers:
-                tracker = self._trackers[role_name]
-            else:
-                tracker = triggers.TrackedRelationship(cls, role_name)
-                self._trackers[role_name] = tracker
-            tracker.initialize(relationship)
-
         self.register_managed_role_constructors()
 
     @property

ansible_base/rbac/triggers.py

@@ -1,5 +1,4 @@
 import logging
-from contextlib import contextmanager
 from typing import Optional, Union
 from uuid import UUID
 
@@ -285,90 +284,6 @@ def post_migration_rbac_setup(sender, *args, **kwargs):
     compute_object_role_permissions()
 
 
-class TrackedRelationship:
-    def __init__(self, cls, role_name):
-        self.cls = cls
-        self.role_name = role_name
-        self.user_relationship = None
-        self.team_relationship = None
-        self._active_sync_flag = False
-
-    def initialize(self, relationship):
-        manager = getattr(self.cls, relationship)
-        related_model_name = manager.field.related_model._meta.model_name
-        if related_model_name == permission_registry.team_model._meta.model_name:
-            self.team_relationship = relationship
-            m2m_changed.connect(self.sync_team_to_role, sender=manager.through)
-        elif related_model_name == permission_registry.user_model._meta.model_name:
-            self.user_relationship = relationship
-            m2m_changed.connect(self.sync_user_to_role, sender=manager.through)
-        else:
-            raise RuntimeError(f'Can only register user or team relationships, obtained {related_model_name}')
-
-    @contextmanager
-    def sync_active(self):
-        try:
-            self._active_sync_flag = True
-            yield
-        finally:
-            self._active_sync_flag = False
-
-    def sync_relationship(self, actor, content_object, giving=True):
-        # Exit if role does not apply for the intended model type, for example
-        # if user is given "team-member" role to organization, do not add user to the team members
-        if content_object._meta.model_name != self.cls._meta.model_name:
-            return
-
-        if actor._meta.model_name == permission_registry.team_model._meta.model_name:
-            if self.team_relationship is None:
-                return
-            manager = getattr(content_object, self.team_relationship)
-        elif actor._meta.model_name == permission_registry.user_model._meta.model_name:
-            if self.user_relationship is None:
-                return
-            manager = getattr(content_object, self.user_relationship)
-
-        if giving:
-            manager.add(actor)
-        else:
-            manager.remove(actor)
-
-    def _sync_actor_to_role(self, actor_model: type, instance: Model, action: str, pk_set: Optional[set[int]]):
-        if self._active_sync_flag:
-            return
-        if action.startswith('pre_'):
-            return
-        rd = RoleDefinition.objects.get(name=self.role_name)
-
-        if action in ('post_add', 'post_remove'):
-            actor_set = pk_set
-        elif action == 'post_clear':
-            ct = permission_registry.content_type_model.objects.get_for_model(instance)
-            role = ObjectRole.objects.get(object_id=instance.pk, content_type=ct, role_definition=rd)
-            if actor_model._meta.model_name == 'team':
-                actor_set = set(role.teams.values_list('id', flat=True))
-            else:
-                actor_set = set(role.users.values_list('id', flat=True))
-
-        giving = bool(action == 'post_add')
-        for actor in actor_model.objects.filter(pk__in=actor_set):
-            rd.give_or_remove_permission(actor, instance, giving=giving, sync_action=True)
-
-    def sync_team_to_role(self, instance: Model, action: str, model: type, pk_set: Optional[set[int]], reverse: bool, **kwargs):
-        if not reverse:
-            self._sync_actor_to_role(permission_registry.team_model, instance, action, pk_set)
-        else:
-            for pk in pk_set:
-                self._sync_actor_to_role(permission_registry.team_model, model(pk=pk), action, {instance.pk})
-
-    def sync_user_to_role(self, instance: Model, action: str, model: type, pk_set: Optional[set[int]], reverse: bool, **kwargs):
-        if not reverse:
-            self._sync_actor_to_role(permission_registry.user_model, instance, action, pk_set)
-        else:
-            for pk in pk_set:
-                self._sync_actor_to_role(permission_registry.user_model, model(pk=pk), action, {instance.pk})
-
-
 def connect_rbac_signals(cls):
     if cls._meta.model_name == permission_registry.team_model._meta.model_name:
         pre_delete.connect(team_pre_delete, sender=cls, dispatch_uid='stash-team-roles-before-delete')

docs/apps/rbac/for_app_developers.md

@@ -344,27 +344,6 @@ If you create (in code) a role definition that sets `managed` to True, then thes
 rules will be disregarded for that particular role definition. Managed role
 definitions can not be created through the API, but can be created in code like migration scripts.
 
-### Tracked Relationships
-
-Let's say that you are introducing RBAC, and you have already set up your API
-with some relationship, like members of a team, and parents of a team
-(to get nested teams).
-This sub-feature will use signals to do bidirectional syncing of memberships of
-that relationship with memberships of their corresponding role.
-
-```
-permission_registry.track_relationship(Team, 'users', 'Team Member')
-permission_registry.track_relationship(Team, 'team_parents', 'Team Member')
-```
-
-This only works with our 2 "actor" types of users and teams.
-Adding these lines will synchronize users and teams of team-object-roles with the "team-member"
-role definition (by name) to the `Team.users`
-and `Team.tracked_parents` ManyToMany relationships, respectively.
-So if you have a team object, `team.users.add(user)` will also give that
-user _member permission_ to that team, where those permissions are defined by the
-role definition with the name "team-member".
-
 
 ### Role assignment callback
 

test_app/migrations/0001_initial.py

@@ -55,8 +55,6 @@ class Migration(migrations.Migration):
                 ('description', models.TextField(blank=True, default='', help_text='The organization description.')),
                 ('created_by', models.ForeignKey(default=None, editable=False, help_text='The user who created this resource', null=True, on_delete=django.db.models.deletion.DO_NOTHING, related_name='%(app_label)s_%(class)s_created+', to=settings.AUTH_USER_MODEL)),
                 ('modified_by', models.ForeignKey(default=None, editable=False, help_text='The user who last modified this resource', null=True, on_delete=django.db.models.deletion.DO_NOTHING, related_name='%(app_label)s_%(class)s_modified+', to=settings.AUTH_USER_MODEL)),
-                ('admins', models.ManyToManyField(blank=True, help_text='The list of admins for this organization', related_name='admin_of_organizations', to=settings.AUTH_USER_MODEL)),
-                ('users', models.ManyToManyField(blank=True, help_text='The list of users on this organization', related_name='member_of_organizations', to=settings.AUTH_USER_MODEL))
             ],
             options={'ordering': ['id'], 'permissions': [('member_organization', 'User is member of this organization')]},
         ),
@@ -85,7 +83,6 @@ class Migration(migrations.Migration):
                 ('created_by', models.ForeignKey(default=None, editable=False, help_text='The user who created this resource', null=True, on_delete=django.db.models.deletion.DO_NOTHING, related_name='%(app_label)s_%(class)s_created+', to=settings.AUTH_USER_MODEL)),
                 ('modified_by', models.ForeignKey(default=None, editable=False, help_text='The user who last modified this resource', null=True, on_delete=django.db.models.deletion.DO_NOTHING, related_name='%(app_label)s_%(class)s_modified+', to=settings.AUTH_USER_MODEL)),
                 ('organization', models.ForeignKey(help_text='The organization of this team.', on_delete=django.db.models.deletion.CASCADE, related_name='teams', to=settings.ANSIBLE_BASE_ORGANIZATION_MODEL)),
-                ('team_parents', models.ManyToManyField(blank=True, related_name='team_children', to=settings.ANSIBLE_BASE_TEAM_MODEL)),
             ],
             options={
                 'ordering': ('organization__name', 'name'),

test_app/migrations/0006_team_admins_team_users.py

@@ -1,7 +1,6 @@
 # Generated by Django 4.2.8 on 2024-03-18 12:43
 
-from django.conf import settings
-from django.db import migrations, models
+from django.db import migrations
 
 
 class Migration(migrations.Migration):
@@ -11,14 +10,5 @@ class Migration(migrations.Migration):
     ]
 
     operations = [
-        migrations.AddField(
-            model_name='team',
-            name='admins',
-            field=models.ManyToManyField(blank=True, help_text='The list of admins for this team', related_name='teams_administered', to=settings.AUTH_USER_MODEL),
-        ),
-        migrations.AddField(
-            model_name='team',
-            name='users',
-            field=models.ManyToManyField(blank=True, help_text='The list of users on this team', related_name='teams', to=settings.AUTH_USER_MODEL),
-        ),
+        # Removed team admins and user relationsips
     ]

test_app/models.py

@@ -30,20 +30,6 @@ class Meta:
 
     resource = AnsibleResourceField(primary_key_field="id")
 
-    users = models.ManyToManyField(
-        settings.AUTH_USER_MODEL,
-        related_name='member_of_organizations',
-        blank=True,
-        help_text="The list of users on this organization",
-    )
-
-    admins = models.ManyToManyField(
-        settings.AUTH_USER_MODEL,
-        related_name='admin_of_organizations',
-        blank=True,
-        help_text="The list of admins for this organization",
-    )
-
     extra_field = models.CharField(max_length=100, null=True)
 
 
@@ -73,7 +59,6 @@ class ManagedUser(User):
 
 class Team(AbstractTeam):
     resource = AnsibleResourceField(primary_key_field="id")
-    team_parents = models.ManyToManyField('Team', related_name='team_children', blank=True)
 
     encryptioner = models.ForeignKey('test_app.EncryptionModel', on_delete=models.SET_NULL, null=True)
 
@@ -89,20 +74,6 @@ class Meta:
         ordering = ('organization__name', 'name')
         permissions = [('member_team', 'Has all roles assigned to this team')]
 
-    users = models.ManyToManyField(
-        User,
-        related_name='teams',
-        blank=True,
-        help_text="The list of users on this team",
-    )
-
-    admins = models.ManyToManyField(
-        User,
-        related_name='teams_administered',
-        blank=True,
-        help_text="The list of admins for this team",
-    )
-
 
 class ResourceMigrationTestModel(models.Model):
     name = models.CharField(max_length=255)
@@ -383,17 +354,6 @@ class Meta:
 permission_registry.register(ExtraExtraUUIDModel, parent_field_name='extra_uuid')
 
 
-# NOTE(cutwater): Using hard coded role names instead of ones defined in ReconcileUser class,
-#   to avoid circular dependency between models and claims modules. This is a temporary workarond,
-#   since we plan to drop support of tracked relationships in future.
-permission_registry.track_relationship(Team, 'users', 'Team Member')
-permission_registry.track_relationship(Team, 'admins', 'Team Admin')
-permission_registry.track_relationship(Team, 'team_parents', 'Team Member')
-
-permission_registry.track_relationship(Organization, 'users', 'Organization Member')
-permission_registry.track_relationship(Organization, 'admins', 'Organization Admin')
-
-
 class MultipleFieldsModel(NamedCommonModel):
     class Meta:
         app_label = "test_app"

test_app/tests/conftest.py

@@ -667,15 +667,15 @@ def org_member_rd():
 
 @pytest.fixture
 def member_rd():
-    "Member role for a team, place in root conftest because it is needed for the team users tracked relationship"
+    "Member role for a team"
     RoleDefinition.objects.managed.clear()
     yield RoleDefinition.objects.managed.team_member
     RoleDefinition.objects.managed.clear()
 
 
 @pytest.fixture
 def admin_rd():
-    "Member role for a team, place in root conftest because it is needed for the team users tracked relationship"
+    "Admin role for a team"
     RoleDefinition.objects.managed.clear()
     yield RoleDefinition.objects.managed.team_admin
     RoleDefinition.objects.managed.clear()