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: Add sensitive flag for MySQL module outputs #303

Merged

Conversation

artemvovk
Copy link
Contributor

running terraform plan on MySQL module was generating errors
because the values in these outputs are considered sensitive.

Before:

After:

Screen Shot 2022-05-20 at 09 42 05

running terraform plan on MySQL module was generating errors
because the values in these outputs are considered sensitive.
@google-cla
Copy link

google-cla bot commented May 20, 2022

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

For more information, open the CLA check for this pull request.

@Tensho
Copy link

Tensho commented May 20, 2022

Seems like it's already in 10.0.2 but only for Postgres. Would be nice to have for MySQL too.

@bharathkkb bharathkkb changed the title Add sensitive flag for MySQL module outputs fix: Add sensitive flag for MySQL module outputs May 25, 2022
@comment-bot-dev
Copy link

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

@Tensho
Copy link

Tensho commented Jun 1, 2022

Any movement with this, ladies and gentlemen?

@bharathkkb bharathkkb merged commit 6a15c26 into terraform-google-modules:master Jun 1, 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.

None yet

4 participants