)]}'
{"ironic/tests/unit/db/utils.py":[{"author":{"_account_id":8106,"name":"Haomeng,Wang","email":"wanghaomeng@gmail.com","username":"whaom"},"change_message_id":"b0a917ddfba2a09ac7f68d556f1ea2582402c724","unresolved":false,"context_lines":[{"line_number":195,"context_line":"    }"},{"line_number":196,"context_line":"    fake_info \u003d {\"foo\": \"bar\", \"fake_password\": \"fakepass\"}"},{"line_number":197,"context_line":"    return {"},{"line_number":198,"context_line":"        \u0027id\u0027: kw.get(\u0027id\u0027, 123),"},{"line_number":199,"context_line":"        \u0027name\u0027: kw.get(\u0027name\u0027, None),"},{"line_number":200,"context_line":"        \u0027uuid\u0027: kw.get(\u0027uuid\u0027, uuidutils.generate_uuid()),"},{"line_number":201,"context_line":"        \u0027chassis_id\u0027: kw.get(\u0027chassis_id\u0027, None),"}],"source_content_type":"text/x-python","patch_set":2,"id":"9a68dd71_0fe312da","line":198,"range":{"start_line":198,"start_character":30,"end_line":198,"end_character":32},"updated":"2016-01-26 02:06:03.000000000","message":"concern: we take the id as internal identfier for programing with the node objects, this will potentially lead to IntegrityErrors for some special case context also, so should we generate a different id for each node also?","commit_id":"50150207996b2ead9c7a3a079896e651972d3c0e"},{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"3ac728a9c81124f4f416c4da4b3895459506e82a","unresolved":false,"context_lines":[{"line_number":195,"context_line":"    }"},{"line_number":196,"context_line":"    fake_info \u003d {\"foo\": \"bar\", \"fake_password\": \"fakepass\"}"},{"line_number":197,"context_line":"    return {"},{"line_number":198,"context_line":"        \u0027id\u0027: kw.get(\u0027id\u0027, 123),"},{"line_number":199,"context_line":"        \u0027name\u0027: kw.get(\u0027name\u0027, None),"},{"line_number":200,"context_line":"        \u0027uuid\u0027: kw.get(\u0027uuid\u0027, uuidutils.generate_uuid()),"},{"line_number":201,"context_line":"        \u0027chassis_id\u0027: kw.get(\u0027chassis_id\u0027, None),"}],"source_content_type":"text/x-python","patch_set":2,"id":"9a68dd71_9873aa15","line":198,"range":{"start_line":198,"start_character":30,"end_line":198,"end_character":32},"in_reply_to":"9a68dd71_0fe312da","updated":"2016-01-26 15:03:47.000000000","message":"There is code that handles the ID [1]. (I don\u0027t know why it was written that way and don\u0027t have time to look into it.) This change makes the handling of ID and UUID inconsistent which isn\u0027t good (as you can see with Haomeng\u0027s question).\n\n[1] https://github.com/openstack/ironic/blob/f350d87af82ba8e33c19fb03eeb67625027838db/ironic/tests/unit/objects/utils.py#L55","commit_id":"50150207996b2ead9c7a3a079896e651972d3c0e"},{"author":{"_account_id":8106,"name":"Haomeng,Wang","email":"wanghaomeng@gmail.com","username":"whaom"},"change_message_id":"b0a917ddfba2a09ac7f68d556f1ea2582402c724","unresolved":false,"context_lines":[{"line_number":247,"context_line":""},{"line_number":248,"context_line":"def get_test_port(**kw):"},{"line_number":249,"context_line":"    return {"},{"line_number":250,"context_line":"        \u0027id\u0027: kw.get(\u0027id\u0027, 987),"},{"line_number":251,"context_line":"        \u0027uuid\u0027: kw.get(\u0027uuid\u0027, uuidutils.generate_uuid()),"},{"line_number":252,"context_line":"        \u0027node_id\u0027: kw.get(\u0027node_id\u0027, 123),"},{"line_number":253,"context_line":"        \u0027address\u0027: kw.get(\u0027address\u0027, \u002752:54:00:cf:2d:31\u0027),"}],"source_content_type":"text/x-python","patch_set":2,"id":"9a68dd71_ef06be84","line":250,"range":{"start_line":250,"start_character":30,"end_line":250,"end_character":32},"updated":"2016-01-26 02:06:03.000000000","message":"ditto.","commit_id":"50150207996b2ead9c7a3a079896e651972d3c0e"},{"author":{"_account_id":8106,"name":"Haomeng,Wang","email":"wanghaomeng@gmail.com","username":"whaom"},"change_message_id":"b0a917ddfba2a09ac7f68d556f1ea2582402c724","unresolved":false,"context_lines":[{"line_number":276,"context_line":""},{"line_number":277,"context_line":"def get_test_chassis(**kw):"},{"line_number":278,"context_line":"    return {"},{"line_number":279,"context_line":"        \u0027id\u0027: kw.get(\u0027id\u0027, 42),"},{"line_number":280,"context_line":"        \u0027uuid\u0027: kw.get(\u0027uuid\u0027, uuidutils.generate_uuid()),"},{"line_number":281,"context_line":"        \u0027extra\u0027: kw.get(\u0027extra\u0027, {}),"},{"line_number":282,"context_line":"        \u0027description\u0027: kw.get(\u0027description\u0027, \u0027data-center-1-chassis\u0027),"}],"source_content_type":"text/x-python","patch_set":2,"id":"9a68dd71_cf841ad4","line":279,"range":{"start_line":279,"start_character":29,"end_line":279,"end_character":31},"updated":"2016-01-26 02:06:03.000000000","message":"ditto","commit_id":"50150207996b2ead9c7a3a079896e651972d3c0e"}]}