ipatests: Test if server setup without dns uninstall properly #5443
Conversation
|
@mrizwan93 |
mrizwan93
force-pushed
the
bz1909876
branch
2 times, most recently
from
ba1c172
to
245e418
Compare
mrizwan93
force-pushed
the
bz1909876
branch
2 times, most recently
from
ddfb3cf
to
48838c4
Compare
| @@ -68,7 +68,7 @@ jobs: | |||
| class: RunPytest | |||
| args: | |||
| build_url: '{fedora-latest/build_url}' | |||
| test_suite: test_integration/test_REPLACEME.py | |||
| test_suite: test_integration/test_uninstall.py::UninstallWithoutDNS | |||
There was a problem hiding this comment.
There is a typo in the temp commit, should be:
test_suite: test_integration/test_uninstall.py::TestUninstallWithoutDNS
|
LGTM, temp commit can be removed. |
|
Thanks @kaleemsiddiqu temp commit results |
There was a problem hiding this comment.
@mrizwan93
as there is a new test file, you need to add job definitions in ipatests/prci_definitions/*.yaml. No need to add the new test in gating.yaml, nightly_latest_pki.yaml or nightly_latest_389ds.yaml but the test needs to be defined in the other files.
IPA server uninstall was failing if dns was not setup. This test check if it uninstalls propelry. related: https://pagure.io/freeipa/issue/8630 Signed-off-by: Mohammad Rizwan <myusuf@redhat.com>
Signed-off-by: Mohammad Rizwan <myusuf@redhat.com>
mrizwan93
added
ipa-4-9
|
@mrizwan93 thanks for the update, the nightly definitions LGTM. |
flo-renaud
added
ack
|
@mrizwan93 a manual backport is required for ipa-4-9 branch (likely because of the yaml files that have different names), can you do it? Thanks |
|
@flo-renaud Thanks, backport PR raised. |
IPA server uninstall was failing if dns was not setup.
This test check if it uninstalls propelry.
related: https://pagure.io/freeipa/issue/8630
Signed-off-by: Mohammad Rizwan myusuf@redhat.com