-
Notifications
You must be signed in to change notification settings - Fork 2
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
Add support for supplying Fargate platform version #26
Conversation
@@ -170,34 +171,29 @@ def run_task(self, config, task_def_arn, security_group_id, subnet_id, cmd): | |||
taskDefinition=task_def_arn | |||
)["taskDefinition"] | |||
|
|||
kwargs = { |
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.
Using a kwargs
dictionary here aids in removing the task_response
code duplication below.
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.
Nice refactor! 🎯
@@ -1,3 +1,5 @@ | |||
from setuptools import setup | |||
|
|||
setup(use_scm_version=True,) | |||
setup( |
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.
Black complained about this during the PR build.
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.
Tested this out with OAR, looks good to go 🚢
Pass through the platformVersion keyword argument when the configured launchType is FARGATE. In addition, default platformVersion to LATEST.
e7955ee
to
892a664
Compare
Overview
Pass through the
platformVersion
keyword argument when the configuredlaunchType
isFARGATE
. In addition, defaultplatformVersion
toLATEST
.Closes #19
Demo
Testing Instructions
See: opensupplyhub/open-apparel-registry#1129