)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":1179,"name":"Clay Gerrard","email":"clay.gerrard@gmail.com","username":"clay-gerrard"},"change_message_id":"cc9fbf9378f401be1e659860c09503751bf9f018","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"efd4e66c_744b937a","updated":"2025-09-29 15:48:07.000000000","message":"idk, this is all non-functional stuff - take it or leave it.","commit_id":"c989ea5737ef8317f18a3dfb64b2879855ec6d1a"},{"author":{"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},"change_message_id":"3a995740b9815b227ab63325077785315754abd8","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"db9641f5_9c1aebb9","updated":"2025-10-06 10:19:38.000000000","message":"LGTM but apparently it\u0027s going to need rebasing once the parent merges","commit_id":"1a372a77a2114ca922f76c71f6019a9e2e01b346"}],"swift/account/backend.py":[{"author":{"_account_id":1179,"name":"Clay Gerrard","email":"clay.gerrard@gmail.com","username":"clay-gerrard"},"change_message_id":"cc9fbf9378f401be1e659860c09503751bf9f018","unresolved":true,"context_lines":[{"line_number":229,"context_line":"        \"\"\""},{"line_number":230,"context_line":"        Logical namespace path used for logging."},{"line_number":231,"context_line":""},{"line_number":232,"context_line":"        For ContainerBroker this is \"\u003caccount\u003e/\u003ccontainer\u003e\";"},{"line_number":233,"context_line":"        for AccountBroker we return just \"\u003caccount\u003e\"."},{"line_number":234,"context_line":"        \"\"\""},{"line_number":235,"context_line":"        if self.account is None:"}],"source_content_type":"text/x-python","patch_set":1,"id":"c24156fd_ce21a54f","side":"PARENT","line":232,"updated":"2025-09-29 15:48:07.000000000","message":"To me the ContainerBroker behavior doesn\u0027t seem super relevant to document here; maybe for a comparison?   It just didn\u0027t seem very DRY.","commit_id":"9009af4b9695b44af75677f3a17c53f2b3b7a01a"},{"author":{"_account_id":34892,"name":"ASHWIN A NAIR","display_name":"indianwhocodes","email":"nairashwin952013@gmail.com","username":"indianwhocodes","status":"Nvidia"},"change_message_id":"3ea09f58981f1268abbd4c31d8c6a4e4475f965f","unresolved":false,"context_lines":[{"line_number":229,"context_line":"        \"\"\""},{"line_number":230,"context_line":"        Logical namespace path used for logging."},{"line_number":231,"context_line":""},{"line_number":232,"context_line":"        For ContainerBroker this is \"\u003caccount\u003e/\u003ccontainer\u003e\";"},{"line_number":233,"context_line":"        for AccountBroker we return just \"\u003caccount\u003e\"."},{"line_number":234,"context_line":"        \"\"\""},{"line_number":235,"context_line":"        if self.account is None:"}],"source_content_type":"text/x-python","patch_set":1,"id":"c329d372_bdedbe40","side":"PARENT","line":232,"in_reply_to":"c24156fd_ce21a54f","updated":"2025-09-29 19:00:24.000000000","message":"Acknowledged and better to remove","commit_id":"9009af4b9695b44af75677f3a17c53f2b3b7a01a"},{"author":{"_account_id":1179,"name":"Clay Gerrard","email":"clay.gerrard@gmail.com","username":"clay-gerrard"},"change_message_id":"cc9fbf9378f401be1e659860c09503751bf9f018","unresolved":true,"context_lines":[{"line_number":661,"context_line":"        indicator that we haven\u0027t populated yet."},{"line_number":662,"context_line":""},{"line_number":663,"context_line":"        On failure, we leave `self.account` as-is (likely None) so that"},{"line_number":664,"context_line":"        a future caller can try again."},{"line_number":665,"context_line":"        \"\"\""},{"line_number":666,"context_line":"        if self.account is None:"},{"line_number":667,"context_line":"            self.get_info()"}],"source_content_type":"text/x-python","patch_set":1,"id":"d439ba20_caa7ae13","side":"PARENT","line":664,"updated":"2025-09-29 15:48:07.000000000","message":"i don\u0027t know what this \"on failure\" was about; i\u0027m not sure what was failing.  If we can\u0027t call get_info we can\u0027t set self.account - but that\u0027s not really a property of this method?","commit_id":"9009af4b9695b44af75677f3a17c53f2b3b7a01a"},{"author":{"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},"change_message_id":"3a995740b9815b227ab63325077785315754abd8","unresolved":false,"context_lines":[{"line_number":661,"context_line":"        indicator that we haven\u0027t populated yet."},{"line_number":662,"context_line":""},{"line_number":663,"context_line":"        On failure, we leave `self.account` as-is (likely None) so that"},{"line_number":664,"context_line":"        a future caller can try again."},{"line_number":665,"context_line":"        \"\"\""},{"line_number":666,"context_line":"        if self.account is None:"},{"line_number":667,"context_line":"            self.get_info()"}],"source_content_type":"text/x-python","patch_set":1,"id":"c924b60a_e31e3941","side":"PARENT","line":664,"in_reply_to":"7a54ee05_78f141ca","updated":"2025-10-06 10:19:38.000000000","message":"\u003e Use of AI for docs in a previous revision resulted in that\n\nno use blaming AI, it\u0027s not listening","commit_id":"9009af4b9695b44af75677f3a17c53f2b3b7a01a"},{"author":{"_account_id":34892,"name":"ASHWIN A NAIR","display_name":"indianwhocodes","email":"nairashwin952013@gmail.com","username":"indianwhocodes","status":"Nvidia"},"change_message_id":"3ea09f58981f1268abbd4c31d8c6a4e4475f965f","unresolved":false,"context_lines":[{"line_number":661,"context_line":"        indicator that we haven\u0027t populated yet."},{"line_number":662,"context_line":""},{"line_number":663,"context_line":"        On failure, we leave `self.account` as-is (likely None) so that"},{"line_number":664,"context_line":"        a future caller can try again."},{"line_number":665,"context_line":"        \"\"\""},{"line_number":666,"context_line":"        if self.account is None:"},{"line_number":667,"context_line":"            self.get_info()"}],"source_content_type":"text/x-python","patch_set":1,"id":"7a54ee05_78f141ca","side":"PARENT","line":664,"in_reply_to":"d439ba20_caa7ae13","updated":"2025-09-29 19:00:24.000000000","message":"Use of AI for docs in a previous revision resulted in that","commit_id":"9009af4b9695b44af75677f3a17c53f2b3b7a01a"},{"author":{"_account_id":1179,"name":"Clay Gerrard","email":"clay.gerrard@gmail.com","username":"clay-gerrard"},"change_message_id":"cc9fbf9378f401be1e659860c09503751bf9f018","unresolved":true,"context_lines":[{"line_number":336,"context_line":""},{"line_number":337,"context_line":"    def get_info(self):"},{"line_number":338,"context_line":"        \"\"\""},{"line_number":339,"context_line":"        Get global data for the account."},{"line_number":340,"context_line":""},{"line_number":341,"context_line":"        :returns: dict with keys: account, created_at, put_timestamp,"},{"line_number":342,"context_line":"                  delete_timestamp, status_changed_at, container_count,"}],"source_content_type":"text/x-python","patch_set":1,"id":"050b6e3f_3b7a36b6","line":339,"updated":"2025-09-29 15:48:07.000000000","message":"FWIW this was what was on master until 960289: account-broker: add resilient path property with lazy cache | https://review.opendev.org/c/openstack/swift/+/960289","commit_id":"c989ea5737ef8317f18a3dfb64b2879855ec6d1a"},{"author":{"_account_id":34892,"name":"ASHWIN A NAIR","display_name":"indianwhocodes","email":"nairashwin952013@gmail.com","username":"indianwhocodes","status":"Nvidia"},"change_message_id":"3ea09f58981f1268abbd4c31d8c6a4e4475f965f","unresolved":false,"context_lines":[{"line_number":336,"context_line":""},{"line_number":337,"context_line":"    def get_info(self):"},{"line_number":338,"context_line":"        \"\"\""},{"line_number":339,"context_line":"        Get global data for the account."},{"line_number":340,"context_line":""},{"line_number":341,"context_line":"        :returns: dict with keys: account, created_at, put_timestamp,"},{"line_number":342,"context_line":"                  delete_timestamp, status_changed_at, container_count,"}],"source_content_type":"text/x-python","patch_set":1,"id":"0e4936c1_91d5262a","line":339,"in_reply_to":"050b6e3f_3b7a36b6","updated":"2025-09-29 19:00:24.000000000","message":"Acknowledged, fair enough to revert","commit_id":"c989ea5737ef8317f18a3dfb64b2879855ec6d1a"}]}