search

lephisto / tesla-apiscraper

API Scraper for pulling Vehicle Statistics from the Tesla Owner API into an InfluxDB + Grafana Dashboards
GNU Lesser General Public License v3.0
364 stars 102 forks source link

Docker improvements #73

Open kurthuwig opened 5 years ago

lephisto commented 5 years ago

I will test the changes later today, thanks for that PR

baylanger commented 4 years ago

Reporting from my installation from last night and it didn't go well because ....

This first commit below with title "Windows fix" breaks apiscraper to start, at least not with Python 3.5.

https://github.com/tkrajina/srtm.py/commit/a7c2fa483fc3d24b4fe60bca94190f052e39f31b

The next commit removes support for Python 3.5 and adding 3.7 & 3.8 to the list.

https://github.com/tkrajina/srtm.py/commit/87f417e59978155b6155e749807173542f972d5b

It is sad srtm is dropping support for 3.5 so. " early ". The Debian image comes with Python 3.5 and based on the above commit, it seems 3.6+ would work.

The below steps installs 3.7 and I confirm this makes latest SRTM commits all happy .. since I'm not Docker pro, please double check the steps.

{noformat} RUN echo 'deb http://ftp.de.debian.org/debian testing main' >> /etc/apt/sources.list && \ echo 'APT::Default-Release "stable";' >> /etc/apt/apt.conf.d/00local && \ cat /etc/apt/sources.list && \ cat /etc/apt/apt.conf.d/00local

RUN apt-get update \ && DEBIAN_FRONTEND=noninteractive apt-get install -y --no-install-recommends -t testing \ python3.7 \ python3-distutils \ python3-pip \ && rm -rf /var/lib/apt/lists/ /var/cache/apt/* {noformat}