Skip to content

Commit

Permalink
Merge pull request #40468 from stefanfreitag/f-aws_fsx_windows_file_s…
Browse files Browse the repository at this point in the history
…ystem-update-validation

fix: update validation for throughput capacity
  • Loading branch information
ewbankkit authored Dec 6, 2024
2 parents 26c65a7 + 45c8fbb commit 622c1b5
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 2 deletions.
3 changes: 3 additions & 0 deletions .changelog/40468.txt
Original file line number Diff line number Diff line change
@@ -0,0 +1,3 @@
```release-note:bug
resource/aws_fsx_windows_file_system: Fix plan-time validation of `throughput_capacity` validation to allow values up to `12228`
```
2 changes: 1 addition & 1 deletion internal/service/fsx/windows_file_system.go
Original file line number Diff line number Diff line change
Expand Up @@ -281,7 +281,7 @@ func resourceWindowsFileSystem() *schema.Resource {
"throughput_capacity": {
Type: schema.TypeInt,
Required: true,
ValidateFunc: validation.IntBetween(8, 2048),
ValidateFunc: validation.IntInSlice([]int{8, 16, 32, 64, 128, 256, 512, 1024, 2048, 4608, 6144, 9216, 12228}),
},
names.AttrVPCID: {
Type: schema.TypeString,
Expand Down
2 changes: 1 addition & 1 deletion website/docs/r/fsx_windows_file_system.html.markdown
Original file line number Diff line number Diff line change
Expand Up @@ -53,7 +53,7 @@ resource "aws_fsx_windows_file_system" "example" {
The following arguments are required:

* `subnet_ids` - (Required) A list of IDs for the subnets that the file system will be accessible from. To specify more than a single subnet set `deployment_type` to `MULTI_AZ_1`.
* `throughput_capacity` - (Required) Throughput (megabytes per second) of the file system in power of 2 increments. Minimum of `8` and maximum of `2048`.
* `throughput_capacity` - (Required) Throughput (megabytes per second) of the file system. For valid values, refer to the [AWS documentation](https://docs.aws.amazon.com/fsx/latest/WindowsGuide/performance.html).

The following arguments are optional:

Expand Down

0 comments on commit 622c1b5

Please sign in to comment.