)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"change_message_id":"6f67bb2c2037c596e264e99dbc416992c1d4878a","unresolved":true,"context_lines":[{"line_number":4,"context_line":"Commit:     Ilya Popov \u003cilya_p@hotmail.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2021-05-29 23:18:03 +0300"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Extra var enable_keystone_integration was added."},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Basically, there are three main installation scenario:"},{"line_number":10,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"fb079d67_e7ea582d","line":7,"range":{"start_line":7,"start_character":10,"end_line":7,"end_character":37},"updated":"2021-06-04 16:26:20.000000000","message":"I would suggest using an ironic_ prefix, since this is specific to ironic.","commit_id":"f28bf0b090d925e91789af8aaab8981f19ab7580"},{"author":{"_account_id":16198,"name":"Ilya Popov","email":"hebulrih@gmail.com","username":"IPO"},"change_message_id":"7998ff4a57975f982bf39986c7ffd08dd3179080","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Ilya Popov \u003cilya_p@hotmail.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2021-05-29 23:18:03 +0300"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Extra var enable_keystone_integration was added."},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Basically, there are three main installation scenario:"},{"line_number":10,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"4f12187b_06276bac","line":7,"range":{"start_line":7,"start_character":10,"end_line":7,"end_character":37},"in_reply_to":"fb079d67_e7ea582d","updated":"2021-07-29 11:53:28.000000000","message":"Done","commit_id":"f28bf0b090d925e91789af8aaab8981f19ab7580"},{"author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"change_message_id":"6f67bb2c2037c596e264e99dbc416992c1d4878a","unresolved":true,"context_lines":[{"line_number":14,"context_line":"together with ironic installation."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"-Ironic installation with connection to already installed"},{"line_number":17,"context_line":"(so-called detached) keystone. In this case we have to set"},{"line_number":18,"context_line":"enable_keystone to False to prevent from new keystone"},{"line_number":19,"context_line":"service installation during the ironic installation process."},{"line_number":20,"context_line":"But in other hand, we need to have correct sections in"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"32aa42ea_f33eaf83","line":17,"range":{"start_line":17,"start_character":0,"end_line":17,"end_character":29},"updated":"2021-06-04 16:26:20.000000000","message":"A common scenario where this happens is with multiple regions sharing a single keystone. Maybe that should be highlighted?","commit_id":"f28bf0b090d925e91789af8aaab8981f19ab7580"},{"author":{"_account_id":16198,"name":"Ilya Popov","email":"hebulrih@gmail.com","username":"IPO"},"change_message_id":"7998ff4a57975f982bf39986c7ffd08dd3179080","unresolved":false,"context_lines":[{"line_number":14,"context_line":"together with ironic installation."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"-Ironic installation with connection to already installed"},{"line_number":17,"context_line":"(so-called detached) keystone. In this case we have to set"},{"line_number":18,"context_line":"enable_keystone to False to prevent from new keystone"},{"line_number":19,"context_line":"service installation during the ironic installation process."},{"line_number":20,"context_line":"But in other hand, we need to have correct sections in"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"3461a7b8_c9740ba1","line":17,"range":{"start_line":17,"start_character":0,"end_line":17,"end_character":29},"in_reply_to":"32aa42ea_f33eaf83","updated":"2021-07-29 11:53:28.000000000","message":"Done","commit_id":"f28bf0b090d925e91789af8aaab8981f19ab7580"},{"author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"change_message_id":"6f67bb2c2037c596e264e99dbc416992c1d4878a","unresolved":true,"context_lines":[{"line_number":24,"context_line":""},{"line_number":25,"context_line":"-No keystone integration. Ironic don\u0027t connect to Keystone."},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"Proposed solution keep the default behaviour: if no"},{"line_number":28,"context_line":"enable_keystone_integration is manually defined by default"},{"line_number":29,"context_line":"it takes value of enable_keystone variable and all behavior"},{"line_number":30,"context_line":"is the same. But if we don\u0027t want to install keystone and"},{"line_number":31,"context_line":"want to connect to existing one at the same time, it will be"},{"line_number":32,"context_line":"possible to set enable_keystone var to False"},{"line_number":33,"context_line":"(preventing keystone from installation) and at the same"},{"line_number":34,"context_line":"time set enable_keystone_integration to True to allow"},{"line_number":35,"context_line":"needed section appear in ironic.conf through templating"},{"line_number":36,"context_line":""},{"line_number":37,"context_line":"Change-Id: I0c7e9a28876a1d4278fb2ed8555c2b08472864b9"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"223632bc_2148ebd3","line":35,"range":{"start_line":27,"start_character":0,"end_line":35,"end_character":55},"updated":"2021-06-04 16:26:20.000000000","message":"I think the question here is whether scenario 2 or 3 is more common. Although, we might be able to keep most cases working by setting the default like this:\n\nironic_enable_keystone_integration: \"{{ enable_keystone | bool or multiple_regions_names | length \u003e 1 }}\"","commit_id":"f28bf0b090d925e91789af8aaab8981f19ab7580"},{"author":{"_account_id":16198,"name":"Ilya Popov","email":"hebulrih@gmail.com","username":"IPO"},"change_message_id":"7998ff4a57975f982bf39986c7ffd08dd3179080","unresolved":false,"context_lines":[{"line_number":24,"context_line":""},{"line_number":25,"context_line":"-No keystone integration. Ironic don\u0027t connect to Keystone."},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"Proposed solution keep the default behaviour: if no"},{"line_number":28,"context_line":"enable_keystone_integration is manually defined by default"},{"line_number":29,"context_line":"it takes value of enable_keystone variable and all behavior"},{"line_number":30,"context_line":"is the same. But if we don\u0027t want to install keystone and"},{"line_number":31,"context_line":"want to connect to existing one at the same time, it will be"},{"line_number":32,"context_line":"possible to set enable_keystone var to False"},{"line_number":33,"context_line":"(preventing keystone from installation) and at the same"},{"line_number":34,"context_line":"time set enable_keystone_integration to True to allow"},{"line_number":35,"context_line":"needed section appear in ironic.conf through templating"},{"line_number":36,"context_line":""},{"line_number":37,"context_line":"Change-Id: I0c7e9a28876a1d4278fb2ed8555c2b08472864b9"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"75d282d3_87e7b08d","line":35,"range":{"start_line":27,"start_character":0,"end_line":35,"end_character":55},"in_reply_to":"223632bc_2148ebd3","updated":"2021-07-29 11:53:28.000000000","message":"I\u0027m note sure that several regions in multiple_regions_names is the reason to assume that these regions basically will be integrated to keystone...May be it is just several regions without keystone or integration of shared neutron...","commit_id":"f28bf0b090d925e91789af8aaab8981f19ab7580"}],"ansible/roles/ironic/defaults/main.yml":[{"author":{"_account_id":14200,"name":"Maksim Malchuk","email":"maksim.malchuk@gmail.com","username":"mmalchuk"},"change_message_id":"733c9bd7c743edfaade8913fbf63a1e08531633a","unresolved":true,"context_lines":[{"line_number":286,"context_line":"####################"},{"line_number":287,"context_line":"# Keystone"},{"line_number":288,"context_line":"####################"},{"line_number":289,"context_line":"enable_keystone_integration: \"{{  enable_keystone | bool }}\""},{"line_number":290,"context_line":"ironic_ks_services:"},{"line_number":291,"context_line":"  - name: \"ironic\""},{"line_number":292,"context_line":"    type: \"baremetal\""}],"source_content_type":"text/x-yaml","patch_set":1,"id":"59791b0f_649a8bbb","line":289,"range":{"start_line":289,"start_character":32,"end_line":289,"end_character":33},"updated":"2021-05-29 20:03:57.000000000","message":"extra space","commit_id":"eb66a0e455279abbf37f69885b9612fe54bbfa05"}],"doc/source/reference/bare-metal/ironic-guide.rst":[{"author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"change_message_id":"d71815de512a8966668f8d8645e12ca344c1dc88","unresolved":true,"context_lines":[{"line_number":106,"context_line":"   [DEFAULT]"},{"line_number":107,"context_line":"   enabled_boot_interfaces \u003d ipxe"},{"line_number":108,"context_line":""},{"line_number":109,"context_line":"Attach ironic to existing keystone (optional)"},{"line_number":110,"context_line":"~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~"},{"line_number":111,"context_line":"In multi-regional deployment keystone could be installed in one region (let\u0027s"},{"line_number":112,"context_line":"say region 1) and ironic - in other region (let\u0027s say region 2)."}],"source_content_type":"text/x-rst","patch_set":6,"id":"409b629c_8d01bbc3","line":109,"range":{"start_line":109,"start_character":17,"end_line":109,"end_character":25},"updated":"2021-08-03 09:06:31.000000000","message":"external?","commit_id":"8ba8f54b73a3cb12c4f26fcd0d0cde49d3d323bd"},{"author":{"_account_id":16198,"name":"Ilya Popov","email":"hebulrih@gmail.com","username":"IPO"},"change_message_id":"ee354f050bb98d541ab109fcf83355e7ba503bd9","unresolved":false,"context_lines":[{"line_number":106,"context_line":"   [DEFAULT]"},{"line_number":107,"context_line":"   enabled_boot_interfaces \u003d ipxe"},{"line_number":108,"context_line":""},{"line_number":109,"context_line":"Attach ironic to existing keystone (optional)"},{"line_number":110,"context_line":"~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~"},{"line_number":111,"context_line":"In multi-regional deployment keystone could be installed in one region (let\u0027s"},{"line_number":112,"context_line":"say region 1) and ironic - in other region (let\u0027s say region 2)."}],"source_content_type":"text/x-rst","patch_set":6,"id":"4ddcef1d_133e9391","line":109,"range":{"start_line":109,"start_character":17,"end_line":109,"end_character":25},"in_reply_to":"409b629c_8d01bbc3","updated":"2021-08-03 21:02:53.000000000","message":"Done","commit_id":"8ba8f54b73a3cb12c4f26fcd0d0cde49d3d323bd"},{"author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"change_message_id":"d71815de512a8966668f8d8645e12ca344c1dc88","unresolved":true,"context_lines":[{"line_number":107,"context_line":"   enabled_boot_interfaces \u003d ipxe"},{"line_number":108,"context_line":""},{"line_number":109,"context_line":"Attach ironic to existing keystone (optional)"},{"line_number":110,"context_line":"~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~"},{"line_number":111,"context_line":"In multi-regional deployment keystone could be installed in one region (let\u0027s"},{"line_number":112,"context_line":"say region 1) and ironic - in other region (let\u0027s say region 2)."},{"line_number":113,"context_line":"In this case we don\u0027t install keystone together with ironic in region 2, but"}],"source_content_type":"text/x-rst","patch_set":6,"id":"71ba1c10_3440ea20","line":110,"updated":"2021-08-03 09:06:31.000000000","message":"This is a nice explanation. I would suggest linking to the multi-region docs (https://docs.openstack.org/kolla-ansible/latest/user/multi-regions.html) for context.","commit_id":"8ba8f54b73a3cb12c4f26fcd0d0cde49d3d323bd"},{"author":{"_account_id":16198,"name":"Ilya Popov","email":"hebulrih@gmail.com","username":"IPO"},"change_message_id":"ee354f050bb98d541ab109fcf83355e7ba503bd9","unresolved":false,"context_lines":[{"line_number":107,"context_line":"   enabled_boot_interfaces \u003d ipxe"},{"line_number":108,"context_line":""},{"line_number":109,"context_line":"Attach ironic to existing keystone (optional)"},{"line_number":110,"context_line":"~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~"},{"line_number":111,"context_line":"In multi-regional deployment keystone could be installed in one region (let\u0027s"},{"line_number":112,"context_line":"say region 1) and ironic - in other region (let\u0027s say region 2)."},{"line_number":113,"context_line":"In this case we don\u0027t install keystone together with ironic in region 2, but"}],"source_content_type":"text/x-rst","patch_set":6,"id":"ee58b41e_d6dd9fdc","line":110,"in_reply_to":"71ba1c10_3440ea20","updated":"2021-08-03 21:02:53.000000000","message":"Done","commit_id":"8ba8f54b73a3cb12c4f26fcd0d0cde49d3d323bd"},{"author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"change_message_id":"1e832d4ea20484331c8af5101d98ddf0c37b695e","unresolved":true,"context_lines":[{"line_number":108,"context_line":""},{"line_number":109,"context_line":"Attach ironic to external keystone (optional)"},{"line_number":110,"context_line":"~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~"},{"line_number":111,"context_line":"In multi-regional deployment (for more information about multiple regions"},{"line_number":112,"context_line":"deployment with kolla see OpenStack `documentation \u003chttps://docs.openstack.org/kolla-ansible/latest/user/multi-regions.html\u003e`_)"},{"line_number":113,"context_line":"keystone could be installed in one region (let\u0027s say region 1) and ironic -"},{"line_number":114,"context_line":"in other region (let\u0027s say region 2). In this case we don\u0027t install keystone"}],"source_content_type":"text/x-rst","patch_set":7,"id":"1fa7345e_22bc89a9","line":111,"range":{"start_line":111,"start_character":3,"end_line":111,"end_character":28},"updated":"2021-08-04 08:41:34.000000000","message":"nit: you could just link from this text.","commit_id":"7b2125c2e7d24e10f007a74488471ebc1ef5867c"},{"author":{"_account_id":16198,"name":"Ilya Popov","email":"hebulrih@gmail.com","username":"IPO"},"change_message_id":"09047401453bcd55351dd9b13137d90c69a42dbd","unresolved":false,"context_lines":[{"line_number":108,"context_line":""},{"line_number":109,"context_line":"Attach ironic to external keystone (optional)"},{"line_number":110,"context_line":"~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~"},{"line_number":111,"context_line":"In multi-regional deployment (for more information about multiple regions"},{"line_number":112,"context_line":"deployment with kolla see OpenStack `documentation \u003chttps://docs.openstack.org/kolla-ansible/latest/user/multi-regions.html\u003e`_)"},{"line_number":113,"context_line":"keystone could be installed in one region (let\u0027s say region 1) and ironic -"},{"line_number":114,"context_line":"in other region (let\u0027s say region 2). In this case we don\u0027t install keystone"}],"source_content_type":"text/x-rst","patch_set":7,"id":"cbd88199_403c6b21","line":111,"range":{"start_line":111,"start_character":3,"end_line":111,"end_character":28},"in_reply_to":"1fa7345e_22bc89a9","updated":"2021-08-04 09:42:14.000000000","message":"I thought about this way, thank you !","commit_id":"7b2125c2e7d24e10f007a74488471ebc1ef5867c"},{"author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"change_message_id":"1e832d4ea20484331c8af5101d98ddf0c37b695e","unresolved":true,"context_lines":[{"line_number":109,"context_line":"Attach ironic to external keystone (optional)"},{"line_number":110,"context_line":"~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~"},{"line_number":111,"context_line":"In multi-regional deployment (for more information about multiple regions"},{"line_number":112,"context_line":"deployment with kolla see OpenStack `documentation \u003chttps://docs.openstack.org/kolla-ansible/latest/user/multi-regions.html\u003e`_)"},{"line_number":113,"context_line":"keystone could be installed in one region (let\u0027s say region 1) and ironic -"},{"line_number":114,"context_line":"in other region (let\u0027s say region 2). In this case we don\u0027t install keystone"},{"line_number":115,"context_line":"together with ironic in region 2, but have to configure ironic to connect to"}],"source_content_type":"text/x-rst","patch_set":7,"id":"50ca4647_19399084","line":112,"range":{"start_line":112,"start_character":36,"end_line":112,"end_character":126},"updated":"2021-08-04 08:41:34.000000000","message":"We have some syntax for linking within the kolla docs that preserves the version.\n\n:kolla-ansible-doc:`documentation \u003cuser/multi-regions.html\u003e`","commit_id":"7b2125c2e7d24e10f007a74488471ebc1ef5867c"},{"author":{"_account_id":16198,"name":"Ilya Popov","email":"hebulrih@gmail.com","username":"IPO"},"change_message_id":"09047401453bcd55351dd9b13137d90c69a42dbd","unresolved":false,"context_lines":[{"line_number":109,"context_line":"Attach ironic to external keystone (optional)"},{"line_number":110,"context_line":"~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~"},{"line_number":111,"context_line":"In multi-regional deployment (for more information about multiple regions"},{"line_number":112,"context_line":"deployment with kolla see OpenStack `documentation \u003chttps://docs.openstack.org/kolla-ansible/latest/user/multi-regions.html\u003e`_)"},{"line_number":113,"context_line":"keystone could be installed in one region (let\u0027s say region 1) and ironic -"},{"line_number":114,"context_line":"in other region (let\u0027s say region 2). In this case we don\u0027t install keystone"},{"line_number":115,"context_line":"together with ironic in region 2, but have to configure ironic to connect to"}],"source_content_type":"text/x-rst","patch_set":7,"id":"f1935f4c_208d12e7","line":112,"range":{"start_line":112,"start_character":36,"end_line":112,"end_character":126},"in_reply_to":"50ca4647_19399084","updated":"2021-08-04 09:42:14.000000000","message":"Done","commit_id":"7b2125c2e7d24e10f007a74488471ebc1ef5867c"},{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"610ba30dbc24060e51c6a5a99251edfd86541ae4","unresolved":true,"context_lines":[{"line_number":110,"context_line":"~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~"},{"line_number":111,"context_line":"In :kolla-ansible-doc:`multi-regional \u003cuser/multi-regions.html\u003e` deployment"},{"line_number":112,"context_line":"keystone could be installed in one region (let\u0027s say region 1) and ironic -"},{"line_number":113,"context_line":"in other region (let\u0027s say region 2). In this case we don\u0027t install keystone"},{"line_number":114,"context_line":"together with ironic in region 2, but have to configure ironic to connect to"},{"line_number":115,"context_line":"existing keystone in region 1. To deploy ironic in this way we have to set"},{"line_number":116,"context_line":"variable enable_keystone to \"no\"."}],"source_content_type":"text/x-rst","patch_set":8,"id":"8a696d81_728388f6","line":113,"range":{"start_line":113,"start_character":3,"end_line":113,"end_character":9},"updated":"2021-08-06 12:59:55.000000000","message":"another","commit_id":"22ccb2f55a42b66a82c85cb3e6f34dddac5ce736"},{"author":{"_account_id":16198,"name":"Ilya Popov","email":"hebulrih@gmail.com","username":"IPO"},"change_message_id":"18857aa60067587e1128025de8644c85b5303a69","unresolved":false,"context_lines":[{"line_number":110,"context_line":"~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~"},{"line_number":111,"context_line":"In :kolla-ansible-doc:`multi-regional \u003cuser/multi-regions.html\u003e` deployment"},{"line_number":112,"context_line":"keystone could be installed in one region (let\u0027s say region 1) and ironic -"},{"line_number":113,"context_line":"in other region (let\u0027s say region 2). In this case we don\u0027t install keystone"},{"line_number":114,"context_line":"together with ironic in region 2, but have to configure ironic to connect to"},{"line_number":115,"context_line":"existing keystone in region 1. To deploy ironic in this way we have to set"},{"line_number":116,"context_line":"variable enable_keystone to \"no\"."}],"source_content_type":"text/x-rst","patch_set":8,"id":"cba620da_c5707544","line":113,"range":{"start_line":113,"start_character":3,"end_line":113,"end_character":9},"in_reply_to":"8a696d81_728388f6","updated":"2021-08-06 13:42:52.000000000","message":"Done","commit_id":"22ccb2f55a42b66a82c85cb3e6f34dddac5ce736"},{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"610ba30dbc24060e51c6a5a99251edfd86541ae4","unresolved":true,"context_lines":[{"line_number":113,"context_line":"in other region (let\u0027s say region 2). In this case we don\u0027t install keystone"},{"line_number":114,"context_line":"together with ironic in region 2, but have to configure ironic to connect to"},{"line_number":115,"context_line":"existing keystone in region 1. To deploy ironic in this way we have to set"},{"line_number":116,"context_line":"variable enable_keystone to \"no\"."},{"line_number":117,"context_line":""},{"line_number":118,"context_line":".. code-block:: yaml"},{"line_number":119,"context_line":""}],"source_content_type":"text/x-rst","patch_set":8,"id":"fcea1296_c0ec9a88","line":116,"range":{"start_line":116,"start_character":28,"end_line":116,"end_character":32},"updated":"2021-08-06 12:59:55.000000000","message":"ditto","commit_id":"22ccb2f55a42b66a82c85cb3e6f34dddac5ce736"},{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"610ba30dbc24060e51c6a5a99251edfd86541ae4","unresolved":true,"context_lines":[{"line_number":113,"context_line":"in other region (let\u0027s say region 2). In this case we don\u0027t install keystone"},{"line_number":114,"context_line":"together with ironic in region 2, but have to configure ironic to connect to"},{"line_number":115,"context_line":"existing keystone in region 1. To deploy ironic in this way we have to set"},{"line_number":116,"context_line":"variable enable_keystone to \"no\"."},{"line_number":117,"context_line":""},{"line_number":118,"context_line":".. code-block:: yaml"},{"line_number":119,"context_line":""}],"source_content_type":"text/x-rst","patch_set":8,"id":"b4b9404e_c4d17a81","line":116,"range":{"start_line":116,"start_character":9,"end_line":116,"end_character":25},"updated":"2021-08-06 12:59:55.000000000","message":"need to wrap in `` `` for proper rendering","commit_id":"22ccb2f55a42b66a82c85cb3e6f34dddac5ce736"},{"author":{"_account_id":16198,"name":"Ilya Popov","email":"hebulrih@gmail.com","username":"IPO"},"change_message_id":"18857aa60067587e1128025de8644c85b5303a69","unresolved":false,"context_lines":[{"line_number":113,"context_line":"in other region (let\u0027s say region 2). In this case we don\u0027t install keystone"},{"line_number":114,"context_line":"together with ironic in region 2, but have to configure ironic to connect to"},{"line_number":115,"context_line":"existing keystone in region 1. To deploy ironic in this way we have to set"},{"line_number":116,"context_line":"variable enable_keystone to \"no\"."},{"line_number":117,"context_line":""},{"line_number":118,"context_line":".. code-block:: yaml"},{"line_number":119,"context_line":""}],"source_content_type":"text/x-rst","patch_set":8,"id":"b7984bf6_36495b3f","line":116,"range":{"start_line":116,"start_character":9,"end_line":116,"end_character":25},"in_reply_to":"b4b9404e_c4d17a81","updated":"2021-08-06 13:42:52.000000000","message":"Done","commit_id":"22ccb2f55a42b66a82c85cb3e6f34dddac5ce736"},{"author":{"_account_id":16198,"name":"Ilya Popov","email":"hebulrih@gmail.com","username":"IPO"},"change_message_id":"18857aa60067587e1128025de8644c85b5303a69","unresolved":false,"context_lines":[{"line_number":113,"context_line":"in other region (let\u0027s say region 2). In this case we don\u0027t install keystone"},{"line_number":114,"context_line":"together with ironic in region 2, but have to configure ironic to connect to"},{"line_number":115,"context_line":"existing keystone in region 1. To deploy ironic in this way we have to set"},{"line_number":116,"context_line":"variable enable_keystone to \"no\"."},{"line_number":117,"context_line":""},{"line_number":118,"context_line":".. code-block:: yaml"},{"line_number":119,"context_line":""}],"source_content_type":"text/x-rst","patch_set":8,"id":"a40ecf30_5b637478","line":116,"range":{"start_line":116,"start_character":28,"end_line":116,"end_character":32},"in_reply_to":"fcea1296_c0ec9a88","updated":"2021-08-06 13:42:52.000000000","message":"Done","commit_id":"22ccb2f55a42b66a82c85cb3e6f34dddac5ce736"},{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"610ba30dbc24060e51c6a5a99251edfd86541ae4","unresolved":true,"context_lines":[{"line_number":119,"context_line":""},{"line_number":120,"context_line":"    enable_keystone: \"no\""},{"line_number":121,"context_line":""},{"line_number":122,"context_line":"It will prevent keystone from been installed in region 2."},{"line_number":123,"context_line":""},{"line_number":124,"context_line":"To add keystone-related sections in ironic.conf, also it is needed to set"},{"line_number":125,"context_line":"variable ironic_enable_keystone_integration to \"yes\""}],"source_content_type":"text/x-rst","patch_set":8,"id":"32f8dabd_25a30920","line":122,"range":{"start_line":122,"start_character":30,"end_line":122,"end_character":35},"updated":"2021-08-06 12:59:55.000000000","message":"being","commit_id":"22ccb2f55a42b66a82c85cb3e6f34dddac5ce736"},{"author":{"_account_id":16198,"name":"Ilya Popov","email":"hebulrih@gmail.com","username":"IPO"},"change_message_id":"18857aa60067587e1128025de8644c85b5303a69","unresolved":false,"context_lines":[{"line_number":119,"context_line":""},{"line_number":120,"context_line":"    enable_keystone: \"no\""},{"line_number":121,"context_line":""},{"line_number":122,"context_line":"It will prevent keystone from been installed in region 2."},{"line_number":123,"context_line":""},{"line_number":124,"context_line":"To add keystone-related sections in ironic.conf, also it is needed to set"},{"line_number":125,"context_line":"variable ironic_enable_keystone_integration to \"yes\""}],"source_content_type":"text/x-rst","patch_set":8,"id":"0a38b4a7_1c60eee6","line":122,"range":{"start_line":122,"start_character":30,"end_line":122,"end_character":35},"in_reply_to":"32f8dabd_25a30920","updated":"2021-08-06 13:42:52.000000000","message":"Done","commit_id":"22ccb2f55a42b66a82c85cb3e6f34dddac5ce736"},{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"610ba30dbc24060e51c6a5a99251edfd86541ae4","unresolved":true,"context_lines":[{"line_number":121,"context_line":""},{"line_number":122,"context_line":"It will prevent keystone from been installed in region 2."},{"line_number":123,"context_line":""},{"line_number":124,"context_line":"To add keystone-related sections in ironic.conf, also it is needed to set"},{"line_number":125,"context_line":"variable ironic_enable_keystone_integration to \"yes\""},{"line_number":126,"context_line":""},{"line_number":127,"context_line":".. code-block:: yaml"}],"source_content_type":"text/x-rst","patch_set":8,"id":"21b6c055_fd193db9","line":124,"range":{"start_line":124,"start_character":49,"end_line":124,"end_character":66},"updated":"2021-08-06 12:59:55.000000000","message":"it is also needed","commit_id":"22ccb2f55a42b66a82c85cb3e6f34dddac5ce736"},{"author":{"_account_id":16198,"name":"Ilya Popov","email":"hebulrih@gmail.com","username":"IPO"},"change_message_id":"18857aa60067587e1128025de8644c85b5303a69","unresolved":false,"context_lines":[{"line_number":121,"context_line":""},{"line_number":122,"context_line":"It will prevent keystone from been installed in region 2."},{"line_number":123,"context_line":""},{"line_number":124,"context_line":"To add keystone-related sections in ironic.conf, also it is needed to set"},{"line_number":125,"context_line":"variable ironic_enable_keystone_integration to \"yes\""},{"line_number":126,"context_line":""},{"line_number":127,"context_line":".. code-block:: yaml"}],"source_content_type":"text/x-rst","patch_set":8,"id":"728ef517_8b7aa00b","line":124,"range":{"start_line":124,"start_character":49,"end_line":124,"end_character":66},"in_reply_to":"21b6c055_fd193db9","updated":"2021-08-06 13:42:52.000000000","message":"Done","commit_id":"22ccb2f55a42b66a82c85cb3e6f34dddac5ce736"},{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"610ba30dbc24060e51c6a5a99251edfd86541ae4","unresolved":true,"context_lines":[{"line_number":122,"context_line":"It will prevent keystone from been installed in region 2."},{"line_number":123,"context_line":""},{"line_number":124,"context_line":"To add keystone-related sections in ironic.conf, also it is needed to set"},{"line_number":125,"context_line":"variable ironic_enable_keystone_integration to \"yes\""},{"line_number":126,"context_line":""},{"line_number":127,"context_line":".. code-block:: yaml"},{"line_number":128,"context_line":""}],"source_content_type":"text/x-rst","patch_set":8,"id":"cc96c7d0_a0aaee09","line":125,"range":{"start_line":125,"start_character":47,"end_line":125,"end_character":52},"updated":"2021-08-06 12:59:55.000000000","message":"ditto","commit_id":"22ccb2f55a42b66a82c85cb3e6f34dddac5ce736"},{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"610ba30dbc24060e51c6a5a99251edfd86541ae4","unresolved":true,"context_lines":[{"line_number":122,"context_line":"It will prevent keystone from been installed in region 2."},{"line_number":123,"context_line":""},{"line_number":124,"context_line":"To add keystone-related sections in ironic.conf, also it is needed to set"},{"line_number":125,"context_line":"variable ironic_enable_keystone_integration to \"yes\""},{"line_number":126,"context_line":""},{"line_number":127,"context_line":".. code-block:: yaml"},{"line_number":128,"context_line":""}],"source_content_type":"text/x-rst","patch_set":8,"id":"92b91669_07dcf239","line":125,"range":{"start_line":125,"start_character":9,"end_line":125,"end_character":44},"updated":"2021-08-06 12:59:55.000000000","message":"ditto `` ``","commit_id":"22ccb2f55a42b66a82c85cb3e6f34dddac5ce736"},{"author":{"_account_id":16198,"name":"Ilya Popov","email":"hebulrih@gmail.com","username":"IPO"},"change_message_id":"18857aa60067587e1128025de8644c85b5303a69","unresolved":false,"context_lines":[{"line_number":122,"context_line":"It will prevent keystone from been installed in region 2."},{"line_number":123,"context_line":""},{"line_number":124,"context_line":"To add keystone-related sections in ironic.conf, also it is needed to set"},{"line_number":125,"context_line":"variable ironic_enable_keystone_integration to \"yes\""},{"line_number":126,"context_line":""},{"line_number":127,"context_line":".. code-block:: yaml"},{"line_number":128,"context_line":""}],"source_content_type":"text/x-rst","patch_set":8,"id":"59d4efbd_bdc13cca","line":125,"range":{"start_line":125,"start_character":9,"end_line":125,"end_character":44},"in_reply_to":"92b91669_07dcf239","updated":"2021-08-06 13:42:52.000000000","message":"Done","commit_id":"22ccb2f55a42b66a82c85cb3e6f34dddac5ce736"},{"author":{"_account_id":16198,"name":"Ilya Popov","email":"hebulrih@gmail.com","username":"IPO"},"change_message_id":"18857aa60067587e1128025de8644c85b5303a69","unresolved":false,"context_lines":[{"line_number":122,"context_line":"It will prevent keystone from been installed in region 2."},{"line_number":123,"context_line":""},{"line_number":124,"context_line":"To add keystone-related sections in ironic.conf, also it is needed to set"},{"line_number":125,"context_line":"variable ironic_enable_keystone_integration to \"yes\""},{"line_number":126,"context_line":""},{"line_number":127,"context_line":".. code-block:: yaml"},{"line_number":128,"context_line":""}],"source_content_type":"text/x-rst","patch_set":8,"id":"c88a0c37_aaf90f23","line":125,"range":{"start_line":125,"start_character":47,"end_line":125,"end_character":52},"in_reply_to":"cc96c7d0_a0aaee09","updated":"2021-08-06 13:42:52.000000000","message":"Done","commit_id":"22ccb2f55a42b66a82c85cb3e6f34dddac5ce736"},{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"610ba30dbc24060e51c6a5a99251edfd86541ae4","unresolved":true,"context_lines":[{"line_number":128,"context_line":""},{"line_number":129,"context_line":"    ironic_enable_keystone_integration: \"yes\""},{"line_number":130,"context_line":""},{"line_number":131,"context_line":"As a result of deployment we will have multi-regional deployment, where"},{"line_number":132,"context_line":"ironic in region 2 configured to connect to keystone in region 1 and there is"},{"line_number":133,"context_line":"no keystone installed in region 2."},{"line_number":134,"context_line":""},{"line_number":135,"context_line":"Deployment"},{"line_number":136,"context_line":"~~~~~~~~~~"}],"source_content_type":"text/x-rst","patch_set":8,"id":"60775c7c_558dffe4","line":133,"range":{"start_line":131,"start_character":0,"end_line":133,"end_character":34},"updated":"2021-08-06 12:59:55.000000000","message":"It would require to set keystone params as well. I would drop this paragraph as it only brings confusion. The recipe has already been provided.","commit_id":"22ccb2f55a42b66a82c85cb3e6f34dddac5ce736"},{"author":{"_account_id":16198,"name":"Ilya Popov","email":"hebulrih@gmail.com","username":"IPO"},"change_message_id":"18857aa60067587e1128025de8644c85b5303a69","unresolved":false,"context_lines":[{"line_number":128,"context_line":""},{"line_number":129,"context_line":"    ironic_enable_keystone_integration: \"yes\""},{"line_number":130,"context_line":""},{"line_number":131,"context_line":"As a result of deployment we will have multi-regional deployment, where"},{"line_number":132,"context_line":"ironic in region 2 configured to connect to keystone in region 1 and there is"},{"line_number":133,"context_line":"no keystone installed in region 2."},{"line_number":134,"context_line":""},{"line_number":135,"context_line":"Deployment"},{"line_number":136,"context_line":"~~~~~~~~~~"}],"source_content_type":"text/x-rst","patch_set":8,"id":"ef503339_5b81764d","line":133,"range":{"start_line":131,"start_character":0,"end_line":133,"end_character":34},"in_reply_to":"60775c7c_558dffe4","updated":"2021-08-06 13:42:52.000000000","message":"Done","commit_id":"22ccb2f55a42b66a82c85cb3e6f34dddac5ce736"}],"releasenotes/notes/update-ironic-template-for-keystone-1ee5f80fda7a21a0.yaml":[{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"610ba30dbc24060e51c6a5a99251edfd86541ae4","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":"    New variable ironic_enable_keystone_integration was added."},{"line_number":5,"context_line":"    It helps to add keystone connection information into"},{"line_number":6,"context_line":"    ironic.conf if we want to connect to existing keystone"},{"line_number":7,"context_line":"    (not installing it at the same time)."}],"source_content_type":"text/x-yaml","patch_set":8,"id":"ed936d85_7697cf59","line":4,"range":{"start_line":4,"start_character":17,"end_line":4,"end_character":52},"updated":"2021-08-06 12:59:55.000000000","message":"need to wrap in `` `` for proper rendering.","commit_id":"22ccb2f55a42b66a82c85cb3e6f34dddac5ce736"},{"author":{"_account_id":16198,"name":"Ilya Popov","email":"hebulrih@gmail.com","username":"IPO"},"change_message_id":"bfd6015577d3b613002bfd49c3f6d15e3cdcb83e","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":"    New variable ironic_enable_keystone_integration was added."},{"line_number":5,"context_line":"    It helps to add keystone connection information into"},{"line_number":6,"context_line":"    ironic.conf if we want to connect to existing keystone"},{"line_number":7,"context_line":"    (not installing it at the same time)."}],"source_content_type":"text/x-yaml","patch_set":8,"id":"2930774e_2f787eba","line":4,"range":{"start_line":4,"start_character":17,"end_line":4,"end_character":52},"in_reply_to":"ed936d85_7697cf59","updated":"2021-08-06 14:57:46.000000000","message":"Done","commit_id":"22ccb2f55a42b66a82c85cb3e6f34dddac5ce736"},{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"610ba30dbc24060e51c6a5a99251edfd86541ae4","unresolved":true,"context_lines":[{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    New variable ironic_enable_keystone_integration was added."},{"line_number":5,"context_line":"    It helps to add keystone connection information into"},{"line_number":6,"context_line":"    ironic.conf if we want to connect to existing keystone"},{"line_number":7,"context_line":"    (not installing it at the same time)."}],"source_content_type":"text/x-yaml","patch_set":8,"id":"a0b3db66_d9f2063c","line":6,"range":{"start_line":6,"start_character":4,"end_line":6,"end_character":15},"updated":"2021-08-06 12:59:55.000000000","message":"ditto","commit_id":"22ccb2f55a42b66a82c85cb3e6f34dddac5ce736"},{"author":{"_account_id":16198,"name":"Ilya Popov","email":"hebulrih@gmail.com","username":"IPO"},"change_message_id":"bfd6015577d3b613002bfd49c3f6d15e3cdcb83e","unresolved":false,"context_lines":[{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    New variable ironic_enable_keystone_integration was added."},{"line_number":5,"context_line":"    It helps to add keystone connection information into"},{"line_number":6,"context_line":"    ironic.conf if we want to connect to existing keystone"},{"line_number":7,"context_line":"    (not installing it at the same time)."}],"source_content_type":"text/x-yaml","patch_set":8,"id":"d205b9a7_095db77b","line":6,"range":{"start_line":6,"start_character":4,"end_line":6,"end_character":15},"in_reply_to":"a0b3db66_d9f2063c","updated":"2021-08-06 14:57:46.000000000","message":"Done","commit_id":"22ccb2f55a42b66a82c85cb3e6f34dddac5ce736"}]}
