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

tests/provider: Update hardcoded AZs (DB instance) #14778

Merged
merged 2 commits into from
Sep 30, 2020

Conversation

YakDriver
Copy link
Member

@YakDriver YakDriver commented Aug 21, 2020

Community Note

  • Please vote on this pull request by adding a 👍 reaction to the original pull request comment to help the community and maintainers prioritize this request
  • Please do not leave "+1" or other comments that do not add relevant new information or questions, they generate extra noise for pull request followers and do not help prioritize the request

Relates #12995
Depends on #15228

Release note for CHANGELOG:

NONE

Output from acceptance testing (GovCloud partition):

--- FAIL: TestAccAWSDbInstanceDataSource_ec2Classic (1.38s)
--- FAIL: TestAccAWSDBInstance_DbSubnetGroupName_RamShared (1.90s)
--- FAIL: TestAccAWSDBInstance_ReplicateSourceDb_DbSubnetGroupName (0.58s)
--- FAIL: TestAccAWSDBInstance_ReplicateSourceDb_DbSubnetGroupName_RamShared (0.41s)
--- FAIL: TestAccAWSDBInstance_S3Import (0.42s)
--- FAIL: TestAccAWSDBInstance_ReplicateSourceDb_ParameterGroupName (1480.70s)
--- FAIL: TestAccAWSDBInstance_SnapshotIdentifier_DbSubnetGroupName_RamShared (0.33s)
--- FAIL: TestAccAWSDBInstance_ReplicateSourceDb_CACertificateIdentifier (751.38s)
--- FAIL: TestAccAWSDBInstance_separate_iops_update (0.49s) (fixed, waiting on testing)
--- PASS: TestAccAWSDBInstance_MinorVersion (437.91s) **
--- FAIL: TestAccAWSDBInstance_ec2Classic (0.41s)
--- FAIL: TestAccAWSDBInstance_PerformanceInsightsEnabled_EnabledToDisabled (3.64s) (fixed, waiting on testing)
--- FAIL: TestAccAWSDBInstance_PerformanceInsightsRetentionPeriod (4.85s) (fixed, waiting on testing)
--- FAIL: TestAccAWSDBInstance_PerformanceInsightsKmsKeyId (5.41s) (fixed, waiting on testing)
--- FAIL: TestAccAWSDBInstance_PerformanceInsightsEnabled_DisabledToEnabled (405.64s) (fixed, waiting on testing)
--- FAIL: TestAccAWSDBInstance_CACertificateIdentifier (428.80s)
--- FAIL: TestAccAWSOpsworksRdsDbInstance_basic (418.61s)
--- FAIL: TestAccAWSDBInstance_ReplicateSourceDb_PerformanceInsightsEnabled (743.03s) (fixed, waiting on testing)
--- PASS: TestAccAWSDBInstance_MySQL_SnapshotRestoreWithEngineVersion (1338.80s) **
--- FAIL: TestAccAWSDBInstance_SnapshotIdentifier_PerformanceInsightsEnabled (1267.70s) (fixed, waiting on testing)
--- FAIL: TestAccAWSDBInstance_MSSQL_Domain (2965.58s) (fixed, waiting on testing) (fixed, waiting on testing)
--- PASS: TestAccAWSDBInstance_MSSQL_DomainSnapshotRestore (3067.96s) **
--- SKIP: TestAccAWSDBInstance_ReplicateSourceDb_DeletionProtection (0.00s)
--- SKIP: TestAccAWSDBInstance_SnapshotIdentifier_Tags_Unset (0.00s)
--- PASS: TestAccAWSDBInstance_kmsKey (456.21s)
--- PASS: TestAccAWSDBInstance_basic (468.92s)
--- PASS: TestAccAWSDBInstance_iamAuth (489.18s)
--- PASS: TestAccAWSDBInstance_namePrefix (489.44s)
--- PASS: TestAccAWSDBInstance_IsAlreadyBeingDeleted (487.70s)
--- PASS: TestAccAWSDBInstance_DeletionProtection (509.36s)
--- PASS: TestAccAWSDbInstanceDataSource_basic (509.64s)
--- PASS: TestAccAWSDBInstance_AllowMajorVersionUpgrade (511.05s)
--- PASS: TestAccAWSDBInstance_generatedName (536.79s)
--- PASS: TestAccAWSDBInstance_DbSubnetGroupName (558.42s)
--- PASS: TestAccAWSDBInstance_DbSubnetGroupName_VpcSecurityGroupIds (559.36s)
--- PASS: TestAccAWSDBInstance_optionGroup (574.92s)
--- PASS: TestAccAWSDBInstance_FinalSnapshotIdentifier (779.54s)
--- PASS: TestAccAWSDBInstance_FinalSnapshotIdentifier_SkipFinalSnapshot (778.32s)
--- PASS: TestAccAWSDBInstance_subnetGroup (904.64s)
--- PASS: TestAccAWSDBInstance_Password (477.09s)
--- PASS: TestAccAWSDBInstance_MaxAllocatedStorage (572.28s)
--- PASS: TestAccAWSDBInstance_ReplicateSourceDb_AvailabilityZone (1385.43s)
--- PASS: TestAccAWSDBInstance_ReplicateSourceDb (1405.95s)
--- PASS: TestAccAWSDBInstance_ReplicateSourceDb_AllowMajorVersionUpgrade (1444.80s)
--- PASS: TestAccAWSDBInstance_ReplicateSourceDb_MaintenanceWindow (1384.71s)
--- PASS: TestAccAWSDBInstance_ReplicateSourceDb_BackupWindow (1485.64s)
--- PASS: TestAccAWSDBInstance_ReplicateSourceDb_IamDatabaseAuthenticationEnabled (1525.73s)
--- PASS: TestAccAWSDBInstance_ReplicateSourceDb_BackupRetentionPeriod (1676.99s)
--- PASS: TestAccAWSDBInstance_ReplicateSourceDb_AutoMinorVersionUpgrade (1717.96s)
--- PASS: TestAccAWSDBInstance_ReplicateSourceDb_MaxAllocatedStorage (1484.84s)
--- PASS: TestAccAWSDBInstance_ReplicateSourceDb_AllocatedStorage (1778.76s)
--- PASS: TestAccAWSDBInstance_ReplicateSourceDb_Monitoring (1586.28s)
--- PASS: TestAccAWSDBInstance_ReplicateSourceDb_Port (1382.50s)
--- PASS: TestAccAWSDBInstance_ReplicateSourceDb_DbSubnetGroupName_VpcSecurityGroupIds (2297.76s)
--- PASS: TestAccAWSDBInstance_ReplicateSourceDb_MultiAZ (2029.48s)
--- PASS: TestAccAWSDBInstance_SnapshotIdentifier (1118.06s)
--- PASS: TestAccAWSDBInstance_SnapshotIdentifier_AvailabilityZone (1007.65s)
--- PASS: TestAccAWSDBInstance_ReplicateSourceDb_VpcSecurityGroupIds (1442.58s)
--- PASS: TestAccAWSDBInstance_SnapshotIdentifier_AutoMinorVersionUpgrade (1188.73s)
--- PASS: TestAccAWSDBInstance_SnapshotIdentifier_BackupWindow (1089.18s)
--- PASS: TestAccAWSDBInstance_SnapshotIdentifier_AllocatedStorage (1562.82s)
--- PASS: TestAccAWSDBInstance_SnapshotIdentifier_DbSubnetGroupName (1198.94s)
--- PASS: TestAccAWSDBInstance_SnapshotIdentifier_DbSubnetGroupName_VpcSecurityGroupIds (1199.07s)
--- PASS: TestAccAWSDBInstance_SnapshotIdentifier_Io1Storage (1704.26s)
--- PASS: TestAccAWSDBInstance_SnapshotIdentifier_BackupRetentionPeriod (1503.50s)
--- PASS: TestAccAWSDBInstance_SnapshotIdentifier_DeletionProtection (1227.15s)
--- PASS: TestAccAWSDBInstance_SnapshotIdentifier_BackupRetentionPeriod_Unset (1704.82s)
--- PASS: TestAccAWSDBInstance_SnapshotIdentifier_MaintenanceWindow (1079.45s)
--- PASS: TestAccAWSDBInstance_SnapshotIdentifier_AllowMajorVersionUpgrade (1968.55s)
--- PASS: TestAccAWSDBInstance_SnapshotIdentifier_IamDatabaseAuthenticationEnabled (1240.71s)
--- PASS: TestAccAWSDBInstance_SnapshotIdentifier_MaxAllocatedStorage (1250.00s)
--- PASS: TestAccAWSDBInstance_portUpdate (528.95s)
--- PASS: TestAccAWSDBInstance_SnapshotIdentifier_Monitoring (1351.77s)
--- PASS: TestAccAWSDBInstance_MonitoringRoleArn_RemovedToEnabled (588.91s)
--- PASS: TestAccAWSDBInstance_MonitoringRoleArn_EnabledToDisabled (772.92s)
--- PASS: TestAccAWSDBInstance_SnapshotIdentifier_VpcSecurityGroupIds (1039.17s)
--- PASS: TestAccAWSDBInstance_SnapshotIdentifier_Port (1180.58s)
--- PASS: TestAccAWSDBInstance_MonitoringRoleArn_EnabledToRemoved (771.22s)
--- PASS: TestAccAWSDBInstance_SnapshotIdentifier_ParameterGroupName (1290.87s)
--- PASS: TestAccAWSDBInstance_SnapshotIdentifier_VpcSecurityGroupIds_Tags (1160.76s)
--- PASS: TestAccAWSDBInstance_SnapshotIdentifier_Tags (1280.58s)
--- PASS: TestAccAWSDBInstance_MonitoringInterval (1126.42s)
--- PASS: TestAccAWSDBInstance_cloudwatchLogsExportConfiguration (466.22s)
--- PASS: TestAccAWSDBInstance_EnabledCloudwatchLogsExports_Postgresql (373.26s)
--- PASS: TestAccAWSDBInstance_SnapshotIdentifier_MultiAZ (1877.62s)
--- PASS: TestAccAWSDBInstance_cloudwatchLogsExportConfigurationUpdate (748.88s)
--- PASS: TestAccAWSDBInstance_NoDeleteAutomatedBackups (655.23s)
--- PASS: TestAccAWSDBInstance_EnabledCloudwatchLogsExports_Oracle (746.82s)
--- PASS: TestAccAWSDBInstance_EnabledCloudwatchLogsExports_MSSQL (837.84s)
--- PASS: TestAccAWSDBInstance_MSSQL_TZ (2062.29s)
--- PASS: TestAccAWSDBInstance_SnapshotIdentifier_MultiAZ_SQLServer (4302.23s)

Output from acceptance testing (commercial/standard partition):

--- FAIL: TestAccAWSDBInstance_DbSubnetGroupName_RamShared (11.33s)
--- FAIL: TestAccAWSDbInstanceDataSource_ec2Classic (25.60s)
--- FAIL: TestAccAWSDBInstance_ReplicateSourceDb_DbSubnetGroupName (1.32s)
--- FAIL: TestAccAWSDBInstance_S3Import (1.29s)
--- FAIL: TestAccAWSDBInstance_separate_iops_update (1.00s)
--- FAIL: TestAccAWSDBInstance_MinorVersion (1.60s)
--- FAIL: TestAccAWSDBInstance_ec2Classic (9.69s)
--- FAIL: TestAccAWSDBInstance_PerformanceInsightsEnabled_EnabledToDisabled (7.67s)
--- FAIL: TestAccAWSDBInstance_PerformanceInsightsKmsKeyId (8.14s)
--- FAIL: TestAccAWSDBInstance_PerformanceInsightsRetentionPeriod (7.56s)
--- FAIL: TestAccAWSDBInstance_PerformanceInsightsEnabled_DisabledToEnabled (497.26s)
--- FAIL: TestAccAWSDBInstance_ReplicateSourceDb_PerformanceInsightsEnabled (665.73s)
--- FAIL: TestAccAWSDBInstance_MySQL_SnapshotRestoreWithEngineVersion (1101.79s)
--- FAIL: TestAccAWSDBInstance_SnapshotIdentifier_PerformanceInsightsEnabled (1180.47s)
--- FAIL: TestAccAWSDBInstance_MSSQL_Domain (2969.55s)
--- SKIP: TestAccAWSDBInstance_ReplicateSourceDb_DeletionProtection (0.00s)
--- SKIP: TestAccAWSDBInstance_SnapshotIdentifier_Tags_Unset (0.00s)
--- PASS: TestAccAWSDBInstance_generatedName (376.58s)
--- PASS: TestAccAWSDBInstance_DeletionProtection (411.38s)
--- PASS: TestAccAWSDBInstance_Password (477.99s)
--- PASS: TestAccAWSDBInstance_basic (482.03s)
--- PASS: TestAccAWSDBInstance_iamAuth (486.41s)
--- PASS: TestAccAWSDBInstance_namePrefix (506.15s)
--- PASS: TestAccAWSDBInstance_kmsKey (508.38s)
--- PASS: TestAccAWSDBInstance_DbSubnetGroupName (509.21s)
--- PASS: TestAccAWSDBInstance_AllowMajorVersionUpgrade (541.00s)
--- PASS: TestAccAWSDBInstance_DbSubnetGroupName_VpcSecurityGroupIds (542.91s)
--- PASS: TestAccAWSDBInstance_IsAlreadyBeingDeleted (561.26s)
--- PASS: TestAccAWSDBInstance_optionGroup (632.05s)
--- PASS: TestAccAWSDbInstanceDataSource_basic (669.20s)
--- PASS: TestAccAWSDBInstance_MaxAllocatedStorage (672.55s)
--- PASS: TestAccAWSDBInstance_FinalSnapshotIdentifier_SkipFinalSnapshot (691.03s)
--- PASS: TestAccAWSDBInstance_FinalSnapshotIdentifier (1036.45s)
--- PASS: TestAccAWSDBInstance_subnetGroup (1069.98s)
--- PASS: TestAccAWSDBInstance_ReplicateSourceDb (1345.67s)
--- PASS: TestAccAWSDBInstance_ReplicateSourceDb_AllowMajorVersionUpgrade (1358.08s)
--- PASS: TestAccAWSDBInstance_ReplicateSourceDb_AutoMinorVersionUpgrade (1080.01s)
--- PASS: TestAccAWSDBInstance_ReplicateSourceDb_AvailabilityZone (1312.78s)
--- PASS: TestAccAWSDBInstance_ReplicateSourceDb_AllocatedStorage (1714.27s)
--- PASS: TestAccAWSDBInstance_ReplicateSourceDb_IamDatabaseAuthenticationEnabled (1279.90s)
--- PASS: TestAccAWSDBInstance_ReplicateSourceDb_MaintenanceWindow (1362.09s)
--- PASS: TestAccAWSDBInstance_ReplicateSourceDb_MaxAllocatedStorage (1347.15s)
--- PASS: TestAccAWSDBInstance_ReplicateSourceDb_BackupWindow (1424.69s)
--- PASS: TestAccAWSDBInstance_ReplicateSourceDb_Port (1299.80s)
--- PASS: TestAccAWSDBInstance_ReplicateSourceDb_VpcSecurityGroupIds (1299.24s)
--- PASS: TestAccAWSDBInstance_ReplicateSourceDb_Monitoring (1567.05s)
--- PASS: TestAccAWSDBInstance_ReplicateSourceDb_BackupRetentionPeriod (1652.83s)
--- PASS: TestAccAWSDBInstance_SnapshotIdentifier (1157.64s)
--- PASS: TestAccAWSDBInstance_ReplicateSourceDb_CACertificateIdentifier (1524.89s)
--- PASS: TestAccAWSDBInstance_ReplicateSourceDb_ParameterGroupName (1621.07s)
--- PASS: TestAccAWSDBInstance_SnapshotIdentifier_AllocatedStorage (1374.80s)
--- PASS: TestAccAWSDBInstance_ReplicateSourceDb_MultiAZ (1904.69s)
--- PASS: TestAccAWSDBInstance_ReplicateSourceDb_DbSubnetGroupName_RamShared (2089.56s)
--- PASS: TestAccAWSDBInstance_SnapshotIdentifier_AutoMinorVersionUpgrade (1178.23s)
--- PASS: TestAccAWSDBInstance_ReplicateSourceDb_DbSubnetGroupName_VpcSecurityGroupIds (2186.02s)
--- PASS: TestAccAWSDBInstance_SnapshotIdentifier_DbSubnetGroupName (1017.86s)
--- PASS: TestAccAWSDBInstance_SnapshotIdentifier_AvailabilityZone (1277.87s)
--- PASS: TestAccAWSDBInstance_SnapshotIdentifier_Io1Storage (1661.11s)
--- PASS: TestAccAWSDBInstance_SnapshotIdentifier_DbSubnetGroupName_VpcSecurityGroupIds (1039.39s)
--- PASS: TestAccAWSDBInstance_SnapshotIdentifier_DeletionProtection (1058.47s)
--- PASS: TestAccAWSDBInstance_SnapshotIdentifier_BackupWindow (1183.09s)
--- PASS: TestAccAWSDBInstance_SnapshotIdentifier_IamDatabaseAuthenticationEnabled (985.16s)
--- PASS: TestAccAWSDBInstance_SnapshotIdentifier_DbSubnetGroupName_RamShared (1311.19s)
--- PASS: TestAccAWSDBInstance_SnapshotIdentifier_MaxAllocatedStorage (1069.46s)
--- PASS: TestAccAWSDBInstance_SnapshotIdentifier_BackupRetentionPeriod_Unset (1508.03s)
--- PASS: TestAccAWSDBInstance_SnapshotIdentifier_MaintenanceWindow (1190.96s)
--- PASS: TestAccAWSDBInstance_SnapshotIdentifier_BackupRetentionPeriod (1642.09s)
--- PASS: TestAccAWSDBInstance_SnapshotIdentifier_Monitoring (1199.81s)
--- PASS: TestAccAWSDBInstance_SnapshotIdentifier_AllowMajorVersionUpgrade (2046.24s)
--- PASS: TestAccAWSDBInstance_SnapshotIdentifier_Tags (987.86s)
--- PASS: TestAccAWSDBInstance_portUpdate (569.62s)
--- PASS: TestAccAWSDBInstance_MonitoringRoleArn_RemovedToEnabled (634.16s)
--- PASS: TestAccAWSDBInstance_MonitoringRoleArn_EnabledToRemoved (656.35s)
--- PASS: TestAccAWSDBInstance_SnapshotIdentifier_Port (1106.56s)
--- PASS: TestAccAWSDBInstance_SnapshotIdentifier_ParameterGroupName (1217.91s)
--- PASS: TestAccAWSDBInstance_SnapshotIdentifier_VpcSecurityGroupIds (1097.54s)
--- PASS: TestAccAWSDBInstance_MonitoringRoleArn_EnabledToDisabled (784.05s)
--- PASS: TestAccAWSDBInstance_cloudwatchLogsExportConfiguration (518.62s)
--- PASS: TestAccAWSDBInstance_SnapshotIdentifier_MultiAZ (1723.47s)
--- PASS: TestAccAWSDBInstance_EnabledCloudwatchLogsExports_Postgresql (382.18s)
--- PASS: TestAccAWSDBInstance_SnapshotIdentifier_VpcSecurityGroupIds_Tags (1131.79s)
--- PASS: TestAccAWSDBInstance_cloudwatchLogsExportConfigurationUpdate (671.40s)
--- PASS: TestAccAWSDBInstance_EnabledCloudwatchLogsExports_MSSQL (675.63s)
--- PASS: TestAccAWSDBInstance_MonitoringInterval (1125.34s)
--- PASS: TestAccAWSDBInstance_EnabledCloudwatchLogsExports_Oracle (755.01s)
--- PASS: TestAccAWSDBInstance_NoDeleteAutomatedBackups (580.20s)
--- PASS: TestAccAWSDBInstance_CACertificateIdentifier (518.97s)
--- PASS: TestAccAWSOpsworksRdsDbInstance_basic (702.31s)
--- PASS: TestAccAWSDBInstance_MSSQL_TZ (1952.91s)
--- PASS: TestAccAWSDBInstance_MSSQL_DomainSnapshotRestore (3189.37s)
--- PASS: TestAccAWSDBInstance_SnapshotIdentifier_MultiAZ_SQLServer (4009.65s)

@YakDriver YakDriver requested a review from a team August 21, 2020 16:45
@ghost ghost added size/XL Managed by automation to categorize the size of a PR. service/rds Issues and PRs that pertain to the rds service. tests PRs: expanded test coverage. Issues: expanded coverage, enhancements to test infrastructure. labels Aug 21, 2020
@YakDriver YakDriver marked this pull request as draft August 21, 2020 16:49
@YakDriver YakDriver force-pushed the t-hardaz-db-instance branch 5 times, most recently from ae5adf1 to b48d7d7 Compare August 25, 2020 16:07
@ghost ghost added the service/opsworks Issues and PRs that pertain to the opsworks service. label Aug 25, 2020
@YakDriver YakDriver force-pushed the t-hardaz-db-instance branch 2 times, most recently from deba2b9 to 3b5d2b3 Compare August 25, 2020 20:56
@YakDriver YakDriver added the partition/aws-us-gov Pertains to the aws-us-gov partition. label Aug 27, 2020
@YakDriver YakDriver force-pushed the t-hardaz-db-instance branch from 3b5d2b3 to c8cb4a6 Compare September 1, 2020 23:39
@ghost ghost added size/XXL Managed by automation to categorize the size of a PR. and removed size/XL Managed by automation to categorize the size of a PR. labels Sep 1, 2020
@YakDriver YakDriver force-pushed the t-hardaz-db-instance branch 2 times, most recently from 03efbfc to 649c366 Compare September 2, 2020 14:15
@YakDriver YakDriver marked this pull request as ready for review September 2, 2020 14:15
@bflad bflad self-assigned this Sep 2, 2020
@YakDriver
Copy link
Member Author

Since the last tctest run, these tests should be fixed.

TestAccAWSDBInstance_DbSubnetGroupName_RamShared
TestAccAWSDBInstance_ReplicateSourceDb_DbSubnetGroupName
TestAccAWSDBInstance_separate_iops_update
TestAccAWSDBInstance_MinorVersion
TestAccAWSDBInstance_PerformanceInsightsEnabled_EnabledToDisabled
TestAccAWSDBInstance_PerformanceInsightsKmsKeyId
TestAccAWSDBInstance_PerformanceInsightsRetentionPeriod
TestAccAWSDBInstance_PerformanceInsightsEnabled_DisabledToEnabled
TestAccAWSDBInstance_ReplicateSourceDb_PerformanceInsightsEnabled
TestAccAWSDBInstance_MySQL_SnapshotRestoreWithEngineVersion
TestAccAWSDBInstance_SnapshotIdentifier_PerformanceInsightsEnabled
TestAccAWSDBInstance_MSSQL_Domain

These tests are not fixed on GovCloud and have unrelated issues:

TestAccAWSDbInstanceDataSource_ec2Classic
TestAccAWSDBInstance_ec2Classic
TestAccAWSDBInstance_S3Import

@YakDriver
Copy link
Member Author

YakDriver commented Sep 2, 2020

TestAccAWSDBInstance_S3Import has not worked for as long as I've been able to find records. I'm not sure what the issue is. I've fixed several but more bubble up.

@bflad
Copy link
Contributor

bflad commented Sep 2, 2020

That test is part of the failures noted in #14384 and will be addressed as part of this quarter's tech debt theme.

@YakDriver
Copy link
Member Author

YakDriver commented Sep 2, 2020

One issue to note is that RDS requires the latest minor version for the s3 import but, for mysql, on commericial that's 5.6.44 and on GovCloud that's 5.6.41.

@bflad
Copy link
Contributor

bflad commented Sep 2, 2020

Can you remove the specification for the engine version?

@bflad
Copy link
Contributor

bflad commented Sep 2, 2020

Oh I guess it would pick some (likely random) version in that case. Maybe a simple workaround of the below?

engine_version = data.aws_partition.current.partition == "aws" ? "5.6.44" : "5.6.41"

Would require a more complicated lookup if the support varies across other partitions 🙁

@YakDriver
Copy link
Member Author

YakDriver commented Sep 2, 2020

I'll try it out. But getting past the version issue just leads to another problem (at least on GovCloud):

unexpected state 'incompatible-restore', wanted target 'available, storage-optimization'

@bflad
Copy link
Contributor

bflad commented Sep 2, 2020

We could also consider a preferred_engine_versions in the data source as well. Bummer that DescribeOrderableDBInstanceOptions doesn't have a SupportsS3Import field.

@bflad
Copy link
Contributor

bflad commented Sep 2, 2020

👍 Cool yeah, #13391 is now the issue for tracking that if you feel super inclined to deal with the special backup file and potentially AWS Support.

@YakDriver
Copy link
Member Author

YakDriver commented Sep 3, 2020

Performance insights aren't supported on GovCloud so I need to add a pre check. #15022 has a pre check that this resource can also use. I'll wait on that to merge before finishing this.

@YakDriver
Copy link
Member Author

YakDriver commented Sep 17, 2020

This PR was waiting on #15022 (merged).

@YakDriver
Copy link
Member Author

YakDriver commented Sep 22, 2020

This was waiting on #15228 (merged).

@YakDriver YakDriver force-pushed the t-hardaz-db-instance branch 2 times, most recently from f05d047 to 2318ec7 Compare September 24, 2020 16:43
@YakDriver YakDriver force-pushed the t-hardaz-db-instance branch 3 times, most recently from 9431aa2 to 063d3a8 Compare September 28, 2020 17:00
@YakDriver YakDriver marked this pull request as ready for review September 28, 2020 17:06
Copy link
Contributor

@bflad bflad left a comment

Choose a reason for hiding this comment

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

🚀

Output from acceptance testing in AWS Commercial (test failures being tracked elsewhere):

--- FAIL: TestAccAWSDBInstance_DbSubnetGroupName_RamShared (2.33s)
--- FAIL: TestAccAWSDBInstance_ReplicateSourceDb_BackupRetentionPeriod (1343.08s)
--- FAIL: TestAccAWSDBInstance_ReplicateSourceDb_DbSubnetGroupName_RamShared (0.95s)
--- FAIL: TestAccAWSDBInstance_S3Import (682.03s)
--- FAIL: TestAccAWSDBInstance_SnapshotIdentifier_DbSubnetGroupName_RamShared (1.08s)
--- PASS: TestAccAWSDBInstance_AllowMajorVersionUpgrade (536.94s)
--- PASS: TestAccAWSDBInstance_basic (467.01s)
--- PASS: TestAccAWSDBInstance_CACertificateIdentifier (428.20s)
--- PASS: TestAccAWSDBInstance_cloudwatchLogsExportConfiguration (522.85s)
--- PASS: TestAccAWSDBInstance_cloudwatchLogsExportConfigurationUpdate (777.66s)
--- PASS: TestAccAWSDBInstance_DbSubnetGroupName (401.44s)
--- PASS: TestAccAWSDBInstance_DbSubnetGroupName_VpcSecurityGroupIds (530.41s)
--- PASS: TestAccAWSDBInstance_DeletionProtection (389.53s)
--- PASS: TestAccAWSDBInstance_EnabledCloudwatchLogsExports_MSSQL (758.11s)
--- PASS: TestAccAWSDBInstance_EnabledCloudwatchLogsExports_Oracle (813.75s)
--- PASS: TestAccAWSDBInstance_EnabledCloudwatchLogsExports_Postgresql (550.69s)
--- PASS: TestAccAWSDBInstance_FinalSnapshotIdentifier (737.80s)
--- PASS: TestAccAWSDBInstance_FinalSnapshotIdentifier_SkipFinalSnapshot (786.45s)
--- PASS: TestAccAWSDBInstance_generatedName (530.22s)
--- PASS: TestAccAWSDBInstance_iamAuth (534.67s)
--- PASS: TestAccAWSDBInstance_IsAlreadyBeingDeleted (369.48s)
--- PASS: TestAccAWSDBInstance_kmsKey (515.03s)
--- PASS: TestAccAWSDBInstance_MaxAllocatedStorage (697.89s)
--- PASS: TestAccAWSDBInstance_MinorVersion (458.03s)
--- PASS: TestAccAWSDBInstance_MonitoringInterval (1072.86s)
--- PASS: TestAccAWSDBInstance_MonitoringRoleArn_EnabledToDisabled (736.29s)
--- PASS: TestAccAWSDBInstance_MonitoringRoleArn_EnabledToRemoved (685.29s)
--- PASS: TestAccAWSDBInstance_MonitoringRoleArn_RemovedToEnabled (440.76s)
--- PASS: TestAccAWSDBInstance_MSSQL_Domain (3674.59s)
--- PASS: TestAccAWSDBInstance_MSSQL_DomainSnapshotRestore (3710.44s)
--- PASS: TestAccAWSDBInstance_MSSQL_TZ (1666.26s)
--- PASS: TestAccAWSDBInstance_MySQL_SnapshotRestoreWithEngineVersion (2072.50s)
--- PASS: TestAccAWSDBInstance_namePrefix (510.86s)
--- PASS: TestAccAWSDBInstance_NoDeleteAutomatedBackups (801.78s)
--- PASS: TestAccAWSDBInstance_optionGroup (495.88s)
--- PASS: TestAccAWSDBInstance_Password (449.57s)
--- PASS: TestAccAWSDBInstance_PerformanceInsightsEnabled_DisabledToEnabled (764.33s)
--- PASS: TestAccAWSDBInstance_PerformanceInsightsEnabled_EnabledToDisabled (856.05s)
--- PASS: TestAccAWSDBInstance_PerformanceInsightsKmsKeyId (1081.61s)
--- PASS: TestAccAWSDBInstance_PerformanceInsightsRetentionPeriod (997.28s)
--- PASS: TestAccAWSDBInstance_portUpdate (528.42s)
--- PASS: TestAccAWSDBInstance_ReplicateSourceDb (1333.69s)
--- PASS: TestAccAWSDBInstance_ReplicateSourceDb_AllocatedStorage (1524.08s)
--- PASS: TestAccAWSDBInstance_ReplicateSourceDb_AllowMajorVersionUpgrade (1474.29s)
--- PASS: TestAccAWSDBInstance_ReplicateSourceDb_AutoMinorVersionUpgrade (1411.78s)
--- PASS: TestAccAWSDBInstance_ReplicateSourceDb_AvailabilityZone (1298.79s)
--- PASS: TestAccAWSDBInstance_ReplicateSourceDb_BackupWindow (1287.28s)
--- PASS: TestAccAWSDBInstance_ReplicateSourceDb_CACertificateIdentifier (1267.96s)
--- PASS: TestAccAWSDBInstance_ReplicateSourceDb_DbSubnetGroupName (2373.24s)
--- PASS: TestAccAWSDBInstance_ReplicateSourceDb_DbSubnetGroupName_VpcSecurityGroupIds (2250.34s)
--- PASS: TestAccAWSDBInstance_ReplicateSourceDb_IamDatabaseAuthenticationEnabled (1429.01s)
--- PASS: TestAccAWSDBInstance_ReplicateSourceDb_MaintenanceWindow (1166.06s)
--- PASS: TestAccAWSDBInstance_ReplicateSourceDb_MaxAllocatedStorage (1540.00s)
--- PASS: TestAccAWSDBInstance_ReplicateSourceDb_Monitoring (1561.64s)
--- PASS: TestAccAWSDBInstance_ReplicateSourceDb_MultiAZ (1893.17s)
--- PASS: TestAccAWSDBInstance_ReplicateSourceDb_ParameterGroupName (1580.56s)
--- PASS: TestAccAWSDBInstance_ReplicateSourceDb_PerformanceInsightsEnabled (1792.83s)
--- PASS: TestAccAWSDBInstance_ReplicateSourceDb_Port (1317.92s)
--- PASS: TestAccAWSDBInstance_ReplicateSourceDb_VpcSecurityGroupIds (1399.70s)
--- PASS: TestAccAWSDBInstance_separateIopsUpdate (892.38s)
--- PASS: TestAccAWSDBInstance_SnapshotIdentifier (1065.64s)
--- PASS: TestAccAWSDBInstance_SnapshotIdentifier_AllocatedStorage (1336.27s)
--- PASS: TestAccAWSDBInstance_SnapshotIdentifier_AllowMajorVersionUpgrade (2034.64s)
--- PASS: TestAccAWSDBInstance_SnapshotIdentifier_AutoMinorVersionUpgrade (1135.07s)
--- PASS: TestAccAWSDBInstance_SnapshotIdentifier_AvailabilityZone (1124.14s)
--- PASS: TestAccAWSDBInstance_SnapshotIdentifier_BackupRetentionPeriod (1479.42s)
--- PASS: TestAccAWSDBInstance_SnapshotIdentifier_BackupRetentionPeriod_Unset (1356.38s)
--- PASS: TestAccAWSDBInstance_SnapshotIdentifier_BackupWindow (1235.87s)
--- PASS: TestAccAWSDBInstance_SnapshotIdentifier_DbSubnetGroupName (1145.18s)
--- PASS: TestAccAWSDBInstance_SnapshotIdentifier_DbSubnetGroupName_VpcSecurityGroupIds (1135.27s)
--- PASS: TestAccAWSDBInstance_SnapshotIdentifier_DeletionProtection (1247.93s)
--- PASS: TestAccAWSDBInstance_SnapshotIdentifier_IamDatabaseAuthenticationEnabled (973.22s)
--- PASS: TestAccAWSDBInstance_SnapshotIdentifier_Io1Storage (1609.23s)
--- PASS: TestAccAWSDBInstance_SnapshotIdentifier_MaintenanceWindow (1023.83s)
--- PASS: TestAccAWSDBInstance_SnapshotIdentifier_MaxAllocatedStorage (1205.47s)
--- PASS: TestAccAWSDBInstance_SnapshotIdentifier_Monitoring (1478.25s)
--- PASS: TestAccAWSDBInstance_SnapshotIdentifier_MultiAZ (2054.68s)
--- PASS: TestAccAWSDBInstance_SnapshotIdentifier_MultiAZ_SQLServer (3841.89s)
--- PASS: TestAccAWSDBInstance_SnapshotIdentifier_ParameterGroupName (1315.93s)
--- PASS: TestAccAWSDBInstance_SnapshotIdentifier_PerformanceInsightsEnabled (1489.33s)
--- PASS: TestAccAWSDBInstance_SnapshotIdentifier_Port (963.86s)
--- PASS: TestAccAWSDBInstance_SnapshotIdentifier_Tags (1205.64s)
--- PASS: TestAccAWSDBInstance_SnapshotIdentifier_VpcSecurityGroupIds (1285.83s)
--- PASS: TestAccAWSDBInstance_SnapshotIdentifier_VpcSecurityGroupIds_Tags (1130.23s)
--- PASS: TestAccAWSDBInstance_subnetGroup (834.37s)
--- PASS: TestAccAWSOpsworksRdsDbInstance_basic (882.56s)
--- SKIP: TestAccAWSDBInstance_ec2Classic (1.14s)
--- SKIP: TestAccAWSDBInstance_ReplicateSourceDb_DeletionProtection (0.00s)
--- SKIP: TestAccAWSDBInstance_SnapshotIdentifier_Tags_Unset (0.00s)

Output from acceptance testing in AWS GovCloud (US) (test failures being tracked elsewhere):

--- FAIL: TestAccAWSDBInstance_CACertificateIdentifier (437.51s)
--- FAIL: TestAccAWSDBInstance_DbSubnetGroupName_RamShared (2.11s)
--- FAIL: TestAccAWSDBInstance_ec2Classic (0.37s)
--- FAIL: TestAccAWSDBInstance_MSSQL_Domain (2580.84s)
--- FAIL: TestAccAWSDBInstance_MSSQL_DomainSnapshotRestore (2879.60s)
--- FAIL: TestAccAWSDBInstance_ReplicateSourceDb_CACertificateIdentifier (619.90s)
--- FAIL: TestAccAWSDBInstance_ReplicateSourceDb_DbSubnetGroupName (9.90s)
--- FAIL: TestAccAWSDBInstance_ReplicateSourceDb_DbSubnetGroupName_RamShared (0.35s)
--- FAIL: TestAccAWSDBInstance_ReplicateSourceDb_DbSubnetGroupName_VpcSecurityGroupIds (9.64s)
--- FAIL: TestAccAWSDBInstance_S3Import (670.25s)
--- FAIL: TestAccAWSDBInstance_SnapshotIdentifier_DbSubnetGroupName_RamShared (0.47s)
--- FAIL: TestAccAWSOpsworksRdsDbInstance_basic (417.92s)
--- PASS: TestAccAWSDBInstance_AllowMajorVersionUpgrade (311.46s)
--- PASS: TestAccAWSDBInstance_basic (454.97s)
--- PASS: TestAccAWSDBInstance_cloudwatchLogsExportConfiguration (505.91s)
--- PASS: TestAccAWSDBInstance_cloudwatchLogsExportConfigurationUpdate (759.30s)
--- PASS: TestAccAWSDBInstance_DbSubnetGroupName (476.84s)
--- PASS: TestAccAWSDBInstance_DbSubnetGroupName_VpcSecurityGroupIds (558.55s)
--- PASS: TestAccAWSDBInstance_DeletionProtection (612.27s)
--- PASS: TestAccAWSDBInstance_EnabledCloudwatchLogsExports_MSSQL (726.20s)
--- PASS: TestAccAWSDBInstance_EnabledCloudwatchLogsExports_Oracle (897.23s)
--- PASS: TestAccAWSDBInstance_EnabledCloudwatchLogsExports_Postgresql (343.35s)
--- PASS: TestAccAWSDBInstance_FinalSnapshotIdentifier (856.53s)
--- PASS: TestAccAWSDBInstance_FinalSnapshotIdentifier_SkipFinalSnapshot (635.91s)
--- PASS: TestAccAWSDBInstance_generatedName (533.60s)
--- PASS: TestAccAWSDBInstance_iamAuth (514.40s)
--- PASS: TestAccAWSDBInstance_IsAlreadyBeingDeleted (409.62s)
--- PASS: TestAccAWSDBInstance_kmsKey (466.67s)
--- PASS: TestAccAWSDBInstance_MaxAllocatedStorage (591.89s)
--- PASS: TestAccAWSDBInstance_MinorVersion (481.90s)
--- PASS: TestAccAWSDBInstance_MonitoringInterval (1119.55s)
--- PASS: TestAccAWSDBInstance_MonitoringRoleArn_EnabledToDisabled (816.62s)
--- PASS: TestAccAWSDBInstance_MonitoringRoleArn_EnabledToRemoved (804.73s)
--- PASS: TestAccAWSDBInstance_MonitoringRoleArn_RemovedToEnabled (690.38s)
--- PASS: TestAccAWSDBInstance_MSSQL_TZ (1724.16s)
--- PASS: TestAccAWSDBInstance_MySQL_SnapshotRestoreWithEngineVersion (1756.31s)
--- PASS: TestAccAWSDBInstance_namePrefix (464.88s)
--- PASS: TestAccAWSDBInstance_NoDeleteAutomatedBackups (572.47s)
--- PASS: TestAccAWSDBInstance_optionGroup (515.22s)
--- PASS: TestAccAWSDBInstance_Password (444.94s)
--- PASS: TestAccAWSDBInstance_portUpdate (508.89s)
--- PASS: TestAccAWSDBInstance_ReplicateSourceDb (1422.57s)
--- PASS: TestAccAWSDBInstance_ReplicateSourceDb_AllocatedStorage (1654.63s)
--- PASS: TestAccAWSDBInstance_ReplicateSourceDb_AllowMajorVersionUpgrade (1251.41s)
--- PASS: TestAccAWSDBInstance_ReplicateSourceDb_AutoMinorVersionUpgrade (1563.31s)
--- PASS: TestAccAWSDBInstance_ReplicateSourceDb_AvailabilityZone (1311.17s)
--- PASS: TestAccAWSDBInstance_ReplicateSourceDb_BackupRetentionPeriod (1564.16s)
--- PASS: TestAccAWSDBInstance_ReplicateSourceDb_BackupWindow (1373.04s)
--- PASS: TestAccAWSDBInstance_ReplicateSourceDb_IamDatabaseAuthenticationEnabled (1523.89s)
--- PASS: TestAccAWSDBInstance_ReplicateSourceDb_MaintenanceWindow (1371.90s)
--- PASS: TestAccAWSDBInstance_ReplicateSourceDb_MaxAllocatedStorage (1463.30s)
--- PASS: TestAccAWSDBInstance_ReplicateSourceDb_Monitoring (1483.21s)
--- PASS: TestAccAWSDBInstance_ReplicateSourceDb_MultiAZ (2230.38s)
--- PASS: TestAccAWSDBInstance_ReplicateSourceDb_ParameterGroupName (1322.48s)
--- PASS: TestAccAWSDBInstance_ReplicateSourceDb_Port (1383.18s)
--- PASS: TestAccAWSDBInstance_ReplicateSourceDb_VpcSecurityGroupIds (1372.90s)
--- PASS: TestAccAWSDBInstance_separateIopsUpdate (711.01s)
--- PASS: TestAccAWSDBInstance_SnapshotIdentifier (1219.29s)
--- PASS: TestAccAWSDBInstance_SnapshotIdentifier_AllocatedStorage (1381.85s)
--- PASS: TestAccAWSDBInstance_SnapshotIdentifier_AllowMajorVersionUpgrade (1836.41s)
--- PASS: TestAccAWSDBInstance_SnapshotIdentifier_AutoMinorVersionUpgrade (1088.88s)
--- PASS: TestAccAWSDBInstance_SnapshotIdentifier_AvailabilityZone (1089.35s)
--- PASS: TestAccAWSDBInstance_SnapshotIdentifier_BackupRetentionPeriod (1392.22s)
--- PASS: TestAccAWSDBInstance_SnapshotIdentifier_BackupRetentionPeriod_Unset (1502.34s)
--- PASS: TestAccAWSDBInstance_SnapshotIdentifier_BackupWindow (1118.30s)
--- PASS: TestAccAWSDBInstance_SnapshotIdentifier_DbSubnetGroupName (958.52s)
--- PASS: TestAccAWSDBInstance_SnapshotIdentifier_DbSubnetGroupName_VpcSecurityGroupIds (1098.60s)
--- PASS: TestAccAWSDBInstance_SnapshotIdentifier_DeletionProtection (1307.04s)
--- PASS: TestAccAWSDBInstance_SnapshotIdentifier_IamDatabaseAuthenticationEnabled (1098.70s)
--- PASS: TestAccAWSDBInstance_SnapshotIdentifier_Io1Storage (1502.29s)
--- PASS: TestAccAWSDBInstance_SnapshotIdentifier_MaintenanceWindow (1239.09s)
--- PASS: TestAccAWSDBInstance_SnapshotIdentifier_MaxAllocatedStorage (1160.11s)
--- PASS: TestAccAWSDBInstance_SnapshotIdentifier_Monitoring (1200.44s)
--- PASS: TestAccAWSDBInstance_SnapshotIdentifier_MultiAZ (1805.21s)
--- PASS: TestAccAWSDBInstance_SnapshotIdentifier_MultiAZ_SQLServer (3818.32s)
--- PASS: TestAccAWSDBInstance_SnapshotIdentifier_ParameterGroupName (1200.87s)
--- PASS: TestAccAWSDBInstance_SnapshotIdentifier_Port (1138.95s)
--- PASS: TestAccAWSDBInstance_SnapshotIdentifier_Tags (1219.38s)
--- PASS: TestAccAWSDBInstance_SnapshotIdentifier_VpcSecurityGroupIds (1088.40s)
--- PASS: TestAccAWSDBInstance_SnapshotIdentifier_VpcSecurityGroupIds_Tags (1059.13s)
--- PASS: TestAccAWSDBInstance_subnetGroup (960.33s)
--- SKIP: TestAccAWSDBInstance_PerformanceInsightsEnabled_DisabledToEnabled (0.27s)
--- SKIP: TestAccAWSDBInstance_PerformanceInsightsEnabled_EnabledToDisabled (0.31s)
--- SKIP: TestAccAWSDBInstance_PerformanceInsightsKmsKeyId (0.27s)
--- SKIP: TestAccAWSDBInstance_PerformanceInsightsRetentionPeriod (0.28s)
--- SKIP: TestAccAWSDBInstance_ReplicateSourceDb_DeletionProtection (0.00s)
--- SKIP: TestAccAWSDBInstance_ReplicateSourceDb_PerformanceInsightsEnabled (0.29s)
--- SKIP: TestAccAWSDBInstance_SnapshotIdentifier_PerformanceInsightsEnabled (0.27s)
--- SKIP: TestAccAWSDBInstance_SnapshotIdentifier_Tags_Unset (0.00s)

@YakDriver YakDriver merged commit 49a20c8 into master Sep 30, 2020
@YakDriver YakDriver deleted the t-hardaz-db-instance branch September 30, 2020 18:47
@ghost
Copy link

ghost commented Oct 31, 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 feel this issue should be reopened, we encourage creating a new issue linking back to this one for added context. Thanks!

@ghost ghost locked as resolved and limited conversation to collaborators Oct 31, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
partition/aws-us-gov Pertains to the aws-us-gov partition. service/opsworks Issues and PRs that pertain to the opsworks service. service/rds Issues and PRs that pertain to the rds service. size/XXL Managed by automation to categorize the size of a PR. tests PRs: expanded test coverage. Issues: expanded coverage, enhancements to test infrastructure.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants