Re: [Freeipa-devel] [PATCH] Fixed host managed-by adder dialog.

2012-01-30 Thread Petr Vobornik
On 01/26/2012 05:33 PM, Endi Sukma Dewata wrote: The host managed-by adder dialog has been fixed to use the new --not-man-hosts option to filter out hosts that are already added. Ticket #1675 ACK -- Petr Vobornik ___ Freeipa-devel mailing list

Re: [Freeipa-devel] [PATCH] 934 don't bind on TLS connect failure

2012-01-30 Thread Martin Kosek
On Fri, 2012-01-27 at 13:22 -0500, Rob Crittenden wrote: Martin Kosek wrote: On Thu, 2012-01-26 at 16:37 -0500, Rob Crittenden wrote: In our installer LDAP library (also used by replication tools) we handle the case where the remote server hasn't started yet (wait_on_bind). What this

Re: [Freeipa-devel] [PATCH] 933 %ghost some UI files

2012-01-30 Thread Martin Kosek
On Fri, 2012-01-27 at 09:26 -0500, Rob Crittenden wrote: Martin Kosek wrote: On Thu, 2012-01-26 at 14:11 -0500, Rob Crittenden wrote: Add a %ghost to some files installed by the UI so that they are owned by freeipa-server when the server is installed and they will be removed by rpm when

Re: [Freeipa-devel] [PATCH] 921 fix existing replication agreements

2012-01-30 Thread Martin Kosek
On Fri, 2012-01-13 at 10:10 -0500, Rob Crittenden wrote: Rob Crittenden wrote: A bug when creating replication agreements has caused memberOf to be dropped from the exclusion list. This patch adds a tool that will find and fix the agreements. It will be run when the package is installed so

Re: [Freeipa-devel] [PATCH] 928 fix automount submounts

2012-01-30 Thread Jakub Hrozek
On Mon, Jan 30, 2012 at 02:51:42PM +0100, Martin Kosek wrote: On Thu, 2012-01-19 at 16:56 -0500, Rob Crittenden wrote: There is a --parentmap option in automountmap-add-indirect. This is used to create submount maps but wasn't working at all. It now creates the proper key entry pointing

Re: [Freeipa-devel] [PATCH] 921 fix existing replication agreements

2012-01-30 Thread Alexander Bokovoy
On Mon, 30 Jan 2012, Martin Kosek wrote: My main concern is if it is conceptually OK to create a separate script for one-time fixes like this one. What if we find another problem with replica agreements where we would need to update existing agreements? Would we create another fix tool or

Re: [Freeipa-devel] [PATCH] 921 fix existing replication agreements

2012-01-30 Thread Martin Kosek
On Mon, 2012-01-30 at 16:28 +0200, Alexander Bokovoy wrote: On Mon, 30 Jan 2012, Martin Kosek wrote: My main concern is if it is conceptually OK to create a separate script for one-time fixes like this one. What if we find another problem with replica agreements where we would need to

Re: [Freeipa-devel] [PATCH] 921 fix existing replication agreements

2012-01-30 Thread Rob Crittenden
Martin Kosek wrote: On Mon, 2012-01-30 at 16:28 +0200, Alexander Bokovoy wrote: On Mon, 30 Jan 2012, Martin Kosek wrote: My main concern is if it is conceptually OK to create a separate script for one-time fixes like this one. What if we find another problem with replica agreements where we

Re: [Freeipa-devel] [PATCH] 921 fix existing replication agreements

2012-01-30 Thread Alexander Bokovoy
On Mon, 30 Jan 2012, Martin Kosek wrote: Maybe we could move this agreement fix to a plugin in ipa-ldap-updater or create a more general tool for one-time fixes like this one. I am thinking about something like this: ipa-server-remedy [-l/--list] [-u/--unit] [-d/--debug] [-t/--test]

Re: [Freeipa-devel] [PATCH] 921 fix existing replication agreements

2012-01-30 Thread Alexander Bokovoy
On Mon, 30 Jan 2012, Rob Crittenden wrote: Yes, this was my first suggestion. I mentioned this general utility as a second option if we need to implement this onetime fix as a separate tool for some reason. Martin This is for FreeIPA 2.1 which doesn't support update plugins. I think we

Re: [Freeipa-devel] [PATCH] 921 fix existing replication agreements

2012-01-30 Thread Rob Crittenden
Alexander Bokovoy wrote: On Mon, 30 Jan 2012, Rob Crittenden wrote: Yes, this was my first suggestion. I mentioned this general utility as a second option if we need to implement this onetime fix as a separate tool for some reason. Martin This is for FreeIPA 2.1 which doesn't support update

Re: [Freeipa-devel] [PATCH] 203 Add argument help to CLI

2012-01-30 Thread Rob Crittenden
Martin Kosek wrote: An example of the improved help: # ipa help user-add Purpose: Add a new user. Usage: ipa [global-options] user-add LOGIN [options] Positional arguments: LOGIN: User login Options: -h, --help show this help message and exit --first=STRFirst name

Re: [Freeipa-devel] [PATCH] 925 Make deleting replication agreements nicer

2012-01-30 Thread Martin Kosek
On Mon, 2012-01-16 at 23:37 -0500, Rob Crittenden wrote: Check for the existence of a replication agreement before deleting it. When using ipa-replica-manage or ipa-csreplica-manage to delete an agreement with a host we would try to make a connection to that host prior to tryign to delete

Re: [Freeipa-devel] [PATCH] 202 Add reverse DNS record when forward is created

2012-01-30 Thread Rob Crittenden
Martin Kosek wrote: Adding reverse DNS record may be a time consuming task, especially for IPv6 addresses. Having a way to automatically create a reverse record when a forward record is created could speed up the process. host-add command already has this possibility. This patch takes advantage

Re: [Freeipa-devel] [PATCH] 933 %ghost some UI files

2012-01-30 Thread Rob Crittenden
Martin Kosek wrote: On Fri, 2012-01-27 at 09:26 -0500, Rob Crittenden wrote: Martin Kosek wrote: On Thu, 2012-01-26 at 14:11 -0500, Rob Crittenden wrote: Add a %ghost to some files installed by the UI so that they are owned by freeipa-server when the server is installed and they will be

Re: [Freeipa-devel] [PATCH] 928 fix automount submounts

2012-01-30 Thread Rob Crittenden
Rob Crittenden wrote: Jakub Hrozek wrote: On Mon, Jan 30, 2012 at 02:51:42PM +0100, Martin Kosek wrote: On Thu, 2012-01-19 at 16:56 -0500, Rob Crittenden wrote: There is a --parentmap option in automountmap-add-indirect. This is used to create submount maps but wasn't working at all. It now

[Freeipa-devel] [PATCH] 936 support defaultNamingContext and basedn in migration

2012-01-30 Thread Rob Crittenden
Add support for defaultNamingContext which is available in 389-ds 1.2.10-0.9.a8. If the attribute isn't returned continue to use namingContexts to determine the basedn. While I was in poking at this I added support to the migration plugin to be able to pass in the basedn of the remote server.

Re: [Freeipa-devel] [PATCH] [WIP] 069 Added refresh button for UI

2012-01-30 Thread Endi Sukma Dewata
On 1/17/2012 9:36 AM, Endi Sukma Dewata wrote: On 1/16/2012 8:02 AM, Petr Vobornik wrote: 1) Button position: I added the button into facet header next to 'add', 'delete', 'reset', 'update' buttons as shown on the picture ( http://pvoborni.fedorapeople.org/images/2051-refresh-button.png ). I'm

Re: [Freeipa-devel] [PATCH] 070 Modifying DNS UI to benefit from new DNS API

2012-01-30 Thread Endi Sukma Dewata
On 1/27/2012 8:51 AM, Petr Vobornik wrote: Updated patch attached. ACK and pushed to master and ipa-2-2. See also the comments below. 5. The following attributes probably should be shown as text areas in the edit dialog and should not be displayed in the table because it could be too long: *