)]}'
{"nova/compute/manager.py":[{"author":{"_account_id":5638,"name":"Davanum Srinivas","email":"davanum@gmail.com","username":"dims-v"},"change_message_id":"168d87cf841ebe03342c122f4b2349b685b6f978","unresolved":false,"context_lines":[{"line_number":4456,"context_line":""},{"line_number":4457,"context_line":"        # Don\u0027t update the cache for instances that are still building,"},{"line_number":4458,"context_line":"        # about to be deleted, or have moved to another host"},{"line_number":4459,"context_line":"        while (not instance or"},{"line_number":4460,"context_line":"               instance.vm_state \u003d\u003d vm_states.BUILDING or"},{"line_number":4461,"context_line":"               instance.task_state \u003d\u003d task_states.DELETING or"},{"line_number":4462,"context_line":"               instance.host !\u003d self.host):"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAATn%2F%2BjVE%3D","line":4459,"updated":"2013-11-24 00:39:47.000000000","message":"@Phil, adding the additional conditions here, is there a possibility of infinite loop? Could we avoid introducing one by adding the 2 conditions just before \"self._get_instance_nw_info(context, instance)\" [1]? \n\n[1] https://github.com/openstack/nova/blob/master/nova/compute/manager.py#L4481","commit_id":"ad1837779b843b8f22503752517b5778149284b1"},{"author":{"_account_id":1501,"name":"Phil Day","email":"philip.day@hp.com","username":"philip-day"},"change_message_id":"b41560bbe3828b5c53350e2937b846fa444e29e3","unresolved":false,"context_lines":[{"line_number":4456,"context_line":""},{"line_number":4457,"context_line":"        # Don\u0027t update the cache for instances that are still building,"},{"line_number":4458,"context_line":"        # about to be deleted, or have moved to another host"},{"line_number":4459,"context_line":"        while (not instance or"},{"line_number":4460,"context_line":"               instance.vm_state \u003d\u003d vm_states.BUILDING or"},{"line_number":4461,"context_line":"               instance.task_state \u003d\u003d task_states.DELETING or"},{"line_number":4462,"context_line":"               instance.host !\u003d self.host):"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAATn%2F%2BjFw%3D","line":4459,"in_reply_to":"AAAATn%2F%2BjVE%3D","updated":"2013-11-24 08:18:36.000000000","message":"Yep - good spot.  If all instances were in Building or Deleting state we would spin here.   I don\u0027t really like moving the test down though as that would waste a run of the periodic task when it might be able to update a valid instance.\n\nI think if I also use the same checks to pop entries from db_instances before L4475 then we can break out at the right place.   Maybe the Building check can just be there - since its always the initial state of an instance (i.e. you can\u0027t go from not-building into building) \n\nCheck for deleting still needs to be here though because the state can change between the uuid list being created in one run and an entry being processed several minutes latter.","commit_id":"ad1837779b843b8f22503752517b5778149284b1"},{"author":{"_account_id":5638,"name":"Davanum Srinivas","email":"davanum@gmail.com","username":"dims-v"},"change_message_id":"1deac82fac4583cc9ca564416bb737502d9dbd66","unresolved":false,"context_lines":[{"line_number":4484,"context_line":"                # We don\u0027t want to refersh the cache for instances"},{"line_number":4485,"context_line":"                # which are building or deleting so remove them now"},{"line_number":4486,"context_line":"                # to avoid doing another get_by_uuid on them"},{"line_number":4487,"context_line":"                instance_uuids \u003d []"},{"line_number":4488,"context_line":"                for inst in db_instances:"},{"line_number":4489,"context_line":"                    if (inst.vm_state !\u003d vm_states.BUILDING and"},{"line_number":4490,"context_line":"                            inst.task_state !\u003d task_states.DELETING):"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAATn%2F%2BiTI%3D","line":4487,"updated":"2013-11-24 23:47:58.000000000","message":"Should we re-init instance here as well? (just like line 4455 above)\n\n    instance \u003d None","commit_id":"13dbc3534ad4d3711537c1334fab04ad9cbbbf74"},{"author":{"_account_id":1501,"name":"Phil Day","email":"philip.day@hp.com","username":"philip-day"},"change_message_id":"bf5e096afbe68e6dbd61acde602dd096b41e59c3","unresolved":false,"context_lines":[{"line_number":4484,"context_line":"                # We don\u0027t want to refersh the cache for instances"},{"line_number":4485,"context_line":"                # which are building or deleting so remove them now"},{"line_number":4486,"context_line":"                # to avoid doing another get_by_uuid on them"},{"line_number":4487,"context_line":"                instance_uuids \u003d []"},{"line_number":4488,"context_line":"                for inst in db_instances:"},{"line_number":4489,"context_line":"                    if (inst.vm_state !\u003d vm_states.BUILDING and"},{"line_number":4490,"context_line":"                            inst.task_state !\u003d task_states.DELETING):"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAATn%2F%2Be5s%3D","line":4487,"in_reply_to":"AAAATn%2F%2BiTI%3D","updated":"2013-11-25 16:12:22.000000000","message":"Done","commit_id":"13dbc3534ad4d3711537c1334fab04ad9cbbbf74"},{"author":{"_account_id":4395,"name":"Aaron Rosen","email":"aaronorosen@gmail.com","username":"arosen"},"change_message_id":"bd9eba80764d2009bcc29b7371f3869b6daff073","unresolved":false,"context_lines":[{"line_number":4464,"context_line":"                try:"},{"line_number":4465,"context_line":"                    instance \u003d instance_obj.Instance.get_by_uuid("},{"line_number":4466,"context_line":"                        context, instance_uuids.pop(0),"},{"line_number":4467,"context_line":"                        expected_attrs\u003d[\u0027system_metadata\u0027])"},{"line_number":4468,"context_line":"                except exception.InstanceNotFound:"},{"line_number":4469,"context_line":"                    # Instance is gone.  Try to grab another."},{"line_number":4470,"context_line":"                    continue"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAATn%2F9Zlo%3D","line":4467,"updated":"2013-12-11 22:24:51.000000000","message":"nit: perhaps we should move the check that\u0027s done on line 4460-4461 to here with the comment of why we skip the instance if the instance is being deleted or has moved. (It took me a little while to figure out the flow of how this works and i think moving that here would be more obvious. \n\nFree free to ignore this comment though (as it could just be me ...and it being friday :)).","commit_id":"f920b64ad575e96f0fe16bf4e5f1395f86a208bb"},{"author":{"_account_id":642,"name":"Matt Dietz","email":"matthew.dietz@gmail.com","username":"cerberus"},"change_message_id":"fd3e64d1d880c400051817af9302ea93549d8570","unresolved":false,"context_lines":[{"line_number":4481,"context_line":"                    # None.. just return."},{"line_number":4482,"context_line":"                    return"},{"line_number":4483,"context_line":"                generated_uuid_list \u003d True"},{"line_number":4484,"context_line":"                # We don\u0027t want to refersh the cache for instances"},{"line_number":4485,"context_line":"                # which are building or deleting so remove them now"},{"line_number":4486,"context_line":"                # to avoid doing another get_by_uuid on them"},{"line_number":4487,"context_line":"                instance_uuids \u003d []"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAUH%2F%2BDd0%3D","line":4484,"updated":"2014-01-30 16:24:58.000000000","message":"femto-nit: \"refresh\" is misspelled","commit_id":"f920b64ad575e96f0fe16bf4e5f1395f86a208bb"},{"author":{"_account_id":4395,"name":"Aaron Rosen","email":"aaronorosen@gmail.com","username":"arosen"},"change_message_id":"bd9eba80764d2009bcc29b7371f3869b6daff073","unresolved":false,"context_lines":[{"line_number":4484,"context_line":"                # We don\u0027t want to refersh the cache for instances"},{"line_number":4485,"context_line":"                # which are building or deleting so remove them now"},{"line_number":4486,"context_line":"                # to avoid doing another get_by_uuid on them"},{"line_number":4487,"context_line":"                instance_uuids \u003d []"},{"line_number":4488,"context_line":"                instance \u003d None"},{"line_number":4489,"context_line":"                for inst in db_instances:"},{"line_number":4490,"context_line":"                    if (inst.vm_state !\u003d vm_states.BUILDING and"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAATn%2F9Zqk%3D","line":4487,"updated":"2013-12-11 22:24:51.000000000","message":"nit: We could remove this line if we set line 4454 to: \n\ninstance_uuids \u003d getattr(self, \u0027_instance_uuids_to_heal\u0027, [])","commit_id":"f920b64ad575e96f0fe16bf4e5f1395f86a208bb"},{"author":{"_account_id":1501,"name":"Phil Day","email":"philip.day@hp.com","username":"philip-day"},"change_message_id":"7fcc4d21f4090854492333f46d25a3d58a6ad357","unresolved":false,"context_lines":[{"line_number":4484,"context_line":"                # We don\u0027t want to refersh the cache for instances"},{"line_number":4485,"context_line":"                # which are building or deleting so remove them now"},{"line_number":4486,"context_line":"                # to avoid doing another get_by_uuid on them"},{"line_number":4487,"context_line":"                instance_uuids \u003d []"},{"line_number":4488,"context_line":"                instance \u003d None"},{"line_number":4489,"context_line":"                for inst in db_instances:"},{"line_number":4490,"context_line":"                    if (inst.vm_state !\u003d vm_states.BUILDING and"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAUH%2F9oso%3D","line":4487,"in_reply_to":"AAAATn%2F9Zqk%3D","updated":"2014-02-05 13:35:17.000000000","message":"I don\u0027t think we can, as line 4454 is outside the loop.   This is building a new (refreshed) list of uuids to work through.","commit_id":"f920b64ad575e96f0fe16bf4e5f1395f86a208bb"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"5938be35db06d2e20732a9548dce82912defae7b","unresolved":false,"context_lines":[{"line_number":4487,"context_line":"                instance_uuids \u003d []"},{"line_number":4488,"context_line":"                instance \u003d None"},{"line_number":4489,"context_line":"                for inst in db_instances:"},{"line_number":4490,"context_line":"                    if (inst.vm_state !\u003d vm_states.BUILDING and"},{"line_number":4491,"context_line":"                            inst.task_state !\u003d task_states.DELETING):"},{"line_number":4492,"context_line":"                        if not instance:"},{"line_number":4493,"context_line":"                            instance \u003d inst"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAUH%2F%2BEog%3D","line":4490,"updated":"2014-01-30 13:03:33.000000000","message":"My head hurts reading this, can we refactor this so its more readable?\n\nIt feels like this check should be shared between the two cases. I know its slightly different, but only because of a deep understanding of the nova state machine can prove that to be true. I think combining the two checks should work, but I might be missing something.","commit_id":"f920b64ad575e96f0fe16bf4e5f1395f86a208bb"},{"author":{"_account_id":1501,"name":"Phil Day","email":"philip.day@hp.com","username":"philip-day"},"change_message_id":"7fcc4d21f4090854492333f46d25a3d58a6ad357","unresolved":false,"context_lines":[{"line_number":4487,"context_line":"                instance_uuids \u003d []"},{"line_number":4488,"context_line":"                instance \u003d None"},{"line_number":4489,"context_line":"                for inst in db_instances:"},{"line_number":4490,"context_line":"                    if (inst.vm_state !\u003d vm_states.BUILDING and"},{"line_number":4491,"context_line":"                            inst.task_state !\u003d task_states.DELETING):"},{"line_number":4492,"context_line":"                        if not instance:"},{"line_number":4493,"context_line":"                            instance \u003d inst"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAUH%2F9or0%3D","line":4490,"in_reply_to":"AAAAUH%2F%2BDd8%3D","updated":"2014-02-05 13:35:17.000000000","message":"I take the point that the loop is a tad convoluted, I\u0027ll see if I can untangle it some","commit_id":"f920b64ad575e96f0fe16bf4e5f1395f86a208bb"},{"author":{"_account_id":642,"name":"Matt Dietz","email":"matthew.dietz@gmail.com","username":"cerberus"},"change_message_id":"fd3e64d1d880c400051817af9302ea93549d8570","unresolved":false,"context_lines":[{"line_number":4487,"context_line":"                instance_uuids \u003d []"},{"line_number":4488,"context_line":"                instance \u003d None"},{"line_number":4489,"context_line":"                for inst in db_instances:"},{"line_number":4490,"context_line":"                    if (inst.vm_state !\u003d vm_states.BUILDING and"},{"line_number":4491,"context_line":"                            inst.task_state !\u003d task_states.DELETING):"},{"line_number":4492,"context_line":"                        if not instance:"},{"line_number":4493,"context_line":"                            instance \u003d inst"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAUH%2F%2BDd8%3D","line":4490,"in_reply_to":"AAAAUH%2F%2BEog%3D","updated":"2014-01-30 16:24:58.000000000","message":"I\u0027m not as convinced as John that it can be rewritten into one case (though I\u0027d love to see) but I\u0027ll absolutely second the \"This is hard to follow\" I think tackling some of the nits Aaron pointed out would help, too.","commit_id":"f920b64ad575e96f0fe16bf4e5f1395f86a208bb"},{"author":{"_account_id":1501,"name":"Phil Day","email":"philip.day@hp.com","username":"philip-day"},"change_message_id":"7fcc4d21f4090854492333f46d25a3d58a6ad357","unresolved":false,"context_lines":[{"line_number":4487,"context_line":"                instance_uuids \u003d []"},{"line_number":4488,"context_line":"                instance \u003d None"},{"line_number":4489,"context_line":"                for inst in db_instances:"},{"line_number":4490,"context_line":"                    if (inst.vm_state !\u003d vm_states.BUILDING and"},{"line_number":4491,"context_line":"                            inst.task_state !\u003d task_states.DELETING):"},{"line_number":4492,"context_line":"                        if not instance:"},{"line_number":4493,"context_line":"                            instance \u003d inst"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAUH%2F9osE%3D","line":4490,"in_reply_to":"AAAAUH%2F%2BEog%3D","updated":"2014-02-05 13:35:17.000000000","message":"I\u0027m not really sure what two cases you\u0027re referring to ?\nWhat this check is doing is not adding instances that are building or deleting to the list of instance uuids that we need to check for.  In the case of deleting, we never need to check, in the case of building we don\u0027t need to check yet (but will add them to the list the next time we rebuild it).","commit_id":"f920b64ad575e96f0fe16bf4e5f1395f86a208bb"},{"author":{"_account_id":8651,"name":"Chen Xiao","email":"chenxiao@cn.ibm.com","username":"chenxiao"},"change_message_id":"652b852e5d61cf2e0cbe5510cad97453c5c4d631","unresolved":false,"context_lines":[{"line_number":4538,"context_line":"            LOG.debug(_(\u0027Rebuilding the list of instances to heal\u0027))"},{"line_number":4539,"context_line":"            db_instances \u003d instance_obj.InstanceList.get_by_host("},{"line_number":4540,"context_line":"                context, self.host, expected_attrs\u003d[], use_slave\u003dTrue)"},{"line_number":4541,"context_line":"            if db_instances:"},{"line_number":4542,"context_line":"                # We don\u0027t want to refersh the cache for instances"},{"line_number":4543,"context_line":"                # which are building or deleting so don\u0027t put them"},{"line_number":4544,"context_line":"                # in the list.   If they are building they will get"}],"source_content_type":"text/x-python","patch_set":8,"id":"AAAAVX%2F%2FpT0%3D","line":4541,"updated":"2014-03-02 03:19:24.000000000","message":"I think do not need this line, if db_instances is [], \"for inst in db_instances\" can work too.","commit_id":"48b57652862a04a8a2194dec6017a41401202c17"},{"author":{"_account_id":8651,"name":"Chen Xiao","email":"chenxiao@cn.ibm.com","username":"chenxiao"},"change_message_id":"652b852e5d61cf2e0cbe5510cad97453c5c4d631","unresolved":false,"context_lines":[{"line_number":4541,"context_line":"            if db_instances:"},{"line_number":4542,"context_line":"                # We don\u0027t want to refersh the cache for instances"},{"line_number":4543,"context_line":"                # which are building or deleting so don\u0027t put them"},{"line_number":4544,"context_line":"                # in the list.   If they are building they will get"},{"line_number":4545,"context_line":"                # added to the list next time we build it. "},{"line_number":4546,"context_line":"                for inst in db_instances:"},{"line_number":4547,"context_line":"                    if (inst.vm_state \u003d\u003d vm_states.BUILDING or"}],"source_content_type":"text/x-python","patch_set":8,"id":"AAAAVX%2F%2FpTg%3D","line":4544,"updated":"2014-03-02 03:19:24.000000000","message":"delete some blank space before \"If\u0027","commit_id":"48b57652862a04a8a2194dec6017a41401202c17"},{"author":{"_account_id":1030,"name":"Chris Behrens","email":"cbehrens@codestud.com","username":"cbehrens"},"change_message_id":"a710b5a5c41dcabadf56b3d0b1a4035322195e75","unresolved":false,"context_lines":[{"line_number":4731,"context_line":"                if (inst.vm_state \u003d\u003d vm_states.BUILDING or"},{"line_number":4732,"context_line":"                        inst.task_state \u003d\u003d task_states.DELETING):"},{"line_number":4733,"context_line":"                    LOG.debug(_(\u0027Skipping cache update for instance\u0027),"},{"line_number":4734,"context_line":"                              instance\u003dinst)"},{"line_number":4735,"context_line":"                else:"},{"line_number":4736,"context_line":"                    if not instance:"},{"line_number":4737,"context_line":"                        # Save the first one we find so we don\u0027t"}],"source_content_type":"text/x-python","patch_set":9,"id":"AAAAWH%2F%2Fjhs%3D","line":4734,"updated":"2014-03-17 03:56:28.000000000","message":"you could \u0027continue\u0027 here so you don\u0027t need the else below and another indentation level with the if contained within... but that\u0027s just a nit.","commit_id":"c87e2ca48a54ef5ddc51eca29e5325e14a95bc20"},{"author":{"_account_id":1501,"name":"Phil Day","email":"philip.day@hp.com","username":"philip-day"},"change_message_id":"440c410a0311b3037a52d7f1ac13fcd0d3b911e6","unresolved":false,"context_lines":[{"line_number":4731,"context_line":"                if (inst.vm_state \u003d\u003d vm_states.BUILDING or"},{"line_number":4732,"context_line":"                        inst.task_state \u003d\u003d task_states.DELETING):"},{"line_number":4733,"context_line":"                    LOG.debug(_(\u0027Skipping cache update for instance\u0027),"},{"line_number":4734,"context_line":"                              instance\u003dinst)"},{"line_number":4735,"context_line":"                else:"},{"line_number":4736,"context_line":"                    if not instance:"},{"line_number":4737,"context_line":"                        # Save the first one we find so we don\u0027t"}],"source_content_type":"text/x-python","patch_set":9,"id":"AAAAWH%2F%2FWJc%3D","line":4734,"in_reply_to":"AAAAWH%2F%2Fjhs%3D","updated":"2014-03-18 13:37:31.000000000","message":"Done","commit_id":"c87e2ca48a54ef5ddc51eca29e5325e14a95bc20"},{"author":{"_account_id":1030,"name":"Chris Behrens","email":"cbehrens@codestud.com","username":"cbehrens"},"change_message_id":"a710b5a5c41dcabadf56b3d0b1a4035322195e75","unresolved":false,"context_lines":[{"line_number":4758,"context_line":"                if (inst.host !\u003d self.host or"},{"line_number":4759,"context_line":"                        inst.task_state \u003d\u003d task_states.DELETING):"},{"line_number":4760,"context_line":"                    LOG.debug(_(\u0027Skipping cache update for instance\u0027),"},{"line_number":4761,"context_line":"                              instance\u003dinst)"},{"line_number":4762,"context_line":"                else:"},{"line_number":4763,"context_line":"                    instance \u003d inst"},{"line_number":4764,"context_line":"                    break"}],"source_content_type":"text/x-python","patch_set":9,"id":"AAAAWH%2F%2Fjhc%3D","line":4761,"updated":"2014-03-17 03:56:28.000000000","message":"Might be good to know why it was skipped...","commit_id":"c87e2ca48a54ef5ddc51eca29e5325e14a95bc20"},{"author":{"_account_id":1501,"name":"Phil Day","email":"philip.day@hp.com","username":"philip-day"},"change_message_id":"440c410a0311b3037a52d7f1ac13fcd0d3b911e6","unresolved":false,"context_lines":[{"line_number":4758,"context_line":"                if (inst.host !\u003d self.host or"},{"line_number":4759,"context_line":"                        inst.task_state \u003d\u003d task_states.DELETING):"},{"line_number":4760,"context_line":"                    LOG.debug(_(\u0027Skipping cache update for instance\u0027),"},{"line_number":4761,"context_line":"                              instance\u003dinst)"},{"line_number":4762,"context_line":"                else:"},{"line_number":4763,"context_line":"                    instance \u003d inst"},{"line_number":4764,"context_line":"                    break"}],"source_content_type":"text/x-python","patch_set":9,"id":"AAAAWH%2F%2FWJU%3D","line":4761,"in_reply_to":"AAAAWH%2F%2Fjhc%3D","updated":"2014-03-18 13:37:31.000000000","message":"Done","commit_id":"c87e2ca48a54ef5ddc51eca29e5325e14a95bc20"},{"author":{"_account_id":1030,"name":"Chris Behrens","email":"cbehrens@codestud.com","username":"cbehrens"},"change_message_id":"a710b5a5c41dcabadf56b3d0b1a4035322195e75","unresolved":false,"context_lines":[{"line_number":4775,"context_line":"                LOG.error(_(\"An error occurred while refreshing the network \""},{"line_number":4776,"context_line":"                          \"cache.\"), instance\u003dinstance, exc_info\u003dTrue)"},{"line_number":4777,"context_line":"        else:"},{"line_number":4778,"context_line":"            LOG.debug(_(\"Didn\u0027t find any instances for cache update\"))"},{"line_number":4779,"context_line":""},{"line_number":4780,"context_line":"    @periodic_task.periodic_task"},{"line_number":4781,"context_line":"    def _poll_rebooting_instances(self, context):"}],"source_content_type":"text/x-python","patch_set":9,"id":"AAAAWH%2F%2FjhU%3D","line":4778,"updated":"2014-03-17 03:56:28.000000000","message":"Might be good to make this say \u0027network cache\u0027, also, just for clarity. I think it\u0027s currently the only cache update we do, so it\u0027s okay for now.","commit_id":"c87e2ca48a54ef5ddc51eca29e5325e14a95bc20"},{"author":{"_account_id":1501,"name":"Phil Day","email":"philip.day@hp.com","username":"philip-day"},"change_message_id":"440c410a0311b3037a52d7f1ac13fcd0d3b911e6","unresolved":false,"context_lines":[{"line_number":4775,"context_line":"                LOG.error(_(\"An error occurred while refreshing the network \""},{"line_number":4776,"context_line":"                          \"cache.\"), instance\u003dinstance, exc_info\u003dTrue)"},{"line_number":4777,"context_line":"        else:"},{"line_number":4778,"context_line":"            LOG.debug(_(\"Didn\u0027t find any instances for cache update\"))"},{"line_number":4779,"context_line":""},{"line_number":4780,"context_line":"    @periodic_task.periodic_task"},{"line_number":4781,"context_line":"    def _poll_rebooting_instances(self, context):"}],"source_content_type":"text/x-python","patch_set":9,"id":"AAAAWH%2F%2FWJQ%3D","line":4778,"in_reply_to":"AAAAWH%2F%2FjhU%3D","updated":"2014-03-18 13:37:31.000000000","message":"Done","commit_id":"c87e2ca48a54ef5ddc51eca29e5325e14a95bc20"},{"author":{"_account_id":1030,"name":"Chris Behrens","email":"cbehrens@codestud.com","username":"cbehrens"},"change_message_id":"76dccc1489f8c3fd6e9866f85876d130bc7d2367","unresolved":false,"context_lines":[{"line_number":4819,"context_line":"                    continue"},{"line_number":4820,"context_line":"                if (inst.task_state \u003d\u003d task_states.DELETING):"},{"line_number":4821,"context_line":"                    LOG.debug(_(\u0027Skipping network cache update for instance \u0027"},{"line_number":4822,"context_line":"                                \u0027beacuse it is being deleted.\u0027), instance\u003dinst)"},{"line_number":4823,"context_line":"                    continue"},{"line_number":4824,"context_line":""},{"line_number":4825,"context_line":"                if not instance:"}],"source_content_type":"text/x-python","patch_set":10,"id":"AAAAWH%2F%2FTHY%3D","line":4822,"updated":"2014-03-18 19:14:35.000000000","message":"hehe, well now \u0027because\u0027 is typoed twice here ;)","commit_id":"6f68f90a99643047153bfb374519f104b47fbcfc"},{"author":{"_account_id":1501,"name":"Phil Day","email":"philip.day@hp.com","username":"philip-day"},"change_message_id":"653e41edea94fe8a95e0cb1414b92bdc0cee8b5d","unresolved":false,"context_lines":[{"line_number":4819,"context_line":"                    continue"},{"line_number":4820,"context_line":"                if (inst.task_state \u003d\u003d task_states.DELETING):"},{"line_number":4821,"context_line":"                    LOG.debug(_(\u0027Skipping network cache update for instance \u0027"},{"line_number":4822,"context_line":"                                \u0027beacuse it is being deleted.\u0027), instance\u003dinst)"},{"line_number":4823,"context_line":"                    continue"},{"line_number":4824,"context_line":""},{"line_number":4825,"context_line":"                if not instance:"}],"source_content_type":"text/x-python","patch_set":10,"id":"AAAAWH%2F%2FRyA%3D","line":4822,"in_reply_to":"AAAAWH%2F%2FTHY%3D","updated":"2014-03-18 22:38:50.000000000","message":"Doh ;-(","commit_id":"6f68f90a99643047153bfb374519f104b47fbcfc"}]}