[Freeipa-devel] [PATCH] enhance sysrestore

[Simo Sorce]

On Mon, 2008-03-24 at 09:24 -0400, Rob Crittenden wrote:
> Simo Sorce wrote:
> > On Mon, 2008-03-24 at 09:02 -0400, Rob Crittenden wrote:
> >> Simo Sorce wrote:
> >>> On Thu, 2008-03-20 at 17:00 -0400, Rob Crittenden wrote:
> >>>> Simo Sorce wrote:
> >>>>> Simo Sorce wrote:
> >>>>>> On Thu, 2008-03-20 at 16:06 -0400, Simo Sorce wrote:
> >>>>>>> This is in preparation to move it into ipa-python so that I can use 
> >>>>>>> sysrestore to make a client --uninstall option.
> >>>>>> Arghh this one does not merge properly with latest code, let me fix it
> >>>>>> and resend.
> >>>>> New patch attached.
> >>>>>
> >>>> You are completely doing away with these restores?
> >>> There is one big restore in the main script now, as we keep track of
> >>> every file it made little sense to me to do the restores one by one.
> >>>
> >>> Simo.
> >>>
> >> Ok, ack then.
> > 
> > I have been thinking about this during the weekend, and it came to mind
> > that we may be restarting some daemon during uninstall. Should we delay
> > restarts after files are restored ? I guess daemons may want to see the
> > original files.
> > 
> > what do you think?
> > 
> > Simo.
> > 
> 
> Yes, I think that the current running process should match the config on 
> disk.

Ok In this case I'll put back 1 by 1 restore of files for daemons we
restart, and leave a global restore in the main only for stuff that is
not affected by a restart.

New patch soon.

Simo.

-- 
Simo Sorce * Red Hat, Inc * New York