Skip to content

fix(template): preserve separated manifests and assets - #7226

Merged
cuisongliu merged 3 commits into
labring:release-v5.1from
mlhiter:cluster-62-app-assets-bug
Aug 16, 2026
Merged

fix(template): preserve separated manifests and assets#7226
cuisongliu merged 3 commits into
labring:release-v5.1from
mlhiter:cluster-62-app-assets-bug

Conversation

@mlhiter

@mlhiter mlhiter commented Aug 16, 2026

Copy link
Copy Markdown
Member

Summary

Fix template deployments and catalog/instance asset responses for repositories that separate template metadata from Kubernetes manifests. The provider now loads nested manifests/**/*.yaml files, excludes them from catalog indexing, and proxies repository-hosted icons through a browser-compatible asset endpoint. The template repository layout is documented and focused unit coverage is included.

Verification

  • git diff --check origin/release-v5.1...HEAD passed.
  • The template build could not run because the existing workspace dependency @sealos/gtm is unavailable, and Vitest is not declared or installed in the workspace.

Sequence Diagram

sequenceDiagram
    participant Browser
    participant TemplateAPI
    participant Repo as Template Repository
    participant AssetAPI

    Browser->>TemplateAPI: Request template or instance data
    TemplateAPI->>Repo: Read index metadata and nested manifests
    Repo-->>TemplateAPI: Return deployment source and asset references
    TemplateAPI->>AssetAPI: Rewrite repository icon URL
    AssetAPI-->>Browser: Serve icon with browser-compatible response
    TemplateAPI-->>Browser: Return separated deployment source and metadata
Loading

@mlhiter
mlhiter requested a review from a team as a code owner August 16, 2026 03:18
@cuisongliu cuisongliu added this to the v5.1 milestone Aug 16, 2026
@cuisongliu
cuisongliu merged commit f43e414 into labring:release-v5.1 Aug 16, 2026
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants