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

EFS: support setting the AZ when setting up a oneZone FileSystem #30005

Closed
2 tasks
kabo opened this issue Apr 29, 2024 · 4 comments · Fixed by #30010
Closed
2 tasks

EFS: support setting the AZ when setting up a oneZone FileSystem #30005

kabo opened this issue Apr 29, 2024 · 4 comments · Fixed by #30010
Labels
@aws-cdk/aws-efs Related to Amazon Elastic File System effort/medium Medium work item – several days of effort feature-request A feature should be added or improved. p2

Comments

@kabo
Copy link

kabo commented Apr 29, 2024

Describe the feature

We'd like to be able to specify which AZ the one-zone FS should be in

Use Case

So it's in the same AZ as the rest of our setup. This is for a test environment so one AZ is fine and cheaper.

Proposed Solution

There's a vpcSubnets constructor prop, it'd be handy if it could work like this

    const efs = new FileSystem(this, 'Efs', {
      vpc,
      encrypted: true,
      oneZone: true,
      vpcSubnets: { availabilityZones: [ 'eu-west-1b' ] },
    })

But maybe some other solution is easier.

Other Information

No response

Acknowledgements

  • I may be able to implement this feature request
  • This feature might incur a breaking change

CDK version used

2.136.0

Environment details (OS name and version, etc.)

Artix Linux

@kabo kabo added feature-request A feature should be added or improved. needs-triage This issue or PR still needs to be triaged. labels Apr 29, 2024
@github-actions github-actions bot added the @aws-cdk/aws-efs Related to Amazon Elastic File System label Apr 29, 2024
@pahud
Copy link
Contributor

pahud commented Apr 30, 2024

Sounds good to me.

At this moment it's determined automatically by selecting the first VPC AZ this way, which is not configurable.

const oneZoneAzName = props.vpc.availabilityZones[0];

Having the support to allow users to configure would be great.

@pahud pahud added p2 effort/medium Medium work item – several days of effort and removed needs-triage This issue or PR still needs to be triaged. labels Apr 30, 2024
@mergify mergify bot closed this as completed in #30010 May 1, 2024
@mergify mergify bot closed this as completed in cbf130e May 1, 2024
Copy link

github-actions bot commented May 1, 2024

⚠️COMMENT VISIBILITY WARNING⚠️

Comments on closed issues are hard for our team to see.
If you need more assistance, please either tag a team member or open a new issue that references this one.
If you wish to keep having a conversation with other community members under this issue feel free to do so.

1 similar comment
Copy link

github-actions bot commented May 1, 2024

⚠️COMMENT VISIBILITY WARNING⚠️

Comments on closed issues are hard for our team to see.
If you need more assistance, please either tag a team member or open a new issue that references this one.
If you wish to keep having a conversation with other community members under this issue feel free to do so.

@aws-cdk-automation
Copy link
Collaborator

Comments on closed issues and PRs are hard for our team to see. If you need help, please open a new issue that references this one.

@aws aws locked as resolved and limited conversation to collaborators Jul 25, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
@aws-cdk/aws-efs Related to Amazon Elastic File System effort/medium Medium work item – several days of effort feature-request A feature should be added or improved. p2
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants