)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":34452,"name":"Joan Gilabert","display_name":"jgilaber","email":"jgilaber@redhat.com","username":"jgilaber"},"change_message_id":"98574e1a5f6eb3365d5a5de4569303f66e3bf9c0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"7172b25f_7df3c50a","updated":"2025-08-19 17:14:59.000000000","message":"lgtm thanks Takashi!","commit_id":"1009c3781b267167651b8ab8c0d3a56d6243d872"},{"author":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"change_message_id":"93428fa13aef10f50cedd3119d7e3217007fd632","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"376fa226_ebe49081","updated":"2025-08-25 11:39:47.000000000","message":"lgtm, thanks","commit_id":"1009c3781b267167651b8ab8c0d3a56d6243d872"}],"watcher/conf/cinder_client.py":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"63d90108177a5684dfc41e46c1f8ce58b055b9d8","unresolved":true,"context_lines":[{"line_number":27,"context_line":"               help\u003d\u0027Version of Cinder API to use in cinderclient.\u0027),"},{"line_number":28,"context_line":"    cfg.StrOpt(\u0027endpoint_type\u0027,"},{"line_number":29,"context_line":"               default\u003d\u0027publicURL\u0027,"},{"line_number":30,"context_line":"               choices\u003d[\u0027publicURL\u0027, \u0027internalURL\u0027, \u0027adminURL\u0027],"},{"line_number":31,"context_line":"               help\u003d\u0027Type of endpoint to use in cinderclient.\u0027),"},{"line_number":32,"context_line":"    cfg.StrOpt(\u0027region_name\u0027,"},{"line_number":33,"context_line":"               help\u003d\u0027Region in Identity service catalog to use for \u0027"}],"source_content_type":"text/x-python","patch_set":3,"id":"d9328e7d_d8683f58","line":30,"updated":"2025-08-19 13:56:49.000000000","message":"if we are dong this we shoudl also include the non URL version as we actually recommend using public,internal and admin without the suffix i think in general.","commit_id":"ca2fcb0ed25975f09809fb7b7cd0491caf3fe031"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"8738a558e219447339d5b01754b5bf42955c5e7d","unresolved":false,"context_lines":[{"line_number":27,"context_line":"               help\u003d\u0027Version of Cinder API to use in cinderclient.\u0027),"},{"line_number":28,"context_line":"    cfg.StrOpt(\u0027endpoint_type\u0027,"},{"line_number":29,"context_line":"               default\u003d\u0027publicURL\u0027,"},{"line_number":30,"context_line":"               choices\u003d[\u0027publicURL\u0027, \u0027internalURL\u0027, \u0027adminURL\u0027],"},{"line_number":31,"context_line":"               help\u003d\u0027Type of endpoint to use in cinderclient.\u0027),"},{"line_number":32,"context_line":"    cfg.StrOpt(\u0027region_name\u0027,"},{"line_number":33,"context_line":"               help\u003d\u0027Region in Identity service catalog to use for \u0027"}],"source_content_type":"text/x-python","patch_set":3,"id":"501a4d77_2bd3e30f","line":30,"in_reply_to":"d9328e7d_d8683f58","updated":"2025-08-19 16:46:18.000000000","message":"Done","commit_id":"ca2fcb0ed25975f09809fb7b7cd0491caf3fe031"}],"watcher/conf/gnocchi_client.py":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"63d90108177a5684dfc41e46c1f8ce58b055b9d8","unresolved":true,"context_lines":[{"line_number":29,"context_line":"               default\u003d\u0027public\u0027,"},{"line_number":30,"context_line":"               choices\u003d[\u0027public\u0027, \u0027internal\u0027, \u0027admin\u0027,"},{"line_number":31,"context_line":"                        \u0027publicURL\u0027, \u0027internalURL\u0027, \u0027adminURL\u0027],"},{"line_number":32,"context_line":"               help\u003d\u0027Type of endpoint to use in gnocchi client.\u0027),"},{"line_number":33,"context_line":"    cfg.StrOpt(\u0027region_name\u0027,"},{"line_number":34,"context_line":"               help\u003d\u0027Region in Identity service catalog to use for \u0027"},{"line_number":35,"context_line":"                    \u0027communication with the OpenStack service.\u0027)"}],"source_content_type":"text/x-python","patch_set":3,"id":"bd6e90a8_16788e72","line":32,"updated":"2025-08-19 13:56:49.000000000","message":"basiclly we shoudl do this for all the endpoints.","commit_id":"ca2fcb0ed25975f09809fb7b7cd0491caf3fe031"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"8738a558e219447339d5b01754b5bf42955c5e7d","unresolved":false,"context_lines":[{"line_number":29,"context_line":"               default\u003d\u0027public\u0027,"},{"line_number":30,"context_line":"               choices\u003d[\u0027public\u0027, \u0027internal\u0027, \u0027admin\u0027,"},{"line_number":31,"context_line":"                        \u0027publicURL\u0027, \u0027internalURL\u0027, \u0027adminURL\u0027],"},{"line_number":32,"context_line":"               help\u003d\u0027Type of endpoint to use in gnocchi client.\u0027),"},{"line_number":33,"context_line":"    cfg.StrOpt(\u0027region_name\u0027,"},{"line_number":34,"context_line":"               help\u003d\u0027Region in Identity service catalog to use for \u0027"},{"line_number":35,"context_line":"                    \u0027communication with the OpenStack service.\u0027)"}],"source_content_type":"text/x-python","patch_set":3,"id":"97a5f12e_762f6034","line":32,"in_reply_to":"bd6e90a8_16788e72","updated":"2025-08-19 16:46:18.000000000","message":"Done","commit_id":"ca2fcb0ed25975f09809fb7b7cd0491caf3fe031"}],"watcher/conf/monasca_client.py":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"a2efc3eb094ab41157e05b813e774f17799c4f07","unresolved":false,"context_lines":[{"line_number":26,"context_line":"               default\u003d\u00272_0\u0027,"},{"line_number":27,"context_line":"               help\u003d\u0027Version of Monasca API to use in monascaclient.\u0027),"},{"line_number":28,"context_line":"    cfg.StrOpt(\u0027interface\u0027,"},{"line_number":29,"context_line":"               default\u003d\u0027internal\u0027,"},{"line_number":30,"context_line":"               choices\u003d[\u0027public\u0027, \u0027internal\u0027, \u0027admin\u0027,"},{"line_number":31,"context_line":"                        \u0027publicURL\u0027, \u0027internalURL\u0027, \u0027adminURL\u0027],"},{"line_number":32,"context_line":"               help\u003d\u0027Type of interface used for monasca endpoint.\u0027),"}],"source_content_type":"text/x-python","patch_set":4,"id":"2461e107_46294bef","line":29,"range":{"start_line":29,"start_character":24,"end_line":29,"end_character":32},"updated":"2025-08-19 16:48:59.000000000","message":"this is also a bug but im making this optional in a sperate patch and we are deleting it soon but note to self we shoud make sure we default to public in general.","commit_id":"1009c3781b267167651b8ab8c0d3a56d6243d872"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"7bbc7e7e528dd8b67ad22b02e8109b71d056b385","unresolved":false,"context_lines":[{"line_number":26,"context_line":"               default\u003d\u00272_0\u0027,"},{"line_number":27,"context_line":"               help\u003d\u0027Version of Monasca API to use in monascaclient.\u0027),"},{"line_number":28,"context_line":"    cfg.StrOpt(\u0027interface\u0027,"},{"line_number":29,"context_line":"               default\u003d\u0027internal\u0027,"},{"line_number":30,"context_line":"               choices\u003d[\u0027public\u0027, \u0027internal\u0027, \u0027admin\u0027,"},{"line_number":31,"context_line":"                        \u0027publicURL\u0027, \u0027internalURL\u0027, \u0027adminURL\u0027],"},{"line_number":32,"context_line":"               help\u003d\u0027Type of interface used for monasca endpoint.\u0027),"}],"source_content_type":"text/x-python","patch_set":4,"id":"fedb410b_cc9e665c","line":29,"range":{"start_line":29,"start_character":24,"end_line":29,"end_character":32},"in_reply_to":"2461e107_46294bef","updated":"2025-08-19 16:50:03.000000000","message":"Yeah I noticed the same but I left this because monasca is being retired.","commit_id":"1009c3781b267167651b8ab8c0d3a56d6243d872"}]}
