algorithmiaio / langpacks

Standardized builder and runners for Algorithmia algorithms
https://algorithmia.com/
MIT License
16 stars 9 forks source link

ALGO-890 added lzma to all python base images #178

Closed zeryx closed 3 years ago

zeryx commented 3 years ago

URGENT: lzma support is a blocker for one of our customers.

To Test

./tools/environment_validator.py -g python[x] -s python[x] -t language -n python[x] ^ please test the construction of each version of python that we support:

Please also verify with a simple import pandas as pd that we no longer get the lzma warning, to sanity check that the lzma package is installed.

aslisabanci commented 3 years ago

✅Tested creating the images with:

./tools/environment_validator.py -g python2 -s python27 -t language -n python2

./tools/environment_validator.py -g python3 -s python36 -t language -n python3

./tools/environment_validator.py -g python3 -s python37 -t language -n python3

./tools/environment_validator.py -g python3 -s python38 -t language -n python3

✅ After the final images are created, tested the running container with: curl localhost:9999 -H 'Content-Type: application/json' -d '{"name": "Anthony"}'

✅ docker exec'd into the final containers to do: