)]}'
{"whitebox_neutron_tempest_plugin/tests/scenario/base.py":[{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"15b671484ebb4756d9feda4a6ad93cdbcbb3ce48","unresolved":true,"context_lines":[{"line_number":587,"context_line":"            if controller_count \u003c 1 or crc_ocp_count \u003c 1 or compute_count \u003c 1:"},{"line_number":588,"context_line":"                LOG.critical("},{"line_number":589,"context_line":"                    \u0027Node discovery insufficient!\\n%s\\ncls.nodes:\\n%s\u0027,"},{"line_number":590,"context_line":"                    info_msg % tuple(node_counts),"},{"line_number":591,"context_line":"                    pprint.pformat(cls.nodes))"},{"line_number":592,"context_line":"                raise RuntimeError(\u0027Insufficient node discovery, check logs\u0027)"},{"line_number":593,"context_line":"        LOG.info(info_msg, *node_counts)"}],"source_content_type":"text/x-python","patch_set":1,"id":"a215583c_bf741cab","line":590,"updated":"2026-06-18 14:31:01.000000000","message":"why do we need to make it in such (IMHO) weird way which is still error prone as if node_counts would be different length then expected by the \"info_msg\". Can\u0027t you instead do something like:\n\n    info msg \u003d \"Discovered %d nodes: %d compute, %d controller\" % (len(cls.nodes), compute_count, controller_count)\n    if not cls.is_devstack:\n        info_msg \u003d info_msg + \", %d networker, %d OCP/CRC\" % (networker_count, crc_ocp_count)\n        ...\n\nThat would make this code more readable in the future","commit_id":"ac107b2145a820459ca776d00e8ead04652de127"}]}