Skip to content

Conversation

rrelledge
Copy link
Collaborator

@rrelledge rrelledge commented Sep 10, 2025

No description provided.

rrelledge and others added 7 commits August 14, 2025 17:24
…d internode encryption certs to RS 7.22.2 release notes
* DOC-5210 RS: Added ccs_internode_encryption and data_internode_encryption to certificates table

* DOC-5210 Added more details to the INE certificate descriptions

* DOC-5210 Added new section for customer-managed internode encryption certificates

* DOC-5210 Copy edits

* Add <img> handling to show full size image on click, similar to image shortcode

* DOC-5210 More copy edits
@rrelledge rrelledge self-assigned this Sep 10, 2025
@rrelledge rrelledge added do not merge yet rs Redis Enterprise Software labels Sep 10, 2025
Copy link
Contributor

github-actions bot commented Sep 10, 2025

DOC-5564

Copy link
Contributor

@andy-stark-redis andy-stark-redis left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link

@yoavredis yoavredis left a comment

Choose a reason for hiding this comment

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

@rrelledge I wrote some comments, other than that it LGTM, 10x.

@rrelledge rrelledge requested a review from yoavredis September 16, 2025 16:38
Copy link

@yoavredis yoavredis left a comment

Choose a reason for hiding this comment

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

@rrelledge LGTM!
I've left some tiny comments.
10x

Copy link

@yoavredis yoavredis left a comment

Choose a reason for hiding this comment

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

@rrelledge I've added an important comment about using RSA keys only - please make sure it's included. The rest LGTM and approved. 10x.

@rrelledge rrelledge requested a review from yoavredis September 22, 2025 14:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
do not merge yet rs Redis Enterprise Software
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants