-
-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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 platform parameter to docker run command. fix #1310 #1471
add platform parameter to docker run command. fix #1310 #1471
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.
The overall change LGTM. Thanks for the contribution. Waiting for the test results.
I see a test failure. Can this be related to your change?
|
Let me check - the weird part is that the same test is failing in the SDKv2 bump branch. I'll take a look today. |
It seems the above failing test was related to a missing HTTP server. This has been fixed as part of #1451 |
Let's rebase the code with the recent change and I'll trigger the test pipeline once again to see if there are any major concern with the change. |
628e9ec
to
2bfafa9
Compare
rebase done |
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. Passing all the relevant tests.
Description
Fixes #1310 add platform parameter to docker run command
TODOs
Read the Gruntwork contribution guidelines.
Release Notes (draft)
Added
platform
parameter to docker run commandMigration Guide
n/a