[Freeipa-devel] [PATCH 0051] Clear SSSD caches when uninstalling the client

[Simo Sorce]

On Tue, 2015-06-30 at 13:19 +0200, Tomas Babej wrote:
> 
> On 06/30/2015 01:08 PM, Martin Basti wrote:
> > On 30/06/15 13:00, Tomas Babej wrote:
> >>
> >> On 06/29/2015 03:50 PM, Martin Basti wrote:
> >>> On 29/06/15 13:46, Jakub Hrozek wrote:
> >>>> On Fri, Jun 05, 2015 at 11:31:54AM -0600, Gabe Alford wrote:
> >>>>> Thanks. Updated patch attached.
> >>>>>
> >>>>> On Fri, Jun 5, 2015 at 9:53 AM, Jakub Hrozek <jhrozek at redhat.com>
> >>>>> wrote:
> >>>>>
> >>>>>> On Fri, Jun 05, 2015 at 09:46:05AM -0600, Gabe Alford wrote:
> >>>>>>> How should ​
> >>>>>>> https://www.redhat.com/archives/freeipa-users/2015-June/msg00116.html
> >>>>>>>
> >>>>>>> be
> >>>>>>> handled where the user cleared out the db cache?
> >>>>>> Ah, I confused that one with another issue Jan Pazdziora had,
> >>>>>> which was
> >>>>>> incidentally about client uninstall as well.
> >>>>>>
> >>>>>> In that case, you can just remove the single ldb file that
> >>>>>> corresponds
> >>>>>> to the domain that the client is leaving. Maybe it would be safer
> >>>>>> to mv
> >>>>>> the files instead of remove them, but I guess if you run --uninstall,
> >>>>>> you really want just to purge everything..
> >>>>>>
> >>>>>> btw do the ipa installer tools support multiple domains at all?
> >>>>>>
> >>>>>> -- 
> >>>>>> 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
> >>>>>>
> >>>>>   From 40f7c3780baaf0b42d10c94c8527c9359a42247f Mon Sep 17 00:00:00
> >>>>> 2001
> >>>>> From: Gabe <redhatrises at gmail.com>
> >>>>> Date: Fri, 5 Jun 2015 11:27:46 -0600
> >>>>> Subject: [PATCH] Clear SSSD caches when uninstalling the client
> >>>>>
> >>>>> https://fedorahosted.org/freeipa/ticket/5049
> >>>> Conceptually LGTM, but I haven't tested the patch.
> >>>>
> >>> ACK, I did testing.
> >>>
> >> Pushed to master: 37729936dd6fe9c3396cbb8a682a4674af8b5537
> >>
> >> For ipa-4-1 the patch requires a rebase.
> > Rebased patch for ipa-4-1 attached.
> > 
> 
> Pushed to ipa-4-1: 222427cb37a037f24ca76a9bcf614a2711a2ba96
> 

This patch break ipa-client-install --uninstall when the first part of
sssd uninstall fails, and exception is thrown and we 'pass', but then
domain is not set and we use it.

Please revert or fix it.

Simo.

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