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

fix for_each issue on the sql_audit_config #340

Merged

Conversation

vponnam
Copy link
Contributor

@vponnam vponnam commented Aug 26, 2022

This change converts the sql_server_audit_config from map(string) to list(map(string)) where for_each was being iterated on each of the map items in sql_server_audit_config map. Also updated the example to test sql_server_audit_config variable.

@vponnam vponnam force-pushed the fix/sql_server_audit branch 2 times, most recently from 6646d36 to 02e7699 Compare August 26, 2022 22:05
@vponnam vponnam force-pushed the fix/sql_server_audit branch from 02e7699 to 8b9c47a Compare August 29, 2022 16:49
@vponnam vponnam force-pushed the fix/sql_server_audit branch 2 times, most recently from 3af3b02 to 7b54a3c Compare September 22, 2022 15:17
@vponnam vponnam force-pushed the fix/sql_server_audit branch from 7b54a3c to 889444a Compare September 22, 2022 15:21
@comment-bot-dev
Copy link

@vponnam
Thanks for the PR! 🚀
✅ Lint checks have passed.

Copy link
Member

@bharathkkb bharathkkb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the PR @vponnam

@bharathkkb bharathkkb merged commit 95e48a1 into terraform-google-modules:master Sep 22, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants