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

docs: 安全组管理 业务下接口文档 #1249

Open
wants to merge 2 commits into
base: feat-sg-mgmt
Choose a base branch
from

Conversation

chenjr15
Copy link
Collaborator

No description provided.

@chenjr15 chenjr15 changed the title docs: 安全组管理范围 业务下接口文档 docs: 安全组支持多业务 业务下接口文档 Jan 16, 2025
@chenjr15 chenjr15 changed the title docs: 安全组支持多业务 业务下接口文档 docs: 安全组管理 业务下接口文档 Jan 16, 2025
@yangyy
Copy link
Collaborator

yangyy commented Jan 17, 2025

添加查询安全组规则数量接口,支持批量

@yangyy
Copy link
Collaborator

yangyy commented Jan 17, 2025

添加 安全组中的规则引用的安全组不是本业务 的查询接口

| name | string | 名称 |
| vendor | string | 供应商(枚举值:tcloud、aws、azure、gcp、huawei) |
| bk_biz_id | int64 | 业务ID |
| account_id | string | 账号ID |
Copy link
Collaborator

Choose a reason for hiding this comment

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

account_id可以返回么?cvm那边我看没有

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Account id 一定和当前安全组account id相同

| cloud_id | string | 云资源ID |
| name | string | 名称 |
| vendor | string | 供应商(枚举值:tcloud、aws、azure、gcp、huawei) |
| account_id | string | 账号ID |
Copy link
Collaborator

Choose a reason for hiding this comment

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

bk_biz_id可以筛选的吧?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

这里在url里面限制了业务id,筛选无意义

|---------------------|--------|--------------------------------------|
| cloud_id | string | 云资源ID |
| name | string | 名称 |
| vendor | string | 供应商(枚举值:tcloud、aws、azure、gcp、huawei) |
Copy link
Collaborator

Choose a reason for hiding this comment

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

关联表和资源表中同时有vendor字段,不能放在filter中

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

这里有sgid 做限制,vendor 筛选无意义,应该去掉

@@ -0,0 +1,211 @@
### 描述
Copy link
Collaborator

Choose a reason for hiding this comment

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

缺少业务下接口:批量查询安全组关联资源业务列表(BatchListBizSGRelBusiness)

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

已参考资源下补充

Copy link
Collaborator

@yangfan-njupt yangfan-njupt Jan 20, 2025

Choose a reason for hiding this comment

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

@yangyy 确认下新补充的文档还有没有问题~

| bk_biz_id | int64 | 业务ID |
| account_id | string | 账号ID |
| region | string | 地域 |
| main_zones | string | 主可用区 |
Copy link
Collaborator

Choose a reason for hiding this comment

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

main_zones?lb的接口文档怎么和实际的接口返回不一致的。。。

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

已修改

| backup_zones | string | 备可用区 |
| cloud_vpc_id | string | 云vpcID |
| vpc_id | string | vpcID |
| network_type | string | 网络类型 |
Copy link
Collaborator

Choose a reason for hiding this comment

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

network_type这个字段现有的负载均衡列表里也没有,需要确认下含义

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

已修改

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