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

KCM: Document KCM debugging #77

Merged
merged 1 commit into from
Jan 18, 2024
Merged

KCM: Document KCM debugging #77

merged 1 commit into from
Jan 18, 2024

Conversation

aplopez
Copy link
Contributor

@aplopez aplopez commented Dec 8, 2023

Added some KCM-specific details to the already present debug information.

@justin-stephenson
Copy link
Contributor

I added some inline comments but changes look good to me overall.

A couple other things I found useful when debugging KCM issues. You can determine if they would be worthwhile to add here or not.

sudo strace -p $(pidof sssd_kcm) -Ttxvs 1024 -e read=$fds -e read,readv,write

@aplopez
Copy link
Contributor Author

aplopez commented Dec 18, 2023

* KCM opcodes and parameters from https://github.com/krb5/krb5/blob/master/src/include/kcm.h

I didn't add this one as it is easily found and the text mainly focuses on debugging techniques.

sudo strace -p $(pidof sssd_kcm) -Ttxvs 1024 -e read=$fds -e read,readv,write

But I did add this one.

Thanks.

Added some KCM-specific details to the already present debug information.
Copy link
Contributor

@justin-stephenson justin-stephenson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ack, thank you.

@aplopez aplopez added the Accepted Pull request is accepted label Dec 19, 2023
@andreboscatto andreboscatto merged commit 948c27a into SSSD:master Jan 18, 2024
2 checks passed
@aplopez aplopez deleted the kcm branch March 25, 2024 10:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Accepted Pull request is accepted
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants