)]}'
{"quantum/plugins/services/agent_loadbalancer/plugin.py":[{"author":{"_account_id":5948,"name":"Oleg Bondarev","email":"obondarev@mirantis.com","username":"obondarev"},"change_message_id":"d1df6d2d3ace888aa81f6fc8a8c343b232337e36","unresolved":false,"context_lines":[{"line_number":275,"context_line":"            pool[\u0027pool\u0027][\u0027status\u0027] \u003d constants.PENDING_UPDATE"},{"line_number":276,"context_line":"        p \u003d super(LoadBalancerPlugin, self).update_pool(context, id, pool)"},{"line_number":277,"context_line":"        if p[\u0027status\u0027] in ACTIVE_PENDING:"},{"line_number":278,"context_line":"            if p[\u0027vip_id\u0027] is not None:"},{"line_number":279,"context_line":"                self.agent_rpc.reload_pool(context, p[\u0027id\u0027])"},{"line_number":280,"context_line":"        else:"},{"line_number":281,"context_line":"            self.agent_rpc.destroy_pool(context, p[\u0027id\u0027])"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAPH%2F%2FkpI%3D","line":278,"updated":"2013-05-30 08:40:50.000000000","message":"why not having both conditions under one if?","commit_id":"75c43c6a2200f88e7c4d74d34149ad71d957c865"},{"author":{"_account_id":6072,"name":"Eugene Nikanorov","email":"enikanorov@mirantis.com","username":"enikanorov"},"change_message_id":"563b9d828f72d3a14fb7c019adc0d4f0bb3f5a5b","unresolved":false,"context_lines":[{"line_number":275,"context_line":"            pool[\u0027pool\u0027][\u0027status\u0027] \u003d constants.PENDING_UPDATE"},{"line_number":276,"context_line":"        p \u003d super(LoadBalancerPlugin, self).update_pool(context, id, pool)"},{"line_number":277,"context_line":"        if p[\u0027status\u0027] in ACTIVE_PENDING:"},{"line_number":278,"context_line":"            if p[\u0027vip_id\u0027] is not None:"},{"line_number":279,"context_line":"                self.agent_rpc.reload_pool(context, p[\u0027id\u0027])"},{"line_number":280,"context_line":"        else:"},{"line_number":281,"context_line":"            self.agent_rpc.destroy_pool(context, p[\u0027id\u0027])"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAPH%2F%2FknI%3D","line":278,"in_reply_to":"AAAAPH%2F%2FkpI%3D","updated":"2013-05-30 09:01:15.000000000","message":"For the case when p[\u0027vip_id\u0027] is None it will go to else: which i don\u0027t want","commit_id":"75c43c6a2200f88e7c4d74d34149ad71d957c865"}]}