feat(query): add cbs disk without encrypted for tencentcloud

[hellertang]

Proposed Changes

• CBS Disk Without Encrypted for TencentCloud

I submit this contribution under the Apache-2.0 license.

[hellertang]

Please help review the pr~

[hellertang]

Already add new cloudProvider : tencentcloud in .github/scripts/queries-validator/metadata-schema.json. Please check and run again. Thanks.

[hellertang]

Why does the e2e-debian-tests fail? I can’t seem to see the detailed error message.
Please tell me how to fix.
Thanks.

[gabriel-cx]

Why does the e2e-debian-tests fail? I can’t seem to see the detailed error message. Please tell me how to fix. Thanks.

Hi @hellertang ,

In this case we just need to rerun the test. The issue is not related to your PR's changes, do not worry :D

[hellertang]

Why does the e2e-debian-tests fail? I can’t seem to see the detailed error message. Please tell me how to fix. Thanks.

Hi @hellertang ,

In this case we just need to rerun the test. The issue is not related to your PR's changes, do not worry :D

Thanks. We sincerely hope that KICS can support the inspection of Tencent Cloud's Terraform resources. I aspire for this PR to be integrated, as a testament to the feasibility of this path. In doing so, we can strive to contribute more inspection rules pertinent to the Tencent Cloud Provider.

[gabriel-cx]

Why does the e2e-debian-tests fail? I can’t seem to see the detailed error message. Please tell me how to fix. Thanks.

Hi @hellertang ,
In this case we just need to rerun the test. The issue is not related to your PR's changes, do not worry :D

Thanks. We sincerely hope that KICS can support the inspection of Tencent Cloud's Terraform resources. I aspire for this PR to be integrated, as a testament to the feasibility of this path. In doing so, we can strive to contribute more inspection rules pertinent to the Tencent Cloud Provider.

Nice to know! Thank you again for this amazing contribution!

Kindly be aware that, since this is a new cloud provider support (tencentcloud), after this PR is merged, in order to run this new queries you need to use the KICS flag --experimental-queries.
After merging this PR, we will update our documentation as well, so we can mention Tencent Cloud as a new support, under our Beta Features section.

[gabriel-cx]

@hellertang kindly add a line for tencentcloud here as well. So the unit tests can run against tencentcloud queries.

[hellertang]

@hellertang kindly add a line for tencentcloud here as well. So the unit tests can run against tencentcloud queries.

Got it, thanks. Already fix just now.

[gabriel-cx]

thanks @hellertang !

kindly be aware of this failing unit test:

[hellertang]

thanks @hellertang !

kindly be aware of this failing unit test:

Thanks @gabriel-cx . Already fix it just now.

I have discerned that if 'experimental' is set to true, 'Remediation' cannot be established in 'query.go', as this would result in the unit test not being able to execute successfully. So I remove it.

[gabriel-cx]

Thank you @hellertang !
This week we will release a new KICS version (v1.7.13) and Tencent Cloud support will be part of it!

[gabriel-cx]

Hi @hellertang ,

I'm happy to let you know that your tencentcloud queries are now part of the KICS 'official queries'!
This means, tencentcloud queries will run by default when KICS detects terraform files.
This functionality will start to be available in the next KICS version (v2.1.0).

Thank you again for all your help on this!