-
Notifications
You must be signed in to change notification settings - Fork 9.2k
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
Reduce minimum file system throughput for aws_fsx_ontap_file_system #22898
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 🚀.
% make testacc TESTS=TestAccFSxOntapFileSystem_ PKG=fsx ACCTEST_PARALLELISM=5
==> Checking that code complies with gofmt requirements...
TF_ACC=1 go test ./internal/service/fsx/... -v -count 1 -parallel 5 -run='TestAccFSxOntapFileSystem_' -timeout 180m
=== RUN TestAccFSxOntapFileSystem_basic
=== PAUSE TestAccFSxOntapFileSystem_basic
=== RUN TestAccFSxOntapFileSystem_fsxAdminPassword
=== PAUSE TestAccFSxOntapFileSystem_fsxAdminPassword
=== RUN TestAccFSxOntapFileSystem_endpointIPAddressRange
=== PAUSE TestAccFSxOntapFileSystem_endpointIPAddressRange
=== RUN TestAccFSxOntapFileSystem_diskIops
=== PAUSE TestAccFSxOntapFileSystem_diskIops
=== RUN TestAccFSxOntapFileSystem_disappears
=== PAUSE TestAccFSxOntapFileSystem_disappears
=== RUN TestAccFSxOntapFileSystem_securityGroupIDs
=== PAUSE TestAccFSxOntapFileSystem_securityGroupIDs
=== RUN TestAccFSxOntapFileSystem_routeTableIDs
=== PAUSE TestAccFSxOntapFileSystem_routeTableIDs
=== RUN TestAccFSxOntapFileSystem_tags
=== PAUSE TestAccFSxOntapFileSystem_tags
=== RUN TestAccFSxOntapFileSystem_weeklyMaintenanceStartTime
=== PAUSE TestAccFSxOntapFileSystem_weeklyMaintenanceStartTime
=== RUN TestAccFSxOntapFileSystem_automaticBackupRetentionDays
=== PAUSE TestAccFSxOntapFileSystem_automaticBackupRetentionDays
=== RUN TestAccFSxOntapFileSystem_kmsKeyID
=== PAUSE TestAccFSxOntapFileSystem_kmsKeyID
=== RUN TestAccFSxOntapFileSystem_dailyAutomaticBackupStartTime
=== PAUSE TestAccFSxOntapFileSystem_dailyAutomaticBackupStartTime
=== CONT TestAccFSxOntapFileSystem_basic
=== CONT TestAccFSxOntapFileSystem_routeTableIDs
=== CONT TestAccFSxOntapFileSystem_diskIops
=== CONT TestAccFSxOntapFileSystem_securityGroupIDs
=== CONT TestAccFSxOntapFileSystem_disappears
--- PASS: TestAccFSxOntapFileSystem_basic (1987.20s)
=== CONT TestAccFSxOntapFileSystem_automaticBackupRetentionDays
--- PASS: TestAccFSxOntapFileSystem_diskIops (2030.51s)
=== CONT TestAccFSxOntapFileSystem_dailyAutomaticBackupStartTime
--- PASS: TestAccFSxOntapFileSystem_disappears (2103.77s)
=== CONT TestAccFSxOntapFileSystem_weeklyMaintenanceStartTime
=== CONT TestAccFSxOntapFileSystem_routeTableIDs
ontap_file_system_test.go:233: Step 1/2 error: Error running apply: exit status 1
Error: error waiting for FSx ONTAP File System (fs-0869b3a29670b19c7) create: timeout while waiting for state to become 'AVAILABLE' (last state: 'CREATING', timeout: 1h0m0s)
with aws_fsx_ontap_file_system.test,
on terraform_plugin_test.tf line 62, in resource "aws_fsx_ontap_file_system" "test":
62: resource "aws_fsx_ontap_file_system" "test" {
--- PASS: TestAccFSxOntapFileSystem_securityGroupIDs (4060.80s)
=== CONT TestAccFSxOntapFileSystem_endpointIPAddressRange
--- PASS: TestAccFSxOntapFileSystem_weeklyMaintenanceStartTime (2064.68s)
=== CONT TestAccFSxOntapFileSystem_kmsKeyID
--- PASS: TestAccFSxOntapFileSystem_dailyAutomaticBackupStartTime (2190.34s)
=== CONT TestAccFSxOntapFileSystem_fsxAdminPassword
--- FAIL: TestAccFSxOntapFileSystem_routeTableIDs (4322.67s)
=== CONT TestAccFSxOntapFileSystem_tags
--- PASS: TestAccFSxOntapFileSystem_automaticBackupRetentionDays (2357.42s)
--- PASS: TestAccFSxOntapFileSystem_endpointIPAddressRange (2129.24s)
--- PASS: TestAccFSxOntapFileSystem_kmsKeyID (2253.67s)
--- PASS: TestAccFSxOntapFileSystem_tags (2134.37s)
--- PASS: TestAccFSxOntapFileSystem_fsxAdminPassword (2263.56s)
FAIL
FAIL github.com/hashicorp/terraform-provider-aws/internal/service/fsx 6488.809s
FAIL
make: *** [testacc] Error 1
Failures are unrelated to this change.
@lvthillo Thanks for the contribution 🎉 👏. |
This functionality has been released in v4.0.0 of the Terraform AWS Provider. Please see the Terraform documentation on provider versioning or reach out if you need any assistance upgrading. For further feature requests or bug reports with this functionality, please create a new GitHub issue following the template. Thank you! |
I'm going to lock this pull request because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues. |
Community Note
Closes #22896
Output from acceptance testing:
Example usage: