-
Notifications
You must be signed in to change notification settings - Fork 4
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
added performance_insights_enabled argument for rds_instance - #22 #77
Conversation
Hi @NishantTomar the workflow we have is running the command Can you lint the terraform? |
@robg-test I ran the |
Hi @NishantTomar we have a small issue in regards to the workflow around forked repos. Will look at fixing it ASAP. See #78 |
Hi @NishantTomar we are a step closer to resolving this issue with the pipeline (@chrisbloe's merge) |
Hi @robg-test thanks for the update. |
Hi again @NishantTomar , we've run into a bit of an issue around execution of this repo from forked repos. The pipeline is meant to automatically commit documentation changes to the repo however this isn't working as your on a fork. Were going to have a look at next steps for supporting forks next week. |
Hi @robg-test is it fixed? |
So in order for forked pull requests to work we need to rethink some of the existing behavior around auto-commiting documentation changes. I will add this in scope for release 3 of these modules to come up with a fix for this. I will replicate this PR in a PR branch directly on this repo. Apologies, you were the first external editor of this repo and you've helped uncover a rather nasty bug in our workflow. |
Hi @NishantTomar, thank you for your PR! And sorry it took us so long to address the issue described above. I've merged a fix in #95 so documentation is updated automatically for all PRs. |
Hi @cmbuckley No problem. It was my pleasure to contribute to the repository. |
added the performance_insights_enabled argument to rds_instance and set the default value as false.
ISSUE- #22