You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@openwhisk.apache.org by "dgrove-oss (via GitHub)" <gi...@apache.org> on 2023/04/26 15:09:46 UTC
[GitHub] [openwhisk-runtime-python] dgrove-oss opened a new pull request, #143: remove Python 3.6 based runtime
dgrove-oss opened a new pull request, #143:
URL: https://github.com/apache/openwhisk-runtime-python/pull/143
Python 3.6 is past its end of life; therefore this runtime is no longer receiving security updates.
--
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
[GitHub] [openwhisk-runtime-python] style95 merged pull request #143: remove Python 3.6 based runtime
Posted by "style95 (via GitHub)" <gi...@apache.org>.
style95 merged PR #143:
URL: https://github.com/apache/openwhisk-runtime-python/pull/143
--
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
[GitHub] [openwhisk-runtime-python] dgrove-oss commented on a diff in pull request #143: remove Python 3.6 based runtime
Posted by "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 runtime 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
[GitHub] [openwhisk-runtime-python] dgrove-oss commented on a diff in pull request #143: remove Python 3.6 based runtime
Posted by "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
[GitHub] [openwhisk-runtime-python] style95 commented on a diff in pull request #143: remove Python 3.6 based runtime
Posted by "style95 (via GitHub)" <gi...@apache.org>.
style95 commented on code in PR #143:
URL: https://github.com/apache/openwhisk-runtime-python/pull/143#discussion_r1195863019
##########
.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:
Fair point.
--
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
[GitHub] [openwhisk-runtime-python] style95 commented on a diff in pull request #143: remove Python 3.6 based runtime
Posted by "style95 (via GitHub)" <gi...@apache.org>.
style95 commented on code in PR #143:
URL: https://github.com/apache/openwhisk-runtime-python/pull/143#discussion_r1194513403
##########
.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:
This will entirely remove AI runtime.
I am not sure we need to archive it for future reference(though we can still refer to the commit).
--
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