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-5] - test_renewal_master: add ipa csreplica-manage test #1579

Closed
wants to merge 1 commit into from

Conversation

Rezney
Copy link
Collaborator

@Rezney Rezney commented Feb 14, 2018

This PR was opened automatically because PR #1573 was pushed to ipa-4-6 and backport to ipa-4-5 is required.

@Rezney
Copy link
Collaborator Author

Rezney commented Feb 14, 2018

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

@Rezney Rezney added the ack Pull Request approved, can be merged label Feb 14, 2018
@tiran tiran changed the title [Backport][ipa-4-5] [Backport][ipa-4-6] - test_renewal_master: add ipa csreplica-manage test [Backport][ipa-4-5] - test_renewal_master: add ipa csreplica-manage test Feb 14, 2018
@stlaz stlaz removed the ack Pull Request approved, can be merged label Feb 14, 2018
@stlaz
Copy link
Contributor

stlaz commented Feb 14, 2018

You're missing import re

Add test case for setting renewal master using command
ipa-csreplica-manage.

Automation related to upstream ticket freeipa#7120. Testing using
config-mod already covered.

https://pagure.io/freeipa/issue/7321

Reviewed-By: Christian Heimes <cheimes@redhat.com>
@stlaz stlaz added the ack Pull Request approved, can be merged label Feb 14, 2018
@Rezney
Copy link
Collaborator Author

Rezney commented Feb 14, 2018

ipa-4-5:

  • dbfec88 test_renewal_master: add ipa csreplica-manage test

@Rezney Rezney added the pushed Pull Request has already been pushed label Feb 14, 2018
@Rezney Rezney closed this Feb 14, 2018
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