[#693] update mysql-connector-python

> #693

Author: eeliu

.github/workflows/main.yml

@@ -276,10 +276,13 @@ jobs:
         # run: docker-compose  -f "testapps/compose.yaml" build python-plugins
       # - name: build python-plugins
       - name: start test environment
-        run: ./docker-compose-linux-x86_64 -f "testapps/compose.yaml" up python-plugins --exit-code-from python-plugins
+        run: |
+          ./docker-compose-linux-x86_64 -f "testapps/compose.yaml" up py3.9-plugins --exit-code-from py3.9-plugins
+          ./docker-compose-linux-x86_64 -f "testapps/compose.yaml" up py-plugins --exit-code-from py-plugins
+          ./docker-compose-linux-x86_64 -f "testapps/compose.yaml" up py3.11-plugins --exit-code-from py3.11-plugins
       - name: Stop containers
         # if: always()
-        run: ./docker-compose-linux-x86_64 -f "testapps/compose.yaml" down python-plugins
+        run: ./docker-compose-linux-x86_64 -f "testapps/compose.yaml" down py3.11-plugins py-plugins py3.9-plugins 
 
   Collector-agent:
     strategy:

plugins/PY/pinpointPy/libs/_MysqlConnector/CMysqlPlugin.py

@@ -15,11 +15,17 @@
 # ------------------------------------------------------------------------------
 
 from pinpointPy import Common, Defines, pinpoint
+import mysql
+import warnings
+
+#
 
 
 class CMysqlPlugin(Common.PinTrace):
 
     def __init__(self, name):
+        warnings.warn(
+            "CMysqlPlugin is deprecated, please use MysqlPlugin instead", DeprecationWarning)
         super().__init__(name)
 
     # -> tuple[Any, tuple[Any, ...], dict[str, Any]]:
@@ -48,4 +54,5 @@ def onException(self, traceId, e):
         pinpoint.add_exception(f'{e}', traceId)
 
     def get_cursor_host(self, cursor):
-        return f'{cursor._cnx.server_host}:{cursor._cnx.server_port}'
+        # https://github.com/mysql/mysql-connector-python/blob/4fbf521f1c6c71621f882f89c0c4946c10ee13ac/mysql-connector-python/lib/mysql/connector/abstracts.py#L692
+        return f'{cursor._connection._host}:{cursor._connection._host}'

plugins/PY/pinpointPy/libs/_MysqlConnector/__init__.py

@@ -55,5 +55,9 @@ def monkey_patch():
 
 
 __all__ = ['monkey_patch']
-__version__ = '0.0.5'
+__version__ = '0.0.6'
 __author__ = '[email protected]'
+
+# Changes
+# ## 0.0.6
+# - remove `CMySQLCursor` and `CMySQLCursorPrepared`

plugins/PY/requirements.txt

@@ -1,16 +1,16 @@
-bottle==0.12.25
-fastapi==0.111.0
-flask==3.0.0
-httpx==0.25.1
-mysql-connector-python==9.1.0
-pymongo==4.6.3
-PyMySQL==1.1.1
-redis==5.0.1
-requests==2.31.0
-SQLAlchemy==2.0.23
-starlette-context==0.3.6
-mysqlclient==2.2.0
+bottle
+fastapi
+flask
+httpx
+mysql-connector-python
+pymongo
+PyMySQL
+redis
+requests
+SQLAlchemy
+starlette-context
+mysqlclient
 grpcio
-grpc-interceptor==0.15.4
+grpc-interceptor
 psycopg2
 pinpointPy

setup.py

@@ -39,7 +39,7 @@
                  Path(cwd, './common/src')]
 
 setup(name='pinpointPy',
-      version="1.3.1",  # don't forget update __version__ in pinpointPy/__init__.py
+      version="1.3.2",  # don't forget update __version__ in pinpointPy/__init__.py
       author="cd_pinpoint members",
       author_email='[email protected]',
       license='Apache License 2.0',

setup_pypi_test.py

@@ -37,7 +37,7 @@
                  Path(cwd, './common/src')]
 
 setup(name='pinpointPy',
-      version="1.3.6",  # don't forget update __version__ in pinpointPy/__init__.py
+      version="1.3.7",  # don't forget update __version__ in pinpointPy/__init__.py
       author="cd_pinpoint members",
       author_email='[email protected]',
       license='Apache License 2.0',

testapps/compose.yaml

@@ -224,9 +224,41 @@ services:
     ports:
       - 27017:27017
 
-  python-plugins:
+  py3.9-plugins:
     build:
-      dockerfile: testapps/python-plugins.dockerfile
+      dockerfile: testapps/python-plugins/py3.9-plugins.dockerfile
+      context: ../
+    profiles:
+      - testing
+    depends_on:
+      redis:
+        condition: service_healthy
+      httpbin:
+        condition: service_healthy
+      mongodb:
+        condition: service_healthy
+      postgres:
+        condition: service_healthy
+
+  py3.11-plugins:
+    build:
+      dockerfile: testapps/python-plugins/py3.11-plugins.dockerfile
+      context: ../
+    profiles:
+      - testing
+    depends_on:
+      redis:
+        condition: service_healthy
+      httpbin:
+        condition: service_healthy
+      mongodb:
+        condition: service_healthy
+      postgres:
+        condition: service_healthy
+
+  py-plugins:
+    build:
+      dockerfile: testapps/python-plugins/py-plugins.dockerfile
       context: ../
     profiles:
       - testing

testapps/python-plugins/py-plugins.dockerfile

@@ -0,0 +1,15 @@
+FROM python:latest
+
+COPY setup.py /pinpoint-c-agent/setup.py
+COPY common/ /pinpoint-c-agent/common
+COPY README /pinpoint-c-agent/README
+COPY plugins/PY /pinpoint-c-agent/plugins/PY
+COPY src/PY /pinpoint-c-agent/src/PY
+
+
+RUN pip install -r /pinpoint-c-agent/plugins/PY/requirements.txt
+RUN cd /pinpoint-c-agent && pip install -e .
+CMD ["python","-m","unittest","discover","-s","/pinpoint-c-agent/plugins/PY/pinpointPy/"]
+# CMD [ "python","-m","unittest","pinpointPy.libs._MysqlConnector.test_case.Test_Case" ]
+# CMD [ "sleep","infinity" ]
+# python -m unittest discover -s /pinpoint-c-agent/plugins/PY/pinpointPy/

testapps/python-plugins/py3.11-plugins.dockerfile

@@ -0,0 +1,15 @@
+FROM python:latest
+
+COPY setup.py /pinpoint-c-agent/setup.py
+COPY common/ /pinpoint-c-agent/common
+COPY README /pinpoint-c-agent/README
+COPY plugins/PY /pinpoint-c-agent/plugins/PY
+COPY src/PY /pinpoint-c-agent/src/PY
+
+
+RUN pip install -r /pinpoint-c-agent/plugins/PY/requirements.txt
+RUN cd /pinpoint-c-agent && pip install -e .
+CMD ["python","-m","unittest","discover","-s","/pinpoint-c-agent/plugins/PY/pinpointPy/"]
+# CMD [ "python","-m","unittest","pinpointPy.libs._MysqlConnector.test_case.Test_Case" ]
+# CMD [ "sleep","infinity" ]
+# python -m unittest discover -s /pinpoint-c-agent/plugins/PY/pinpointPy/

testapps/python-plugins.dockerfile …/python-plugins/py3.9-plugins.dockerfiletestapps/python-plugins.dockerfile renamed to testapps/python-plugins/py3.9-plugins.dockerfile testapps/python-plugins.dockerfile renamed to testapps/python-plugins/py3.9-plugins.dockerfile

@@ -1,4 +1,4 @@
-FROM python:3.13
+FROM python:3.9
 
 COPY setup.py /pinpoint-c-agent/setup.py
 COPY common/ /pinpoint-c-agent/common
@@ -11,5 +11,5 @@ RUN pip install -r /pinpoint-c-agent/plugins/PY/requirements.txt
 RUN cd /pinpoint-c-agent && pip install -e .
 CMD ["python","-m","unittest","discover","-s","/pinpoint-c-agent/plugins/PY/pinpointPy/"]
 # CMD [ "python","-m","unittest","pinpointPy.libs._MysqlConnector.test_case.Test_Case" ]
-# CMD [ "bash" ]
+# CMD [ "sleep","infinity" ]
 # python -m unittest discover -s /pinpoint-c-agent/plugins/PY/pinpointPy/