)]}'
{"tempest/scenario/test_security_groups_basic_ops.py":[{"author":{"_account_id":4656,"name":"Sean M. Collins","email":"sean@coreitpro.com","username":"scollins"},"change_message_id":"214af7983f8c71c1df028082ac244baa05633c3a","unresolved":false,"context_lines":[{"line_number":91,"context_line":"    \"\"\""},{"line_number":92,"context_line":""},{"line_number":93,"context_line":"    credentials \u003d [\u0027primary\u0027, \u0027alt\u0027, \u0027admin\u0027]"},{"line_number":94,"context_line":"    _ip_version \u003d 4"},{"line_number":95,"context_line":""},{"line_number":96,"context_line":"    class TenantProperties(object):"},{"line_number":97,"context_line":"        \"\"\"helper class to save tenant details"}],"source_content_type":"text/x-python","patch_set":5,"id":"9a68dd71_f1f66fd6","line":94,"updated":"2016-01-21 03:24:10.000000000","message":"No need to prefix with an underscore, since credentials is is not declared as \"_credentials\"","commit_id":"441a43c78c75a58cb4882b76723fe6435e7ad5fa"},{"author":{"_account_id":4656,"name":"Sean M. Collins","email":"sean@coreitpro.com","username":"scollins"},"change_message_id":"1aaa3816558fe5b65d7adbe9961f8236ad3320cd","unresolved":false,"context_lines":[{"line_number":91,"context_line":"    \"\"\""},{"line_number":92,"context_line":""},{"line_number":93,"context_line":"    credentials \u003d [\u0027primary\u0027, \u0027alt\u0027, \u0027admin\u0027]"},{"line_number":94,"context_line":"    _ip_version \u003d 4"},{"line_number":95,"context_line":""},{"line_number":96,"context_line":"    class TenantProperties(object):"},{"line_number":97,"context_line":"        \"\"\"helper class to save tenant details"}],"source_content_type":"text/x-python","patch_set":5,"id":"9a68dd71_f5d9ff57","line":94,"in_reply_to":"9a68dd71_9428e587","updated":"2016-01-23 06:28:20.000000000","message":"OK that makes sense.","commit_id":"441a43c78c75a58cb4882b76723fe6435e7ad5fa"},{"author":{"_account_id":19050,"name":"Johan Pas","email":"j20012@outlook.com","username":"jpas"},"change_message_id":"c81ddd80b3d7ed5e0b0413d9078f389d6ba6d99b","unresolved":false,"context_lines":[{"line_number":91,"context_line":"    \"\"\""},{"line_number":92,"context_line":""},{"line_number":93,"context_line":"    credentials \u003d [\u0027primary\u0027, \u0027alt\u0027, \u0027admin\u0027]"},{"line_number":94,"context_line":"    _ip_version \u003d 4"},{"line_number":95,"context_line":""},{"line_number":96,"context_line":"    class TenantProperties(object):"},{"line_number":97,"context_line":"        \"\"\"helper class to save tenant details"}],"source_content_type":"text/x-python","patch_set":5,"id":"9a68dd71_9428e587","line":94,"in_reply_to":"9a68dd71_f1f66fd6","updated":"2016-01-22 14:55:08.000000000","message":"I used the leading underscore in _ip_version to be consistent with the naming in the API tests, e.g. https://github.com/openstack/tempest/blob/master/tempest/api/network/base.py#L51","commit_id":"441a43c78c75a58cb4882b76723fe6435e7ad5fa"}]}