)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":1179,"name":"Clay Gerrard","email":"clay.gerrard@gmail.com","username":"clay-gerrard"},"change_message_id":"cd9024408b395be3357a5dad326d2a914d795911","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"102289a6_d4564179","updated":"2023-06-03 00:23:00.000000000","message":"I think the test failures are mostly just complaining about logging:\n\n\ttest/unit/container/test_sharder.py:6268: in _do_test_audit_shard_container\n\t    sharder.logger.get_lines_for_level(\u0027info\u0027))\n\tE   AssertionError: Lists differ: [\"Updated own shard range from... !\u003d [\u0027_really_merge_items took 0.0...\n\tE   \n\tE   First differing element 0:\n\tE   \"Updated own shard range from ShardRange\u003c\u0027j\u0027 to \u0027k\u0027 as of 1685747349.00000, (0, 0) as of 1685747349.00000, active as of 1685747349.00000\u003e to ShardRange\u003c\u0027k\u0027 to \u0027t\u0027 as of 1685747350.00000, (0, 0) as of 1685747350.00000, active as of 1685747350.00000\u003e, path: .shards_a/shard_c, db: /home/zuul/xfstmp/tmpuPvwhL/sda/containers/0/f3b/b3fac948f9f029603d4248b5e2d39f3b/b3fac948f9f029603d4248b5e2d39f3b.db\"\n\tE   \u0027_really_merge_items took 0.00626492500305\u0027\n\tE   \n\tE   Second list contains 1 additional elements.\n\tE   First extra element 1:\n\tE   \"Updated own shard range from ShardRange\u003c\u0027j\u0027 to \u0027k\u0027 as of 1685747349.00000, (0, 0) as of 1685747349.00000, active as of 1685747349.00000\u003e to ShardRange\u003c\u0027k\u0027 to \u0027t\u0027 as of 1685747350.00000, (0, 0) as of 1685747350.00000, active as of 1685747350.00000\u003e, path: .shards_a/shard_c, db: /home/zuul/xfstmp/tmpuPvwhL/sda/containers/0/f3b/b3fac948f9f029603d4248b5e2d39f3b/b3fac948f9f029603d4248b5e2d39f3b.db\"\n\tE   \n\tE   Diff is 862 characters long. Set self.maxDiff to None to see it.","commit_id":"f9656cbc7bd3846bc0b292e7f084551a58a5e01a"},{"author":{"_account_id":1179,"name":"Clay Gerrard","email":"clay.gerrard@gmail.com","username":"clay-gerrard"},"change_message_id":"7f6685ef483574ae627ccabd04b6bb30d75f0092","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"29ac360d_3deb9fed","updated":"2023-06-02 22:54:32.000000000","message":"none of this should merge really, but i wanted to share what i\u0027ve learned","commit_id":"f9656cbc7bd3846bc0b292e7f084551a58a5e01a"}],"swift/container/server.py":[{"author":{"_account_id":1179,"name":"Clay Gerrard","email":"clay.gerrard@gmail.com","username":"clay-gerrard"},"change_message_id":"7f6685ef483574ae627ccabd04b6bb30d75f0092","unresolved":true,"context_lines":[{"line_number":567,"context_line":"        started \u003d time.time()"},{"line_number":568,"context_line":"        spi \u003d broker.storage_policy_index"},{"line_number":569,"context_line":"        self.logger.info(\u0027spi property took %s\u0027 % ("},{"line_number":570,"context_line":"            time.time() - started))"},{"line_number":571,"context_line":"        if created:"},{"line_number":572,"context_line":"            return HTTPCreated(request\u003dreq, headers\u003d{"},{"line_number":573,"context_line":"                \u0027x-backend-storage-policy-index\u0027: spi})"}],"source_content_type":"text/x-python","patch_set":1,"id":"84259c4c_f20a0900","line":570,"updated":"2023-06-02 22:54:32.000000000","message":"Jun  2 22:51:08 saio container-6041: _really_merge_items took 0.024205923080444336\nJun  2 22:51:08 saio container-6041: spi property took 0.49563121795654297\n\n^ this is rebased on the \"use the db for shard range sum in get_info\" change - it was even worse on master.\n\nI\u0027ll re-write the property to do something smarter.","commit_id":"f9656cbc7bd3846bc0b292e7f084551a58a5e01a"}]}