On 18.12.2015 09:35, Oleg Fayans wrote:
Hi Petr,

On 12/17/2015 08:19 PM, Petr Spacek wrote:
On 17.12.2015 14:27, Oleg Fayans wrote:
Commit message was updated. Thanks!

On 12/17/2015 02:05 PM, Lukas Slebodnik wrote:
On (17/12/15 13:53), Oleg Fayans wrote:
--
Oleg Fayans
Quality Engineer
FreeIPA team
RedHat.
>From ed4630140386c1043e36733eb42ec402cc276bee Mon Sep 17 00:00:00 2001
From: Oleg Fayans <ofay...@redhat.com>
Date: Thu, 17 Dec 2015 13:50:19 +0100
Subject: [PATCH] Enabled automatic creation of reverse zone during master
installation

The commit message does not contain ticket.
It is also not explained why this change was done.
Could you update commit message with verbose explanation?

Proper commit message might prevent removal of this change in future
based on "git blame"
Seems reasonable in general but this should not happen in BRQ or ABCDE labs.
Where do you see the problem?

Otherwise ACK.
The problem occurs in local libvirt-powered VMs. Also it may (or may not
- I did not try it yet) be met in Beaker.


NACK

ipa-server-install: error: You cannot specify a --auto-reverse option without the --setup-dns option

--
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