[GitHub] [openwhisk-runtime-python] dgrove-oss commented on a diff in pull request #143: remove Python 3.6 based runtime

["dgrove-oss (via GitHub)" <gi...@apache.org>]

dgrove-oss commented on code in PR #143:
URL: https://github.com/apache/openwhisk-runtime-python/pull/143#discussion_r1195688877


##########
.github/workflows/ci.yaml:
##########
@@ -90,8 +90,6 @@ jobs:
           SHORT_COMMIT=$(git rev-parse --short "$GITHUB_SHA")
           ./gradlew :core:python3Action:distDocker -PdockerRegistry=docker.io -PdockerImagePrefix=openwhisk -PdockerImageTag=nightly
           ./gradlew :core:python3Action:distDocker -PdockerRegistry=docker.io -PdockerImagePrefix=openwhisk -PdockerImageTag=$SHORT_COMMIT
-          ./gradlew :core:python36AiAction:distDocker -PdockerRegistry=docker.io -PdockerImagePrefix=openwhisk -PdockerImageTag=nightly
-          ./gradlew :core:python36AiAction:distDocker -PdockerRegistry=docker.io -PdockerImagePrefix=openwhisk -PdockerImageTag=$SHORT_COMMIT

Review Comment:
   Yeah, I hesitated on this which is why I didn't remove this back in early 2022 when we first hit the EOL date.  But at this point, Python 3.6 is more than a year past its EOL and all of the AI libraries included in this image are also obsolete.  So, I think this runtime is no longer useful to provide from the core project.   There is enough diversity in Python packages for AI, that its also not clear to me that we can hope to provide the "one true" combination of packages as a core runtime (as opposed to people building custom images and running as a blackbox or as a python virtualenv).



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscribe@openwhisk.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org