On 10.03.2016 08:08, Jan Cholasta wrote:
On 10.3.2016 07:57, Fraser Tweedale wrote:
On Thu, Mar 10, 2016 at 07:15:59AM +0100, Jan Cholasta wrote:
On 10.3.2016 03:35, Fraser Tweedale wrote:
On Wed, Mar 09, 2016 at 01:30:01PM +0100, Martin Basti wrote:

Patch attached.


Shouldn't we remove these on upgrade as well?

The parameters are doing no harm in existing installations, so it is
a tradeoff between ensuring consistency between existing and new
installations, or adding more steps to upgrade script.  What is the
guiding principle here?

Install and upgrade should result in the same configuration, but given we have no sane infrastructure for that and the parameters are harmless as you pointed out, I think we are fine here. So ACK.

Pushed to:
master: 0cb870e5657a5e30d66e967644ab54bfa9bf54b4
ipa-4-3: beb2b4cb90e68cfa36e880dc9654756944b731c4

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

Reply via email to