)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":7,"name":"Jay Pipes","email":"jaypipes@gmail.com","username":"jaypipes"},"change_message_id":"3087de4e15480bea4bc3d7c13e66bf8394db0fd1","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"commit I7423907ef33648669decc561fc3461415b877ba6 updated"},{"line_number":10,"context_line":"the Neutron port information of the allocated PCI device on"},{"line_number":11,"context_line":"migration resize-confirm. This patch addes the logic for"},{"line_number":12,"context_line":"migration resize-revert"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Co-Authored-By: Ludovic Beliveau \u003cludovic.beliveau@windriver.com\u003e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"9ad45d7e_7d693225","line":11,"updated":"2016-08-09 14:09:52.000000000","message":"s/addes/adds/","commit_id":"3af7cb6e1e6e9128345f9506652093b5c78b3344"},{"author":{"_account_id":17973,"name":"Ludovic Beliveau","email":"lbeliveau@gmail.com","username":"lbeliveau"},"change_message_id":"0502428fc7a35b46e0dd7bc6cc2efe126cab5943","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"commit I7423907ef33648669decc561fc3461415b877ba6 updated"},{"line_number":10,"context_line":"the Neutron port information of the allocated PCI device on"},{"line_number":11,"context_line":"migration resize-confirm. This patch addes the logic for"},{"line_number":12,"context_line":"migration resize-revert"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Co-Authored-By: Ludovic Beliveau \u003cludovic.beliveau@windriver.com\u003e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"9ad45d7e_aa7cfdeb","line":11,"in_reply_to":"9ad45d7e_7d693225","updated":"2016-08-09 15:05:14.000000000","message":"Done","commit_id":"3af7cb6e1e6e9128345f9506652093b5c78b3344"}],"nova/network/neutronv2/api.py":[{"author":{"_account_id":7,"name":"Jay Pipes","email":"jaypipes@gmail.com","username":"jaypipes"},"change_message_id":"3087de4e15480bea4bc3d7c13e66bf8394db0fd1","unresolved":false,"context_lines":[{"line_number":2215,"context_line":""},{"line_number":2216,"context_line":"    def _get_pci_devices_from_migration_context(self, migration_context,"},{"line_number":2217,"context_line":"                                                migration):"},{"line_number":2218,"context_line":"        if migration and migration.get(\u0027status\u0027) \u003d\u003d \u0027reverted\u0027:"},{"line_number":2219,"context_line":"            # In case of revert, swap old and new devices to"},{"line_number":2220,"context_line":"            # update the ports back to the original devices."},{"line_number":2221,"context_line":"            return (migration_context.new_pci_devices,"}],"source_content_type":"text/x-python","patch_set":3,"id":"9ad45d7e_bd73aa14","line":2218,"updated":"2016-08-09 14:09:52.000000000","message":"Please use object notation -- i.e. migration.status \u003d\u003d \u0027reverted\u0027 -- instead of dict notation.","commit_id":"3af7cb6e1e6e9128345f9506652093b5c78b3344"},{"author":{"_account_id":12171,"name":"Moshe Levi","email":"moshele@nvidia.com","username":"moshele"},"change_message_id":"56094d283385e9793ba482fa4b196159a85a91ef","unresolved":false,"context_lines":[{"line_number":2215,"context_line":""},{"line_number":2216,"context_line":"    def _get_pci_devices_from_migration_context(self, migration_context,"},{"line_number":2217,"context_line":"                                                migration):"},{"line_number":2218,"context_line":"        if migration and migration.get(\u0027status\u0027) \u003d\u003d \u0027reverted\u0027:"},{"line_number":2219,"context_line":"            # In case of revert, swap old and new devices to"},{"line_number":2220,"context_line":"            # update the ports back to the original devices."},{"line_number":2221,"context_line":"            return (migration_context.new_pci_devices,"}],"source_content_type":"text/x-python","patch_set":3,"id":"9ad45d7e_4b1f395f","line":2218,"in_reply_to":"9ad45d7e_016ed400","updated":"2016-08-09 18:31:16.000000000","message":"the object notation doen\u0027t work \nline 2218, in _get_pci_devices_from_migration_context\n2016-08-09 20:43:09.329 29509 ERROR nova.compute.manager [instance: 9c3a1648-4e27-480d-a809-35ce68524bdf]     if migration and migration.status \u003d\u003d \u0027reverted\u0027:\n2016-08-09 20:43:09.329 29509 ERROR nova.compute.manager [instance: 9c3a1648-4e27-480d-a809-35ce68524bdf] AttributeError: \u0027dict\u0027 object has no attribute \u0027status\u0027","commit_id":"3af7cb6e1e6e9128345f9506652093b5c78b3344"},{"author":{"_account_id":17973,"name":"Ludovic Beliveau","email":"lbeliveau@gmail.com","username":"lbeliveau"},"change_message_id":"0502428fc7a35b46e0dd7bc6cc2efe126cab5943","unresolved":false,"context_lines":[{"line_number":2215,"context_line":""},{"line_number":2216,"context_line":"    def _get_pci_devices_from_migration_context(self, migration_context,"},{"line_number":2217,"context_line":"                                                migration):"},{"line_number":2218,"context_line":"        if migration and migration.get(\u0027status\u0027) \u003d\u003d \u0027reverted\u0027:"},{"line_number":2219,"context_line":"            # In case of revert, swap old and new devices to"},{"line_number":2220,"context_line":"            # update the ports back to the original devices."},{"line_number":2221,"context_line":"            return (migration_context.new_pci_devices,"}],"source_content_type":"text/x-python","patch_set":3,"id":"9ad45d7e_016ed400","line":2218,"in_reply_to":"9ad45d7e_bd73aa14","updated":"2016-08-09 15:05:14.000000000","message":"Done","commit_id":"3af7cb6e1e6e9128345f9506652093b5c78b3344"},{"author":{"_account_id":7,"name":"Jay Pipes","email":"jaypipes@gmail.com","username":"jaypipes"},"change_message_id":"3087de4e15480bea4bc3d7c13e66bf8394db0fd1","unresolved":false,"context_lines":[{"line_number":2231,"context_line":""},{"line_number":2232,"context_line":"        :param context:  The request context."},{"line_number":2233,"context_line":"        :param instance: Get PCI mapping for this instance."},{"line_number":2234,"context_line":"        :param migration: The migration context for this instance."},{"line_number":2235,"context_line":"        :Returns: dictionary of mapping {\u0027\u003cold pci address\u003e\u0027: \u003cNew PciDevice\u003e}"},{"line_number":2236,"context_line":"        \"\"\""},{"line_number":2237,"context_line":"        migration_context \u003d instance.migration_context"}],"source_content_type":"text/x-python","patch_set":3,"id":"9ad45d7e_1d61163a","line":2234,"updated":"2016-08-09 14:09:52.000000000","message":"Agreed with snikitin. Docstring comment is confusing.","commit_id":"3af7cb6e1e6e9128345f9506652093b5c78b3344"},{"author":{"_account_id":9569,"name":"Sergey Nikitin","email":"nikitinserv@gmail.com","username":"snikitin"},"change_message_id":"4cbb2db8011309e280f8eb15870204be7cc52b96","unresolved":false,"context_lines":[{"line_number":2231,"context_line":""},{"line_number":2232,"context_line":"        :param context:  The request context."},{"line_number":2233,"context_line":"        :param instance: Get PCI mapping for this instance."},{"line_number":2234,"context_line":"        :param migration: The migration context for this instance."},{"line_number":2235,"context_line":"        :Returns: dictionary of mapping {\u0027\u003cold pci address\u003e\u0027: \u003cNew PciDevice\u003e}"},{"line_number":2236,"context_line":"        \"\"\""},{"line_number":2237,"context_line":"        migration_context \u003d instance.migration_context"}],"source_content_type":"text/x-python","patch_set":3,"id":"9ad45d7e_7f74cafc","line":2234,"range":{"start_line":2234,"start_character":26,"end_line":2234,"end_character":66},"updated":"2016-08-08 12:56:16.000000000","message":"I think you need to change this description because in the next line 2237 we are getting \"migration context\" from the \"instance\".","commit_id":"3af7cb6e1e6e9128345f9506652093b5c78b3344"},{"author":{"_account_id":17973,"name":"Ludovic Beliveau","email":"lbeliveau@gmail.com","username":"lbeliveau"},"change_message_id":"0502428fc7a35b46e0dd7bc6cc2efe126cab5943","unresolved":false,"context_lines":[{"line_number":2231,"context_line":""},{"line_number":2232,"context_line":"        :param context:  The request context."},{"line_number":2233,"context_line":"        :param instance: Get PCI mapping for this instance."},{"line_number":2234,"context_line":"        :param migration: The migration context for this instance."},{"line_number":2235,"context_line":"        :Returns: dictionary of mapping {\u0027\u003cold pci address\u003e\u0027: \u003cNew PciDevice\u003e}"},{"line_number":2236,"context_line":"        \"\"\""},{"line_number":2237,"context_line":"        migration_context \u003d instance.migration_context"}],"source_content_type":"text/x-python","patch_set":3,"id":"9ad45d7e_6154f020","line":2234,"in_reply_to":"9ad45d7e_1d61163a","updated":"2016-08-09 15:05:14.000000000","message":"Done","commit_id":"3af7cb6e1e6e9128345f9506652093b5c78b3344"},{"author":{"_account_id":17973,"name":"Ludovic Beliveau","email":"lbeliveau@gmail.com","username":"lbeliveau"},"change_message_id":"0502428fc7a35b46e0dd7bc6cc2efe126cab5943","unresolved":false,"context_lines":[{"line_number":2231,"context_line":""},{"line_number":2232,"context_line":"        :param context:  The request context."},{"line_number":2233,"context_line":"        :param instance: Get PCI mapping for this instance."},{"line_number":2234,"context_line":"        :param migration: The migration context for this instance."},{"line_number":2235,"context_line":"        :Returns: dictionary of mapping {\u0027\u003cold pci address\u003e\u0027: \u003cNew PciDevice\u003e}"},{"line_number":2236,"context_line":"        \"\"\""},{"line_number":2237,"context_line":"        migration_context \u003d instance.migration_context"}],"source_content_type":"text/x-python","patch_set":3,"id":"9ad45d7e_415f2c41","line":2234,"range":{"start_line":2234,"start_character":26,"end_line":2234,"end_character":66},"in_reply_to":"9ad45d7e_7f74cafc","updated":"2016-08-09 15:05:14.000000000","message":"Done","commit_id":"3af7cb6e1e6e9128345f9506652093b5c78b3344"}],"nova/tests/unit/network/test_neutronv2.py":[{"author":{"_account_id":7,"name":"Jay Pipes","email":"jaypipes@gmail.com","username":"jaypipes"},"change_message_id":"3087de4e15480bea4bc3d7c13e66bf8394db0fd1","unresolved":false,"context_lines":[{"line_number":3757,"context_line":"        instance.migration_context.old_pci_devices \u003d old_pci_devices"},{"line_number":3758,"context_line":"        instance.migration_context.new_pci_devices \u003d new_pci_devices"},{"line_number":3759,"context_line":"        instance.pci_devices \u003d instance.migration_context.old_pci_devices"},{"line_number":3760,"context_line":"        migration \u003d {\u0027status\u0027: \u0027confirmed\u0027}"},{"line_number":3761,"context_line":""},{"line_number":3762,"context_line":"        pci_mapping \u003d self.api._get_pci_mapping_for_migration("},{"line_number":3763,"context_line":"            self.context, instance, migration)"}],"source_content_type":"text/x-python","patch_set":3,"id":"9ad45d7e_dd781ef5","line":3760,"updated":"2016-08-09 14:09:52.000000000","message":"Here and below, please use a Migration object, not a dict...","commit_id":"3af7cb6e1e6e9128345f9506652093b5c78b3344"},{"author":{"_account_id":17973,"name":"Ludovic Beliveau","email":"lbeliveau@gmail.com","username":"lbeliveau"},"change_message_id":"0502428fc7a35b46e0dd7bc6cc2efe126cab5943","unresolved":false,"context_lines":[{"line_number":3757,"context_line":"        instance.migration_context.old_pci_devices \u003d old_pci_devices"},{"line_number":3758,"context_line":"        instance.migration_context.new_pci_devices \u003d new_pci_devices"},{"line_number":3759,"context_line":"        instance.pci_devices \u003d instance.migration_context.old_pci_devices"},{"line_number":3760,"context_line":"        migration \u003d {\u0027status\u0027: \u0027confirmed\u0027}"},{"line_number":3761,"context_line":""},{"line_number":3762,"context_line":"        pci_mapping \u003d self.api._get_pci_mapping_for_migration("},{"line_number":3763,"context_line":"            self.context, instance, migration)"}],"source_content_type":"text/x-python","patch_set":3,"id":"9ad45d7e_240fa62f","line":3760,"in_reply_to":"9ad45d7e_dd781ef5","updated":"2016-08-09 15:05:14.000000000","message":"Done","commit_id":"3af7cb6e1e6e9128345f9506652093b5c78b3344"}]}