[sssd PR#996][opened] 1.16: failover: make sure we switch to anoter server if only port differs

[pbrezina]

URL: https://github.com/SSSD/sssd/pull/996
Author: pbrezina
 Title: #996: 1.16: failover: make sure we switch to anoter server if only port differs
Action: opened

PR body:
"""
This is a regression introduced in 735af71a8e169f17fa5462db610a1567c9618e29.
After this commit we checked only server name instead of name and port combo.

Steps to reproduce:
1. Configure SSSD to use two servers with same name but different ports
```
[domain/LDAP]
debug_level=0xFFF0
id_provider = ldap
ldap_uri = ldap://$SERVER1:12345,ldap://$SERVER1:389
ldap_tls_cacertdir = /etc/openldap/certs
ldap_search_base = dc=example,dc=com
```

2. The first server port is unreachable, the second is working.
3. Run sssd and try to resolve user.
4. Without the patch SSSD tries to connect to the first server twice
   because new URI is not constructed and goes offline.

Resolves:
https://pagure.io/SSSD/sssd/issue/4112

Reviewed-by: Michal Židek 
"""

To pull the PR as Git branch:
git remote add ghsssd https://github.com/SSSD/sssd
git fetch ghsssd pull/996/head:pr996
git checkout pr996

[sssd PR#996][+Waiting for review] 1.16: failover: make sure we switch to anoter server if only port differs

[pbrezina]

URL: https://github.com/SSSD/sssd/pull/996
Title: #996: 1.16: failover: make sure we switch to anoter server if only port differs

Label: +Waiting for review

[sssd PR#996][synchronized] 1.16: failover: make sure we switch to anoter server if only port differs

[pbrezina]

URL: https://github.com/SSSD/sssd/pull/996
Author: pbrezina
 Title: #996: 1.16: failover: make sure we switch to anoter server if only port differs
Action: synchronized

To pull the PR as Git branch:
git remote add ghsssd https://github.com/SSSD/sssd
git fetch ghsssd pull/996/head:pr996
git checkout pr996

[sssd PR#996][comment] 1.16: failover: make sure we switch to anoter server if only port differs

[pbrezina]

URL: https://github.com/SSSD/sssd/pull/996
Title: #996: 1.16: failover: make sure we switch to anoter server if only port differs

pbrezina commented:
"""
(cherry picked from commit 08f0159078aa00d04533ca227679e25d64c452e1)
"""

See the full comment at https://github.com/SSSD/sssd/pull/996#issuecomment-593335883

[sssd PR#996][synchronized] 1.16: failover: make sure we switch to anoter server if only port differs

[pbrezina]

URL: https://github.com/SSSD/sssd/pull/996
Author: pbrezina
 Title: #996: 1.16: failover: make sure we switch to anoter server if only port differs
Action: synchronized

To pull the PR as Git branch:
git remote add ghsssd https://github.com/SSSD/sssd
git fetch ghsssd pull/996/head:pr996
git checkout pr996

[sssd PR#996][edited] 1.16: failover: make sure we switch to another server if only port differs

[pbrezina]

URL: https://github.com/SSSD/sssd/pull/996
Author: pbrezina
 Title: #996: 1.16: failover: make sure we switch to another server if only port differs
Action: edited

Changed field: title
Original value:
"""
1.16: failover: make sure we switch to anoter server if only port differs
"""

[sssd PR#996][comment] 1.16: failover: make sure we switch to another server if only port differs

[mzidek-gh]

URL: https://github.com/SSSD/sssd/pull/996
Title: #996: 1.16: failover: make sure we switch to another server if only port differs

mzidek-gh commented:
"""
In 1.16 branch:
Issue reproducible without the patch. Not reproducible with the patch.

ACK.
"""

See the full comment at https://github.com/SSSD/sssd/pull/996#issuecomment-594561399

[sssd PR#996][+Accepted] 1.16: failover: make sure we switch to another server if only port differs

[mzidek-gh]

URL: https://github.com/SSSD/sssd/pull/996
Title: #996: 1.16: failover: make sure we switch to another server if only port differs

Label: +Accepted

[sssd PR#996][-Waiting for review] 1.16: failover: make sure we switch to another server if only port differs

[mzidek-gh]

URL: https://github.com/SSSD/sssd/pull/996
Title: #996: 1.16: failover: make sure we switch to another server if only port differs

Label: -Waiting for review

[sssd PR#996][+Ready to push] 1.16: failover: make sure we switch to another server if only port differs

[pbrezina]

URL: https://github.com/SSSD/sssd/pull/996
Title: #996: 1.16: failover: make sure we switch to another server if only port differs

Label: +Ready to push

[sssd PR#996][comment] 1.16: failover: make sure we switch to another server if only port differs

[pbrezina]

URL: https://github.com/SSSD/sssd/pull/996
Title: #996: 1.16: failover: make sure we switch to another server if only port differs

pbrezina commented:
"""
* `sssd-1-16`
    * 4b1d1a099d89c5de320d233e57d65ea9484ca205 - failover: make sure we switch to another server if only port differs

"""

See the full comment at https://github.com/SSSD/sssd/pull/996#issuecomment-595122831

[sssd PR#996][-Accepted] 1.16: failover: make sure we switch to another server if only port differs

[pbrezina]

URL: https://github.com/SSSD/sssd/pull/996
Title: #996: 1.16: failover: make sure we switch to another server if only port differs

Label: -Accepted

[sssd PR#996][+Pushed] 1.16: failover: make sure we switch to another server if only port differs

[pbrezina]

URL: https://github.com/SSSD/sssd/pull/996
Title: #996: 1.16: failover: make sure we switch to another server if only port differs

Label: +Pushed

[sssd PR#996][-Ready to push] 1.16: failover: make sure we switch to another server if only port differs

[pbrezina]

URL: https://github.com/SSSD/sssd/pull/996
Title: #996: 1.16: failover: make sure we switch to another server if only port differs

Label: -Ready to push

[sssd PR#996][closed] 1.16: failover: make sure we switch to another server if only port differs

[pbrezina]

URL: https://github.com/SSSD/sssd/pull/996
Author: pbrezina
 Title: #996: 1.16: failover: make sure we switch to another server if only port differs
Action: closed

To pull the PR as Git branch:
git remote add ghsssd https://github.com/SSSD/sssd
git fetch ghsssd pull/996/head:pr996
git checkout pr996