Skip to content

Conversation

@hwbyuehen
Copy link
Contributor

What this PR does / why we need it:
add SMS source server resource
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:

add SMS source server resource

PR Checklist

  • Tests added/passed.
==> Checking that code complies with gofmt requirements...
TF_ACC=1 go test ./huaweicloud/services/acceptance/sms/ -v -run TestAccResourceSourceServer_basic -timeout 360m -parallel 4
=== RUN   TestAccResourceSourceServer_basic
=== PAUSE TestAccResourceSourceServer_basic
=== CONT  TestAccResourceSourceServer_basic
--- PASS: TestAccResourceSourceServer_basic (323.04s)
PASS
ok      github.com/huaweicloud/terraform-provider-huaweicloud/huaweicloud/services/acceptance/sms       323.111s
  • Documentation updated.

  • Schema updated.

  • CheckDeleted.

    • a. During query operation (Read Context)
      aa. Resource not found
      r_check0625
    • b. During delete/disassociate/unbind operation (Delete Context)
      ba. Resource not found
      d_check0625

@hwbyuehen hwbyuehen force-pushed the add_sms_source_server_resource branch 7 times, most recently from ed24736 to 72ebfb5 Compare June 27, 2025 02:47
@github-ci-robot github-ci-robot added the needs-rebase This branch has conflicts that must be resolved. label Jul 1, 2025
@hwbyuehen hwbyuehen force-pushed the add_sms_source_server_resource branch 2 times, most recently from 9e1a7a8 to 5bb02f3 Compare July 1, 2025 08:59
@github-ci-robot github-ci-robot removed the needs-rebase This branch has conflicts that must be resolved. label Jul 1, 2025
@hwbyuehen hwbyuehen force-pushed the add_sms_source_server_resource branch 2 times, most recently from 24bfef8 to fb31c7a Compare July 2, 2025 03:39
@hwbyuehen hwbyuehen force-pushed the add_sms_source_server_resource branch from fb31c7a to 8d0a912 Compare July 7, 2025 01:43
@houpeng80
Copy link
Collaborator

/lgtm
/approve

@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 Nov 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: @houpeng80

/lgtm
/approve

@github-ci-robot github-ci-robot merged commit 0978f48 into huaweicloud:master Nov 25, 2025
11 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/XL

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants