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

Feat/mariner deploy aws gateway. #1767

Merged
merged 2 commits into from
Nov 4, 2021

Conversation

UchicagoZchen138
Copy link
Contributor

@UchicagoZchen138 UchicagoZchen138 commented Oct 28, 2021

Jira Ticket: PXP-8909

New Features

  • mariner persistent volume and persistent volume claim added as part of automatic deployment
  • add mariner pod and mariner engine to indexd ingress policy

@UchicagoZchen138 UchicagoZchen138 force-pushed the feat/mariner-deploy-aws-gateway branch 6 times, most recently from b86127b to d431343 Compare October 28, 2021 20:43
@haraprasadj haraprasadj changed the title Feat/mariner deploy aws gateway Feat/mariner deploy aws gateway. Oct 29, 2021
@UchicagoZchen138 UchicagoZchen138 force-pushed the feat/mariner-deploy-aws-gateway branch from d431343 to 04dfd7b Compare October 29, 2021 16:29
accessModes:
- ReadWriteMany
nfs:
server: 172.24.211.63
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this should not be hardcoded, but need to get this dynamically, either from manifest or programmatically.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

AWS CLI can return this:

aws storagegateway describe-gateway-information --gateway-arn <gateway_arn> | jq -r '.GatewayNetworkInterfaces[].Ipv4Address'

The only thing we'll need a gateway ARN somewhere in the manifest, unless we can map S3 bucket to gateway ARN and use that.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actually, we can use aws storagegateway list-file-shares to list available file shares, use aws storagegateway describe-nfs-file-shares --file-share-arn-list <file_share_arn> to get S3 bucket.

Something like this:

aws storagegateway describe-gateway-information --gateway-arn \
  $(aws storagegateway describe-nfs-file-shares \
    --file-share-arn-list \
    $(aws storagegateway list-file-shares | jq -r '.FileShareInfoList[].FileShareARN') \
    | jq -r '.NFSFileShareInfoList[] | select(.Path == "<replace_this_by_mount_path>") | .GatewayARN') \
  | jq -r '.GatewayNetworkInterfaces[].Ipv4Address'

In this case, <replace_this_by_mount_path> equals to /newstoragegw-poc.

Also, there may be multiple file share to the same bucket and mount path, so it's probably should be configurable. And there may be multiple IP addresses.

@UchicagoZchen138 UchicagoZchen138 force-pushed the feat/mariner-deploy-aws-gateway branch 2 times, most recently from fc21d8d to 816bd26 Compare November 3, 2021 20:31
@UchicagoZchen138 UchicagoZchen138 force-pushed the feat/mariner-deploy-aws-gateway branch from 816bd26 to 0c817a2 Compare November 4, 2021 15:07
@UchicagoZchen138 UchicagoZchen138 force-pushed the feat/mariner-deploy-aws-gateway branch from 0c817a2 to 98a50ab Compare November 4, 2021 16:02
@UchicagoZchen138 UchicagoZchen138 merged commit f3c07ab into master Nov 4, 2021
@UchicagoZchen138 UchicagoZchen138 deleted the feat/mariner-deploy-aws-gateway branch November 4, 2021 20:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants