[freeipa PR#2585][opened] ipa-replica-install: password and admin-password options mutually exclusive
by flo-renaud
URL: https://github.com/freeipa/freeipa/pull/2585
Author: flo-renaud
Title: #2585: ipa-replica-install: password and admin-password options mutually exclusive
Action: opened
PR body:
"""
## ipa-replica-install: make password and admin-password options mutually exclusive
Currently it is possible to run ipa-replica-install in one step, and provide --password and --admin-password simultaneously.
This is confusing as --password is intended for one-time pwd when the ipa-replica-install command is delegated to a user who doesn't know the admin password.
The fix makes --password and --admin-password options mutually exclusive.
Fixes https://pagure.io/freeipa/issue/6353
## ipatests: add test for ipa-replica-install options
Add a test checking that --password and --admin-password options are mutually exclusive.
"""
To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/2585/head:pr2585
git checkout pr2585
5 years, 5 months
[freeipa PR#2574][opened] ipatests: fix test_replica_uninstall_deletes_ruvs
by flo-renaud
URL: https://github.com/freeipa/freeipa/pull/2574
Author: flo-renaud
Title: #2574: ipatests: fix test_replica_uninstall_deletes_ruvs
Action: opened
PR body:
"""
test_topology.py is failing because of a wrong scenario.
Currently, test_replica_uninstall_deletes_ruvs does:
- install master + replica with CA
- `ipa-replica-manage list-ruv` to check that the repl is propery setup
- `ipa-replica-manage del $replica`
- (on replica) `ipa-server-install --uninstall -U`
- `ipa-replica-manage list-ruv` to check that replica does not appear any more in the RUV list
When ipa-replica-manage del is run, the topology plugin creates 2 tasks cleanallruvs (one for the domain, one for the ca) and they are run asynchronously. This means that the ruvs may still be present when the test moves forward and calls list-ruv.
The test should wait for the cleanallruvs tasks to finish before checking that list-ruv does not display replica anymore.
Fixes https://pagure.io/freeipa/issue/7545
"""
To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/2574/head:pr2574
git checkout pr2574
5 years, 5 months