Skip to content

docker: remove alpine build#19837

Merged
phlax merged 1 commit intoenvoyproxy:mainfrom
daixiang0:alipine-sh
Feb 7, 2022
Merged

docker: remove alpine build#19837
phlax merged 1 commit intoenvoyproxy:mainfrom
daixiang0:alipine-sh

Conversation

@daixiang0
Copy link
Copy Markdown
Member

Signed-off-by: Loong loong.dai@intel.com

Part of #19781

Commit Message: do not build for alpine anymore.
Additional Description:
Risk Level:
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Fixes commit #PR or SHA]
[Optional Deprecated:]
[Optional API Considerations:]

@daixiang0 daixiang0 changed the title docker: remove alpine docker: remove alpine build Feb 7, 2022
@phlax phlax self-assigned this Feb 7, 2022
Copy link
Copy Markdown
Member

@phlax phlax left a comment

Choose a reason for hiding this comment

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

@daixiang0 could you remove it from the Dockerfile also please

@daixiang0
Copy link
Copy Markdown
Member Author

Please review again.

Copy link
Copy Markdown
Member

@phlax phlax left a comment

Choose a reason for hiding this comment

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

lgtm, thanks @daixiang0

@phlax phlax enabled auto-merge (squash) February 7, 2022 07:20
PiotrSikora added a commit to PiotrSikora/envoy that referenced this pull request Feb 7, 2022
Signed-off-by: Piotr Sikora <piotrsikora@google.com>
@phlax phlax merged commit bdd22a6 into envoyproxy:main Feb 7, 2022
Signed-off-by: Loong <loong.dai@intel.com>
@daixiang0 daixiang0 deleted the alipine-sh branch February 8, 2022 00:40
joshperry pushed a commit to joshperry/envoy that referenced this pull request Feb 13, 2022
Signed-off-by: Loong <loong.dai@intel.com>
Signed-off-by: Josh Perry <josh.perry@mx.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.

2 participants