netcmd: silo member add and remove should not set assigned_silo

The Windows tools don't do this either

Signed-off-by: Rob van der Linde <rob@catalyst.net.nz>
Reviewed-by: Douglas Bagnall <douglas.bagnall@catalyst.net.nz>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>

diff --git a/python/samba/netcmd/domain/auth/silo_member.py b/python/samba/netcmd/domain/auth/silo_member.py
index 638ad59b8ca3c91825232cdc0e1a9cf9832745ad..4094ba768406d94c950dfd7ee763e8c994744b28 100644 (file)
--- a/python/samba/netcmd/domain/auth/silo_member.py
+++ b/python/samba/netcmd/domain/auth/silo_member.py
@@ -69,13 +69,9 @@ class cmd_domain_auth_silo_member_add(Command):
         if user is None:
             raise CommandError(f"User '{member}' not found.")
 
-        # Set the assigned silo.
-        user.assigned_silo = silo.dn
-
-        # Add member and save user.
+        # Add member.
         try:
             silo.add_member(ldb, user)
-            user.save(ldb)
         except ModelError as e:
             raise CommandError(e)
 
@@ -172,13 +168,9 @@ class cmd_domain_auth_silo_member_remove(Command):
         if user is None:
             raise CommandError(f"User '{member}' not found.")
 
-        # Unset the assigned silo.
-        user.assigned_silo = None
-
-        # Remove member and save user.
+        # Remove member.
         try:
             silo.remove_member(ldb, user)
-            user.save(ldb)
         except ModelError as e:
             raise CommandError(e)