weizhouapache commented on a change in pull request #4727: URL: https://github.com/apache/cloudstack/pull/4727#discussion_r596633192
########## File path: debian/cloudstack-marvin.postinst ########## @@ -19,5 +19,6 @@ set -e -pip install --upgrade http://cdn.mysql.com/Downloads/Connector-Python/mysql-connector-python-2.0.4.zip#md5=3df394d89300db95163f17c843ef49df -pip install --upgrade /usr/share/cloudstack-marvin/Marvin-*.tar.gz +python3 -m pip install --upgrade pip +python3 -m pip install --upgrade http://cdn.mysql.com/Downloads/Connector-Python/mysql-connector-python-2.0.4.zip#md5=3df394d89300db95163f17c843ef49df Review comment: @GabrielBrascher line 23 is not needed, as pip3 install latest mysql-connector-python 8.0.15 ``` root@mgt01:~# pip3 list |grep mysql-connector-python mysql-connector-python 8.0.15 ``` ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: us...@infra.apache.org
