-
Notifications
You must be signed in to change notification settings - Fork 143
Feat/cm support #2212
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/cm support #2212
Conversation
22d83e3
to
5d5aa5c
Compare
paramMap["Module"] = helper.String(v.(string)) | ||
} | ||
|
||
if v, _ := d.GetOk("start_time"); v != nil { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
这里不需要使用GetOkExists吗
return err | ||
} | ||
|
||
time.Sleep(3 * time.Second) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
这里为啥需要sleep呢
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
因为修改配置后实例状态没有立即更新
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
这里等待3秒时间够吗?如果3秒状态未返回会有问题吗
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
我们是不是应该封装一个公共的异步请求的方法呢?
|
||
time.Sleep(3 * time.Second) | ||
service := MonitorService{client: meta.(*TencentCloudClient).apiV3Conn} | ||
err = resource.Retry(1*readRetryTimeout, func() *resource.RetryError { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
修改完再查询一次的目的是什么呢
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
轮询等待实例状态ready
fc10bfb
to
5b31e7a
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
No description provided.