Skip to content

chore: add upgrade workbench AI-23623#94

Merged
EdisonSu768 merged 1 commit intomasterfrom
feat/update-workbench
Feb 9, 2026
Merged

chore: add upgrade workbench AI-23623#94
EdisonSu768 merged 1 commit intomasterfrom
feat/update-workbench

Conversation

@EdisonSu768
Copy link
Copy Markdown
Member

@EdisonSu768 EdisonSu768 commented Feb 5, 2026

Summary by CodeRabbit

  • Documentation
    • Expanded upgrade guide with new sections covering cluster plugin upgrades and enabling fine-tuning/training features.
    • Added detailed step-by-step workflows and verification checks for Alauda AI components.
    • Improved organization and formatting for enhanced readability.

@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Feb 5, 2026

Walkthrough

This PR updates documentation for the Alauda AI upgrade process. It converts quote styles in version constants from double to single quotes and substantially expands the upgrade guide with new sections covering cluster plugins, fine-tuning features, and verification steps.

Changes

Cohort / File(s) Summary
Documentation Update
docs/en/upgrade/upgrade-from-previous-version.mdx
Expanded upgrade guide with new sections for upgrading cluster plugins, enabling fine-tuning/training features, and verification procedures. Updated exported version constants (prevVersion, curVer) from double-quoted to single-quoted strings. Added informational callouts, YAML examples, and CLI verification steps.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Possibly related PRs

Suggested reviewers

  • zhaomingkun1030
  • typhoonzero

Poem

🐰 A rabbit's tale of upgrade guides so bright,
New sections bloom like clover in the light,
With quotes now single, and features newly told,
Our documentation grows, more helpful and bold!

🚥 Pre-merge checks | ✅ 2 | ❌ 1
❌ Failed checks (1 inconclusive)
Check name Status Explanation Resolution
Title check ❓ Inconclusive The title references 'add upgrade workbench' but the changes are substantial documentation updates to an upgrade guide with new sections on cluster plugins and fine-tuning features, not primarily about adding a workbench component. Clarify whether the title should reflect the main documentation updates (e.g., 'docs: expand upgrade guide with cluster plugins and fine-tuning steps') or if 'add upgrade workbench' accurately describes the primary objective.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch feat/update-workbench

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

@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: 3

🤖 Fix all issues with AI agents
In `@docs/en/upgrade/upgrade-from-previous-version.mdx`:
- Around line 133-138: The closing admonition marker for the MDX note block (the
`:::` that follows the list mentioning Volcano and MLflow) is indented and may
not close the block correctly; remove any leading spaces so the closing `:::`
starts at the beginning of the line (no indentation) to properly terminate the
`:::note` block.
- Around line 127-131: The text inside the Callouts component has awkward
mid-sentence breaks; update the content of the <Callouts> block so each numbered
item is a single continuous line or use a proper ordered list (e.g., "1. When
set to `true`, the **Training** item appears in the left navigation menu." on
its own line and "2. When set to `true`, the **Fine-Tuning** item appears in the
left navigation menu." on the next line) to remove the "menu. 2." break and
improve readability.
- Around line 96-98: The documentation currently uses two different tab names
("Functional Components" and "Features") — check the actual UI to confirm the
correct tab label and make them consistent; update the text in
upgrade-from-previous-version.mdx to use the verified tab name in both places
(replace the "Functional Components" mention or the "Features" mention
accordingly) and ensure any surrounding context (e.g., "cluster details page" or
"Features tab of the cluster details page") matches the UI wording exactly.

Comment on lines +96 to +98
:::info
Once the new version is uploaded and recognized by the platform, the **Upgrade** button will become available for each plugin on the **Features** tab of the cluster details page.
:::
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

⚠️ Potential issue | 🟡 Minor

Potential inconsistency in tab name reference.

Line 89 refers to the "Functional Components tab" but line 97 mentions the "Features tab". Please verify these are the correct tab names or if they should be consistent.

Suggested fix if they should be the same
 :::info
-Once the new version is uploaded and recognized by the platform, the **Upgrade** button will become available for each plugin on the **Features** tab of the cluster details page.
+Once the new version is uploaded and recognized by the platform, the **Upgrade** button will become available for each plugin on the **Functional Components** tab of the cluster details page.
 :::
📝 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
:::info
Once the new version is uploaded and recognized by the platform, the **Upgrade** button will become available for each plugin on the **Features** tab of the cluster details page.
:::
:::info
Once the new version is uploaded and recognized by the platform, the **Upgrade** button will become available for each plugin on the **Functional Components** tab of the cluster details page.
:::
🤖 Prompt for AI Agents
In `@docs/en/upgrade/upgrade-from-previous-version.mdx` around lines 96 - 98, The
documentation currently uses two different tab names ("Functional Components"
and "Features") — check the actual UI to confirm the correct tab label and make
them consistent; update the text in upgrade-from-previous-version.mdx to use the
verified tab name in both places (replace the "Functional Components" mention or
the "Features" mention accordingly) and ensure any surrounding context (e.g.,
"cluster details page" or "Features tab of the cluster details page") matches
the UI wording exactly.

Comment on lines +127 to +131
<Callouts>
1. When set to `true`, the **Training** item appears in the left navigation
menu. 2. When set to `true`, the **Fine-Tuning** item appears in the left
navigation menu.
</Callouts>
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

⚠️ Potential issue | 🟡 Minor

Awkward line breaks in Callouts content.

The numbered items inside the <Callouts> component have line breaks in the middle of sentences, making them hard to read. The text "menu. 2." on line 129 shows where item 1 ends and item 2 begins mid-line.

Suggested formatting fix
 <Callouts>
-  1. When set to `true`, the **Training** item appears in the left navigation
-  menu. 2. When set to `true`, the **Fine-Tuning** item appears in the left
-  navigation menu.
+  1. When set to `true`, the **Training** item appears in the left navigation menu.
+  2. When set to `true`, the **Fine-Tuning** item appears in the left navigation menu.
 </Callouts>
📝 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
<Callouts>
1. When set to `true`, the **Training** item appears in the left navigation
menu. 2. When set to `true`, the **Fine-Tuning** item appears in the left
navigation menu.
</Callouts>
<Callouts>
1. When set to `true`, the **Training** item appears in the left navigation menu.
2. When set to `true`, the **Fine-Tuning** item appears in the left navigation menu.
</Callouts>
🤖 Prompt for AI Agents
In `@docs/en/upgrade/upgrade-from-previous-version.mdx` around lines 127 - 131,
The text inside the Callouts component has awkward mid-sentence breaks; update
the content of the <Callouts> block so each numbered item is a single continuous
line or use a proper ordered list (e.g., "1. When set to `true`, the
**Training** item appears in the left navigation menu." on its own line and "2.
When set to `true`, the **Fine-Tuning** item appears in the left navigation
menu." on the next line) to remove the "menu. 2." break and improve readability.

Comment on lines +133 to +138
:::note
After enabling these features, ensure that the required cluster plugins are installed:

- **Volcano** cluster plugin for training job scheduling
- **MLflow** cluster plugin for training experiment monitoring (requires PostgreSQL)
:::
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

⚠️ Potential issue | 🟡 Minor

Note block closing marker may be incorrectly indented.

The ::: closing marker on line 138 appears to be indented (preceded by spaces). In MDX, the closing fence should typically start at the beginning of the line to properly close the admonition block. This could cause rendering issues.

Suggested fix
 :::note
 After enabling these features, ensure that the required cluster plugins are installed:

 - **Volcano** cluster plugin for training job scheduling
 - **MLflow** cluster plugin for training experiment monitoring (requires PostgreSQL)
-  :::
+:::
📝 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
:::note
After enabling these features, ensure that the required cluster plugins are installed:
- **Volcano** cluster plugin for training job scheduling
- **MLflow** cluster plugin for training experiment monitoring (requires PostgreSQL)
:::
:::note
After enabling these features, ensure that the required cluster plugins are installed:
- **Volcano** cluster plugin for training job scheduling
- **MLflow** cluster plugin for training experiment monitoring (requires PostgreSQL)
:::
🤖 Prompt for AI Agents
In `@docs/en/upgrade/upgrade-from-previous-version.mdx` around lines 133 - 138,
The closing admonition marker for the MDX note block (the `:::` that follows the
list mentioning Volcano and MLflow) is indented and may not close the block
correctly; remove any leading spaces so the closing `:::` starts at the
beginning of the line (no indentation) to properly terminate the `:::note`
block.

@cloudflare-workers-and-pages
Copy link
Copy Markdown

Deploying alauda-ai with  Cloudflare Pages  Cloudflare Pages

Latest commit: cb2f710
Status: ✅  Deploy successful!
Preview URL: https://7633a55e.alauda-ai.pages.dev
Branch Preview URL: https://feat-update-workbench.alauda-ai.pages.dev

View logs

@EdisonSu768 EdisonSu768 merged commit f70fa52 into master Feb 9, 2026
3 checks passed
This was referenced Feb 26, 2026
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