selftest: Remove duplicate setup of "spn/upn namespaces" in the customdc testenv
authorAndrew Bartlett <abartlet@samba.org>
Thu, 11 Apr 2024 02:33:16 +0000 (14:33 +1200)
committerAndrew Bartlett <abartlet@samba.org>
Tue, 16 Apr 2024 03:58:31 +0000 (03:58 +0000)
The call to $self->setup_namespaces() was allways in error, as the design
is to have the in the state that it was backed up in, but before commit
08be28241b808845c4b51a4c47765a9416ca3aa7 the error return was not
checked and so this was harmless.

The customdc environment is not tested in selftest currently, as
it is intended to be used for manual testing of domains from backup
files not as an automatically constructed environment.

This makes:
 BACKUP_FILE=samba-backup-2024-04-11T14-10-20.437096.tar.bz2 SELFTEST_TESTENV=customdc make testenv
work again.

Signed-off-by: Andrew Bartlett <abartlet@samba.org>
Reviewed-by: Jo Sutton <josutton@catalyst.net.nz>
selftest/target/Samba4.pm

index f2b84b4f9b7b321b9d27f25db38778deff87f468..48a78b2c8d2950f6931228050625b32b14c312ad 100755 (executable)
@@ -3725,10 +3725,6 @@ sub setup_customdc
        my $upn_array = ["$env->{REALM}.upn"];
        my $spn_array = ["$env->{REALM}.spn"];
 
-       if ($self->setup_namespaces($env, $upn_array, $spn_array) != 0) {
-               return undef;
-       }
-
        return $env;
 }