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

fix(cbs): [119747862] tencentcloud_cbs_storage add auto_mount_configuration params #2954

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

Conversation

SevenEarth
Copy link
Collaborator

No description provided.

@@ -246,6 +302,11 @@ func resourceTencentCloudCbsStorageCreate(d *schema.ResourceData, meta interface
}
}

if autoMount {
// Skip the initial UNATTACHED state of the CBS
time.Sleep(10 * time.Second)
Copy link
Collaborator

Choose a reason for hiding this comment

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

sleep这么久?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

自动挂载磁盘操作大概是3、5分钟
sleep10秒是为了100%跳过磁盘刚创建好 默认待挂载的状态
这里是有判断的sleep,整体不会影响挂载的过程

@@ -124,6 +125,34 @@ func ResourceTencentCloudCbsStorage() *schema.Resource {
Computed: true,
Description: "The quota of backup points of cloud disk.",
},
"auto_mount_configuration": {
Type: schema.TypeList,
Copy link
Collaborator

Choose a reason for hiding this comment

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

只支持创建,不支持读取么

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

该参数mount_point和file_system_type是无返回的,单纯返回insId意义不大,并且该功能是一次性设置所以我没设置read

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.

2 participants