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-6] upgrade: Run configuration upgrade under empty ccache collection #1732

Closed
wants to merge 1 commit into from

Conversation

abbra
Copy link
Contributor

@abbra abbra commented Mar 23, 2018

Manual backport for ipa-4-6 as KCM is also a default ccache type in Fedora 27

Use temporary empty DIR-based ccache collection to prevent upgrade
failures in case KCM: or KEYRING: ccache type is used by default in
krb5.conf and is not available. We don't need any user credentials
during upgrade procedure but kadmin.local would attempt to resolve
default ccache and if that's not available, kadmin.local will fail.

Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1558818

Use temporary empty DIR-based ccache collection to prevent upgrade
failures in case KCM: or KEYRING: ccache type is used by default in
krb5.conf and is not available. We don't need any user credentials
during upgrade procedure but kadmin.local would attempt to resolve
default ccache and if that's not available, kadmin.local will fail.

Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1558818
Signed-off-by: Alexander Bokovoy <abokovoy@redhat.com>
@tiran
Copy link
Member

tiran commented Mar 23, 2018

This is a manual backport of #1725 to 4.6.

@tiran tiran added the ack Pull Request approved, can be merged label Mar 23, 2018
@tiran tiran self-assigned this Mar 23, 2018
@tiran tiran self-requested a review March 23, 2018 15:04
@tiran tiran added the pushed Pull Request has already been pushed label Mar 24, 2018
@tiran
Copy link
Member

tiran commented Mar 24, 2018

ipa-4-6:

  • 7645dc2 upgrade: Run configuration upgrade under empty ccache collection

@tiran tiran closed this Mar 24, 2018
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
2 participants