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

fix docs to use cern-get-keytab #59

Merged
merged 3 commits into from
Nov 9, 2023
Merged

Conversation

bendylan
Copy link
Contributor

@bendylan bendylan commented Jul 5, 2023

The existing docs with ktutil use the wrong enctypes and will fail. However rather than fixing that, I think using cern-get-keytab is probably the right thing to suggest, since that tool takes care of the details.

Note that existing docs are probably doing the wrong enctypes,
whilst the utility does the right thing.
@welcome
Copy link

welcome bot commented Jul 5, 2023

Thanks for opening your first pull request here 🚀! If you have any questions, feel free to mention one of the conveners, previous contributors, or attend our weekly meeting (see
our homepage). Also, sometimes PRs go unnoticed, so don't hesitate to @mention some of us, if we do not come back to you within a few days.

@jonas-eschle
Copy link
Collaborator

Hey, thanks for contributing and sorry for the long silence! I am unfortunately rather unfamiliar with the section, are you confident that the change is the right thing? If so, lgtm

@goi42
Copy link
Contributor

goi42 commented Aug 24, 2023

I just opened #66 without realizing this PR already existed--sorry about that. I'll leave it open for now since it makes some additional changes not included here.

@stale
Copy link

stale bot commented Oct 24, 2023

This issue or pull request has been automatically marked as stale because it has not had recent activity. Please manually close it, if it is no longer relevant, or ask for help or support to help getting it unstuck. Let me bring this to the attention of @klieret @wdconinc @michmx for now.

@stale stale bot added the stale label Oct 24, 2023
@stale stale bot removed the stale label Nov 9, 2023
@jonas-eschle jonas-eschle merged commit 3bf10ce into hsf-training:master Nov 9, 2023
2 checks passed
Copy link

welcome bot commented Nov 9, 2023

Congrats on merging your first pull request 🎉! We greatly appreciate it.
You might be eligible to be added to the HSF Training Community page (see the instructions on the page for how to create a profile). If you already have created a profile previously, make sure you're also added to the current year. If this repository features a list of contributors at the bottom of the readme, you might also be eligible to add yourself there.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants