Re: [Freeipa-devel] [PATCH] 0012 Fix session cookies

2016-07-22 Thread Petr Spacek
On 22.7.2016 10:08, Florence Blanc-Renaud wrote: > Hi, > > please find attached a patch related to session cookies used by IPA API. > > https://fedorahosted.org/freeipa/ticket/5984 ACK -- Petr^2 Spacek -- Manage your subscription for the Freeipa-devel mailing list:

Re: [Freeipa-devel] [PATCH 0195] Create indexes for krbCanonicalName attribute

2016-07-22 Thread Martin Babinsky
On 07/22/2016 02:37 PM, thierry bordaz wrote: Hi Martin, The patch looks good. Just a question krbPrincipalName is caseExactIA5Match but is also indexed caseIgnoreIA5Match. Do you think it would be need for krbCanonicalName as well ? thanks thierry On 07/22/2016 01:27 PM, Martin Babinsky

Re: [Freeipa-devel] [PATCH 0555] AVC: use copy during instalation to keep SELinux context valid

2016-07-22 Thread Petr Spacek
On 21.7.2016 19:49, Martin Basti wrote: > https://fedorahosted.org/freeipa/ticket/6111 > > I was able to reproduce this locally with vagrant, but I haven't been able to > reproduce this in LAB, I don't know where differences are (cloud vs desktop > fedora?) > > > Patch attached. ACK --

[Freeipa-devel] [PATCH 0432] Prevent crash while reloading an invalid DNS zone

2016-07-22 Thread Petr Spacek
Hello, Prevent crash while reloading an invalid DNS zone. The crash happened under these circumstances: - create a DNS zone (test.) with NS record relative to this zone (ns.test.) - make sure that name pointed to by NS record does not have any A/ records - restart BIND - add missing A/

Re: [Freeipa-devel] [PATCH] 0012 Fix session cookies

2016-07-22 Thread Martin Babinsky
On 07/22/2016 03:47 PM, Petr Spacek wrote: On 22.7.2016 10:08, Florence Blanc-Renaud wrote: Hi, please find attached a patch related to session cookies used by IPA API. https://fedorahosted.org/freeipa/ticket/5984 ACK Pushed to: master: bc7eb99a2959980c1abf31f77610cec2f098744b ipa-4-3:

Re: [Freeipa-devel] [PATCH 0555] AVC: use copy during instalation to keep SELinux context valid

2016-07-22 Thread Martin Basti
On 22.07.2016 16:38, Martin Babinsky wrote: On 07/22/2016 03:49 PM, Petr Spacek wrote: On 21.7.2016 19:49, Martin Basti wrote: https://fedorahosted.org/freeipa/ticket/6111 I was able to reproduce this locally with vagrant, but I haven't been able to reproduce this in LAB, I don't know

Re: [Freeipa-devel] [PATCH 0555] AVC: use copy during instalation to keep SELinux context valid

2016-07-22 Thread Martin Babinsky
On 07/22/2016 03:49 PM, Petr Spacek wrote: On 21.7.2016 19:49, Martin Basti wrote: https://fedorahosted.org/freeipa/ticket/6111 I was able to reproduce this locally with vagrant, but I haven't been able to reproduce this in LAB, I don't know where differences are (cloud vs desktop fedora?)

Re: [Freeipa-devel] [PATCH 0555] AVC: use copy during instalation to keep SELinux context valid

2016-07-22 Thread Martin Babinsky
On 07/22/2016 04:45 PM, Martin Basti wrote: On 22.07.2016 16:38, Martin Babinsky wrote: On 07/22/2016 03:49 PM, Petr Spacek wrote: On 21.7.2016 19:49, Martin Basti wrote: https://fedorahosted.org/freeipa/ticket/6111 I was able to reproduce this locally with vagrant, but I haven't been able

Re: [Freeipa-devel] [PATCH 0432] Prevent crash while reloading an invalid DNS zone

2016-07-22 Thread Martin Basti
On 22.07.2016 16:50, Petr Spacek wrote: Hello, Prevent crash while reloading an invalid DNS zone. The crash happened under these circumstances: - create a DNS zone (test.) with NS record relative to this zone (ns.test.) - make sure that name pointed to by NS record does not have any A/

Re: [Freeipa-devel] [PATCH 0553] CI tests: improve log collecting in tests

2016-07-22 Thread Martin Basti
On 20.07.2016 17:41, Martin Basti wrote: On 19.07.2016 17:05, Martin Basti wrote: On 19.07.2016 16:18, Martin Basti wrote: Patch attached. self-NACK, my assumptions were wrong, this doesn't work if any of log files do not exist updated patches attached Please note, that in

[Freeipa-devel] [PATCH] 0012 Fix session cookies

2016-07-22 Thread Florence Blanc-Renaud
Hi, please find attached a patch related to session cookies used by IPA API. https://fedorahosted.org/freeipa/ticket/5984 >From 7cc3b19c21a66483bff14e7a9c84b29c969673de Mon Sep 17 00:00:00 2001 From: Florence Blanc-Renaud Date: Thu, 21 Jul 2016 16:54:43 +0200 Subject: [PATCH]

Re: [Freeipa-devel] [PATCH 0195] Create indexes for krbCanonicalName attribute

2016-07-22 Thread thierry bordaz
Hi Martin, The patch looks good. Just a question krbPrincipalName is caseExactIA5Match but is also indexed caseIgnoreIA5Match. Do you think it would be need for krbCanonicalName as well ? thanks thierry On 07/22/2016 01:27 PM, Martin Babinsky wrote:

Re: [Freeipa-devel] [PATCH 0029][Tests] Adding authentication test to trust test suite

2016-07-22 Thread Lenka Doudova
On 07/20/2016 02:28 PM, Martin Babinsky wrote: On 07/19/2016 10:41 AM, Lenka Doudova wrote: Hi, this patch adds authentication test (specifically "kinit -E ipauser@IPADOMAIN") to basic trust test suite, as requested by Sumit. Intended to be applied after my patches 25.4 and 26.3 (already

Re: [Freeipa-devel] [PATCH 0556] host-del: fix behavior of --updatedns and PTR records

2016-07-22 Thread Petr Spacek
On 21.7.2016 20:01, Martin Basti wrote: > https://fedorahosted.org/freeipa/ticket/6060 > > > Patch attached. ACK -- Petr^2 Spacek -- Manage your subscription for the Freeipa-devel mailing list: https://www.redhat.com/mailman/listinfo/freeipa-devel Contribute to FreeIPA:

[Freeipa-devel] [PATCH 0195] Create indexes for krbCanonicalName attribute

2016-07-22 Thread Martin Babinsky
https://fedorahosted.org/freeipa/ticket/6100 -- Martin^3 Babinsky From 618f68499cd3d9537ef0947132155fad6fa61da4 Mon Sep 17 00:00:00 2001 From: Martin Babinsky Date: Fri, 22 Jul 2016 13:02:38 +0200 Subject: [PATCH] Create indexes for krbCanonicalName attribute

[Freeipa-devel] [PATCH 0557] DNS: fix update-system-records unpacking error

2016-07-22 Thread Martin Basti
https://fedorahosted.org/freeipa/ticket/6117 Patch attached From dddbb97def78d6888b7127ab342d7d1fdc9856ad Mon Sep 17 00:00:00 2001 From: Martin Basti Date: Fri, 22 Jul 2016 13:32:31 +0200 Subject: [PATCH] DNS Locations: fix update-system-records unpacking error Method

Re: [Freeipa-devel] [PATCH 0556] host-del: fix behavior of --updatedns and PTR records

2016-07-22 Thread Martin Basti
On 22.07.2016 13:24, Petr Spacek wrote: On 21.7.2016 20:01, Martin Basti wrote: https://fedorahosted.org/freeipa/ticket/6060 Patch attached. ACK Pushed to master: 8aba4f63439853d524e8b394b7919159c86d2a08 -- Manage your subscription for the Freeipa-devel mailing list:

Re: [Freeipa-devel] [PATCH 0149] help: Add dnsserver commands to help topic 'dns'

2016-07-22 Thread Martin Basti
On 22.07.2016 13:07, Petr Spacek wrote: On 15.7.2016 12:05, David Kupka wrote: On 12/07/16 12:54, Petr Spacek wrote: Hello, help: Add dnsserver commands to help topic 'dns' https://bugzilla.redhat.com/show_bug.cgi?id=1353888 Hi! Your patch turns dnsserver topic to a subtopic of dns

Re: [Freeipa-devel] [PATCH 0149] help: Add dnsserver commands to help topic 'dns'

2016-07-22 Thread Petr Spacek
On 15.7.2016 12:05, David Kupka wrote: > On 12/07/16 12:54, Petr Spacek wrote: >> Hello, >> >> help: Add dnsserver commands to help topic 'dns' >> >> https://bugzilla.redhat.com/show_bug.cgi?id=1353888 >> > Hi! > > Your patch turns dnsserver topic to a subtopic of dns topic. I'm sorry I gave >

Re: [Freeipa-devel] [PATCH 0002][Tests] Small fix for dns_plugin tests

2016-07-22 Thread Ganna Kaihorodova
Hello! Thank you! Best regards, Ganna Kaihorodova Associate Software Quality Engineer - Original Message - From: "Martin Basti" To: "Ganna Kaihorodova" Cc: freeipa-devel@redhat.com Sent: Thursday, July 21, 2016 8:03:53 PM Subject: Re:

Re: [Freeipa-devel] [PATCH 0557] DNS: fix update-system-records unpacking error

2016-07-22 Thread Nikhil Dehadrai
ACK. On 07/22/2016 05:09 PM, Martin Basti wrote: https://fedorahosted.org/freeipa/ticket/6117 Patch attached -- Manage your subscription for the Freeipa-devel mailing list: https://www.redhat.com/mailman/listinfo/freeipa-devel Contribute to FreeIPA:

Re: [Freeipa-devel] [PATCH 0557] DNS: fix update-system-records unpacking error

2016-07-22 Thread Martin Basti
On 22.07.2016 15:12, Nikhil Dehadrai wrote: ACK. On 07/22/2016 05:09 PM, Martin Basti wrote: https://fedorahosted.org/freeipa/ticket/6117 Patch attached Thanks Pushed to master: 524719f420fa331b3a1d53d5d8bebdfee39c8371 -- Manage your subscription for the Freeipa-devel mailing list: