)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":597,"name":"Pete Zaitcev","email":"zaitcev@kotori.zaitcev.us","username":"zaitcev"},"change_message_id":"4dcea3211996ff19fb524303804bc28a851d7bbb","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"90fe8907_96e161fd","updated":"2021-12-03 21:07:13.000000000","message":"\"KeyError: \u0027replication_ip\u0027\" ugh.\nLooks like tests weren\u0027t sufficient.","commit_id":"22784e2931c82fe050dd7133fdc7fd787acfc9b6"},{"author":{"_account_id":597,"name":"Pete Zaitcev","email":"zaitcev@kotori.zaitcev.us","username":"zaitcev"},"change_message_id":"1d748eabe7f0108c3b478bf91fc6c88bcdbe7992","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"1eda0160_d74b09fb","updated":"2021-12-03 20:05:00.000000000","message":"Fully onboard with the idea; the patch looks fine; test updates are sufficient, I think.","commit_id":"22784e2931c82fe050dd7133fdc7fd787acfc9b6"},{"author":{"_account_id":597,"name":"Pete Zaitcev","email":"zaitcev@kotori.zaitcev.us","username":"zaitcev"},"change_message_id":"ef3e529d83c131a909b9e19a234b2167d58b77d8","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"357fadcc_64aa88fe","updated":"2021-12-04 00:59:08.000000000","message":"I take it you agree with the fix-up.","commit_id":"11831ffaaf7aaf271265214f12cd121371e0d3d3"},{"author":{"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},"change_message_id":"397e393035e71b0c66288f76954bc11b8dcece65","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"aabc85d6_76df812c","updated":"2021-12-06 18:11:49.000000000","message":"LGTM, tests could cover replication_ip a little better, see follow up at https://review.opendev.org/c/openstack/swift/+/820579\n\nI\u0027m happy for this to be merged as it is and the follow up separately, or for the follow up to be squashed in.","commit_id":"11831ffaaf7aaf271265214f12cd121371e0d3d3"},{"author":{"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"},"change_message_id":"5c372d7fe92cc840bf0255177bebe99742ae8449","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"7393cea4_03f70894","updated":"2021-12-03 23:23:48.000000000","message":"recheck\n\nI feel like I\u0027ve seen a fair few timeouts now where things seem to hang somewhere in test.unit.common.test_utils.TestStatsdLogging -- though not always the same test 🤔","commit_id":"11831ffaaf7aaf271265214f12cd121371e0d3d3"},{"author":{"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"},"change_message_id":"8df1e51d3401012db40ae6319b847d5d8394b39f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"d272f711_4031afc0","updated":"2021-12-06 18:39:07.000000000","message":"Both sets of fixups look great, thanks all :-)","commit_id":"0d94072dcf9d2e24da087a516945e79d02a06f9a"},{"author":{"_account_id":597,"name":"Pete Zaitcev","email":"zaitcev@kotori.zaitcev.us","username":"zaitcev"},"change_message_id":"4206de74bd977e497c019a71fd19cf329f9d1697","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"e34a31d3_ae6f5a4b","updated":"2021-12-06 19:04:42.000000000","message":"Well that string value of port may be","commit_id":"fed1facad483c3baf2843f5750735b01f68da154"},{"author":{"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"},"change_message_id":"e02471e6fb5887d4269a8bb64e2f2a46d378d823","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"13238406_8f091b57","updated":"2021-12-06 20:54:16.000000000","message":"recheck","commit_id":"fed1facad483c3baf2843f5750735b01f68da154"},{"author":{"_account_id":597,"name":"Pete Zaitcev","email":"zaitcev@kotori.zaitcev.us","username":"zaitcev"},"change_message_id":"792c1807a01af6e3c0c5231c4b12bc87af27a04f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"7054843d_bae61805","updated":"2021-12-07 05:49:20.000000000","message":"recheck","commit_id":"fed1facad483c3baf2843f5750735b01f68da154"},{"author":{"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},"change_message_id":"c96cc2e7fd0be6f5d109cedf68e0580332789fb0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"1414baca_9a5dc3f5","updated":"2021-12-07 10:27:34.000000000","message":"fixed my attempt to be helpful (replication_port should not be a string!)","commit_id":"b75519f0e0e47169f4695a8d393bbe2f8921234e"},{"author":{"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"},"change_message_id":"1a0d769a97ae36a4927ffb89980a6e1a8d4854ef","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"42387c4b_9f111443","updated":"2021-12-08 19:05:30.000000000","message":"recheck","commit_id":"b75519f0e0e47169f4695a8d393bbe2f8921234e"},{"author":{"_account_id":597,"name":"Pete Zaitcev","email":"zaitcev@kotori.zaitcev.us","username":"zaitcev"},"change_message_id":"08bc1992a7accab5285a169d34122ec82d5dc497","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"6b15cf32_230bf25d","updated":"2021-12-09 06:03:50.000000000","message":"recheck\n\nFailed twice in the same job (grenade) but in different places of it.","commit_id":"b75519f0e0e47169f4695a8d393bbe2f8921234e"},{"author":{"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"},"change_message_id":"b05d8c554c2f980f87d07a1a1a67a7ba5181b4a5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"5f94a8c1_0fb2dd0b","updated":"2021-12-10 17:49:36.000000000","message":"recheck","commit_id":"eec595b123612a58e26116c7d57c0afcde006429"}]}
