On 30.06.2016 14:40, Oleg Fayans wrote:
Hi Martin,

Attached is a new version of the patch with two test cases separated.

On 06/29/2016 12:23 PM, Martin Basti wrote:

On 29.06.2016 10:56, Oleg Fayans wrote:


+        assert_error(result,
+                     "Failed to verify that %s is an IPA Server" %
+                     self.master.hostname)

I would expect this error there:

"Cannot promote this client to a replica. Local domain '{local}' does
not match IPA domain '{ipadomain}'. "
Right, that's what this ticket is about:

Once these changes are implemented, we can update this test


You get exactly the right message from ipa-replica-install, tested, reviewed by several people.

You should not use random REALM, in this case you don't test domains but
realms. You can leave the test with incorrect realm there, but as
separated testcase
Oh, ok. But it does not seem possible to setup client providing only
--realm without --domain: installer would not do it.

Try to read again: "should not use *random* REALM". Nothing prevents you to use, --realm=TEST.REALM --domain=random-blah-domain



+    domain_name = 'exxample.test'
+    realm_name = domain_name.upper()

you still use random realm name, and you still don't test ipa-replica-install, that ticket has nothing related to domain in ipa-client-install, it is related to replica promotion

Manage your subscription for the Freeipa-devel mailing list:
Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code

Reply via email to