Skip to content
This repository has been archived by the owner on Sep 30, 2024. It is now read-only.

Docs: update docs for src validate command to add new kube subcommand #46794

Merged
merged 3 commits into from
Jan 25, 2023

Conversation

jdpleiness
Copy link
Contributor

This is a documentation update for the new kube subcommand for src validate that will be introduced in sourcegraph/src-cli#926

  • Updated examples with new commands and configuration options
  • Updated CLI reference pages

Add docs to CLI reference pages

  • cli/references/validate

Screenshot 2023-01-23 at 2 12 19 PM

  • cli/references/validate/kube

Screenshot 2023-01-23 at 2 12 54 PM

  • cli/references/validate/install

Screenshot 2023-01-23 at 2 13 08 PM

Add docs to admin/validation page

localhost_5080_admin_validation (1)

Test plan

Run yarn docsite:serve to stand up documentation.

@sourcegraph-bot
Copy link
Contributor

sourcegraph-bot commented Jan 23, 2023

Codenotify: Notifying subscribers in CODENOTIFY files for diff dc4eee6...b32e105.

Notify File(s)
@sourcegraph/delivery doc/admin/validation.md

* Persistent Volume Claim (PVC) validation
* Inter-Service network connection validation

These validations can also include warnings for non-failure states that should be address, e.g. high restart counts.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
These validations can also include warnings for non-failure states that should be address, e.g. high restart counts.
These validations can also include warnings for non-failure states that should be addressed, e.g. high restart counts.

@jdpleiness jdpleiness merged commit 24e5ce9 into main Jan 25, 2023
@jdpleiness jdpleiness deleted the jdp/src-validate-kube-docs branch January 25, 2023 14:38
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants