Skip to content

Fix incorrect rladmin command in create-certificates.md (Fixes #3737) - #3806

Closed
harshh6305 wants to merge 1 commit into
redis:mainfrom
harshh6305:fix/rladmin-certificates
Closed

Fix incorrect rladmin command in create-certificates.md (Fixes #3737)#3806
harshh6305 wants to merge 1 commit into
redis:mainfrom
harshh6305:fix/rladmin-certificates

Conversation

@harshh6305

@harshh6305 harshh6305 commented Aug 13, 2026

Copy link
Copy Markdown

Fixes #3737

Problem

The "Install certificates" section instructed users to check certificate status using:

rladmin status certificates

This command doesn't work — rladmin has no certificates subcommand, resulting in:

ERROR: invalid token 'certificates'

Fix

Replaced the incorrect command with an openssl x509 command to verify the installed certificate's expiration dates, as suggested in the issue.


Note

Low Risk
Documentation-only correction; no product code or operational behavior changes.

Overview
Fixes #3737 in the Install certificates section of create-certificates.md.

The post-install step Check certificate status no longer tells users to run rladmin status certificates (invalid rladmin subcommand). It now uses openssl x509 -in /path/to/redis-cert-chain.pem -noout -dates, aligned with the validation section elsewhere in the same doc. The following View certificate details step (rladmin info cluster) is unchanged.

Reviewed by Cursor Bugbot for commit a313dee. Bugbot is set up for automated code reviews on this repo. Configure here.

@CLAassistant

CLAassistant commented Aug 13, 2026

Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.


harshh6305 seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

@harshh6305
harshh6305 force-pushed the fix/rladmin-certificates branch from 352387c to 3a6eac3 Compare August 13, 2026 05:55
@harshh6305
harshh6305 force-pushed the fix/rladmin-certificates branch from 3a6eac3 to a313dee Compare August 13, 2026 06:02
@harshh6305 harshh6305 closed this Aug 13, 2026
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.

Feedback: Create certificates

2 participants