Skip to content

docs: restructure CVF documentation and eliminate duplication - #106

Open
yuva29 wants to merge 1 commit into
mainfrom
restructure-cvf-docs
Open

docs: restructure CVF documentation and eliminate duplication#106
yuva29 wants to merge 1 commit into
mainfrom
restructure-cvf-docs

Conversation

@yuva29

@yuva29 yuva29 commented Aug 12, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Move roce-workload.md into cluster_validation_framework/ to match its TOC section
  • Consolidate prerequisites to CVF README as single source of truth
  • Clear ownership: docs page owns usage/deployment, docker README owns building
  • Fix heading hierarchy in CVF README for correct Sphinx rendering
  • Update all cross-references and TOC for new file location

🤖 Generated with Claude Code

@yuva29
yuva29 requested review from sundar-pds and a lite review from Copilot August 12, 2026 04:18

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR restructures the Cluster Validation Framework (CVF) documentation so the RoCE workload docs live under the CVF section, removes duplicated prerequisite/build guidance, and updates navigation/cross-references to match the new layout.

Changes:

  • Update Sphinx TOC to reference cluster_validation_framework/roce-workload under the CVF subtree.
  • Update installation docs and CVF README cross-links to point at the new RoCE workload documentation location.
  • Refactor docs/cluster_validation_framework/roce-workload.md to clearly separate “getting/building” vs “using” (manual quick start vs CVF automation), and simplify docker/roce-workload/README.md to point to the docs site.

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
docs/sphinx/_toc.yml.in Updates TOC entry to match the RoCE workload doc’s new location under the CVF subtree.
docs/installation/workload.md Updates the “RoCE Workload Image” link to the new CVF doc path.
docs/cluster_validation_framework/roce-workload.md Reorganizes RoCE workload documentation (image acquisition/build + usage, manual quick start, CVF automation).
docs/cluster_validation_framework/README.md Fixes heading hierarchy for Sphinx rendering and updates references to the RoCE workload doc.
docker/roce-workload/README.md Removes duplicated deployment/usage content and redirects readers to the central docs page.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

# Cluster Validation and Job Scheduling Framework

The **Cluster Validation and Job Scheduling Framework** is a Kubernetes-based framework designed to periodically verify the health and readiness of worker nodes present in the cluster through active tests. This framework can also be used for validating a set of newly added worker nodes to the kubernetes cluster before scheduling distributed training,inference workloads on them.
The **Cluster Validation and Job Scheduling Framework** is a Kubernetes-based framework designed to periodically verify the health and readiness of worker nodes present in the cluster through active tests. This framework can also be used for validating a set of newly added worker nodes to the kubernetes cluster before scheduling distributed training,inference workloads on them.
Comment thread docker/roce-workload/README.md Outdated
Move roce-workload.md into cluster_validation_framework/ to match TOC
section. Consolidate prerequisites to CVF README as single source of
truth. Establish clear ownership: docs page for usage/deployment, docker
README for building. Fix heading hierarchy and cross-references.

Co-Authored-By: Claude Opus 4 <noreply@anthropic.com>
@yuva29
yuva29 force-pushed the restructure-cvf-docs branch from f5eca70 to e51914b Compare August 12, 2026 04:44
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