[Freeipa-devel] [PATCH 0437] dogtag: remove redundant parameters from CS.cfg

[Martin Basti]

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:
>>>>> https://fedorahosted.org/freeipa/ticket/5298
>>>>>
>>>>> Patch attached.
>>>>>
>>>> ACK
>>>
>>> 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