Skip to content

examples/front-proxy: base dockerfile on alpine#2326

Merged
junr03 merged 1 commit intoenvoyproxy:masterfrom
kragniz:front-proxy-alpine
Jan 9, 2018
Merged

examples/front-proxy: base dockerfile on alpine#2326
junr03 merged 1 commit intoenvoyproxy:masterfrom
kragniz:front-proxy-alpine

Conversation

@kragniz
Copy link
Copy Markdown
Contributor

@kragniz kragniz commented Jan 8, 2018

Previously, the example wouldn't build due to the wrong version of
python being used.

Rather than go through the pain of installing a modern version of python
on ubuntu:14.04, this patch just bases examples dockerfile on the envoy
alpine image which includes the necessary version of python3 by default.

Fixes #2262.

Risk Level: Low

Release Notes: N/A

Release Notes: N/A

Previously, the example wouldn't build due to the wrong version of
python being used.

Rather than go through the pain of installing a modern version of python
on ubuntu:14.04, this patch just bases examples dockerfile on the envoy
alpine image which includes the necessary version of python3 by default.

Fixes envoyproxy#2262.

Signed-off-by: Louis Taylor <louis@kragniz.eu>
@mattklein123
Copy link
Copy Markdown
Member

LGTM, thank you @kragniz! @junr03 do you mind giving this a quick review?

@mattklein123 mattklein123 requested a review from junr03 January 9, 2018 00:51
Copy link
Copy Markdown
Member

@junr03 junr03 left a comment

Choose a reason for hiding this comment

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

thanks

@junr03 junr03 merged commit 5c6cb3c into envoyproxy:master Jan 9, 2018
Shikugawa pushed a commit to Shikugawa/envoy that referenced this pull request Mar 28, 2020
jpsim added a commit that referenced this pull request Nov 28, 2022
To pull in bazelbuild/rules_apple#1478

```
Envoy.xcframework
├── Info.plist
├── ios-arm64
│ ├── Envoy.a
│ ├── Envoy.swiftmodule
│ │ ├── arm64.swiftdoc
│ │ └── arm64.swiftinterface
│ └── Headers
│     ├── Envoy.h
│     └── module.modulemap
└── ios-arm64_x86_64-simulator
    ├── Envoy.a
    ├── Envoy.swiftmodule
    │ ├── arm64.swiftdoc
    │ ├── arm64.swiftinterface
    │ ├── x86_64.swiftdoc
    │ └── x86_64.swiftinterface
    └── Headers
        ├── Envoy.h
        └── module.modulemap
```

Risk Level: Low
Testing: Validated by pulling in this version in Xcode and running on device + simulator
Docs Changes: N/A
Release Notes: N/A

Signed-off-by: JP Simard <jp@jpsim.com>
jpsim added a commit that referenced this pull request Nov 29, 2022
To pull in bazelbuild/rules_apple#1478

```
Envoy.xcframework
├── Info.plist
├── ios-arm64
│ ├── Envoy.a
│ ├── Envoy.swiftmodule
│ │ ├── arm64.swiftdoc
│ │ └── arm64.swiftinterface
│ └── Headers
│     ├── Envoy.h
│     └── module.modulemap
└── ios-arm64_x86_64-simulator
    ├── Envoy.a
    ├── Envoy.swiftmodule
    │ ├── arm64.swiftdoc
    │ ├── arm64.swiftinterface
    │ ├── x86_64.swiftdoc
    │ └── x86_64.swiftinterface
    └── Headers
        ├── Envoy.h
        └── module.modulemap
```

Risk Level: Low
Testing: Validated by pulling in this version in Xcode and running on device + simulator
Docs Changes: N/A
Release Notes: N/A

Signed-off-by: JP Simard <jp@jpsim.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