)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":34860,"name":"Amit Uniyal","email":"auniyal@redhat.com","username":"auniyal"},"change_message_id":"b92fb5f00c009b0513d91d896a9cd27a0f12a68d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"7108befb_167f75a8","updated":"2024-04-15 20:26:01.000000000","message":"can you please update the other docs as well for service_user under same directory.\ndoc/source/install/compute-install-obs.rst\ndoc/source/install/controller-install-obs.rst\n.\netc..\n\nrest lgtm, I\u0027ll update my vote on respin, thanks for the patch.","commit_id":"36c4aac5cfc6239be0e8cf6cd5939f3cf3e833dc"},{"author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"change_message_id":"3bf76d9a777fa5a22a13ea7f17a5f9e8bf26b381","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"25e16b63_5ffa7b4a","updated":"2024-09-05 22:14:03.000000000","message":"OK, I see now based on a duplicate bug recently filed [1], that the point of this patch is the bring the [service_user] example into consistency with the rest of the examples in the file. I think this makes sense. -1 because https should be changed to http (the rest of the examples show http).\n\n[1] https://bugs.launchpad.net/nova/+bug/2078843","commit_id":"eeb6fe4c6ba20fd753d96aaad42fc8efe9237140"},{"author":{"_account_id":37598,"name":"Ivan Anfimov","display_name":"Ivan Anfimov","email":"lazekteam@gmail.com","username":"anfimovir"},"change_message_id":"39322e79ddac842d88e64ef72e406c255710e6ab","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"09151c20_9cc97b66","updated":"2025-05-30 14:27:04.000000000","message":"https://review.opendev.org/q/topic:%22doc-https%22\n\nI think we can close current MR.","commit_id":"eeb6fe4c6ba20fd753d96aaad42fc8efe9237140"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"7c3d5380c90b58086577d618e8b8a9306cd974d1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"8aeeefee_b4685201","updated":"2025-01-08 18:02:35.000000000","message":"i mentioned this on https://review.opendev.org/c/openstack/nova/+/938680\n\nbut i dont think this doc update is somethign we shoudl do.\n\nthe doc is not invliad it depend on the how you deployed keystone\n\ndeploy without tls and on the old default keyston port of 5000 is not what i would consider the best practice so i dont think we shoudl update the doc to provide examples of the leagcy way of deploying.","commit_id":"eeb6fe4c6ba20fd753d96aaad42fc8efe9237140"},{"author":{"_account_id":34860,"name":"Amit Uniyal","email":"auniyal@redhat.com","username":"auniyal"},"change_message_id":"a13efab62f4243ee5264c1ff6f79374be83fd732","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"dee0c36d_d3cd5501","updated":"2024-04-22 10:36:25.000000000","message":"lgtm","commit_id":"eeb6fe4c6ba20fd753d96aaad42fc8efe9237140"},{"author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"change_message_id":"f64f6acce4e055f3b4a4e923e27601ee7dad4dfc","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"87054e3e_bda20bde","in_reply_to":"8aeeefee_b4685201","updated":"2025-01-08 18:25:28.000000000","message":"While it\u0027s true the Keystone url depends on how it\u0027s deployed, I realized (and commented on Sep 05) that this patch is meaning to make the Keystone url consistent with the rest of the text on the doc page. I -1 because the other Keystone urls in these doc pages are http not https. So in its current state, this patch would not be adding consistency.","commit_id":"eeb6fe4c6ba20fd753d96aaad42fc8efe9237140"}],"doc/source/install/compute-install-obs.rst":[{"author":{"_account_id":34860,"name":"Amit Uniyal","email":"auniyal@redhat.com","username":"auniyal"},"change_message_id":"fbcac84e805686a009b93cc0415f95291425a727","unresolved":true,"context_lines":[{"line_number":100,"context_line":""},{"line_number":101,"context_line":"        [service_user]"},{"line_number":102,"context_line":"        send_service_user_token \u003d true"},{"line_number":103,"context_line":"        auth_url \u003d https://controller:5000/"},{"line_number":104,"context_line":"        auth_strategy \u003d keystone"},{"line_number":105,"context_line":"        auth_type \u003d password"},{"line_number":106,"context_line":"        project_domain_name \u003d Default"}],"source_content_type":"text/x-rst","patch_set":2,"id":"6798c50f_3c612213","line":103,"updated":"2024-05-02 05:22:59.000000000","message":"this is mostly depends on how identity endpoint is created, like devstack do not use ports for any service.\n\nbut in this doc because everywhere port is used and I could not find refrence of identity creation cmd in corrosponding docs https://opendev.org/openstack/keystone/src/branch/master/doc/source/install\n\nso I think this change is correct. but adding @Sean to confirm and help in merge this change.","commit_id":"eeb6fe4c6ba20fd753d96aaad42fc8efe9237140"}],"doc/source/install/compute-install-rdo.rst":[{"author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"change_message_id":"3bf76d9a777fa5a22a13ea7f17a5f9e8bf26b381","unresolved":true,"context_lines":[{"line_number":92,"context_line":""},{"line_number":93,"context_line":"        [service_user]"},{"line_number":94,"context_line":"        send_service_user_token \u003d true"},{"line_number":95,"context_line":"        auth_url \u003d https://controller:5000/"},{"line_number":96,"context_line":"        auth_strategy \u003d keystone"},{"line_number":97,"context_line":"        auth_type \u003d password"},{"line_number":98,"context_line":"        project_domain_name \u003d Default"}],"source_content_type":"text/x-rst","patch_set":2,"id":"21a0aa7d_f817a364","line":95,"range":{"start_line":95,"start_character":19,"end_line":95,"end_character":24},"updated":"2024-09-05 22:14:03.000000000","message":"This should be `http` to match the other `auth_url` in this file.","commit_id":"eeb6fe4c6ba20fd753d96aaad42fc8efe9237140"}]}