Conversation
There was a problem hiding this comment.
Pull request overview
This PR refreshes the documentation site structure/content and expands the AWS Terraform “kubeadm on EC2” stack with configurable node IAM policies, optional AWS Client VPN access, additional outputs, and an operator-focused terraform/Makefile workflow. It also updates Clouder CLI branding/help text and adds a clouder about command.
Changes:
- AWS Terraform: configurable node managed-policy attachments, optional Client VPN resources, richer outputs, and a dedicated
terraform/Makefileworkflow. - Docs: reorganize and expand Services/Deployments/Operations/Integrations content; add new Addons and Architecture pages; refresh links and screenshots.
- CLI: add
clouder aboutand adjust “CRIU support” wording in help/docs.
Reviewed changes
Copilot reviewed 70 out of 83 changed files in this pull request and generated 5 comments.
Show a summary per file
| File | Description |
|---|---|
| terraform/variables.tf | Adds variables for node IAM policies and optional Client VPN configuration. |
| terraform/terraform.tfvars.example | Documents new tfvars for IAM policies and Client VPN. |
| terraform/outputs.tf | Extends outputs for network/VPN and IAM role/profile metadata. |
| terraform/modules/aws/network/variables.tf | Adds Client VPN input variables to the network module. |
| terraform/modules/aws/network/outputs.tf | Exposes IGW/route table/VPN-related outputs. |
| terraform/modules/aws/network/main.tf | Implements optional AWS Client VPN resources and subnet tagging. |
| terraform/modules/aws/iam/variables.tf | Adds configurable node managed policy ARN list input. |
| terraform/modules/aws/iam/outputs.tf | Outputs the attached managed policy ARNs. |
| terraform/modules/aws/iam/main.tf | Switches to for_each attachments for configurable managed policies. |
| terraform/Makefile | Adds Terraform lifecycle/guardrail targets for kubeadm workflow. |
| terraform/main.tf | Wires new network and IAM inputs into modules. |
| README.md | Fixes CRIU link text (“checkpoint and restore”). |
| Makefile | Removes root-level Makefile. |
| docs/docs/services/index.mdx | Reworks Services landing page and Terraform-first snippet. |
| docs/docs/services/14-addons/index.mdx | Adds Addons landing page. |
| docs/docs/services/14-addons/2-ray-cluster/index.mdx | Adds Ray Cluster addon guide. |
| docs/docs/services/14-addons/2-ray-cluster/category.yml | Adds Ray Cluster addon category metadata. |
| docs/docs/services/14-addons/1-stripe/index.mdx | Adds Stripe addon documentation. |
| docs/docs/services/14-addons/1-stripe/category.yml | Adds Stripe addon category metadata. |
| docs/docs/services/14-addons/category.yml | Adjusts Addons section position. |
| docs/docs/services/13-status/index.mdx | Updates Status OpenAPI docs link. |
| docs/docs/services/13-status/category.yml | Adjusts Status section position. |
| docs/docs/services/12-manager/index.mdx | Updates Manager OpenAPI docs link. |
| docs/docs/services/12-manager/category.yml | Adjusts Manager section position. |
| docs/docs/services/11-spider/index.mdx | Updates Spider title marker. |
| docs/docs/services/11-spider/category.yml | Adjusts Spider section position. |
| docs/docs/services/10-scheduler/index.mdx | Updates Scheduler title marker. |
| docs/docs/services/10-scheduler/category.yml | Adjusts Scheduler section position. |
| docs/docs/services/09-functions/index.mdx | Updates Functions title marker. |
| docs/docs/services/09-functions/category.yml | Adjusts Functions section position. |
| docs/docs/services/08-ai-inference/index.mdx | Adds AI Inference page stub. |
| docs/docs/services/08-ai-inference/category.yml | Adds AI Inference category metadata. |
| docs/docs/services/07-ai-agents/index.mdx | Updates AI Agents OpenAPI docs link. |
| docs/docs/services/06-library/index.mdx | Updates Library “REST API” label and OpenAPI docs link. |
| docs/docs/services/05-spacer/index.mdx | Updates Spacer “REST API” label and OpenAPI docs link. |
| docs/docs/services/04-runtimes/index.mdx | Updates Runtimes OpenAPI docs link (host/path). |
| docs/docs/services/03-operator/index.mdx | Adds Operator service page with deploy instructions. |
| docs/docs/services/03-operator/category.yml | Adds Operator category metadata. |
| docs/docs/services/02-iam/index.mdx | Updates IAM OpenAPI docs link. |
| docs/docs/services/02-iam/category.yml | Adds IAM category metadata. |
| docs/docs/operations/index.mdx | Reworks Operations landing content and recommended sequence. |
| docs/docs/integrations/index.mdx | Reworks Integrations landing content and sequence. |
| docs/docs/integrations/1-jupyterhub/index.mdx | Comments out large Helm install block using MDX comment syntax. |
| docs/docs/index.mdx | Updates docs homepage title/content and onboarding link targets. |
| docs/docs/deployments/index.mdx | Expands Deployments landing page with recommended paths/order. |
| docs/docs/deployments/5-gitub-actions/index.mdx | Adjusts GitHub Actions deployment page header. |
| docs/docs/deployments/4-terraform/index.mdx | Major expansion of Terraform deployment guide and Makefile workflow. |
| docs/docs/deployments/2-plane/index.mdx | Adjusts Plane deployment page header. |
| docs/docs/deployments/1-configuration/index.mdx | Updates minimal configuration wording (Jupyter → Runtimes). |
| docs/docs/cluster/3-kubernetes/8-namespaces/index.mdx | Updates namespaces image URLs. |
| docs/docs/cluster/3-kubernetes/3-azure-aks/index.mdx | Updates AKS image URLs. |
| docs/docs/cluster/3-kubernetes/2-aws-eks/index.mdx | Adds frontmatter, updates title casing, updates image URLs, comments large block. |
| docs/docs/cluster/1-requirements/9-license/category.yml | Updates label and whitespace. |
| docs/docs/cluster/1-requirements/8-terraform/category.yml | Updates Terraform CLI label. |
| docs/docs/cluster/1-requirements/7-openfga/index.mdx | Adds OpenFGA CLI requirement page. |
| docs/docs/cluster/1-requirements/7-openfga/category.yml | Updates OpenFGA category position. |
| docs/docs/cluster/1-requirements/6-cmctl/index.mdx | Adds cmctl CLI requirement page. |
| docs/docs/cluster/1-requirements/6-cmctl/category.yml | Updates cmctl category position. |
| docs/docs/cluster/1-requirements/5-helm/index.mdx | Adds Helm CLI requirement page. |
| docs/docs/cluster/1-requirements/5-helm/category.yml | Updates Helm category position. |
| docs/docs/cluster/1-requirements/4-kubectl/index.mdx | Adds kubectl CLI requirement page. |
| docs/docs/cluster/1-requirements/4-kubectl/category.yml | Updates kubectl category position. |
| docs/docs/cluster/1-requirements/3-datalayer/index.mdx | Adds optional Datalayer CLI requirement page. |
| docs/docs/cluster/1-requirements/3-datalayer/category.yml | Updates Datalayer CLI category position. |
| docs/docs/cluster/1-requirements/2-plane/index.mdx | Updates Plane CLI heading. |
| docs/docs/cluster/1-requirements/2-plane/category.yml | Updates Plane CLI category position. |
| docs/docs/cluster/1-requirements/1-clouder/index.mdx | Updates Clouder CLI snippet and comments out dev block. |
| docs/docs/cluster/1-requirements/1-clouder/category.yml | Updates Clouder CLI category position. |
| docs/docs/cli/kubeadm.mdx | Removes “CRIU support” wording from kubeadm CLI docs. |
| docs/docs/cli/index.mdx | Removes “CRIU support” wording from CLI index table. |
| docs/docs/architecture/5-byon/index.mdx | Adds BYON architecture page. |
| docs/docs/architecture/5-byon/category.yml | Adds BYON category metadata. |
| docs/docs/architecture/4-byoc/index.mdx | Adds BYOC architecture page. |
| docs/docs/architecture/4-byoc/category.yml | Adds BYOC category metadata. |
| docs/docs/architecture/3-user-storage/index.mdx | Updates user storage architecture text and fixes “Linux” spelling. |
| clouder/sbin/header.sh | Updates copyright URL to datalayer.ai. |
| clouder/sbin/about.sh | Updates/about banner output and copyright URL. |
| clouder/cli/main.py | Updates CLI help text and adds clouder about command. |
| clouder/cli/kubeadm/setup.py | Updates kubeadm setup command docstring summary wording. |
Comments suppressed due to low confidence (1)
clouder/cli/kubeadm/setup.py:47
- The updated docstring removes "with CRIU support" from the summary line, but the subsequent Steps list still explicitly says "enable CRIU feature gates". Please make these consistent (either keep CRIU in the summary, or update the steps text if CRIU is no longer part of kubeadm setup).
"""Set up a kubeadm cluster on previously created VMs.
Steps: install prerequisites → kubeadm init (master) → install CNI →
kubeadm join (workers) → enable CRIU feature gates (all nodes).
"""
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
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.
No description provided.