Skip to content

Conversation

@alemorvan
Copy link
Contributor

@alemorvan alemorvan commented Jul 15, 2022

Author checklist (Completed by original Author)

  • Contribution a good fit for the Rocky project? Title and Author MetaTags inserted ?
  • Is this a non-English contribution?
  • If applicable, steps and instructions have been tested to work on a real system
  • Did you perform an initial self-review to fix basic typos and grammatical correctness

Rocky Documentation checklist (Completed by Rocky team)

  • 1st Pass (Check that document is good fit for project and author checklist completed)
  • 2nd Pass (Technical Review - check for technical correctness)
  • 3rd Pass (Basic Editorial Review)
  • 4th Pass (Detailed Editorial Review and Peer Review)
  • Final pass/approval (Final Review)

@alemorvan
Copy link
Contributor Author

As request by @jimcat8, we need a PAM documentation.

Here is the translation of my initial PAM doc (for centos 7).

Please consider making the tests needed to validate the good functionnement of the modules (does they still exist?).

Hope this doc will help you @jimcat8

Thx @sspencerwire for the review

@EzequielBruni
Copy link
Collaborator

Adding this to my to-do list.

@sspencerwire
Copy link
Contributor

sspencerwire commented Jul 20, 2022

Adding this to my to-do list.

As you are planning on doing the editing on this @EzequielBruni, I'm just going to run it through some testing on both 8.6 and 9.0. It should work the same way as here, but since it was written for CentOS 7, it's probably a good idea to test it. Don't let it hold up your edit and merge, however. I can always push any changes or notes needed after testing. Thank you, and welcome back!!

@sspencerwire
Copy link
Contributor

Adding this to my to-do list.

As you are planning on doing the editing on this @EzequielBruni, I'm just going to run it through some testing on both 8.6 and 9.0. It should work the same way as here, but since it was written for CentOS 7, it's probably a good idea to test it. Don't let it hold up your edit and merge, however. I can always push any changes or notes needed after testing. Thank you, and welcome back!!

Instead of testing, I checked out release notes associated with upstream between 7 and 8 and 8 and 9, and the only item that I found that is different, isn't mentioned in @alemorvan 's document, so you can edit and merge when ready. Thank you!!

@EzequielBruni
Copy link
Collaborator

@sspencerwire I made a few significant edits, and for some reason I was forced to make a new branch. If you wouldn't mind double checking my work, I'd appreciate it.

https://github.com/rocky-linux/documentation/blob/pr/746/docs/guides/security/pam.md

@sspencerwire
Copy link
Contributor

@sspencerwire I made a few significant edits, and for some reason I was forced to make a new branch. If you wouldn't mind double checking my work, I'd appreciate it.

https://github.com/rocky-linux/documentation/blob/pr/746/docs/guides/security/pam.md

I'm on it! Thank you!

@sspencerwire sspencerwire merged commit 55c2ef5 into rocky-linux:main Jul 25, 2022
@alemorvan alemorvan deleted the lemorvan/pam branch June 4, 2023 19:07
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