-
Notifications
You must be signed in to change notification settings - Fork 12
changed test_flavor to s3.medium.2 #41
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
playbooks/scenario2_simple_ecs.yaml
Outdated
| test_image: Standard_Fedora_33_latest | ||
| test_flavor: s3.medium.2 | ||
|
|
||
| - name: Check if s3.medium.2 flavor exists |
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.
please rename this task name to Query specific flavor
or something similar
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.
changed
playbooks/scenario2a_ecs_az.yaml
Outdated
| test_keypair_name: "{{ (prefix + '-kp') }}" | ||
| test_network_name: "{{ ( prefix + '-net') }}" | ||
|
|
||
| - name: Check if s3.medium.2 flavor exists |
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.
same here
playbooks/scenario2c_coreos_az.yaml
Outdated
| test_keypair_name: "{{ (prefix + '-kp') }}" | ||
| test_network_name: "{{ ( prefix + '-net') }}" | ||
|
|
||
| - name: Check if s3.medium.2 flavor exists |
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.
and here
changed the default flavor to s3.medium.2, but extended with further logic to keep s2.medium.2 in case s3 is not available yet