Adding example for S3 Express use case #92
Conversation
dlakhaws
changed the title
docs/install.md
Outdated
| @@ -115,6 +115,9 @@ Review the [configuration values](https://github.com/awslabs/mountpoint-s3-csi-d | |||
| kubectl get pods -n kube-system -l app.kubernetes.io/name=aws-mountpoint-s3-csi-driver | |||
| ``` | |||
|
|
|||
| ### Driver example | |||
There was a problem hiding this comment.
Something like "Volume Configuration Example" makes more sense to me here.
|
|
||
| ## Configure | ||
| ### Edit [Persistent Volume](https://github.com/awslabs/mountpoint-s3-csi-driver/blob/main/examples/kubernetes/static_provisioning/static_provisioning.yaml) | ||
| > Note: This example assumes your S3 bucket has already been created. If you need to create a bucket, follow the [S3 documentation](https://docs.aws.amazon.com/AmazonS3/latest/userguide/creating-bucket.html). | ||
| > Note: This example assumes your S3 bucket has already been created. If you need to create a bucket, follow the [S3 documentation](https://docs.aws.amazon.com/AmazonS3/latest/userguide/creating-bucket.html) for a regular bucket and [S3 One Zone documentation](https://docs.aws.amazon.com/AmazonS3/latest/userguide/directory-bucket-create.html) for an S3 Express bucket (aka a Directory bucket). |
There was a problem hiding this comment.
This says S3 One Zone, in line with the above comment it should probably be S3 Express One Zone
There was a problem hiding this comment.
fixed in latest commit
| - key: topology.kubernetes.io/zone | ||
| operator: In | ||
| values: | ||
| - us-west-2a # specify az where bucket lives |
There was a problem hiding this comment.
do you know if there's a selector that supports AZ IDs like usw2-az2? otherwise folks will need to look up the mapping between the AZ ID of their directory bucket and the AZ name in their account (https://docs.aws.amazon.com/ram/latest/userguide/working-with-az-ids.html), which is maybe worth a comment
There was a problem hiding this comment.
nope, that's not supported here so I added a reference to this link since it indicates all the codes that can be used here -- we'll make sure to include support for AZ ID when we work on this issue
Co-authored-by: James Bornholt <[email protected]>
Co-authored-by: James Bornholt <[email protected]>
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.