[Freeipa-devel] [PATCH] more DS uninstall cleanup

[Simo Sorce]

On Tue, 2008-05-13 at 16:35 -0400, Rob Crittenden wrote:
> Simo Sorce wrote:
> > On Tue, 2008-05-13 at 14:38 -0400, Rob Crittenden wrote:
> >> Simo Sorce wrote:
> >>> On Fri, 2008-05-09 at 16:04 -0400, Rob Crittenden wrote:
> >>>> +        shutil.rmtree("/var/lib/dirsrv/slapd-%s/db" % serverid)
> >>> Shouldn't we remove the whole directory and not just /db ?
> >>>
> >> We delete the whole directory now. I'm proposing we keep any ldif's and 
> >> backups.
> > 
> > We should ask, backups may contain security sensitive material you may
> > really want to remove from the system.
> > 
> > Simo.
> > 
> 
> Well, we clearly state the we "will delete all data and configuration!" 
> so I guess we can go ahead and punt on this.
> 
> Is the rest of the patch acceptable?

yup
Simo.

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