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

Set "KDC:Disable Last Success" by default #641

Closed
wants to merge 1 commit into from
Closed

Set "KDC:Disable Last Success" by default #641

wants to merge 1 commit into from

Conversation

MartinBasti
Copy link
Contributor

@MartinBasti MartinBasti commented Mar 22, 2017

In big deployments enabled recording of the last sucesfull login
this creates a huge changelog on DS side and cause performance
issues even if this is excluded from replication.

Actually this is not used directly by FreeIPA so it is safe to remove
in new installations. User who need this must manually remove
"KDC:Disable Last Success" using ipa config-mod command or WebUI.

https://pagure.io/freeipa/issue/5313

In big deployments enabled recording of the last sucesfull login
this creates a huge changelog on DS side and cause performance
issues even if this is excluded from replication.

Actually this is not used directly by FreeIPA so it is safe to remove
in new installations. User who need this must manually remove
"KDC:Disable Last Success" using `ipa config-mod` command or WebUI.

https://pagure.io/freeipa/issue/5313
@MartinBasti
Copy link
Contributor Author

Bump for review

@stlaz stlaz self-assigned this Mar 27, 2017
@stlaz
Copy link
Contributor

stlaz commented Mar 27, 2017

This change of default behavior seems to have removed a lot of write-load from DS so I believe we should go with it. However, add comment about it to the issue "changelog", please.

@stlaz stlaz added the ack Pull Request approved, can be merged label Mar 27, 2017
@pvomacka
Copy link

ipa-4-5:

  • fdcd5f4 Set "KDC:Disable Last Success" by default
    master:

  • eeaf428 Set "KDC:Disable Last Success" by default

@pvomacka pvomacka added the pushed Pull Request has already been pushed label Mar 27, 2017
@pvomacka pvomacka closed this Mar 27, 2017
@MartinBasti MartinBasti deleted the disable-lst-success branch March 27, 2017 16:56
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