Skip to content
lint

feat: cluster-manager支持自定义配置projectcode注解(v1.29.x) #3941

Sign in to view logs

feat: cluster-manager支持自定义配置projectcode注解(v1.29.x)

feat: cluster-manager支持自定义配置projectcode注解(v1.29.x) #3941

Triggered via pull request December 19, 2024 06:57
@yuyudeqiuyuyudeqiu
opened #3649
yuyudeqiu:feat_annokey_cm_129
Status Failure
Total duration 6m 46s
Artifacts –

lint.yml

on: pull_request
bcs-webconsole
4m 24s
bcs-webconsole
bcs-monitor
5m 34s
bcs-monitor
bcs-bscp
4m 15s
bcs-bscp
bcs-cluster-manager
6m 37s
bcs-cluster-manager
bcs-helm-manager
5m 13s
bcs-helm-manager
bcs-project-manager
4m 22s
bcs-project-manager
bcs-user-manager
3m 43s
bcs-user-manager
bcs-cluster-resources
5m 1s
bcs-cluster-resources
Fit to window
Zoom out
Zoom in

Annotations

14 errors and 18 warnings
bcs-cluster-resources: bcs-services/cluster-resources/pkg/action/view/view.go#L434
S1032: should use sort.Strings(...) instead of sort.Sort(sort.StringSlice(...)) (gosimple)
bcs-cluster-resources: bcs-services/cluster-resources/pkg/action/view/view.go#L449
S1032: should use sort.Strings(...) instead of sort.Sort(sort.StringSlice(...)) (gosimple)
bcs-cluster-resources: bcs-services/cluster-resources/pkg/action/view/view.go#L464
S1032: should use sort.Strings(...) instead of sort.Sort(sort.StringSlice(...)) (gosimple)
bcs-cluster-resources
issues found
bcs-cluster-manager: bcs-services/bcs-cluster-manager/internal/remote/nodeman/client.go#L1
: # github.com/Tencent/bk-bcs/bcs-services/bcs-cluster-manager/internal/remote/nodeman [github.com/Tencent/bk-bcs/bcs-services/bcs-cluster-manager/internal/remote/nodeman.test]
bcs-cluster-manager
not enough arguments in call to cli.CloudList
bcs-cluster-manager: bcs-services/bcs-cluster-manager/internal/cloudprovider/aws/tasks/import_cluster.go#L136
Error return value of `(github.com/Tencent/bk-bcs/bcs-services/bcs-cluster-manager/internal/store.ClusterManagerModel).UpdateCluster` is not checked (errcheck)
bcs-cluster-manager: bcs-services/bcs-cluster-manager/internal/cloudprovider/eop/tasks/create_cluster.go#L822
Function 'checkClusterNodesStatus' is too long (130 > 90) (funlen)
bcs-cluster-manager: bcs-services/bcs-cluster-manager/internal/cloudprovider/aws/tasks/create_node_group.go#L316
Function 'CheckCloudNodeGroupStatusTask' is too long (110 > 90) (funlen)
bcs-cluster-manager: bcs-services/bcs-cluster-manager/internal/cloudprovider/qcloud-public/business/tke.go#L927
Function 'CheckClusterAllInstanceStatus' is too long (98 > 90) (funlen)
bcs-cluster-manager: bcs-services/bcs-cluster-manager/internal/cloudprovider/qcloud-public/tasks/updateDesiredNodes.go#L36
Function 'ApplyInstanceMachinesTask' is too long (120 > 90) (funlen)
bcs-cluster-manager: bcs-services/bcs-cluster-manager/internal/actions/cluster/create.go#L440
Function 'Handle' is too long (96 > 90) (funlen)
bcs-cluster-manager: bcs-services/bcs-cluster-manager/internal/store/machinery/task.go#L22
File is not `gci`-ed with --skip-generated -s standard -s default -s prefix(github.com/Tencent/bk-bcs/bcs-services/bcs-cluster-manager) (gci)
bcs-cluster-manager: bcs-services/bcs-cluster-manager/internal/store/machinery/task.go#L26
File is not `gci`-ed with --skip-generated -s standard -s default -s prefix(github.com/Tencent/bk-bcs/bcs-services/bcs-cluster-manager) (gci)
bcs-user-manager
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
bcs-bscp
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
bcs-project-manager
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
bcs-webconsole
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
bcs-cluster-resources
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
bcs-helm-manager
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
bcs-monitor
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
bcs-cluster-manager
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
bcs-cluster-manager: bcs-services/bcs-cluster-manager/internal/store/types/machinery_task.go#L36
exported: exported type Task should have comment or be unexported (revive)
bcs-cluster-manager: bcs-services/bcs-cluster-manager/internal/store/types/machinery_task.go#L13
package-comments: should have a package comment (revive)
bcs-cluster-manager: bcs-services/bcs-cluster-manager/internal/store/machinery/task.go#L13
package-comments: should have a package comment (revive)
bcs-cluster-manager: bcs-services/bcs-cluster-manager/internal/actions/health/clean_history_data.go#L28
exported: exported const DataTask should have comment (or a comment on this block) or be unexported (revive)
bcs-cluster-manager: bcs-services/bcs-cluster-manager/internal/remote/cmdb/types.go#L53
exported: exported const FieldHostIP should have comment (or a comment on this block) or be unexported (revive)
bcs-cluster-manager: bcs-services/bcs-cluster-manager/internal/remote/cmdb/types.go#L61
exported: comment on exported const FieldAssetId should be of the form "FieldAssetId ..." (revive)
bcs-cluster-manager: bcs-services/bcs-cluster-manager/internal/cloudprovider/eop/api/constant.go#L13
package-comments: should have a package comment (revive)
bcs-cluster-manager: bcs-services/bcs-cluster-manager/internal/cloudprovider/aws/api/as.go#L13
package-comments: should have a package comment (revive)
bcs-cluster-manager: bcs-services/bcs-cluster-manager/internal/cloudprovider/eop/tasks/create_cluster.go#L13
package-comments: should have a package comment (revive)
bcs-cluster-manager: bcs-services/bcs-cluster-manager/internal/cloudprovider/aws/tasks/clean_nodes_ingroup.go#L13
package-comments: should have a package comment (revive)