Closed rpmcginty closed 2 days ago
Hi @rpmcginty, thanks for raising a PR here.
Do you know what AWS actually uses as the length of this identifier? The documentation is a bit confused:
FileSystemId
The system-generated, unique 17-digit ID of the file system.
Type: String
Length Constraints: Minimum length of 11. Maximum length of 21.
Pattern: ^(fs-[0-9a-f]{8,})$
Required: No
Based on this I can't tell whether it's supposed to 8, 17, or anything between 11 and 21.
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 94.50%. Comparing base (
d8da0ac
) to head (6c76aac
). Report is 11 commits behind head on master.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Hi @rpmcginty, thanks for raising a PR here.
Do you know what AWS actually uses as the length of this identifier? The documentation is a bit confused:
FileSystemId
The system-generated, unique 17-digit ID of the file system. Type: String Length Constraints: Minimum length of 11. Maximum length of 21. Pattern: ^(fs-[0-9a-f]{8,})$ Required: No
Based on this I can't tell whether it's supposed to 8, 17, or anything between 11 and 21.
yeah it is a bit confusing I suppose. The character count does include the "fs-" so I think the minimum is in fact 11 characters (not 8). Having dealt with fsx ids I have only seen 17 characters so I can update it if you would like me to.
This change intends to address https://github.com/getmoto/moto/issues/8148
Modifies how fsx fs id is generated so that: