)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"change_message_id":"e6dc88b7ce77cebe7fc3e2341b8922226fba12f2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"705991e7_173cae52","updated":"2026-02-05 15:32:13.000000000","message":"Can we have unit test for this!!","commit_id":"954c21ba475a9c85c355957852794f2141921008"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"5bd3d1aba6a28bb528ac0c96eef234505e2ba704","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":3,"id":"8b9ef5da_e0c3bf93","in_reply_to":"705991e7_173cae52","updated":"2026-02-06 15:51:51.000000000","message":"I\u0027m wondering what is the additional coverage we can add.\n\nThe key point is that we should always pass down request context to get_updated_store_location from glance\u0027s perspective because the other parts are within glance_store.\n\nThis change updates get_updated_store_location to require context and the existing unit test coverage for API call to replace/add location allows us to verify that.\n\nI could probably inject additional mocking of get_updated_store_location to assert the context argument value being passed but I wonder if that\u0027s \"too-deep\" inspection.","commit_id":"954c21ba475a9c85c355957852794f2141921008"},{"author":{"_account_id":8122,"name":"Cyril Roelandt","email":"cyril@redhat.com","username":"cyril.roelandt.enovance"},"change_message_id":"1f5412034b13c20e22c4c4581e0c50cdff0feae7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"558bbaee_9d0f7894","updated":"2026-03-17 16:11:24.000000000","message":"Is this a bugfix?","commit_id":"78e61c6ce7425e928770b4bc1eb04f229bd65783"},{"author":{"_account_id":8122,"name":"Cyril Roelandt","email":"cyril@redhat.com","username":"cyril.roelandt.enovance"},"change_message_id":"2efe104da719f452d6f46bed02e34fd2672da5f2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"b926f61b_f8a0b31d","updated":"2026-03-17 16:19:38.000000000","message":"OK apparently this has not triggered a bug yet, so no need for release notes/LP tracking, let\u0027s merge this as is.","commit_id":"78e61c6ce7425e928770b4bc1eb04f229bd65783"}]}
