Re: [Freeipa-devel] I plan to delete my FreeIPA COPR repos

On 17.06.2016 10:33, Martin Kosek wrote: On 05/13/2016 01:43 PM, Martin Kosek wrote: Hi all, When we were starting building FreeIPA in the Fedora COPR service [1], the service did not support the organizations as it can do now and we did the official repos in my personal name space [2] as I

Re: [Freeipa-devel] [DESIGN] Lightweight CA renewal

On 17.6.2016 09:34, Fraser Tweedale wrote: On Mon, May 09, 2016 at 09:35:06AM +0200, Jan Cholasta wrote: Hi, On 6.5.2016 08:01, Fraser Tweedale wrote: Hullo all, FreeIPA Lightweight CAs implementation is progressing well. The remaining big unknown in the design is how to do renewal. I have

Re: [Freeipa-devel] [PATCH 0503-0513, 0515-0519, 0520-0528] DNS locations

On 17.6.2016 12:25, Martin Basti wrote: > > > On 17.06.2016 08:46, Petr Spacek wrote: >> On 16.6.2016 22:14, Martin Basti wrote: >>> >>> On 16.06.2016 15:59, Petr Spacek wrote: On 16.6.2016 13:57, Martin Basti wrote: > On 16.06.2016 12:09, Petr Spacek wrote: >> On 15.6.2016 17:24,

Re: [Freeipa-devel] [PATCH 0503-0513, 0515-0519, 0520-0528] DNS locations

On 17.06.2016 15:17, Petr Spacek wrote: On 17.6.2016 12:25, Martin Basti wrote: On 17.06.2016 08:46, Petr Spacek wrote: On 16.6.2016 22:14, Martin Basti wrote: On 16.06.2016 15:59, Petr Spacek wrote: On 16.6.2016 13:57, Martin Basti wrote: On 16.06.2016 12:09, Petr Spacek wrote: On

Re: [Freeipa-devel] [PATCH 0153-0158] move ipa-replica-manage del functionality into server-del

On 06/16/2016 12:45 PM, Martin Basti wrote: On 15.06.2016 15:29, Martin Babinsky wrote: On 06/15/2016 10:30 AM, Jan Cholasta wrote: Hi, On 12.6.2016 17:31, Martin Babinsky wrote: On 06/09/2016 08:12 PM, Martin Babinsky wrote: These patches expand `server_del` to a full fledged IPA master

Re: [Freeipa-devel] [PATCH 0048] Remove sys.exit() from installer modules

On 06/17/2016 07:45 AM, Petr Spacek wrote: On 16.6.2016 17:33, Stanislav Laznicka wrote: Hello, This patch removes most sys.exits() from installer modules and scripts and replaces them with ScriptError. I only left sys.exits at places where the user decides yes/no on continuation of the

Re: [Freeipa-devel] [PATCH 0503-0513, 0515-0519, 0520-0528] DNS locations

On 16.6.2016 22:14, Martin Basti wrote: > > > On 16.06.2016 15:59, Petr Spacek wrote: >> On 16.6.2016 13:57, Martin Basti wrote: >>> >>> On 16.06.2016 12:09, Petr Spacek wrote: On 15.6.2016 17:24, Petr Spacek wrote: > On 15.6.2016 15:45, Martin Basti wrote: >> On 15.06.2016 14:52,

Re: [Freeipa-devel] [PATCH 0048] Remove sys.exit() from installer modules

On 17.6.2016 08:43, Stanislav Laznicka wrote: > On 06/17/2016 07:45 AM, Petr Spacek wrote: >> On 16.6.2016 17:33, Stanislav Laznicka wrote: >>> Hello, >>> >>> This patch removes most sys.exits() from installer modules and scripts and >>> replaces them with ScriptError. I only left sys.exits at

Re: [Freeipa-devel] [PATCH] 0069 renew_ca_cert: bootstrap api with in_server=True

On Fri, Jun 17, 2016 at 08:35:45AM +0200, Jan Cholasta wrote: > Hi, > > On 17.6.2016 06:55, Fraser Tweedale wrote: > > Attached patch fixes https://fedorahosted.org/freeipa/ticket/5968 > > This should be fixed for all the restart scripts, not just renew_ca_cert. > Updated patch attached. From

[Freeipa-devel] [PATCH] 0070..0071 Fix replica installation from IPA v4.2

The attached patches fix https://fedorahosted.org/freeipa/ticket/5963 Thanks Milan for reporting. Cheers, Fraser From aef64476fef6e3a850d7a4375dbbff098a9439ba Mon Sep 17 00:00:00 2001 From: Fraser Tweedale Date: Fri, 17 Jun 2016 10:57:32 +1000 Subject: [PATCH 70/71] Split

Re: [Freeipa-devel] [PATCH 0042] Removed dead code from LDAPRemoveReverseMember

On 06/14/2016 04:40 PM, Jan Cholasta wrote: On 14.6.2016 16:35, Martin Basti wrote: On 14.06.2016 16:37, Jan Cholasta wrote: On 14.6.2016 16:29, Martin Basti wrote: On 08.06.2016 14:17, Stanislav Laznicka wrote: On 06/07/2016 10:42 AM, Martin Basti wrote: On 07.06.2016 10:43, Jan Cholasta

Re: [Freeipa-devel] [PATCH 0048] Remove sys.exit() from installer modules

On 06/17/2016 08:48 AM, Petr Spacek wrote: On 17.6.2016 08:43, Stanislav Laznicka wrote: On 06/17/2016 07:45 AM, Petr Spacek wrote: On 16.6.2016 17:33, Stanislav Laznicka wrote: Hello, This patch removes most sys.exits() from installer modules and scripts and replaces them with ScriptError.

Re: [Freeipa-devel] [PATCH 0048] Remove sys.exit() from installer modules

On 06/17/2016 09:51 AM, Petr Vobornik wrote: On 17.6.2016 09:24, Stanislav Laznicka wrote: On 06/17/2016 08:48 AM, Petr Spacek wrote: On 17.6.2016 08:43, Stanislav Laznicka wrote: On 06/17/2016 07:45 AM, Petr Spacek wrote: On 16.6.2016 17:33, Stanislav Laznicka wrote: Hello, This patch

Re: [Freeipa-devel] [Test][Patch-0047] Added a test for Ticket N 5964

One more test was added to the patch-0048 On 06/17/2016 09:43 AM, Oleg Fayans wrote: > Fixed a bug in the previous patch, automated 2 more testcases from > http://www.freeipa.org/page/V4/Manage_replication_topology_4_4/Test_Plan > > On 06/16/2016 04:46 PM, Oleg Fayans wrote: >> >> >> > > >

Re: [Freeipa-devel] [PATCH 0161] Fix listing of enabled roles in `server-find`

On 17.06.2016 16:21, Martin Babinsky wrote: There is an issue with outputting enabled roles in server-find regarding our lovely `--raw`, `--all` and `--no-member` options. This quick patch fixes that. ACK Added ticket to commit message https://fedorahosted.org/freeipa/ticket/5181

Re: [Freeipa-devel] [DESIGN] Lightweight CA renewal

On Fri, Jun 17, 2016 at 03:21:07PM +0200, Jan Cholasta wrote: > On 17.6.2016 09:34, Fraser Tweedale wrote: > > On Mon, May 09, 2016 at 09:35:06AM +0200, Jan Cholasta wrote: > > > Hi, > > > > > > On 6.5.2016 08:01, Fraser Tweedale wrote: > > > > Hullo all, > > > > > > > > FreeIPA Lightweight CAs

Re: [Freeipa-devel] [PATCH 0153-0158] move ipa-replica-manage del functionality into server-del

On 17.06.2016 16:18, Martin Babinsky wrote: On 06/17/2016 03:56 PM, Martin Babinsky wrote: On 06/16/2016 12:45 PM, Martin Basti wrote: On 15.06.2016 15:29, Martin Babinsky wrote: On 06/15/2016 10:30 AM, Jan Cholasta wrote: Hi, On 12.6.2016 17:31, Martin Babinsky wrote: On 06/09/2016

Re: [Freeipa-devel] [PATCH 0153-0158] move ipa-replica-manage del functionality into server-del

On 06/17/2016 03:56 PM, Martin Babinsky wrote: On 06/16/2016 12:45 PM, Martin Basti wrote: On 15.06.2016 15:29, Martin Babinsky wrote: On 06/15/2016 10:30 AM, Jan Cholasta wrote: Hi, On 12.6.2016 17:31, Martin Babinsky wrote: On 06/09/2016 08:12 PM, Martin Babinsky wrote: These patches

[Freeipa-devel] [PATCH 0161] Fix listing of enabled roles in `server-find`

There is an issue with outputting enabled roles in server-find regarding our lovely `--raw`, `--all` and `--no-member` options. This quick patch fixes that. -- Martin^3 Babinsky From 24eaec37476afdd9019492b5b4b79cef9c853a4b Mon Sep 17 00:00:00 2001 From: Martin Babinsky

Re: [Freeipa-devel] [PATCH] 0069 renew_ca_cert: bootstrap api with in_server=True

On 17.6.2016 08:53, Fraser Tweedale wrote: > On Fri, Jun 17, 2016 at 08:35:45AM +0200, Jan Cholasta wrote: >> Hi, >> >> On 17.6.2016 06:55, Fraser Tweedale wrote: >>> Attached patch fixes https://fedorahosted.org/freeipa/ticket/5968 >> >> This should be fixed for all the restart scripts, not just

Re: [Freeipa-devel] [PATCH 0503-0513, 0515-0519, 0520-0528] DNS locations

On 17.06.2016 18:00, Petr Spacek wrote: On 17.6.2016 17:05, Martin Basti wrote: On 17.06.2016 15:17, Petr Spacek wrote: On 17.6.2016 12:25, Martin Basti wrote: On 17.06.2016 08:46, Petr Spacek wrote: On 16.6.2016 22:14, Martin Basti wrote: On 16.06.2016 15:59, Petr Spacek wrote: On

Re: [Freeipa-devel] bind-dyndb-ldap reviews

On 17.6.2016 13:58, Martin Sehnoutka wrote: > ACK Thanks, pushed to master: 75d1dbabc3893deac89d36d56c9ff86d88ed3b55 Make task selection in syncrepl_update() deterministic. 431b8c47b4c6a27a933c842ed875cb5155f968fb Avoid redundant sync_task_add() calls. 62b19b4d423164e4a0dfb00a13358611c86ed789

Re: [Freeipa-devel] [PATCH 0503-0513, 0515-0519, 0520-0528] DNS locations

On 17.06.2016 15:17, Petr Spacek wrote: On 17.6.2016 12:25, Martin Basti wrote: On 17.06.2016 08:46, Petr Spacek wrote: On 16.6.2016 22:14, Martin Basti wrote: On 16.06.2016 15:59, Petr Spacek wrote: On 16.6.2016 13:57, Martin Basti wrote: On 16.06.2016 12:09, Petr Spacek wrote: On

Re: [Freeipa-devel] [PATCH 0503-0513, 0515-0519, 0520-0528] DNS locations

On 17.6.2016 17:05, Martin Basti wrote: > > > On 17.06.2016 15:17, Petr Spacek wrote: >> On 17.6.2016 12:25, Martin Basti wrote: >>> >>> On 17.06.2016 08:46, Petr Spacek wrote: On 16.6.2016 22:14, Martin Basti wrote: > On 16.06.2016 15:59, Petr Spacek wrote: >> On 16.6.2016 13:57,

[Freeipa-devel] [PATCH 0105] Remove unused locking "context manager"

Just dropping tiny piece of no longer used code. -- David Kupka From c493fd213690e10960b805900c62504a63740d43 Mon Sep 17 00:00:00 2001 From: David Kupka Date: Fri, 17 Jun 2016 15:12:29 +0200 Subject: [PATCH] Remove unused locking "context manager" Class ods_db_lock is unused

Re: [Freeipa-devel] [PATCH 0105] Remove unused locking "context manager"

On 17.06.2016 16:31, David Kupka wrote: Just dropping tiny piece of no longer used code. ACK Pushed to master: 45bb2ad045654c020fe6ac4e77ed2741cd35d717 Only master, feel free to open tickets if backport is needed -- Manage your subscription for the Freeipa-devel mailing list:

Re: [Freeipa-devel] [PATCH] 0069 renew_ca_cert: bootstrap api with in_server=True

Hi, On 17.6.2016 06:55, Fraser Tweedale wrote: Attached patch fixes https://fedorahosted.org/freeipa/ticket/5968 This should be fixed for all the restart scripts, not just renew_ca_cert. Thanks, Fraser Honza -- Jan Cholasta -- Manage your subscription for the Freeipa-devel mailing

Re: [Freeipa-devel] [DESIGN] Lightweight CA renewal

On Mon, May 09, 2016 at 09:35:06AM +0200, Jan Cholasta wrote: > Hi, > > On 6.5.2016 08:01, Fraser Tweedale wrote: > > Hullo all, > > > > FreeIPA Lightweight CAs implementation is progressing well. The > > remaining big unknown in the design is how to do renewal. I have > > put my ideas into

Re: [Freeipa-devel] [Test][Patch-0047] Added a test for Ticket N 5964

Fixed a bug in the previous patch, automated 2 more testcases from http://www.freeipa.org/page/V4/Manage_replication_topology_4_4/Test_Plan On 06/16/2016 04:46 PM, Oleg Fayans wrote: > > > -- Oleg Fayans Quality Engineer FreeIPA team RedHat. From 2b087b1dc64500d58e72296f287aabd82cbd011c Mon

Re: [Freeipa-devel] [PATCH 0048] Remove sys.exit() from installer modules

On 17.6.2016 09:24, Stanislav Laznicka wrote: > On 06/17/2016 08:48 AM, Petr Spacek wrote: >> On 17.6.2016 08:43, Stanislav Laznicka wrote: >>> On 06/17/2016 07:45 AM, Petr Spacek wrote: On 16.6.2016 17:33, Stanislav Laznicka wrote: > Hello, > > This patch removes most sys.exits()

Re: [Freeipa-devel] [PATCH 0048] Remove sys.exit() from installer modules

On 17.6.2016 09:24, Stanislav Laznicka wrote: > On 06/17/2016 08:48 AM, Petr Spacek wrote: >> On 17.6.2016 08:43, Stanislav Laznicka wrote: >>> On 06/17/2016 07:45 AM, Petr Spacek wrote: On 16.6.2016 17:33, Stanislav Laznicka wrote: > Hello, > > This patch removes most sys.exits()

Re: [Freeipa-devel] I plan to delete my FreeIPA COPR repos

On 05/13/2016 01:43 PM, Martin Kosek wrote: > Hi all, > > When we were starting building FreeIPA in the Fedora COPR service [1], the > service did not support the organizations as it can do now and we did the > official repos in my personal name space [2] as I was the common denominator > for the

Re: [Freeipa-devel] [PATCH 0048] Remove sys.exit() from installer modules

On 17.6.2016 12:12, Stanislav Laznicka wrote: > On 06/17/2016 09:51 AM, Petr Vobornik wrote: >> On 17.6.2016 09:24, Stanislav Laznicka wrote: >>> On 06/17/2016 08:48 AM, Petr Spacek wrote: On 17.6.2016 08:43, Stanislav Laznicka wrote: > On 06/17/2016 07:45 AM, Petr Spacek wrote: >> On

Re: [Freeipa-devel] [PATCH 0048] Remove sys.exit() from installer modules

On 06/17/2016 01:01 PM, Petr Vobornik wrote: On 17.6.2016 12:12, Stanislav Laznicka wrote: On 06/17/2016 09:51 AM, Petr Vobornik wrote: On 17.6.2016 09:24, Stanislav Laznicka wrote: On 06/17/2016 08:48 AM, Petr Spacek wrote: On 17.6.2016 08:43, Stanislav Laznicka wrote: On 06/17/2016 07:45

Re: [Freeipa-devel] bind-dyndb-ldap reviews

ACK Martin Sehnoutka - Original Message - From: "Petr Spacek" To: msehn...@redhat.com Sent: Wednesday, June 15, 2016 4:25:15 PM Subject: bind-dyndb-ldap reviews Hello, please review patches mentioned below. Do not hesitate to contact me whenever you have a