)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":13997,"name":"SHIGEMATSU Mitsuhiro","email":"shigematsu.mitsuhiro@lab.ntt.co.jp","username":"pshige"},"change_message_id":"529dbe453112161bf952e1216a46707c32dfca87","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Instead of directly using the os module in the agent module"},{"line_number":10,"context_line":"have the utility module contain a connection class (that is"},{"line_number":11,"context_line":"very similar to the multiprocess.connnection.Connnection"},{"line_number":12,"context_line":"class) that wraps the operations used; allowing them to be"},{"line_number":13,"context_line":"abstracted away from the underlying os operations."},{"line_number":14,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"da20952f_4e442573","line":11,"updated":"2015-09-03 14:00:10.000000000","message":"s/multiprocess.connnection.Connnection/multiprocess.connection.Connection/","commit_id":"e0ae5abe60760cd69496d70b6ab36affe9171a6a"},{"author":{"_account_id":10342,"name":"Jay Faulkner","display_name":"JayF","email":"jay@jvf.cc","username":"JayF","status":"youtube.com/@oss-gr / podcast.gr-oss.io"},"change_message_id":"e2190c6d4d91b6c01f9399149cb078b979e5fcef","unresolved":false,"context_lines":[{"line_number":12,"context_line":"class) that wraps the operations used; allowing them to be"},{"line_number":13,"context_line":"abstracted away from the underlying os operations."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Change-Id: I3f8a4bd71d69402a0326fa352bb3a775913446fc"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"da20952f_3e590836","line":15,"updated":"2015-09-09 18:25:15.000000000","message":"Given the nature of this change, I think a bug describing the problem we\u0027re fixing would be good.","commit_id":"bd4be5107204cd83bd25d8225689891130c2e96b"}],"ironic_python_agent/agent.py":[{"author":{"_account_id":10342,"name":"Jay Faulkner","display_name":"JayF","email":"jay@jvf.cc","username":"JayF","status":"youtube.com/@oss-gr / podcast.gr-oss.io"},"change_message_id":"e2190c6d4d91b6c01f9399149cb078b979e5fcef","unresolved":false,"context_lines":[{"line_number":18,"context_line":"import time"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"try:"},{"line_number":21,"context_line":"    from contextlib import ExitStack  # noqa"},{"line_number":22,"context_line":"except ImportError:"},{"line_number":23,"context_line":"    from contextlib2 import ExitStack  # noqa"},{"line_number":24,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"da20952f_9eb8542c","line":21,"updated":"2015-09-09 18:25:15.000000000","message":"I\u0027m confused as to why this is needed? Both py2.7 + py3 have contextlib, without the 2, on my system. \n\nIt it belongs here, a comment as to why we\u0027re doing a try:except would be nice.","commit_id":"bd4be5107204cd83bd25d8225689891130c2e96b"},{"author":{"_account_id":1297,"name":"Joshua Harlow","email":"jxharlow@godaddy.com","username":"harlowja"},"change_message_id":"57aee7067e4cf755d1af43b9fbf3bfb303b8e5e9","unresolved":false,"context_lines":[{"line_number":18,"context_line":"import time"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"try:"},{"line_number":21,"context_line":"    from contextlib import ExitStack  # noqa"},{"line_number":22,"context_line":"except ImportError:"},{"line_number":23,"context_line":"    from contextlib2 import ExitStack  # noqa"},{"line_number":24,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"da20952f_c4103203","line":21,"in_reply_to":"da20952f_9eb8542c","updated":"2015-09-10 00:25:37.000000000","message":"Python 2.7 doesn\u0027t have this class.\n\n    Python 2.7.10 (default, Jun 18 2015, 15:58:36) \n    [GCC 4.1.2 20080704 (Red Hat 4.1.2-46)] on linux2\n    Type \"help\", \"copyright\", \"credits\" or \"license\" for more information.\n    \u003e\u003e\u003e from contextlib import ExitStack\n    Traceback (most recent call last):\n      File \"\u003cstdin\u003e\", line 1, in \u003cmodule\u003e\n    ImportError: cannot import name ExitStack\n\nSo contextlib2 provides it for 2.7 (and 2.6).","commit_id":"bd4be5107204cd83bd25d8225689891130c2e96b"}]}