SQLAlchemy implicit unicode conversion

[Federico Ceratto]

During testing, SQLAlchemy warns about unicode conversions:

SAWarning: Unicode type received non-unicode bind param value '192.168.2.170'. (this warning may be suppressed after 10 occurrences)

Different databases and Python versions treat unicode in different ways and this could lead to bugs.

Those "managed_*" fields are Unicode and maybe should be Strings:
https://github.com/openstack/designate/blob/master/designate/storage/impl_sqlalchemy/tables.py#L183

[OpenStack Infra (hudson-openstack)]

Related fix proposed to branch: master
Review: https://review.openstack.org/266353

[OpenStack Infra (hudson-openstack)]

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

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/276448
Committed: https://git.openstack.org/cgit/openstack/designate/commit/?id=a999159607908ac9a82fdab979eaa52f289586b7
Submitter: Jenkins
Branch: master

commit a999159607908ac9a82fdab979eaa52f289586b7
Author: Federico Ceratto <email address hidden>
Date: Thu Feb 4 21:02:19 2016 +0000

    Switch "managed" fields from unicode to string

    Change-Id: I8c6e921dab3084d4006567b98da3d6fa85e96574
    Closes-Bug: 1533193

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/266353
Committed: https://git.openstack.org/cgit/openstack/designate/commit/?id=b442fe709bd7ec258104f5a064c5330e52b55563
Submitter: Jenkins
Branch: master

commit b442fe709bd7ec258104f5a064c5330e52b55563
Author: Federico Ceratto <email address hidden>
Date: Tue Jan 12 12:59:04 2016 +0000

    Minor unicode test fixes

    Change-Id: I9d4eb3a4e0985960fa06bf17fe3454d61c34a5a4
    Related-Bug: #1533193

[Thierry Carrez (ttx)]

This issue was fixed in the openstack/designate 2.0.0.0b3 development milestone.