)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":32704,"name":"Alfredo Garcia","email":"alfrgarc@redhat.com","username":"alfrgarc"},"change_message_id":"7445785ff0d4983e09ac29c867c6b38b63c9cbf0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"3eabdd85_76bd2c82","updated":"2026-04-27 13:34:38.000000000","message":"Code changes look good but I do have one concern before I give a +1","commit_id":"d5633344f508bedcf3035fd1013510fcc779aec8"},{"author":{"_account_id":34510,"name":"Maxim Sava","email":"msava@redhat.com","username":"msava@redhat.com"},"change_message_id":"54bab9ca95f03cddc1f62c07e1433c08493e1b94","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"4472536f_ad49cdf2","updated":"2026-04-23 09:33:04.000000000","message":"recheck","commit_id":"d5633344f508bedcf3035fd1013510fcc779aec8"}],"tempest/api/image/v2/test_images.py":[{"author":{"_account_id":32704,"name":"Alfredo Garcia","email":"alfrgarc@redhat.com","username":"alfrgarc"},"change_message_id":"7445785ff0d4983e09ac29c867c6b38b63c9cbf0","unresolved":true,"context_lines":[{"line_number":449,"context_line":"        # Get default store ID for download test"},{"line_number":450,"context_line":"        default_store_id \u003d None"},{"line_number":451,"context_line":"        for store in self.available_stores:"},{"line_number":452,"context_line":"            if store.get(\u0027default\u0027) \u003d\u003d \u0027true\u0027:"},{"line_number":453,"context_line":"                default_store_id \u003d store[\u0027id\u0027]"},{"line_number":454,"context_line":"                break"},{"line_number":455,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"20095fb6_43c0d133","line":452,"range":{"start_line":452,"start_character":0,"end_line":452,"end_character":46},"updated":"2026-04-27 13:34:38.000000000","message":"This is checking for the string \u0027true\u0027 and not if it is an actual bool. Is it possible that the value would be a Boolean and not a String? If so, you should do something like this instead:\nif store.get(\u0027default\u0027) \u003d\u003d \u0027true\u0027 or store.get(\u0027default\u0027) is True:\n\nThis also happens a few more times, I\u0027ll mark it below","commit_id":"d5633344f508bedcf3035fd1013510fcc779aec8"},{"author":{"_account_id":32704,"name":"Alfredo Garcia","email":"alfrgarc@redhat.com","username":"alfrgarc"},"change_message_id":"7445785ff0d4983e09ac29c867c6b38b63c9cbf0","unresolved":true,"context_lines":[{"line_number":476,"context_line":"        # Get default store ID"},{"line_number":477,"context_line":"        default_store_id \u003d None"},{"line_number":478,"context_line":"        for store in self.available_stores:"},{"line_number":479,"context_line":"            if store.get(\u0027default\u0027) \u003d\u003d \u0027true\u0027:"},{"line_number":480,"context_line":"                default_store_id \u003d store[\u0027id\u0027]"},{"line_number":481,"context_line":"                break"},{"line_number":482,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"ca9a839b_3caf60b4","line":479,"updated":"2026-04-27 13:34:38.000000000","message":"here^","commit_id":"d5633344f508bedcf3035fd1013510fcc779aec8"},{"author":{"_account_id":32704,"name":"Alfredo Garcia","email":"alfrgarc@redhat.com","username":"alfrgarc"},"change_message_id":"7445785ff0d4983e09ac29c867c6b38b63c9cbf0","unresolved":true,"context_lines":[{"line_number":485,"context_line":"        store2_id \u003d None"},{"line_number":486,"context_line":"        for store in self.available_stores:"},{"line_number":487,"context_line":"            if (store[\u0027id\u0027] !\u003d default_store_id and"},{"line_number":488,"context_line":"                    store.get(\u0027read-only\u0027) !\u003d \u0027true\u0027):"},{"line_number":489,"context_line":"                store2_id \u003d store[\u0027id\u0027]"},{"line_number":490,"context_line":"                break"},{"line_number":491,"context_line":"        if not store2_id:"}],"source_content_type":"text/x-python","patch_set":1,"id":"eaf586c3_4d1ca788","line":488,"updated":"2026-04-27 13:34:38.000000000","message":"here^","commit_id":"d5633344f508bedcf3035fd1013510fcc779aec8"},{"author":{"_account_id":32704,"name":"Alfredo Garcia","email":"alfrgarc@redhat.com","username":"alfrgarc"},"change_message_id":"7445785ff0d4983e09ac29c867c6b38b63c9cbf0","unresolved":true,"context_lines":[{"line_number":515,"context_line":"        # Get default store ID"},{"line_number":516,"context_line":"        default_store_id \u003d None"},{"line_number":517,"context_line":"        for store in self.available_stores:"},{"line_number":518,"context_line":"            if store.get(\u0027default\u0027) \u003d\u003d \u0027true\u0027:"},{"line_number":519,"context_line":"                default_store_id \u003d store[\u0027id\u0027]"},{"line_number":520,"context_line":"                break"},{"line_number":521,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"07134921_dc5a97a9","line":518,"updated":"2026-04-27 13:34:38.000000000","message":"here^","commit_id":"d5633344f508bedcf3035fd1013510fcc779aec8"},{"author":{"_account_id":32704,"name":"Alfredo Garcia","email":"alfrgarc@redhat.com","username":"alfrgarc"},"change_message_id":"7445785ff0d4983e09ac29c867c6b38b63c9cbf0","unresolved":true,"context_lines":[{"line_number":524,"context_line":"        store2_id \u003d None"},{"line_number":525,"context_line":"        for store in self.available_stores:"},{"line_number":526,"context_line":"            if (store[\u0027id\u0027] !\u003d default_store_id and"},{"line_number":527,"context_line":"                    store.get(\u0027read-only\u0027) !\u003d \u0027true\u0027):"},{"line_number":528,"context_line":"                store2_id \u003d store[\u0027id\u0027]"},{"line_number":529,"context_line":"                break"},{"line_number":530,"context_line":"        if not store2_id:"}],"source_content_type":"text/x-python","patch_set":1,"id":"16c7aabf_e946b908","line":527,"updated":"2026-04-27 13:34:38.000000000","message":"here^","commit_id":"d5633344f508bedcf3035fd1013510fcc779aec8"}]}
