)]}'
{"ironic_python_agent/agent.py":[{"author":{"_account_id":7680,"name":"Alex Gaynor","email":"alex.gaynor@gmail.com","username":"alex_gaynor"},"change_message_id":"7a888978084147873f4b9dbca14c8823933f14f1","unresolved":false,"context_lines":[{"line_number":192,"context_line":"                result \u003d base.SyncCommandResult(command_name,"},{"line_number":193,"context_line":"                                                kwargs,"},{"line_number":194,"context_line":"                                                False,"},{"line_number":195,"context_line":"                                                e)"},{"line_number":196,"context_line":""},{"line_number":197,"context_line":"            self.command_results[result.id] \u003d result"},{"line_number":198,"context_line":"            return result"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAWH%2F9ZH0%3D","line":195,"updated":"2014-04-04 20:38:26.000000000","message":"I\u0027m not sure this is right, seems like you might want to use `six.text_type(e)` here?","commit_id":"8ace3332862135fde1d59fdaed17d0dd8fc8f30f"}],"ironic_python_agent/base.py":[{"author":{"_account_id":7680,"name":"Alex Gaynor","email":"alex.gaynor@gmail.com","username":"alex_gaynor"},"change_message_id":"7a888978084147873f4b9dbca14c8823933f14f1","unresolved":false,"context_lines":[{"line_number":31,"context_line":""},{"line_number":32,"context_line":"class BaseCommandResult(encoding.Serializable):"},{"line_number":33,"context_line":"    def __init__(self, command_name, command_params):"},{"line_number":34,"context_line":"        self.id \u003d str(uuid.uuid4())"},{"line_number":35,"context_line":"        self.command_name \u003d command_name"},{"line_number":36,"context_line":"        self.command_params \u003d command_params"},{"line_number":37,"context_line":"        self.command_status \u003d AgentCommandStatus.RUNNING"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAWH%2F9ZHg%3D","line":34,"updated":"2014-04-04 20:38:26.000000000","message":"Similarly, you might want `six.text_type` instead of `str`?","commit_id":"8ace3332862135fde1d59fdaed17d0dd8fc8f30f"}],"ironic_python_agent/hardware.py":[{"author":{"_account_id":7680,"name":"Alex Gaynor","email":"alex.gaynor@gmail.com","username":"alex_gaynor"},"change_message_id":"7a888978084147873f4b9dbca14c8823933f14f1","unresolved":false,"context_lines":[{"line_number":170,"context_line":""},{"line_number":171,"context_line":"        # There will always be at least one extension available (the"},{"line_number":172,"context_line":"        # GenericHardwareManager)."},{"line_number":173,"context_line":"        try:"},{"line_number":174,"context_line":"            preferred_extension \u003d sorted(extension_manager, _compare_extensions)[0]"},{"line_number":175,"context_line":"        except TypeError:"},{"line_number":176,"context_line":"            preferred_extension \u003d sorted("}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAWH%2F9ZGQ%3D","line":173,"updated":"2014-04-04 20:38:26.000000000","message":"I think an explicit version check is better than a try/except here.","commit_id":"8ace3332862135fde1d59fdaed17d0dd8fc8f30f"}],"ironic_python_agent/tests/agent.py":[{"author":{"_account_id":7680,"name":"Alex Gaynor","email":"alex.gaynor@gmail.com","username":"alex_gaynor"},"change_message_id":"7a888978084147873f4b9dbca14c8823933f14f1","unresolved":false,"context_lines":[{"line_number":33,"context_line":""},{"line_number":34,"context_line":"EXPECTED_ERROR \u003d RuntimeError(\u0027command execution failed\u0027)"},{"line_number":35,"context_line":""},{"line_number":36,"context_line":"if sys.version_info[:2][0] \u003d\u003d 2:"},{"line_number":37,"context_line":"    OPEN_FUNCTION_NAME \u003d \u0027__builtin__.open\u0027"},{"line_number":38,"context_line":"else:"},{"line_number":39,"context_line":"    OPEN_FUNCTION_NAME \u003d \u0027builtins.open\u0027"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAWH%2F9ZF8%3D","line":36,"updated":"2014-04-04 20:38:26.000000000","message":"`six.PY2` here.","commit_id":"8ace3332862135fde1d59fdaed17d0dd8fc8f30f"}],"ironic_python_agent/tests/api.py":[{"author":{"_account_id":7680,"name":"Alex Gaynor","email":"alex.gaynor@gmail.com","username":"alex_gaynor"},"change_message_id":"7a888978084147873f4b9dbca14c8823933f14f1","unresolved":false,"context_lines":[{"line_number":164,"context_line":"        response \u003d self.get_json(\u0027/v1\u0027, path_prefix\u003d\u0027\u0027)"},{"line_number":165,"context_line":"        data \u003d response.json"},{"line_number":166,"context_line":"        self.assertTrue(\u0027status\u0027 in list(data.keys()))"},{"line_number":167,"context_line":"        self.assertTrue(\u0027commands\u0027 in list(data.keys()))"},{"line_number":168,"context_line":""},{"line_number":169,"context_line":"    def test_get_agent_status(self):"},{"line_number":170,"context_line":"        status \u003d agent.IronicPythonAgentStatus(time.time(),"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAWH%2F9ZFc%3D","line":167,"updated":"2014-04-04 20:38:26.000000000","message":"The `list` here aren\u0027t needed.","commit_id":"8ace3332862135fde1d59fdaed17d0dd8fc8f30f"}],"ironic_python_agent/tests/configdrive.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":"70851a4f59c3fdf50ff05c4012ace70d796023ac","unresolved":false,"context_lines":[{"line_number":14,"context_line":"limitations under the License."},{"line_number":15,"context_line":"\"\"\""},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"from __future__ import unicode_literals"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"import base64"},{"line_number":20,"context_line":"import json"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAWH%2F9ZE4%3D","side":"PARENT","line":17,"updated":"2014-04-04 20:38:23.000000000","message":"unintentionally removed, will fix in next patchset","commit_id":"1a87ebfcce7be6e10bed23b64f87a0110e76a22b"},{"author":{"_account_id":7680,"name":"Alex Gaynor","email":"alex.gaynor@gmail.com","username":"alex_gaynor"},"change_message_id":"7a888978084147873f4b9dbca14c8823933f14f1","unresolved":false,"context_lines":[{"line_number":25,"context_line":"from ironic_python_agent import configdrive"},{"line_number":26,"context_line":"from ironic_python_agent import utils"},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"if sys.version_info[:2][0] \u003d\u003d 2:"},{"line_number":29,"context_line":"    OPEN_FUNCTION_NAME \u003d \u0027__builtin__.open\u0027"},{"line_number":30,"context_line":"else:"},{"line_number":31,"context_line":"    OPEN_FUNCTION_NAME \u003d \u0027builtins.open\u0027"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAWH%2F9ZFA%3D","line":28,"updated":"2014-04-04 20:38:26.000000000","message":"same","commit_id":"8ace3332862135fde1d59fdaed17d0dd8fc8f30f"}],"requirements.txt":[{"author":{"_account_id":10343,"name":"Jim Rollenhagen","email":"jim@jimrollenhagen.com","username":"jimrollenhagen"},"change_message_id":"f68d333215a65f540f9692e9597b584cc1e23b71","unresolved":false,"context_lines":[{"line_number":5,"context_line":"WSME\u003e\u003d0.6"},{"line_number":6,"context_line":"six\u003e\u003d1.5.2"},{"line_number":7,"context_line":"#eventlet\u003e\u003d0.13.0"},{"line_number":8,"context_line":"-e git+https://github.com/jstasiak/eventlet@301c71210a1e91a146010a8e42637b9be47e1940#egg\u003deventlet"},{"line_number":9,"context_line":"oslo.config\u003e\u003d1.2.0"},{"line_number":10,"context_line":"Babel\u003e\u003d1.3"},{"line_number":11,"context_line":"iso8601\u003e\u003d0.1.9"}],"source_content_type":"text/plain","patch_set":5,"id":"AAAAWH%2F9Pwc%3D","line":8,"updated":"2014-04-07 17:22:49.000000000","message":"Looks like this snuck in here with the latest patchset, I don\u0027t think it should be there.","commit_id":"c98665583a3c7e1c87a92f3af6283f55df48d859"}],"tox.ini":[{"author":{"_account_id":10343,"name":"Jim Rollenhagen","email":"jim@jimrollenhagen.com","username":"jimrollenhagen"},"change_message_id":"47a426061e32653733658b46849aeccc14e9e729","unresolved":false,"context_lines":[{"line_number":1,"context_line":"[tox]"},{"line_number":2,"context_line":"minversion \u003d 1.6"},{"line_number":3,"context_line":"skipsdist \u003d True"},{"line_number":4,"context_line":"envlist \u003d py26,py27,py33,pypy,pep8"},{"line_number":5,"context_line":""},{"line_number":6,"context_line":"[testenv]"},{"line_number":7,"context_line":"usedevelop \u003d True"}],"source_content_type":"text/x-properties","patch_set":4,"id":"AAAAWH%2F9QY8%3D","line":4,"updated":"2014-04-07 16:02:16.000000000","message":"I don\u0027t think py33 should be added to envlist until tests pass there. `tox -epy33` can still be run manually.","commit_id":"1968a08243641a9c57407a439e5198afab467607"}]}