Re: [Freeipa-devel] [PATCH] 1007 remove all state when uninstalling

2012-04-17 Thread Martin Kosek
On Mon, 2012-04-16 at 17:28 -0400, Rob Crittenden wrote: We no longer use the running state when uninstalling DS instances but we need to pull it in case it is there in an upgraded instance. rob ACK. Pushed to master, ipa-2-2. I added a clear reproduction steps to the linked Bugzilla.

[Freeipa-devel] [PATCH] 1007 remove all state when uninstalling

2012-04-16 Thread Rob Crittenden
We no longer use the running state when uninstalling DS instances but we need to pull it in case it is there in an upgraded instance. rob From 2862b1d40e688c931c6d7ac2b4fcdfc5defaca0e Mon Sep 17 00:00:00 2001 From: Rob Crittenden rcrit...@redhat.com Date: Mon, 16 Apr 2012 17:03:46 -0400

Re: [Freeipa-devel] [PATCH] 1007 remove all state when uninstalling

2012-04-16 Thread John Dennis
On 04/16/2012 05:28 PM, Rob Crittenden wrote: We no longer use the running state when uninstalling DS instances but we need to pull it in case it is there in an upgraded instance. Ah, so the issue is that StateFile.restore_state() has the side effect of removing the key from the state file