Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Backport][ipa-4-7] Move DS's Kerberos env vars to unit file #2986

Closed
wants to merge 3 commits into from

Conversation

tiran
Copy link
Member

@tiran tiran commented Apr 2, 2019

Manual backport of PR #2982 to ipa-4-7 branch.

config_replace_variables() can now also remove lines from a file.

Related: https://pagure.io/freeipa/issue/7860
Signed-off-by: Christian Heimes <cheimes@redhat.com>
systemd daemon must be reloaded after a config file is added, changed,
or removed. Provide a common API endpoint in ipaplatform.tasks.

Related: https://pagure.io/freeipa/issue/7860
Signed-off-by: Christian Heimes <cheimes@redhat.com>
The IPA specific env vars KRB5_KTNAME and KRB5CCNAME are now defined in
a instance specific ipa-env.conf unit file.

Fixes: https://pagure.io/freeipa/issue/7860
Signed-off-by: Christian Heimes <cheimes@redhat.com>
@abbra abbra added the re-run Trigger a new run of PR-CI label Apr 3, 2019
@freeipa-pr-ci freeipa-pr-ci removed re-run Trigger a new run of PR-CI labels Apr 3, 2019
@tiran tiran added the re-run Trigger a new run of PR-CI label Apr 3, 2019
@freeipa-pr-ci freeipa-pr-ci removed re-run Trigger a new run of PR-CI labels Apr 3, 2019
@tiran
Copy link
Member Author

tiran commented Apr 3, 2019

Auto-ACK backport

@tiran tiran added ack Pull Request approved, can be merged pushed Pull Request has already been pushed labels Apr 3, 2019
@tiran
Copy link
Member Author

tiran commented Apr 3, 2019

ipa-4-7:

  • f9f01f5 Add option to remove lines from a file
  • f9b3fc1 Add tasks.systemd_daemon_reload()
  • b6c810e Move DS's Kerberos env vars to unit file

@tiran tiran closed this Apr 3, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ack Pull Request approved, can be merged pushed Pull Request has already been pushed
Projects
None yet
3 participants