search

Open-EO / openeo-geotrellis-kubernetes

Contains scripts to run openeo geotrellis backend on a Kubernetes cluster on DIAS.
Apache License 2.0
3 stars 4 forks source link

Eliminate PYTHONPATH and python38 references in openEO helm chart docs #24

Open soxofaan opened 1 month ago

soxofaan commented 1 month ago

Stumbled on this one from https://github.com/EOEPCA/deployment-guide/pull/8:

The openEO webapp driver Sparkpplication helm chart usage docs show setting PYTHONPATH, containing static python38 references, e.g.

https://github.com/Open-EO/openeo-geotrellis-kubernetes/blob/d67111da5cf6ce2f6d477f6e66759caaa302dea4/kubernetes/charts/sparkapplication/README.md?plain=1#L41

I think these things should be baked in internally into the image, and the user of the image should not be bothered with that.

Moreover, we want to upgrade from python 3.8 and these of hard-coded references make that quite hard and annoying to roll out

soxofaan commented 1 month ago

this should also be addressed in https://github.com/EOEPCA/deployment-guide/blob/main/docs/eoepca/openeo-geotrellis.md

jdries commented 1 month ago

Note that I may have solved this already when working on the images, but the yaml file is indeed still the old approach.