rohityadavcloud commented on code in PR #7111:
URL: https://github.com/apache/cloudstack/pull/7111#discussion_r1081260629
##########
tools/marvin/setup.py:
##########
@@ -46,7 +46,7 @@
"marvin.sandbox.basic"],
license="LICENSE.txt",
install_requires=[
- "mysql-connector-python >= 1.1.6",
+ "mysql-connector-python == 8.0.20",
Review Comment:
Yup that may work as well; but we might be specific like we are with the
pom.xml. As with EL8, cpython version is 3.6 we want to use a dependency that
is certified for cpython3.6 ; see
https://pypi.org/project/mysql-connector-python/8.0.20/#files supports both
cp3.6 and cp2.7. Happy to include your workaround as well. I think 8.0.30 and
8.0.31 might work, it's the new 8.0.32 that breaks. I see 8.0.31 used here
https://github.com/apache/cloudstack/blob/main/packaging/centos8/cloud.spec#L309
(we bundle these for cloudstack-management)
--
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]
