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: 安全组-资源 关联关系迁移到common表 --story=121678936 #1246

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

Conversation

KooKouse
Copy link
Contributor

No description provided.

@KooKouse KooKouse force-pushed the github/feat-121678936 branch from 55f5a29 to 75c957a Compare January 16, 2025 09:01
@KooKouse KooKouse force-pushed the github/feat-121678936 branch from 75c957a to f79a5d9 Compare January 17, 2025 08:20
err = g.createSGCommonRels(kt, enumor.TCloud, cvmID, sgCloudIDs, sgCloudIDToIDMap)
if err != nil {
// 不抛出err, 尽最大努力交付
logs.Errorf("create sg common rels failed, err: %v, rid: %s", err, kt.Rid)
Copy link
Collaborator

@fxdgg fxdgg Jan 21, 2025

Choose a reason for hiding this comment

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

1、记录下cvmID, sgCloudIDs, one.InstanceId等参数吧
2、这里创建rel失败后,会有什么影响吗?是否需要配置日志告警?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

rel创建失败,本地暂时看不到关联关系,需要由同步进行数据兜底

cmd/hc-service/service/security-group/logics.go Outdated Show resolved Hide resolved
cmd/hc-service/service/security-group/logics.go Outdated Show resolved Hide resolved
cmd/hc-service/service/cvm/cvm.go Outdated Show resolved Hide resolved
cmd/hc-service/service/cvm/aws.go Outdated Show resolved Hide resolved
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.

4 participants