magic-modules icon indicating copy to clipboard operation
magic-modules copied to clipboard

feat: Add Audit Manager product definition

Open krgauravKum opened this issue 4 weeks ago • 8 comments

Adding support for audit manager resource in Terraform provider

`google_audit_manager_framework_audit`
`google_audit_manager_framework_audit_scope_report`

krgauravKum avatar Dec 02 '25 10:12 krgauravKum

Hello! I am a robot. Tests will require approval from a repository maintainer to run.

Googlers: For automatic test runs see go/terraform-auto-test-runs.

@ScottSuarez, a repository maintainer, has been assigned to review your changes. If you have not received review feedback within 2 business days, please leave a comment on this PR asking them to take a look.

You can help make sure that review is quick by doing a self-review and by running impacted tests locally.

github-actions[bot] avatar Dec 02 '25 10:12 github-actions[bot]

also please create a release note in the primary PR comment following https://googlecloudplatform.github.io/magic-modules/code-review/release-notes/

and please follow go/terraform-contribution-guide#before-you-begin to get automatic build approvals

also please create a release note in the primary PR comment following https://googlecloudplatform.github.io/magic-modules/code-review/release-notes/

and please follow go/terraform-contribution-guide#before-you-begin to get automatic build approvals

added

krgauravKum avatar Dec 03 '25 16:12 krgauravKum

I've updated the changelog notes, when you make your new PR, you can cut/paste the datasource one.

I'm going to be OOO for over a week, I'll reassign the reviewer here to take over after you've made the appropriate changes.

c2thorn avatar Dec 03 '25 21:12 c2thorn

@modular-magician reassign-reviewer

c2thorn avatar Dec 03 '25 21:12 c2thorn

Generally, we prefer to split out pull requests to be one resource/data source at a time.

Additionally, have you considered the updatability of all fields? I see none are marked immutable, which means this resource should only have output-only or updatable fields. You'll need to follow https://googlecloudplatform.github.io/magic-modules/test/test/#add-an-update-test if that is the case.

i have updated file and mark as immutable and push in same pull request

krgauravKum avatar Dec 04 '25 10:12 krgauravKum

Generally, we prefer to split out pull requests to be one resource/data source at a time.

Additionally, have you considered the updatability of all fields? I see none are marked immutable, which means this resource should only have output-only or updatable fields. You'll need to follow https://googlecloudplatform.github.io/magic-modules/test/test/#add-an-update-test if that is the case.

Hey i have split this PR in two 1 for datasource - https://github.com/GoogleCloudPlatform/magic-modules/pull/15880/files 2 for resource - https://github.com/GoogleCloudPlatform/magic-modules/pull/15879

please review

krgauravKum avatar Dec 04 '25 11:12 krgauravKum

@ScottSuarez This PR has been waiting for review for 3 weekdays. Please take a look! Use the label disable-review-reminders to disable these notifications.

github-actions[bot] avatar Dec 09 '25 09:12 github-actions[bot]

@GoogleCloudPlatform/terraform-team @ScottSuarez This PR has been waiting for review for 1 week. Please take a look! Use the label disable-review-reminders to disable these notifications.

github-actions[bot] avatar Dec 11 '25 09:12 github-actions[bot]