-
Notifications
You must be signed in to change notification settings - Fork 3k
Amazon KMS + SES extensions #9413
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
Conversation
|
Hopefully, this makes it in 1.5! |
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.
Looks good to me, thanks! Do you think you will be able to write the guides before next Tuesday? Just to know if we should target 1.5 or push this to 1.6.
|
I might not have it ready by Tuesday. So 1.6 seems more realistic
śr., 20 maj 2020, 10:01 użytkownik Guillaume Smet <[email protected]>
napisał:
… ***@***.**** requested changes on this pull request.
Looks good to me, thanks! Do you think you will be able to write the
guides before next Tuesday? Just to know if we should target 1.5 or push
this to 1.6.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#9413 (review)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ABAF4G2BJFMWRTYNIHBJPEDRSOE43ANCNFSM4NEEAKXQ>
.
|
| - name: Amazon Services | ||
| run: | | ||
| docker run --rm --publish 8000:4569 --publish 8008:4572 --publish 8009:4575 --publish 8010:4576 --name build-amazon-service-clients -e SERVICES=s3,dynamodb,sns,sqs -e START_WEB=0 \ | ||
| docker run --rm --publish 8000:4569 --publish 8008:4572 --publish 8009:4575 --publish 8010:4576 --publish 8011:4599 --publish 8012:4566 --name build-amazon-service-clients -e SERVICES=s3,dynamodb,sns,sqs,kms,ses -e START_WEB=0 \ |
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.
You probably need to make the timeout for the Amazon services native job higher. It's 20 right now, maybe make it 30.
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.
Yep, i noticed that
1d66346 to
b9a4301
Compare
|
@gsmet - I managed to produce guides and quickstart. |
|
OK, I will merge that one and backport. I will probably tweak the guides a bit later if I have some time. Thanks for the quick turnaround! |
Amazon KMS (key management service) + SES (Simple email service) extensions