[SSSD] [sssd PR#66][comment] Minor Dynamic DNS fixes

2017-02-22 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/66 Title: #66: Minor Dynamic DNS fixes jhrozek commented: """ * master: * d694d4fdcc81f24c2f9e3bb5a0dbe0a52498f196 * fccd8f9ab7a0ac9868c43ea0e8c3af142b2809fa """ See the full comment at https://github.com/SSSD/sssd/pull/66#issuecomment-281643200

[SSSD] [sssd PR#66][comment] Minor Dynamic DNS fixes

2017-02-20 Thread fidencio
URL: https://github.com/SSSD/sssd/pull/66 Title: #66: Minor Dynamic DNS fixes fidencio commented: """ CI: http://sssd-ci.duckdns.org/logs/job/63/03/summary.html """ See the full comment at https://github.com/SSSD/sssd/pull/66#issuecomment-281198976

[SSSD] [sssd PR#66][comment] Minor Dynamic DNS fixes

2017-02-20 Thread fidencio
URL: https://github.com/SSSD/sssd/pull/66 Title: #66: Minor Dynamic DNS fixes fidencio commented: """ @justin-stephenson: Thanks for the patches and explanation. I was able to reproduce the behavior you mentioned and, IMO, the patches look fine. I've just submitted them to our internal CI and

[SSSD] [sssd PR#66][comment] Minor Dynamic DNS fixes

2017-02-20 Thread justin-stephenson
URL: https://github.com/SSSD/sssd/pull/66 Title: #66: Minor Dynamic DNS fixes justin-stephenson commented: """ @fidencio No problem, thanks for looking into this. 1. Once the DNS role is installed, create a Reverse Zone in the AD DNS MMC(right click Reverse-Lookup zones and click New Zone)

[SSSD] [sssd PR#66][comment] Minor Dynamic DNS fixes

2017-02-19 Thread fidencio
URL: https://github.com/SSSD/sssd/pull/66 Title: #66: Minor Dynamic DNS fixes fidencio commented: """ @justin-stephenson: I've tried to reproduce the issue but, so far, I've failed (mainly due to a lack of knowledge). Let me drop a few questions in order to try to understand better how to

[SSSD] [sssd PR#66][comment] Minor Dynamic DNS fixes

2017-02-16 Thread justin-stephenson
URL: https://github.com/SSSD/sssd/pull/66 Title: #66: Minor Dynamic DNS fixes justin-stephenson commented: """ Thank you @fidencio @jhrozek 2 different customers confirmed this fix addressed their issue with the PTR record not being created. If you are not able to easily reproduce easily then

[SSSD] [sssd PR#66][comment] Minor Dynamic DNS fixes

2017-02-16 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/66 Title: #66: Minor Dynamic DNS fixes jhrozek commented: """ btw I'm fine with the patch, I was just waiting for Justin's confirmation that the patch indeed helps the customer. But please, go ahead and review it again, I only scrolled through the patch

[SSSD] [sssd PR#66][comment] Minor Dynamic DNS fixes

2017-02-16 Thread fidencio
URL: https://github.com/SSSD/sssd/pull/66 Title: #66: Minor Dynamic DNS fixes fidencio commented: """ @justin-stephenson, Firstly, sorry this patch has been waiting for so long. I'll try to reproduce the issue and review it Tomorrow. """ See the full comment at

[SSSD] [sssd PR#66][comment] Minor Dynamic DNS fixes

2017-02-13 Thread justin-stephenson
URL: https://github.com/SSSD/sssd/pull/66 Title: #66: Minor Dynamic DNS fixes justin-stephenson commented: """ The commit `DYNDNS: Update PTR record after non-fatal error` has been successfully tested and confirmed fixed by a SSSD user with RHEL 6 and RHEL7 systems. """ See the full comment

[SSSD] [sssd PR#66][comment] Minor Dynamic DNS fixes

2016-11-24 Thread lslebodn
URL: https://github.com/SSSD/sssd/pull/66 Title: #66: Minor Dynamic DNS fixes lslebodn commented: """ I let fellow developers to review it. """ See the full comment at https://github.com/SSSD/sssd/pull/66#issuecomment-262816234 ___ sssd-devel

[SSSD] [sssd PR#66][comment] Minor Dynamic DNS fixes

2016-11-23 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/66 Title: #66: Minor Dynamic DNS fixes jhrozek commented: """ On Wed, Nov 23, 2016 at 06:05:10AM -0800, lslebodn wrote: > @jhrozek Do you pan to review the patch? Feel free to take over the review, I'm currently busy with other work. """ See the full

[SSSD] [sssd PR#66][comment] Minor Dynamic DNS fixes

2016-11-23 Thread lslebodn
URL: https://github.com/SSSD/sssd/pull/66 Title: #66: Minor Dynamic DNS fixes lslebodn commented: """ @jhrozek Do you pan to review the patch? """ See the full comment at https://github.com/SSSD/sssd/pull/66#issuecomment-262520613 ___ sssd-devel

[SSSD] [sssd PR#66][comment] Minor Dynamic DNS fixes

2016-11-02 Thread justin-stephenson
URL: https://github.com/SSSD/sssd/pull/66 Title: #66: Minor Dynamic DNS fixes justin-stephenson commented: """ Please try reproducing this on a SSSD - AD provider configured system using AD as a DNS server with the below steps: 1. In the AD Properties of a DNS Forward Zone, change `Dynamic

[SSSD] [sssd PR#66][comment] Minor Dynamic DNS fixes

2016-11-02 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/66 Title: #66: Minor Dynamic DNS fixes jhrozek commented: """ Thank you for the patch. I wonder if you were actually able to reproduce the bug? If so, can you help me reproduce it locally? """ See the full comment at

[SSSD] [sssd PR#66][comment] Minor Dynamic DNS fixes

2016-10-25 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/66 Title: #66: Minor Dynamic DNS fixes jhrozek commented: """ ok to test """ See the full comment at https://github.com/SSSD/sssd/pull/66#issuecomment-256084328 ___ sssd-devel mailing list --

[SSSD] [sssd PR#66][comment] Minor Dynamic DNS fixes

2016-10-25 Thread centos-ci
URL: https://github.com/SSSD/sssd/pull/66 Title: #66: Minor Dynamic DNS fixes centos-ci commented: """ Can one of the admins verify this patch? """ See the full comment at https://github.com/SSSD/sssd/pull/66#issuecomment-25603 ___ sssd-devel

[SSSD] [sssd PR#66][comment] Minor Dynamic DNS fixes

2016-10-25 Thread centos-ci
URL: https://github.com/SSSD/sssd/pull/66 Title: #66: Minor Dynamic DNS fixes centos-ci commented: """ Can one of the admins verify this patch? """ See the full comment at https://github.com/SSSD/sssd/pull/66#issuecomment-25607 ___ sssd-devel