)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"943a4cbbacfe79a213f86ea2999b13ecb16dbcc7","unresolved":true,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Remove nonexistent [backend] backend_host"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"The [backend] backend_host option has never been implemented in cinder"},{"line_number":10,"context_line":"but it was wrongly added to the cinder.conf file generated by"},{"line_number":11,"context_line":"oslo-config-generator."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Fix the script to generate the opts.py file to exclude the option."},{"line_number":14,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":8,"id":"f3210ba8_bc3cfbb1","line":11,"range":{"start_line":9,"start_character":0,"end_line":11,"end_character":22},"updated":"2025-01-18 01:01:23.000000000","message":"I think this is a misreading of what\u0027s going on.  The \u0027backend_host\u0027 option is implemented (see cinder/cmd/volume.py), it\u0027s just that the code won\u0027t look for a \u0027backend\u0027 group (unless someone has named their backend, \u0027backend\u0027, i guess).  I think the idea was that we want this option\u0027s help text to appear somewhere, and we want to illustrate that it needs to be in its own group.","commit_id":"c1645f8173fdff61fa71c15f7ba5ede46147673f"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":31016,"name":"Ivan Pchelintsev","email":"i.pchelintsev@yadro.com","username":"pcheli"},"change_message_id":"6649d98724d53848833c4a9d040d8d4829928a2c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"396591ed_6c45e1e3","updated":"2024-09-23 15:49:15.000000000","message":"run-Yadro Tatlin Unified CI","commit_id":"6a816eaa247759c447b0337576c58e289ea6d976"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"943a4cbbacfe79a213f86ea2999b13ecb16dbcc7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"bf8032e0_8e9a4aa6","updated":"2025-01-18 01:01:23.000000000","message":"I can see how this is confusing, but I don\u0027t think we want to completely delete the option from the sample configuration.  See comments inline.","commit_id":"c1645f8173fdff61fa71c15f7ba5ede46147673f"}],"cinder/cmd/volume.py":[{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"943a4cbbacfe79a213f86ea2999b13ecb16dbcc7","unresolved":true,"context_lines":[{"line_number":95,"context_line":"def _launch_service(launcher: \u0027oslo_service.ProcessLauncher\u0027,"},{"line_number":96,"context_line":"                    backend: str) -\u003e None:"},{"line_number":97,"context_line":"    CONF.register_opt(host_opt, group\u003dbackend)"},{"line_number":98,"context_line":"    backend_host \u003d getattr(CONF, backend).backend_host"},{"line_number":99,"context_line":"    host \u003d \"%s@%s\" % (backend_host or CONF.host, backend)"},{"line_number":100,"context_line":"    # We also want to set cluster to None on empty strings, and we"},{"line_number":101,"context_line":"    # ignore leading and trailing spaces."}],"source_content_type":"text/x-python","patch_set":8,"id":"cea60a01_2d933b01","line":98,"range":{"start_line":98,"start_character":4,"end_line":98,"end_character":54},"updated":"2025-01-18 01:01:23.000000000","message":"Here\u0027s the implementation.","commit_id":"c1645f8173fdff61fa71c15f7ba5ede46147673f"}],"cinder/opts.py":[{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"943a4cbbacfe79a213f86ea2999b13ecb16dbcc7","unresolved":true,"context_lines":[{"line_number":217,"context_line":""},{"line_number":218,"context_line":"def list_opts():"},{"line_number":219,"context_line":"    return ["},{"line_number":220,"context_line":"        (\u0027backend\u0027,"},{"line_number":221,"context_line":"            itertools.chain("},{"line_number":222,"context_line":"                [cinder_cmd_volume.host_opt],"},{"line_number":223,"context_line":"            )),"}],"source_content_type":"text/x-python","patch_set":8,"id":"92bd205a_423f3792","side":"PARENT","line":220,"updated":"2025-01-18 01:01:23.000000000","message":"Instead of deleting this, I suggest doing what the two following items do and rename it \u0027backend_example\u0027.","commit_id":"4aadc0784d33a616340ca0340f79ef2124bd581f"}]}
