)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":9107,"name":"Victor Stinner","email":"vstinner@redhat.com","username":"haypo"},"change_message_id":"53be9b7fbbf6a1f4cc41ccfc6c434f46c0cad868","unresolved":false,"context_lines":[{"line_number":22,"context_line":"behavior is retained."},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"Change-Id: I5c211092d282e724d1c87ce4d06b6c44b592e764"},{"line_number":25,"context_line":"Depends-On: Id33c9f8c17102ba1fe24c12b053c336b6d265501"},{"line_number":26,"context_line":"Closes-bug: #1518430"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"da6895a0_a4c7fe10","line":25,"updated":"2016-10-24 17:45:40.000000000","message":"I\u0027m not sure that a Depends-On is enough, we need a release of oslo.utils and require the new version in Oslo Messaging requirements.txt, no?","commit_id":"a4ab606c56bff7529c0850bbde4d41ade14199b5"}],"oslo_messaging/_drivers/impl_rabbit.py":[{"author":{"_account_id":2813,"name":"Mehdi Abaakouk","email":"sileht@sileht.net","username":"sileht"},"change_message_id":"540b3ff5ae09acbb559495dc2edd574838630705","unresolved":false,"context_lines":[{"line_number":26,"context_line":"import time"},{"line_number":27,"context_line":"import uuid"},{"line_number":28,"context_line":""},{"line_number":29,"context_line":"from eventlet import event"},{"line_number":30,"context_line":"from eventlet import timeout"},{"line_number":31,"context_line":"import kombu"},{"line_number":32,"context_line":"import kombu.connection"}],"source_content_type":"text/x-python","patch_set":1,"id":"fa6399be_6ccabf1c","line":29,"updated":"2016-10-14 15:11:43.000000000","message":"Can you use importutils.try_import(), we try to remove eventlet dep in oslo.messaging, this is almost done.","commit_id":"d21cdf71a00efed91f906a95e6a4509fdd85ac25"},{"author":{"_account_id":20864,"name":"Gevorg Davoian","email":"gdavoian@gmail.com","username":"gdavoian"},"change_message_id":"a5c996652d1aed9c1e5845d9e9fc3e5b2410255e","unresolved":false,"context_lines":[{"line_number":50,"context_line":"from oslo_messaging import _utils"},{"line_number":51,"context_line":"from oslo_messaging import exceptions"},{"line_number":52,"context_line":""},{"line_number":53,"context_line":"event \u003d importutils.try_import(\u0027eventlet.event\u0027)"},{"line_number":54,"context_line":"timeout \u003d importutils.try_import(\u0027eventlet.timeout\u0027)"},{"line_number":55,"context_line":""},{"line_number":56,"context_line":"# NOTE(sileht): don\u0027t exists in py2 socket module"}],"source_content_type":"text/x-python","patch_set":3,"id":"fa6399be_72637542","line":53,"range":{"start_line":53,"start_character":0,"end_line":53,"end_character":5},"updated":"2016-10-14 17:05:52.000000000","message":"Just a small remark: I think that eventlet_event and eventlet_timeout are better names for imported modules, since event and timeout are quite common names and may be used as variable names in many places.","commit_id":"490d418c3ddb982b337c727330e321b49ba20019"},{"author":{"_account_id":9257,"name":"John Eckersberg","email":"jeckersb@redhat.com","username":"jeckersb"},"change_message_id":"986bff988e8d6e40016147d60dbd23554ee2c6c3","unresolved":false,"context_lines":[{"line_number":50,"context_line":"from oslo_messaging import _utils"},{"line_number":51,"context_line":"from oslo_messaging import exceptions"},{"line_number":52,"context_line":""},{"line_number":53,"context_line":"event \u003d importutils.try_import(\u0027eventlet.event\u0027)"},{"line_number":54,"context_line":"timeout \u003d importutils.try_import(\u0027eventlet.timeout\u0027)"},{"line_number":55,"context_line":""},{"line_number":56,"context_line":"# NOTE(sileht): don\u0027t exists in py2 socket module"}],"source_content_type":"text/x-python","patch_set":3,"id":"fa6399be_4d76d23e","line":53,"in_reply_to":"fa6399be_72637542","updated":"2016-10-14 17:28:46.000000000","message":"Done","commit_id":"490d418c3ddb982b337c727330e321b49ba20019"}]}