)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":13362,"name":"Tan Lin","email":"lin.tan@intel.com","username":"zero"},"change_message_id":"43e2836d6e6c9d51b5d30497f04c9ccce46449ba","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Add CLI to list nodes using the same driver."},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Add CLI function to list nodes using the same driver."},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"Change-Id: I3ee3dd4fd52931a4eebb6ff037d83eb95e4fb1ba"},{"line_number":12,"context_line":"Partial-Bug: #1530626"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"9a68dd71_f6f665d6","line":10,"updated":"2016-01-21 02:53:38.000000000","message":"nit: better add a depends on to your API endpoint patch.","commit_id":"7a4a28618e25275f3037264f374ba9b30f1d5ac1"}],"ironicclient/tests/unit/v1/test_node_shell.py":[{"author":{"_account_id":8106,"name":"Haomeng,Wang","email":"wanghaomeng@gmail.com","username":"whaom"},"change_message_id":"efe72da42e94414c09245b9a4f4dfa8a9022a6d5","unresolved":false,"context_lines":[{"line_number":585,"context_line":""},{"line_number":586,"context_line":"        n_shell.do_node_list(client_mock, args)"},{"line_number":587,"context_line":"        client_mock.node.list.assert_called_once_with(driver\u003d\u0027fake\u0027,"},{"line_number":588,"context_line":"                                                      detail\u003dTrue)"},{"line_number":589,"context_line":""},{"line_number":590,"context_line":"    def test_do_node_list_sort_key(self):"},{"line_number":591,"context_line":"        client_mock \u003d mock.MagicMock()"}],"source_content_type":"text/x-python","patch_set":3,"id":"fa69d971_0c1ebe6f","line":588,"range":{"start_line":588,"start_character":65,"end_line":588,"end_character":66},"updated":"2016-01-07 08:06:56.000000000","message":"nit: it is better that we add some negative cases.","commit_id":"844fec6f9ea53e2ccd9cca6ee58cbf613cca018b"},{"author":{"_account_id":14810,"name":"Kan","email":"sofaopenstack@gmail.com","username":"Kan"},"change_message_id":"a1a8e6943363ac10a7edb818cf7e5a63fb3bab5c","unresolved":false,"context_lines":[{"line_number":585,"context_line":""},{"line_number":586,"context_line":"        n_shell.do_node_list(client_mock, args)"},{"line_number":587,"context_line":"        client_mock.node.list.assert_called_once_with(driver\u003d\u0027fake\u0027,"},{"line_number":588,"context_line":"                                                      detail\u003dTrue)"},{"line_number":589,"context_line":""},{"line_number":590,"context_line":"    def test_do_node_list_sort_key(self):"},{"line_number":591,"context_line":"        client_mock \u003d mock.MagicMock()"}],"source_content_type":"text/x-python","patch_set":3,"id":"fa69d971_7123fc58","line":588,"range":{"start_line":588,"start_character":65,"end_line":588,"end_character":66},"in_reply_to":"fa69d971_0c1ebe6f","updated":"2016-01-11 02:40:15.000000000","message":"Thanks Haomeng. I think the negative cases of the driver argument are not needed here for the negative cases are in the API side.","commit_id":"844fec6f9ea53e2ccd9cca6ee58cbf613cca018b"}]}