Skip to content

Conversation

@946596029
Copy link
Contributor

What this PR does / why we need it:
feat(workspace): add new resource for batch modify volume QOS

Which issue this PR fixes:
nothing

Special notes for your reviewer:
nothing

Release note:

1. add provider implement
2. add test case
3. add document

PR Checklist

  • Tests added/passed.
  • Documentation updated.
  • Schema updated.
  • CheckDeleted.

@946596029 946596029 force-pushed the br_workspace_batch_modify_volume_qos branch 5 times, most recently from 870546e to 71404f1 Compare November 28, 2025 03:48
@946596029 946596029 force-pushed the br_workspace_batch_modify_volume_qos branch from 71404f1 to 9c41dd3 Compare December 2, 2025 09:36
@946596029 946596029 force-pushed the br_workspace_batch_modify_volume_qos branch 2 times, most recently from 0c89b91 to d86cba4 Compare December 5, 2025 06:20
@946596029 946596029 changed the title feat(workspace): add new resource for batch modify volume QOS feat(workspace): add new resource for batch modify volume QoS Dec 5, 2025
@946596029 946596029 force-pushed the br_workspace_batch_modify_volume_qos branch 2 times, most recently from b168f1c to 037e17e Compare December 5, 2025 09:00
@946596029
Copy link
Contributor Author

Prepare to calculate the coverage the following command:
TF_ACC=1 go test "./huaweicloud/services/acceptance/workspace" -v -coverprofile="./huaweicloud/services/acceptance/workspace/workspace_coverage.cov" -coverpkg="./huaweicloud/services/workspace" -run TestAccDesktop_basic -timeout 360m -parallel 10
=== RUN TestAccDesktop_basic
--- PASS: TestAccDesktop_basic (1097.24s)
PASS
coverage: 7.9% of statements in ./huaweicloud/services/workspace
ok github.com/huaweicloud/terraform-provider-huaweicloud/huaweicloud/services/acceptance/workspace 1097.394s coverage: 7.9% of statements in ./huaweicloud/services/workspace

@946596029
Copy link
Contributor Author

Prepare to calculate the coverage the following command:
TF_ACC=1 go test "./huaweicloud/services/acceptance/workspace" -v -coverprofile="./huaweicloud/services/acceptance/workspace/workspace_coverage.cov" -coverpkg="./huaweicloud/services/workspace" -run TestAccDesktop_UpdateWithEpsId -timeout 360m -parallel 10
=== RUN TestAccDesktop_UpdateWithEpsId
--- PASS: TestAccDesktop_UpdateWithEpsId (665.23s)
PASS
coverage: 6.6% of statements in ./huaweicloud/services/workspace
ok github.com/huaweicloud/terraform-provider-huaweicloud/huaweicloud/services/acceptance/workspace 665.371s coverage: 6.6% of statements in ./huaweicloud/services/workspace

@946596029
Copy link
Contributor Author

Prepare to calculate the coverage the following command:
TF_ACC=1 go test "./huaweicloud/services/acceptance/workspace" -v -coverprofile="./huaweicloud/services/acceptance/workspace/workspace_coverage.cov" -coverpkg="./huaweicloud/services/workspace" -run TestAccDesktop_powerAction -timeout 360m -parallel 10
=== RUN TestAccDesktop_powerAction
--- PASS: TestAccDesktop_powerAction (1371.03s)
PASS
coverage: 6.9% of statements in ./huaweicloud/services/workspace
ok github.com/huaweicloud/terraform-provider-huaweicloud/huaweicloud/services/acceptance/workspace 1371.157s coverage: 6.9% of statements in ./huaweicloud/services/workspace

@946596029
Copy link
Contributor Author

Prepare to calculate the coverage the following command:
TF_ACC=1 go test "./huaweicloud/services/acceptance/workspace" -v -coverprofile="./huaweicloud/services/acceptance/workspace/workspace_coverage.cov" -coverpkg="./huaweicloud/services/workspace" -run TestAccDesktop_rootVolumeModify -timeout 360m -parallel 10
=== RUN TestAccDesktop_rootVolumeModify
--- PASS: TestAccDesktop_rootVolumeModify (913.02s)
PASS
coverage: 7.8% of statements in ./huaweicloud/services/workspace
ok github.com/huaweicloud/terraform-provider-huaweicloud/huaweicloud/services/acceptance/workspace 913.155s coverage: 7.8% of statements in ./huaweicloud/services/workspace

@946596029 946596029 force-pushed the br_workspace_batch_modify_volume_qos branch from 037e17e to a56bc1c Compare December 8, 2025 02:00
@github-ci-robot github-ci-robot added the needs-rebase This branch has conflicts that must be resolved. label Dec 8, 2025
@github-ci-robot
Copy link
Collaborator

This PR needs rebase.

@946596029
Copy link
Contributor Author

Prepare to calculate the coverage the following command:
TF_ACC=1 go test "./huaweicloud/services/acceptance/workspace" -v -coverprofile="./huaweicloud/services/acceptance/workspace/workspace_coverage.cov" -coverpkg="./huaweicloud/services/workspace" -run TestAccDesktop_dataVolumeModify -timeout 360m -parallel 10
=== RUN TestAccDesktop_dataVolumeModify
--- PASS: TestAccDesktop_dataVolumeModify (2085.28s)
PASS
coverage: 8.2% of statements in ./huaweicloud/services/workspace
ok github.com/huaweicloud/terraform-provider-huaweicloud/huaweicloud/services/acceptance/workspace 2085.432s coverage: 8.2% of statements in ./huaweicloud/services/workspace

@946596029
Copy link
Contributor Author

Prepare to calculate the coverage the following command:
TF_ACC=1 go test "./huaweicloud/services/acceptance/workspace" -v -coverprofile="./huaweicloud/services/acceptance/workspace/workspace_coverage.cov" -coverpkg="./huaweicloud/services/workspace" -run TestAccDesktop_volumeChangeErrors -timeout 360m -parallel 10
=== RUN TestAccDesktop_volumeChangeErrors
--- PASS: TestAccDesktop_volumeChangeErrors (795.36s)
PASS
coverage: 7.7% of statements in ./huaweicloud/services/workspace
ok github.com/huaweicloud/terraform-provider-huaweicloud/huaweicloud/services/acceptance/workspace 795.528s coverage: 7.7% of statements in ./huaweicloud/services/workspace

@946596029 946596029 force-pushed the br_workspace_batch_modify_volume_qos branch 2 times, most recently from a9be47d to 615276c Compare December 8, 2025 09:20
@github-ci-robot github-ci-robot removed the needs-rebase This branch has conflicts that must be resolved. label Dec 8, 2025
@946596029 946596029 force-pushed the br_workspace_batch_modify_volume_qos branch 3 times, most recently from 7ae22c4 to a78a3e9 Compare December 8, 2025 12:09
@946596029
Copy link
Contributor Author

Prepare to calculate the coverage the following command:
TF_ACC=1 go test "./huaweicloud/services/acceptance/workspace" -v -coverprofile="./huaweicloud/services/acceptance/workspace/workspace_coverage.cov" -coverpkg="./huaweicloud/services/workspace" -run TestAccDesktop_volumeChangeErrors -timeout 360m -parallel 10
=== RUN TestAccDesktop_volumeChangeErrors
--- PASS: TestAccDesktop_volumeChangeErrors (819.53s)
PASS
coverage: 7.6% of statements in ./huaweicloud/services/workspace
ok github.com/huaweicloud/terraform-provider-huaweicloud/huaweicloud/services/acceptance/workspace 819.779s coverage: 7.6% of statements in ./huaweicloud/services/workspace

@946596029 946596029 force-pushed the br_workspace_batch_modify_volume_qos branch 3 times, most recently from 543f3bc to 5f4fc2b Compare December 9, 2025 01:21
@946596029 946596029 force-pushed the br_workspace_batch_modify_volume_qos branch from 5f4fc2b to 594c40d Compare December 9, 2025 01:28
@946596029 946596029 force-pushed the br_workspace_batch_modify_volume_qos branch from 594c40d to 161d109 Compare December 9, 2025 01:48
@Lance52259
Copy link
Contributor

/approve

@github-ci-robot github-ci-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Dec 9, 2025
Copy link
Collaborator

@github-ci-robot github-ci-robot left a comment

Choose a reason for hiding this comment

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

This PR has been approved by: @Lance52259

/approve

- change implement for modify volume QoS
- add test case for volume normal change
- add test case for volume abnormal change
@946596029 946596029 force-pushed the br_workspace_batch_modify_volume_qos branch from 161d109 to 0dbbe23 Compare December 9, 2025 01:51
@Lance52259
Copy link
Contributor

/lgtm

@github-ci-robot github-ci-robot added the LGTM /lgtm label Dec 9, 2025
@github-ci-robot github-ci-robot merged commit 364c955 into huaweicloud:master Dec 9, 2025
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. feature LGTM /lgtm size/L size/XL

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants