Skip to content

Add python3.8-dev to the Ubuntu image#144

Merged
phlax merged 4 commits intoenvoyproxy:mainfrom
lfpino:patch-1
Sep 8, 2021
Merged

Add python3.8-dev to the Ubuntu image#144
phlax merged 4 commits intoenvoyproxy:mainfrom
lfpino:patch-1

Conversation

@lfpino
Copy link
Copy Markdown
Contributor

@lfpino lfpino commented Sep 6, 2021

The python headers are necessary to build envoy-mobile with remote build execution

@phlax
Copy link
Copy Markdown
Member

phlax commented Sep 7, 2021

hi @lfpino i dont use this image myself, but in my local dev image i have python3.8-dev enabled to allow installing deps that require compilation, so seems a good change to me

would you mind fixing dco - i think the other ci fail was a flake so i have rerun the job

@phlax
Copy link
Copy Markdown
Member

phlax commented Sep 7, 2021

ci breakage is unrelated, i opened a pr here #145 to address

The python headers are necessary to build envoy-mobile with remote build execution

Signed-off-by: Luis Fernando Pino Duque <luis@engflow.com>
@lfpino lfpino marked this pull request as ready for review September 7, 2021 12:36
@lfpino lfpino requested a review from a team as a code owner September 7, 2021 12:36
@lfpino
Copy link
Copy Markdown
Contributor Author

lfpino commented Sep 7, 2021

@phlax thanks for taking a look! I fixed the DCO now

Signed-off-by: Luis Fernando Pino Duque <luis@engflow.com>
Signed-off-by: Luis Fernando Pino Duque <luis@engflow.com>
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 @lfpino

@phlax phlax merged commit 072df4c into envoyproxy:main Sep 8, 2021
htuch pushed a commit that referenced this pull request Sep 8, 2021
  [skip ci]
  Add python3.8-dev to the Ubuntu image (#144)

* Add python3.8-dev to the Ubuntu image

The python headers are necessary to build envoy-mobile with remote build execution

Signed-off-by: Luis Fernando Pino Duque <luis@engflow.com>
htuch pushed a commit that referenced this pull request Sep 8, 2021
  [skip ci]
  Regenerate linux toolchains from 072df4c

  [skip ci]
  Add python3.8-dev to the Ubuntu image (#144)

* Add python3.8-dev to the Ubuntu image

The python headers are necessary to build envoy-mobile with remote build execution

Signed-off-by: Luis Fernando Pino Duque <luis@engflow.com>
goaway pushed a commit to envoyproxy/envoy-mobile that referenced this pull request Sep 13, 2021
Description: Move python_tests workflow to Engflow's remote execution and make build times >10x better

We also need to update the Docker image to one of the latest in envoy-build-ubuntu so that it includes envoyproxy/envoy-build-tools#144. The image can be found at: https://hub.docker.com/layers/envoyproxy/envoy-build-ubuntu/b51ac7a0f2cb5dd729984a379a301ce029ac765d/images/sha256-239015d203837f2bdcd4cfdd710d7db60acc3fb1f002f1c926b92b42c59afdd6?context=explore

Risk Level: Low
Testing: See python_tests workflow
Docs Changes: N/A
Release Notes: N/A

Signed-off-by: Luis Fernando Pino Duque <luis@engflow.com>
jpsim pushed a commit to envoyproxy/envoy that referenced this pull request Nov 28, 2022
Description: Move python_tests workflow to Engflow's remote execution and make build times >10x better

We also need to update the Docker image to one of the latest in envoy-build-ubuntu so that it includes envoyproxy/envoy-build-tools#144. The image can be found at: https://hub.docker.com/layers/envoyproxy/envoy-build-ubuntu/b51ac7a0f2cb5dd729984a379a301ce029ac765d/images/sha256-239015d203837f2bdcd4cfdd710d7db60acc3fb1f002f1c926b92b42c59afdd6?context=explore

Risk Level: Low
Testing: See python_tests workflow
Docs Changes: N/A
Release Notes: N/A

Signed-off-by: Luis Fernando Pino Duque <luis@engflow.com>
Signed-off-by: JP Simard <jp@jpsim.com>
jpsim pushed a commit to envoyproxy/envoy that referenced this pull request Nov 29, 2022
Description: Move python_tests workflow to Engflow's remote execution and make build times >10x better

We also need to update the Docker image to one of the latest in envoy-build-ubuntu so that it includes envoyproxy/envoy-build-tools#144. The image can be found at: https://hub.docker.com/layers/envoyproxy/envoy-build-ubuntu/b51ac7a0f2cb5dd729984a379a301ce029ac765d/images/sha256-239015d203837f2bdcd4cfdd710d7db60acc3fb1f002f1c926b92b42c59afdd6?context=explore

Risk Level: Low
Testing: See python_tests workflow
Docs Changes: N/A
Release Notes: N/A

Signed-off-by: Luis Fernando Pino Duque <luis@engflow.com>
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.

2 participants