sssd-devel December 2021

sssd-devel@lists.fedorahosted.org

2 participants
83 discussions

[sssd PR#5883][comment] Various fixes related forest and site name discovery
by alexey-tikhonov 06 Dec '21

06 Dec '21

URL: https://github.com/SSSD/sssd/pull/5883 Title: #5883: Various fixes related forest and site name discovery alexey-tikhonov commented: """ @pbrezina, it's not the only problem it is twice in master. Another problem is commit in master doesn't match patch in PR (new file `src/man/include/dns_options.xml` absent in master). """ See the full comment at https://github.com/SSSD/sssd/pull/5883#issuecomment-986705230

1 0

[sssd PR#5883][comment] Various fixes related forest and site name discovery
by pbrezina 06 Dec '21

06 Dec '21

URL: https://github.com/SSSD/sssd/pull/5883 Title: #5883: Various fixes related forest and site name discovery pbrezina commented: """ On 12/3/21 20:42, Alexey A Tikhonov wrote: > only the last patch fails because the man page entry for > |dns_resolver_server_timeout| is missing in 1-16, the option itself > exists. So what would be the preferred solution, drop the hunk with > the man page change or add a patch with the man page entries first? > > Since option exists, I would vote to backport man page patch. > > Funny enough, the entry was added twice in master 43b9b09 > <https://github.com/SSSD/sssd/commit/43b9b0922aa24a03ea466c673646d5e3079403fe> > and 7190f6b > <https://github.com/SSSD/sssd/commit/7190f6b5d5b523e18e8c320df73c44b082c4d99b>, > maybe one of the commits was meant for 1-16? > > Well, something really weird happened. > Looking at https://github.com/SSSD/sssd/pull/5645/files > <https://github.com/SSSD/sssd/pull/5645/files> : new file > |src/man/include/dns_options.xml| was meant to be introduced. > But actual commits are quite different: #5645 (comment) > <https://github.com/SSSD/sssd/pull/5645#issuecomment-847150345> > > @pbrezina <https://github.com/pbrezina>, could you please take a look? Who knows, but it is clearly twice in master and not once in 1.16. Sumit, would you to mind to fix it together with backport PR? """ See the full comment at https://github.com/SSSD/sssd/pull/5883#issuecomment-986703792

1 0

[sssd PR#5883][comment] Various fixes related forest and site name discovery
by alexey-tikhonov 04 Dec '21

04 Dec '21

URL: https://github.com/SSSD/sssd/pull/5883 Title: #5883: Various fixes related forest and site name discovery alexey-tikhonov commented: """ > only the last patch fails because the man page entry for `dns_resolver_server_timeout` is missing in 1-16, the option itself exists. So what would be the preferred solution, drop the hunk with the man page change or add a patch with the man page entries first? Since option exists, I would vote to backport man page patch. > Funny enough, the entry was added twice in master [43b9b09](https://github.com/SSSD/sssd/commit/43b9b0922aa24a03ea466c673646d5… and [7190f6b](https://github.com/SSSD/sssd/commit/7190f6b5d5b523e18e8c320df73c44…, maybe one of the commits was meant for 1-16? Well, something really weird happened. Looking at https://github.com/SSSD/sssd/pull/5645/files : new file ` src/man/include/dns_options.xml` was meant to be introduced. But actual commits are quite different: https://github.com/SSSD/sssd/pull/5645#issuecomment-847150345 @pbrezina, could you please take a look? """ See the full comment at https://github.com/SSSD/sssd/pull/5883#issuecomment-985781202

1 0

[sssd PR#5883][comment] Various fixes related forest and site name discovery
by sumit-bose 03 Dec '21

03 Dec '21

URL: https://github.com/SSSD/sssd/pull/5883 Title: #5883: Various fixes related forest and site name discovery sumit-bose commented: """ > Since this claims to fix https://bugzilla.redhat.com/show_bug.cgi?id=2006866 , this also need to be backported to 1-16 branch. > > @sumit-bose, does this apply cleanly or is dedicated PR required? Hi, only the last patch fails because the man page entry for `dns_resolver_server_timeout` is missing in 1-16, the option itself exists. So what would be the preferred solution, drop the hunk with the man page change or add a patch with the man page entries first? Funny enough, the entry was added twice in master 43b9b0922a and 7190f6b5d5, maybe one of the commits was meant for 1-16? bye, Sumit """ See the full comment at https://github.com/SSSD/sssd/pull/5883#issuecomment-985601568

1 0

[sssd PR#5883][comment] Various fixes related forest and site name discovery
by alexey-tikhonov 03 Dec '21

03 Dec '21

URL: https://github.com/SSSD/sssd/pull/5883 Title: #5883: Various fixes related forest and site name discovery alexey-tikhonov commented: """ Since this claims to fix https://bugzilla.redhat.com/show_bug.cgi?id=2006866 , this also need to be backported to 1-16 branch. @sumit-bose, does this apply cleanly or is dedicated PR required? """ See the full comment at https://github.com/SSSD/sssd/pull/5883#issuecomment-985516674

1 0

[sssd PR#5883][comment] Various fixes related forest and site name discovery
by alexey-tikhonov 03 Dec '21

03 Dec '21

URL: https://github.com/SSSD/sssd/pull/5883 Title: #5883: Various fixes related forest and site name discovery alexey-tikhonov commented: """ Since this claims to fix https://bugzilla.redhat.com/show_bug.cgi?id=2006866 , this also need to ba backported to 1-16 branch. @sumit-bose, does this apply cleanly or is dedicated PR required? """ See the full comment at https://github.com/SSSD/sssd/pull/5883#issuecomment-985516674

1 0

[sssd PR#5883][closed] Various fixes related forest and site name discovery
by pbrezina 03 Dec '21

03 Dec '21

URL: https://github.com/SSSD/sssd/pull/5883 Author: sumit-bose Title: #5883: Various fixes related forest and site name discovery Action: closed To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/5883/head:pr5883 git checkout pr5883

1 0

[sssd PR#5883][-Ready to push] Various fixes related forest and site name discovery
by pbrezina 03 Dec '21

03 Dec '21

URL: https://github.com/SSSD/sssd/pull/5883 Title: #5883: Various fixes related forest and site name discovery Label: -Ready to push

1 0

[sssd PR#5883][-Accepted] Various fixes related forest and site name discovery
by pbrezina 03 Dec '21

03 Dec '21

URL: https://github.com/SSSD/sssd/pull/5883 Title: #5883: Various fixes related forest and site name discovery Label: -Accepted

1 0

[sssd PR#5883][comment] Various fixes related forest and site name discovery
by pbrezina 03 Dec '21

03 Dec '21

URL: https://github.com/SSSD/sssd/pull/5883 Title: #5883: Various fixes related forest and site name discovery pbrezina commented: """ Pushed PR: https://github.com/SSSD/sssd/pull/5883 * `master` * c0941810fc3c3d74a00697349723f14e2f6bbdd2 - cldap: use dns_resolver_server_timeout timeout for cldap ping * 724293d0873ee3229866ae4c13e1c8829375146f - ad: only send cldap-ping to our local domain * 918abaf37d7f13d72b29863933e133bcbd24d87c - ad: make ad_srv_plugin_ctx_switch_site() public * 99c4161910e542dd40c740032196d268c4163d07 - ad: use already discovered forest name * 4508ef5f7183c640191393605ea163044d9ac267 - ad: move current site and forest name to a more global context * b37e2713a9b86936f5b82a17e47757562900b911 - ad: require name when looking up root domain """ See the full comment at https://github.com/SSSD/sssd/pull/5883#issuecomment-985499303

1 0

← Newer
1
2
3
4
5
6
7
8
9
Older →

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

sssd-devel December 2021