)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":16312,"name":"Alfredo Moralejo","email":"amoralej@redhat.com","username":"amoralej"},"change_message_id":"ea20e437d12acf20f4ca4c307eaf7725ceebe9fc","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"6460f766_ddd03721","updated":"2026-05-07 10:31:52.000000000","message":"LGTM, methods are not used at all in watcher codebase and there is no contract from helper methods to be used outside.","commit_id":"925e5f7f1d28ce2556688a01895bc6de40f45212"},{"author":{"_account_id":36630,"name":"Winicius Allan Bezerra da Silva","display_name":"Winicius Allan","email":"winiciusab12@gmail.com","username":"winiciusallan"},"change_message_id":"6b7a6a62f97f17af60cd99cf612aa10f4142c62e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"28306f80_1aecc508","updated":"2026-05-07 21:13:52.000000000","message":"Looks like these methods are not used anymore indeed. They were added a long time ago in this change https://review.opendev.org/c/openstack/watcher/+/477204 and it has no indication that is being used at this time.\n\nAny change of being used in the future?","commit_id":"925e5f7f1d28ce2556688a01895bc6de40f45212"},{"author":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"change_message_id":"da3bb43fa75592380ad2515e11f8ce3fa610d307","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"29361996_484cdb42","updated":"2026-05-15 14:20:14.000000000","message":"lgtm, this code is not used anymore and can be removed. I may W+1 together with the following ones.","commit_id":"925e5f7f1d28ce2556688a01895bc6de40f45212"},{"author":{"_account_id":34452,"name":"Joan Gilabert","display_name":"jgilaber","email":"jgilaber@redhat.com","username":"jgilaber"},"change_message_id":"7fa37405e2b8be3d4986e103641899c64df9915c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"0d6d5d01_0fcafcb6","in_reply_to":"28306f80_1aecc508","updated":"2026-05-08 06:25:43.000000000","message":"In case some of it is needed in the future we can reintroduce it but I strongly doubt it will. For context, this was used until a few releases ago, but it was used mostly to migrate volumes that were attached to a instance, but did so in a way that created a potential security issue https://bugs.launchpad.net/watcher/+bug/2112187","commit_id":"925e5f7f1d28ce2556688a01895bc6de40f45212"},{"author":{"_account_id":16312,"name":"Alfredo Moralejo","email":"amoralej@redhat.com","username":"amoralej"},"change_message_id":"9982b2d9aa1916ce08afb3c8db4701bcd6d44495","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"d4440413_a712c6b5","updated":"2026-05-19 07:03:47.000000000","message":"recheck","commit_id":"0bba1ac1da099aee6a9a2134893a63ceaa21b7b7"},{"author":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"change_message_id":"1463e43e8f9cd5a7321129c8a001e3da77c7f559","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"4ffe00ef_c816e1f6","updated":"2026-05-18 19:40:35.000000000","message":"recheck\n\ngrenade job failing deploy glance\nhttps://6d82362f2cdc504b27f1-9f757b11a1d2b00e739d31e1ecad199a.ssl.cf5.rackcdn.com/openstack/b323a88ff23a439e9ba0cf78895b8c6a/controller/logs/screen-g-api.txt","commit_id":"0bba1ac1da099aee6a9a2134893a63ceaa21b7b7"}]}
