)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":21129,"name":"Alan Bishop","email":"abishopsweng@gmail.com","username":"ASBishop","status":"ex Red Hat"},"change_message_id":"b29e232a31c87b1c474d67b5ea850f957db99fac","unresolved":true,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Fix in api-ref"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"\u0027snapshot\u0027 instead of \u0027backup\u0027 for force request param in"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"https: //docs.openstack.org/api-ref/block-storage/v3/?expanded\u003dcreate"},{"line_number":12,"context_line":"Change-Id: Ib6a153562aec1c73ceb7ad9d4f6c4ee104d768e2"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"ac3725a1_6c000534","line":9,"updated":"2021-02-22 13:39:04.000000000","message":"-1: This statement is no longer correct.","commit_id":"129fbfacbf55b234f1dfdc5728c3bf3959fc083e"}],"api-ref/source/v3/parameters.yaml":[{"author":{"_account_id":21129,"name":"Alan Bishop","email":"abishopsweng@gmail.com","username":"ASBishop","status":"ex Red Hat"},"change_message_id":"405a646072f08a58b689cf505dac0ccd302e7710","unresolved":true,"context_lines":[{"line_number":1178,"context_line":"  type: string"},{"line_number":1179,"context_line":"force:"},{"line_number":1180,"context_line":"  description: |"},{"line_number":1181,"context_line":"    Indicates whether to snapshot, even if the volume"},{"line_number":1182,"context_line":"    is attached. Default is ``false``."},{"line_number":1183,"context_line":"    See :ref:`valid boolean values \u003cvalid-boolean-values\u003e`"},{"line_number":1184,"context_line":"  in: body"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"190f937b_41bea29a","line":1181,"updated":"2021-02-19 15:03:48.000000000","message":"The problem is this parameter is used by both operations, so changing this word to \"snapshots\" makes the documentation for the backup operation incorrect.\n\nHow about \"Indicates whether to create a backup or snapshot even if the volume is attached.","commit_id":"60f65c69427e8e797d211bdae46ff651efd2bbb7"},{"author":{"_account_id":20813,"name":"Sofia Enriquez","email":"lsofia.enriquez@gmail.com","username":"enriquetaso"},"change_message_id":"464c0f002c5d48bfdc18c02fc6a45774d13968b1","unresolved":true,"context_lines":[{"line_number":1178,"context_line":"  type: string"},{"line_number":1179,"context_line":"force:"},{"line_number":1180,"context_line":"  description: |"},{"line_number":1181,"context_line":"    Indicates whether to snapshot, even if the volume"},{"line_number":1182,"context_line":"    is attached. Default is ``false``."},{"line_number":1183,"context_line":"    See :ref:`valid boolean values \u003cvalid-boolean-values\u003e`"},{"line_number":1184,"context_line":"  in: body"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"2908bc09_f8a64bed","line":1181,"in_reply_to":"190f937b_41bea29a","updated":"2021-02-19 15:49:27.000000000","message":"+1","commit_id":"60f65c69427e8e797d211bdae46ff651efd2bbb7"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"ff02f2f3050054be3dcc9b9601ced3d7af838c23","unresolved":true,"context_lines":[{"line_number":1178,"context_line":"  type: string"},{"line_number":1179,"context_line":"force:"},{"line_number":1180,"context_line":"  description: |"},{"line_number":1181,"context_line":"    Indicates whether to snapshot, even if the volume"},{"line_number":1182,"context_line":"    is attached. Default is ``false``."},{"line_number":1183,"context_line":"    See :ref:`valid boolean values \u003cvalid-boolean-values\u003e`"},{"line_number":1184,"context_line":"  in: body"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"9c1dd243_80aa2b74","line":1181,"in_reply_to":"2908bc09_f8a64bed","updated":"2021-02-23 16:14:19.000000000","message":"I think we should create a separate var here for snapshot and not to combine both API in single. reading \u0027backup\u0027 word in snapshot API still confusing the user. like below\n\nforce_snapshot:\n  description: |\n    Indicates whether to snapshot, even if the volume\n    is attached. Default is ``false``.\n    See :ref:`valid boolean values \u003cvalid-boolean-values\u003e`\n  in: body\n  required: false\n  type: boolean\n\nand use this in https://github.com/openstack/cinder/blob/master/api-ref/source/v3/volumes-v3-snapshots.inc change\n \n- force: force_snapshot","commit_id":"60f65c69427e8e797d211bdae46ff651efd2bbb7"},{"author":{"_account_id":21129,"name":"Alan Bishop","email":"abishopsweng@gmail.com","username":"ASBishop","status":"ex Red Hat"},"change_message_id":"b29e232a31c87b1c474d67b5ea850f957db99fac","unresolved":true,"context_lines":[{"line_number":1178,"context_line":"  type: string"},{"line_number":1179,"context_line":"force:"},{"line_number":1180,"context_line":"  description: |"},{"line_number":1181,"context_line":"    Indicates whether to create backup or snapshot,"},{"line_number":1182,"context_line":"    even if the volume is attached. Default is ``false``."},{"line_number":1183,"context_line":"    See :ref:`valid boolean values \u003cvalid-boolean-values\u003e`"},{"line_number":1184,"context_line":"  in: body"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"a4c6c69e_eac12c48","line":1181,"updated":"2021-02-22 13:39:04.000000000","message":"grammar nit: \"create a backup...\" (add \" a \")","commit_id":"129fbfacbf55b234f1dfdc5728c3bf3959fc083e"},{"author":{"_account_id":21129,"name":"Alan Bishop","email":"abishopsweng@gmail.com","username":"ASBishop","status":"ex Red Hat"},"change_message_id":"ae7c77bed6c59cc12a39f8b4d5ab601df1adf41b","unresolved":true,"context_lines":[{"line_number":1178,"context_line":"  type: string"},{"line_number":1179,"context_line":"force:"},{"line_number":1180,"context_line":"  description: |"},{"line_number":1181,"context_line":"    Indicates whether to backup,even if the volume"},{"line_number":1182,"context_line":"    is attached. Default is ``false``."},{"line_number":1183,"context_line":"    See :ref:`valid boolean values \u003cvalid-boolean-values\u003e`"},{"line_number":1184,"context_line":"  in: body"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"0dcfc813_e6037288","line":1181,"updated":"2021-02-24 14:49:53.000000000","message":"sorry, you inadvertently removed a space. It should be \"backup, even\"","commit_id":"d34796370d747c3239057ed63b25a5453b48e531"}]}
