craton

Ensure utf8 as db charset during db init

Bug #1606859 reported by Sulochan Acharya
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
craton
In Progress
Medium
Syed Ahsan Shamim Zaidi

Bug Description

We should ensure that utf8 is the db default charset.

Sulochan Acharya (sulochan-acharya)
Changed in craton:
status: New → Triaged
importance: Undecided → Medium
Revision history for this message
Syed Ahsan Shamim Zaidi (ahsanmohsin04) wrote :

Hi Sulo,

Can you kindly provide more description about this bug ?

Changed in craton:
assignee: nobody → Syed Ahsan Shamim Zaidi (ahsanmohsin04)
Revision history for this message
Jim Baker (jimbaker) wrote :

See http://docs.sqlalchemy.org/en/latest/dialects/mysql.html#unicode

I believe we are doing this for MySQL by virtue of our docker_run.sh (or equivalent admn setup that follows that docker approach); and that SQLAlchemy defaults to 'utf8' encoding.

But arguably we should be using the MySQL-specific encoding 'utf8mb4', which allows for the full Unicode standard.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to craton (master)

Fix proposed to branch: master
Review: https://review.openstack.org/351915

Changed in craton:
status: Triaged → In Progress
Revision history for this message
Syed Ahsan Shamim Zaidi (ahsanmohsin04) wrote :

Thank you Jim for your comments, I have made the relevant changes, kindly review the patch set.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on craton (master)

Change abandoned by Syed Ahsan Shamim Zaidi (<email address hidden>) on branch: master
Review: https://review.openstack.org/351915
Reason: Will be uploading new patch set

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to craton (master)

Fix proposed to branch: master
Review: https://review.openstack.org/353634

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on craton (master)

Change abandoned by Syed Ahsan Shamim Zaidi (<email address hidden>) on branch: master
Review: https://review.openstack.org/353634

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to craton (master)

Fix proposed to branch: master
Review: https://review.openstack.org/355657

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on craton (master)

Change abandoned by Syed Ahsan Shamim Zaidi (<email address hidden>) on branch: master
Review: https://review.openstack.org/355657

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to craton (master)

Fix proposed to branch: master
Review: https://review.openstack.org/364561

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on craton (master)

Change abandoned by Jim Baker (<email address hidden>) on branch: master
Review: https://review.openstack.org/364561
Reason: Revisit in the future

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.