)]}'
{"nova/compute/resource_tracker.py":[{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"70bc6f586020a6b83fec3f6f07f88410d7b0dd28","unresolved":false,"context_lines":[{"line_number":705,"context_line":"                    migration.old_instance_type_id):"},{"line_number":706,"context_line":"                itype \u003d self._get_instance_type(context, instance, \u0027new_\u0027,"},{"line_number":707,"context_line":"                        migration.new_instance_type_id)"},{"line_number":708,"context_line":"                numa_topology \u003d instance.migration_context.new_numa_topology"},{"line_number":709,"context_line":"            else:"},{"line_number":710,"context_line":"                # instance record already has new flavor, hold space for a"},{"line_number":711,"context_line":"                # possible revert to the old instance type:"}],"source_content_type":"text/x-python","patch_set":3,"id":"da20952f_98ac2c37","line":708,"updated":"2015-09-01 16:43:02.000000000","message":"If an older node started this migration (or is the destination? wherever the claim gets setup) then migration_context will be None and we\u0027ll fail here.\n\nI think you need to be defensive in that case and presumably just ignore the numa bits like we did before.","commit_id":"82e347a23c845cb2819743386d2697b7a9291823"},{"author":{"_account_id":5511,"name":"Nikola Dipanov","email":"ndipanov@redhat.com","username":"ndipanov"},"change_message_id":"35a6ca13b335427c6281dbbaaac2ab1e50283fdf","unresolved":false,"context_lines":[{"line_number":705,"context_line":"                    migration.old_instance_type_id):"},{"line_number":706,"context_line":"                itype \u003d self._get_instance_type(context, instance, \u0027new_\u0027,"},{"line_number":707,"context_line":"                        migration.new_instance_type_id)"},{"line_number":708,"context_line":"                numa_topology \u003d instance.migration_context.new_numa_topology"},{"line_number":709,"context_line":"            else:"},{"line_number":710,"context_line":"                # instance record already has new flavor, hold space for a"},{"line_number":711,"context_line":"                # possible revert to the old instance type:"}],"source_content_type":"text/x-python","patch_set":3,"id":"da20952f_d66b55f0","line":708,"in_reply_to":"da20952f_98ac2c37","updated":"2015-09-01 17:26:17.000000000","message":"Yes you are right - we need to make sure that migration_context is not None here","commit_id":"82e347a23c845cb2819743386d2697b7a9291823"}]}