#104 ipa-{server,replica}-install: add --setup-kra
Opened 2 years ago by fcami. Modified 2 years ago
https://github.com/fcami/fedora-qa_os-autoinst-distri-fedora freeipa-setup-kra  into  master

ipa-replica-install: add --setup-kra
François Cami • 2 years ago  
tests/realmd_join_sssd.pm
file modified
+1 -1
@@ -54,7 +54,7 @@

          assert_script_run "systemctl restart firewalld.service";

  

          # deploy as a replica

-         my $args = "--setup-dns --setup-ca --allow-zone-overlap -U --principal admin --admin-password monkeys123";

+         my $args = "--setup-dns --setup-ca --setup-kra --allow-zone-overlap -U --principal admin --admin-password monkeys123";

          for my $fwd (@forwards) {

              $args .= " --forwarder=$fwd";

          }

no initial comment

Some installer bugs were recently discovered with fs.protected_regular=1
Add --setup-kra to both master and replica to exercise more codepaths.
Essentially test for https://bugzilla.redhat.com/show_bug.cgi?id=1698384

Signed-off-by: François Cami fcami@redhat.com

hmm, so, the problem I see here is that the functionality covered by the existing tests - especially the regular role_deploy_domain_controller - is what's considered release-blocking; if the test fails we've probably got a release-blocking bug.

This functionality is beyond what we block the release on, I think...so if we wind up with the test failing because of this, we don't actually know if the functionality we actually block the release on is working or not.

Perhaps we could do this only for the replica tests, since they already go a bit beyond what's strictly in the release criteria? Way to do that would be to have it conditional on an openQA variable, and set that variable in the templates for the replica test suites but not the others...

Doing it replica-only should be fine.
Another possibly is to run ipa-kra-install post install in another task on replica or master.
For simplicity purposes I think having --setup-kra only in the replica install directly is fine.

rebased onto 084303f

2 years ago

@adamwill Please wait until the fix for https://bugzilla.redhat.com/show_bug.cgi?id=1698384 is present in a f30 build.

OK. It will need to be pushed stable, then, otherwise the tests will fail for all other updates.

Metadata