[Freeipa-devel] [PATCH] 115 ipa-client-install breaks network configuration

2011-08-29 Thread Martin Kosek
I would like to get this patch included in upcoming 6.2, the issue is quite serious. --- Do not forget to add new line in updated /etc/sysconfig/network configuration. Move the actual change of the hostname after the user confirmation about proceeding with installation. It confused users when the h

Re: [Freeipa-devel] [PATCH] 115 ipa-client-install breaks network configuration

2011-08-29 Thread Alexander Bokovoy
On 29.08.2011 11:27, Martin Kosek wrote: > I would like to get this patch included in upcoming 6.2, the issue is > quite serious. > --- > Do not forget to add new line in updated /etc/sysconfig/network > configuration. Move the actual change of the hostname after the > user confirmation about proce

Re: [Freeipa-devel] [PATCH] 115 ipa-client-install breaks network configuration

2011-08-29 Thread Simo Sorce
On Mon, 2011-08-29 at 11:36 +0300, Alexander Bokovoy wrote: > On 29.08.2011 11:27, Martin Kosek wrote: > > I would like to get this patch included in upcoming 6.2, the issue is > > quite serious. > > --- > > Do not forget to add new line in updated /etc/sysconfig/network > > configuration. Move the

Re: [Freeipa-devel] [PATCH] 009 Modifying sudo options refreshes the whole page

2011-08-29 Thread Petr Vobornik
On 08/26/2011 11:04 PM, Endi Sukma Dewata wrote: On 8/26/2011 11:41 AM, Petr Vobornik wrote: https://fedorahosted.org/freeipa/ticket/1689 Currently adding or deleting sudo options will refresh the entire page. It's not a problem but the code could be optimized to refresh only the sudo options t

Re: [Freeipa-devel] [PATCH] 845 add external source hosts to HBAC

2011-08-29 Thread Martin Kosek
On Wed, 2011-08-17 at 11:56 -0400, Rob Crittenden wrote: > You should be able to specify hosts that aren't in IPA in HBAC source > host rules. This adds support for it. > > ticket https://fedorahosted.org/freeipa/ticket/1574 > > rob ACK. Works fine. Although, I am thinking that we now have the

[Freeipa-devel] [PATCH] 257 Fixed host adder dialog to show default DNS zone.

2011-08-29 Thread Endi Sukma Dewata
The DNS zone widget for host adder dialog has been modified not to provide an empty option, so it will show the first available zone. Ticket #1685 -- Endi S. Dewata From b2f14d73824bc12e6d7bb47e3907312efd1aaa97 Mon Sep 17 00:00:00 2001 From: Endi S. Dewata Date: Fri, 26 Aug 2011 14:27:48 -0500

[Freeipa-devel] [PATCH] 859 bump dogtag min n-v-r

2011-08-29 Thread Rob Crittenden
Update pki-* n-v-r to pick up SELinux fix in https://bugzilla.redhat.com/show_bug.cgi?id=700505. rob >From 228977079575b0069b30133fbabd36db8404ad85 Mon Sep 17 00:00:00 2001 From: Rob Crittenden Date: Mon, 29 Aug 2011 10:27:17 -0400 Subject: [PATCH] Set min nvr of pki-ca to 9.0.12 for fix in BZ

Re: [Freeipa-devel] [PATCH] 859 bump dogtag min n-v-r

2011-08-29 Thread Martin Kosek
On Mon, 2011-08-29 at 10:29 -0400, Rob Crittenden wrote: > Update pki-* n-v-r to pick up SELinux fix in > https://bugzilla.redhat.com/show_bug.cgi?id=700505. > > rob Looks good. Just one thing - the spec log confused me. Who is doing this change - you or Honza? :-) Martin _

Re: [Freeipa-devel] [PATCH] 859 bump dogtag min n-v-r

2011-08-29 Thread Martin Kosek
On Mon, 2011-08-29 at 16:45 +0200, Martin Kosek wrote: > On Mon, 2011-08-29 at 10:29 -0400, Rob Crittenden wrote: > > Update pki-* n-v-r to pick up SELinux fix in > > https://bugzilla.redhat.com/show_bug.cgi?id=700505. > > > > rob > > Looks good. Just one thing - the spec log confused me. Who is

Re: [Freeipa-devel] [PATCH] 859 bump dogtag min n-v-r

2011-08-29 Thread Rob Crittenden
Martin Kosek wrote: On Mon, 2011-08-29 at 16:45 +0200, Martin Kosek wrote: On Mon, 2011-08-29 at 10:29 -0400, Rob Crittenden wrote: Update pki-* n-v-r to pick up SELinux fix in https://bugzilla.redhat.com/show_bug.cgi?id=700505. rob Looks good. Just one thing - the spec log confused me. Who

Re: [Freeipa-devel] [PATCH] 859 bump dogtag min n-v-r

2011-08-29 Thread Rob Crittenden
Martin Kosek wrote: On Mon, 2011-08-29 at 10:29 -0400, Rob Crittenden wrote: Update pki-* n-v-r to pick up SELinux fix in https://bugzilla.redhat.com/show_bug.cgi?id=700505. rob Looks good. Just one thing - the spec log confused me. Who is doing this change - you or Honza? :-) Oh, heh. I ju

Re: [Freeipa-devel] [PATCH] 859 bump dogtag min n-v-r

2011-08-29 Thread Martin Kosek
On Mon, 2011-08-29 at 11:05 -0400, Rob Crittenden wrote: > Martin Kosek wrote: > > On Mon, 2011-08-29 at 16:45 +0200, Martin Kosek wrote: > >> On Mon, 2011-08-29 at 10:29 -0400, Rob Crittenden wrote: > >>> Update pki-* n-v-r to pick up SELinux fix in > >>> https://bugzilla.redhat.com/show_bug.cgi?i

[Freeipa-devel] [PATCH] 860 make uninstall logging more robust

2011-08-29 Thread Rob Crittenden
The uninstaller would not often report problems, may exceptions were ignored with pass. This beefs up the debug logging a bit more and will attempt to find anything left over that may prevent a re-install. This also unifies the installed-status functions. ipa-server-install looked at the insta

Re: [Freeipa-devel] [PATCH] 009 Modifying sudo options refreshes the whole page

2011-08-29 Thread Endi Sukma Dewata
On 8/29/2011 7:58 AM, Petr Vobornik wrote: I would say that the network issue and the same concurrent edit issue can be so rare, that the update isn't much necessary and it slows down more frequent failures like non-concurrent adding of the same option. If we want UI to be faster, we should remo

Re: [Freeipa-devel] [PATCH] 859 bump dogtag min n-v-r

2011-08-29 Thread Rob Crittenden
Martin Kosek wrote: On Mon, 2011-08-29 at 11:05 -0400, Rob Crittenden wrote: Martin Kosek wrote: On Mon, 2011-08-29 at 16:45 +0200, Martin Kosek wrote: On Mon, 2011-08-29 at 10:29 -0400, Rob Crittenden wrote: Update pki-* n-v-r to pick up SELinux fix in https://bugzilla.redhat.com/show_bug.cg

[Freeipa-devel] [PATCH] 861 roll back client changes on failed install

2011-08-29 Thread Rob Crittenden
There are still places where the client installation can fail that can cause /etc/sysconfig/network to be restored. I went through a number of iterations on restoring this and finally decided that we should just call uninstall() and undo everything. When we do automatic uninstallation I made i

Re: [Freeipa-devel] [PATCH] 0283-enable-proxy-for-dogtag

2011-08-29 Thread Simo Sorce
On Fri, 2011-08-26 at 22:28 -0400, Adam Young wrote: > On 08/26/2011 08:57 PM, Adam Young wrote: > > On 08/26/2011 06:30 PM, Simo Sorce wrote: > >> On Fri, 2011-08-26 at 17:41 -0400, Adam Young wrote: > >>> On 08/26/2011 02:34 PM, Simo Sorce wrote: > On Fri, 2011-08-26 at 14:03 -0400, Simo Sor