Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Backport][ipa-4-9] ipatests: various enhancement to hidden replica tests #5695

Closed

Conversation

flo-renaud
Copy link
Contributor

This PR was opened automatically because PR #5556 was pushed to master and backport to ipa-4-9 is required.

wait_for_ipa_to_start(host) waits for ipactl to return RUNNING for all
IPA services on the specified host.

Related: https://pagure.io/freeipa/issue/8534
Signed-off-by: François Cami <fcami@redhat.com>
Add a frontend to "ipa dns-update-system-records" to tasks.py.

Related: https://pagure.io/freeipa/issue/8534
Signed-off-by: François Cami <fcami@redhat.com>
Use wait_for_ipa_to_start to wait until the restored replica is online.

Related: https://pagure.io/freeipa/issue/8534
Signed-off-by: François Cami <fcami@redhat.com>
Previously, hidden replica checks were run without waiting for replication
to complete, potentially leading to unstable behavior.
Use wait_for_replication.

Fixes: https://pagure.io/freeipa/issue/8534
Signed-off-by: François Cami <fcami@redhat.com>
Use dns_update_system_records after restoring the replica
to force-update the DNS records.

Related: https://pagure.io/freeipa/issue/8534
Signed-off-by: François Cami <fcami@redhat.com>
Rename a test and split a test in two.

Related: https://pagure.io/freeipa/issue/8534
Signed-off-by: François Cami <fcami@redhat.com>
test_ipahealthcheck_hidden_replica fails due to:
https://pagure.io/freeipa/issue/8582
Mark it as expected failure.

Related: https://pagure.io/freeipa/issue/8534
Signed-off-by: François Cami <fcami@redhat.com>
@flo-renaud flo-renaud added the ack Pull Request approved, can be merged label Apr 1, 2021
@flo-renaud
Copy link
Contributor Author

PR was ACKed automatically because this is backport of PR #5556. Wait for CI to finish before pushing. In case of questions or problems contact @fcami who is author of the original PR.

@flo-renaud flo-renaud added the pushed Pull Request has already been pushed label Apr 2, 2021
@flo-renaud
Copy link
Contributor Author

ipa-4-9:

  • 7f598c8 ipatests: tasks.py: add wait_for_ipa_to_start
  • 18b0350 ipatests: tasks.py: add dns_update_system_records
  • 9ad9c38 ipatests: hiddenreplica: use wait_for_ipa_to_start after restore
  • 834adfc ipatests: use wait_for_replication for hidden replica checks
  • b1fef6b ipatests: hidden replica: use dns_update_system_records
  • 45fa104 ipatests: hidden replica: misc fixes
  • 0ffaf29 ipatests: mark test_ipahealthcheck_hidden_replica as expected failure

@flo-renaud flo-renaud closed this Apr 2, 2021
@flo-renaud flo-renaud deleted the backport_pr5556_ipa-4-9 branch April 2, 2021 07:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ack Pull Request approved, can be merged pushed Pull Request has already been pushed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants