[sssd PR#368][opened] ipa: make sure view name is initialized at startup

[sumit-bose]

   URL: https://github.com/SSSD/sssd/pull/368
Author: sumit-bose
 Title: #368: ipa: make sure view name is initialized at startup
Action: opened

PR body:
"""
sysdb_master_domain_update() can only set the view name properly if it was not
set before but it might be called multiple times before the view name is
available if the cache is empty. Since ipa_apply_view() keeps track if
the view name was already set at startup or not the name can safely be
cleaned here before sysdb_master_domain_update() is called.

Resolves https://pagure.io/SSSD/sssd/issue/3501
"""

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

[sssd PR#368][+Accepted] ipa: make sure view name is initialized at startup

[jhrozek]

  URL: https://github.com/SSSD/sssd/pull/368
Title: #368: ipa: make sure view name is initialized at startup

Label: +Accepted

[sssd PR#368][comment] ipa: make sure view name is initialized at startup

[jhrozek]

  URL: https://github.com/SSSD/sssd/pull/368
Title: #368: ipa: make sure view name is initialized at startup

jhrozek commented:
"""
CI and Coverity passed -> accepted.
"""

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

[sssd PR#368][comment] ipa: make sure view name is initialized at startup

[lslebodn]

  URL: https://github.com/SSSD/sssd/pull/368
Title: #368: ipa: make sure view name is initialized at startup

lslebodn commented:
"""
@sumit-bose Upstream ticket has not been triaged yet. So I am not sure where to push the
patch.
Is this bug also in 1.13 and 1.14?
"""

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

[sssd PR#368][comment] ipa: make sure view name is initialized at startup

[sumit-bose]

  URL: https://github.com/SSSD/sssd/pull/368
Title: #368: ipa: make sure view name is initialized at startup

sumit-bose commented:
"""
@lslebodn, yes I would say the bug is present since the views/override support was added.
Since this issue can only be seen with non-default views and the names of the Kerberos
credential cache files seem to be the only place in the backend where the proper view name
is really needed for the operation it was not noticed so far.
"""

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

[sssd PR#368][comment] ipa: make sure view name is initialized at startup

[lslebodn]

  URL: https://github.com/SSSD/sssd/pull/368
Title: #368: ipa: make sure view name is initialized at startup

lslebodn commented:
"""
master:
* f00591a4615720640cf01b1c408315b57dd397dc
"""

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

[sssd PR#368][closed] ipa: make sure view name is initialized at startup

[jhrozek]

   URL: https://github.com/SSSD/sssd/pull/368
Author: sumit-bose
 Title: #368: ipa: make sure view name is initialized at startup
Action: closed

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

[sssd PR#368][+Pushed] ipa: make sure view name is initialized at startup

[jhrozek]

  URL: https://github.com/SSSD/sssd/pull/368
Title: #368: ipa: make sure view name is initialized at startup

Label: +Pushed

[sssd PR#368][comment] ipa: make sure view name is initialized at startup

[jhrozek]

  URL: https://github.com/SSSD/sssd/pull/368
Title: #368: ipa: make sure view name is initialized at startup

jhrozek commented:
"""
The ticket will be discussed during today's triage, I suspect we will apply the patch
to also the stable branches, but we can already close the PR.
"""

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