[sssd PR#998][opened] spec: Do not overwrite /etc/pam.d/sssd-shadowutils on update

[mzidek-gh]

URL: https://github.com/SSSD/sssd/pull/998
Author: mzidek-gh
 Title: #998: spec: Do not overwrite /etc/pam.d/sssd-shadowutils on update
Action: opened

PR body:
"""
We should not overwrite this file when sssd-common is
updated.
"""

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

[sssd PR#998][+Waiting for review] spec: Do not overwrite /etc/pam.d/sssd-shadowutils on update

[mzidek-gh]

URL: https://github.com/SSSD/sssd/pull/998
Title: #998: spec: Do not overwrite /etc/pam.d/sssd-shadowutils on update

Label: +Waiting for review

[sssd PR#998][-Waiting for review] spec: Do not overwrite /etc/pam.d/sssd-shadowutils on update

[pbrezina]

URL: https://github.com/SSSD/sssd/pull/998
Title: #998: spec: Do not overwrite /etc/pam.d/sssd-shadowutils on update

Label: -Waiting for review

[sssd PR#998][+Accepted] spec: Do not overwrite /etc/pam.d/sssd-shadowutils on update

[pbrezina]

URL: https://github.com/SSSD/sssd/pull/998
Title: #998: spec: Do not overwrite /etc/pam.d/sssd-shadowutils on update

Label: +Accepted

[sssd PR#998][+Ready to push] spec: Do not overwrite /etc/pam.d/sssd-shadowutils on update

[pbrezina]

URL: https://github.com/SSSD/sssd/pull/998
Title: #998: spec: Do not overwrite /etc/pam.d/sssd-shadowutils on update

Label: +Ready to push

[sssd PR#998][comment] spec: Do not overwrite /etc/pam.d/sssd-shadowutils on update

[pbrezina]

URL: https://github.com/SSSD/sssd/pull/998
Title: #998: spec: Do not overwrite /etc/pam.d/sssd-shadowutils on update

pbrezina commented:
"""
Ack.

I agree we should not modify it on update but I'd like to ask what is the meaning of this file?
"""

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

[sssd PR#998][comment] spec: Do not overwrite /etc/pam.d/sssd-shadowutils on update

[mzidek-gh]

URL: https://github.com/SSSD/sssd/pull/998
Title: #998: spec: Do not overwrite /etc/pam.d/sssd-shadowutils on update

mzidek-gh commented:
"""
IIUC it is only used as pam target for proxy domains. Probably so that people do not need to craft their own pam config from scratch and we do not need to rely on something external (in examples and howtos). But I am not 100% sure. I do see it in example proxy domain configuration.
"""

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

[sssd PR#998][comment] spec: Do not overwrite /etc/pam.d/sssd-shadowutils on update

[pbrezina]

URL: https://github.com/SSSD/sssd/pull/998
Title: #998: spec: Do not overwrite /etc/pam.d/sssd-shadowutils on update

pbrezina commented:
"""
* `master`
    * e698d53e0ddd3c2778e04fd8e405f8c0cee0a766 - spec: Do not overwrite /etc/pam.d/sssd-shadowutils

"""

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

[sssd PR#998][+Pushed] spec: Do not overwrite /etc/pam.d/sssd-shadowutils on update

[pbrezina]

URL: https://github.com/SSSD/sssd/pull/998
Title: #998: spec: Do not overwrite /etc/pam.d/sssd-shadowutils on update

Label: +Pushed

[sssd PR#998][-Accepted] spec: Do not overwrite /etc/pam.d/sssd-shadowutils on update

[pbrezina]

URL: https://github.com/SSSD/sssd/pull/998
Title: #998: spec: Do not overwrite /etc/pam.d/sssd-shadowutils on update

Label: -Accepted

[sssd PR#998][-Ready to push] spec: Do not overwrite /etc/pam.d/sssd-shadowutils on update

[pbrezina]

URL: https://github.com/SSSD/sssd/pull/998
Title: #998: spec: Do not overwrite /etc/pam.d/sssd-shadowutils on update

Label: -Ready to push

[sssd PR#998][closed] spec: Do not overwrite /etc/pam.d/sssd-shadowutils on update

[pbrezina]

URL: https://github.com/SSSD/sssd/pull/998
Author: mzidek-gh
 Title: #998: spec: Do not overwrite /etc/pam.d/sssd-shadowutils on update
Action: closed

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