opentelemetry-lambda icon indicating copy to clipboard operation
opentelemetry-lambda copied to clipboard

Delete python/src/otel/Makefile

Open anuraaga opened this issue 3 years ago • 3 comments

python/src/otel/Makefile has commands like yum install - presumably this isn't for normal use. Do we still need it or can delete?

anuraaga avatar Apr 08 '21 07:04 anuraaga

OTel Python SDK otlp exporter has to be packaged in Linux, so run.sh use sam build -u to make sure the makefile is running in docker linux.

https://github.com/open-telemetry/opentelemetry-lambda/blob/main/utils/sam/run.sh#L92

wangzlei avatar Apr 14 '21 16:04 wangzlei

Since now we use TF to build python layer, the Makefile is no longer required.

wangzlei avatar May 04 '21 18:05 wangzlei

Thanks you so much for providing more context, referring to these steps in the README.MD. Could there be customers still using the sam build for building the layer?

vasireddy99 avatar Apr 06 '22 18:04 vasireddy99