)]}'
{"ironic_tempest_plugin/tests/scenario/baremetal_standalone_manager.py":[{"author":{"_account_id":9542,"name":"Pavlo Shchelokovskyy","email":"pshchelokovskyy@mirantis.com","username":"pshchelo"},"change_message_id":"0da0393bfd1ae4c38077ff193c79cba6e0cc21d8","unresolved":false,"context_lines":[{"line_number":150,"context_line":"        \"\"\"Removes floating IP.\"\"\""},{"line_number":151,"context_line":"        body \u003d cls.os_admin.floating_ips_client.list_floatingips()"},{"line_number":152,"context_line":"        floating_ip_id \u003d [f[\u0027id\u0027] for f in body[\u0027floatingips\u0027] if"},{"line_number":153,"context_line":"                          f[\u0027floating_ip_address\u0027] \u003d\u003d ip_address][0]"},{"line_number":154,"context_line":"        cls.os_admin.floating_ips_client.delete_floatingip(floating_ip_id)"},{"line_number":155,"context_line":""},{"line_number":156,"context_line":"    @classmethod"}],"source_content_type":"text/x-python","patch_set":1,"id":"9f91af0f_faaa7b17","line":153,"updated":"2017-12-29 15:39:30.000000000","message":"there\u0027s a small bug here - if the resource setup fails, there might not be any IPs in the list, and there is IndexError thrown.. better do it like\n\n  \n  for ip in [f[...\u003d\u003d ip_address]:\n      cls....delete_floatingip(ip)","commit_id":"857926293af867bb6f562e7fa9498477e2f07200"},{"author":{"_account_id":10343,"name":"Jim Rollenhagen","email":"jim@jimrollenhagen.com","username":"jimrollenhagen"},"change_message_id":"944b6d300b64e978ad0feb0691d5be2c1fda85cd","unresolved":false,"context_lines":[{"line_number":313,"context_line":"                    \u0027enabled_hw_types\u0027: CONF.baremetal.enabled_hardware_types})"},{"line_number":314,"context_line":"        if (cls.deploy_interface and cls.deploy_interface not in"},{"line_number":315,"context_line":"                CONF.baremetal.enabled_deploy_interfaces):"},{"line_number":316,"context_line":"            cls.skipException("},{"line_number":317,"context_line":"                \"Deploy interface %(iface)s required by test is not \""},{"line_number":318,"context_line":"                \"in the list of enabled deploy interfaces %(enabled)s\" % {"},{"line_number":319,"context_line":"                    \u0027iface\u0027: cls.deploy_interface,"}],"source_content_type":"text/x-python","patch_set":1,"id":"9f91af0f_2512085a","line":316,"updated":"2017-12-29 15:31:20.000000000","message":"Needs a \"raise\" here. :)","commit_id":"857926293af867bb6f562e7fa9498477e2f07200"},{"author":{"_account_id":9542,"name":"Pavlo Shchelokovskyy","email":"pshchelokovskyy@mirantis.com","username":"pshchelo"},"change_message_id":"0da0393bfd1ae4c38077ff193c79cba6e0cc21d8","unresolved":false,"context_lines":[{"line_number":313,"context_line":"                    \u0027enabled_hw_types\u0027: CONF.baremetal.enabled_hardware_types})"},{"line_number":314,"context_line":"        if (cls.deploy_interface and cls.deploy_interface not in"},{"line_number":315,"context_line":"                CONF.baremetal.enabled_deploy_interfaces):"},{"line_number":316,"context_line":"            cls.skipException("},{"line_number":317,"context_line":"                \"Deploy interface %(iface)s required by test is not \""},{"line_number":318,"context_line":"                \"in the list of enabled deploy interfaces %(enabled)s\" % {"},{"line_number":319,"context_line":"                    \u0027iface\u0027: cls.deploy_interface,"}],"source_content_type":"text/x-python","patch_set":1,"id":"9f91af0f_855bd42c","line":316,"in_reply_to":"9f91af0f_2512085a","updated":"2017-12-29 15:39:30.000000000","message":"ooops :)","commit_id":"857926293af867bb6f562e7fa9498477e2f07200"}]}