You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@marvin.apache.org by GitBox <gi...@apache.org> on 2021/11/24 16:27:48 UTC
[GitHub] [incubator-marvin] cardosolucas opened a new pull request #59: Pre-release modifications
cardosolucas opened a new pull request #59:
URL: https://github.com/apache/incubator-marvin/pull/59
Some modifications to create engine-executor docker image and package name changing.
--
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: notifications-unsubscribe@marvin.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-marvin] cardosolucas commented on a change in pull request #59: Pre-release modifications
Posted by GitBox <gi...@apache.org>.
cardosolucas commented on a change in pull request #59:
URL: https://github.com/apache/incubator-marvin/pull/59#discussion_r786349782
##########
File path: .travis.yml
##########
@@ -37,7 +37,7 @@ matrix:
- language: python
os: linux
python:
- - 3.6
+ - "3.8"
Review comment:
It was getting stuck on validation.
--
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: notifications-unsubscribe@marvin.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-marvin] Wei-1 commented on pull request #59: Pre-release modifications
Posted by GitBox <gi...@apache.org>.
Wei-1 commented on pull request #59:
URL: https://github.com/apache/incubator-marvin/pull/59#issuecomment-980525401
I tested with a minimum tweak, but it doesn't work in #60
I had also print more logs from `tox` in that PR, you can check if there is anything helpful, although it doesn't seem useful to me.
--
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: notifications-unsubscribe@marvin.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-marvin] cardosolucas commented on pull request #59: Pre-release modifications
Posted by GitBox <gi...@apache.org>.
cardosolucas commented on pull request #59:
URL: https://github.com/apache/incubator-marvin/pull/59#issuecomment-1014980661
It's working now. I changed the ubuntu version to get the right openssl version installed.
--
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: notifications-unsubscribe@marvin.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-marvin] Wei-1 commented on pull request #59: Pre-release modifications
Posted by GitBox <gi...@apache.org>.
Wei-1 commented on pull request #59:
URL: https://github.com/apache/incubator-marvin/pull/59#issuecomment-1014988728
I will check and merge it before weekend!
--
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: notifications-unsubscribe@marvin.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-marvin] cardosolucas commented on pull request #59: Pre-release modifications
Posted by GitBox <gi...@apache.org>.
cardosolucas commented on pull request #59:
URL: https://github.com/apache/incubator-marvin/pull/59#issuecomment-981197621
It seems like the problem is with the installation of cryptography library. The openssl required version is > 1.1.0 but we have > 1.0.0. I have tried some fixes but it doesn't work. I'll continue to work on it tomorrow.
--
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: notifications-unsubscribe@marvin.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-marvin] Wei-1 commented on a change in pull request #59: Pre-release modifications
Posted by GitBox <gi...@apache.org>.
Wei-1 commented on a change in pull request #59:
URL: https://github.com/apache/incubator-marvin/pull/59#discussion_r785596047
##########
File path: .travis.yml
##########
@@ -37,7 +37,7 @@ matrix:
- language: python
os: linux
python:
- - 3.6
+ - "3.8"
Review comment:
Weird, the build configuration had not been changed: https://github.com/apache/incubator-marvin/runs/4835434559
--
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: notifications-unsubscribe@marvin.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-marvin] asfgit merged pull request #59: Pre-release modifications
Posted by GitBox <gi...@apache.org>.
asfgit merged pull request #59:
URL: https://github.com/apache/incubator-marvin/pull/59
--
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: notifications-unsubscribe@marvin.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org