chore: add .catalog registration for harbor-cli#15
Conversation
Add Backstage catalog YAML following the Confluence catalog collection standard. Registers harbor-cli as a tool component with open-source metadata (upstream: goharbor/harbor-cli, Apache-2.0). No container images — CLI tool only. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
| acp.cpaas.io/open-source-component-license: Apache-2.0 | ||
| # 开源组件的license地址,格式 ${url}/${project}/${repo}/${license-path} | ||
| acp.cpaas.io/open-source-component-license_url: https://github.com/goharbor/harbor-cli/blob/main/LICENSE | ||
| acp.cpaas.io/owner: jtcheng@alauda.io |
There was a problem hiding this comment.
Critical (bug/format): Incorrect value format. The spec.system field should contain just the system name (devops-tools), not the prefixed format (system:devops-tools). According to Backstage spec, spec.system is an optional field that references the system name directly. Using the system: prefix may cause validation failures.
| acp.cpaas.io/owner: jtcheng@alauda.io | |
| system: devops-tools |
|
|
||
| # 组件属性，区别是否部署时必带的核心组件,可选值为 core 和 plugin。 | ||
| acp.cpaas.io/functional-attributes: plugin | ||
|
|
There was a problem hiding this comment.
Suggestion (refactor/consistency): Consider aligning spec.owner (devops) with metadata.annotations.acp.cpaas.io/owner (jtcheng@alauda.io) for consistency in ownership representation across the catalog entity.
🤖 AI Code Review
SummaryThis PR adds a Backstage catalog registration file ( Review Statistics
Critical Issues
Warnings
(None) Suggestions
Positive FeedbackThe catalog file is well-structured and follows the Backstage Component entity format correctly. The annotations include proper ACP-specific metadata for the Alauda platform (open-source component info, license, owner, functional attributes). The file is properly placed in the ℹ️ About this reviewThis review was automatically generated using the
|
2 participants
Summary
.catalog/harbor-cli.yamlfollowing the Confluence catalog collection standardtoolcomponent (CLI, no container images)Test plan
🤖 Generated with Claude Code