)]}'
{"nova/tests/unit/virt/ironic/test_driver.py":[{"author":{"_account_id":13689,"name":"Hironori Shiina","email":"Hironori.Shiina@fujitsu.com","username":"shiina"},"change_message_id":"21f1d50481a93e0a897ff3180d897f7eab9ffcc4","unresolved":false,"context_lines":[{"line_number":518,"context_line":"                                                             id\u003di,"},{"line_number":519,"context_line":"                                                             uuid\u003duuid))"},{"line_number":520,"context_line":"            nodes.append(ironic_utils.get_test_node(instance_uuid\u003duuid,"},{"line_number":521,"context_line":"                                                    fields\u003d[\u0027instance_uuid\u0027]))"},{"line_number":522,"context_line":""},{"line_number":523,"context_line":"        mock_inst_by_uuid.side_effect \u003d instances"},{"line_number":524,"context_line":"        mock_call.return_value \u003d nodes"}],"source_content_type":"text/x-python","patch_set":2,"id":"bf659307_37fd563a","line":521,"range":{"start_line":521,"start_character":52,"end_line":521,"end_character":76},"updated":"2018-03-26 05:10:39.000000000","message":"nit: This is not necessary.","commit_id":"8b9215586aa069c1ba1a087f4963d9db4e4fe17f"},{"author":{"_account_id":13689,"name":"Hironori Shiina","email":"Hironori.Shiina@fujitsu.com","username":"shiina"},"change_message_id":"141005768dac8b4ea1ca22c568aac0f69e0d6f30","unresolved":false,"context_lines":[{"line_number":518,"context_line":"                                                             id\u003di,"},{"line_number":519,"context_line":"                                                             uuid\u003duuid))"},{"line_number":520,"context_line":"            nodes.append(ironic_utils.get_test_node(instance_uuid\u003duuid,"},{"line_number":521,"context_line":"                                                    fields\u003d[\u0027instance_uuid\u0027]))"},{"line_number":522,"context_line":""},{"line_number":523,"context_line":"        mock_inst_by_uuid.side_effect \u003d instances"},{"line_number":524,"context_line":"        mock_call.return_value \u003d nodes"}],"source_content_type":"text/x-python","patch_set":2,"id":"bf659307_c3dc5c48","line":521,"range":{"start_line":521,"start_character":52,"end_line":521,"end_character":76},"in_reply_to":"bf659307_37fd563a","updated":"2018-03-26 08:15:35.000000000","message":"I noticed this parameter will be added by another patch[1]. It seems that the dependency was lost in PS2.\n\n[1] https://review.openstack.org/#/c/539508/","commit_id":"8b9215586aa069c1ba1a087f4963d9db4e4fe17f"},{"author":{"_account_id":13689,"name":"Hironori Shiina","email":"Hironori.Shiina@fujitsu.com","username":"shiina"},"change_message_id":"21f1d50481a93e0a897ff3180d897f7eab9ffcc4","unresolved":false,"context_lines":[{"line_number":549,"context_line":"        for n in range(num_nodes):"},{"line_number":550,"context_line":"            nodes.append(ironic_utils.get_test_node("},{"line_number":551,"context_line":"                                      instance_uuid\u003duuidutils.generate_uuid(),"},{"line_number":552,"context_line":"                                      fields\u003d[\u0027instance_uuid\u0027]))"},{"line_number":553,"context_line":""},{"line_number":554,"context_line":"        mock_call.return_value \u003d nodes"},{"line_number":555,"context_line":"        uuids \u003d self.driver.list_instance_uuids()"}],"source_content_type":"text/x-python","patch_set":2,"id":"bf659307_d7a3ba4e","line":552,"range":{"start_line":552,"start_character":38,"end_line":552,"end_character":62},"updated":"2018-03-26 05:10:39.000000000","message":"ditto","commit_id":"8b9215586aa069c1ba1a087f4963d9db4e4fe17f"}]}