[Freeipa-devel] [PATCH] install dies if selinux is disabled

[Simo Sorce]

On Wed, 2007-10-03 at 14:07 -0400, William Jon McCann wrote:
> On 10/3/07, Simo Sorce <ssorce at redhat.com> wrote:
> > On Wed, 2007-10-03 at 13:00 -0400, William Jon McCann wrote:
> > > Hi,
> > >
> > > Tiny patch to fix the installer crashing if selinux is disabled.  Also
> > > changes the exception to contain the complete command.
> > >
> > > FYI: the installer goes kinda wonky if you have run it multiple times
> > > and you don't apply this:
> > > https://bugzilla.redhat.com/show_bug.cgi?id=317071
> > >
> > > Perhaps we can make it more robust to dirsrv failing to start.
> >
> > Thanks, yeah at this stage we still don't support interrupting the
> > install script.
> > We are still thinking on which is the best way to handle it. I think we
> > will probably prompt a very alarming message about deleting your
> > existing directory configuration and data and then wipe it out if you
> > really want to.
> > We are also considering using the realm name instead of a UUID in the
> > file names so that it will make it possible to better detect the
> > intentions (create a new server vs replacing an existing one).
> 
> Yeah, that would work for me.  I figured out that I need to do the
> following before running the install script:
> sudo rm -rf /var/lib/dirsrv/slapd-* /var/lock/dirsrv/* /etc/dirsrv/slapd-*
> sudo pkill -U dirsrv
> 
> The only part that was missing was the chown of /var/run/dirsrv.
> 
> I guess we should also try to clean up the configurations when the
> script doesn't run to completion...

Uhmm that would mean catching ctrl+c and signals, not sure how it works
with python, may be it is just simpler to clean up on each start-up, it
will also catch legitimate existing cruft :)

Simo.

> 
> Thanks,
> Jon