)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":12999,"name":"James Arendt","email":"james.arendt@hpe.com","username":"jwarendt"},"change_message_id":"3206f8d3f040ab90f18b0fc5d51d8cb42f131f8a","unresolved":false,"context_lines":[{"line_number":5,"context_line":"CommitDate: 2016-01-28 15:11:05 -0800"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Replace depcricated Nova networks with Nova interfaces"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Nova network (os-networks) is depcrecated and thus it\u0027s"},{"line_number":10,"context_line":"usage should be replaced by Nova interface (os-interface)."},{"line_number":11,"context_line":"Create corresponding test cases."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"7a5de9d1_2d154a3f","line":8,"updated":"2016-01-29 01:02:00.000000000","message":"Nit: deprecated","commit_id":"66f0f23764b3482e7d332152fd313ee7616d420a"},{"author":{"_account_id":12999,"name":"James Arendt","email":"james.arendt@hpe.com","username":"jwarendt"},"change_message_id":"3206f8d3f040ab90f18b0fc5d51d8cb42f131f8a","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Replace depcricated Nova networks with Nova interfaces"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Nova network (os-networks) is depcrecated and thus it\u0027s"},{"line_number":10,"context_line":"usage should be replaced by Nova interface (os-interface)."},{"line_number":11,"context_line":"Create corresponding test cases."},{"line_number":12,"context_line":"Ref url: http://developer.openstack.org/api-ref-compute-v2.1.html#listAttachedInterfaces"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"7a5de9d1_4d3766e3","line":9,"updated":"2016-01-29 01:02:00.000000000","message":"Nit: deprecated","commit_id":"66f0f23764b3482e7d332152fd313ee7616d420a"}],"octavia/compute/drivers/nova_driver.py":[{"author":{"_account_id":6951,"name":"Brandon Logan","email":"brandon.logan@rackspace.com","username":"brandon-logan"},"change_message_id":"d34f79de5d0df30aca0a9a2ee1dc643b27a0027a","unresolved":false,"context_lines":[{"line_number":140,"context_line":"        # fields"},{"line_number":141,"context_line":""},{"line_number":142,"context_line":"        try:"},{"line_number":143,"context_line":"            inf_list \u003d self.manager.interface_list(server\u003dnova_response.id)"},{"line_number":144,"context_line":"            lb_network_ip \u003d None"},{"line_number":145,"context_line":"            for interface in inf_list:"},{"line_number":146,"context_line":"                if (getattr(interface, \u0027net_id\u0027) is"}],"source_content_type":"text/x-python","patch_set":3,"id":"7a5de9d1_609d8458","line":143,"updated":"2016-01-29 03:40:42.000000000","message":"we need to support the old way and this new way.  This will break some clouds, including ours.  I believe we wanted to use this in beginning but it wouldn\u0027t work for everyone.  I\u0027m okay if this is the first method tried but if a 404 exception is returned from this call then the old way should be attempted as well.","commit_id":"66f0f23764b3482e7d332152fd313ee7616d420a"},{"author":{"_account_id":12999,"name":"James Arendt","email":"james.arendt@hpe.com","username":"jwarendt"},"change_message_id":"064d48333ce33390ccadb46d3933fec0d31d5192","unresolved":false,"context_lines":[{"line_number":156,"context_line":""},{"line_number":157,"context_line":"        if not lb_network_ip:"},{"line_number":158,"context_line":"            # Try networks extension"},{"line_number":159,"context_line":"            # TODO Remove when RAX doesn\u0027t need that any longer"},{"line_number":160,"context_line":"            try:"},{"line_number":161,"context_line":"                net_name \u003d self._nova_client.networks.get("},{"line_number":162,"context_line":"                CONF.controller_worker.amp_network).label"}],"source_content_type":"text/x-python","patch_set":6,"id":"7a5de9d1_166ca806","line":159,"updated":"2016-02-01 19:15:54.000000000","message":"Nit: The OpenStack style guide at http://docs.openstack.org/developer/hacking/ wants a username with a TODO - i.e. [H101] Include your name with TODOs as in # TODO(yourname)","commit_id":"f5319c27a258b2e08dd799934cf52d7e88aa7bbf"}],"octavia/controller/worker/tasks/compute_tasks.py":[{"author":{"_account_id":12999,"name":"James Arendt","email":"james.arendt@hpe.com","username":"jwarendt"},"change_message_id":"064d48333ce33390ccadb46d3933fec0d31d5192","unresolved":false,"context_lines":[{"line_number":157,"context_line":"        time.sleep(CONF.controller_worker.amp_active_wait_sec)"},{"line_number":158,"context_line":"        amp \u003d self.compute.get_amphora(compute_id)"},{"line_number":159,"context_line":"        if amp.status \u003d\u003d constants.ACTIVE and amp.lb_network_ip:"},{"line_number":160,"context_line":"            return amp"},{"line_number":161,"context_line":""},{"line_number":162,"context_line":"        raise exceptions.ComputeWaitTimeoutException()"}],"source_content_type":"text/x-python","patch_set":6,"id":"7a5de9d1_f60a2c25","line":160,"updated":"2016-02-01 19:15:54.000000000","message":"Isn\u0027t not having a lb_network_ip a different error than not transitioning to ACTIVE?  This lumps both under a WaitTimeout - would prefer a separate exception for not able to get IP.","commit_id":"f5319c27a258b2e08dd799934cf52d7e88aa7bbf"}]}