Skip to content

Remove unused powershell call from Windows docker file#17472

Merged
wrowe merged 1 commit intoenvoyproxy:mainfrom
davinci26:removeUnusedCallWindowsDocker
Jul 26, 2021
Merged

Remove unused powershell call from Windows docker file#17472
wrowe merged 1 commit intoenvoyproxy:mainfrom
davinci26:removeUnusedCallWindowsDocker

Conversation

@davinci26
Copy link
Copy Markdown
Member

@davinci26 davinci26 commented Jul 23, 2021

Ever since #15461 we no longer have a reference to /tmp/ in envoyproxy_io_proxy.yaml so we do not need this powershell call.

Removing this call is important because it allows to build the Envoy Windows Container on Nanoserver which is required for #16759

Risk Level: Low
Testing: Manually build the container
Docs Changes: N/A
Release Notes: N/A (Null-Op)
Platform Specific Features: Windows only

Signed-off-by: Sotiris Nanopoulos sonanopo@microsoft.com

Signed-off-by: Sotiris Nanopoulos <sonanopo@microsoft.com>
@davinci26
Copy link
Copy Markdown
Member Author

cc @envoyproxy/windows-dev

Copy link
Copy Markdown
Contributor

@wrowe wrowe left a comment

Choose a reason for hiding this comment

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

Looks sharp, thx!

@wrowe wrowe merged commit 2f49895 into envoyproxy:main Jul 26, 2021
leyao-daily pushed a commit to leyao-daily/envoy that referenced this pull request Sep 30, 2021
)

Signed-off-by: Sotiris Nanopoulos <sonanopo@microsoft.com>
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.

3 participants