Skip to content

Conversation

@andy-stark-redis
Copy link
Contributor

@andy-stark-redis andy-stark-redis commented Nov 28, 2024

DOC-4605

I actually added the new stuff after the section about values.yaml (because I figured you would add the new values there) but let me know if the new info should be emphasised by moving it further up the page. Content is basically the same as suggested with a few extra links.

@andy-stark-redis andy-stark-redis requested a review from a team November 28, 2024 13:14
@andy-stark-redis andy-stark-redis self-assigned this Nov 28, 2024
@github-actions
Copy link
Contributor

Copy link
Contributor

@stoyanr stoyanr left a comment

Choose a reason for hiding this comment

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

Thanks, lgtm

@andy-stark-redis
Copy link
Contributor Author

Thanks for the review @stoyanr

Copy link
Collaborator

@dwdougherty dwdougherty left a comment

Choose a reason for hiding this comment

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

LGTM.

@andy-stark-redis
Copy link
Contributor Author

Thanks @dwdougherty !

@andy-stark-redis andy-stark-redis merged commit 55e1edc into main Dec 2, 2024
5 checks passed
@andy-stark-redis andy-stark-redis deleted the DOC-4605-rdi-nginx-ingress branch December 2, 2024 14:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants