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 -- Simo Sorce * Red Hat, Inc * New York -- 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
