)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":9061,"name":"David Moreau Simard","email":"moi@dmsimard.com","username":"dmsimard"},"change_message_id":"700cec76659a8a779ee137fdb3b3a708d1b0a2cf","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"When Glance is set up to use https, we saw OpenSSL.SSL.WantReadError"},{"line_number":10,"context_line":"errors in image operations (see"},{"line_number":11,"context_line":"https://ci.centos.org/view/rdo/view/promotion-pipeline/job/weirdo-master-promote-puppet-openstack-scenario003/73/consoleFull"},{"line_number":12,"context_line":"for an example)."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"This error was fixed by glanceclient in"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"1a122d0e_16acf686","line":11,"updated":"2016-05-05 14:03:56.000000000","message":"I think the description of the issue should go in a bug (which would include a link to the issues on a logs.openstack.org link, i.e, http://logs.openstack.org/86/305886/29/check/gate-puppet-openstack-integration-3-scenario003-tempest-centos-7/616bb73/console.html#_2016-05-04_23_34_14_652 ) and then refer to that bug in the commit message (closes-bug)","commit_id":"5e772b6a69269827c935bea7f4b9f466131eb13c"}],"tempest/common/glance_http.py":[{"author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"change_message_id":"f9e4c75b78b42dc2b27e5a6f0ee88f65ae5ea1f8","unresolved":false,"context_lines":[{"line_number":29,"context_line":"from six.moves import http_client as httplib"},{"line_number":30,"context_line":"from six.moves.urllib import parse as urlparse"},{"line_number":31,"context_line":""},{"line_number":32,"context_line":"try:"},{"line_number":33,"context_line":"    from eventlet import patcher"},{"line_number":34,"context_line":"    # Handle case where we are running in a monkey patched environment"},{"line_number":35,"context_line":"    if patcher.is_monkey_patched(\u0027socket\u0027):"},{"line_number":36,"context_line":"        from eventlet.green.httplib import HTTPSConnection"},{"line_number":37,"context_line":"        from eventlet.green.OpenSSL.SSL import GreenConnection as Connection"},{"line_number":38,"context_line":"    else:"},{"line_number":39,"context_line":"        raise ImportError"},{"line_number":40,"context_line":"except ImportError:"},{"line_number":41,"context_line":"    from OpenSSL import SSL"},{"line_number":42,"context_line":"    from six.moves import http_client"}],"source_content_type":"text/x-python","patch_set":2,"id":"1a122d0e_8603b30c","line":39,"range":{"start_line":32,"start_character":0,"end_line":39,"end_character":25},"updated":"2016-05-05 15:08:48.000000000","message":"We\u0027re not going to optionally require eventlet in tempest. This doesn\u0027t make sense in this context","commit_id":"2bf553db1f2eca7e90c14f7bac22063a320d568d"},{"author":{"_account_id":13294,"name":"Javier Peña","email":"jpena@redhat.com","username":"jpena"},"change_message_id":"36ff1ca1e099e04c7bb491c60ef46a385d145f31","unresolved":false,"context_lines":[{"line_number":29,"context_line":"from six.moves import http_client as httplib"},{"line_number":30,"context_line":"from six.moves.urllib import parse as urlparse"},{"line_number":31,"context_line":""},{"line_number":32,"context_line":"try:"},{"line_number":33,"context_line":"    from eventlet import patcher"},{"line_number":34,"context_line":"    # Handle case where we are running in a monkey patched environment"},{"line_number":35,"context_line":"    if patcher.is_monkey_patched(\u0027socket\u0027):"},{"line_number":36,"context_line":"        from eventlet.green.httplib import HTTPSConnection"},{"line_number":37,"context_line":"        from eventlet.green.OpenSSL.SSL import GreenConnection as Connection"},{"line_number":38,"context_line":"    else:"},{"line_number":39,"context_line":"        raise ImportError"},{"line_number":40,"context_line":"except ImportError:"},{"line_number":41,"context_line":"    from OpenSSL import SSL"},{"line_number":42,"context_line":"    from six.moves import http_client"}],"source_content_type":"text/x-python","patch_set":2,"id":"1a122d0e_b5788ff9","line":39,"range":{"start_line":32,"start_character":0,"end_line":39,"end_character":25},"in_reply_to":"1a122d0e_8603b30c","updated":"2016-05-05 16:50:04.000000000","message":"Thanks for the comment. I\u0027m not very familiar with the glanceclient internals, do you know of a cleaner way to fix this?\n\nWhat I see in the glanceclient code is that they optionally require eventlet, too. That\u0027s why I tried to port the code here.","commit_id":"2bf553db1f2eca7e90c14f7bac22063a320d568d"}]}