Skip to content

docs: clarify that workspace name derives the Kubernetes namespace#1606

Open
redhat-chai-bot wants to merge 1 commit into
ambient-code:mainfrom
redhat-chai-bot:05270555-fix-workspace-name-docs
Open

docs: clarify that workspace name derives the Kubernetes namespace#1606
redhat-chai-bot wants to merge 1 commit into
ambient-code:mainfrom
redhat-chai-bot:05270555-fix-workspace-name-docs

Conversation

@redhat-chai-bot
Copy link
Copy Markdown
Collaborator

@redhat-chai-bot redhat-chai-bot commented May 27, 2026

The docs described the workspace name field as just a "Display name". In reality, the platform derives the project name (Kubernetes namespace) from it. This name is globally unique on the cluster, immutable after creation, and subject to Kubernetes naming rules.

Changes

  • Updated the creation table to note that the display name derives the internal project name
  • Added a new Workspace name and project name section documenting:
    • The derivation rules (generateWorkspaceName)
    • Global uniqueness constraint (409 Conflict on collision)
    • Immutability after creation
  • Updated the General settings section to list the project name as read-only

Slack thread: https://redhat-internal.enterprise.slack.com/archives/C09KMNSNUDT/p1779781417071479

Summary by CodeRabbit

  • Documentation
    • Updated workspace creation documentation to clarify the relationship between workspace name and the derived Kubernetes project name, including derivation rules, examples, and immutability details.
    • Clarified that workspace display name is editable while project name is read-only after creation.

The docs described the workspace name field as just a "Display name".
In reality, the platform derives the project name (Kubernetes namespace)
from it. This is globally unique, immutable after creation, and subject
to K8s naming rules.

Add a new section documenting the derivation rules, uniqueness
constraint, and immutability. Update the creation table and General
settings section accordingly.
@netlify
Copy link
Copy Markdown

netlify Bot commented May 27, 2026

Deploy Preview for cheerful-kitten-f556a0 ready!

Name Link
🔨 Latest commit 6dccdc8
🔍 Latest deploy log https://app.netlify.com/projects/cheerful-kitten-f556a0/deploys/6a16877cf4e1920008d2bdb9
😎 Deploy Preview https://deploy-preview-1606--cheerful-kitten-f556a0.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented May 27, 2026

📝 Walkthrough

Walkthrough

Documentation updated to clarify workspace naming conventions. The "Creating a workspace" section now explains that "Workspace Name" input yields a derived Kubernetes "project name," including derivation rules, collision behavior, and immutability. The "Workspace settings" section distinguishes editable fields from the read-only "project name."

Changes

Workspace Field Documentation

Layer / File(s) Summary
Workspace creation and settings field documentation
docs/src/content/docs/concepts/workspaces.md
Workspace creation section documents "Workspace Name" field and explains the derived "project name" (derivation rules, example, global uniqueness, immutability). Workspace settings section distinguishes editable name/description from read-only project name.
🚥 Pre-merge checks | ✅ 8
✅ Passed checks (8 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed Title follows Conventional Commits format (docs: scope) and accurately describes the main documentation change about workspace name deriving the Kubernetes namespace.
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.
Performance And Algorithmic Complexity ✅ Passed PR is documentation-only (markdown file). No code changes, algorithms, loops, API calls, or performance-sensitive logic present. Check criteria are not applicable.
Security And Secret Handling ✅ Passed PR is documentation-only (workspace.md) with no code/config changes. No hardcoded secrets, credentials, or API keys in plaintext; proper placeholder usage and security best practices documented.
Kubernetes Resource Safety ✅ Passed PR modifies only markdown documentation (workspaces.md). No Kubernetes manifests, resources, RBAC rules, or container configs are changed, so the Kubernetes Resource Safety check is not applicable.

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

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

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.

Copy link
Copy Markdown
Contributor

@coderabbitai coderabbitai Bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@docs/src/content/docs/concepts/workspaces.md`:
- Around line 53-54: The docs claim "Workspace name (display name) and
description — Update at any time" but this is only true on OpenShift; on vanilla
Kubernetes these fields are not editable. Update the text for the "Workspace
name (display name) and description" entry in
docs/src/content/docs/concepts/workspaces.md to clearly state the difference
(e.g., "OpenShift: editable at any time; vanilla Kubernetes: read-only after
creation") and/or link to the "Workspace name and project name" section for
platform-specific behavior so readers know the editability depends on the
underlying platform.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Enterprise

Run ID: 068b9d0b-5070-48c8-ab8e-1269579107a8

📥 Commits

Reviewing files that changed from the base of the PR and between d45e631 and 6dccdc8.

📒 Files selected for processing (1)
  • docs/src/content/docs/concepts/workspaces.md

Comment on lines +53 to +54
- **Workspace name (display name) and description** -- Update at any time.
- **Project name** -- The Kubernetes namespace for this workspace. Read-only after creation (see [Workspace name and project name](#workspace-name-and-project-name)).
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.

⚠️ Potential issue | 🟠 Major | ⚡ Quick win

General settings editability is overstated for vanilla Kubernetes.

Line 53 says “Workspace name (display name) and description — Update at any time,” but current behavior is OpenShift-only; on vanilla Kubernetes these fields are not editable. This creates a user-facing docs mismatch.

Proposed doc fix
-- **Workspace name (display name) and description** -- Update at any time.
+- **Workspace display name and description** -- Editable on OpenShift clusters.
+  On vanilla Kubernetes, these metadata fields are not editable in the UI.
📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
- **Workspace name (display name) and description** -- Update at any time.
- **Project name** -- The Kubernetes namespace for this workspace. Read-only after creation (see [Workspace name and project name](#workspace-name-and-project-name)).
- **Workspace display name and description** -- Editable on OpenShift clusters.
On vanilla Kubernetes, these metadata fields are not editable in the UI.
- **Project name** -- The Kubernetes namespace for this workspace. Read-only after creation (see [Workspace name and project name](`#workspace-name-and-project-name`)).
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@docs/src/content/docs/concepts/workspaces.md` around lines 53 - 54, The docs
claim "Workspace name (display name) and description — Update at any time" but
this is only true on OpenShift; on vanilla Kubernetes these fields are not
editable. Update the text for the "Workspace name (display name) and
description" entry in docs/src/content/docs/concepts/workspaces.md to clearly
state the difference (e.g., "OpenShift: editable at any time; vanilla
Kubernetes: read-only after creation") and/or link to the "Workspace name and
project name" section for platform-specific behavior so readers know the
editability depends on the underlying platform.

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.

1 participant