[Freeipa-devel] [freeipa PR#4560][opened] [ipa-4-6] ipatests: fix teardown method

[flo-renaud]

   URL: https://github.com/freeipa/freeipa/pull/4560
Author: flo-renaud
 Title: #4560: [ipa-4-6] ipatests: fix teardown method
Action: opened

PR body:
"""
The teardown method of the integration tests is calling
  tasks.uninstall_master(replica)
which in turn does ipa server-del + ipa-server-install --uninstall

The uninstallation is called by default with --ignore-last-of-role
and --ignore-topology-disconnect excepted if the domain-level is 0,
in which case these options are not supported by uninstall.
    
The issue is that the call to get domain level fails because the
server has already been removed, and the code handles that failure by
assuming domain level =0, resulting in missing --ignore-last-of-role.

To workaround the issue, the call to tasks.uninstall should be done
with the actual domain level:
  tasks.uninstall_master(replica, domain_level=...)

Fixes: https://pagure.io/freeipa/issue/8280

"""

To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/4560/head:pr4560
git checkout pr4560