Removed sbcli parts from the Kubernetes authentication page - #204
Merged
Conversation
The Kubernetes page was a copy of the non-Kubernetes page with a CRD section appended, so every sbctl command on it was duplicated from the plain Linux page. The three CLI-driven sections are removed and the page now runs entirely through the StoragePool CR. A closing link points to the plain Linux page for the sbctl flow. The anchor in k8s-storage-plane.md pointed at a heading that no longer exists and is retargeted to the section that carries the allowedNodes reconciliation statement it cites. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
…ding the openapi spec Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
…uilding the openapi spec The image mirrors the sbcli dependencies by hand. sbcli has since added prometheus_fastapi_instrumentator, and ec2-metadata became reachable from simplyblock_web.app, both of which broke the openapi spec generation. py-cpuinfo, flask-openapi3 and graypy are not reachable from the app entry point today, but are required by other simplyblock_web and simplyblock_core modules, so they are added as well to avoid breaking the build again. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Contributor
|
🤖: Deployment available as https://docstest.simplyblock.io/docs-split-k8s-auth-encryption |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
docs/kubernetes/operations/security/authentication-encryption.mdwas a copy of the non-Kubernetes page with a CRD section appended, so every{{ cliname }}command on it was duplicated from the plain Linux page. The Kubernetes aspects are now separate from the sbcli aspects.Changes
docs/kubernetes/operations/security/authentication-encryption.mdstorage-pool add --dhchap,storage-pool add-host/remove-host, andvolume connect --host-nqn. They remain indocs/non-kubernetes/operations/security/authentication-encryption.md, which is unchanged.## Enable Host Authentication and Encryptionnow carries thedhchap/allowedNodesYAML example instead of the CLI command.## Reconciliation by the Operatorfor what the operator does in place of the manual steps, and## Managing Allowed Nodesfor the immutability rules.{{ cliname }}flow, matching howmulti-tenancy.mdandstorage-class.mdcross-reference.docs/kubernetes/installation/k8s-storage-plane.md#configuring-dhchap-via-the-storagepool-crdheading and is retargeted to#managing-allowed-nodes, where theallowedNodesreconciliation statement it cites now lives.Quality gates
./scripts/quality-gate.shreports no new findings on either file. The 17 syntax errors it reports are pre-existing links intodocs/reference/cli/, which is generated from the sbcli repository and not present in the working tree.🤖 Generated with Claude Code