)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"a03068eba13b44a61c9212d384f3b7fef3cffc8a","unresolved":false,"context_lines":[{"line_number":19,"context_line":"   127.0.0.1 \"POST /v2.1/6f70656e737461636b20342065766572/servers\""},{"line_number":20,"context_line":"   status: 400 len: 164 microversion: 2.1 time: 0.129485"},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"One could argue that I8a3e7e6c4b72eb1c3707d54049d18dc29f606fe5"},{"line_number":23,"context_line":"is a behavior change that should have gone with a microversion,"},{"line_number":24,"context_line":"which if people agree with that, could be reverted separately."},{"line_number":25,"context_line":""},{"line_number":26,"context_line":"Change-Id: I14d44dbc0b6a8fb25932c333e695cad9edaefbed"},{"line_number":27,"context_line":"Closes-Bug: #1766306"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"9f6a8fd7_72f583ec","line":24,"range":{"start_line":22,"start_character":0,"end_line":24,"end_character":62},"updated":"2018-04-23 18:37:15.000000000","message":"This is fixed in the change on top. I need to keep the patches separate since adding the check back in needs to be backported to stable/queens but this docs fix is for rocky only.","commit_id":"048fa2310014e6282e183da0abee87c73b6d71b9"}],"api-ref/source/parameters.yaml":[{"author":{"_account_id":5754,"name":"Alex Xu","email":"hejie.xu@intel.com","username":"xuhj"},"change_message_id":"fb7d0b79a211fe414ad7dd9a2c6c9f0cbe4f0bc0","unresolved":false,"context_lines":[{"line_number":1812,"context_line":"    hypervisors support booting from multiple devices but only if the devices are"},{"line_number":1813,"context_line":"    of different types. For example, a disk and CD-ROM."},{"line_number":1814,"context_line":"  in: body"},{"line_number":1815,"context_line":"  required: true"},{"line_number":1816,"context_line":"  type: integer"},{"line_number":1817,"context_line":"certificate:"},{"line_number":1818,"context_line":"  description: |"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"9f6a8fd7_c93e1db7","line":1815,"range":{"start_line":1815,"start_character":2,"end_line":1815,"end_character":16},"updated":"2018-04-24 06:44:26.000000000","message":"I\u0027m not sure revert still workable for now. If we aren\u0027t going to revert, we should have some note in the api-ref for this bug.\n\nAnd we shouldn\u0027t change this before we decide to revert or not.","commit_id":"048fa2310014e6282e183da0abee87c73b6d71b9"},{"author":{"_account_id":5754,"name":"Alex Xu","email":"hejie.xu@intel.com","username":"xuhj"},"change_message_id":"55a64db531d667ca9b2683bc3bf57bf92126a39a","unresolved":false,"context_lines":[{"line_number":1812,"context_line":"    hypervisors support booting from multiple devices but only if the devices are"},{"line_number":1813,"context_line":"    of different types. For example, a disk and CD-ROM."},{"line_number":1814,"context_line":"  in: body"},{"line_number":1815,"context_line":"  required: true"},{"line_number":1816,"context_line":"  type: integer"},{"line_number":1817,"context_line":"certificate:"},{"line_number":1818,"context_line":"  description: |"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"9f6a8fd7_ee90abef","line":1815,"range":{"start_line":1815,"start_character":2,"end_line":1815,"end_character":16},"in_reply_to":"9f6a8fd7_a2f73b59","updated":"2018-04-25 07:29:06.000000000","message":"I did a test with a request:\n{\"server\": {\"name\": \"vm1\", \"imageRef\": \"\", \"block_device_mapping_v2\": [{\"source_type\": \"image\", \"volume_size\": \"1\", \"uuid\": \"09a0bbed-a17b-490e-8ecc-dd6a8a9233dc\", \"destination_type\": \"volume\"}], \"flavorRef\": \"1\", \"max_count\": 1, \"min_count\": 1, \"networks\": [{\"uuid\": \"c3a7823f-7fa1-4a37-8f2d-07d1e994fe50\"}]}}\u0027\n\nThat request doesn\u0027t provide boot index. Then I get an error instance. I see a ImageNotFound exception, but I\u0027m not sure why is that exception.\n\nThen I add boot index in the request, the instance boot correctly.\n\nThat means we remove that check, the API doesn\u0027t work, it is bug, that means we can fix API without microversion. \n\nIf so, we should revert the patch?","commit_id":"048fa2310014e6282e183da0abee87c73b6d71b9"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"42f3e9310df2deaaf34c59cd72a18248a4df2702","unresolved":false,"context_lines":[{"line_number":1812,"context_line":"    hypervisors support booting from multiple devices but only if the devices are"},{"line_number":1813,"context_line":"    of different types. For example, a disk and CD-ROM."},{"line_number":1814,"context_line":"  in: body"},{"line_number":1815,"context_line":"  required: true"},{"line_number":1816,"context_line":"  type: integer"},{"line_number":1817,"context_line":"certificate:"},{"line_number":1818,"context_line":"  description: |"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"9f6a8fd7_a2f73b59","line":1815,"range":{"start_line":1815,"start_character":2,"end_line":1815,"end_character":16},"in_reply_to":"9f6a8fd7_c93e1db7","updated":"2018-04-24 13:20:44.000000000","message":"My thinking is:\n\n1. This is required for any release prior to Queens.\n2. Providing boot_index in \u003e\u003d Queens is OK. At worst it\u0027s unnecessary. However, not providing it \u003c Queens results in a failure.\n\nRather than revert, I\u0027ve put up a fix in the patch on top of this one in the series.","commit_id":"048fa2310014e6282e183da0abee87c73b6d71b9"},{"author":{"_account_id":5754,"name":"Alex Xu","email":"hejie.xu@intel.com","username":"xuhj"},"change_message_id":"38b2b7cb7e37b56baf91c261ef053aa2e5d2ba5e","unresolved":false,"context_lines":[{"line_number":1812,"context_line":"    hypervisors support booting from multiple devices but only if the devices are"},{"line_number":1813,"context_line":"    of different types. For example, a disk and CD-ROM."},{"line_number":1814,"context_line":"  in: body"},{"line_number":1815,"context_line":"  required: true"},{"line_number":1816,"context_line":"  type: integer"},{"line_number":1817,"context_line":"certificate:"},{"line_number":1818,"context_line":"  description: |"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"9f6a8fd7_5fd85fed","line":1815,"range":{"start_line":1815,"start_character":2,"end_line":1815,"end_character":16},"in_reply_to":"9f6a8fd7_ee90abef","updated":"2018-04-25 09:34:09.000000000","message":"Oh, yes, this fix is right, I\u0027m really reverse the option/required.","commit_id":"048fa2310014e6282e183da0abee87c73b6d71b9"}]}