Skip to content

chore: add user namespace requirement#725

Merged
chalindukodikara merged 1 commit into
openchoreo:mainfrom
chalindukodikara:issue-latest_2026_06_16-1
Jun 16, 2026
Merged

chore: add user namespace requirement#725
chalindukodikara merged 1 commit into
openchoreo:mainfrom
chalindukodikara:issue-latest_2026_06_16-1

Conversation

@chalindukodikara

@chalindukodikara chalindukodikara commented Jun 16, 2026

Copy link
Copy Markdown
Contributor

Purpose

$subject

Related Issues

Include any related issues that are resolved by this PR.

Checklist

  • Updated sidebars.ts if adding a new documentation page
  • Run npm run start to preview the changes locally
  • Run npm run build to ensure the build passes without errors
  • Verified all links are working (no broken links)

Signed-off-by: Chalindu Kodikara <chalindumkodikara@gmail.com>
@coderabbitai

coderabbitai Bot commented Jun 16, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: a520aaac-970b-490a-9597-2749ddd0a347

📥 Commits

Reviewing files that changed from the base of the PR and between a9da6d4 and 2eaf219.

📒 Files selected for processing (2)
  • docs/getting-started/try-it-out/on-k3d-locally.mdx
  • docs/getting-started/try-it-out/on-your-environment.mdx

📝 Walkthrough

Summary by CodeRabbit

  • Documentation
    • Updated minimum Kubernetes version requirement to v1.33+ for local deployment setups
    • Added explicit prerequisites for Linux kernel (6.3+), container runtime versions (containerd 2.0+, CRI-O 1.25+), and related components
    • Added warning about required Kubernetes user namespaces configuration for build workflows
    • Added new verification command to check node component versions

Walkthrough

Two getting-started documentation files are updated to raise the minimum supported Kubernetes and kubectl version from v1.32+ to v1.33+. The environment setup guide also adds a user namespaces required warning with component version minimums (kernel, containerd/CRI-O, runc/crun) and a new node verification command.

Changes

Kubernetes 1.33+ Prerequisite Docs Update

Layer / File(s) Summary
Version bumps and user namespace requirements
docs/getting-started/try-it-out/on-k3d-locally.mdx, docs/getting-started/try-it-out/on-your-environment.mdx
kubectl minimum bumped to v1.33+ in the k3d guide. The environment guide bumps Kubernetes to v1.33+, adds a user namespaces warning (hostUsers: false) with minimum versions for kernel 6.3+, containerd 2.0+/CRI-O 1.25+, and runc 1.2+/crun 1.9+, and inserts a kubectl get nodes verification command showing kernel, container runtime, and kubelet versions per node.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Description check ⚠️ Warning The description is largely incomplete with the Purpose section containing an unreplaced template variable ($subject) and missing substantive content about the changes. Replace the $subject placeholder with a clear description of what the PR does, summarizing the documentation updates to Kubernetes version requirements and user namespace prerequisites.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately reflects the main change: adding user namespace requirements to documentation prerequisites.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@LakshanSS LakshanSS 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.

LGTM

@chalindukodikara chalindukodikara merged commit 13c952d into openchoreo:main Jun 16, 2026
3 checks passed
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