-
Notifications
You must be signed in to change notification settings - Fork 0
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
PM-1343 S3 fallback #2
Conversation
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.
I like the kc > ctx 👍
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. Just a minor request for README update.
README.md
Outdated
**3. AWS S3 Configuration**: | ||
|
||
- `AWS_S3_BUCKET` - AWS S3 Bucket | ||
- `NETWORK_NAME` - Network name (in case block does not exist in Cassandra | ||
we attampt to download it from AWS S3 from `AWS_S3_BUCKET`\\`NETWORK_NAME`\blocks) | ||
- `AWS_REGION` - The AWS region. | ||
- `AWS_ACCESS_KEY_ID` - Your AWS Access Key ID. | ||
- `AWS_SECRET_ACCESS_KEY` - Your AWS Secret Access Key. | ||
|
||
|
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.
It would be nice to mention whether these options are mandatory or not and how does the program behave in their absence (if it does not exit).
This PR introduces S3 fallback functionality. In case
raw_block
is missing for particular submission in Cassandra we attempt to download it fromAWS_S3_BUCKET
/NETWORK_NAME
/blocks/blockhash.dat. Such updated submission list we pass todelegation-verify
.