[SSSD] [sssd PR#75][opened] Add configuirable max payload size limit of a secret

URL: https://github.com/SSSD/sssd/pull/75 Author: fidencio Title: #75: Add configuirable max payload size limit of a secret Action: opened PR body: """ This series may conflict with PR53[0]. So either one or another will need to be rebased after the first one gets merged. [0]:

[SSSD] [sssd PR#75][synchronized] Add configuirable max payload size limit of a secret

URL: https://github.com/SSSD/sssd/pull/75 Author: fidencio Title: #75: Add configuirable max payload size limit of a secret Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/75/head:pr75 git checkout pr75 From

[SSSD] [sssd PR#39][synchronized] RESPONDER: Enable sudoRule in case insen. domains (1.13)

URL: https://github.com/SSSD/sssd/pull/39 Author: celestian Title: #39: RESPONDER: Enable sudoRule in case insen. domains (1.13) Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/39/head:pr39 git checkout pr39 From

[SSSD] Re: [PATCH SET] SYSDB: Adding message to inform about cache

On (07/11/16 14:42), Petr Cech wrote: >Hi all, > >after chat with Lukas I attached only first two patches. Author of the third >one is Lukas and I am not sure if he is finished. (There was question of >LD_PRELOAD.) > >Regards > >-- >Petr^4 Čech >From c67ccc872eb5dacc98f626c10740424cef205334 Mon

[SSSD] [sssd PR#75][synchronized] Add configuirable max payload size limit of a secret

URL: https://github.com/SSSD/sssd/pull/75 Author: fidencio Title: #75: Add configuirable max payload size limit of a secret Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/75/head:pr75 git checkout pr75 From

[SSSD] [sssd PR#75][comment] Add configuirable max payload size limit of a secret

URL: https://github.com/SSSD/sssd/pull/75 Title: #75: Add configuirable max payload size limit of a secret fidencio commented: """ Thanks for catching this, @lslebodn. I've ran the CI locally and it has passed. I was wondering what was my mistake and `git stash pop` gave me the answer.

[SSSD] [sssd PR#76][comment] AD: Fix crash in AD subdomain reinit

URL: https://github.com/SSSD/sssd/pull/76 Title: #76: AD: Fix crash in AD subdomain reinit centos-ci commented: """ Can one of the admins verify this patch? """ See the full comment at https://github.com/SSSD/sssd/pull/76#issuecomment-259315385 ___

[SSSD] [sssd PR#76][comment] AD: Fix crash in AD subdomain reinit

URL: https://github.com/SSSD/sssd/pull/76 Title: #76: AD: Fix crash in AD subdomain reinit centos-ci commented: """ Can one of the admins verify this patch? """ See the full comment at https://github.com/SSSD/sssd/pull/76#issuecomment-259315383 ___

[SSSD] [sssd PR#76][comment] AD: Fix crash in AD subdomain reinit

URL: https://github.com/SSSD/sssd/pull/76 Title: #76: AD: Fix crash in AD subdomain reinit celestian commented: """ ok to test """ See the full comment at https://github.com/SSSD/sssd/pull/76#issuecomment-259348408 ___ sssd-devel mailing list --

[SSSD] [sssd PR#76][comment] AD: Fix crash in AD subdomain reinit

URL: https://github.com/SSSD/sssd/pull/76 Title: #76: AD: Fix crash in AD subdomain reinit sumit-bose commented: """ PR #74 has the same fix and a similar forx for the IPA provider as well. I'm fine with revoking #74 if the fix for the IPA provider is added here as well. """ See the full

[SSSD] [sssd PR#73][synchronized] AD_PROVIDER: Enabled subdomains (1.13)

URL: https://github.com/SSSD/sssd/pull/73 Author: celestian Title: #73: AD_PROVIDER: Enabled subdomains (1.13) Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/73/head:pr73 git checkout pr73 From

[SSSD] Re: Design discussion: Fleet Commander integration

On Fri, Oct 7, 2016 at 10:22 AM, Jakub Hrozek wrote: > On Thu, Oct 06, 2016 at 06:38:23PM +0200, Sumit Bose wrote: >> On Thu, Oct 06, 2016 at 04:41:10PM +0200, Jakub Hrozek wrote: >> > Hi, >> > >> > with Alexander's help, I wrote up a design page about how SSSD should >> >

[SSSD] about letting the responder choose the sysdb optimization level

Hi, I would like to ask for opinions about: https://fedorahosted.org/sssd/ticket/3126 The basic idea is that the responder would choose what kind of optimization would the back end perform when saving the sysdb entries. Requests that just return information might choose to optimize very

[SSSD] [sssd PR#70][+Changes requested] check_duplicate: check name member before using it

URL: https://github.com/SSSD/sssd/pull/70 Title: #70: check_duplicate: check name member before using it Label: +Changes requested ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to

[SSSD] [sssd PR#70][comment] check_duplicate: check name member before using it

URL: https://github.com/SSSD/sssd/pull/70 Title: #70: check_duplicate: check name member before using it lslebodn commented: """ I would appreciate integration test for the crash. Feel free to extend `test_extra_attribute_already_exists` or create new one. """ See the full comment at

[SSSD] [sssd PR#70][-Accepted] check_duplicate: check name member before using it

URL: https://github.com/SSSD/sssd/pull/70 Title: #70: check_duplicate: check name member before using it Label: -Accepted ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to

[SSSD] [sssd PR#71][comment] MONITOR: Do not set up watchdog for monitor

URL: https://github.com/SSSD/sssd/pull/71 Title: #71: MONITOR: Do not set up watchdog for monitor lslebodn commented: """ ACK LS """ See the full comment at https://github.com/SSSD/sssd/pull/71#issuecomment-259090619 ___ sssd-devel mailing list --

[SSSD] [sssd PR#71][closed] MONITOR: Do not set up watchdog for monitor

URL: https://github.com/SSSD/sssd/pull/71 Author: jhrozek Title: #71: MONITOR: Do not set up watchdog for monitor Action: closed To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/71/head:pr71 git checkout pr71

[SSSD] [sssd PR#71][+Pushed] MONITOR: Do not set up watchdog for monitor

URL: https://github.com/SSSD/sssd/pull/71 Title: #71: MONITOR: Do not set up watchdog for monitor Label: +Pushed ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org

[SSSD] [sssd PR#71][comment] MONITOR: Do not set up watchdog for monitor

URL: https://github.com/SSSD/sssd/pull/71 Title: #71: MONITOR: Do not set up watchdog for monitor jhrozek commented: """ master: fbe6644aa28d93f492434950680c5618eb567712 sssd-1-14: 2d88a121918e800b266d018d43dad9bd374b10a7 """ See the full comment at

[SSSD] [sssd PR#73][-Changes requested] AD_PROVIDER: Enabled subdomains (1.13)

URL: https://github.com/SSSD/sssd/pull/73 Title: #73: AD_PROVIDER: Enabled subdomains (1.13) Label: -Changes requested ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to

[SSSD] [sssd PR#73][comment] AD_PROVIDER: Enabled subdomains (1.13)

URL: https://github.com/SSSD/sssd/pull/73 Title: #73: AD_PROVIDER: Enabled subdomains (1.13) celestian commented: """ I pushed new version with ```(cherry picked from commit...``` (It is done manually) """ See the full comment at https://github.com/SSSD/sssd/pull/73#issuecomment-259075858

[SSSD] Re: [PATCHES] views: properly override group member names - 1.13 backport

On (08/11/16 10:19), Lukas Slebodnik wrote: >On (24/10/16 16:59), Sumit Bose wrote: >>Hi, >> >>please find attached a backport to 1.13 of the given patch set including >>fixes from Lukas. >> >>It's intention is to fix https://fedorahosted.org/sssd/ticket/3118 /

[SSSD] [sssd PR#43][comment] RESPONDER: Enable sudoRule in case insen. domains​ (1.15)

URL: https://github.com/SSSD/sssd/pull/43 Title: #43: RESPONDER: Enable sudoRule in case insen. domains​ (1.15) pbrezina commented: """ Ok, I'm fine with this patch due to the indexing issue. """ See the full comment at https://github.com/SSSD/sssd/pull/43#issuecomment-259093311

[SSSD] Re: [PATCHES] views: properly override group member names

On Tue, Nov 08, 2016 at 08:14:20AM +0100, Lukas Slebodnik wrote: > On (26/07/16 22:05), Sumit Bose wrote: > >On Tue, Jul 26, 2016 at 06:06:48PM +0200, Jakub Hrozek wrote: > >> On Tue, Jul 26, 2016 at 05:25:11PM +0200, Jakub Hrozek wrote: > >> > On Tue, Jul 26, 2016 at 01:51:56PM +0200, Sumit Bose

[SSSD] Re: [PATCHES] views: properly override group member names - 1.13 backport

On (24/10/16 16:59), Sumit Bose wrote: >Hi, > >please find attached a backport to 1.13 of the given patch set including >fixes from Lukas. > >It's intention is to fix https://fedorahosted.org/sssd/ticket/3118 / >https://bugzilla.redhat.com/show_bug.cgi?id=1367802 without introducing >a new patch.

[SSSD] [sssd PR#70][comment] check_duplicate: check name member before using it

URL: https://github.com/SSSD/sssd/pull/70 Title: #70: check_duplicate: check name member before using it celestian commented: """ I tested manually with IPA provider. It works and I was informed about attribute colision: ``` [sdap_extend_map] (0x0010): Attribute entryUSN (abc in LDAP) is

[SSSD] [sssd PR#70][+Accepted] check_duplicate: check name member before using it

URL: https://github.com/SSSD/sssd/pull/70 Title: #70: check_duplicate: check name member before using it Label: +Accepted ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to

[SSSD] [sssd PR#70][comment] check_duplicate: check name member before using it

URL: https://github.com/SSSD/sssd/pull/70 Title: #70: check_duplicate: check name member before using it sumit-bose commented: """ @lslebodn, the patch adds a new unit-test test_extra_opts_empty_name() which covers the crash. What would be the benefit of checking it in the integration tests

[SSSD] [sssd PR#69][comment] krb5: Use command line arguments instead env vars for krb5_child

URL: https://github.com/SSSD/sssd/pull/69 Title: #69: krb5: Use command line arguments instead env vars for krb5_child celestian commented: """ Code looks good to me. CI tests passed: http://sssd-ci.duckdns.org/logs/job/56/34/summary.html => ACK """ See the full comment at

[SSSD] [sssd PR#69][+Accepted] krb5: Use command line arguments instead env vars for krb5_child

URL: https://github.com/SSSD/sssd/pull/69 Title: #69: krb5: Use command line arguments instead env vars for krb5_child Label: +Accepted ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to

[SSSD] [sssd PR#39][synchronized] RESPONDER: Enable sudoRule in case insen. domains (1.13)

URL: https://github.com/SSSD/sssd/pull/39 Author: celestian Title: #39: RESPONDER: Enable sudoRule in case insen. domains (1.13) Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/39/head:pr39 git checkout pr39 From

[SSSD] [sssd PR#73][+Accepted] AD_PROVIDER: Enabled subdomains (1.13)

URL: https://github.com/SSSD/sssd/pull/73 Title: #73: AD_PROVIDER: Enabled subdomains (1.13) Label: +Accepted ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org

[SSSD] [sssd PR#43][comment] RESPONDER: Enable sudoRule in case insen. domains​ (1.15)

URL: https://github.com/SSSD/sssd/pull/43 Title: #43: RESPONDER: Enable sudoRule in case insen. domains​ (1.15) jhrozek commented: """ master: f4a1046bb88d7a0ab3617e49ae94bfa849d10645 23637e2fd2b1fe42bdd2335893a11ac8016f56bc sssd-1-14: 143b1dcbbe865a139616a22b139e19bd772e46f0

[SSSD] [sssd PR#43][-Accepted] RESPONDER: Enable sudoRule in case insen. domains​ (1.15)

URL: https://github.com/SSSD/sssd/pull/43 Title: #43: RESPONDER: Enable sudoRule in case insen. domains​ (1.15) Label: -Accepted ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to

[SSSD] [sssd PR#43][+Pushed] RESPONDER: Enable sudoRule in case insen. domains​ (1.15)

URL: https://github.com/SSSD/sssd/pull/43 Title: #43: RESPONDER: Enable sudoRule in case insen. domains​ (1.15) Label: +Pushed ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to

[SSSD] [sssd PR#43][closed] RESPONDER: Enable sudoRule in case insen. domains​ (1.15)

URL: https://github.com/SSSD/sssd/pull/43 Author: celestian Title: #43: RESPONDER: Enable sudoRule in case insen. domains​ (1.15) Action: closed To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/43/head:pr43 git checkout pr43

[SSSD] [sssd PR#39][comment] RESPONDER: Enable sudoRule in case insen. domains (1.13)

URL: https://github.com/SSSD/sssd/pull/39 Title: #39: RESPONDER: Enable sudoRule in case insen. domains (1.13) celestian commented: """ I pushed new version, only one difference -- I fix cherry-pick pointer. The patch works without ```sudoUserAlias``` but it still adds fq names to sudoUser. Is

[SSSD] [sssd PR#39][comment] RESPONDER: Enable sudoRule in case insen. domains (1.13)

URL: https://github.com/SSSD/sssd/pull/39 Title: #39: RESPONDER: Enable sudoRule in case insen. domains (1.13) celestian commented: """ Since we have patch set for 1.15 pushed I will prepare proper cherry picking. """ See the full comment at

[SSSD] [sssd PR#73][closed] AD_PROVIDER: Enabled subdomains (1.13)

URL: https://github.com/SSSD/sssd/pull/73 Author: celestian Title: #73: AD_PROVIDER: Enabled subdomains (1.13) Action: closed To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/73/head:pr73 git checkout pr73

[SSSD] [sssd PR#74][opened] IPA/AD: check auth ctx before using it

URL: https://github.com/SSSD/sssd/pull/74 Author: sumit-bose Title: #74: IPA/AD: check auth ctx before using it Action: opened PR body: """ In e6b6b9fa79c67d7d2698bc7e33d2e2f6bb53d483 a feature was introduced to set the 'canonicalize' option in the system-wide Kerberos configuration according

[SSSD] [sssd PR#69][comment] krb5: Use command line arguments instead env vars for krb5_child

URL: https://github.com/SSSD/sssd/pull/69 Title: #69: krb5: Use command line arguments instead env vars for krb5_child lslebodn commented: """ On (08/11/16 03:33), celestian wrote: >Code looks good to me. CI tests passed: >http://sssd-ci.duckdns.org/logs/job/56/34/summary.html > >=> ACK just

[SSSD] [sssd PR#39][comment] RESPONDER: Enable sudoRule in case insen. domains (1.13)

URL: https://github.com/SSSD/sssd/pull/39 Title: #39: RESPONDER: Enable sudoRule in case insen. domains (1.13) pbrezina commented: """ So what is the current plan here? """ See the full comment at https://github.com/SSSD/sssd/pull/39#issuecomment-259118986

[SSSD] [sssd PR#73][comment] AD_PROVIDER: Enabled subdomains (1.13)

URL: https://github.com/SSSD/sssd/pull/73 Title: #73: AD_PROVIDER: Enabled subdomains (1.13) lslebodn commented: """ On (08/11/16 03:06), Jakub Hrozek wrote: >I tested the patches with a setup that consists of three domains >(win.trust.test, child.win.trust.test and sibling.win.trust.test).

[SSSD] [sssd PR#39][comment] RESPONDER: Enable sudoRule in case insen. domains (1.13)

URL: https://github.com/SSSD/sssd/pull/39 Title: #39: RESPONDER: Enable sudoRule in case insen. domains (1.13) jhrozek commented: """ On Tue, Nov 08, 2016 at 04:13:43AM -0800, celestian wrote: > I pushed new version, only one difference -- I fix cherry-pick pointer. > The patch works without

[SSSD] [sssd PR#73][+Pushed] AD_PROVIDER: Enabled subdomains (1.13)

URL: https://github.com/SSSD/sssd/pull/73 Title: #73: AD_PROVIDER: Enabled subdomains (1.13) Label: +Pushed ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org

[SSSD] [sssd PR#53][comment] Fixes in the config API related to secrets responder

URL: https://github.com/SSSD/sssd/pull/53 Title: #53: Fixes in the config API related to secrets responder fidencio commented: """ @jhrozek: For the first patch the tests are correct. @lslebodn also complained that [secrets/users/] could be a valid case in the way the code is in git right now

[SSSD] [sssd PR#39][comment] RESPONDER: Enable sudoRule in case insen. domains (1.13)

URL: https://github.com/SSSD/sssd/pull/39 Title: #39: RESPONDER: Enable sudoRule in case insen. domains (1.13) jhrozek commented: """ On Tue, Nov 08, 2016 at 05:06:41AM -0800, celestian wrote: > Yes, the second patch explicitly qualifies the names. I don't know if there > is possibility to add

[SSSD] [sssd PR#65][comment] Fixing of nitpicks

URL: https://github.com/SSSD/sssd/pull/65 Title: #65: Fixing of nitpicks lslebodn commented: """ On (08/11/16 05:26), celestian wrote: >celestian commented on this pull request. >> @@ -269,6 +269,10 @@ static void >> rdp_message_send_and_reply_done(DBusPendingCall *pending, > sbus_req =

[SSSD] [sssd PR#53][comment] Fixes in the config API related to secrets responder

URL: https://github.com/SSSD/sssd/pull/53 Title: #53: Fixes in the config API related to secrets responder lslebodn commented: """ On (08/11/16 05:15), fidencio wrote: >@jhrozek: >For the first patch the tests are correct. @lslebodn also complained that >[secrets/users/] could be a valid case

[SSSD] [sssd PR#39][comment] RESPONDER: Enable sudoRule in case insen. domains (1.13)

URL: https://github.com/SSSD/sssd/pull/39 Title: #39: RESPONDER: Enable sudoRule in case insen. domains (1.13) celestian commented: """ Yes, the second patch explicitly qualifies the names. I don't know if there is possibility to add wrong domain to the given user name this way. That's the

[SSSD] [sssd PR#53][synchronized] Fixes in the config API related to secrets responder

URL: https://github.com/SSSD/sssd/pull/53 Author: fidencio Title: #53: Fixes in the config API related to secrets responder Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/53/head:pr53 git checkout pr53 From

[SSSD] [sssd PR#65][comment] Fixing of nitpicks

URL: https://github.com/SSSD/sssd/pull/65 Title: #65: Fixing of nitpicks celestian commented: """ ```UTIL: Removing of never read value``` ``` @@ -1104,7 +1104,6 @@ bool sss_krb5_realm_has_proxy(const char *realm) kerr = profile_get_values(profile, profile_path, ); if (kerr ==