)]}'
{"doc/source/install/deploy-ramdisk.rst":[{"author":{"_account_id":24828,"name":"Kaifeng Wang","email":"kaifeng.w@gmail.com","username":"wangkf"},"change_message_id":"6752527a48af95a7e1787a3fac4675784deef32f","unresolved":false,"context_lines":[{"line_number":77,"context_line":""},{"line_number":78,"context_line":".. _BuildingDibBasedDeployRamdisk:"},{"line_number":79,"context_line":""},{"line_number":80,"context_line":"disk-image-builder"},{"line_number":81,"context_line":"~~~~~~~~~~~~~~~~~~"},{"line_number":82,"context_line":""},{"line_number":83,"context_line":"#. Follow `diskimage-builder installation documentation`_ to install"}],"source_content_type":"text/x-rst","patch_set":1,"id":"5faad753_729f6550","side":"PARENT","line":80,"updated":"2019-09-09 03:51:34.000000000","message":"As I understand it, user could still build a ramdisk by using disk-image-builder, maybe just leave a link here so they know there is an alternative? I am not sure our IPA-B supports all elements.","commit_id":"9eebf2603e0d47dae036f79359a57ab26dfea7c2"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"63e989d329f64e6b2daedfd57ac7db27f3547f49","unresolved":false,"context_lines":[{"line_number":77,"context_line":""},{"line_number":78,"context_line":".. _BuildingDibBasedDeployRamdisk:"},{"line_number":79,"context_line":""},{"line_number":80,"context_line":"disk-image-builder"},{"line_number":81,"context_line":"~~~~~~~~~~~~~~~~~~"},{"line_number":82,"context_line":""},{"line_number":83,"context_line":"#. Follow `diskimage-builder installation documentation`_ to install"}],"source_content_type":"text/x-rst","patch_set":1,"id":"5faad753_9a9b0f71","side":"PARENT","line":80,"in_reply_to":"5faad753_729f6550","updated":"2019-09-09 13:59:21.000000000","message":"The link is above. IPA-builder documentation will cover everything that is here right now (the patch is already proposed).","commit_id":"9eebf2603e0d47dae036f79359a57ab26dfea7c2"}],"ironic/drivers/modules/deploy_utils.py":[{"author":{"_account_id":24828,"name":"Kaifeng Wang","email":"kaifeng.w@gmail.com","username":"wangkf"},"change_message_id":"6752527a48af95a7e1787a3fac4675784deef32f","unresolved":false,"context_lines":[{"line_number":881,"context_line":"    agent_config_opts \u003d {"},{"line_number":882,"context_line":"        \u0027ipa-api-url\u0027: get_ironic_api_url(),"},{"line_number":883,"context_line":"        # NOTE: The below entry is a temporary workaround for bug/1433812"},{"line_number":884,"context_line":"        \u0027coreos.configdrive\u0027: 0,"},{"line_number":885,"context_line":"    }"},{"line_number":886,"context_line":"    return agent_config_opts"},{"line_number":887,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"5faad753_b2887d1f","side":"PARENT","line":884,"updated":"2019-09-09 03:51:34.000000000","message":"I feel like suddenly we lost all the background on this, could we add the bug number to release note?","commit_id":"9eebf2603e0d47dae036f79359a57ab26dfea7c2"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"63e989d329f64e6b2daedfd57ac7db27f3547f49","unresolved":false,"context_lines":[{"line_number":881,"context_line":"    agent_config_opts \u003d {"},{"line_number":882,"context_line":"        \u0027ipa-api-url\u0027: get_ironic_api_url(),"},{"line_number":883,"context_line":"        # NOTE: The below entry is a temporary workaround for bug/1433812"},{"line_number":884,"context_line":"        \u0027coreos.configdrive\u0027: 0,"},{"line_number":885,"context_line":"    }"},{"line_number":886,"context_line":"    return agent_config_opts"},{"line_number":887,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"5faad753_da9f474e","side":"PARENT","line":884,"in_reply_to":"5faad753_b2887d1f","updated":"2019-09-09 13:59:21.000000000","message":"Good point","commit_id":"9eebf2603e0d47dae036f79359a57ab26dfea7c2"}],"ironic/tests/unit/drivers/modules/test_deploy_utils.py":[{"author":{"_account_id":24828,"name":"Kaifeng Wang","email":"kaifeng.w@gmail.com","username":"wangkf"},"change_message_id":"6752527a48af95a7e1787a3fac4675784deef32f","unresolved":false,"context_lines":[{"line_number":877,"context_line":"class SwitchPxeConfigTestCase(tests_base.TestCase):"},{"line_number":878,"context_line":""},{"line_number":879,"context_line":"    # NOTE(TheJulia): Remove elilo support after the deprecation period,"},{"line_number":880,"context_line":"    # in the Queens release."},{"line_number":881,"context_line":"    def _create_config(self, ipxe\u003dFalse, boot_mode\u003dNone, boot_loader\u003d\u0027elilo\u0027):"},{"line_number":882,"context_line":"        (fd, fname) \u003d tempfile.mkstemp()"},{"line_number":883,"context_line":"        if boot_mode \u003d\u003d \u0027uefi\u0027 and not ipxe:"}],"source_content_type":"text/x-python","patch_set":1,"id":"5faad753_b2b63dd9","line":880,"updated":"2019-09-09 03:51:34.000000000","message":"Not familiar with elilo, but I remember there was some obstacle for removing it, so I guess those are gone.\n\nAnd I think we can remove these tests as well.","commit_id":"700ed253086e205f8b44b81e26199d68ff347bcf"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"63e989d329f64e6b2daedfd57ac7db27f3547f49","unresolved":false,"context_lines":[{"line_number":877,"context_line":"class SwitchPxeConfigTestCase(tests_base.TestCase):"},{"line_number":878,"context_line":""},{"line_number":879,"context_line":"    # NOTE(TheJulia): Remove elilo support after the deprecation period,"},{"line_number":880,"context_line":"    # in the Queens release."},{"line_number":881,"context_line":"    def _create_config(self, ipxe\u003dFalse, boot_mode\u003dNone, boot_loader\u003d\u0027elilo\u0027):"},{"line_number":882,"context_line":"        (fd, fname) \u003d tempfile.mkstemp()"},{"line_number":883,"context_line":"        if boot_mode \u003d\u003d \u0027uefi\u0027 and not ipxe:"}],"source_content_type":"text/x-python","patch_set":1,"id":"5faad753_5a0b578d","line":880,"in_reply_to":"5faad753_b2b63dd9","updated":"2019-09-09 13:59:21.000000000","message":"Actually, we\u0027ve removed elilo support this cycle. This file has to be cleaned up, but I\u0027d prefer it to be done separately. I\u0027ve removed the elilo template to avoid updating it with the coreos.configdrive change.","commit_id":"700ed253086e205f8b44b81e26199d68ff347bcf"}]}