-
Notifications
You must be signed in to change notification settings - Fork 9
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
Uninversal aws infrastructure #23
Conversation
@g4s8, can you review this PR? |
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.
@Sammers21 why to create new terraform script for AWS infrastructure? We already have working scripts and you can modify it. New implementation doesn't have few important details, such as security groups to access instances via SSH, VPC configuration to setup private network for these instances. Also, we have a script for waiting instances boot for complete https://github.com/artipie/benchmarks/blob/master/files/wait-instance-init.sh
@g4s8, review the PR once again, please |
@g4s8, please, review the PR. |
@g4s8, review the PR, please. |
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.
@Sammers21 please check my comments
@g4s8, review the PR once again, please. |
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.
@Sammers21 thanks
Add universal aws infrastructure with a purpose to be used in every benchmark
Related to #21