[Freeipa-devel] [PATCH 0307] Server Install: print message that client is being installed

[Martin Basti]

On 09/11/2015 04:00 PM, Simo Sorce wrote:
> On Fri, 2015-09-11 at 14:49 +0200, Martin Basti wrote:
>> On 09/03/2015 03:56 PM, Simo Sorce wrote:
>>> On Thu, 2015-09-03 at 15:32 +0200, Martin Basti wrote:
>>>> On 09/03/2015 02:42 PM, Simo Sorce wrote:
>>>>> On Thu, 2015-09-03 at 10:19 +0200, Martin Basti wrote:
>>>>>> On 09/02/2015 06:00 PM, Simo Sorce wrote:
>>>>>>> On Wed, 2015-09-02 at 17:57 +0200, Martin Basti wrote:
>>>>>>>> Client installation is done as "Restarting web server". This step
>>>>>>>> deserve own message.
>>>>>>>>
>>>>>>>> Patch attached
>>>>>>> I've seen various cases like this. And I can't understand why these
>>>>>>> steps aren't embedded in the actual instance install steps that need the
>>>>>>> restart (which implicitly also provide a message).
>>>>>>>
>>>>>>> In the promotion patchset I did move steps like this into the proper
>>>>>>> instances, so I would prefer you do the same with the install path as
>>>>>>> that is more appropriate.
>>>>>>>
>>>>>>> Simo.
>>>>>>>
>>>>>> We need restart httpd after CA, DNS(optional) installation, so thats why
>>>>>> it is outside of httpd instance.
>>>>> You need to restart httpd always after CA install as it changes the
>>>>> proxy settings, but why do you need to restart it after DNS
>>>>> installation ?
>>>> It is needed due changes in resolv.conf
>>>>> (I think it is fine to restart it twice if it is really needed after DNS
>>>>> change).
>>>> IMO it is waste of time to restart httpd twice in one minute
>>>>
>>>> Can we resolve this in 4.4, where might be place to finish improvements
>>>> in installer? (If this is not blocking replica promotion)
>>> Ok, it is not important enough to waste time now.
>>>
>>> Simo.
>>>
>> Can be this patch pushed to master then?
> Yes
>
Pushed to master: 7f0076b9a5f2aced1f27b976217309be2eec0b1c