)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"899c8221df011352b9c9e8f6923880ae17a9bd54","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Nikola Dipanov \u003cndipanov@redhat.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2015-08-14 05:08:00 +0100"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"rt: move filtering of mygration by type lower in the call stack"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"We want to filter out the migrations that are not yet created for"},{"line_number":10,"context_line":"tracking resources, and the best place to do this is in the method that"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":8,"id":"1a4dcd0f_1e10e058","line":7,"range":{"start_line":7,"start_character":22,"end_line":7,"end_character":31},"updated":"2015-08-14 10:56:50.000000000","message":"s/mygration/migration/","commit_id":"3bbaf57a1374c80359e0b1ed91a233f0449b0419"},{"author":{"_account_id":5511,"name":"Nikola Dipanov","email":"ndipanov@redhat.com","username":"ndipanov"},"change_message_id":"707383100f4746a9d7d9213f16e060dfd9fde2c2","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Nikola Dipanov \u003cndipanov@redhat.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2015-08-14 05:08:00 +0100"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"rt: move filtering of mygration by type lower in the call stack"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"We want to filter out the migrations that are not yet created for"},{"line_number":10,"context_line":"tracking resources, and the best place to do this is in the method that"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":8,"id":"fa1b9901_36556df9","line":7,"in_reply_to":"1a4dcd0f_1e10e058","updated":"2015-08-19 19:24:47.000000000","message":"oops sorry I missed this - will fix this in the next patch.","commit_id":"3bbaf57a1374c80359e0b1ed91a233f0449b0419"},{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"2c2098cf5944d9fd6ec0bd7d1e582c00407d3e56","unresolved":false,"context_lines":[{"line_number":12,"context_line":"_update_usage_from_migration()."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"This also fixes a bug that existed in the previous check where we used"},{"line_number":15,"context_line":"\u0027migrate\u0027 instead of \u0027migration\u0027 to check for the type."},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"Change-Id: I740fdadc8461e9a0ba871171262d976f39d84b25"},{"line_number":18,"context_line":"Related-blueprint: migration-fix-resource-tracking"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":10,"id":"fa1b9901_330b4a26","line":15,"updated":"2015-08-25 14:28:48.000000000","message":"Can haz bug for this?\n\nIf this was really broken, then people could have deployed it and may come asking about it later...","commit_id":"f670d2545847511ecf8a4b9fc5519e1ad60da314"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"fb9303fb566092f7346012cbb15dc96714393c19","unresolved":false,"context_lines":[{"line_number":12,"context_line":"_update_usage_from_migration()."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"This also fixes a bug that existed in the previous check where we used"},{"line_number":15,"context_line":"\u0027migrate\u0027 instead of \u0027migration\u0027 to check for the type."},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"Change-Id: I740fdadc8461e9a0ba871171262d976f39d84b25"},{"line_number":18,"context_line":"Related-blueprint: migration-fix-resource-tracking"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":10,"id":"da20952f_6035ad44","line":15,"in_reply_to":"fa1b9901_330b4a26","updated":"2015-08-27 14:52:50.000000000","message":"yeah, thats true, it would be good to track this one.","commit_id":"f670d2545847511ecf8a4b9fc5519e1ad60da314"}],"nova/compute/resource_tracker.py":[{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"ad079eb141f5798224824e47af0570fbdca5b87c","unresolved":false,"context_lines":[{"line_number":468,"context_line":"        # notified when instances are deleted, we need remove all usages"},{"line_number":469,"context_line":"        # from deleted instances."},{"line_number":470,"context_line":"        if self.pci_tracker:"},{"line_number":471,"context_line":"            self.pci_tracker.clean_usage(instances, migrations, orphans)"},{"line_number":472,"context_line":"            dev_pools_obj \u003d self.pci_tracker.stats.to_device_pools_obj()"},{"line_number":473,"context_line":"            self.compute_node.pci_device_pools \u003d dev_pools_obj"},{"line_number":474,"context_line":"        else:"}],"source_content_type":"text/x-python","patch_set":13,"id":"da20952f_008e3178","line":471,"updated":"2015-08-27 14:51:39.000000000","message":"I guess we are OK with the migrations getting passed here now, yeah, I think so.","commit_id":"45ed1e719a1569d53863542d169f82e9f9ca3557"},{"author":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"change_message_id":"4271152c045acd7856ff0cb808d66c9fe12f3c8c","unresolved":false,"context_lines":[{"line_number":468,"context_line":"        # notified when instances are deleted, we need remove all usages"},{"line_number":469,"context_line":"        # from deleted instances."},{"line_number":470,"context_line":"        if self.pci_tracker:"},{"line_number":471,"context_line":"            self.pci_tracker.clean_usage(instances, migrations, orphans)"},{"line_number":472,"context_line":"            dev_pools_obj \u003d self.pci_tracker.stats.to_device_pools_obj()"},{"line_number":473,"context_line":"            self.compute_node.pci_device_pools \u003d dev_pools_obj"},{"line_number":474,"context_line":"        else:"}],"source_content_type":"text/x-python","patch_set":13,"id":"da20952f_d886dbda","line":471,"in_reply_to":"da20952f_008e3178","updated":"2015-08-27 22:52:15.000000000","message":"Yup, that looks fine","commit_id":"45ed1e719a1569d53863542d169f82e9f9ca3557"},{"author":{"_account_id":7,"name":"Jay Pipes","email":"jaypipes@gmail.com","username":"jaypipes"},"change_message_id":"1cc83f760421500ccb116a75b9482dfba8883e77","unresolved":false,"context_lines":[{"line_number":622,"context_line":"        # Only look at resize/migrate migration records"},{"line_number":623,"context_line":"        # NOTE(danms): RT should probably examine live migration"},{"line_number":624,"context_line":"        # records as well and do something smart. However, ignore"},{"line_number":625,"context_line":"        # those for now to avoid them being included in below calculations."},{"line_number":626,"context_line":"        return migration.migration_type in (\u0027resize\u0027, \u0027migration\u0027)"},{"line_number":627,"context_line":""},{"line_number":628,"context_line":"    def _update_usage_from_migration(self, context, instance, image_meta,"}],"source_content_type":"text/x-python","patch_set":13,"id":"da20952f_15a45155","line":625,"updated":"2015-08-27 05:26:11.000000000","message":"The above comment should probably be above line 633 below instead of here, but OK :)","commit_id":"45ed1e719a1569d53863542d169f82e9f9ca3557"}],"nova/tests/unit/compute/test_resource_tracker.py":[{"author":{"_account_id":7,"name":"Jay Pipes","email":"jaypipes@gmail.com","username":"jaypipes"},"change_message_id":"1cc83f760421500ccb116a75b9482dfba8883e77","unresolved":false,"context_lines":[{"line_number":1259,"context_line":"        self.claim_method \u003d self.tracker.resize_claim"},{"line_number":1260,"context_line":""},{"line_number":1261,"context_line":"    def test_move_type_not_tracked(self):"},{"line_number":1262,"context_line":"        self.skipTest(\"Resize_claim does already sets the move_type.\")"},{"line_number":1263,"context_line":""},{"line_number":1264,"context_line":""},{"line_number":1265,"context_line":"class OrphanTestCase(BaseTrackerTestCase):"}],"source_content_type":"text/x-python","patch_set":13,"id":"da20952f_959061ee","line":1262,"updated":"2015-08-27 05:26:11.000000000","message":"s/does already sets/already sets/","commit_id":"45ed1e719a1569d53863542d169f82e9f9ca3557"}]}