Skip to content

refactor(database): unify KubeBlocks 0.8 and 0.9 workspaces - #25

Open
mlhiter wants to merge 1 commit into
mainfrom
accra/release-v5.1-main
Open

mlhiter wants to merge 1 commit into
mainfrom
accra/release-v5.1-main

Conversation

@mlhiter

@mlhiter mlhiter commented Oct 25, 2026

Copy link
Copy Markdown
Collaborator

Summary

Consolidate the main and release-v5.1 source trees into one Next.js workspace and runtime image, selecting complete KubeBlocks 0.8 or 0.9 UI/API implementations from validated mounted configuration while preserving legacy configuration behavior.
Retain both deployment profiles and publishing channels, propagate the version selector through Helm and installer inputs, and update CI, bundle assembly, documentation, and compatibility tests.

Validation

Passed with Node 20.20.2: TypeScript checking, 20 Vitest tests, 44 Node contract tests, 14 Python deployment/bundle tests, both Helm chart lint checks, shell syntax checks, and git diff checks; the commit hook also passed its secret scan.
Container builds and live deployment acceptance were not run locally.

Sequence Diagram

sequenceDiagram
    participant Operator
    participant Helm
    participant Config as Mounted configuration
    participant App as Unified Database UI/API
    participant Implementation as Version implementation
    Operator->>Helm: Select deployment profile and optional version override
    Helm->>Config: Render database.kubeblocks.version
    App->>Config: Load and validate configuration
    alt KubeBlocks 0.8
        App->>Implementation: Dispatch to kb08 UI/API
    else KubeBlocks 0.9
        App->>Implementation: Dispatch to kb09 UI/API
    end
    alt Operation available
        Implementation-->>App: Version-specific response
    else Operation unavailable in selected version
        App-->>Operator: 404 without cross-version fallback
    end
Loading

Comment thread .github/workflows/ci.yml
- uses: actions/setup-node@v4
with:
node-version: '20.20.2'
- uses: azure/setup-helm@v4

@github-advanced-security github-advanced-security AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

CodeQL found more than 20 potential problems in the proposed changes. Check the Files changed tab for more details.

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