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

ipatests: increase test_caless_TestReplicaInstall timeout #4841

Closed
wants to merge 1 commit into from
Closed

ipatests: increase test_caless_TestReplicaInstall timeout #4841

wants to merge 1 commit into from

Conversation

fcami
Copy link
Contributor

@fcami fcami commented Jun 22, 2020

test_caless_TestReplicaInstall timeout seems too short.
Extend it.

Fixes: https://pagure.io/freeipa/issue/8377
Signed-off-by: François Cami fcami@redhat.com

test_caless_TestReplicaInstall timeout seems too short.
Extend it.

Fixes: https://pagure.io/freeipa/issue/8377
Signed-off-by: François Cami <fcami@redhat.com>
@fcami fcami added trivial ipa-4-8 Mark for backport to ipa 4.8 labels Jun 22, 2020
@fcami fcami requested a review from netoarmando June 22, 2020 14:41
@fcami
Copy link
Contributor Author

fcami commented Jun 22, 2020

hi @netoarmando the test suites completed in roughly 4000-4200s, so I wonder why we hit 5400s even with provisioning and teardown included. Still, I'd rather have that change in rather than having a repeat.

@flo-renaud
Copy link
Contributor

Hi @fcami
change LGTM. Please remove the temp commit and I will ACK as soon as PRCI completes.

@flo-renaud flo-renaud self-assigned this Jun 22, 2020
@flo-renaud flo-renaud added ack Pull Request approved, can be merged pushed Pull Request has already been pushed labels Jun 23, 2020
@flo-renaud
Copy link
Contributor

master:

  • c5e6fe0 ipatests: increase test_caless_TestReplicaInstall timeout

@flo-renaud flo-renaud closed this Jun 23, 2020
@flo-renaud
Copy link
Contributor

@fcami
a manual backport to ipa-4-8 is required, can you handle it? Thanks

@fcami
Copy link
Contributor Author

fcami commented Jun 23, 2020

hi @flo-renaud thanks for merging. I will do the backport.

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 ipa-4-8 Mark for backport to ipa 4.8 pushed Pull Request has already been pushed trivial
Projects
None yet
3 participants