Re: [Freeipa-devel] [PATCH] [WIP] 0469 - Remove the unused ipalib.frontend.Property class

2014-02-20 Thread Gabe Alford
Sorry about that Petr. I got a little overzealous :) I can give it back to you since you already have a patch in progress. Gabe On Thu, Feb 20, 2014 at 1:54 AM, Petr Viktorin pvikt...@redhat.com wrote: Hello, I had this patch sitting around for some time but didn't get around to

Re: [Freeipa-devel] [PATCH] [WIP] 0469 - Remove the unused ipalib.frontend.Property class

2014-02-20 Thread Gabe Alford
if he is OK with letting this ticket go. The person may already have some work in progress (as Petr did in this case) or at least an idea how to implement the ticket. Thanks, Martin On 02/20/2014 03:35 PM, Gabe Alford wrote: Sorry about that Petr. I got a little overzealous :) I can

Re: [Freeipa-devel] [PATCH] ntp sync order in ipa-client-install

2014-02-20 Thread Gabe Alford
ooops! Missed that. Updated Trac. Thanks, Gabe On Thu, Feb 20, 2014 at 1:48 AM, Petr Spacek pspa...@redhat.com wrote: On 20.2.2014 05:47, Darth Vader wrote: Hi, Changed when ntp sync's in ipa-client-install for the ticket below: https://fedorahosted.org/freeipa/ticket/3957 Thanks,

[Freeipa-devel] [PATCH 0007][DOC] Tip on restoring admin account

2014-02-26 Thread Gabe Alford
Hi all, I added a tip in the deleting users section on restoring admin account. Please review. https://fedorahosted.org/freeipa/ticket/2746 Thanks, Gabe freeipa-rga-0007-DOC-Document-steps-to-restore-deleted-admin-account.patch Description: Binary data

Re: [Freeipa-devel] [PATCH 0007][DOC] Tip on restoring admin account

2014-02-28 Thread Gabe Alford
That does make more sense to merge them under the same note. I can also include a little blurb about ipa user-del and ipa group-remove-member. On Fri, Feb 28, 2014 at 5:54 AM, Petr Viktorin pvikt...@redhat.com wrote: On 02/26/2014 04:01 PM, Gabe Alford wrote: Hi all, I added a tip

Re: [Freeipa-devel] [PATCH 0007][DOC] Tip on restoring admin account

2014-03-02 Thread Gabe Alford
Here is an updated patch that merges the notes and adds info about preventing removal of the last admin. Gabe On Fri, Feb 28, 2014 at 8:39 AM, Gabe Alford redhatri...@gmail.com wrote: That does make more sense to merge them under the same note. I can also include a little blurb about ipa

Re: [Freeipa-devel] [PATCH 0007][DOC] Tip on restoring admin account

2014-03-03 Thread Gabe Alford
Yes, the attached patch looks good. I see what you are saying how it did look misleading. Thanks, Gabe On Mon, Mar 3, 2014 at 3:13 AM, Petr Viktorin pvikt...@redhat.com wrote: On 03/02/2014 11:26 PM, Gabe Alford wrote: Here is an updated patch that merges the notes and adds info about

[Freeipa-devel] [PATCH 0008] Typo in warning message where IPA realm and domain name differ

2014-03-03 Thread Gabe Alford
Hi all, Quick one line change to fix. https://fedorahosted.org/freeipa/ticket/4211 Thanks, Gabe freeipa-rga-0008-Typo-in-warning-message-where-IPA-realm-and-domain-n.patch Description: Binary data ___ Freeipa-devel mailing list

Re: [Freeipa-devel] [PATCH] Review: rga-0005 Fix order of synchronizing time when running ipa-client-install

2014-03-18 Thread Gabe Alford
Patch was posted for review on Feb 19th, but may have missed due to volume, priorities, etc. What always seemed odd to me was where the time sync occurred not the association of entering a password immediately after a user prompt. It made more sense to me to have time sync before any

[Freeipa-devel] Use of NetworkManager - to disable or not disable

2014-03-18 Thread Gabe Alford
All, Looking at https://fedorahosted.org/freeipa/ticket/4156, I wanted to see if disabling NetworkManager is still required or not. I could see in some environments admins may disable it so having it in the docs wouldn't be a bad idea IMHO. Thoughts? Gabe

[Freeipa-devel] [PATCH] typo in migrate-ds

2014-03-18 Thread Gabe Alford
All, It looks like the only typos exist in the uk and fr .po files for this ticket https://fedorahosted.org/freeipa/ticket/2546 Point me in the right direction if I am wrong. Thanks, Gabe From 16d438bd0a652cdc86485d7beae3514452be394a Mon Sep 17 00:00:00 2001 From: Gabe redhatri...@gmail.com

[Freeipa-devel] [PATCH] [DOC] login as another user in the WebUI

2014-03-18 Thread Gabe Alford
All, Just added a couple of steps under section 4.3.6 Let me know if I need to change anything. https://fedorahosted.org/freeipa/ticket/3064 Thanks, Gabe From fdc73fe2f6d3204b60ac04a9fb39fa666549f7d0 Mon Sep 17 00:00:00 2001 From: Gabe redhatri...@gmail.com Date: Tue, 18 Mar 2014 19:01:24

Re: [Freeipa-devel] [PATCH] [DOC] login as another user in the WebUI

2014-03-19 Thread Gabe Alford
Thanks for the review. Here's an updated patch. Gabe On Wed, Mar 19, 2014 at 3:35 AM, Petr Vobornik pvobo...@redhat.com wrote: On 19.3.2014 02:29, Gabe Alford wrote: All, Just added a couple of steps under section 4.3.6 Let me know if I need to change anything. https

[Freeipa-devel] [PATCH] [DOC] Review section on NetworkManager

2014-03-26 Thread Gabe Alford
All, Please review this patch for https://fedorahosted.org/freeipa/ticket/4156 Added links to documentation on configuring NetworkManager. Thanks, Gabe From 87bb9bc096ab8dfd73a6d9716dc42c94089057d3 Mon Sep 17 00:00:00 2001 From: Gabe redhatri...@gmail.com Date: Mon, 24 Mar 2014 17:13:18 -0600

[Freeipa-devel] [PATCH] [DOC] Add note about additional nameservers in resolv.conf

2014-03-26 Thread Gabe Alford
All, Please review patch for https://fedorahosted.org/freeipa/ticket/3085 Added note that 'nameserver 127.0.0.1' is added to resolv.conf, that it is recommended to add more replicas to resolv.conf, and the max nameservers allowed in resolv.conf. thanks, Gabe From

Re: [Freeipa-devel] [PATCH] [DOC] Add note about additional nameservers in resolv.conf

2014-03-27 Thread Gabe Alford
: On Thu, 2014-03-27 at 10:33 +0100, Petr Spacek wrote: On 27.3.2014 10:23, Martin Basti wrote: On Wed, 2014-03-26 at 17:40 -0600, Gabe Alford wrote: All, Please review patch for https://fedorahosted.org/freeipa/ticket/3085 Added note that 'nameserver 127.0.0.1' is added to resolv.conf

Re: [Freeipa-devel] [PATCH] [DOC] Add note about additional nameservers in resolv.conf

2014-03-28 Thread Gabe Alford
Changed 127.0.0.1 to 192.0.2.1 On Fri, Mar 28, 2014 at 1:38 AM, Petr Spacek pspa...@redhat.com wrote: On 28.3.2014 02:09, Gabe Alford wrote: I believe that Martin is right about the server installer no longer putting 127.0.0.1 in the resolv.conf. Here is a mod patch to address Martin's

[Freeipa-devel] [PATCH] [DOC] document that wildcards are not supported in FreeIPA = 3.2

2014-04-08 Thread Gabe Alford
Hello, Not sure how relevant this patch is to the current documentation considering (I believe) that wildcards are supported in versions 3.3 and up. Patch for https://fedorahosted.org/freeipa/ticket/3616 Thanks, Gabe From 1cc5d540027e1f01912263f83d6a2cceb0731cea Mon Sep 17 00:00:00 2001 From:

Re: [Freeipa-devel] [PATCH] [DOC] document that wildcards are not supported in FreeIPA = 3.2

2014-04-09 Thread Gabe Alford
I am good with it. Gabe On Wed, Apr 9, 2014 at 4:20 AM, Petr Spacek pspa...@redhat.com wrote: On 9.4.2014 10:29, Martin Basti wrote: On Tue, 2014-04-08 at 21:19 -0600, Gabe Alford wrote: Hello, Not sure how relevant this patch is to the current documentation considering (I believe

Re: [Freeipa-devel] global account lockout

2014-04-09 Thread Gabe Alford
I came across these articles that may be of some use in this topic. I humbly admit that I am no expert on this topic, and these may not be of any use. Plus, I am not a fan of the product, but maybe it helps? http://technet.microsoft.com/en-us/library/cc772726%28v=ws.10%29.aspx

[Freeipa-devel] [PATCH][DOC] Update Solaris Documentation, add proxy agent, and profile

2014-04-15 Thread Gabe Alford
Hello, The following patches update the Solaris documentation and add a proxy agent/profile for Solaris. - Solaris documentation update https://fedorahosted.org/freeipa/ticket/3731 - Patch adds default Proxy Agent and default_secure profile through 20-nss_ldap.update when ipa-server-install is

[Freeipa-devel] [PATCH][DOC] Sometimes users set bad uid ranges

2014-04-21 Thread Gabe Alford
Hello, Patch for the following ticket to add note in documentation about bad uid ranges: https://fedorahosted.org/freeipa/ticket/2090 Thanks, Gabe From ad9af09ef2250c497d75a53846539328df865578 Mon Sep 17 00:00:00 2001 From: Gabe redhatri...@gmail.com Date: Mon, 21 Apr 2014 17:13:13 -0600

[Freeipa-devel] [PATCH] Stop ntpd before running ntpdate

2014-04-25 Thread Gabe Alford
Hello, Here is a patch for https://fedorahosted.org/freeipa/ticket/3735. It seemed better to try to stop ntpd before running ntpdate rather than not running ntpdate if ntpd was already running. I believe this patch only applies to the ipa-3-3 branch as ntpdate is not used anymore in the

Re: [Freeipa-devel] [PATCH] Stop ntpd before running ntpdate

2014-04-29 Thread Gabe Alford
Updated patch to not run ntpdate if ntpd is running. Gabe On Tue, Apr 29, 2014 at 8:16 AM, Gabe Alford redhatri...@gmail.com wrote: Thanks Petr! Will rework patch to just skip ntpdate if ntpd is already running. On Tue, Apr 29, 2014 at 12:59 AM, Petr Spacek pspa...@redhat.com wrote

Re: [Freeipa-devel] [PATCH] Stop ntpd before running ntpdate

2014-05-08 Thread Gabe Alford
Re-factored my second patch. :) Gabe On Tue, Apr 29, 2014 at 8:04 PM, Gabe Alford redhatri...@gmail.com wrote: Updated patch to not run ntpdate if ntpd is running. Gabe On Tue, Apr 29, 2014 at 8:16 AM, Gabe Alford redhatri...@gmail.comwrote: Thanks Petr! Will rework patch to just

Re: [Freeipa-devel] [PATCH][DOC] Sometimes users set bad uid ranges

2014-05-08 Thread Gabe Alford
Hello, Just wondering if there are any takers in reviewing this patch. Gabe On Mon, Apr 21, 2014 at 5:48 PM, Gabe Alford redhatri...@gmail.com wrote: Hello, Patch for the following ticket to add note in documentation about bad uid ranges: https://fedorahosted.org/freeipa/ticket

[Freeipa-devel] [PATCH 0018] ipa-client-automount should not configure nsswitch.conf

2014-05-08 Thread Gabe Alford
Hello, Came across this issue in my environment the other day and thought I would send a quick patch for review for https://fedorahosted.org/freeipa/ticket/3733 Thanks, Gabe From b6b67ce978d173a8789346a756fb67e35559b7dc Mon Sep 17 00:00:00 2001 From: Gabe redhatri...@gmail.com Date: Thu,

Re: [Freeipa-devel] [PATCH][DOC] Sometimes users set bad uid ranges

2014-05-13 Thread Gabe Alford
based on if it is Fedora or RHEL. On Tue, May 13, 2014 at 7:55 AM, Petr Viktorin pvikt...@redhat.com wrote: On 05/09/2014 04:14 AM, Gabe Alford wrote: Hello, Just wondering if there are any takers in reviewing this patch. Sorry, looks like it fell through the cracks :( AFAIK the OS

Re: [Freeipa-devel] [PATCH 0018] ipa-client-automount should not configure nsswitch.conf

2014-05-14 Thread Gabe Alford
Thanks. Updated patch. On Wed, May 14, 2014 at 3:49 AM, Petr Viktorin pvikt...@redhat.com wrote: On 05/09/2014 05:09 AM, Gabe Alford wrote: Hello, Came across this issue in my environment the other day and thought I would send a quick patch for review for https://fedorahosted.org

Re: [Freeipa-devel] [PATCH][DOC] Sometimes users set bad uid ranges

2014-05-14 Thread Gabe Alford
What are your thoughts on this? Added UID and SSSUID entities and cleaned up the original patch down to a regular paragraph. Gabe On Wed, May 14, 2014 at 2:32 AM, Petr Spacek pspa...@redhat.com wrote: On 14.5.2014 01:05, Gabe Alford wrote: I had a typo with the one of the UIDs which

[Freeipa-devel] [PATCH 0020] ipa recursively adds old backups

2014-05-29 Thread Gabe Alford
Hello, This is a patch for https://fedorahosted.org/freeipa/ticket/4331 It's a one liner that just adds an exclude to the tar command to ignore the /var/lib/ipa/backup folder. Thanks, Gabe From f1e54b57cfd07f370bc7143cfe14753f2ee145a9 Mon Sep 17 00:00:00 2001 From: Gabe redhatri...@gmail.com

[Freeipa-devel] [PATCHES 21-22] ipautil log messages and API version to env

2014-06-06 Thread Gabe Alford
Hello, Patch 21: This is a patch for https://fedorahosted.org/freeipa/ticket/3724 Patch 22: This is part of https://fedorahosted.org/freeipa/ticket/4316 and adds the API to 'ipa env' One question that I had for adding 'ipa --version' is if using the default optparse version argument would be

Re: [Freeipa-devel] [PATCHES 21-22] ipautil log messages and API version to env

2014-06-06 Thread Gabe Alford
Patch 21: Update per recommendation Patch 22: Added version option as well as updated the manpage. Thanks, Gabe On Fri, Jun 6, 2014 at 5:47 AM, Petr Viktorin pvikt...@redhat.com wrote: On 06/06/2014 08:36 AM, Gabe Alford wrote: Hello, Patch 21: This is a patch for https

[Freeipa-devel] [PATCH] Enable debug pid in smb.conf

2014-07-15 Thread Gabe Alford
Hello, Adds debug pid = yes to smb.conf when ipa-adtrust-install command is run. https://fedorahosted.org/freeipa/ticket/3485 Thanks, Gabe From 646640eff93334fe08e3aa7531293b4a4eb0d914 Mon Sep 17 00:00:00 2001 From: Gabe redhatri...@gmail.com Date: Mon, 14 Jul 2014 16:18:00 -0600 Subject:

[Freeipa-devel] [PATCH] ipa trust-add command should be interactive

2014-07-15 Thread Gabe Alford
Hello, Adds AD admin and password to interactive commands. https://fedorahosted.org/freeipa/ticket/3034 Thanks, Gabe From ddea4f0a8915a10aa8d9ac5dd2b78e2032ee335f Mon Sep 17 00:00:00 2001 From: Gabe redhatri...@gmail.com Date: Tue, 15 Jul 2014 20:34:12 -0600 Subject: [PATCH] ipa trust-add

[Freeipa-devel] [PATCH] Fix typos in dns.py

2014-07-15 Thread Gabe Alford
Hello, Fixes https://fedorahosted.org/freeipa/ticket/4429 Thanks, Gabe From 032d8e8d406ed9adb4c7e0bc948679f51cfedc09 Mon Sep 17 00:00:00 2001 From: Gabe redhatri...@gmail.com Date: Tue, 15 Jul 2014 20:54:57 -0600 Subject: [PATCH] Fix typos in dns.py https://fedorahosted.org/freeipa/ticket/4429

Re: [Freeipa-devel] [PATCH] ipa trust-add command should be interactive

2014-07-22 Thread Gabe Alford
Forgot about --trust-secret. Here is an updated patch. On Mon, Jul 21, 2014 at 2:31 AM, Jan Cholasta jchol...@redhat.com wrote: On 21.7.2014 10:28, Martin Kosek wrote: On 07/21/2014 09:56 AM, Jan Cholasta wrote: Hi, On 16.7.2014 05:48, Gabe Alford wrote: Hello, Adds AD admin

Re: [Freeipa-devel] [PATCH] ipa trust-add command should be interactive

2014-07-23 Thread Gabe Alford
Nope. Somehow in my head it felt cleaner. Updated patched attached. On Wed, Jul 23, 2014 at 1:18 AM, Jan Cholasta jchol...@redhat.com wrote: On 23.7.2014 01:01, Gabe Alford wrote: Forgot about --trust-secret. Here is an updated patch. On Mon, Jul 21, 2014 at 2:31 AM, Jan Cholasta jchol

[Freeipa-devel] [PATCH 0026][DOC] Type in sudocmd in documentation

2014-07-23 Thread Gabe Alford
Hello, Fix for https://fedorahosted.org/freeipa/ticket/4451 Thanks, Gabe From e995aa908933b31509ce02ba6a57fc20fa4fc245 Mon Sep 17 00:00:00 2001 From: Gabe redhatri...@gmail.com Date: Wed, 23 Jul 2014 16:19:18 -0600 Subject: [PATCH] Typo in upstream documentation - Fix typo with --sudocmds

Re: [Freeipa-devel] [PATCH] ipa trust-add command should be interactive

2014-07-31 Thread Gabe Alford
Right. The reason I added it in there is that I could see that in the future trust_type could be more than just 'ad' (maybe 'ipa', 'krb', etc?) which at that point I'm not sure a default makes sense. So, I thought to go ahead and add the check for future use cases so that it doesn't have to be

Re: [Freeipa-devel] [PATCH] ipa trust-add command should be interactive

2014-07-31 Thread Gabe Alford
. Martin On 07/31/2014 03:17 PM, Gabe Alford wrote: Right. The reason I added it in there is that I could see that in the future trust_type could be more than just 'ad' (maybe 'ipa', 'krb', etc?) which at that point I'm not sure a default makes sense. So, I thought to go ahead and add

Re: [Freeipa-devel] [PATCH 0030][DOC] Chapter 1 and 2 updates to documentation

2014-08-11 Thread Gabe Alford
. On 9.8.2014 04:44, Gabe Alford wrote: - Patch 0030 update DNS instructions, installation options/examples, prerequisites, replica information, etc. I started to read the patch and found following: + notetitleNOTE/title

[Freeipa-devel] [DOC] Update firewall sections from iptables to firewalld

2014-08-17 Thread Gabe Alford
Hello, This patch updates the iptables references to firewalld and links to the firewalld in the security guides for configuring ports. Thanks, Gabe From bed0669b2be86c1277f1dec65670087ab5f0148d Mon Sep 17 00:00:00 2001 From: Gabe redhatri...@gmail.com Date: Sun, 17 Aug 2014 20:12:00 -0600

Re: [Freeipa-devel] [PATCH] ipa trust-add command should be interactive

2014-08-21 Thread Gabe Alford
Hello, Just wondering if this needs to be re-ack'd. Thanks, Gabe On Thu, Jul 31, 2014 at 7:57 AM, Gabe Alford redhatri...@gmail.com wrote: Okay. Sounds good. Update patch attached. On Thu, Jul 31, 2014 at 7:18 AM, Martin Kosek mko...@redhat.com wrote: Ah, right. But I still

[Freeipa-devel] [PATCH 0032] Hardcoded lib dir in freeipa.spec

2014-09-08 Thread Gabe Alford
Hello, This patch should fix https://fedorahosted.org/freeipa/ticket/4528 Thanks, Gabe From c015531054b9392981b15617953ccc34d840f0ba Mon Sep 17 00:00:00 2001 From: Gabe redhatri...@gmail.com Date: Mon, 8 Sep 2014 07:57:50 -0600 Subject: [PATCH] Hardcoded lib dir in freeipa.spec -

[Freeipa-devel] [PATCH 0033] Remove trivial path constants

2014-09-24 Thread Gabe Alford
Hello, Patch for https://fedorahosted.org/freeipa/ticket/4399. Let me know if I missed any. Thanks, Gabe From f55ea771a16f1083ad2ea4cdf4cb81abc9e5c194 Mon Sep 17 00:00:00 2001 From: Gabe redhatri...@gmail.com Date: Wed, 24 Sep 2014 20:14:11 -0600 Subject: [PATCH] Remove trivial path constants

Re: [Freeipa-devel] [PATCH][DOC] Update Solaris Documentation, add proxy agent, and profile

2014-09-26 Thread Gabe Alford
Hello, Just wondering if we have found a reviewer for this patch set? It would be nice to have this as a part of the docs. Thanks, Gabe On Wed, Apr 16, 2014 at 5:13 AM, Petr Spacek pspa...@redhat.com wrote: On 16.4.2014 05:01, Gabe Alford wrote: The following patches update

Re: [Freeipa-devel] [PATCH][DOC] Update Solaris Documentation, add proxy agent, and profile

2014-09-26 Thread Gabe Alford
/howto on FreeIPA.org so that Solaris users can easily update it. Martin On 09/26/2014 03:08 PM, Gabe Alford wrote: Hello, Just wondering if we have found a reviewer for this patch set? It would be nice to have this as a part of the docs. Thanks, Gabe On Wed, Apr 16, 2014

Re: [Freeipa-devel] [PATCH 0033] Remove trivial path constants

2014-09-29 Thread Gabe Alford
Updated patch to fix merge conflicts from recent changes. On Wed, Sep 24, 2014 at 8:34 PM, Gabe Alford redhatri...@gmail.com wrote: Hello, Patch for https://fedorahosted.org/freeipa/ticket/4399. Let me know if I missed any. Thanks, Gabe From 51ea42d4ecb16a425ea4ff6871ed9e0d5ce67327 Mon

Re: [Freeipa-devel] [PATCH 0033] Remove trivial path constants

2014-10-03 Thread Gabe Alford
Thanks Petr. Updated patch attached. On Tue, Sep 30, 2014 at 10:59 AM, Petr Viktorin pvikt...@redhat.com wrote: On 09/30/2014 05:13 AM, Gabe Alford wrote: Updated patch to fix merge conflicts from recent changes. On Wed, Sep 24, 2014 at 8:34 PM, Gabe Alford redhatri...@gmail.com

Re: [Freeipa-devel] [PATCH 0034] Missing requires on python-dns

2014-10-07 Thread Gabe Alford
Forgot to add patch. On Tue, Oct 7, 2014 at 7:58 AM, Gabe Alford redhatri...@gmail.com wrote: Hello, Fix for https://fedorahosted.org/freeipa/ticket/4613 Thanks, Gabe From 2506c1ac5c9d92a1817bfc4fb321c206107222a0 Mon Sep 17 00:00:00 2001 From: Gabe redhatri...@gmail.com Date: Tue

[Freeipa-devel] [PATCH 0034] Missing requires on python-dns

2014-10-07 Thread Gabe Alford
Hello, Fix for https://fedorahosted.org/freeipa/ticket/4613 Thanks, Gabe ___ Freeipa-devel mailing list Freeipa-devel@redhat.com https://www.redhat.com/mailman/listinfo/freeipa-devel

Re: [Freeipa-devel] [PATCH 0034] Missing requires on python-dns

2014-10-07 Thread Gabe Alford
Done. Update patch to use python-dns = 1.11.1 On Tue, Oct 7, 2014 at 11:26 AM, Martin Basti mba...@redhat.com wrote: On 07/10/14 15:58, Gabe Alford wrote: Forgot to add patch. On Tue, Oct 7, 2014 at 7:58 AM, Gabe Alford redhatri...@gmail.com wrote: Hello, Fix for https

Re: [Freeipa-devel] [PATCH] 482 Update contributors

2014-10-17 Thread Gabe Alford
Not sure I can do this, but ACK. Gabe On Wed, Sep 24, 2014 at 6:32 AM, Martin Kosek mko...@redhat.com wrote: Add missing developers contributing to project git. Cancel Past and Occcasional section and merge the people in the right categories. Update .mailmap so that the Developer list can

[Freeipa-devel] [PATCH 0035] fix misleading Directory Manager --help output

2014-11-07 Thread Gabe Alford
Hello. Fix for https://fedorahosted.org/freeipa/ticket/4694 Thanks, Gabe From 815b0e65889ba602359278b0d07c55236a32f2b5 Mon Sep 17 00:00:00 2001 From: Gabe redhatri...@gmail.com Date: Fri, 7 Nov 2014 08:47:56 -0700 Subject: [PATCH] ipa-server-install Directory Manager help incorrect

[Freeipa-devel] [PATCH 0036] Add missing python files to Makefile

2014-11-11 Thread Gabe Alford
Hello, Fix for https://fedorahosted.org/freeipa/ticket/4700 Thanks, Gabe From b6674996bb00ac82943b90248a20114d4ca43073 Mon Sep 17 00:00:00 2001 From: Gabe redhatri...@gmail.com Date: Tue, 11 Nov 2014 07:02:15 -0700 Subject: [PATCH] Add missing python files to Makefiles

[Freeipa-devel] [PATCH 0037] ipa-managed-entries prompts for password with -p option and incorrect passowrd

2014-11-23 Thread Gabe Alford
Hello, I have a fix for https://fedorahosted.org/freeipa/ticket/4089 thanks, Gabe From 0e5937d00ed65d945c1c5ab68429ae5ccbd80936 Mon Sep 17 00:00:00 2001 From: Gabe redhatri...@gmail.com Date: Sun, 23 Nov 2014 20:33:57 -0700 Subject: [PATCH] ipa-managed-entries requires password with bad

Re: [Freeipa-devel] [PATCH 0036] Add missing python files to Makefile

2014-11-26 Thread Gabe Alford
Hello, Wondering if I could get a review. Updated patch attached. Thanks, Gabe On Tue, Nov 11, 2014 at 7:21 AM, Gabe Alford redhatri...@gmail.com wrote: Hello, Fix for https://fedorahosted.org/freeipa/ticket/4700 Thanks, Gabe From 4af61cbaafdadb1b338601699d160bd4a15db551 Mon Sep

Re: [Freeipa-devel] [PATCH 0036] Add missing python files to Makefile

2014-11-27 Thread Gabe Alford
Thanks guys. Sounds like it would be better to submit a patch that removes app_PYTHON if it is considered dead code. Gabe On Thursday, November 27, 2014, Petr Spacek pspa...@redhat.com wrote: On 27.11.2014 11:00, Martin Basti wrote: On 27/11/14 00:50, Gabe Alford wrote: Hello

[Freeipa-devel] [PATCH 0038] Update default NTP Configuration

2014-11-29 Thread Gabe Alford
Hello, Fix for https://fedorahosted.org/freeipa/ticket/4583 Thanks, Gabe From c35c3a7160cb710a39721c5a49e9fa9889c2c8ff Mon Sep 17 00:00:00 2001 From: Gabe redhatri...@gmail.com Date: Sat, 29 Nov 2014 17:53:03 -0700 Subject: [PATCH] Update default NTP configuration - Add in missing 4th default

Re: [Freeipa-devel] [PATCH 0036] Add missing python files to Makefile

2014-11-29 Thread Gabe Alford
/27/2014 04:27 PM, Gabe Alford wrote: Thanks guys. Sounds like it would be better to submit a patch that removes app_PYTHON if it is considered dead code. Gabe On Thursday, November 27, 2014, Petr Spacek pspa...@redhat.com wrote: On 27.11.2014 11:00, Martin Basti wrote: On 27/11/14

Re: [Freeipa-devel] [PATCH 0036] Add missing python files to Makefile

2014-11-29 Thread Gabe Alford
Ignore the last patch. Updated patch attached. On Sat, Nov 29, 2014 at 6:03 PM, Gabe Alford redhatri...@gmail.com wrote: This patch removes the app_PYTHON usage. Thanks, Gabe On Thu, Nov 27, 2014 at 9:40 AM, Martin Kosek mko...@redhat.com wrote: Exactly, this was the message from Martin

Re: [Freeipa-devel] [PATCH 0038] Update default NTP Configuration

2014-12-01 Thread Gabe Alford
Thanks, David. You're totally right. I am good with it. thanks, Gabe On Mon, Dec 1, 2014 at 5:20 AM, David Kupka dku...@redhat.com wrote: On 11/30/2014 02:03 AM, Gabe Alford wrote: Hello, Fix for https://fedorahosted.org/freeipa/ticket/4583 Thanks, Gabe

Re: [Freeipa-devel] [PATCH 0036] Add missing python files to Makefile

2014-12-01 Thread Gabe Alford
On Mon, Dec 1, 2014 at 6:05 AM, Martin Kosek mko...@redhat.com wrote: On 11/30/2014 03:28 AM, Gabe Alford wrote: Ignore the last patch. Updated patch attached. On Sat, Nov 29, 2014 at 6:03 PM, Gabe Alford redhatri...@gmail.com wrote: This patch removes the app_PYTHON usage. Thanks

Re: [Freeipa-devel] [PATCH 0036] Add missing python files to Makefile

2014-12-02 Thread Gabe Alford
This patch removes the changelog and Makefile.am for ipaclient as well. Thanks, Gabe On Mon, Dec 1, 2014 at 8:28 AM, Martin Kosek mko...@redhat.com wrote: On 12/01/2014 04:25 PM, Rob Crittenden wrote: Gabe Alford wrote: On Mon, Dec 1, 2014 at 6:05 AM, Martin Kosek mko...@redhat.com

[Freeipa-devel] [PATCH 0039] Add test case for unsupported arg for ipa-advise

2014-12-02 Thread Gabe Alford
Hello, I was going to try my hand at attempting a patch for ipa-tests. However in wanting to test my patch, I am not sure how to run ipa-tests to check if it works or not. Documentation is not really clear on what needs to be done to start a test and run a test. This is for

Re: [Freeipa-devel] [PATCH 0036] Add missing python files to Makefile

2014-12-03 Thread Gabe Alford
/03/2014 05:13 AM, Gabe Alford wrote: This patch removes the changelog and Makefile.am for ipaclient as well. Thanks, Gabe On Mon, Dec 1, 2014 at 8:28 AM, Martin Kosek mko...@redhat.com wrote: On 12/01/2014 04:25 PM, Rob Crittenden wrote: Gabe Alford wrote: On Mon, Dec 1, 2014

Re: [Freeipa-devel] [PATCH 0036] Add missing python files to Makefile

2014-12-04 Thread Gabe Alford
Thanks for the assistance Lukas! I have an updated patch attached. Thanks, Gabe On Wed, Dec 3, 2014 at 9:30 AM, Lukas Slebodnik lsleb...@redhat.com wrote: On (02/12/14 21:13), Gabe Alford wrote: This patch removes the changelog and Makefile.am for ipaclient as well. Thanks, Gabe

[Freeipa-devel] [PATCH 0040] Remove dependency on subscription-manager

2014-12-10 Thread Gabe Alford
Hello, Fix for https://fedorahosted.org/freeipa/ticket/4783 Thanks, Gabe From 11c688db6f741e9747716de023dcc86728e090eb Mon Sep 17 00:00:00 2001 From: Gabe redhatri...@gmail.com Date: Wed, 10 Dec 2014 07:18:35 -0700 Subject: [PATCH] Remove dependency on subscription-manager

Re: [Freeipa-devel] [PATCH 0040] Remove dependency on subscription-manager

2014-12-10 Thread Gabe Alford
Updated patch attached. Thanks, Gabe On Wed, Dec 10, 2014 at 8:02 AM, Martin Basti mba...@redhat.com wrote: On 10/12/14 15:45, Gabe Alford wrote: Hello, Fix for https://fedorahosted.org/freeipa/ticket/4783 Thanks, Gabe Hello, thanks for patch. The patch needs rebase for IPA-4-1

Re: [Freeipa-devel] [PATCH 0039] Add test case for unsupported arg for ipa-advise

2015-01-14 Thread Gabe Alford
On Wed, Jan 14, 2015 at 10:05 AM, Tomas Babej tba...@redhat.com wrote: On 01/14/2015 06:00 PM, Tomas Babej wrote: On 01/14/2015 05:37 PM, Tomas Babej wrote: On 01/14/2015 02:55 PM, Gabe Alford wrote: Hello, In looking into https://fedorahosted.org/freeipa/ticket/4029 I am

Re: [Freeipa-devel] [PATCH 0041] permission-add does not prompt for ipapermright option in interactive mode

2015-02-11 Thread Gabe Alford
: your name+email is missing in commit message, is it on purpose? And time wasn't correct in previous patch. On 11/02/15 15:06, Gabe Alford wrote: Good point. I personally was not aware of all that the API can do. Thanks Martin^2! Updated patch attached. On Tue, Feb 10, 2015 at 11:42 AM

Re: [Freeipa-devel] [PATCH 0041] permission-add does not prompt for ipapermright option in interactive mode

2015-02-11 Thread Gabe Alford
Good point. I personally was not aware of all that the API can do. Thanks Martin^2! Updated patch attached. On Tue, Feb 10, 2015 at 11:42 AM, Martin Basti mba...@redhat.com wrote: On 29/01/15 17:10, Gabe Alford wrote: Hello, Fix for https://fedorahosted.org/freeipa/ticket/4872 Thanks

Re: [Freeipa-devel] [PATCH 0039] Add test case for unsupported arg for ipa-advise

2015-02-17 Thread Gabe Alford
Hello, I was wondering if I could get a review of this patch. Thanks, Gabe On Thursday, January 29, 2015, Gabe Alford redhatri...@gmail.com wrote: Hello, Here is a patch for https://fedorahosted.org/freeipa/ticket/4029 I added test cases for valid and invalid advice

Re: [Freeipa-devel] [PATCH 0039] Add test case for unsupported arg for ipa-advise

2015-01-29 Thread Gabe Alford
Hello, Here is a patch for https://fedorahosted.org/freeipa/ticket/4029 I added test cases for valid and invalid advice. Thanks, Gabe On Wed, Jan 14, 2015 at 10:23 AM, Tomas Babej tba...@redhat.com wrote: On 01/14/2015 06:13 PM, Gabe Alford wrote: On Wed, Jan 14, 2015 at 10:05 AM

[Freeipa-devel] [PATCH 0041] permission-add does not prompt for ipapermright option in interactive mode

2015-01-29 Thread Gabe Alford
Hello, Fix for https://fedorahosted.org/freeipa/ticket/4872 Thanks, Gabe freeipa-rga-0041-permission-add-does-not-prompt-for-ipapermright-in-i.patch Description: Binary data ___ Freeipa-devel mailing list Freeipa-devel@redhat.com

Re: [Freeipa-devel] [PATCH 0039] Add test case for unsupported arg for ipa-advise

2015-01-14 Thread Gabe Alford
, 2014 at 9:21 PM, Gabe Alford redhatri...@gmail.com wrote: Hello, I was going to try my hand at attempting a patch for ipa-tests. However in wanting to test my patch, I am not sure how to run ipa-tests to check if it works or not. Documentation is not really clear on what needs to be done

[Freeipa-devel] [PATCH 0042] Typos in ipa-rmkeytab

2015-02-09 Thread Gabe Alford
Hello, Fix for https://fedorahosted.org/freeipa/ticket/4890 Thanks, Gabe From 6c760974951325419414ef4d474293c1af089004 Mon Sep 17 00:00:00 2001 From: Gabe redhatri...@gmail.com Date: Mon, 9 Feb 2015 20:44:31 -0700 Subject: [PATCH] Typos in ipa-rmkeytab options help and man page

[Freeipa-devel] [PATCH 0044] Man pages: ipa-replica-prepare can only be created on first master

2015-03-12 Thread Gabe Alford
Hello, Fix for https://fedorahosted.org/freeipa/ticket/4944. Since there seems to be plenty of time, I added it to the freeipa-4-1 branch. Thanks, Gabe From 0887f4f4595e62ce4d24f1b031418e47da7586fb Mon Sep 17 00:00:00 2001 From: Gabe redhatri...@gmail.com Date: Thu, 12 Mar 2015 07:26:34 -0600

Re: [Freeipa-devel] Time-based account policies

2015-03-10 Thread Gabe Alford
On Tue, Mar 10, 2015 at 9:51 AM, Stanislav Láznička s...@seznam.cz wrote: On 03/10/2015 04:06 PM, Jakub Hrozek wrote: On Tue, Mar 10, 2015 at 03:47:10PM +0100, Martin Kosek wrote: This is where importing iCal is helpful because it allows you to outsource the task of creating such event to

Re: [Freeipa-devel] [PATCH 0044] Man pages: ipa-replica-prepare can only be created on first master

2015-03-13 Thread Gabe Alford
On Thu, Mar 12, 2015 at 8:26 AM, Martin Kosek mko...@redhat.com wrote: On 03/12/2015 02:37 PM, Gabe Alford wrote: Hello, Fix for https://fedorahosted.org/freeipa/ticket/4944. Since there seems to be plenty of time, I added it to the freeipa-4-1 branch. Thanks Gabe! I would still

Re: [Freeipa-devel] [PATCH 0044] Man pages: ipa-replica-prepare can only be created on first master

2015-03-13 Thread Gabe Alford
Limitations is fine with me. Updated patch attached. On Fri, Mar 13, 2015 at 7:17 AM, Martin Kosek mko...@redhat.com wrote: On 03/13/2015 02:13 PM, Gabe Alford wrote: On Thu, Mar 12, 2015 at 8:26 AM, Martin Kosek mko...@redhat.com mailto:mko...@redhat.com wrote: On 03/12/2015 02:37 PM

[Freeipa-devel] [PATCH 0045] Add message for skipping NTP configuration during client install

2015-03-30 Thread Gabe Alford
Hello, With the merging of ticket 4842 https://fedorahosted.org/freeipa/ticket/4842, I believe that half of ticket 3092 https://fedorahosted.org/freeipa/ticket/3092 has been done. This patch just adds a message that says that NTP configuration was skipped which I believe should finish 3092

Re: [Freeipa-devel] [PATCH 0039] Add test case for unsupported arg for ipa-advise

2015-02-25 Thread Gabe Alford
No worries about the delay. Thanks for taking the time! Updated patch attached. Thanks, Gabe On Tue, Feb 24, 2015 at 11:03 AM, Tomas Babej tba...@redhat.com wrote: Hi Gabe, sorry for the delay. Here comes the review! 1.) All the tests fail, since the IPA master is not installed at all:

[Freeipa-devel] [PATCH 0042] ipa-replica-prepare should document ipv6 options

2015-02-25 Thread Gabe Alford
Hello, Fix for https://fedorahosted.org/freeipa/ticket/4877. I just took what was in the ticket. Thanks, Gabe freeipa-rga-0042-ipa-replica-prepare-should-document-ipv6-options.patch Description: Binary data ___ Freeipa-devel mailing list

Re: [Freeipa-devel] [PATCH 0039] Add test case for unsupported arg for ipa-advise

2015-02-26 Thread Gabe Alford
the same level of conciseness than with the separate test classes. Good luck! HTH, Tomas On 02/25/2015 03:52 PM, Gabe Alford wrote: No worries about the delay. Thanks for taking the time! Updated patch attached. Thanks, Gabe On Tue, Feb 24, 2015 at 11:03 AM, Tomas Babej tba

Re: [Freeipa-devel] [PATCH 001] Remove recommendation from ipa-adtrust-install

2015-04-20 Thread Gabe Alford
Ack from me. Thanks, Gabe On Fri, Apr 10, 2015 at 7:35 AM, Thorsten Scherf tsch...@redhat.com wrote: -- 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

Re: [Freeipa-devel] [PATCH 0082] Update python-yubico dependency version

2015-04-22 Thread Gabe Alford
Ack. Thanks, Gabe On Wed, Apr 22, 2015 at 1:45 PM, Nathaniel McCallum npmccal...@redhat.com wrote: On Tue, 2015-03-31 at 10:25 -0400, Nathaniel McCallum wrote: This change enables support for all current YubiKey hardware. Can someone please review this patch? Nathaniel -- Manage your

[Freeipa-devel] [PATCH 0046] Remove unneeded --ip-address option in ipa-adtrust-install

2015-04-27 Thread Gabe Alford
Hello, Fix for https://fedorahosted.org/freeipa/ticket/4575 Thanks, Gabe From 6c9ac52a18df8bbce33db09c16494159258ff104 Mon Sep 17 00:00:00 2001 From: Gabe redhatri...@gmail.com Date: Wed, 15 Apr 2015 09:18:58 -0600 Subject: [PATCH] Remove unneeded ip-address option in ipa-adtrust-install

[Freeipa-devel] [PATCH 0047] Unsaved changes dialog inconsistent

2015-04-27 Thread Gabe Alford
Hello, Fix for https://fedorahosted.org/freeipa/ticket/4926 Thanks, Gabe From 053f7dd53e9d1acd6dec4688ab515f138d832ef4 Mon Sep 17 00:00:00 2001 From: Gabe redhatri...@gmail.com Date: Mon, 27 Apr 2015 06:49:25 -0600 Subject: [PATCH] Unsaved changes dialog internally inconsistent - Change Update

Re: [Freeipa-devel] [PATCH 0047] Unsaved changes dialog inconsistent

2015-04-30 Thread Gabe Alford
Thanks Kyle and Petr. Update patch attached. On Wed, Apr 29, 2015 at 7:59 AM, Kyle Baker kyba...@redhat.com wrote: - Original Message - On 04/27/2015 03:03 PM, Gabe Alford wrote: Hello, Fix for https://fedorahosted.org/freeipa/ticket/4926 Thanks, Gabe

Re: [Freeipa-devel] [PATCH 0047] Unsaved changes dialog inconsistent

2015-05-05 Thread Gabe Alford
Thanks Petr. I thought I had grepped all that out. Guess I didn't do it from the top of the tree. Updated patch attached. On Tue, May 5, 2015 at 5:15 AM, Petr Vobornik pvobo...@redhat.com wrote: On 04/30/2015 07:43 PM, Gabe Alford wrote: Thanks Kyle and Petr. Update patch attached

Re: [Freeipa-devel] [PATCH 0045] Add message for skipping NTP configuration during client install

2015-04-07 Thread Gabe Alford
Stupid me. I realized that chronyd was running which messed up my testing and such (sorry about that). New patch attached that implements 'else' On Tue, Apr 7, 2015 at 2:32 AM, Martin Basti mba...@redhat.com wrote: On 02/04/15 17:47, Gabe Alford wrote: On Thu, Apr 2, 2015 at 8:59 AM, Martin

Re: [Freeipa-devel] [PATCH 0045] Add message for skipping NTP configuration during client install

2015-04-02 Thread Gabe Alford
On Thu, Apr 2, 2015 at 8:59 AM, Martin Basti mba...@redhat.com wrote: On 30/03/15 15:25, Gabe Alford wrote: Hello, With the merging of ticket 4842 https://fedorahosted.org/freeipa/ticket/4842, I believe that half of ticket 3092 https://fedorahosted.org/freeipa/ticket/3092 has been done

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

2015-06-05 Thread Gabe Alford
How should ​ https://www.redhat.com/archives/freeipa-users/2015-June/msg00116.html be handled where the user cleared out the db cache? On Fri, Jun 5, 2015 at 9:08 AM, Jakub Hrozek jhro...@redhat.com wrote: On Fri, Jun 05, 2015 at 05:03:08PM +0200, Martin Basti wrote: On 05/06/15 16:13, Gabe

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

2015-06-05 Thread Gabe Alford
Thanks. Updated patch attached. On Fri, Jun 5, 2015 at 9:53 AM, Jakub Hrozek jhro...@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

Re: [Freeipa-devel] [PATCH 0050] Fix client ca.crt to match the server's cert

2015-06-16 Thread Gabe Alford
I know you guys are busy. Bump for review. Thanks, Gabe On Tue, May 26, 2015 at 8:16 AM, Gabe Alford redhatri...@gmail.com wrote: Hello, Fix for https://fedorahosted.org/freeipa/ticket/3809 Thanks, Gabe -- Manage your subscription for the Freeipa-devel mailing list: https

[Freeipa-devel] [PATCH 0048] fix ipa help command output errors

2015-05-22 Thread Gabe Alford
Hello, This should fix https://fedorahosted.org/freeipa/ticket/3584, and as requested in the ticket, this should also fix https://fedorahosted.org/freeipa/ticket/2284 Thanks, Gabe From 3d4e7b60287f30e70455facb0035fa30df913c34 Mon Sep 17 00:00:00 2001 From: Gabe redhatri...@gmail.com Date: Fri,

Re: [Freeipa-devel] [PATCH 0048] fix ipa help command output errors

2015-05-22 Thread Gabe Alford
On Fri, May 22, 2015 at 9:01 AM, Martin Basti mba...@redhat.com wrote: On 22/05/15 16:08, Gabe Alford wrote: Hello, This should fix https://fedorahosted.org/freeipa/ticket/3584, and as requested in the ticket, this should also fix https://fedorahosted.org/freeipa/ticket/2284 Thanks

[Freeipa-devel] [PATCH 0050] Fix client ca.crt to match the server's cert

2015-05-26 Thread Gabe Alford
Hello, Fix for https://fedorahosted.org/freeipa/ticket/3809 Thanks, Gabe From b6a852f82e9335ac04fb5d9b96f31013fb2a3bdb Mon Sep 17 00:00:00 2001 From: Gabe redhatri...@gmail.com Date: Tue, 26 May 2015 08:06:12 -0600 Subject: [PATCH] Fix client ca.crt to match the server's cert

  1   2   >