Skip to content

Conversation

wuzhuanhong
Copy link
Contributor

What this PR does / why we need it:
Add a new data source (huaweicloud_dms_rocketmq_availability_zones) to query availability zone list of the DMS rocketMQ under specified region.

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:

Release note:

1. add a new data source.
2. add corresponding documation and acceptance test.

PR Checklist

  • Tests added/passed.
./scripts/coverage.sh -o rocketmq -f TestAccDat
aRocketMQAvailabilityZones_basic
Prepare to calculate the coverage the following command:
TF_ACC=1 go test "./huaweicloud/services/acceptance/rocketmq" -v -coverprofile="./huaweicloud/services/acceptance/rocketmq/rocketmq_coverage.cov" -coverpkg="./huaweicloud/services/rocketmq" -run TestAccDataRocketMQAvailabilityZones_basic -timeout 360m -parallel 10
=== RUN   TestAccDataRocketMQAvailabilityZones_basic
=== PAUSE TestAccDataRocketMQAvailabilityZones_basic
=== CONT  TestAccDataRocketMQAvailabilityZones_basic
--- PASS: TestAccDataRocketMQAvailabilityZones_basic (19.96s)
PASS
coverage: 5.4% of statements in ./huaweicloud/services/rocketmq
ok      github.com/huaweicloud/terraform-provider-huaweicloud/huaweicloud/services/acceptance/rocketmq  20.115s coverage: 5.4% of statements in ./huaweicloud/services/rocketmq
image
  • 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 <<<<<<

@wuzhuanhong wuzhuanhong changed the title feat(rocketmar): add new data source to get azs feat(rocketmq): add new data source to get azs Aug 21, 2025
@Lance52259
Copy link
Contributor

/approve
/lgtm

@github-ci-robot github-ci-robot added LGTM /lgtm approved Indicates a PR has been approved by an approver from all required OWNERS files. labels Aug 25, 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
/lgtm

@github-ci-robot github-ci-robot merged commit ea85d11 into huaweicloud:master Aug 25, 2025
15 checks passed
@wuzhuanhong wuzhuanhong deleted the rocketmk_azs branch August 26, 2025 01:20
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/M
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants