Skip to content

Conversation

@Lance52259
Copy link
Contributor

What this PR does / why we need it:

Supports a new data source that used to query associated desktop pools for specified users.

Which issue this PR fixes:
(optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged)
fixes #xxx

Special notes for your reviewer:

image

Release note:

1. add new data source to query associated desktop pools for users

PR Checklist

  • Tests added/passed.
./scripts/coverage.sh -o workspace -f TestAccDataUserDesktopPoolAssociations_basic
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 TestAccDataUserDesktopPoolAssociations_basic -timeout 360m -parallel 10
=== RUN   TestAccDataUserDesktopPoolAssociations_basic
--- PASS: TestAccDataUserDesktopPoolAssociations_basic (367.01s)
PASS
coverage: 9.0% of statements in ./huaweicloud/services/workspace
ok      github.com/huaweicloud/terraform-provider-huaweicloud/huaweicloud/services/acceptance/workspace 367.093s        coverage: 9.0% of statements in ./huaweicloud/services/workspace
  • Documentation updated.

  • Schema updated.

  • CheckDeleted.

    • a. During query operation (Read Context)
      aa. Resource not found
      >>>>>> Paste the screenshot here <<<<<<

    • b. During delete/disassociate/unbind operation (Delete Context)
      ba. Resource not found
      >>>>>> Paste the screenshot here <<<<<<

@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 1, 2025
@github-ci-robot
Copy link
Collaborator

[APPROVALNOTIFIER] This PR is APPROVED

This PR has been approved by: @Lance52259

@Lance52259 Lance52259 force-pushed the br_workspace_user_associated_desktop_pools branch from 0fd9ebd to 31284aa Compare December 1, 2025 09:59
@Lance52259
Copy link
Contributor Author

./scripts/coverage.sh -o workspace -f TestAccDataUserDesktopPoolAssociations_basic
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 TestAccDataUserDesktopPoolAssociations_basic -timeout 360m -parallel 10
=== RUN   TestAccDataUserDesktopPoolAssociations_basic
--- PASS: TestAccDataUserDesktopPoolAssociations_basic (367.52s)
PASS
coverage: 9.0% of statements in ./huaweicloud/services/workspace
ok      github.com/huaweicloud/terraform-provider-huaweicloud/huaweicloud/services/acceptance/workspace 367.609s        coverage: 9.0% of statements in ./huaweicloud/services/workspace

@Lance52259 Lance52259 force-pushed the br_workspace_user_associated_desktop_pools branch from 31284aa to 599c1a8 Compare December 2, 2025 06:54
@github-ci-robot
Copy link
Collaborator

This PR needs rebase.

@github-ci-robot github-ci-robot added the needs-rebase This branch has conflicts that must be resolved. label Dec 3, 2025
@Lance52259 Lance52259 force-pushed the br_workspace_user_associated_desktop_pools branch from 599c1a8 to dd75898 Compare December 3, 2025 08:16
@github-ci-robot github-ci-robot removed the needs-rebase This branch has conflicts that must be resolved. label Dec 3, 2025
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. bugfix feature size/L

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants