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:

Hello,

+        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:
https://fedorahosted.org/freeipa/ticket/6006

Once these changes are implemented, we can update this test

Wat?

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

Martin^2





NACK

+    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

Martin^2
-- 
Manage your subscription for the Freeipa-devel mailing list:
https://www.redhat.com/mailman/listinfo/freeipa-devel
Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code

Reply via email to