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

Support greater than twenty db parameters #7364

Merged

Conversation

ParthaDutta
Copy link
Contributor

@ParthaDutta ParthaDutta commented Jun 27, 2016

Test to deal with creating and updating greater than 20 database parameters in AWS

Thanks to Christopher Tiwald for his work https://github.com/hashicorp/terraform/pull/3082

Christopher can be found here https://github.com/ctiwald

ctiwald and others added 7 commits August 26, 2015 14:26
Cannot create a passing test for database parameters "innodb_file_per_table" and "binlog_format"
It seems that these parameters can be created and tested successfully
BUT after the "parameter group" has been destroyed, it then makes a "DescribeDBParameterGroups" call
This fails with a 404 error...makes sense since the group does not exist

Have very little understanding of how the test framework works, so am struggling to debug

Currently commented out to have a passing test
@ParthaDutta
Copy link
Contributor Author

Issues with certain key value database parameters

Cannot create a passing test for database parameters innodb_file_per_table and binlog_format
Seems that these database parameter(s) are created successfully
But testing it is failing; what being returned is different to expected values
Remove testing these parameters does not improve the situation; fails later on in the test run
For now the test is ignoring these parameters

Have limited understanding of how the test framework works, so am struggling to debug
Really appreciate another pair of eyes looking into this

The database key/value pairs are listed below

"binlog_format" : value = "MIXED"
"innodb_file_per_table" : value = 1

@ParthaDutta ParthaDutta changed the title WIP: Support greater than twenty db parameters Support greater than twenty db parameters Jun 28, 2016
@tomelliff
Copy link
Contributor

@stack72 is there anything needed to be done on this to get this advanced at all? Would be good to get this in at some point as we've got a pretty awkward workaround to get more than 20 parameters changed.

@stack72
Copy link
Contributor

stack72 commented Jul 25, 2016

Hi @ParthaDutta

Thanks so much here for continuing the awesome work that @ctiwald started! Just ran the tests and all looks good :)

% make testacc TEST=./builtin/providers/aws TESTARGS='-run=TestAccAWSDBParameterGroup_'                        1 ↵
==> Checking that code complies with gofmt requirements...
go generate $(go list ./... | grep -v /terraform/vendor/)
TF_ACC=1 go test ./builtin/providers/aws -v -run=TestAccAWSDBParameterGroup_ -timeout 120m
=== RUN   TestAccAWSDBParameterGroup_importBasic
--- PASS: TestAccAWSDBParameterGroup_importBasic (31.87s)
=== RUN   TestAccAWSDBParameterGroup_limit
--- PASS: TestAccAWSDBParameterGroup_limit (55.99s)
=== RUN   TestAccAWSDBParameterGroup_basic
--- PASS: TestAccAWSDBParameterGroup_basic (53.71s)
=== RUN   TestAccAWSDBParameterGroup_Only
--- PASS: TestAccAWSDBParameterGroup_Only (26.97s)
PASS
ok      github.com/hashicorp/terraform/builtin/providers/aws    168.552s

@ghost
Copy link

ghost commented Apr 24, 2020

I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues.

If you have found a problem that seems similar to this, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.

@ghost ghost locked and limited conversation to collaborators Apr 24, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants