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

upgrade: do not explicitly set principal for services #262

Closed
wants to merge 1 commit into from

Conversation

tkrizek
Copy link
Contributor

@tkrizek tkrizek commented Nov 21, 2016

After installer refactoring, principal is a property of service.

https://fedorahosted.org/freeipa/ticket/6500

After installer refactoring, principal is a property of service.

https://fedorahosted.org/freeipa/ticket/6500
@martbab
Copy link
Contributor

martbab commented Nov 21, 2016

Heh, I have opened #262 for this same issue recently. Since you assigned yourself to the ticket and filled all required fields you win this race :).

@martbab martbab self-assigned this Nov 21, 2016
@martbab martbab added ack Pull Request approved, can be merged pushed Pull Request has already been pushed labels Nov 21, 2016
@martbab
Copy link
Contributor

martbab commented Nov 21, 2016

@martbab martbab closed this Nov 21, 2016
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