)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":9550,"name":"Eugeniya Kudryashova","email":"ekudryashova@mirantis.com","username":"ekudryashova"},"change_message_id":"1b293a07069a24a92c3ca6553c531d6bf03be6b2","unresolved":false,"context_lines":[{"line_number":7,"context_line":"hook up pci allocation for resizing"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"in API stage save a \u0027_new\u0027 prefix flavor for pci devices,"},{"line_number":10,"context_line":"then pci manager could alloction new devices based on new"},{"line_number":11,"context_line":"instance type."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: I4722d094d50108c897dcb1e0763efc50ff7607c5"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"da86d52c_6da184f6","line":10,"updated":"2015-02-18 16:30:52.000000000","message":"alloction \u003e allocate","commit_id":"f8d2ae04839add57da1eb4c518b2b61fd5117587"},{"author":{"_account_id":7543,"name":"Yongli He","email":"yongli.he@intel.com","username":"yongli.he"},"change_message_id":"dfec11d61ea64829f45269872bdb583e03d4a4e7","unresolved":false,"context_lines":[{"line_number":7,"context_line":"hook up pci allocation for resizing"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"in API stage save a \u0027_new\u0027 prefix flavor for pci devices,"},{"line_number":10,"context_line":"then pci manager could alloction new devices based on new"},{"line_number":11,"context_line":"instance type."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: I4722d094d50108c897dcb1e0763efc50ff7607c5"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"ba7be1f8_dd3432e1","line":10,"in_reply_to":"da86d52c_6da184f6","updated":"2015-03-02 06:01:13.000000000","message":"Done","commit_id":"f8d2ae04839add57da1eb4c518b2b61fd5117587"},{"author":{"_account_id":17973,"name":"Ludovic Beliveau","email":"lbeliveau@gmail.com","username":"lbeliveau"},"change_message_id":"18052aa1d2450789d6c03605687b1b39debeecfd","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"save new pci request for resizing"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"API stage should save the \u0027_new\u0027 prefix flavor\u0027s pci request,"},{"line_number":10,"context_line":"then the pci manager could allocate new devices based on new"},{"line_number":11,"context_line":"new pci request."},{"line_number":12,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"1a122d0e_7fe78d12","line":9,"updated":"2016-04-15 16:41:24.000000000","message":"This phrase doesn\u0027t make much sense.","commit_id":"f21cf6a7bf739c0ad0fa659e2f22fa7c04940e47"},{"author":{"_account_id":17973,"name":"Ludovic Beliveau","email":"lbeliveau@gmail.com","username":"lbeliveau"},"change_message_id":"18052aa1d2450789d6c03605687b1b39debeecfd","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"API stage should save the \u0027_new\u0027 prefix flavor\u0027s pci request,"},{"line_number":10,"context_line":"then the pci manager could allocate new devices based on new"},{"line_number":11,"context_line":"new pci request."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: I4722d094d50108c897dcb1e0763efc50ff7607c5"},{"line_number":14,"context_line":"Partial-Bug: #1368201"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"1a122d0e_7ff94dc8","line":11,"updated":"2016-04-15 16:41:24.000000000","message":"delete one \"new\".","commit_id":"f21cf6a7bf739c0ad0fa659e2f22fa7c04940e47"}],"nova/compute/api.py":[{"author":{"_account_id":9550,"name":"Eugeniya Kudryashova","email":"ekudryashova@mirantis.com","username":"ekudryashova"},"change_message_id":"1b293a07069a24a92c3ca6553c531d6bf03be6b2","unresolved":false,"context_lines":[{"line_number":2629,"context_line":""},{"line_number":2630,"context_line":"        pci_request_info \u003d pci_request.get_pci_requests_from_flavor("},{"line_number":2631,"context_line":"            new_instance_type, is_new\u003dTrue)"},{"line_number":2632,"context_line":"        pci_request_info.instance_uuid \u003d instance[\u0027uuid\u0027]"},{"line_number":2633,"context_line":"        pci_request_info.save(context)"},{"line_number":2634,"context_line":""},{"line_number":2635,"context_line":"        filter_properties \u003d {\u0027ignore_hosts\u0027: []}"}],"source_content_type":"text/x-python","patch_set":1,"id":"da86d52c_32ee0d8c","line":2632,"updated":"2015-02-18 16:30:52.000000000","message":"Can we use here instance as an object instead as a dict?","commit_id":"f8d2ae04839add57da1eb4c518b2b61fd5117587"},{"author":{"_account_id":7543,"name":"Yongli He","email":"yongli.he@intel.com","username":"yongli.he"},"change_message_id":"dfec11d61ea64829f45269872bdb583e03d4a4e7","unresolved":false,"context_lines":[{"line_number":2629,"context_line":""},{"line_number":2630,"context_line":"        pci_request_info \u003d pci_request.get_pci_requests_from_flavor("},{"line_number":2631,"context_line":"            new_instance_type, is_new\u003dTrue)"},{"line_number":2632,"context_line":"        pci_request_info.instance_uuid \u003d instance[\u0027uuid\u0027]"},{"line_number":2633,"context_line":"        pci_request_info.save(context)"},{"line_number":2634,"context_line":""},{"line_number":2635,"context_line":"        filter_properties \u003d {\u0027ignore_hosts\u0027: []}"}],"source_content_type":"text/x-python","patch_set":1,"id":"ba7be1f8_a416cda5","line":2632,"in_reply_to":"da86d52c_32ee0d8c","updated":"2015-03-02 06:01:13.000000000","message":"done","commit_id":"f8d2ae04839add57da1eb4c518b2b61fd5117587"},{"author":{"_account_id":17973,"name":"Ludovic Beliveau","email":"lbeliveau@gmail.com","username":"lbeliveau"},"change_message_id":"18052aa1d2450789d6c03605687b1b39debeecfd","unresolved":false,"context_lines":[{"line_number":2705,"context_line":"            new_instance_type, is_new\u003dTrue)"},{"line_number":2706,"context_line":"        pci_request_info.instance_uuid \u003d instance.uuid"},{"line_number":2707,"context_line":"        pci_request_info.save(context)"},{"line_number":2708,"context_line":""},{"line_number":2709,"context_line":"        filter_properties \u003d {\u0027ignore_hosts\u0027: []}"},{"line_number":2710,"context_line":""},{"line_number":2711,"context_line":"        if not CONF.allow_resize_to_same_host:"}],"source_content_type":"text/x-python","patch_set":3,"id":"1a122d0e_bf776501","line":2708,"updated":"2016-04-15 16:41:24.000000000","message":"Don\u0027t you have to do this as well \"instance.pci_requests \u003d pci_request_info\" ?\n\nWouldn\u0027t it be simpler to set \"is_new\" to true for all the instance.pci_requests instead of creating new objects ?","commit_id":"f21cf6a7bf739c0ad0fa659e2f22fa7c04940e47"}],"nova/pci/manager.py":[{"author":{"_account_id":5511,"name":"Nikola Dipanov","email":"ndipanov@redhat.com","username":"ndipanov"},"change_message_id":"2af1c766362684701928d256064f736e4cf32f2b","unresolved":false,"context_lines":[{"line_number":172,"context_line":"    def _claim_instance(self, context, instance, is_new\u003dFalse):"},{"line_number":173,"context_line":"        PCIrequest \u003d objects.InstancePCIRequests"},{"line_number":174,"context_line":"        if is_new:"},{"line_number":175,"context_line":"            pci_requests \u003d PCIrequest.get_by_instance_uuid_and_newness(context,"},{"line_number":176,"context_line":"                                                 instance[\u0027uuid\u0027],"},{"line_number":177,"context_line":"                                                 is_new\u003dTrue)"},{"line_number":178,"context_line":"        else:"}],"source_content_type":"text/x-python","patch_set":3,"id":"1a122d0e_d078efc6","line":175,"range":{"start_line":175,"start_character":38,"end_line":175,"end_character":70},"updated":"2016-04-19 09:18:25.000000000","message":"This is obviously a hack and should be removed. I\u0027m happier with Moshe\u0027s approach here which does that exact thing.\n\nhttps://review.openstack.org/#/c/307124/","commit_id":"f21cf6a7bf739c0ad0fa659e2f22fa7c04940e47"}]}