)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":27339,"name":"Michal Arbet","email":"michal.arbet@ultimum.io","username":"michalarbet"},"change_message_id":"b4094474f20f1097abdb53332b1e7a48fca2bf30","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"91e9b61c_5b586ee6","updated":"2022-05-24 13:29:51.000000000","message":"Hmm, nova is failing because it can\u0027t connect to 35357 port because keystone is already reconfigured and upgraded, what about during the keystone role start \"migration_keystone_container\" to provide 35357 port during the upgrade and then just remove it when all will be reconfigured ? \n\n","commit_id":"ede5c1aad40aa6df33d5dabb30de05bc450f74a4"},{"author":{"_account_id":27339,"name":"Michal Arbet","email":"michal.arbet@ultimum.io","username":"michalarbet"},"change_message_id":"5bc3987400e25f5e208e7a2c7e2fe337135ebcbf","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"d6ad846e_c5624750","updated":"2022-05-24 13:30:12.000000000","message":"Set Work In Progress","commit_id":"ede5c1aad40aa6df33d5dabb30de05bc450f74a4"},{"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":"309fae2940f8630379f9791e5f9bdaae8fd66b17","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"f59415ab_e2182b8c","in_reply_to":"91e9b61c_5b586ee6","updated":"2022-05-24 15:41:26.000000000","message":"I guess the issue is we first need both keystone and haproxy to keep listening on upgrade and then allow for cleanup. The way the plays are wired currently, it does not look straightforward to do without limiting the cleanup to a new deploy, i.e. keep the upgrade code providing the admin endpoint but not the deployment code.","commit_id":"ede5c1aad40aa6df33d5dabb30de05bc450f74a4"},{"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":"c7ef6fd79e706a64ea5e4cca17326caf5a41b067","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"a3386cea_f03f62c2","updated":"2022-05-24 16:00:35.000000000","message":"This should work better. Needs a reno and some further verification.","commit_id":"f6059e5fa02187dacdb9c867c33bbf2025b2d00d"},{"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":"8fdd4b954a974b4f59362b5864e4c23fd446b7b1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"49af2b56_fef93ee8","updated":"2022-05-25 09:25:03.000000000","message":"we are still missing the following:\n\n- recreation of admin environment variables files in CI (so that it uses the new port)\n- endpoint cleaning redeployment of keystone in CI\n- update of the endpoints catalogue to include the new port of admin interface\n\nThe first two seem to be easy. The last one might be harder.","commit_id":"f6059e5fa02187dacdb9c867c33bbf2025b2d00d"},{"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":"9a5191964bc85ccba9b7606b6d3fc762a3758cc7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"c60d2cac_53413271","in_reply_to":"2c933f20_6f7f33c2","updated":"2022-05-25 13:01:47.000000000","message":"Yes, you are.","commit_id":"f6059e5fa02187dacdb9c867c33bbf2025b2d00d"},{"author":{"_account_id":27339,"name":"Michal Arbet","email":"michal.arbet@ultimum.io","username":"michalarbet"},"change_message_id":"fefc8d6c3f9a79c4b7924624746b9e582bb2d4b6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"2c933f20_6f7f33c2","in_reply_to":"49af2b56_fef93ee8","updated":"2022-05-25 11:08:45.000000000","message":"Hmm, if I am correct, last one is done by below command also : \nkolla-ansible -i ${RAW_INVENTORY} -vvv deploy --tags keystone","commit_id":"f6059e5fa02187dacdb9c867c33bbf2025b2d00d"},{"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":"440ed3ef64e403754bdee75138ba122f064bb538","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"7d0453fa_a8551779","updated":"2022-05-25 13:02:02.000000000","message":"(and docs)","commit_id":"bae84f0705629e31c72099528271ed51f96784fb"},{"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":"c0db11b5961acad4b676cad9ca79a43b5974a0bb","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"fc9512b0_e90b8f8a","updated":"2022-05-25 13:01:20.000000000","message":"ok, it automatically fixed the catalogue, awesome","commit_id":"bae84f0705629e31c72099528271ed51f96784fb"},{"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":"b35adead3027c98fbc36116a98292d304733f4f2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"b8dc7dae_6b7c7be6","updated":"2022-05-25 13:01:53.000000000","message":"only renos to go","commit_id":"bae84f0705629e31c72099528271ed51f96784fb"},{"author":{"_account_id":27339,"name":"Michal Arbet","email":"michal.arbet@ultimum.io","username":"michalarbet"},"change_message_id":"f4a6e5bdb6beacc9c5b5fd1619b9a83005afae82","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"f36c42b8_31521850","in_reply_to":"b8dc7dae_6b7c7be6","updated":"2022-05-26 13:02:50.000000000","message":"go ahead :D","commit_id":"bae84f0705629e31c72099528271ed51f96784fb"},{"author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"change_message_id":"7f32bae7377b411490f558c5ccd77dde0cc02d4d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"969e681c_a5510469","updated":"2022-05-30 10:47:10.000000000","message":"Do we need to update the endpoint during the upgrade?","commit_id":"42c2520144e4361d7da6a7b2b339061b96359785"},{"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":"d6eb6aa96527484d011d47c651484e0a4451c35c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"2e1a5b58_8d078a77","updated":"2022-05-27 07:22:16.000000000","message":"Thanks for the reviews. I\u0027m merging this with followup pending.","commit_id":"42c2520144e4361d7da6a7b2b339061b96359785"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"65f3091a408b58076098b79007fccd292549897a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"4a076c91_da94bd55","updated":"2022-05-26 21:26:39.000000000","message":"Thx for doing this, suggestion for further cleanup inline.","commit_id":"42c2520144e4361d7da6a7b2b339061b96359785"},{"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":"4824de41d8b61b5989035fff00ecc21bf771ed4d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"b248eee3_b43a49a3","in_reply_to":"145435cb_34458333","updated":"2022-05-30 11:47:23.000000000","message":"I say it will if we merge that one. And propose a discussion to support endpoint removal (also for the other services). This one is good as it is at the moment,","commit_id":"42c2520144e4361d7da6a7b2b339061b96359785"},{"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":"f74e0bd308e97989a0384dbae85c26eae3c1060c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"f984e0da_9a981c15","in_reply_to":"384d3591_4c73a775","updated":"2022-05-31 08:58:27.000000000","message":"The issue with your suggestion is that it will make keystone\u0027s role work differently. However, maybe it makes sense to actually always/everywhere call this tasklist on upgrade - and then also make it able to clean up useless endpoints. WDYT?","commit_id":"42c2520144e4361d7da6a7b2b339061b96359785"},{"author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"change_message_id":"70185368cb16e837d0f97c29456b9847a91c07d9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"384d3591_4c73a775","in_reply_to":"4dee7683_905216b5","updated":"2022-05-31 08:45:59.000000000","message":"Indeed, hence my original question, \"during the upgrade\". Why not just get it over with? If the user never does a deploy, they\u0027ll get into trouble when they upgrade to 2023.1. Simple:\n\n- import_tasks: register.yml","commit_id":"42c2520144e4361d7da6a7b2b339061b96359785"},{"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":"9b1736fe38b32f8ba76e9cc8da53688590223d1a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"4dee7683_905216b5","in_reply_to":"60bbb1a3_8ef6f5a2","updated":"2022-05-31 08:38:33.000000000","message":"Yes, that\u0027s why this change advises to run deploy to get rid of the old admin endpoint.","commit_id":"42c2520144e4361d7da6a7b2b339061b96359785"},{"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":"bdcd5319738bcc38debc40c02f182bd3abec7fd0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"f1c8e838_a9951b9c","in_reply_to":"7b9a3a4c_993d9c10","updated":"2022-05-31 09:11:13.000000000","message":"Ack, let\u0027s go with temporary then.","commit_id":"42c2520144e4361d7da6a7b2b339061b96359785"},{"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":"a752a93d39a35401f4ba6303fc408cf0c696075c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"f9162ac8_14f78dc1","in_reply_to":"969e681c_a5510469","updated":"2022-05-30 10:58:31.000000000","message":"It updates itself. See also: https://review.opendev.org/c/openstack/kolla/+/843751","commit_id":"42c2520144e4361d7da6a7b2b339061b96359785"},{"author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"change_message_id":"b201e7589ebac58a98e531611577cd5b522618a2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"60bbb1a3_8ef6f5a2","in_reply_to":"b248eee3_b43a49a3","updated":"2022-05-31 08:23:37.000000000","message":"That change affects the keystone bootstrap process, but AFAICT, we don\u0027t run that on upgrade.","commit_id":"42c2520144e4361d7da6a7b2b339061b96359785"},{"author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"change_message_id":"ce3c482f1b1a8848c1ca19579200b479b217f817","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"145435cb_34458333","in_reply_to":"f9162ac8_14f78dc1","updated":"2022-05-30 11:37:39.000000000","message":"How? In https://review.opendev.org/c/openstack/kolla/+/843751 you say it leaves a broken endpoint on upgrade.","commit_id":"42c2520144e4361d7da6a7b2b339061b96359785"},{"author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"change_message_id":"905ae2605227c510319bafe0673b514349ad6f18","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"7b9a3a4c_993d9c10","in_reply_to":"f984e0da_9a981c15","updated":"2022-05-31 09:04:07.000000000","message":"In general, I\u0027d say that when we perform a config migration such as this, where possible and sensible, the upgrade should just handle it. That might mean a temporary tweak to the upgrade procedure that we revert on the next upgrade.\n\nIf we haven\u0027t performed a config migration, the endpoints would only need updating if the user has changed config. In that case I\u0027d expect them to do it using deploy, ideally outside of the upgrade window.","commit_id":"42c2520144e4361d7da6a7b2b339061b96359785"}],"ansible/group_vars/all.yml":[{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"65f3091a408b58076098b79007fccd292549897a","unresolved":true,"context_lines":[{"line_number":846,"context_line":"keystone_internal_fqdn: \"{{ kolla_internal_fqdn }}\""},{"line_number":847,"context_line":"keystone_external_fqdn: \"{{ kolla_external_fqdn }}\""},{"line_number":848,"context_line":""},{"line_number":849,"context_line":"keystone_admin_url: \"{{ admin_protocol }}://{{ keystone_internal_fqdn | put_address_in_context(\u0027url\u0027) }}:{{ keystone_public_port }}\""},{"line_number":850,"context_line":"keystone_internal_url: \"{{ internal_protocol }}://{{ keystone_internal_fqdn | put_address_in_context(\u0027url\u0027) }}:{{ keystone_public_port }}\""},{"line_number":851,"context_line":"keystone_public_url: \"{{ public_protocol }}://{{ keystone_external_fqdn | put_address_in_context(\u0027url\u0027) }}:{{ keystone_public_port }}\""},{"line_number":852,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":6,"id":"169945d3_37b8ba2d","line":849,"updated":"2022-05-26 21:26:39.000000000","message":"I think we should also deprecate admin_protocol at the same time and replace with internal_protocol, doesn\u0027t make sense to keep these different, does it? Might be fine in a followup though.\n\nOr even deprecate this variable too and replace with keystone_internal_url?","commit_id":"42c2520144e4361d7da6a7b2b339061b96359785"},{"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":"d6eb6aa96527484d011d47c651484e0a4451c35c","unresolved":true,"context_lines":[{"line_number":846,"context_line":"keystone_internal_fqdn: \"{{ kolla_internal_fqdn }}\""},{"line_number":847,"context_line":"keystone_external_fqdn: \"{{ kolla_external_fqdn }}\""},{"line_number":848,"context_line":""},{"line_number":849,"context_line":"keystone_admin_url: \"{{ admin_protocol }}://{{ keystone_internal_fqdn | put_address_in_context(\u0027url\u0027) }}:{{ keystone_public_port }}\""},{"line_number":850,"context_line":"keystone_internal_url: \"{{ internal_protocol }}://{{ keystone_internal_fqdn | put_address_in_context(\u0027url\u0027) }}:{{ keystone_public_port }}\""},{"line_number":851,"context_line":"keystone_public_url: \"{{ public_protocol }}://{{ keystone_external_fqdn | put_address_in_context(\u0027url\u0027) }}:{{ keystone_public_port }}\""},{"line_number":852,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":6,"id":"49f2a9fd_82ac8167","line":849,"in_reply_to":"169945d3_37b8ba2d","updated":"2022-05-27 07:22:16.000000000","message":"Let\u0027s do both, in a followup. I will attack both the issues in there.","commit_id":"42c2520144e4361d7da6a7b2b339061b96359785"},{"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":"750d13da818964d0ef0220a674aef4d2d315b24d","unresolved":false,"context_lines":[{"line_number":846,"context_line":"keystone_internal_fqdn: \"{{ kolla_internal_fqdn }}\""},{"line_number":847,"context_line":"keystone_external_fqdn: \"{{ kolla_external_fqdn }}\""},{"line_number":848,"context_line":""},{"line_number":849,"context_line":"keystone_admin_url: \"{{ admin_protocol }}://{{ keystone_internal_fqdn | put_address_in_context(\u0027url\u0027) }}:{{ keystone_public_port }}\""},{"line_number":850,"context_line":"keystone_internal_url: \"{{ internal_protocol }}://{{ keystone_internal_fqdn | put_address_in_context(\u0027url\u0027) }}:{{ keystone_public_port }}\""},{"line_number":851,"context_line":"keystone_public_url: \"{{ public_protocol }}://{{ keystone_external_fqdn | put_address_in_context(\u0027url\u0027) }}:{{ keystone_public_port }}\""},{"line_number":852,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":6,"id":"22cdb4e8_6daa7ead","line":849,"in_reply_to":"49f2a9fd_82ac8167","updated":"2022-05-28 16:42:26.000000000","message":"https://review.opendev.org/c/openstack/kolla-ansible/+/843727","commit_id":"42c2520144e4361d7da6a7b2b339061b96359785"}],"ansible/roles/keystone/defaults/main.yml":[{"author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"change_message_id":"7f32bae7377b411490f558c5ccd77dde0cc02d4d","unresolved":true,"context_lines":[{"line_number":28,"context_line":"      # NOTE(yoctozepto): Admin port settings are kept only for upgrade compatibility."},{"line_number":29,"context_line":"      # TODO(yoctozepto): Remove after Zed."},{"line_number":30,"context_line":"      keystone_admin:"},{"line_number":31,"context_line":"        enabled: \"{{ enable_keystone and kolla_action \u003d\u003d \u0027upgrade\u0027 }}\""},{"line_number":32,"context_line":"        mode: \"http\""},{"line_number":33,"context_line":"        external: false"},{"line_number":34,"context_line":"        tls_backend: \"{{ keystone_enable_tls_backend }}\""}],"source_content_type":"text/x-yaml","patch_set":6,"id":"7de54934_5d33a6ae","line":31,"range":{"start_line":31,"start_character":21,"end_line":31,"end_character":36},"updated":"2022-05-30 10:47:10.000000000","message":"| bool","commit_id":"42c2520144e4361d7da6a7b2b339061b96359785"},{"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":"a752a93d39a35401f4ba6303fc408cf0c696075c","unresolved":false,"context_lines":[{"line_number":28,"context_line":"      # NOTE(yoctozepto): Admin port settings are kept only for upgrade compatibility."},{"line_number":29,"context_line":"      # TODO(yoctozepto): Remove after Zed."},{"line_number":30,"context_line":"      keystone_admin:"},{"line_number":31,"context_line":"        enabled: \"{{ enable_keystone and kolla_action \u003d\u003d \u0027upgrade\u0027 }}\""},{"line_number":32,"context_line":"        mode: \"http\""},{"line_number":33,"context_line":"        external: false"},{"line_number":34,"context_line":"        tls_backend: \"{{ keystone_enable_tls_backend }}\""}],"source_content_type":"text/x-yaml","patch_set":6,"id":"9b7b6cd1_96fe39bc","line":31,"range":{"start_line":31,"start_character":21,"end_line":31,"end_character":36},"in_reply_to":"7de54934_5d33a6ae","updated":"2022-05-30 10:58:31.000000000","message":"tbh, it\u0027s irrelevant, I could drop it entirely; the role depends on enable_keystone","commit_id":"42c2520144e4361d7da6a7b2b339061b96359785"}],"ansible/roles/venus/templates/venus.conf.j2":[{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"65f3091a408b58076098b79007fccd292549897a","unresolved":true,"context_lines":[{"line_number":23,"context_line":"project_name \u003d service\r"},{"line_number":24,"context_line":"password \u003d {{ venus_keystone_password }}\r"},{"line_number":25,"context_line":"username \u003d {{ venus_keystone_user }}\r"},{"line_number":26,"context_line":"auth_url \u003d {{ internal_protocol }}://{{ kolla_internal_fqdn }}:{{ keystone_public_port }}\r"},{"line_number":27,"context_line":"project_domain_id \u003d {{ default_project_domain_id }}\r"},{"line_number":28,"context_line":"user_domain_id \u003d {{ default_user_domain_id }}\r"},{"line_number":29,"context_line":"auth_type \u003d password\r"}],"source_content_type":"text/x-jinja2","patch_set":6,"id":"f7e948d2_cac167aa","line":26,"updated":"2022-05-26 21:26:39.000000000","message":"This was inconsistent before, using internal_protocol together with admin_port.","commit_id":"42c2520144e4361d7da6a7b2b339061b96359785"},{"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":"750d13da818964d0ef0220a674aef4d2d315b24d","unresolved":false,"context_lines":[{"line_number":23,"context_line":"project_name \u003d service\r"},{"line_number":24,"context_line":"password \u003d {{ venus_keystone_password }}\r"},{"line_number":25,"context_line":"username \u003d {{ venus_keystone_user }}\r"},{"line_number":26,"context_line":"auth_url \u003d {{ internal_protocol }}://{{ kolla_internal_fqdn }}:{{ keystone_public_port }}\r"},{"line_number":27,"context_line":"project_domain_id \u003d {{ default_project_domain_id }}\r"},{"line_number":28,"context_line":"user_domain_id \u003d {{ default_user_domain_id }}\r"},{"line_number":29,"context_line":"auth_type \u003d password\r"}],"source_content_type":"text/x-jinja2","patch_set":6,"id":"956ebf43_8346d885","line":26,"in_reply_to":"02e4a60b_05965218","updated":"2022-05-28 16:42:26.000000000","message":"https://review.opendev.org/c/openstack/kolla-ansible/+/843727","commit_id":"42c2520144e4361d7da6a7b2b339061b96359785"},{"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":"d6eb6aa96527484d011d47c651484e0a4451c35c","unresolved":true,"context_lines":[{"line_number":23,"context_line":"project_name \u003d service\r"},{"line_number":24,"context_line":"password \u003d {{ venus_keystone_password }}\r"},{"line_number":25,"context_line":"username \u003d {{ venus_keystone_user }}\r"},{"line_number":26,"context_line":"auth_url \u003d {{ internal_protocol }}://{{ kolla_internal_fqdn }}:{{ keystone_public_port }}\r"},{"line_number":27,"context_line":"project_domain_id \u003d {{ default_project_domain_id }}\r"},{"line_number":28,"context_line":"user_domain_id \u003d {{ default_user_domain_id }}\r"},{"line_number":29,"context_line":"auth_type \u003d password\r"}],"source_content_type":"text/x-jinja2","patch_set":6,"id":"02e4a60b_05965218","line":26,"in_reply_to":"f7e948d2_cac167aa","updated":"2022-05-27 07:22:16.000000000","message":"Yeah, it\u0027s wrong in fact. Need to use keystone_admin_url or rather keystone_internal_url going forward. (They include proper address serialisation). This will be done in a followup.","commit_id":"42c2520144e4361d7da6a7b2b339061b96359785"}],"doc/source/user/operating-kolla.rst":[{"author":{"_account_id":27339,"name":"Michal Arbet","email":"michal.arbet@ultimum.io","username":"michalarbet"},"change_message_id":"05ab6f8a59a1b62853064d8d5d4737f0f36cc56d","unresolved":true,"context_lines":[{"line_number":201,"context_line":""},{"line_number":202,"context_line":"   kolla-ansible deploy --tags keystone"},{"line_number":203,"context_line":""},{"line_number":204,"context_line":"After this command is complete, there are no leftovers of the admin port."},{"line_number":205,"context_line":""},{"line_number":206,"context_line":"Tips and Tricks"},{"line_number":207,"context_line":"~~~~~~~~~~~~~~~"}],"source_content_type":"text/x-rst","patch_set":4,"id":"cc5e9fab_7fe4a9c7","line":204,"range":{"start_line":204,"start_character":0,"end_line":204,"end_character":73},"updated":"2022-05-26 13:35:05.000000000","message":"Maybe it would be good to let users know there is leftover in admin.rc file which can be also regenerated by post deploy ?","commit_id":"1e7d53fcf37447566281ce73bc22c0c61ba9dadc"},{"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":"7babcf9f2f792a8a2362eee1b16be5de584b75bd","unresolved":false,"context_lines":[{"line_number":201,"context_line":""},{"line_number":202,"context_line":"   kolla-ansible deploy --tags keystone"},{"line_number":203,"context_line":""},{"line_number":204,"context_line":"After this command is complete, there are no leftovers of the admin port."},{"line_number":205,"context_line":""},{"line_number":206,"context_line":"Tips and Tricks"},{"line_number":207,"context_line":"~~~~~~~~~~~~~~~"}],"source_content_type":"text/x-rst","patch_set":4,"id":"403c52dd_a56eb9b6","line":204,"range":{"start_line":204,"start_character":0,"end_line":204,"end_character":73},"in_reply_to":"2de8ac23_05a4de67","updated":"2022-05-26 13:39:15.000000000","message":"Done","commit_id":"1e7d53fcf37447566281ce73bc22c0c61ba9dadc"},{"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":"fea6746b236250e3f988485b6e506e0ec240a906","unresolved":true,"context_lines":[{"line_number":201,"context_line":""},{"line_number":202,"context_line":"   kolla-ansible deploy --tags keystone"},{"line_number":203,"context_line":""},{"line_number":204,"context_line":"After this command is complete, there are no leftovers of the admin port."},{"line_number":205,"context_line":""},{"line_number":206,"context_line":"Tips and Tricks"},{"line_number":207,"context_line":"~~~~~~~~~~~~~~~"}],"source_content_type":"text/x-rst","patch_set":4,"id":"2de8ac23_05a4de67","line":204,"range":{"start_line":204,"start_character":0,"end_line":204,"end_character":73},"in_reply_to":"cc5e9fab_7fe4a9c7","updated":"2022-05-26 13:35:41.000000000","message":"Good catch, will improve.","commit_id":"1e7d53fcf37447566281ce73bc22c0c61ba9dadc"}],"releasenotes/notes/keystone-admin-port-gone-1a28302df63aa70b.yaml":[{"author":{"_account_id":27339,"name":"Michal Arbet","email":"michal.arbet@ultimum.io","username":"michalarbet"},"change_message_id":"05ab6f8a59a1b62853064d8d5d4737f0f36cc56d","unresolved":true,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"upgrade:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    Keystone\u0027s admin interface no longer points to a separate port."},{"line_number":5,"context_line":"    On upgrade, the port is preserved to maintain the intermediate"},{"line_number":6,"context_line":"    compatibility. Users are advised to run the deploy command"},{"line_number":7,"context_line":"    afterwards to ensure port\u0027s cleanup. For more information,"},{"line_number":8,"context_line":"    please refer to the docs."}],"source_content_type":"text/x-yaml","patch_set":4,"id":"211f0906_381f887b","line":8,"range":{"start_line":4,"start_character":0,"end_line":8,"end_character":29},"updated":"2022-05-26 13:35:05.000000000","message":"Ditto as in docs.","commit_id":"1e7d53fcf37447566281ce73bc22c0c61ba9dadc"},{"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":"7babcf9f2f792a8a2362eee1b16be5de584b75bd","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"upgrade:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    Keystone\u0027s admin interface no longer points to a separate port."},{"line_number":5,"context_line":"    On upgrade, the port is preserved to maintain the intermediate"},{"line_number":6,"context_line":"    compatibility. Users are advised to run the deploy command"},{"line_number":7,"context_line":"    afterwards to ensure port\u0027s cleanup. For more information,"},{"line_number":8,"context_line":"    please refer to the docs."}],"source_content_type":"text/x-yaml","patch_set":4,"id":"7984fd1c_fda91142","line":8,"range":{"start_line":4,"start_character":0,"end_line":8,"end_character":29},"in_reply_to":"211f0906_381f887b","updated":"2022-05-26 13:39:15.000000000","message":"Done","commit_id":"1e7d53fcf37447566281ce73bc22c0c61ba9dadc"}]}