Skip to content

feat(hss): add new data source to query hss process statistics #7029

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

feat(hss): add new data source to query hss process statistics #7029

wants to merge 1 commit into from
+295 −0

Conversation

@shi2duxinxia
Copy link
Contributor

@shi2duxinxia shi2duxinxia commented Jun 17, 2025

What this PR does / why we need it:

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:

PR Checklist

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

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

@github-ci-robot github-ci-robot added the needs-rebase This branch has conflicts that must be resolved. label Jun 17, 2025
jnzr123
jnzr123 reviewed Jun 17, 2025
View reviewed changes
@jnzr123
Copy link
Contributor

jnzr123 commented Jun 17, 2025

LGTM

@shi2duxinxia shi2duxinxia force-pushed the huweimin_process_statistics branch from 2328666 to 7d08369 Compare June 17, 2025 06:32
@github-ci-robot github-ci-robot removed the needs-rebase This branch has conflicts that must be resolved. label Jun 17, 2025
jinyangyang222
jinyangyang222 reviewed Jun 17, 2025
View reviewed changes
@shi2duxinxia shi2duxinxia force-pushed the huweimin_process_statistics branch from 7d08369 to 99f6ea7 Compare June 17, 2025 11:36
@github-ci-robot github-ci-robot added the needs-rebase This branch has conflicts that must be resolved. label Jun 18, 2025
@github-ci-robot
Copy link
Collaborator

github-ci-robot commented Jun 18, 2025

This PR needs rebase.

@shi2duxinxia shi2duxinxia force-pushed the huweimin_process_statistics branch from 99f6ea7 to 5fb57ff Compare June 19, 2025 09:35
@shi2duxinxia
Copy link
Contributor Author

shi2duxinxia commented Jun 19, 2025

make testacc TEST='./huaweicloud/services/acceptance/hss' TESTARGS='-run TestAccDataSourceAssetProcessStatistics_basic'
==> Checking that code complies with gofmt requirements...
TF_ACC=1 go test ./huaweicloud/services/acceptance/hss -v -run TestAccDataSourceAssetProcessStatistics_basic -timeout 360m -parallel 4
=== RUN TestAccDataSourceAssetProcessStatistics_basic
=== PAUSE TestAccDataSourceAssetProcessStatistics_basic
=== CONT TestAccDataSourceAssetProcessStatistics_basic
--- PASS: TestAccDataSourceAssetProcessStatistics_basic (63.17s)
PASS
ok github.com/huaweicloud/terraform-provider-huaweicloud/huaweicloud/services/acceptance/hss 63.280s

@shi2duxinxia shi2duxinxia force-pushed the huweimin_process_statistics branch from 5fb57ff to 17af89e Compare June 19, 2025 09:40
@github-ci-robot github-ci-robot added needs-rebase This branch has conflicts that must be resolved. and removed needs-rebase This branch has conflicts that must be resolved. labels Jun 19, 2025
@github-ci-robot
Copy link
Collaborator

github-ci-robot commented Jun 20, 2025

This PR needs rebase.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

2 more reviewers

@jnzr123 jnzr123 jnzr123 left review comments

@jinyangyang222 jinyangyang222 jinyangyang222 left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

feature needs-rebase This branch has conflicts that must be resolved. size/M

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@shi2duxinxia @jnzr123 @github-ci-robot @jinyangyang222