)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":21129,"name":"Alan Bishop","email":"abishopsweng@gmail.com","username":"ASBishop","status":"ex Red Hat"},"change_message_id":"04a452487e7f5a188bebad677f3705cbdc43099c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"4812e543_0ebfc794","updated":"2025-01-14 18:57:22.000000000","message":"A thought to consider, but -1 for typo in the release note.","commit_id":"a2754ed8a685ababa13236498443507e642125b5"}],"manifests/scheduler.pp":[{"author":{"_account_id":21129,"name":"Alan Bishop","email":"abishopsweng@gmail.com","username":"ASBishop","status":"ex Red Hat"},"change_message_id":"04a452487e7f5a188bebad677f3705cbdc43099c","unresolved":true,"context_lines":[{"line_number":8,"context_line":"#   (Optional) Default scheduler driver to use"},{"line_number":9,"context_line":"#   Defaults to $facts[\u0027os_service_default\u0027]."},{"line_number":10,"context_line":"#"},{"line_number":11,"context_line":"# [*scheduler_driver_init_wait_time*]"},{"line_number":12,"context_line":"#   (Optional) Maximum time in seconds to wait for the driver to report as"},{"line_number":13,"context_line":"#   ready."},{"line_number":14,"context_line":"#   Defaults to $facts[\u0027os_service_default\u0027]."}],"source_content_type":"text/x-puppet","patch_set":2,"id":"7619178f_b43afff0","line":11,"updated":"2025-01-14 18:57:22.000000000","message":"I know the cinder option names include the \"scheduler_\" prefix, probably because they all appear in the [DEFAULT] section. Would it make sense to drop the prefix from the puppet variables because they\u0027d be prefaced with the \"scheduler::\" class?\n\ncinder::scheduler::driver_init_wait_time\n\nI don\u0027t have a strong position on this, it\u0027s just a thought to consider.","commit_id":"a2754ed8a685ababa13236498443507e642125b5"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"8c8f2b20c8e8404d8df37c77f63f6730fcad04ff","unresolved":false,"context_lines":[{"line_number":8,"context_line":"#   (Optional) Default scheduler driver to use"},{"line_number":9,"context_line":"#   Defaults to $facts[\u0027os_service_default\u0027]."},{"line_number":10,"context_line":"#"},{"line_number":11,"context_line":"# [*scheduler_driver_init_wait_time*]"},{"line_number":12,"context_line":"#   (Optional) Maximum time in seconds to wait for the driver to report as"},{"line_number":13,"context_line":"#   ready."},{"line_number":14,"context_line":"#   Defaults to $facts[\u0027os_service_default\u0027]."}],"source_content_type":"text/x-puppet","patch_set":2,"id":"ae5ac175_5345ee00","line":11,"in_reply_to":"7619178f_b43afff0","updated":"2025-01-15 02:09:40.000000000","message":"+1 for removing the prefix. Updated the change (and the one proposed for puppet-manila as well)","commit_id":"a2754ed8a685ababa13236498443507e642125b5"}],"releasenotes/notes/more-scheduler-opts-7f53704ed415f9be.yaml":[{"author":{"_account_id":21129,"name":"Alan Bishop","email":"abishopsweng@gmail.com","username":"ASBishop","status":"ex Red Hat"},"change_message_id":"04a452487e7f5a188bebad677f3705cbdc43099c","unresolved":true,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"features:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    The following parmaeters have been added to the ``cinder::scheduler``"},{"line_number":5,"context_line":"    class."},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"    - ``scheduler_driver_init_wait_time``"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"d671b0f0_e9b01a21","line":4,"range":{"start_line":4,"start_character":18,"end_line":4,"end_character":28},"updated":"2025-01-14 18:57:22.000000000","message":"typo: parameters","commit_id":"a2754ed8a685ababa13236498443507e642125b5"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"8c8f2b20c8e8404d8df37c77f63f6730fcad04ff","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"features:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    The following parmaeters have been added to the ``cinder::scheduler``"},{"line_number":5,"context_line":"    class."},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"    - ``scheduler_driver_init_wait_time``"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"436e8ce2_b57fad9a","line":4,"range":{"start_line":4,"start_character":18,"end_line":4,"end_character":28},"in_reply_to":"d671b0f0_e9b01a21","updated":"2025-01-15 02:09:40.000000000","message":"Done","commit_id":"a2754ed8a685ababa13236498443507e642125b5"}]}
