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 timeout value in test_getcert_list_profile_using_subca #4924

Closed
wants to merge 1 commit into from

Conversation

menonsudhir
Copy link
Contributor

test_getcert_list_profile_using_subca test had a timeout value of 50 waiting for the cert to be in MONITORING state, this has now been replaced with 300, since the certmonger request was in state SUBMITTING instead of MONITORING causing the test to fail.

…ubca

test_getcert_list_profile_using_subca test had a timeout value of 50
waiting for the cert to be in MONITORING state, this has now been
replaced with 300, since the certmonger request was in state SUBMITTING
instead of MONITORING causing the test to fail.
@flo-renaud
Copy link
Contributor

Hi @menonsudhir
thanks for the fix, LGTM. Let's wait for PR-CI to finish

@menonsudhir
Copy link
Contributor Author

/AzurePipelines run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@flo-renaud
Copy link
Contributor

The azure failure is linked to tomcat update, please see https://bugzilla.redhat.com/show_bug.cgi?id=1857043

@flo-renaud
Copy link
Contributor

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@flo-renaud
Copy link
Contributor

The test is green, please remove the temp commit.

@flo-renaud
Copy link
Contributor

ACK when PR-CI completes.

@flo-renaud flo-renaud added ack Pull Request approved, can be merged ipa-4-8 Mark for backport to ipa 4.8 pushed Pull Request has already been pushed labels Jul 17, 2020
@flo-renaud
Copy link
Contributor

master:

  • 04d25dd ipatests: Increase timeout value in test_getcert_list_profile_using_subca

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
Projects
None yet
2 participants