)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":28619,"name":"Dmitriy Rabotyagov","email":"noonedeadpunk@gmail.com","username":"noonedeadpunk"},"change_message_id":"278238d37c3de38b8c8c135f06bc4e1626ef5122","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"05e97886_87b3dc76","updated":"2023-05-18 15:20:24.000000000","message":"recheck focal failed lxc cache preparation","commit_id":"70c48b47b26d3743b7b1befa3562dd69becac05a"},{"author":{"_account_id":28619,"name":"Dmitriy Rabotyagov","email":"noonedeadpunk@gmail.com","username":"noonedeadpunk"},"change_message_id":"c21d7db8fed3573422ea26f097f99c68ab41af38","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"bd17ff2d_fdcf5af2","updated":"2023-05-22 11:06:25.000000000","message":"recheck rocky upgrade","commit_id":"70c48b47b26d3743b7b1befa3562dd69becac05a"},{"author":{"_account_id":28619,"name":"Dmitriy Rabotyagov","email":"noonedeadpunk@gmail.com","username":"noonedeadpunk"},"change_message_id":"2a1a7d12f442c0705c06290bf752a392afe7c795","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"81527b7b_611b4bb3","updated":"2023-05-22 15:03:28.000000000","message":"recheck stepca","commit_id":"70c48b47b26d3743b7b1befa3562dd69becac05a"}],"playbooks/healthcheck-openstack.yml":[{"author":{"_account_id":25023,"name":"Jonathan Rosser","email":"jonathan.rosser@rd.bbc.co.uk","username":"jrosser"},"change_message_id":"b0b3a1ce1d3d7aa55df89b2519368d673bbb9c8e","unresolved":true,"context_lines":[{"line_number":38,"context_line":"            verify: \"{{ keystone_service_internaluri_insecure | ternary(false, true) }}\""},{"line_number":39,"context_line":"          with_items:"},{"line_number":40,"context_line":"            - internal"},{"line_number":41,"context_line":"            - public"},{"line_number":42,"context_line":"          register: service_catalog"},{"line_number":43,"context_line":""},{"line_number":44,"context_line":"        - name: Show service catalog"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"74651c57_245c87a1","line":41,"range":{"start_line":41,"start_character":14,"end_line":41,"end_character":20},"updated":"2023-05-18 13:01:14.000000000","message":"this is fine for AIO/testing scenario but i don\u0027t think it is guaranteed that utility has external connectivity","commit_id":"70c48b47b26d3743b7b1befa3562dd69becac05a"},{"author":{"_account_id":25023,"name":"Jonathan Rosser","email":"jonathan.rosser@rd.bbc.co.uk","username":"jrosser"},"change_message_id":"961a851406d7b9d89a0ce272d2c6db13c84a9195","unresolved":true,"context_lines":[{"line_number":38,"context_line":"            verify: \"{{ keystone_service_internaluri_insecure | ternary(false, true) }}\""},{"line_number":39,"context_line":"          with_items:"},{"line_number":40,"context_line":"            - internal"},{"line_number":41,"context_line":"            - public"},{"line_number":42,"context_line":"          register: service_catalog"},{"line_number":43,"context_line":""},{"line_number":44,"context_line":"        - name: Show service catalog"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"9c31317b_3353c80d","line":41,"range":{"start_line":41,"start_character":14,"end_line":41,"end_character":20},"in_reply_to":"04a95bb5_6a78186b","updated":"2023-05-18 14:35:55.000000000","message":"the code is fine, so long as we understand what its for - i\u0027m happy with this","commit_id":"70c48b47b26d3743b7b1befa3562dd69becac05a"},{"author":{"_account_id":28619,"name":"Dmitriy Rabotyagov","email":"noonedeadpunk@gmail.com","username":"noonedeadpunk"},"change_message_id":"68fbf04d1bde37a34c1ee88795aef5527f836770","unresolved":true,"context_lines":[{"line_number":38,"context_line":"            verify: \"{{ keystone_service_internaluri_insecure | ternary(false, true) }}\""},{"line_number":39,"context_line":"          with_items:"},{"line_number":40,"context_line":"            - internal"},{"line_number":41,"context_line":"            - public"},{"line_number":42,"context_line":"          register: service_catalog"},{"line_number":43,"context_line":""},{"line_number":44,"context_line":"        - name: Show service catalog"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"04a95bb5_6a78186b","line":41,"range":{"start_line":41,"start_character":14,"end_line":41,"end_character":20},"in_reply_to":"74651c57_245c87a1","updated":"2023-05-18 14:33:02.000000000","message":"well, yes, but I\u0027m not sure if this playbook is really used anywhere by anyone, as it\u0027s quite opionated already.\n\nThe problem I faced, is that keystone_service_\u003ctype\u003euri is avaialable only for keystone_all group_vars. Also then we really don\u0027t know what would have access to the public network.","commit_id":"70c48b47b26d3743b7b1befa3562dd69becac05a"},{"author":{"_account_id":32666,"name":"Damian Dąbrowski","email":"damian@dabrowski.cloud","username":"ddabrowski"},"change_message_id":"b4b2d0f9753e257ec1e8f5bd0167a52babfe6708","unresolved":true,"context_lines":[{"line_number":38,"context_line":"            verify: \"{{ keystone_service_internaluri_insecure | ternary(false, true) }}\""},{"line_number":39,"context_line":"          with_items:"},{"line_number":40,"context_line":"            - internal"},{"line_number":41,"context_line":"            - public"},{"line_number":42,"context_line":"          register: service_catalog"},{"line_number":43,"context_line":""},{"line_number":44,"context_line":"        - name: Show service catalog"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"b48406b6_a5bc1c14","line":41,"range":{"start_line":41,"start_character":14,"end_line":41,"end_character":20},"in_reply_to":"9c31317b_3353c80d","updated":"2023-05-23 12:13:05.000000000","message":"so if we focus just on supporting this file for AIO/testing, maybe `scripts/` directory is a better place for it?\n\nAnother option would be to set a proper comment inside this file.","commit_id":"70c48b47b26d3743b7b1befa3562dd69becac05a"},{"author":{"_account_id":28619,"name":"Dmitriy Rabotyagov","email":"noonedeadpunk@gmail.com","username":"noonedeadpunk"},"change_message_id":"99edeb21d04629cf1cd4c5192b8e25fccb2b2032","unresolved":true,"context_lines":[{"line_number":38,"context_line":"            verify: \"{{ keystone_service_internaluri_insecure | ternary(false, true) }}\""},{"line_number":39,"context_line":"          with_items:"},{"line_number":40,"context_line":"            - internal"},{"line_number":41,"context_line":"            - public"},{"line_number":42,"context_line":"          register: service_catalog"},{"line_number":43,"context_line":""},{"line_number":44,"context_line":"        - name: Show service catalog"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"e8c83794_2e03648c","line":41,"range":{"start_line":41,"start_character":14,"end_line":41,"end_character":20},"in_reply_to":"b48406b6_a5bc1c14","updated":"2023-05-25 21:31:27.000000000","message":"We don\u0027t really have playbooks under scripts, do we? Except ones, that are called from bootstrap-ansible.sh?\n\nI would rather think of a solution how to make it more usable outside of CI to be frank, but not sure how to do that in a good way. As indeed some envs don\u0027t have access to public endpoints, and some do. Maybe we can make this as a configurable variable... But would rather do this in follow-up to be frank","commit_id":"70c48b47b26d3743b7b1befa3562dd69becac05a"},{"author":{"_account_id":32666,"name":"Damian Dąbrowski","email":"damian@dabrowski.cloud","username":"ddabrowski"},"change_message_id":"b4b2d0f9753e257ec1e8f5bd0167a52babfe6708","unresolved":true,"context_lines":[{"line_number":50,"context_line":""},{"line_number":51,"context_line":""},{"line_number":52,"context_line":"    - name: Test Glance"},{"line_number":53,"context_line":"      block:"},{"line_number":54,"context_line":"        - name: Image(s) download"},{"line_number":55,"context_line":"          get_url:"},{"line_number":56,"context_line":"            url: \"{{ item.url }}\""}],"source_content_type":"text/x-yaml","patch_set":1,"id":"b3624a10_429bb239","line":53,"updated":"2023-05-23 12:13:05.000000000","message":"note: this block is not really needed here and in few other places as we have only one task here.\n\nBut I can understand that we still may want to use it for grouping tasks on service basis.","commit_id":"70c48b47b26d3743b7b1befa3562dd69becac05a"},{"author":{"_account_id":28619,"name":"Dmitriy Rabotyagov","email":"noonedeadpunk@gmail.com","username":"noonedeadpunk"},"change_message_id":"99edeb21d04629cf1cd4c5192b8e25fccb2b2032","unresolved":true,"context_lines":[{"line_number":50,"context_line":""},{"line_number":51,"context_line":""},{"line_number":52,"context_line":"    - name: Test Glance"},{"line_number":53,"context_line":"      block:"},{"line_number":54,"context_line":"        - name: Image(s) download"},{"line_number":55,"context_line":"          get_url:"},{"line_number":56,"context_line":"            url: \"{{ item.url }}\""}],"source_content_type":"text/x-yaml","patch_set":1,"id":"32d576c0_fa294710","line":53,"in_reply_to":"b3624a10_429bb239","updated":"2023-05-25 21:31:27.000000000","message":"yeah, I added it more for consistency","commit_id":"70c48b47b26d3743b7b1befa3562dd69becac05a"}]}