)]}'
{"tempest/api/compute/servers/test_create_server.py":[{"author":{"_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},"change_message_id":"ba8fdbf3dc58da92806e9dfeaca6136d20b84b95","unresolved":false,"context_lines":[{"line_number":182,"context_line":"                addresses[name_net2][0][\u0027addr\u0027]]"},{"line_number":183,"context_line":"        networks \u003d [netaddr.IPNetwork(\u002719.80.0.0/24\u0027),"},{"line_number":184,"context_line":"                    netaddr.IPNetwork(\u002719.86.0.0/24\u0027)]"},{"line_number":185,"context_line":"        for address, network in zip(addr, networks):"},{"line_number":186,"context_line":"            self.assertIn(address, network)"},{"line_number":187,"context_line":""},{"line_number":188,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"fa81d914_b8127060","line":185,"updated":"2015-01-28 07:57:40.000000000","message":"loop for two element is overkill.\nBut not good enough reason for blocking this change after 9 day.\n\nThis would be 2 line instead of 4 and also faster:\nself.assertIn(addr[0], netaddr.IPNetwork[\u002719.80.0.0/24\u0027])\nself.assertIn(addr[1], netaddr.IPNetwork[\u002719.86.0.0/24\u0027])\n\nIf you need to resin this change for whatever reason, please remove the loop.","commit_id":"bd2d49af14d6323c112204a8f8e0e53f68d745df"}]}