)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":15435,"name":"Al Bailey","email":"albailey1974@gmail.com","username":"albailey"},"change_message_id":"57024c6fded074f59223a3d89794d240c60e88e6","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Create test framework for python with stestr."},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Add add the first unit test case for commont/timeutils."},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"Story: 2007082"},{"line_number":12,"context_line":"Task: 38077"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"3fa7e38b_04ddab3e","line":9,"range":{"start_line":9,"start_character":37,"end_line":9,"end_character":44},"updated":"2020-01-13 13:48:11.000000000","message":"typo","commit_id":"9db68b0ec808777c184840d1588935c4256d6920"},{"author":{"_account_id":28470,"name":"Yan Chen","email":"yan.chen@intel.com","username":"chenyan"},"change_message_id":"ea4bc1addc6097ed78d9ae152115483f6e8a4904","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Create test framework for python with stestr."},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Add add the first unit test case for commont/timeutils."},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"Story: 2007082"},{"line_number":12,"context_line":"Task: 38077"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"3fa7e38b_0a212dfe","line":9,"range":{"start_line":9,"start_character":37,"end_line":9,"end_character":44},"in_reply_to":"3fa7e38b_04ddab3e","updated":"2020-01-14 02:45:42.000000000","message":"Done","commit_id":"9db68b0ec808777c184840d1588935c4256d6920"},{"author":{"_account_id":14174,"name":"Bart Wensley","email":"bartwensley@hotmail.com","username":"bartwensley"},"change_message_id":"c7b77cebfcb2e3da07cda90c112088f43fc01dc7","unresolved":false,"context_lines":[{"line_number":9,"context_line":"Add add the first unit test for fm-rest-api/fm/fm/common/timeutils."},{"line_number":10,"context_line":"Add tox task for py27/35 and add py27 test as a zuul checking job."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Will add py27/35 as zuul gating jobs once fixed all issues found"},{"line_number":13,"context_line":"in code."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Story: 2007082"},{"line_number":16,"context_line":"Task: 38077"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":8,"id":"3fa7e38b_870d0be9","line":13,"range":{"start_line":12,"start_character":0,"end_line":13,"end_character":8},"updated":"2020-01-14 13:43:34.000000000","message":"The py27 job is passing, so why not add it as a gating job now?","commit_id":"63c34e4aa6749f59e130bdd8a8738062316b909b"},{"author":{"_account_id":15435,"name":"Al Bailey","email":"albailey1974@gmail.com","username":"albailey"},"change_message_id":"a8b0cfc805fab39b8ab3339d8d8868cc027fb7c2","unresolved":false,"context_lines":[{"line_number":9,"context_line":"Add add the first unit test for fm-rest-api/fm/fm/common/timeutils."},{"line_number":10,"context_line":"Add tox task for py27/35 and add py27 test as a zuul checking job."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Will add py27/35 as zuul gating jobs once fixed all issues found"},{"line_number":13,"context_line":"in code."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Story: 2007082"},{"line_number":16,"context_line":"Task: 38077"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":8,"id":"3fa7e38b_e28b8dbe","line":13,"range":{"start_line":12,"start_character":0,"end_line":13,"end_character":8},"in_reply_to":"3fa7e38b_870d0be9","updated":"2020-01-14 14:38:51.000000000","message":"I ran py27 locally and its passing, so I would suggest adding it to both the check and gate sections in the .zuul.yaml\n\nYou can get py35 to pass by making the following change in the timeutils.py file\n\nchange this line (it appears twice)\n       raise ValueError(e.message)\n\nwith this:\n       raise ValueError(str(e))\n\nThis is how we fixed that python3 issue in other subsystems","commit_id":"63c34e4aa6749f59e130bdd8a8738062316b909b"}],"fm-rest-api/fm/fm/common/timeutils.py":[{"author":{"_account_id":14174,"name":"Bart Wensley","email":"bartwensley@hotmail.com","username":"bartwensley"},"change_message_id":"c698f27045fb0ef5dfcc6684c1e34a1def61913c","unresolved":false,"context_lines":[{"line_number":46,"context_line":"    try:"},{"line_number":47,"context_line":"        return iso8601.parse_date(timestr)"},{"line_number":48,"context_line":"    except iso8601.ParseError as e:"},{"line_number":49,"context_line":"        raise ValueError(e)"},{"line_number":50,"context_line":"    except TypeError as e:"},{"line_number":51,"context_line":"        raise ValueError(e)"},{"line_number":52,"context_line":""}],"source_content_type":"text/x-python","patch_set":9,"id":"3fa7e38b_14404e06","line":49,"range":{"start_line":49,"start_character":25,"end_line":49,"end_character":26},"updated":"2020-01-15 12:49:03.000000000","message":"I believe this needs to be a string, so \"str(e)\".","commit_id":"81b99f87959be7ddcda66fc91d16b35e84758c11"},{"author":{"_account_id":28470,"name":"Yan Chen","email":"yan.chen@intel.com","username":"chenyan"},"change_message_id":"e0973806d25af36c6ab35c5ac949e8ad47290c61","unresolved":false,"context_lines":[{"line_number":46,"context_line":"    try:"},{"line_number":47,"context_line":"        return iso8601.parse_date(timestr)"},{"line_number":48,"context_line":"    except iso8601.ParseError as e:"},{"line_number":49,"context_line":"        raise ValueError(e)"},{"line_number":50,"context_line":"    except TypeError as e:"},{"line_number":51,"context_line":"        raise ValueError(e)"},{"line_number":52,"context_line":""}],"source_content_type":"text/x-python","patch_set":9,"id":"3fa7e38b_f211f851","line":49,"range":{"start_line":49,"start_character":25,"end_line":49,"end_character":26},"in_reply_to":"3fa7e38b_14404e06","updated":"2020-01-16 00:03:11.000000000","message":"Done","commit_id":"81b99f87959be7ddcda66fc91d16b35e84758c11"}],"fm-rest-api/fm/tests/base.py":[{"author":{"_account_id":14174,"name":"Bart Wensley","email":"bartwensley@hotmail.com","username":"bartwensley"},"change_message_id":"8a9c3ee94023cc61f50b369efc0410785d65b462","unresolved":false,"context_lines":[{"line_number":1,"context_line":"# vim: tabstop\u003d4 shiftwidth\u003d4 softtabstop\u003d4"},{"line_number":2,"context_line":""},{"line_number":3,"context_line":"# Copyright 2010 United States Government as represented by the"},{"line_number":4,"context_line":"# Administrator of the National Aeronautics and Space Administration."}],"source_content_type":"text/x-python","patch_set":3,"id":"3fa7e38b_c44a3323","line":1,"range":{"start_line":1,"start_character":0,"end_line":1,"end_character":43},"updated":"2020-01-13 13:51:11.000000000","message":"Remove the vim settings.","commit_id":"9db68b0ec808777c184840d1588935c4256d6920"},{"author":{"_account_id":28470,"name":"Yan Chen","email":"yan.chen@intel.com","username":"chenyan"},"change_message_id":"ea4bc1addc6097ed78d9ae152115483f6e8a4904","unresolved":false,"context_lines":[{"line_number":1,"context_line":"# vim: tabstop\u003d4 shiftwidth\u003d4 softtabstop\u003d4"},{"line_number":2,"context_line":""},{"line_number":3,"context_line":"# Copyright 2010 United States Government as represented by the"},{"line_number":4,"context_line":"# Administrator of the National Aeronautics and Space Administration."}],"source_content_type":"text/x-python","patch_set":3,"id":"3fa7e38b_aa1259b2","line":1,"range":{"start_line":1,"start_character":0,"end_line":1,"end_character":43},"in_reply_to":"3fa7e38b_c44a3323","updated":"2020-01-14 02:45:42.000000000","message":"Done","commit_id":"9db68b0ec808777c184840d1588935c4256d6920"}],"fm-rest-api/fm/tests/test_utils.py":[{"author":{"_account_id":14174,"name":"Bart Wensley","email":"bartwensley@hotmail.com","username":"bartwensley"},"change_message_id":"8a9c3ee94023cc61f50b369efc0410785d65b462","unresolved":false,"context_lines":[{"line_number":1,"context_line":""},{"line_number":2,"context_line":""},{"line_number":3,"context_line":"# Copyright 2011 Justin Santa Barbara"},{"line_number":4,"context_line":"# Copyright 2012 Hewlett-Packard Development Company, L.P."},{"line_number":5,"context_line":"#"},{"line_number":6,"context_line":"#    Licensed under the Apache License, Version 2.0 (the \"License\"); you may"},{"line_number":7,"context_line":"#    not use this file except in compliance with the License. You may obtain"}],"source_content_type":"text/x-python","patch_set":3,"id":"3fa7e38b_84d19b31","line":4,"range":{"start_line":3,"start_character":0,"end_line":4,"end_character":58},"updated":"2020-01-13 13:51:11.000000000","message":"Since this file is essentially new code, why would you keep these old copyright notices?","commit_id":"9db68b0ec808777c184840d1588935c4256d6920"},{"author":{"_account_id":28470,"name":"Yan Chen","email":"yan.chen@intel.com","username":"chenyan"},"change_message_id":"ea4bc1addc6097ed78d9ae152115483f6e8a4904","unresolved":false,"context_lines":[{"line_number":1,"context_line":""},{"line_number":2,"context_line":""},{"line_number":3,"context_line":"# Copyright 2011 Justin Santa Barbara"},{"line_number":4,"context_line":"# Copyright 2012 Hewlett-Packard Development Company, L.P."},{"line_number":5,"context_line":"#"},{"line_number":6,"context_line":"#    Licensed under the Apache License, Version 2.0 (the \"License\"); you may"},{"line_number":7,"context_line":"#    not use this file except in compliance with the License. You may obtain"}],"source_content_type":"text/x-python","patch_set":3,"id":"3fa7e38b_4a072572","line":4,"range":{"start_line":3,"start_character":0,"end_line":4,"end_character":58},"in_reply_to":"3fa7e38b_84d19b31","updated":"2020-01-14 02:45:42.000000000","message":"Done","commit_id":"9db68b0ec808777c184840d1588935c4256d6920"}]}
