)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"830a5041fa53e3b6ce89ea46e02fcd031b6f9727","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"83c21eed_c0b6dbca","updated":"2025-10-03 17:10:51.000000000","message":"just an fyi but it looks like we dont really depend on neutron client at all\nthe code was just never deleted so this will take care fo that usage.\n\nill be looking at the same for glance next week as i think its also dead code form the same orgianl feature.","commit_id":"95e2647efde104c8ed0e076fc2cd2d8eca1b7d00"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"e5007e2a406354f4ec2e5e270c0048d82ba5c4b8","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"78afa8da_2859da49","updated":"2025-10-06 08:15:49.000000000","message":"thanks, I just checked watcher but had no strength to dig deep to remove neutronclient dependency","commit_id":"95e2647efde104c8ed0e076fc2cd2d8eca1b7d00"},{"author":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"change_message_id":"52d8765f288fd26235be244b86a94fc94610273a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"c154cd6d_d0a0a3ea","updated":"2025-10-21 18:55:57.000000000","message":"LGTM. I think that we need to remove neutron from services integration doc[1], since we are removing it from the code. We can have that in a follow up patch. Thanks!\n\n[1] https://docs.openstack.org/watcher/latest/integrations/index.html","commit_id":"24afb27d3c2b89659dd90b62124c734bbe6cfc85"},{"author":{"_account_id":34452,"name":"Joan Gilabert","display_name":"jgilaber","email":"jgilaber@redhat.com","username":"jgilaber"},"change_message_id":"8c9d0eadfb30cc26e0cd07f29635024ebdbbbfd6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"4524e851_3ef705df","updated":"2025-10-15 10:25:11.000000000","message":"lgtm, good to remove unused code","commit_id":"24afb27d3c2b89659dd90b62124c734bbe6cfc85"}],"watcher/common/nova_helper.py":[{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"e5007e2a406354f4ec2e5e270c0048d82ba5c4b8","unresolved":true,"context_lines":[{"line_number":21,"context_line":"from novaclient import api_versions"},{"line_number":22,"context_line":"from oslo_log import log"},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"import glanceclient.exc as glexceptions"},{"line_number":25,"context_line":"import novaclient.exceptions as nvexceptions"},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"from watcher.common import clients"}],"source_content_type":"text/x-python","patch_set":1,"id":"d2a835ed_ab83e3a8","side":"PARENT","line":24,"range":{"start_line":24,"start_character":27,"end_line":24,"end_character":39},"updated":"2025-10-06 08:15:49.000000000","message":"perhaps mention that glance dependency is also touched by this patch","commit_id":"0439681e4a9a5eb9e3515cb1335ffadadff53d5c"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"1e84f6588035471af40490c45d301fac52a6f238","unresolved":true,"context_lines":[{"line_number":21,"context_line":"from novaclient import api_versions"},{"line_number":22,"context_line":"from oslo_log import log"},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"import glanceclient.exc as glexceptions"},{"line_number":25,"context_line":"import novaclient.exceptions as nvexceptions"},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"from watcher.common import clients"}],"source_content_type":"text/x-python","patch_set":1,"id":"a477d98c_3f84f5da","side":"PARENT","line":24,"range":{"start_line":24,"start_character":27,"end_line":24,"end_character":39},"in_reply_to":"d2a835ed_ab83e3a8","updated":"2025-10-06 08:16:23.000000000","message":"I see now in the commit msg","commit_id":"0439681e4a9a5eb9e3515cb1335ffadadff53d5c"}]}
