cloudstack-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "ASF GitHub Bot (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (CLOUDSTACK-8818) Python scripts should depend on mysql.connector instead of MySQLdb
Date Tue, 10 Nov 2015 08:11:10 GMT

    [ https://issues.apache.org/jira/browse/CLOUDSTACK-8818?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14998227#comment-14998227
] 

ASF GitHub Bot commented on CLOUDSTACK-8818:
--------------------------------------------

Github user wido commented on the pull request:

    https://github.com/apache/cloudstack/pull/1054#issuecomment-155353598
  
    Pylint result:
    
    ```
    Global evaluation
    -----------------
    Your code has been rated at 6.48/10 (previous run: 6.48/10, +0.00)
    ```
    
    I also wrote a small test script:
    
    ```
    from db import Database
    
    db = Database(username='root', password='XXXX')
    
    try:
        db.testConnection()
        print("Succes!")
    except:
        raise
    ```
    
    The result:
    
    ```
    wido@wido-laptop:~/repos/cloudstack/python/lib/cloudutils$ python smallthin.py 
    Succes!
    wido@wido-laptop:~/repos/cloudstack/python/lib/cloudutils$
    ```


> Python scripts should depend on mysql.connector instead of MySQLdb
> ------------------------------------------------------------------
>
>                 Key: CLOUDSTACK-8818
>                 URL: https://issues.apache.org/jira/browse/CLOUDSTACK-8818
>             Project: CloudStack
>          Issue Type: Improvement
>      Security Level: Public(Anyone can view this level - this is the default.) 
>            Reporter: Wido den Hollander
>              Labels: mysql, python, python3
>
> Our current Python scripts depend on MySQLdb for MySQL connections.
> The best way to go is the native mysql.connector which implements the MySQL protocol
in native Python instead of depending on external libraries.
> It would be best if we drop MySQLdb and use mysql.connector since that also supports
Python 3.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Mime
View raw message