Delete 5 pages#22813
Conversation
✅ Deploy Preview for cockroachdb-api-docs canceled.
|
✅ Deploy Preview for cockroachdb-api-docs canceled.
|
❌ Deploy Preview for cockroachdb-docs failed. Why did it fail? →
|
✅ Deploy Preview for cockroachdb-interactivetutorials-docs canceled.
|
✅ Deploy Preview for cockroachdb-interactivetutorials-docs canceled.
|
✅ Netlify Preview
To edit notification comments on pull requests, go to your Netlify project configuration. |
…chdb/docs into 20260219-doc-16178-page-deletion
Also removed unnecessary note
Fixed bad redirects (hopefully)
| <ul> | ||
| <li><a href="orchestrate-a-local-cluster-with-kubernetes.html">Orchestrate CockroachDB Locally with Minikube</a></li> | ||
| <li><a href="deploy-cockroachdb-with-kubernetes.html">Orchestrate CockroachDB in a Single Kubernetes Cluster</a></li> | ||
| <li><a href="orchestrate-cockroachdb-with-kubernetes-multi-cluster.html">Orchestrate CockroachDB Across Multiple Kubernetes Clusters</a></li> |
There was a problem hiding this comment.
my instinct is to retain this section and just delete the link to the one page we're removing in this PR
i bet you and @jhlodin are more aware of current state of our k8s world
but from pure tech writerly POV I'd just remove the first link to a page being deleted
There was a problem hiding this comment.
Should leave these sections across all three operating systems but wholly replace their contents with "To orchestrate CockroachDB using Kubernetes, use the official CockroachDB operator" linking to https://www.cockroachlabs.com/docs/stable/cockroachdb-operator-overview#cockroachdb-operator
| "/cockroachcloud/egress-private-endpoints.html" | ||
| ] | ||
| }, | ||
| { |
There was a problem hiding this comment.
i was wondering why just in this version, then remembered that per our discussions at offsite we're only deleting for v26.1+
rmloveland
left a comment
rmloveland
left a comment
There was a problem hiding this comment.
LGTM!
I had one q re: removing a whole k8s block in the install page instead of the one link but nothing blocking
jhlodin
left a comment
jhlodin
left a comment
There was a problem hiding this comment.
Confirmed that we should be clear to remove the local k8s deployment page because its contents are covered in this section of the docs. We do need to keep a mention of Kubernetes in the get started content though, and it's a good opportunity to update that content to link to the new CRDB operator docs.
| <ul> | ||
| <li><a href="orchestrate-a-local-cluster-with-kubernetes.html">Orchestrate CockroachDB Locally with Minikube</a></li> | ||
| <li><a href="deploy-cockroachdb-with-kubernetes.html">Orchestrate CockroachDB in a Single Kubernetes Cluster</a></li> | ||
| <li><a href="orchestrate-cockroachdb-with-kubernetes-multi-cluster.html">Orchestrate CockroachDB Across Multiple Kubernetes Clusters</a></li> |
There was a problem hiding this comment.
Should leave these sections across all three operating systems but wholly replace their contents with "To orchestrate CockroachDB using Kubernetes, use the official CockroachDB operator" linking to https://www.cockroachlabs.com/docs/stable/cockroachdb-operator-overview#cockroachdb-operator
|
Please hold: need to add a piece of content to the kubernetes page we're redirecting to. Will commit directly to this PR |
jhlodin
force-pushed
the
20260219-doc-16178-page-deletion
branch
from
66daa48 to
12307aa
Compare
jhlodin
left a comment
jhlodin
left a comment
There was a problem hiding this comment.
Added callouts that the reader should use minikube to evaluate a kubernetes deployment locally, covering the content gap added by removing the local deployment. LGTM, but please review the contents of my commit before merging.
|
Looks good, thanks Joe |
3 participants
Fixes: DOC-16178
Removes 5 pages as determined by the onsite page deletion project:
cockroachcloud/learn-cockroachdb-sql.md
stable/security-reference/config-secure-hba.md
stable/satori-integration.md
stable/orchestrate-a-local-cluster-with-kubernetes.md
stable/orchestrate-a-local-cluster-with-kubernetes-insecure.md