-
Notifications
You must be signed in to change notification settings - Fork 6
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: add the EIPFS infra for the blob protocol #361
Conversation
joaosa
commented
Apr 29, 2024
•
edited
Loading
edited
- adds the EIPFS multihashes queue;
- adds the EIPFS blocks CAR positions table.
dd9e7d5
to
8be72e7
Compare
8be72e7
to
7e63fa2
Compare
View stack outputs
|
.env.tpl
Outdated
@@ -13,6 +13,8 @@ | |||
|
|||
EIPFS_INDEXER_SQS_ARN = 'arn:aws:sqs:us-west-2:505595374361:staging-ep-indexer-topic' | |||
EIPFS_INDEXER_SQS_URL = 'https://sqs.us-west-2.amazonaws.com/505595374361/staging-ep-indexer-topic' | |||
EIPFS_MULTIHASHES_SQS_ARN = 'arn:aws:sqs:us-west-2:505595374361:multihashes-topic' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There is no "staging-multihashes-topic"?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
AFAICT no, so we maybe we should create it.
my bad. found it!
@@ -89,6 +116,7 @@ export function UploadApiStack({ stack, app }) { | |||
PIECE_TABLE_NAME: pieceTable.tableName, | |||
PIECE_OFFER_QUEUE_URL: pieceOfferQueue.queueUrl, | |||
FILECOIN_SUBMIT_QUEUE_URL: filecoinSubmitQueue.queueUrl, | |||
BLOCKS_CAR_POSITION_TABLE_NAME: blocksCarPositionTable.tableName, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Also the multihashes queue URL?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
alright wasn't sure about that one 👍 . adding it
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks perfect, thanks!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Can you please set these env vars in seed.run?
done! |