)]}'
{"nova/objects/pci_device.py":[{"author":{"_account_id":12171,"name":"Moshe Levi","email":"moshele@nvidia.com","username":"moshele"},"change_message_id":"f47adef6b30db3e4685ac6a14c4415f8d4525eb1","unresolved":false,"context_lines":[{"line_number":367,"context_line":"        self.status \u003d fields.PciDeviceStatus.ALLOCATED"},{"line_number":368,"context_line":"        self.instance_uuid \u003d instance[\u0027uuid\u0027]"},{"line_number":369,"context_line":""},{"line_number":370,"context_line":"        # NOTE(lbeliveau): Update the status in the migration_context.  Else"},{"line_number":371,"context_line":"        # calling apply_migration_context() can cause the PCI devices in the"},{"line_number":372,"context_line":"        # instance to still show as \u0027claimed\u0027."},{"line_number":373,"context_line":"        migration_context \u003d instance.migration_context"},{"line_number":374,"context_line":"        if migration_context:"},{"line_number":375,"context_line":"            for dev in migration_context.new_pci_devices:"},{"line_number":376,"context_line":"                if dev.id \u003d\u003d self.id:"},{"line_number":377,"context_line":"                    dev.status \u003d fields.PciDeviceStatus.ALLOCATED"},{"line_number":378,"context_line":""},{"line_number":379,"context_line":"        # Notes(yjiang5): remove this check when instance object for"},{"line_number":380,"context_line":"        # compute manager is finished"}],"source_content_type":"text/x-python","patch_set":1,"id":"3aaa91ec_76c29954","line":377,"range":{"start_line":370,"start_character":0,"end_line":377,"end_character":65},"updated":"2016-06-30 06:59:43.000000000","message":"shouldn\u0027t we do that when we apply the migration context","commit_id":"3081d93ed8c03636ba54dead739adf8f58351111"},{"author":{"_account_id":21813,"name":"Andrey Volkov","email":"m@amadev.ru","username":"avolkov"},"change_message_id":"3528ed67174a868c60a0e1c98d9728f8c9befc6e","unresolved":false,"context_lines":[{"line_number":374,"context_line":"        if migration_context:"},{"line_number":375,"context_line":"            for dev in migration_context.new_pci_devices:"},{"line_number":376,"context_line":"                if dev.id \u003d\u003d self.id:"},{"line_number":377,"context_line":"                    dev.status \u003d fields.PciDeviceStatus.ALLOCATED"},{"line_number":378,"context_line":""},{"line_number":379,"context_line":"        # Notes(yjiang5): remove this check when instance object for"},{"line_number":380,"context_line":"        # compute manager is finished"}],"source_content_type":"text/x-python","patch_set":3,"id":"fa7ab95a_2def7de9","line":377,"updated":"2016-08-26 11:13:14.000000000","message":"I\u0027m totally not against having a test for this code.","commit_id":"2c4b8481c2e7bdf5fe1f762f70e7ed72a350510e"},{"author":{"_account_id":9569,"name":"Sergey Nikitin","email":"nikitinserv@gmail.com","username":"snikitin"},"change_message_id":"97e68f105928a16d09676dc6966e18ac600b564a","unresolved":false,"context_lines":[{"line_number":374,"context_line":"        if migration_context:"},{"line_number":375,"context_line":"            for dev in migration_context.new_pci_devices:"},{"line_number":376,"context_line":"                if dev.id \u003d\u003d self.id:"},{"line_number":377,"context_line":"                    dev.status \u003d fields.PciDeviceStatus.ALLOCATED"},{"line_number":378,"context_line":""},{"line_number":379,"context_line":"        # Notes(yjiang5): remove this check when instance object for"},{"line_number":380,"context_line":"        # compute manager is finished"}],"source_content_type":"text/x-python","patch_set":3,"id":"fa7ab95a_2c24d50b","line":377,"in_reply_to":"fa7ab95a_2def7de9","updated":"2016-08-26 13:49:26.000000000","message":"yes, I think you need a test here","commit_id":"2c4b8481c2e7bdf5fe1f762f70e7ed72a350510e"}]}