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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
