Skip to content

update ramendr docs to reflect change to defaults - #732

Open
OAharoni-RedHat wants to merge 1 commit into
validatedpatterns:mainfrom
OAharoni-RedHat:main
Open

OAharoni-RedHat wants to merge 1 commit into
validatedpatterns:mainfrom
OAharoni-RedHat:main

Conversation

@OAharoni-RedHat

@OAharoni-RedHat OAharoni-RedHat commented Sep 28, 2026 •

Copy link
Copy Markdown
Contributor

BYOC is now the default for RamenDR-starter-kit. We want to reflect that in our documentation.

@openshift-ci openshift-ci Bot added the size/M label Sep 28, 2026
@openshift-ci
openshift-ci Bot requested review from beekhof and mhjacks September 28, 2026 16:26
@ocpdocs-previewbot

ocpdocs-previewbot commented Sep 28, 2026 •

Copy link
Copy Markdown

🤖 Tue Sep 29 12:42:32 - The preview is ready at:
https://732--patternsdocs-pr.netlify.app

@mhjacks mhjacks left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Really minor issue, but I'm glad it showed up, because the statement about hub/rdr/values.yaml is now obsolete and also must be changed. That's not your fault, I missed it the last time I updated docs here.


. The pattern will infer the baseDomain of your cluster based on the clusterDomain which is tracked by the pattern
operator. Previously, this required the pattern to be forked to be useful - but this is no longer the case (you may
operator. Previously, this required the pattern to be forked to be useful - but this is no longer the case. you may

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Capitalization for "you may." Also, the link to the hub/rdr/values.yaml file is wrong - we should refer them to values-names in overrides instead I think.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

good catch, should be fixed now!

This branch has not been deployed

No deployments
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