)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"0f25652ddc5e5834ab3b8bb6710ba78fe7078839","unresolved":false,"context_lines":[{"line_number":5,"context_line":"CommitDate: 2017-06-28 08:00:28 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Use six to correctly handle text in Python 2 and 3 in the Role schema"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Change-Id: I23a7dab693ff39377ee1e012aec106bc5b7aed0e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"5f201791_6006c054","line":8,"updated":"2017-06-28 08:07:30.000000000","message":"bug would be ideal but even just another line to say this fixes the py27 tests if you get another pass please thanks","commit_id":"92edeb9f6d3a07c70d56f59d51760c5d4c804e99"}],"tripleo_common/utils/roles.py":[{"author":{"_account_id":6928,"name":"Ben Nemec","email":"openstack@nemebean.com","username":"bnemec"},"change_message_id":"cdd540ad8cada4f82d04f1b4383f6ba0d67572bb","unresolved":false,"context_lines":[{"line_number":100,"context_line":"        raise RoleMetadataError(\u0027Unable to parse role yaml\u0027)"},{"line_number":101,"context_line":""},{"line_number":102,"context_line":"    schema \u003d {"},{"line_number":103,"context_line":"        \u0027name\u0027: {\u0027type\u0027: six.text_type},"},{"line_number":104,"context_line":"        \u0027CountDefault\u0027: {\u0027type\u0027: int},"},{"line_number":105,"context_line":"        \u0027HostnameFormatDefault\u0027: {\u0027type\u0027: six.text_type},"},{"line_number":106,"context_line":"        \u0027disable_constraints\u0027: {\u0027type\u0027: bool},"}],"source_content_type":"text/x-python","patch_set":1,"id":"5f201791_529af06d","line":103,"updated":"2017-06-23 14:28:25.000000000","message":"Note that this is a change of type.  In Python 2, six.text_type is \"unicode\".  These types should probably be unicode anyway since they\u0027re arbitrary strings passed in by the user that could contain unicode characters, unless we\u0027re passing this to something that can\u0027t handle unicode.","commit_id":"837adcb4ad9d0eda57b6975999dc5fc8613ab19d"},{"author":{"_account_id":9712,"name":"Dougal Matthews","email":"dougal.matthews@canonical.com","username":"dougal"},"change_message_id":"2b8979fc8b2a26467fe90db4a77f49761bdc00b5","unresolved":false,"context_lines":[{"line_number":100,"context_line":"        raise RoleMetadataError(\u0027Unable to parse role yaml\u0027)"},{"line_number":101,"context_line":""},{"line_number":102,"context_line":"    schema \u003d {"},{"line_number":103,"context_line":"        \u0027name\u0027: {\u0027type\u0027: six.text_type},"},{"line_number":104,"context_line":"        \u0027CountDefault\u0027: {\u0027type\u0027: int},"},{"line_number":105,"context_line":"        \u0027HostnameFormatDefault\u0027: {\u0027type\u0027: six.text_type},"},{"line_number":106,"context_line":"        \u0027disable_constraints\u0027: {\u0027type\u0027: bool},"}],"source_content_type":"text/x-python","patch_set":1,"id":"5f201791_6d75e731","line":103,"in_reply_to":"5f201791_529af06d","updated":"2017-06-26 10:05:48.000000000","message":"Good point.","commit_id":"837adcb4ad9d0eda57b6975999dc5fc8613ab19d"},{"author":{"_account_id":15895,"name":"Adriano Petrich","email":"apetrich@redhat.com","username":"apetrich"},"change_message_id":"6447c51f1938535ed4d053c8989c003d9e052e9d","unresolved":false,"context_lines":[{"line_number":100,"context_line":"        raise RoleMetadataError(\u0027Unable to parse role yaml\u0027)"},{"line_number":101,"context_line":""},{"line_number":102,"context_line":"    schema \u003d {"},{"line_number":103,"context_line":"        \u0027name\u0027: {\u0027type\u0027: six.text_type},"},{"line_number":104,"context_line":"        \u0027CountDefault\u0027: {\u0027type\u0027: int},"},{"line_number":105,"context_line":"        \u0027HostnameFormatDefault\u0027: {\u0027type\u0027: six.text_type},"},{"line_number":106,"context_line":"        \u0027disable_constraints\u0027: {\u0027type\u0027: bool},"}],"source_content_type":"text/x-python","patch_set":1,"id":"5f201791_054efe36","line":103,"in_reply_to":"5f201791_6d75e731","updated":"2017-06-28 07:15:06.000000000","message":"Could we use six.string_types that will default to basestring on 2 and str on 3 and should make it work?\n\nUsing string_types works for me locally.","commit_id":"837adcb4ad9d0eda57b6975999dc5fc8613ab19d"}]}
