)]}'
{"nova/compute/manager.py":[{"author":{"_account_id":5511,"name":"Nikola Dipanov","email":"ndipanov@redhat.com","username":"ndipanov"},"change_message_id":"37e16604731e8e892707dee94b698e8443c6ae99","unresolved":false,"context_lines":[{"line_number":4942,"context_line":""},{"line_number":4943,"context_line":"    def _attach_interface(self, context, instance, network_id, port_id,"},{"line_number":4944,"context_line":"                          requested_ip):"},{"line_number":4945,"context_line":"        network_info \u003d self.network_api.allocate_port_for_instance("},{"line_number":4946,"context_line":"            context, instance, port_id, network_id, requested_ip)"},{"line_number":4947,"context_line":"        if len(network_info) !\u003d 1:"},{"line_number":4948,"context_line":"            LOG.error(_LE(\u0027allocate_port_for_instance returned %(ports)s \u0027"},{"line_number":4949,"context_line":"                          \u0027ports\u0027), {\u0027ports\u0027: len(network_info)})"}],"source_content_type":"text/x-python","patch_set":1,"id":"fa69d971_303d538f","line":4946,"range":{"start_line":4945,"start_character":24,"end_line":4946,"end_character":65},"updated":"2016-01-11 19:36:32.000000000","message":"This fix puts a call to a method of another completely unrelated class inside a critical section but this method could be (and in fact is) called from a number of other places without grabbing the same lock.\n\nThis is an antipattern in concurrent programming and we should not do this.","commit_id":"b4baa58839e3b622ff520481c28cfee3b10cc288"}]}