Commit 842201c
authored
fix(efs): cannot run an integ test when
### Issue # (if applicable)
N/A
### Reason for this change
I encountered this problem when fixing a bug. This PR fixes an issue where we cannot rerun [test/aws-efs/test/integ.efs-transition.js](https://github.com/aws/aws-cdk/blob/main/packages/%40aws-cdk-testing/framework-integ/test/aws-efs/test/integ.efs-transition.ts).
When attempting to rerun this test, the following error is encountered:
```sh
The ThroughputMode value for the file system does not support TransitionToArchive. Either change the ThroughputMode value to Elastic or remove the TransitionToArchive parameter.
```
When `throughputMode` is `undefined`, throughput mode is set to `Bursting`.
However, `transitionToArchive` is only supported in the Elastic throughput mode. ([Ref](https://docs.aws.amazon.com/efs/latest/ug/API_PutLifecycleConfiguration.html))
> The Archive storage class is available only for file systems that use the Elastic throughput mode and the General Purpose performance mode.
### Description of changes
Set `throughputMode` to `Elastic` when `transitionToArchivePolicy` is specified in a unit test and an integ test.
I also considered adding validations. However, I decided against it as it might cause regression issues.
### Describe any new or updated permissions being added
Nothing
### Description of how you validated changes
Successfully reran the integration test.
### Checklist
- [x] My code adheres to the [CONTRIBUTING GUIDE](https://github.com/aws/aws-cdk/blob/main/CONTRIBUTING.md) and [DESIGN GUIDELINES](https://github.com/aws/aws-cdk/blob/main/docs/DESIGN_GUIDELINES.md)
----
*By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*transitionToArchivePolicy is specified and throughputMode is undefined (#33713)1 parent 25619a0 commit 842201c
File tree
9 files changed
+322
-58
lines changed- packages
- @aws-cdk-testing/framework-integ/test/aws-efs/test
- integ.efs-transition.js.snapshot
- aws-cdk-lib/aws-efs/test
9 files changed
+322
-58
lines changedLines changed: 1 addition & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.
Lines changed: 1 addition & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.
Lines changed: 152 additions & 2 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.
Lines changed: 3 additions & 3 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.
Lines changed: 2 additions & 1 deletion
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
395 | 395 | | |
396 | 396 | | |
397 | 397 | | |
398 | | - | |
| 398 | + | |
| 399 | + | |
399 | 400 | | |
400 | 401 | | |
401 | 402 | | |
| |||
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.
0 commit comments