)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":13362,"name":"Tan Lin","email":"lin.tan@intel.com","username":"zero"},"change_message_id":"806497238f2424af59bdf0f270d26ddebe6e346b","unresolved":false,"context_lines":[{"line_number":12,"context_line":"command."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"This patch replaces those commands with the \"sanboot\" command from iPXE"},{"line_number":15,"context_line":"which is used to chain the boot process to the local disk."},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"Closes-Bug: #1524403"},{"line_number":18,"context_line":"Change-Id: I7d2310d94e40b1c9c76957382ab19c0076bf42cd"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"7a740942_e8f8c8c6","line":15,"updated":"2015-12-10 03:10:15.000000000","message":"nit: this description makes me feel `sanboot` is only for localdisk, But boot from local disk is only part of  its work.","commit_id":"fd8e06252b10434d851d51e44a7b70e46247901a"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"4b67d41b9b846644c97fe9eed49a92b0144048a9","unresolved":false,"context_lines":[{"line_number":12,"context_line":"command."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"This patch replaces those commands with the \"sanboot\" command from iPXE"},{"line_number":15,"context_line":"which is used to chain the boot process to the local disk."},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"Closes-Bug: #1524403"},{"line_number":18,"context_line":"Change-Id: I7d2310d94e40b1c9c76957382ab19c0076bf42cd"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"7a740942_6ab49bf1","line":15,"in_reply_to":"7a740942_e8f8c8c6","updated":"2015-12-10 20:16:22.000000000","message":"Right, that\u0027s only one of the use cases for sanboot","commit_id":"fd8e06252b10434d851d51e44a7b70e46247901a"}],"ironic/drivers/modules/ipxe_config.template":[{"author":{"_account_id":13362,"name":"Tan Lin","email":"lin.tan@intel.com","username":"zero"},"change_message_id":"806497238f2424af59bdf0f270d26ddebe6e346b","unresolved":false,"context_lines":[{"line_number":16,"context_line":"boot"},{"line_number":17,"context_line":""},{"line_number":18,"context_line":":boot_whole_disk"},{"line_number":19,"context_line":"sanboot --no-describe"}],"source_content_type":"application/octet-stream","patch_set":1,"id":"7a740942_88a584b6","line":19,"updated":"2015-12-10 03:10:15.000000000","message":"nit: From iPXE instruction, it seems like better to explicitly declare the local driver as  SAN drive: \n* sanboot --no-describe --drive 0x80","commit_id":"fd8e06252b10434d851d51e44a7b70e46247901a"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"4b67d41b9b846644c97fe9eed49a92b0144048a9","unresolved":false,"context_lines":[{"line_number":16,"context_line":"boot"},{"line_number":17,"context_line":""},{"line_number":18,"context_line":":boot_whole_disk"},{"line_number":19,"context_line":"sanboot --no-describe"}],"source_content_type":"application/octet-stream","patch_set":1,"id":"7a740942_102bc895","line":19,"in_reply_to":"7a740942_88a584b6","updated":"2015-12-10 20:16:22.000000000","message":"Thanks, yeah if none is passed it will assume 0x80","commit_id":"fd8e06252b10434d851d51e44a7b70e46247901a"}],"releasenotes/notes/fix-ipxe-template-for-whole-disk-image-943da0311ca7aeb5.yaml":[{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"e3dd6541de280ecd20cf90dcdb272b4c9e6009c1","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"fixes:"},{"line_number":3,"context_line":"  - Fixes the iPXE command from the upstream iPXE template when chainloading"},{"line_number":4,"context_line":"    the boot process to the local disk for whole disk images."}],"source_content_type":"text/x-yaml","patch_set":1,"id":"7a740942_f208952a","line":4,"updated":"2015-12-10 14:50:26.000000000","message":"Will \u0027upstream\u0027 iPXE template make sense to people? Maybe \u0027provided\u0027 or \u0027included\u0027 iPXE template?\n\nHow about \u0027Fixes an issue with the provided iPXE template where whole disk images could not be booted. See https://bugs.launchpad.net/ironic/+bug/1524403.\u0027\n\n(That \u0027Fixes an issue...\u0027 is what Jim did for other bug fixes.)","commit_id":"fd8e06252b10434d851d51e44a7b70e46247901a"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"375c898beb883bc0dad52f30bd238658f85f02d6","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"fixes:"},{"line_number":3,"context_line":"  - Fixes the iPXE command from the upstream iPXE template when chainloading"},{"line_number":4,"context_line":"    the boot process to the local disk for whole disk images."}],"source_content_type":"text/x-yaml","patch_set":1,"id":"7a740942_851ca6c2","line":4,"updated":"2015-12-09 16:39:45.000000000","message":"it looks a bit too detailed to me, but I can\u0027t think of better wording","commit_id":"fd8e06252b10434d851d51e44a7b70e46247901a"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"4b67d41b9b846644c97fe9eed49a92b0144048a9","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"fixes:"},{"line_number":3,"context_line":"  - Fixes the iPXE command from the upstream iPXE template when chainloading"},{"line_number":4,"context_line":"    the boot process to the local disk for whole disk images."}],"source_content_type":"text/x-yaml","patch_set":1,"id":"7a740942_6aab3b8a","line":4,"in_reply_to":"7a740942_f208952a","updated":"2015-12-10 20:16:22.000000000","message":"Done","commit_id":"fd8e06252b10434d851d51e44a7b70e46247901a"}]}