Skip to content

Conversation

@jimcat8
Copy link
Contributor

@jimcat8 jimcat8 commented Oct 7, 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

Hello @jimcat8
Thanks for your review.
You will find my comments on my review.

Regards.

@jimcat8 jimcat8 changed the title A supplement to the newgrp command Changes to the file 06-users.md, For a long time Oct 7, 2022
Copy link
Contributor

@SergeCroise SergeCroise left a comment

Choose a reason for hiding this comment

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

proposal:

  • 3: The time when the password was last changed, the timestamp format, in days. The so-called timestamp is based on January 1st 1970, as the standard time. Every time one day goes by, the timestamp is +1.
  • 4: Minimum lifetime of the password. That is, the time interval between two password changes (related to the third field), in days.

sspencerwire and others added 2 commits October 12, 2022 07:27
Co-authored-by: Serge Croisé <SergeCroise@users.noreply.github.com>
Co-authored-by: Serge Croisé <SergeCroise@users.noreply.github.com>
| `-x DAYS` | Maximum password lifetime. Permanent change. For root (uid=0) use only. |
| `-w DAYS` | Warning time before expiration. Permanent change. For root (uid=0) use only. |
| `-i DAYS` | Delay before deactivation when the password expires. Permanent change. For root (uid=0) use only.|

Copy link
Contributor

Choose a reason for hiding this comment

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

Since all of these commands are "For root (uid=0) only." Why not specify that ahead of the table and limit the number of words in the table itself? If you are trying to add clarity to the document, (which appears to be your goal) then this would help.

jimcat8 and others added 2 commits October 24, 2022 23:04
returned original wording of `chage` example per author
@sspencerwire sspencerwire merged commit f4be585 into rocky-linux:main Oct 24, 2022
@github-actions
Copy link

Test results for 5011569:

Number of broken URLs: 28

URL,RESULT,FILENAME
 https://bugs.rockylinux.org/show_bug.cgi?id=174,failed,release_notes/8-changelog.md
 https://bugs.rockylinux.org/show_bug.cgi?id=14,failed,release_notes/8_4.md
 https://bugs.rockylinux.org/show_bug.cgi?id=20,failed,release_notes/8_4.md
 https://lxd.readthedocs.io/en/stable-4.0/instances/,failed,books/lxd_server/07-configurations.md
 https://lxd.readthedocs.io/en/latest/clustering/,failed,books/lxd_server/03-lxdinit.md
 https://lxd.readthedocs.io/en/latest/clustering/,failed,books/lxd_server/30-appendix_a.md
 https://www.cyber.gov.au/,failed,books/disa_stig/disa_stig_part2.md
 http://mirror.netweaver.uk/rocky/8.5/PowerTools/aarch64/os/,failed,books/admin_guide/13-softwares.md
 https://www.ourownwiki.com,failed,guides/security/ssl_keys_https.md
 https://lxd.readthedocs.io/en/latest/clustering/,failed,guides/containers/lxd_web_servers.md
 http://10.1.1.160,failed,guides/containers/podman-nextcloud.md
 http://downloads.asterisk.org/pub/telephony/asterisk/asterisk-18.6.0.tar.gz,failed,guides/communications/asterisk_installation.md
 http://rpms.remirepo.net/enterprise/remi-release-,failed,guides/network/librenms_monitoring_server.md
 https://rocky.lowend.ninja/RockyDevel/MOCK_RAW/,failed,guides/package_management/package_build_troubleshooting.md
 https://wiki.rockylinux.org/en/team/development/Mock_Build_Howto,failed,guides/package_management/package_build_troubleshooting.md
 https://wiki.rockylinux.org/en/team/development/Package_Error_Tracking,failed,guides/package_management/package_build_troubleshooting.md
 https://wiki.rockylinux.org/en/team/development/Build_Order/Build_Pass_10_Failure,failed,guides/package_management/package_build_troubleshooting.md
 https://rocky.lowend.ninja/RockyDevel/mock_configs/,failed,guides/package_management/package_build_troubleshooting.md
 https://wiki.rockylinux.org/en/team/development/debranding/how-to,failed,guides/package_management/package_debranding.md
 https://github,failed,guides/automation/cronie.md
 http://mybrokenaxel.com,failed,guides/file_sharing/sftp.md
 http://docs.gluster.org/en/latest/Administrator%20Guide/Split%20brain%20and%20ways%20to%20deal%20with%20it/,failed,guides/file_sharing/glusterfs.md
 http://wiki-doc/install.php,failed,guides/cms/dokuwiki_server.md
 https://www.ourownwiki.com/,failed,guides/web/apache-sites-enabled.md
 http://yourip/info.php,failed,guides/web/php.md
 http://10.56.233.189:8000,failed,guides/contribute/mkdocs_lsyncd.md
 https://docs.rockylinux.org/guides/containers/lxd_server,failed,guides/proxies/haproxy_apache_lxd.md
 https://docs.rockylinux.org/guides/containers/lxd_server/#centos-macvlan,failed,guides/proxies/haproxy_apache_lxd.md

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.

4 participants