[PATCH] Changes to allow SSSD to be used for access control with a machine account belonging to a domain controller.
Hi,
I'm resending a patch for ticket #2870 on behalf of the original reporter who also kindly submitted a patch.
The patch looks good to me, as soon as we fix CI, I'll submit it as well and I think we can push it..
On Mon, Nov 23, 2015 at 11:53:46AM +0100, Jakub Hrozek wrote:
Hi,
I'm resending a patch for ticket #2870 on behalf of the original reporter who also kindly submitted a patch.
The patch looks good to me, as soon as we fix CI, I'll submit it as well and I think we can push it..
From 79e3fe4052b0bdb0ca6b9c62686cacb581df0044 Mon Sep 17 00:00:00 2001 From: Mathieu Deaudelin-Lemay contrib@mdeaudelin.net Date: Fri, 20 Nov 2015 11:56:11 -0500 Subject: [PATCH] Changes to allow SSSD to be used for access control with a machine account belonging to a domain controller.
CI passed: http://sssd-ci.duckdns.org/logs/job/33/92/summary.html
Conceptually the patch was already agreen on by Simo in the ticket. The code looks good to me as well, so if there is no opposition, I'll push it.
On Thu, Nov 26, 2015 at 03:47:13PM +0100, Jakub Hrozek wrote:
On Mon, Nov 23, 2015 at 11:53:46AM +0100, Jakub Hrozek wrote:
Hi,
I'm resending a patch for ticket #2870 on behalf of the original reporter who also kindly submitted a patch.
The patch looks good to me, as soon as we fix CI, I'll submit it as well and I think we can push it..
From 79e3fe4052b0bdb0ca6b9c62686cacb581df0044 Mon Sep 17 00:00:00 2001 From: Mathieu Deaudelin-Lemay contrib@mdeaudelin.net Date: Fri, 20 Nov 2015 11:56:11 -0500 Subject: [PATCH] Changes to allow SSSD to be used for access control with a machine account belonging to a domain controller.
CI passed: http://sssd-ci.duckdns.org/logs/job/33/92/summary.html
Conceptually the patch was already agreen on by Simo in the ticket. The code looks good to me as well, so if there is no opposition, I'll push it.
I checked that the value of UAC_SERVER_TRUST_ACCOUNT is the right one. Since you already run the CI I'd say ACK.
bye, Sumit
sssd-devel mailing list sssd-devel@lists.fedorahosted.org https://lists.fedorahosted.org/admin/lists/sssd-devel@lists.fedorahosted.org
On Thu, Nov 26, 2015 at 04:46:07PM +0100, Sumit Bose wrote:
On Thu, Nov 26, 2015 at 03:47:13PM +0100, Jakub Hrozek wrote:
On Mon, Nov 23, 2015 at 11:53:46AM +0100, Jakub Hrozek wrote:
Hi,
I'm resending a patch for ticket #2870 on behalf of the original reporter who also kindly submitted a patch.
The patch looks good to me, as soon as we fix CI, I'll submit it as well and I think we can push it..
From 79e3fe4052b0bdb0ca6b9c62686cacb581df0044 Mon Sep 17 00:00:00 2001 From: Mathieu Deaudelin-Lemay contrib@mdeaudelin.net Date: Fri, 20 Nov 2015 11:56:11 -0500 Subject: [PATCH] Changes to allow SSSD to be used for access control with a machine account belonging to a domain controller.
CI passed: http://sssd-ci.duckdns.org/logs/job/33/92/summary.html
Conceptually the patch was already agreen on by Simo in the ticket. The code looks good to me as well, so if there is no opposition, I'll push it.
I checked that the value of UAC_SERVER_TRUST_ACCOUNT is the right one. Since you already run the CI I'd say ACK.
bye, Sumit
Thank you for checking, pushed with your RB: 5c129880ae10c80b4f79cb2994e9d127dc6dfbef
On (26/11/15 16:51), Jakub Hrozek wrote:
On Thu, Nov 26, 2015 at 04:46:07PM +0100, Sumit Bose wrote:
On Thu, Nov 26, 2015 at 03:47:13PM +0100, Jakub Hrozek wrote:
On Mon, Nov 23, 2015 at 11:53:46AM +0100, Jakub Hrozek wrote:
Hi,
I'm resending a patch for ticket #2870 on behalf of the original reporter who also kindly submitted a patch.
The patch looks good to me, as soon as we fix CI, I'll submit it as well and I think we can push it..
From 79e3fe4052b0bdb0ca6b9c62686cacb581df0044 Mon Sep 17 00:00:00 2001 From: Mathieu Deaudelin-Lemay contrib@mdeaudelin.net Date: Fri, 20 Nov 2015 11:56:11 -0500 Subject: [PATCH] Changes to allow SSSD to be used for access control with a machine account belonging to a domain controller.
CI passed: http://sssd-ci.duckdns.org/logs/job/33/92/summary.html
Conceptually the patch was already agreen on by Simo in the ticket. The code looks good to me as well, so if there is no opposition, I'll push it.
I checked that the value of UAC_SERVER_TRUST_ACCOUNT is the right one. Since you already run the CI I'd say ACK.
bye, Sumit
Thank you for checking, pushed with your RB: 5c129880ae10c80b4f79cb2994e9d127dc6dfbef
Is there a reason why this patch could not be in stable branch? Timo asked it on #sssd.
LS
On Fri, Jul 01, 2016 at 08:55:05AM +0200, Lukas Slebodnik wrote:
On (26/11/15 16:51), Jakub Hrozek wrote:
On Thu, Nov 26, 2015 at 04:46:07PM +0100, Sumit Bose wrote:
On Thu, Nov 26, 2015 at 03:47:13PM +0100, Jakub Hrozek wrote:
On Mon, Nov 23, 2015 at 11:53:46AM +0100, Jakub Hrozek wrote:
Hi,
I'm resending a patch for ticket #2870 on behalf of the original reporter who also kindly submitted a patch.
The patch looks good to me, as soon as we fix CI, I'll submit it as well and I think we can push it..
From 79e3fe4052b0bdb0ca6b9c62686cacb581df0044 Mon Sep 17 00:00:00 2001 From: Mathieu Deaudelin-Lemay contrib@mdeaudelin.net Date: Fri, 20 Nov 2015 11:56:11 -0500 Subject: [PATCH] Changes to allow SSSD to be used for access control with a machine account belonging to a domain controller.
CI passed: http://sssd-ci.duckdns.org/logs/job/33/92/summary.html
Conceptually the patch was already agreen on by Simo in the ticket. The code looks good to me as well, so if there is no opposition, I'll push it.
I checked that the value of UAC_SERVER_TRUST_ACCOUNT is the right one. Since you already run the CI I'd say ACK.
bye, Sumit
Thank you for checking, pushed with your RB: 5c129880ae10c80b4f79cb2994e9d127dc6dfbef
Is there a reason why this patch could not be in stable branch? Timo asked it on #sssd.
No, just push it.
sssd-devel@lists.fedorahosted.org
-
Jakub Hrozek -
Lukas Slebodnik -
Sumit Bose