Re: [Freeipa-devel] [PATCHES] backport of Firefox extension to FreeIPA 2.2

2012-10-23 Thread Martin Kosek
On 10/22/2012 05:38 PM, Petr Vobornik wrote: On 10/22/2012 05:01 PM, Martin Kosek wrote: On 10/17/2012 12:02 PM, Petr Vobornik wrote: On 10/16/2012 06:10 PM, Endi Sukma Dewata wrote: On 10/12/2012 5:55 AM, Petr Viktorin wrote: On 10/11/2012 02:55 PM, Petr Vobornik wrote: This bunch of

Re: [Freeipa-devel] Experimental patchwork server

2012-10-23 Thread Petr Viktorin
On 10/23/2012 03:04 AM, Simo Sorce wrote: On Mon, 2012-10-22 at 18:03 -0400, Simo Sorce wrote: Hello fellow developers, We have discussed for a while among us about how to improve patch tracking for review purposes. Various method have been discussed for quite some time now (including gerrit

Re: [Freeipa-devel] What is platform services' is_installed() supposed to do?

2012-10-23 Thread Petr Viktorin
On 10/22/2012 08:48 PM, Alexander Bokovoy wrote: On Mon, 22 Oct 2012, Petr Viktorin wrote: For example, ipapython.platform.systemd.SystemdService.is_installed(). What is it supposed to do? Is it checking if the package is installed, or if IPΑ's instance of the service is configured and ready to

[Freeipa-devel] Announcing FreeIPA v2.2.1 Release

2012-10-23 Thread Martin Kosek
The FreeIPA team is proud to announce a bug fix release FreeIPA v2.2.1. It can be downloaded from http://www.freeipa.org/page/Downloads. A build is on the way to updates-testing for Fedora 17. Fedora 15 and 16 are not supported by FreeIPA 2.2.1 due to missing dependencies. == Highlights in

Re: [Freeipa-devel] Experimental patchwork server

2012-10-23 Thread Petr Vobornik
On 10/23/2012 11:18 AM, Petr Viktorin wrote: On 10/23/2012 03:04 AM, Simo Sorce wrote: On Mon, 2012-10-22 at 18:03 -0400, Simo Sorce wrote: Hello fellow developers, We have discussed for a while among us about how to improve patch tracking for review purposes. Various method have been

Re: [Freeipa-devel] Experimental patchwork server

2012-10-23 Thread Petr Vobornik
On 10/23/2012 01:29 PM, Petr Vobornik wrote: On 10/23/2012 11:18 AM, Petr Viktorin wrote: On 10/23/2012 03:04 AM, Simo Sorce wrote: On Mon, 2012-10-22 at 18:03 -0400, Simo Sorce wrote: Hello fellow developers, We have discussed for a while among us about how to improve patch tracking for

Re: [Freeipa-devel] [PATCH] 1064 Improve error messages during ipa-replica-manage

2012-10-23 Thread Petr Viktorin
On 10/17/2012 05:56 PM, Rob Crittenden wrote: Make some of the errors in ipa-replica-manage clearer. See ticket for more reproduction details. rob Looks good, ACK. -- Petr³ ___ Freeipa-devel mailing list Freeipa-devel@redhat.com

Re: [Freeipa-devel] Experimental patchwork server

2012-10-23 Thread Simo Sorce
On Tue, 2012-10-23 at 11:18 +0200, Petr Viktorin wrote: On 10/23/2012 03:04 AM, Simo Sorce wrote: On Mon, 2012-10-22 at 18:03 -0400, Simo Sorce wrote: Hello fellow developers, We have discussed for a while among us about how to improve patch tracking for review purposes. Various method

Re: [Freeipa-devel] What is platform services' is_installed() supposed to do?

2012-10-23 Thread Simo Sorce
On Tue, 2012-10-23 at 12:58 +0200, Petr Viktorin wrote: On 10/22/2012 08:48 PM, Alexander Bokovoy wrote: On Mon, 22 Oct 2012, Petr Viktorin wrote: For example, ipapython.platform.systemd.SystemdService.is_installed(). What is it supposed to do? Is it checking if the package is installed,

Re: [Freeipa-devel] Experimental patchwork server

2012-10-23 Thread John Dennis
On 10/23/2012 09:00 AM, Simo Sorce wrote: I strongly suggest you use git-send-email instead of thunderbird, it makes everything a lot faster, see the instructions I sent in my followup email. I wrote a python script to manage my patch submissions a while ago which might be useful to folks,

Re: [Freeipa-devel] Experimental patchwork server

2012-10-23 Thread Simo Sorce
On Tue, 2012-10-23 at 09:53 -0400, John Dennis wrote: On 10/23/2012 09:00 AM, Simo Sorce wrote: I strongly suggest you use git-send-email instead of thunderbird, it makes everything a lot faster, see the instructions I sent in my followup email. I wrote a python script to manage my patch

Re: [Freeipa-devel] Experimental patchwork server

2012-10-23 Thread Petr Viktorin
I'll try this a bit before spamming the list again. Just one note: you need to `yum install git-email` to get the send-email subcommand. On 10/23/2012 03:53 PM, John Dennis wrote: On 10/23/2012 09:00 AM, Simo Sorce wrote: I strongly suggest you use git-send-email instead of thunderbird, it

[Freeipa-devel] [PATCH] 1065 Close connection after each request

2012-10-23 Thread Rob Crittenden
Close connection after each request, avoid NSS shutdown problem. The unit tests were failing when executed against an Apache server in F-18 due to dangling references causing NSS shutdown to fail, and potentially other places like adding host keys during client enrollment. rob From

[Freeipa-devel] [RFE] Warnings and client capabilities (Was: [PATCH] 0062 Don't crash when server returns extra output)

2012-10-23 Thread Petr Viktorin
Here is a draft design document for ticket 2732. Please comment on both the feature itself, and on how to write design documents. Petr¹, please add how the UI should handle this. == Ticket summary ([https://fedorahosted.org/freeipa/ticket/2732 #2732]) == Currently the only way to display a

Re: [Freeipa-devel] [RFE] Warnings and client capabilities (Was: [PATCH] 0062 Don't crash when server returns extra output)

2012-10-23 Thread Dmitri Pal
On 10/23/2012 11:57 AM, Petr Viktorin wrote: Here is a draft design document for ticket 2732. Please comment on both the feature itself, and on how to write design documents. Petr¹, please add how the UI should handle this. == Ticket summary ([https://fedorahosted.org/freeipa/ticket/2732

Re: [Freeipa-devel] [PATCH] 1064 Improve error messages during ipa-replica-manage

2012-10-23 Thread Rob Crittenden
Petr Viktorin wrote: On 10/17/2012 05:56 PM, Rob Crittenden wrote: Make some of the errors in ipa-replica-manage clearer. See ticket for more reproduction details. rob Looks good, ACK. pushed to master and ipa-3-0 ___ Freeipa-devel mailing

Re: [Freeipa-devel] Experimental patchwork server

2012-10-23 Thread John Dennis
On 10/23/2012 10:17 AM, Simo Sorce wrote: Does this script send separate emails for each patch ? Otherwise it will cause issues for the patchwork server. As I said it can only handle one patch per email, that's why git-send-email is handy however git-send-email will not respect the rules we

[Freeipa-devel] [PATCH 0/1] Check for leftover certmonger certs

2012-10-23 Thread rcritten
From: Rob Crittenden rcrit...@redhat.com Rob Crittenden (1): After unininstall see if certmonger is still tracking any of our certs. install/tools/ipa-server-install | 10 +- ipapython/certmonger.py | 33 + 2 files changed, 42

[Freeipa-devel] [PATCH 1/1] After unininstall see if certmonger is still tracking any of our certs.

2012-10-23 Thread rcritten
From: Rob Crittenden rcrit...@redhat.com Rather than providing a list of nicknames I'm going to look at the NSS databases directly. Anything in there is suspect and this will help future-proof us. certmonger may be tracking other certificates but we only care about a subset of them, so don't

Re: [Freeipa-devel] [PATCH 1/1] After unininstall see if certmonger is still tracking any of our certs.

2012-10-23 Thread Rob Crittenden
rcrit...@redhat.com wrote: From: Rob Crittenden rcrit...@redhat.com Rather than providing a list of nicknames I'm going to look at the NSS databases directly. Anything in there is suspect and this will help future-proof us. certmonger may be tracking other certificates but we only care about a

Re: [Freeipa-devel] [PATCH 1/1] After unininstall see if certmonger is still tracking any of our certs.

2012-10-23 Thread Simo Sorce
On Tue, 2012-10-23 at 16:41 -0400, Rob Crittenden wrote: rcrit...@redhat.com wrote: From: Rob Crittenden rcrit...@redhat.com Rather than providing a list of nicknames I'm going to look at the NSS databases directly. Anything in there is suspect and this will help future-proof us.

Re: [Freeipa-devel] [PATCH 0/1] Check for leftover certmonger certs

2012-10-23 Thread Simo Sorce
On Tue, 2012-10-23 at 16:39 -0400, rcrit...@redhat.com wrote: From: Rob Crittenden rcrit...@redhat.com Rob Crittenden (1): After unininstall see if certmonger is still tracking any of our certs. install/tools/ipa-server-install | 10 +- ipapython/certmonger.py |

Re: [Freeipa-devel] [PATCH] 0093 ipa-replica-install: Use configured IPA DNS servers in forward/reverse resolution check

2012-10-23 Thread Rob Crittenden
Petr Viktorin wrote: Fix a regression in one of my earlier patches. https://fedorahosted.org/freeipa/ticket/3194 ACK, pushed to master and ipa-3-0 rob ___ Freeipa-devel mailing list Freeipa-devel@redhat.com

Re: [Freeipa-devel] [PATCH] 0092 Make sure the CA is running when starting services

2012-10-23 Thread Rob Crittenden
Petr Viktorin wrote: On 10/19/2012 05:42 PM, Rob Crittenden wrote: Petr Viktorin wrote: https://fedorahosted.org/freeipa/ticket/3084 See ticket commit message. Please tell me of a better way to extend the Services. What's interesting is that usually the CA is running right after the ports

Re: [Freeipa-devel] [PATCH 0017] Improve error message in ipa-replica-manage

2012-10-23 Thread Rob Crittenden
Tomas Babej wrote: On 10/19/2012 09:55 AM, Petr Viktorin wrote: On 10/18/2012 08:01 PM, Rob Crittenden wrote: Tomas Babej wrote: On 10/02/2012 03:55 PM, Rob Crittenden wrote: Tomas Babej wrote: Hi, When executing ipa-replica-manage connect to an unknown or irrelevant master, we now print a

Re: [Freeipa-devel] [PATCH] 222 Fixed incorrect link to browser config after session expiration

2012-10-23 Thread Endi Sukma Dewata
On 10/17/2012 5:07 AM, Petr Vobornik wrote: Fixed typo in message placeholder. https://fedorahosted.org/freeipa/ticket/3187 ACK. -- Endi S. Dewata ___ Freeipa-devel mailing list Freeipa-devel@redhat.com