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: 【告警】告警通知里,针对集群信息补充环境类型字段 --story=121108605 #4401

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

tang202388
Copy link
Collaborator

No description provided.

Copy link

请在 PR 中添加项目标签,例如:project/monitorproject/apmproject/logproject/public

Copy link

请在 PR 中添加类型标签,例如:fixfeatdocsstylerefactortestchoremerge, perf

@tang202388 tang202388 added feat A new feature. Correlates with MINOR in SemVer project/monitor project monitor labels Dec 13, 2024
@tang202388
Copy link
Collaborator Author

自测截图:
修改后,关于集群的关联信息正常加上 环境类型了
image


if environment_types and bk_set_ids:
environments = [
environment_mapping[environment_types[bk_set_id]]
Copy link
Collaborator

Choose a reason for hiding this comment

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

需要考虑cmdb迭代的问题。 新增了环境类型的时候, 此时我们写死123,则会有异常。这块如果没有对应翻译,就直接给到对应的类型id即可。

@tang202388 tang202388 force-pushed the feat/add_env_type/#1010158081121108605 branch from 1eca6bc to e847b2e Compare December 30, 2024 08:05
@tang202388
Copy link
Collaborator Author

自测截图补充:
当环境类型没有对应的翻译时,返回环境类型ID
image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feat A new feature. Correlates with MINOR in SemVer project/monitor project monitor
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants