Skip to content

replace legacy Operator version remote include with manual version nu…#23393

Merged
taroface merged 2 commits into
mainfrom
k8s-remove-remote-include
May 22, 2026
Merged

replace legacy Operator version remote include with manual version nu…#23393
taroface merged 2 commits into
mainfrom
k8s-remove-remote-include

Conversation

@taroface
Copy link
Copy Markdown
Contributor

Replace the latest_operator_version variable, which used a remote include, with the hardcoded version number. This applies only to the legacy Operator that we do not expect to update. cc @jhlodin

@taroface taroface requested a review from a team May 21, 2026 22:13
@netlify
Copy link
Copy Markdown

netlify Bot commented May 21, 2026

Deploy Preview for cockroachdb-api-docs canceled.

Name Link
🔨 Latest commit 132443d
🔍 Latest deploy log https://app.netlify.com/projects/cockroachdb-api-docs/deploys/6a10718dfce5b800082abf0a

@netlify
Copy link
Copy Markdown

netlify Bot commented May 21, 2026

Deploy Preview for cockroachdb-interactivetutorials-docs canceled.

Name Link
🔨 Latest commit 132443d
🔍 Latest deploy log https://app.netlify.com/projects/cockroachdb-interactivetutorials-docs/deploys/6a10718d11d7b70008105694

@github-actions
Copy link
Copy Markdown

Files changed:

@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 21, 2026

Cross-Version Link Check Failed

Found 1 cross-version link violation that must be fixed:


File: src/current/_includes/v20.2/orchestration/start-cockroachdb-operator-secure.md

Line 93: Link from v20.2 to v21.1 detected

[Operate CockroachDB on Kubernetes](../v21.1/operate-cockroachdb-kubernetes.html#use-a-custom-ca)

Suggested fix:

[Operate CockroachDB on Kubernetes]({% link {{ page.version.version }}/operate-cockroachdb-kubernetes.html#use-a-custom-ca %})

Action Required: Please update all cross-version links to use version variables or ensure links stay within the same version.

For more information about proper link formatting, see the CockroachDB Docs Style Guide.

@netlify
Copy link
Copy Markdown

netlify Bot commented May 21, 2026

Netlify Preview

Name Link
🔨 Latest commit 132443d
🔍 Latest deploy log https://app.netlify.com/projects/cockroachdb-docs/deploys/6a10718d9406a80008f306a3
😎 Deploy Preview https://deploy-preview-23393--cockroachdb-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@rmloveland
Copy link
Copy Markdown
Contributor

mentioning https://cockroachlabs.atlassian.net/browse/DOC-17061 for the cross-linkage

Copy link
Copy Markdown
Contributor

@rmloveland rmloveland left a comment

Choose a reason for hiding this comment

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

LGTM

@taroface taroface enabled auto-merge (squash) May 22, 2026 15:09
@taroface taroface merged commit 1e73e6c into main May 22, 2026
7 of 8 checks passed
@taroface taroface deleted the k8s-remove-remote-include branch May 22, 2026 15:17
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.

2 participants