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

feat(cci): add datasource cci v2 namespaces #6482

Merged
merged 1 commit into from
Mar 28, 2025

Conversation

luoping-12345
Copy link
Contributor

@luoping-12345 luoping-12345 commented Mar 10, 2025

What this PR does / why we need it:
add datasource cci v2 namespaces

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 datasource cci v2 namespaces

PR Checklist

  • Tests added/passed.
make testacc TEST="./huaweicloud/services/acceptance/cci" TESTARGS="-run TestAccDataSourceV2Namespaces_basic"
==> Checking that code complies with gofmt requirements...
TF_ACC=1 go test ./huaweicloud/services/acceptance/cci -v -run TestAccDataSourceV2Namespaces_basic -timeout 360m -parallel 4
=== RUN   TestAccDataSourceV2Namespaces_basic
=== PAUSE TestAccDataSourceV2Namespaces_basic
=== CONT  TestAccDataSourceV2Namespaces_basic
--- PASS: TestAccDataSourceV2Namespaces_basic(81.31s)
PASS
ok      github.com/huaweicloud/terraform-provider-huaweicloud/huaweicloud/services/acceptance/cci       82.430s

  • 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 <<<<<<

@luoping-12345 luoping-12345 force-pushed the cciv2_namespaces branch 2 times, most recently from 88e481e to fb29f9a Compare March 10, 2025 07:11
@github-actions github-actions bot added size/M and removed size/L labels Mar 27, 2025
@luoping-12345 luoping-12345 force-pushed the cciv2_namespaces branch 2 times, most recently from 523efa7 to 43cf0a2 Compare March 28, 2025 02:12
@github-actions github-actions bot added size/L and removed size/M labels Mar 28, 2025
@github-actions github-actions bot added size/M and removed size/L labels Mar 28, 2025
@luoping-12345 luoping-12345 force-pushed the cciv2_namespaces branch 2 times, most recently from df13040 to 1d2004f Compare March 28, 2025 02:46
@houpeng80
Copy link
Collaborator

/lgtm

@github-ci-robot github-ci-robot added the LGTM /lgtm label Mar 28, 2025
@Jason-Zhang9309
Copy link
Collaborator

/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 Mar 28, 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: @Jason-Zhang9309

/approve

@github-ci-robot github-ci-robot merged commit 5e9382d into huaweicloud:master Mar 28, 2025
23 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/M
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants