)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"change_message_id":"a68c4b2e0d1f9360e3cdc844332d08d1a4a72aed","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     xianming mao \u003cxianming.mao@easystack.cn\u003e"},{"line_number":5,"context_line":"CommitDate: 2016-10-17 09:52:13 +0800"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Add test_volumes_list functions in test_volumes_list.py."},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This patch has added two test functions which includes"},{"line_number":10,"context_line":"    test_volumes_list_by_bootable"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":7,"id":"fa6399be_a24834b4","line":7,"updated":"2016-10-17 08:57:41.000000000","message":"I think it\u0027s better to mention about \u0027bootable\u0027 thing in the subject because it\u0027s the different thing from others, right?\nAnd one nit, We don\u0027t need the trailing period :)","commit_id":"58fe2b3728f029fb8f4bad1b9506d44f6c499bdf"}],"tempest/api/volume/test_volumes_list.py":[{"author":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"change_message_id":"a68c4b2e0d1f9360e3cdc844332d08d1a4a72aed","unresolved":false,"context_lines":[{"line_number":35,"context_line":"        \"\"\"Check out the list."},{"line_number":36,"context_line":""},{"line_number":37,"context_line":"        This function is aim at check out whether all of the volumes in"},{"line_number":38,"context_line":"        expected_list are in fetched_list."},{"line_number":39,"context_line":"        \"\"\""},{"line_number":40,"context_line":"        if fields:"},{"line_number":41,"context_line":"            fieldsgetter \u003d operator.itemgetter(*fields)"}],"source_content_type":"text/x-python","patch_set":7,"id":"fa6399be_c2c9a827","line":38,"updated":"2016-10-17 08:57:41.000000000","message":"I\u0027m actually not sure this is useful or not because the method name already indicates it. But the \u0027fields\u0027 parameter is unclear for me. I just understand it through reading the below code.","commit_id":"58fe2b3728f029fb8f4bad1b9506d44f6c499bdf"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"aafe3877fdbd2adf697bbfe66718956928376739","unresolved":false,"context_lines":[{"line_number":35,"context_line":"        \"\"\"Check out the list."},{"line_number":36,"context_line":""},{"line_number":37,"context_line":"        This function is aim at check out whether all of the volumes in"},{"line_number":38,"context_line":"        expected_list are in fetched_list."},{"line_number":39,"context_line":"        \"\"\""},{"line_number":40,"context_line":"        if fields:"},{"line_number":41,"context_line":"            fieldsgetter \u003d operator.itemgetter(*fields)"}],"source_content_type":"text/x-python","patch_set":8,"id":"fa6399be_13c71b13","line":38,"range":{"start_line":38,"start_character":41,"end_line":38,"end_character":42},"updated":"2016-10-19 05:58:16.000000000","message":"yea, should mention about fields also if adding doc string.","commit_id":"0bed624ff5ac8473403fb8c5885a57aa15a9fe2e"}]}