-
Notifications
You must be signed in to change notification settings - Fork 48
DOC-1287 hide defaults in cloud cluster properties #1107
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
DOC-1287 hide defaults in cloud cluster properties #1107
Conversation
|
Important Review skippedAuto incremental reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the 📝 WalkthroughWalkthroughThe changes update the documentation for several cluster configuration properties in the Assessment against linked issues
Possibly related PRs
Suggested reviewers
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
✅ Deploy Preview for redpanda-docs-preview ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
asimms41
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just a couple of updates to make, but LGTM
|
|
||
| *Type:* boolean | ||
|
|
||
| *Enterprise license required*: `true` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Wouldn't this line be excluded for Cloud as well?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fixed, thank you!
| endif::[] | ||
|
|
||
| **Related topics**: | ||
|
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should link on line below be Enable Iceberg Integration to match the page heading?
|
|
||
| *Type:* string | ||
|
|
||
| ifndef::env-cloud[] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Anchor link above <<iceberg_rest_catalog_trust_file, iceberg_rest_catalog_trust_file>> is not working.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fixed
Description
This pull request updates the
modules/reference/pages/properties/cluster-properties.adocfile to conditionally exclude default property values in Cloud docs. Specifically, it wraps default values inifndef::env-cloud[]blocks to exclude them when theenv-cloudenvironment is active.Conditional Default Value Updates:
ifndef::env-cloud[]blocks around default values for various cluster properties, such as audit logging, WebAssembly-powered data transforms, Iceberg catalog configurations, and more. These blocks exclude default values from the documentation when theenv-cloudenvironment is active. [1] [2] [3] [4] [5] [6] [7] [8] [9] [10] [11] [12] [13] [14] [15] [16] [17] [18] [19] [20] [21] [22] [23] [24]Resolves https://redpandadata.atlassian.net/browse/DOC-1287
Review deadline:
Page previews
Cluster config properties in Cloud
Cluster config properties in Self-Managed
Checks