)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},"change_message_id":"3790f11af2c65675066aa3a38af4e1d4861d4e3f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"b1b97598_be4216d9","updated":"2021-12-17 14:42:36.000000000","message":"It would be nice to have similar for container_info lookups in memcache,but that will be much easier once we have this change [1] which will make the proxy app and its logger available\n\n[1] https://review.opendev.org/c/openstack/swift/+/821921/3/swift/proxy/controllers/base.py","commit_id":"e33b28b1d788744bc835a8afbc356ae6f0badc2f"},{"author":{"_account_id":1179,"name":"Clay Gerrard","email":"clay.gerrard@gmail.com","username":"clay-gerrard"},"change_message_id":"6ed304fd2fe6db3ad031cf2fc8e79ac2d2966166","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"68bc2c25_92fbb3b1","updated":"2021-12-17 17:26:34.000000000","message":"i\u0027m sure these new stats are great","commit_id":"e33b28b1d788744bc835a8afbc356ae6f0badc2f"}],"swift/proxy/controllers/base.py":[{"author":{"_account_id":1179,"name":"Clay Gerrard","email":"clay.gerrard@gmail.com","username":"clay-gerrard"},"change_message_id":"6ed304fd2fe6db3ad031cf2fc8e79ac2d2966166","unresolved":true,"context_lines":[{"line_number":2398,"context_line":"            shard_ranges, response \u003d self._get_shard_ranges("},{"line_number":2399,"context_line":"                req, account, container, states\u003d\u0027updating\u0027)"},{"line_number":2400,"context_line":"            self.app.logger.increment("},{"line_number":2401,"context_line":"                \u0027shard_updating.backend.%s\u0027 % response.status_int)"},{"line_number":2402,"context_line":"            if shard_ranges:"},{"line_number":2403,"context_line":"                cached_ranges \u003d [dict(sr) for sr in shard_ranges]"},{"line_number":2404,"context_line":"                # went to disk; cache it"}],"source_content_type":"text/x-python","patch_set":1,"id":"249bdb2f_5cbd98a5","line":2401,"updated":"2021-12-17 17:26:34.000000000","message":"what values do we think are reasonable for status_int - just 200?\n\ni guess 404 and 503 would both be valid and expected, in this case shard_ranges would be None","commit_id":"e33b28b1d788744bc835a8afbc356ae6f0badc2f"},{"author":{"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"},"change_message_id":"e61de0cd42a0212d2cfbbf7d6b8a12b8eca3ac20","unresolved":true,"context_lines":[{"line_number":2396,"context_line":"            shard_ranges, response \u003d self._get_shard_ranges("},{"line_number":2397,"context_line":"                req, account, container, states\u003d\u0027updating\u0027)"},{"line_number":2398,"context_line":"            self.app.logger.increment("},{"line_number":2399,"context_line":"                \u0027shard_updating.backend.%s\u0027 % response.status_int)"},{"line_number":2400,"context_line":"            if shard_ranges:"},{"line_number":2401,"context_line":"                cached_ranges \u003d [dict(sr) for sr in shard_ranges]"},{"line_number":2402,"context_line":"                # went to disk; cache it"}],"source_content_type":"text/x-python","patch_set":2,"id":"7b837ca1_b17eff08","line":2399,"updated":"2021-12-20 23:31:43.000000000","message":"OK, so the expectation is that we\u0027ll periodically see spikes on both shard_updater.cache.miss and shard_updating.backend.503 until we get the next patch in the chain working for us, at which point those both go to nil and we start seeing this low-level background of shard_updating.backend.200\n\nI like it.","commit_id":"40e0f577a9d5fea6f9cee30092ac05f98fdb1687"}]}
