Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: 共享集群支持查询集群下所有命名空间的release #3657

Merged
merged 2 commits into from
Jan 14, 2025

Conversation

LidolLxf
Copy link
Contributor

feat: 共享集群支持查询集群下所有命名空间的release

@@ -102,10 +102,6 @@ func (l *ListReleaseV1Action) list() (*helmmanager.ReleaseListData, error) {
dbReleases = append(dbReleases, item.Transfer2Proto())
}

if cluster.IsShared && len(option.Namespace) == 0 {
Copy link
Contributor

Choose a reason for hiding this comment

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

要先获取命名空间列表,然后根据列表并发去获取 release

if err != nil {
return nil, err
}
p, err := cli.Namespace.ListNamespaces(context.Background(), &bcsproject.ListNamespacesRequest{
Copy link
Contributor

Choose a reason for hiding this comment

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

这里的 context 可以传进来

@LidolLxf LidolLxf closed this Jan 14, 2025
@LidolLxf LidolLxf reopened this Jan 14, 2025
@ifooth ifooth merged commit 7f1be1b into TencentBlueKing:master Jan 14, 2025
21 of 22 checks passed
evanlixin added a commit that referenced this pull request Jan 15, 2025
* github-bk-bcs/master:
  feat: 模板文件多个资源模块表单添加默认值 (#3658)
  feat: 共享集群支持查询集群下所有命名空间的release (#3657)
  feat: cluster-resource 封装storage events 接口 (#3665)
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.

3 participants