Skip to content

fix: remove docker runtime version for test_commands#4723

Merged
mergify[bot] merged 6 commits intoaws:mainlinefrom
0xO0O0:mainline
Apr 5, 2023
Merged

fix: remove docker runtime version for test_commands#4723
mergify[bot] merged 6 commits intoaws:mainlinefrom
0xO0O0:mainline

Conversation

@0xO0O0
Copy link
Copy Markdown
Contributor

@0xO0O0 0xO0O0 commented Apr 1, 2023

Updates docker version from 18 --> 20

Fixes #4721

To address a requirement when running test_commands

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the Apache 2.0 License.

@0xO0O0 0xO0O0 requested a review from a team as a code owner April 1, 2023 02:57
@0xO0O0 0xO0O0 requested review from iamhopaul123 and removed request for a team April 1, 2023 02:57
@Lou1415926 Lou1415926 changed the title Updated Docker Version chore: updated docker runtime version for test_commands Apr 3, 2023
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 3, 2023

🍕 Here are the new binary sizes!

Name New size (kiB) size (kiB) Delta (%)
macOS (amd) 50480 50324 +0.31
macOS (arm) 50300 50128 +0.34
linux (amd) 44436 44300 +0.31
linux (arm) 42756 42564 +0.45
windows (amd) 41320 41204 +0.28

@Lou1415926
Copy link
Copy Markdown
Contributor

Hey @0xO0O0! Copilot loves external contributions Thank you very much ❤️!!

What do you think of @efekarakus's proposal here #4721 (comment)? If you think it sounds good, instead of bumping docker from 18 to 20, can we remove the line?

We will need to apply the same changes in five places:

  1. The actual change:
  2. bb, cc, and dd (the three files that you've changed in this PR) to pass the tests
  3. and also for the testing purpose

Thank you again!!!

@0xO0O0
Copy link
Copy Markdown
Contributor Author

0xO0O0 commented Apr 5, 2023

@Lou1415926 very helpful, thank you! One thing, you posted the gh_template.yaml file twice, was there a third file that also needed updating ?

@Lou1415926
Copy link
Copy Markdown
Contributor

ohh sorry one of the gh_template.yml was meant to be gvv1_template.yml!

@0xO0O0
Copy link
Copy Markdown
Contributor Author

0xO0O0 commented Apr 5, 2023

updated!

@0xO0O0
Copy link
Copy Markdown
Contributor Author

0xO0O0 commented Apr 5, 2023

If the docker version is no longer required, removing would be a good move. Let me know if that's the direction we want to move in and I'll update the pr.

@efekarakus
Copy link
Copy Markdown
Contributor

@0xO0O0

If the docker version is no longer required, removing would be a good move. Let me know if that's the direction we want to move in and I'll update the pr.

That would be awesome, if you don't mind just deleting the install phase 🙏

Copy link
Copy Markdown
Contributor

@paragbhingre paragbhingre left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you so much for your contribution ❤️

Copy link
Copy Markdown
Contributor

@efekarakus efekarakus left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you!!

@efekarakus efekarakus changed the title chore: updated docker runtime version for test_commands fix: remove docker runtime version for test_commands Apr 5, 2023
@mergify mergify Bot merged commit fd0a1b6 into aws:mainline Apr 5, 2023
@KollaAdithya
Copy link
Copy Markdown
Contributor

Fix is now released in v1.28 https://github.com/aws/copilot-cli/releases/tag/v1.28.0. Thanks @0xO0O0 for reporting and contribution!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

docker version 18 in buildspec of test_commands gives 'Unknown runtime version'

6 participants