)]}'
{"id":"openstack%2Fswift~774393","triplet_id":"openstack%2Fswift~master~I5fa39d41f9cd3b211e45d2012fd709f4135f595e","project":"openstack/swift","branch":"master","topic":"update_stats_sharded_split","hashtags":[],"change_id":"I5fa39d41f9cd3b211e45d2012fd709f4135f595e","subject":"Report final in_progress when sharding is complete","status":"MERGED","created":"2021-02-08 06:12:10.000000000","updated":"2021-02-26 18:45:51.000000000","submitted":"2021-02-26 18:42:36.000000000","submitter":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"total_comment_count":39,"unresolved_comment_count":31,"has_review_started":true,"submission_id":"774393-update_stats_sharded_split","meta_rev_id":"ef3e3983d4d4ac3d0ad974bbe4192cc1c64e79fc","_number":774393,"virtual_id_number":774393,"owner":{"_account_id":7233,"name":"Matthew Oliver","email":"matt@oliver.net.au","username":"mattoliverau"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"value":0,"_account_id":7233,"name":"Matthew Oliver","email":"matt@oliver.net.au","username":"mattoliverau"},{"value":0,"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},{"value":0,"_account_id":1179,"name":"Clay Gerrard","email":"clay.gerrard@gmail.com","username":"clay-gerrard"},{"tag":"autogenerated:zuul:gate","value":2,"date":"2021-02-26 18:42:31.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"values":{"-2":"Fails","-1":"Doesn\u0027t seem to work"," 0":"No score","+1":"Works for me","+2":"Verified"},"description":"","default_value":0,"optional":true},"Code-Review":{"approved":{"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},"all":[{"value":0,"_account_id":7233,"name":"Matthew Oliver","email":"matt@oliver.net.au","username":"mattoliverau"},{"value":2,"date":"2021-02-26 15:59:58.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},{"value":2,"date":"2021-02-26 16:17:24.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":1179,"name":"Clay Gerrard","email":"clay.gerrard@gmail.com","username":"clay-gerrard"},{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"values":{"-2":"Do not merge","-1":"This patch needs further work before it can be merged"," 0":"No score","+1":"Looks good to me, but someone else must approve","+2":"Looks good to me (core reviewer)"},"description":"","default_value":0,"optional":true},"Workflow":{"approved":{"_account_id":1179,"name":"Clay Gerrard","email":"clay.gerrard@gmail.com","username":"clay-gerrard"},"all":[{"value":0,"_account_id":7233,"name":"Matthew Oliver","email":"matt@oliver.net.au","username":"mattoliverau"},{"value":0,"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},{"value":1,"date":"2021-02-26 16:17:24.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":1179,"name":"Clay Gerrard","email":"clay.gerrard@gmail.com","username":"clay-gerrard"},{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":1179,"name":"Clay Gerrard","email":"clay.gerrard@gmail.com","username":"clay-gerrard"},{"_account_id":7233,"name":"Matthew Oliver","email":"matt@oliver.net.au","username":"mattoliverau"},{"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2021-02-08 07:20:37.000000000","updated_by":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"reviewer":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"state":"REVIEWER"},{"updated":"2021-02-08 13:52:09.000000000","updated_by":{"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},"reviewer":{"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},"state":"CC"},{"updated":"2021-02-23 13:25:05.000000000","updated_by":{"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},"reviewer":{"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},"state":"REVIEWER"},{"updated":"2021-02-26 16:17:24.000000000","updated_by":{"_account_id":1179,"name":"Clay Gerrard","email":"clay.gerrard@gmail.com","username":"clay-gerrard"},"reviewer":{"_account_id":1179,"name":"Clay Gerrard","email":"clay.gerrard@gmail.com","username":"clay-gerrard"},"state":"REVIEWER"}],"messages":[{"id":"c2683a51c97d89a7504baa882563d038362c3f37","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":7233,"name":"Matthew Oliver","email":"matt@oliver.net.au","username":"mattoliverau"},"date":"2021-02-08 06:12:10.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"177af4e427b720140682f551c2c68e7a9414bf40","author":{"_account_id":7233,"name":"Matthew Oliver","email":"matt@oliver.net.au","username":"mattoliverau"},"date":"2021-02-08 06:16:43.000000000","message":"Patch Set 1: Workflow-1\n\n(1 comment)\n\nStill need test and add tests. Just putting something up to start.","accounts_in_message":[],"_revision_number":1},{"id":"a0f8623365bde3f0d31018c369079eeca34a52d4","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-02-08 07:20:37.000000000","message":"Patch Set 1: Verified-1\n\nBuild failed (check pipeline).  For information on how to proceed, see\nhttps://docs.opendev.org/opendev/infra-manual/latest/developers.html#automated-testing\n\n\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/3db0e0285f2743eb90a0868e696e0cb6 : SUCCESS in 9m 32s\n- grenade https://zuul.opendev.org/t/openstack/build/32165efdf9914ee289e1f60d1cba09d1 : SUCCESS in 59m 12s\n- tempest-integrated-object-storage https://zuul.opendev.org/t/openstack/build/168b68aa5a764afdb1ea6602055fe712 : SUCCESS in 44m 30s\n- swift-build-image https://zuul.opendev.org/t/openstack/build/514d0726c65443f7a3ba5bfa643baa46 : SUCCESS in 8m 45s (non-voting)\n- swift-build-image-py3 https://zuul.opendev.org/t/openstack/build/6f9423ccbc05448f8b082d5357d46f19 : SUCCESS in 6m 09s (non-voting)\n- swift-tox-py27 https://zuul.opendev.org/t/openstack/build/b71f978b96ed4cf4a21f40841adf025b : FAILURE in 13m 52s\n- swift-tox-py36 https://zuul.opendev.org/t/openstack/build/523c7466b4ae4178afb46d6f6b22cde7 : FAILURE in 14m 23s\n- swift-tox-py37 https://zuul.opendev.org/t/openstack/build/ad43949a7df34de98bad155c8a725e8d : FAILURE in 14m 43s\n- swift-tox-py38 https://zuul.opendev.org/t/openstack/build/9fdeb919575046f2b76dca6721a863a2 : FAILURE in 13m 58s\n- swift-tox-func-py27 https://zuul.opendev.org/t/openstack/build/8f04bc1db7d64bae807baf46cc8b2883 : SUCCESS in 17m 07s\n- swift-tox-func-encryption-py27 https://zuul.opendev.org/t/openstack/build/21a82ac5152649ddb9bf9b8566e63469 : SUCCESS in 16m 15s\n- swift-tox-func-ec-py27 https://zuul.opendev.org/t/openstack/build/d342b96fc3204239940114993e14a3fb : SUCCESS in 17m 54s\n- swift-tox-func-py37 https://zuul.opendev.org/t/openstack/build/ad381b63ec5b41598a091a464ed4eeb1 : SUCCESS in 17m 47s\n- swift-tox-func-encryption-py37 https://zuul.opendev.org/t/openstack/build/2a293a198f874b71910d79b343cc2f49 : SUCCESS in 17m 36s\n- swift-tox-func-ec-py37 https://zuul.opendev.org/t/openstack/build/7646438b70a948728c68fe230c2e3b04 : SUCCESS in 19m 41s\n- swift-tox-func-s3api-ceph-s3tests-tempauth https://zuul.opendev.org/t/openstack/build/8959e6061cc947338cd92ce91da10f36 : SUCCESS in 24m 21s (non-voting)\n- swift-probetests-centos-7 https://zuul.opendev.org/t/openstack/build/23863f711af54262b98eecfa5e594fe0 : SUCCESS in 43m 46s\n- swift-probetests-centos-8 https://zuul.opendev.org/t/openstack/build/79bdc5d6a95f4dfa8edeeb5f79ec2191 : SUCCESS in 45m 38s\n- swift-dsvm-functional https://zuul.opendev.org/t/openstack/build/abffe8c1b1764dc89ec3673742bae498 : SUCCESS in 36m 13s\n- swift-dsvm-functional-ipv6 https://zuul.opendev.org/t/openstack/build/3dfa3220b9f64b19ad7623e18493e580 : SUCCESS in 35m 32s\n- swift-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/8143f7c40a264277a274999329ecee76 : FAILURE in 15m 18s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/07b318d3963846d392908772c347ae69 : SUCCESS in 6m 15s\n- swift-multinode-rolling-upgrade https://zuul.opendev.org/t/openstack/build/8a6cc7b5a48b4e6981b82bce48ee4161 : FAILURE in 9m 07s (non-voting)\n- tempest-ipv6-only https://zuul.opendev.org/t/openstack/build/92f1049456224e0d89b88ab0f6cf7108 : SUCCESS in 1h 00m 45s","accounts_in_message":[],"_revision_number":1},{"id":"1a033ee8c82423d3ee793b271167bb2d95ec7d47","author":{"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},"date":"2021-02-08 13:52:09.000000000","message":"Patch Set 1:\n\n(1 comment)","accounts_in_message":[],"_revision_number":1},{"id":"5ba33aed03112412cf1fa9c6efa5639e81534cf8","author":{"_account_id":7233,"name":"Matthew Oliver","email":"matt@oliver.net.au","username":"mattoliverau"},"date":"2021-02-09 00:37:27.000000000","message":"Patch Set 1:\n\n(1 comment)","accounts_in_message":[],"_revision_number":1},{"id":"bf70f236d190b59ee47a573045ede5dde5a6dfb4","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":7233,"name":"Matthew Oliver","email":"matt@oliver.net.au","username":"mattoliverau"},"date":"2021-02-18 10:01:59.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"5af8f6bbd97e78c3f7d9ae8556a838948eda5728","author":{"_account_id":7233,"name":"Matthew Oliver","email":"matt@oliver.net.au","username":"mattoliverau"},"date":"2021-02-18 10:03:00.000000000","message":"Patch Set 2:\n\n(1 comment)","accounts_in_message":[],"_revision_number":2},{"id":"67c01f4535fdbc20d3d581c5de80d337c158506a","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":7233,"name":"Matthew Oliver","email":"matt@oliver.net.au","username":"mattoliverau"},"date":"2021-02-18 10:04:16.000000000","message":"Uploaded patch set 3.","accounts_in_message":[],"_revision_number":3},{"id":"6a1cefd5efb93890fb4f77e791e69c2d02f7757e","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-02-18 13:30:18.000000000","message":"Patch Set 3: Verified-1\n\nBuild failed (check pipeline).  For information on how to proceed, see\nhttps://docs.opendev.org/opendev/infra-manual/latest/developers.html#automated-testing\n\n\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/067904ebfc7f41be9a5fe1fddc8cd525 : SUCCESS in 8m 41s\n- grenade https://zuul.opendev.org/t/openstack/build/b899e718b60844eebcec89a9ae75b757 : SUCCESS in 1h 08m 35s\n- tempest-integrated-object-storage https://zuul.opendev.org/t/openstack/build/7803a5c0de474a059b66b927987c566e : SUCCESS in 44m 19s\n- swift-build-image https://zuul.opendev.org/t/openstack/build/d218b6b5548445b8bf4453f95bd9d231 : SUCCESS in 9m 19s (non-voting)\n- swift-build-image-py3 https://zuul.opendev.org/t/openstack/build/c3736bf0bb4a479699075e6353ee742f : SUCCESS in 8m 23s (non-voting)\n- swift-tox-py27 https://zuul.opendev.org/t/openstack/build/30db9f44c1574cb9bdb442b7fb7f4758 : FAILURE in 15m 55s\n- swift-tox-py36 https://zuul.opendev.org/t/openstack/build/74f9301af612457d9ae4598306cca853 : FAILURE in 17m 18s\n- swift-tox-py37 https://zuul.opendev.org/t/openstack/build/020da848c2954930a61dc0c1373191f0 : FAILURE in 16m 45s\n- swift-tox-py38 https://zuul.opendev.org/t/openstack/build/e563db2b188d493cb81b3d40238b1b61 : FAILURE in 13m 57s\n- swift-tox-py39 https://zuul.opendev.org/t/openstack/build/8c58a6dd29384aa7983e9466ef064bc4 : FAILURE in 15m 54s\n- swift-tox-func-py27 https://zuul.opendev.org/t/openstack/build/7182beef54624878acc1b9cf94fba008 : SUCCESS in 16m 55s\n- swift-tox-func-encryption-py27 https://zuul.opendev.org/t/openstack/build/7a18e3d398664f00bbbb0287ebb70841 : SUCCESS in 17m 55s\n- swift-tox-func-ec-py27 https://zuul.opendev.org/t/openstack/build/60c117928de24d64827cf4134e480656 : FAILURE in 19m 52s\n- swift-tox-func-py37 https://zuul.opendev.org/t/openstack/build/b54c1422e3c54785885ff8849d777545 : FAILURE in 21m 48s\n- swift-tox-func-encryption-py37 https://zuul.opendev.org/t/openstack/build/b2641dad07a74f8db65e0e5fa06abf9c : SUCCESS in 17m 35s\n- swift-tox-func-ec-py37 https://zuul.opendev.org/t/openstack/build/e445ce42371f4be08f45f8b46eab93d7 : SUCCESS in 18m 42s\n- swift-tox-func-s3api-ceph-s3tests-tempauth https://zuul.opendev.org/t/openstack/build/5c542bd06571489f832f70273cb21cfd : FAILURE in 19m 35s (non-voting)\n- swift-probetests-centos-7 https://zuul.opendev.org/t/openstack/build/d06a9fbd0bde432d8f9d711dafb60fc5 : FAILURE in 31m 36s\n- swift-probetests-centos-8 https://zuul.opendev.org/t/openstack/build/a6fddb40fa744c23bf7d9ff10a091688 : FAILURE in 29m 08s\n- swift-dsvm-functional https://zuul.opendev.org/t/openstack/build/1c501383bead47a6a28351a15dc4a385 : SUCCESS in 1h 03m 21s\n- swift-dsvm-functional-ipv6 https://zuul.opendev.org/t/openstack/build/7da863074a36464fb34682b59f62959b : SUCCESS in 54m 51s\n- swift-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/ae4b6c20ca924218b6e253817cdbca96 : FAILURE in 17m 05s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/203712b81ea14ff09babca08975aa96c : SUCCESS in 7m 47s\n- swift-multinode-rolling-upgrade https://zuul.opendev.org/t/openstack/build/d7ac493447f64c598754862203b42005 : FAILURE in 10m 36s (non-voting)\n- tempest-ipv6-only https://zuul.opendev.org/t/openstack/build/1b69d545fe6f49d28c2e0603477ebf1b : SUCCESS in 1h 02m 39s","accounts_in_message":[],"_revision_number":3},{"id":"2250fd94d87e691edc260d3bb378e79301066656","author":{"_account_id":7233,"name":"Matthew Oliver","email":"matt@oliver.net.au","username":"mattoliverau"},"date":"2021-02-19 03:58:11.000000000","message":"Patch Set 3:\n\nThis needs a rebase. But that can wait until the rehashing finishes landing.","accounts_in_message":[],"_revision_number":3},{"id":"6a3a019e6ebee3e03a3bd3b13479f99fcb077eb5","author":{"_account_id":7233,"name":"Matthew Oliver","email":"matt@oliver.net.au","username":"mattoliverau"},"date":"2021-02-19 03:59:16.000000000","message":"Patch Set 3:\n\nopps wrong change, I have too mant tabs :P","accounts_in_message":[],"_revision_number":3},{"id":"22357aea2a80ffa8b1ae8e144c52e55e6a9330c9","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":7233,"name":"Matthew Oliver","email":"matt@oliver.net.au","username":"mattoliverau"},"date":"2021-02-19 05:12:12.000000000","message":"Uploaded patch set 4.","accounts_in_message":[],"_revision_number":4},{"id":"4dacc8e3e0dab529d773de5318caeeda8c117325","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-02-19 06:19:52.000000000","message":"Patch Set 4: Verified+1\n\nBuild succeeded (check pipeline).\n\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/c03a1983df9646f8bb2c84a3642dd975 : SUCCESS in 9m 07s\n- grenade https://zuul.opendev.org/t/openstack/build/42a40fcac3424334b0c9b1eb800be4f3 : SUCCESS in 1h 06m 51s\n- tempest-integrated-object-storage https://zuul.opendev.org/t/openstack/build/66c519be42cc48d9b7beccb51a60390b : SUCCESS in 51m 01s\n- swift-build-image https://zuul.opendev.org/t/openstack/build/e7e231f52fe14d70b9461cae8b0b1daa : SUCCESS in 8m 29s (non-voting)\n- swift-build-image-py3 https://zuul.opendev.org/t/openstack/build/b27b5a3ef3064a53b655dc8984e289fe : SUCCESS in 5m 56s (non-voting)\n- swift-tox-py27 https://zuul.opendev.org/t/openstack/build/be413066b056490598c78ff4c5b1e7f2 : SUCCESS in 14m 10s\n- swift-tox-py36 https://zuul.opendev.org/t/openstack/build/00ce10fb2e594912869365e365395252 : SUCCESS in 15m 33s\n- swift-tox-py37 https://zuul.opendev.org/t/openstack/build/0007236c4be74361ae6b2e3b2c493daf : SUCCESS in 15m 00s\n- swift-tox-py38 https://zuul.opendev.org/t/openstack/build/645ee1599a72446ab0860b092fdd9696 : SUCCESS in 14m 20s\n- swift-tox-py39 https://zuul.opendev.org/t/openstack/build/4cb98196181a4d03a5e4052b950051a3 : SUCCESS in 14m 25s\n- swift-tox-func-py27 https://zuul.opendev.org/t/openstack/build/5ea96444fbfb40f7a41e1ad9011579d4 : SUCCESS in 16m 28s\n- swift-tox-func-encryption-py27 https://zuul.opendev.org/t/openstack/build/9c27e0f76fd648a5a61b2395fd2e960a : SUCCESS in 17m 21s\n- swift-tox-func-ec-py27 https://zuul.opendev.org/t/openstack/build/690eba7895b84464a478606859fe0f7a : SUCCESS in 17m 26s\n- swift-tox-func-py37 https://zuul.opendev.org/t/openstack/build/d7598e9eb2c94f1bb3374bd6d91d2abb : SUCCESS in 18m 01s\n- swift-tox-func-encryption-py37 https://zuul.opendev.org/t/openstack/build/11ce2ed341744f3d87e92830103b30e2 : SUCCESS in 17m 10s\n- swift-tox-func-ec-py37 https://zuul.opendev.org/t/openstack/build/0c4a27a3a43c4e12a9f232e7798e4fa9 : SUCCESS in 18m 35s\n- swift-tox-func-s3api-ceph-s3tests-tempauth https://zuul.opendev.org/t/openstack/build/da2c59c8735e423b9aeae2efb5d1b722 : FAILURE in 20m 13s (non-voting)\n- swift-probetests-centos-7 https://zuul.opendev.org/t/openstack/build/582183d1a54f4772989b34a6f65e4e05 : SUCCESS in 44m 06s\n- swift-probetests-centos-8 https://zuul.opendev.org/t/openstack/build/e3e141f2d0f74ac9b19f07ddf310484c : SUCCESS in 49m 52s\n- swift-dsvm-functional https://zuul.opendev.org/t/openstack/build/5cea88d9899f43ffa0971fe99296fb12 : SUCCESS in 40m 23s\n- swift-dsvm-functional-ipv6 https://zuul.opendev.org/t/openstack/build/2938b2e478544bf894f9d834e3ab0698 : SUCCESS in 49m 14s\n- swift-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/46703f602bde48a4b5a4d39d2c8f57a7 : SUCCESS in 18m 55s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/9475a849b2f14312b8f353c11aff49f2 : SUCCESS in 6m 25s\n- swift-multinode-rolling-upgrade https://zuul.opendev.org/t/openstack/build/407bd3694d4141af9867c3cf790bab1b : FAILURE in 10m 33s (non-voting)\n- tempest-ipv6-only https://zuul.opendev.org/t/openstack/build/9b5ede816cd94122867ffa2dfc3ad907 : SUCCESS in 1h 02m 04s","accounts_in_message":[],"_revision_number":4},{"id":"c56d5c68c56eac52d139afc0f7a314690b71a8cb","author":{"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},"date":"2021-02-23 13:25:05.000000000","message":"Patch Set 4: Code-Review-1\n\n(6 comments)\n\nThis is going to be really useful. I wonder what you think about making it a fixed reporting period rather than being at the mercy of the cycle time?\n\nThe recon middleware change could be expedited on its own.","accounts_in_message":[],"_revision_number":4},{"id":"b52aba218514d81df98148c1f65cbb043bb82543","author":{"_account_id":7233,"name":"Matthew Oliver","email":"matt@oliver.net.au","username":"mattoliverau"},"date":"2021-02-24 00:17:42.000000000","message":"Patch Set 4:\n\n(4 comments)","accounts_in_message":[],"_revision_number":4},{"id":"6511e3e2cef5ec05c24f9668bea6d100c79f972b","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":7233,"name":"Matthew Oliver","email":"matt@oliver.net.au","username":"mattoliverau"},"date":"2021-02-24 05:03:25.000000000","message":"Uploaded patch set 5.","accounts_in_message":[],"_revision_number":5},{"id":"6d1c6b156207758ade0cb8f808a2dcb262e35cf4","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":7233,"name":"Matthew Oliver","email":"matt@oliver.net.au","username":"mattoliverau"},"date":"2021-02-24 05:22:36.000000000","message":"Uploaded patch set 6.","accounts_in_message":[],"_revision_number":6},{"id":"49abaa93166cf787ff14c58c892072950eca545b","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-02-24 06:16:32.000000000","message":"Patch Set 6: Verified+1\n\nBuild succeeded (check pipeline).\n\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/6ef223decf9f41e29a17c8fb0cb0cb68 : SUCCESS in 9m 49s\n- grenade https://zuul.opendev.org/t/openstack/build/ebf0adc132854344aff0f41c960c35db : SUCCESS in 52m 20s\n- tempest-integrated-object-storage https://zuul.opendev.org/t/openstack/build/0f3cc90989f74b1fa8329eecaf59a7ec : SUCCESS in 34m 40s\n- swift-build-image https://zuul.opendev.org/t/openstack/build/b2758dea7e404b46b4f5fd41f34eee68 : SUCCESS in 7m 28s (non-voting)\n- swift-build-image-py3 https://zuul.opendev.org/t/openstack/build/dba7f2d918174627a644b01cf709f4c2 : SUCCESS in 7m 41s (non-voting)\n- swift-tox-py27 https://zuul.opendev.org/t/openstack/build/0418d63b59ca410ab8efa67677399bac : SUCCESS in 23m 40s\n- swift-tox-py36 https://zuul.opendev.org/t/openstack/build/138c9cbae6db43659fd1283f54c5cd59 : SUCCESS in 33m 29s\n- swift-tox-py37 https://zuul.opendev.org/t/openstack/build/c147d2621e4a4c66813869972d123824 : SUCCESS in 20m 09s\n- swift-tox-py38 https://zuul.opendev.org/t/openstack/build/152813a8498d443dae27ee7cdc6b5ce0 : SUCCESS in 15m 00s\n- swift-tox-py39 https://zuul.opendev.org/t/openstack/build/0724169503004561b250462406d80219 : SUCCESS in 27m 56s\n- swift-tox-func-py27 https://zuul.opendev.org/t/openstack/build/41a0732219844fdca213c21c9f3df162 : SUCCESS in 20m 24s\n- swift-tox-func-encryption-py27 https://zuul.opendev.org/t/openstack/build/a19efe59e8e745c38b523f208fac4985 : SUCCESS in 37m 41s\n- swift-tox-func-ec-py27 https://zuul.opendev.org/t/openstack/build/bf3db32ce32f4716841c081e6bb72312 : SUCCESS in 38m 29s\n- swift-tox-func-py37 https://zuul.opendev.org/t/openstack/build/9472194383624355a01bb2dabccf41fc : SUCCESS in 26m 23s\n- swift-tox-func-encryption-py37 https://zuul.opendev.org/t/openstack/build/09ec1d5def8548e080d668e8353064e6 : SUCCESS in 31m 10s\n- swift-tox-func-ec-py37 https://zuul.opendev.org/t/openstack/build/ce66a0efd9f143bf8e652e728e827c03 : SUCCESS in 24m 34s\n- swift-tox-func-s3api-ceph-s3tests-tempauth https://zuul.opendev.org/t/openstack/build/f861dc2087db4bb386e280f71c163590 : SUCCESS in 22m 30s (non-voting)\n- swift-probetests-centos-7 https://zuul.opendev.org/t/openstack/build/58db720d4748439ab67c09af89b61737 : SUCCESS in 48m 06s\n- swift-probetests-centos-8 https://zuul.opendev.org/t/openstack/build/1fc295164bf34dc4a6239c18a5fecd67 : SUCCESS in 39m 53s\n- swift-dsvm-functional https://zuul.opendev.org/t/openstack/build/0fb01abcde8a42e2a76f94f8ff214f83 : SUCCESS in 35m 39s\n- swift-dsvm-functional-ipv6 https://zuul.opendev.org/t/openstack/build/b8be16920c844f258c95612931eea8d0 : SUCCESS in 35m 37s\n- swift-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/98a00817688f44da823f03331076056d : SUCCESS in 18m 04s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/46f7c97f02b14a91ab39bf40261246a6 : SUCCESS in 6m 11s\n- swift-multinode-rolling-upgrade https://zuul.opendev.org/t/openstack/build/0658c8239cb24c41ab4085a7b96b596a : FAILURE in 11m 02s (non-voting)\n- tempest-ipv6-only https://zuul.opendev.org/t/openstack/build/4ba86ae605a94a159196399bd103a6ff : SUCCESS in 51m 47s","accounts_in_message":[],"_revision_number":6},{"id":"d45ce9b92946ad228445dafe1668791a6530f440","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":7233,"name":"Matthew Oliver","email":"matt@oliver.net.au","username":"mattoliverau"},"date":"2021-02-25 04:51:27.000000000","message":"Uploaded patch set 7.","accounts_in_message":[],"_revision_number":7},{"id":"64f329ec9a75c6174f9fcd75c7ec75d6425d0df9","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":7233,"name":"Matthew Oliver","email":"matt@oliver.net.au","username":"mattoliverau"},"date":"2021-02-25 05:36:06.000000000","message":"Uploaded patch set 8: Patch Set 7 was rebased.","accounts_in_message":[],"_revision_number":8},{"id":"73c26ab32dfafaec59334273d5e094ad24161eee","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":7233,"name":"Matthew Oliver","email":"matt@oliver.net.au","username":"mattoliverau"},"date":"2021-02-25 05:55:37.000000000","message":"Uploaded patch set 9.","accounts_in_message":[],"_revision_number":9},{"id":"541f7f7171e365eb2de10ab4f41c1cfd549b0b81","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-02-25 07:23:12.000000000","message":"Patch Set 9: Verified+1\n\nBuild succeeded (check pipeline).\n\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/767b812ad6ba4b11bde8df5ed17b19fd : SUCCESS in 11m 22s\n- grenade https://zuul.opendev.org/t/openstack/build/f6b09e6bbc8b4a70a5e9aa050c5a024c : SUCCESS in 1h 26m 30s\n- tempest-integrated-object-storage https://zuul.opendev.org/t/openstack/build/6614da33d7a24c91b10fd60935959310 : SUCCESS in 47m 08s\n- swift-build-image https://zuul.opendev.org/t/openstack/build/c4448a05325c4e1da71e5763efccfe93 : SUCCESS in 10m 19s (non-voting)\n- swift-build-image-py3 https://zuul.opendev.org/t/openstack/build/1d4267bac8124fa89391ae835ba92983 : SUCCESS in 7m 47s (non-voting)\n- swift-tox-py27 https://zuul.opendev.org/t/openstack/build/0986de36726c4d07b8ea3efe6026f2d0 : SUCCESS in 25m 23s\n- swift-tox-py36 https://zuul.opendev.org/t/openstack/build/cdb6be2636124aea826904bf4d0b89af : SUCCESS in 26m 19s\n- swift-tox-py37 https://zuul.opendev.org/t/openstack/build/9b2e4f44d0d04c608de314e48a60e0f0 : SUCCESS in 27m 04s\n- swift-tox-py38 https://zuul.opendev.org/t/openstack/build/8527489e05154ff3a210e24b8e803a2d : SUCCESS in 31m 18s\n- swift-tox-py39 https://zuul.opendev.org/t/openstack/build/feac8de143d64a36a1bfca768d5c6715 : SUCCESS in 31m 02s\n- swift-tox-func-py27 https://zuul.opendev.org/t/openstack/build/9629a11c0a644b9fa1b2fae0bcfd7185 : SUCCESS in 35m 19s\n- swift-tox-func-encryption-py27 https://zuul.opendev.org/t/openstack/build/311d2323e5a04ffa94803517a7b9600f : SUCCESS in 31m 35s\n- swift-tox-func-ec-py27 https://zuul.opendev.org/t/openstack/build/dabdc3ff667f494693df9c68ba382f2a : SUCCESS in 19m 55s\n- swift-tox-func-py37 https://zuul.opendev.org/t/openstack/build/350567c025454d7fb94e20932af98887 : SUCCESS in 28m 20s\n- swift-tox-func-encryption-py37 https://zuul.opendev.org/t/openstack/build/d6d46065aea14fb49a2df2c90adb6afb : SUCCESS in 39m 05s\n- swift-tox-func-ec-py37 https://zuul.opendev.org/t/openstack/build/bebae9222c804c7bafef0008a92461bd : SUCCESS in 51m 15s\n- swift-tox-func-s3api-ceph-s3tests-tempauth https://zuul.opendev.org/t/openstack/build/22c671eebf69436ab8b1d734b642d7ea : FAILURE in 24m 21s (non-voting)\n- swift-probetests-centos-7 https://zuul.opendev.org/t/openstack/build/4db5a85f45a34c2a889167658704235b : SUCCESS in 45m 07s\n- swift-probetests-centos-8 https://zuul.opendev.org/t/openstack/build/7fa4d7560b504270804911a24bcad160 : SUCCESS in 50m 28s\n- swift-dsvm-functional https://zuul.opendev.org/t/openstack/build/aaad82d1c193436f84fc827e3e33d11c : SUCCESS in 38m 13s\n- swift-dsvm-functional-ipv6 https://zuul.opendev.org/t/openstack/build/4efa87e34141438fa535d0c6d94c688f : SUCCESS in 47m 48s\n- swift-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/b4dcd0608dfe486b9f4c66d8755dd403 : SUCCESS in 14m 34s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/d2a42bdb84ca42cab77250bf6fdce718 : SUCCESS in 5m 53s\n- swift-multinode-rolling-upgrade https://zuul.opendev.org/t/openstack/build/2441e47c5e5f432597b4aba18783cba3 : FAILURE in 10m 43s (non-voting)\n- tempest-ipv6-only https://zuul.opendev.org/t/openstack/build/202f52a9288b4c4eb52e28c87439df52 : SUCCESS in 1h 00m 58s","accounts_in_message":[],"_revision_number":9},{"id":"670e1dfa33eab013a1b4c0b1142eec7a5c991900","author":{"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},"date":"2021-02-25 15:05:33.000000000","message":"Patch Set 9:\n\n(4 comments)\n\nThis is definitely heading in a good direction.\n\nI may have missed something but I\u0027m not sure why we need to freeze the deleted timestamp. It seems that the contexts tell us what we need to know via CleavingContext.done(), and by the time contexts are deleted we don\u0027t care any more.","accounts_in_message":[],"_revision_number":9},{"id":"83afc87198997dcbb5866b02a03553f8ac9faa41","author":{"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},"date":"2021-02-25 18:36:01.000000000","message":"Patch Set 9:\n\n(2 comments)\n\nAfter chatting about this some more with Clay and Tim I realised my last review missed the fact that the cleaving contexts get deleted in _complete_sharding() - the audit step is just a fallback. So I understand better why you were testing for the existence of and last modified time of cleaving context *keys*. That in turn led to needing to force the delete timestamp to retain the last modified time.\n\nHowever, the cleaving contexts are exactly the state we need to continue reporting completed sharded containers. So we shouldn\u0027t delete them in _complete_sharding().\n\nSo here\u0027s a suggestion:\n\nInstead, keep cleaving contexts around in _complete_sharding() and then in  _audit_cleave_contexts():\n\n  delete_age \u003d  recon_sharded_timeout if context.done() else reclaim_age\n  if last_mod + delete_age \u003c now:\n      context.delete()\n\nIn other words, don\u0027t delete any cleaving context until its at least older than the recon_sharded_timeout.\n\nNote: we *do* need the catch all at reclaim age for stale contexts (db replica went awol).\n\nThen in record_sharding_progress() the extra condition is to test the max_ts of the undeleted CleavingContext values, no need to include any deleted contexts because they are known to be older than anything we care about recording.","accounts_in_message":[],"_revision_number":9},{"id":"3b4dcfc42c08227003336455080b7a19d2085617","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":7233,"name":"Matthew Oliver","email":"matt@oliver.net.au","username":"mattoliverau"},"date":"2021-02-26 01:03:50.000000000","message":"Uploaded patch set 10.","accounts_in_message":[],"_revision_number":10},{"id":"3325ad49bb823d91067c3d5488ebe73b132f7cf1","author":{"_account_id":7233,"name":"Matthew Oliver","email":"matt@oliver.net.au","username":"mattoliverau"},"date":"2021-02-26 01:34:02.000000000","message":"Patch Set 10:\n\n(1 comment)","accounts_in_message":[],"_revision_number":10},{"id":"bd36361262f9021ec9fa6b9d202914fc4a0454a9","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":7233,"name":"Matthew Oliver","email":"matt@oliver.net.au","username":"mattoliverau"},"date":"2021-02-26 01:35:55.000000000","message":"Uploaded patch set 11.","accounts_in_message":[],"_revision_number":11},{"id":"86dbd06e35cb342e054d6c8820127ad623f2cfb8","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":7233,"name":"Matthew Oliver","email":"matt@oliver.net.au","username":"mattoliverau"},"date":"2021-02-26 01:40:47.000000000","message":"Uploaded patch set 12: Patch Set 11 was rebased.","accounts_in_message":[],"_revision_number":12},{"id":"5e908deee1fe938bb823ca1f8b6e831eee348ed3","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-02-26 03:08:02.000000000","message":"Patch Set 12: Verified-1\n\nBuild failed (check pipeline).  For information on how to proceed, see\nhttps://docs.opendev.org/opendev/infra-manual/latest/developers.html#automated-testing\n\n\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/ae6cc72115674d4fb2095b1e998032d4 : SUCCESS in 8m 44s\n- grenade https://zuul.opendev.org/t/openstack/build/6b369bed97c54cd99c7ee902ef23dd65 : SUCCESS in 59m 00s\n- tempest-integrated-object-storage https://zuul.opendev.org/t/openstack/build/9d1e1b5f3b9e4efdab09b6f5da634f7e : SUCCESS in 40m 21s\n- swift-build-image https://zuul.opendev.org/t/openstack/build/6722f8b0752a400a9dce9f237595ef2d : SUCCESS in 8m 14s (non-voting)\n- swift-build-image-py3 https://zuul.opendev.org/t/openstack/build/ce2ed96f813744fb8ea6a0bb97c3d535 : SUCCESS in 7m 21s (non-voting)\n- swift-tox-py27 https://zuul.opendev.org/t/openstack/build/d7cee3e0e16241d089b011c249e6924e : SUCCESS in 14m 59s\n- swift-tox-py36 https://zuul.opendev.org/t/openstack/build/32f18187efbf4f6badeeda2bd4a345c5 : SUCCESS in 20m 08s\n- swift-tox-py37 https://zuul.opendev.org/t/openstack/build/f04cc91b763342598288dc9cba1654cd : SUCCESS in 13m 30s\n- swift-tox-py38 https://zuul.opendev.org/t/openstack/build/cade04e179994044b9581c5d796df5e4 : SUCCESS in 14m 15s\n- swift-tox-py39 https://zuul.opendev.org/t/openstack/build/dd96109ff2b5486783698f35778dc7e0 : SUCCESS in 14m 37s\n- swift-tox-func-py27 https://zuul.opendev.org/t/openstack/build/1174a202ed6b465ebd4e0360b80d034d : FAILURE in 16m 48s\n- swift-tox-func-encryption-py27 https://zuul.opendev.org/t/openstack/build/95b983aa20d24b6e8792b91546aaaef6 : SUCCESS in 22m 42s\n- swift-tox-func-ec-py27 https://zuul.opendev.org/t/openstack/build/51f56bc93eff4d5ebca5c10ae6e74158 : SUCCESS in 18m 29s\n- swift-tox-func-py37 https://zuul.opendev.org/t/openstack/build/924881dcd771401389ae9f984f45089f : SUCCESS in 23m 45s\n- swift-tox-func-encryption-py37 https://zuul.opendev.org/t/openstack/build/97e9526b10ab49edb108ce1968acf7c7 : SUCCESS in 23m 52s\n- swift-tox-func-ec-py37 https://zuul.opendev.org/t/openstack/build/88a6ef5110554e68a4aa4ff9f67baca1 : SUCCESS in 16m 24s\n- swift-tox-func-s3api-ceph-s3tests-tempauth https://zuul.opendev.org/t/openstack/build/1070df3e0fac4e63aa903b577b8e94a4 : FAILURE in 28m 48s (non-voting)\n- swift-probetests-centos-7 https://zuul.opendev.org/t/openstack/build/df37d352ae474bbcbcaa1d847e18089b : FAILURE in 39m 10s\n- swift-probetests-centos-8 https://zuul.opendev.org/t/openstack/build/853123d78c124369bfb342e50e80c91b : FAILURE in 43m 32s\n- swift-dsvm-functional https://zuul.opendev.org/t/openstack/build/0417c3d92b77415c9a248c56323d9750 : SUCCESS in 45m 05s\n- swift-dsvm-functional-ipv6 https://zuul.opendev.org/t/openstack/build/3e2b92dd8ccb47448c8a91c7bb62ef56 : SUCCESS in 45m 11s\n- swift-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/59586991c6924ac38275ac58e505f332 : SUCCESS in 20m 23s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/f19cb5acafc1478787c07a3bf0960aac : SUCCESS in 6m 15s\n- swift-multinode-rolling-upgrade https://zuul.opendev.org/t/openstack/build/fcddca5cc89948dfb55d90b8af8d9421 : FAILURE in 9m 40s (non-voting)\n- tempest-ipv6-only https://zuul.opendev.org/t/openstack/build/ee2c8acfb59b4668aff0637c27cc6b58 : SUCCESS in 1h 21m 54s","accounts_in_message":[],"_revision_number":12},{"id":"9ebf5c011124ee0a1062277b9e6f23f19022072a","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":7233,"name":"Matthew Oliver","email":"matt@oliver.net.au","username":"mattoliverau"},"date":"2021-02-26 05:13:19.000000000","message":"Uploaded patch set 13.","accounts_in_message":[],"_revision_number":13},{"id":"1dc3a0c01c298205d3ca7e7db1f43dc0a2441d4a","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-02-26 06:14:47.000000000","message":"Patch Set 13: Verified-1\n\nBuild failed (check pipeline).  For information on how to proceed, see\nhttps://docs.opendev.org/opendev/infra-manual/latest/developers.html#automated-testing\n\n\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/e0828330d1414ce8a94cef1bd6056631 : SUCCESS in 8m 48s\n- grenade https://zuul.opendev.org/t/openstack/build/3fc4e9de73fb43d79cc2f38b09abea7b : SUCCESS in 55m 01s\n- tempest-integrated-object-storage https://zuul.opendev.org/t/openstack/build/3dabf3d3a6b8475abcd64171054557dc : SUCCESS in 46m 15s\n- swift-build-image https://zuul.opendev.org/t/openstack/build/aa646594a9dc4e4da47c59c1cea98b84 : SUCCESS in 7m 54s (non-voting)\n- swift-build-image-py3 https://zuul.opendev.org/t/openstack/build/aa9b31e99249470ab9d27fc66921860b : SUCCESS in 5m 58s (non-voting)\n- swift-tox-py27 https://zuul.opendev.org/t/openstack/build/5387b19a9fd84d68b08e2b6fd00b6501 : SUCCESS in 17m 02s\n- swift-tox-py36 https://zuul.opendev.org/t/openstack/build/69a86a82d2db4726bbeaf9d8c2491c94 : SUCCESS in 21m 20s\n- swift-tox-py37 https://zuul.opendev.org/t/openstack/build/20e9e5d26fcb4fdca7149c56f6ebcc2c : SUCCESS in 17m 49s\n- swift-tox-py38 https://zuul.opendev.org/t/openstack/build/cb6d960571634618b4a42069338b7655 : SUCCESS in 21m 07s\n- swift-tox-py39 https://zuul.opendev.org/t/openstack/build/ea1519673d0f473ca3b437b0bb220df1 : SUCCESS in 15m 26s\n- swift-tox-func-py27 https://zuul.opendev.org/t/openstack/build/c7bcd9b914074ba3adb67d1bb522385d : SUCCESS in 21m 21s\n- swift-tox-func-encryption-py27 https://zuul.opendev.org/t/openstack/build/7331d15d54cf41a1afb9ea181309d948 : SUCCESS in 19m 32s\n- swift-tox-func-ec-py27 https://zuul.opendev.org/t/openstack/build/11bd1755b8c0409bb435d897f05f6f77 : SUCCESS in 16m 13s\n- swift-tox-func-py37 https://zuul.opendev.org/t/openstack/build/cab8b7417e644510b533805c8a768754 : SUCCESS in 21m 54s\n- swift-tox-func-encryption-py37 https://zuul.opendev.org/t/openstack/build/2c5994ba91404791a0917427b6e170c0 : SUCCESS in 19m 40s\n- swift-tox-func-ec-py37 https://zuul.opendev.org/t/openstack/build/0229b18fda9b48c8b2bc17c752b075eb : SUCCESS in 17m 19s\n- swift-tox-func-s3api-ceph-s3tests-tempauth https://zuul.opendev.org/t/openstack/build/0d1d22347daa4eeea3316b4133eb9456 : SUCCESS in 31m 43s (non-voting)\n- swift-probetests-centos-7 https://zuul.opendev.org/t/openstack/build/9e399e69feb74334a655e77c50b83065 : FAILURE in 47m 48s\n- swift-probetests-centos-8 https://zuul.opendev.org/t/openstack/build/2ea429fc6cb949b6812381a8cb0a12c5 : FAILURE in 39m 58s\n- swift-dsvm-functional https://zuul.opendev.org/t/openstack/build/984c4761631b473d928dd251ed02755e : SUCCESS in 38m 42s\n- swift-dsvm-functional-ipv6 https://zuul.opendev.org/t/openstack/build/d62c00e2fb4847dc8b1a552892c28974 : SUCCESS in 34m 57s\n- swift-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/8f123856be10463b9c9a1e160440ff1b : SUCCESS in 18m 14s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/cb3c7f2298fe447baaa25e86c8faa3d1 : SUCCESS in 6m 59s\n- swift-multinode-rolling-upgrade https://zuul.opendev.org/t/openstack/build/e884b86e61c44d5895924d9bac909029 : FAILURE in 9m 51s (non-voting)\n- tempest-ipv6-only https://zuul.opendev.org/t/openstack/build/050d09c8b04f4317952d2b2abb794b11 : SUCCESS in 59m 15s","accounts_in_message":[],"_revision_number":13},{"id":"b8378d0beaeb26e0184c0300241debe6ade50059","author":{"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},"date":"2021-02-26 14:48:10.000000000","message":"Patch Set 13: Code-Review-1\n\n(11 comments)\n\nGetting close\n\nI see the probe test failures same as zuul 😞...investigating.","accounts_in_message":[],"_revision_number":13},{"id":"d6f81693c48ab766aafa63b3615cbe173958f9a4","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},"date":"2021-02-26 15:56:46.000000000","message":"Uploaded patch set 14.","accounts_in_message":[],"_revision_number":14},{"id":"cafd6ff2ade029b57a5eb4e0406a872d7e0b74da","author":{"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},"date":"2021-02-26 15:59:58.000000000","message":"Patch Set 14: Code-Review+2\n\n@Matt Hope you don\u0027t mind me pushing some fixes. LGTM.","accounts_in_message":[],"_revision_number":14},{"id":"f59ca96d7c4f6af73c421b742783002d89cdcca0","author":{"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},"date":"2021-02-26 16:03:30.000000000","message":"Patch Set 14:\n\n(1 comment)","accounts_in_message":[],"_revision_number":14},{"id":"a592cbceeba0aa1c68e2507b6368ac071bc01643","author":{"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},"date":"2021-02-26 16:04:43.000000000","message":"Patch Set 14:\n\nBTW I could only get the probe tests to pass OMM with https://review.opendev.org/c/openstack/swift/+/777765/1 - I can\u0027t see any reason why this patch broke the tests other than bad luck w.r.t. the garbage collection.","accounts_in_message":[],"_revision_number":14},{"id":"683c1ef2008729897b682cb5db936557412ea097","author":{"_account_id":1179,"name":"Clay Gerrard","email":"clay.gerrard@gmail.com","username":"clay-gerrard"},"date":"2021-02-26 16:17:24.000000000","message":"Patch Set 14: Code-Review+2 Workflow+1\n\n(6 comments)\n\nlooks great!","accounts_in_message":[],"_revision_number":14},{"id":"ceb67db8bfe0be0042a77a1d1a798758aa65a249","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-02-26 17:03:05.000000000","message":"Patch Set 14: Verified+1\n\nBuild succeeded (check pipeline).\n\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/6689c78dbcc4432c98238f29f35fd05e : SUCCESS in 8m 45s\n- grenade https://zuul.opendev.org/t/openstack/build/3c845440d0fe413e95dab0e79f36ee5d : SUCCESS in 44m 31s\n- tempest-integrated-object-storage https://zuul.opendev.org/t/openstack/build/31d2e1e75e5b403ab138adc94add4ac7 : SUCCESS in 56m 42s\n- swift-build-image https://zuul.opendev.org/t/openstack/build/cc182bffc9d94ca69537513bac252faa : SUCCESS in 8m 25s (non-voting)\n- swift-build-image-py3 https://zuul.opendev.org/t/openstack/build/42153fafe2954c36a6e8c5e6cc1057a1 : SUCCESS in 5m 57s (non-voting)\n- swift-tox-py27 https://zuul.opendev.org/t/openstack/build/1430411794b840b5bd482d90c57d22e2 : SUCCESS in 13m 48s\n- swift-tox-py36 https://zuul.opendev.org/t/openstack/build/f498c31da5c24b098f18ccd215dd1774 : SUCCESS in 17m 56s\n- swift-tox-py37 https://zuul.opendev.org/t/openstack/build/5c6bf733cb994b2b8f5f08774014f8b8 : SUCCESS in 15m 27s\n- swift-tox-py38 https://zuul.opendev.org/t/openstack/build/fea9760617ba41299d469c1a3c356a06 : SUCCESS in 15m 03s\n- swift-tox-py39 https://zuul.opendev.org/t/openstack/build/c39b275b31f144caba33a1f02835b591 : SUCCESS in 15m 05s\n- swift-tox-func-py27 https://zuul.opendev.org/t/openstack/build/50611f7903cf46828b376cd743bfba09 : SUCCESS in 15m 39s\n- swift-tox-func-encryption-py27 https://zuul.opendev.org/t/openstack/build/247008028e1f43ce8d73008dc8a670f4 : SUCCESS in 18m 28s\n- swift-tox-func-ec-py27 https://zuul.opendev.org/t/openstack/build/c5b528da0ee346efbe30557d147abe60 : SUCCESS in 16m 53s\n- swift-tox-func-py37 https://zuul.opendev.org/t/openstack/build/fafaaa2e428c4505a16e1a3fd6cfaf4c : SUCCESS in 19m 06s\n- swift-tox-func-encryption-py37 https://zuul.opendev.org/t/openstack/build/7b98781a98f04c93aff8332e6f28e67d : SUCCESS in 17m 04s\n- swift-tox-func-ec-py37 https://zuul.opendev.org/t/openstack/build/8621bd48d0e448e4a4fe8a868a91d94b : SUCCESS in 19m 59s\n- swift-tox-func-s3api-ceph-s3tests-tempauth https://zuul.opendev.org/t/openstack/build/ebe58f908e444faca163f7a560aeadf8 : FAILURE in 20m 26s (non-voting)\n- swift-probetests-centos-7 https://zuul.opendev.org/t/openstack/build/d7560d521405488fad265769baec9bf8 : SUCCESS in 45m 56s\n- swift-probetests-centos-8 https://zuul.opendev.org/t/openstack/build/6c115668c0304524a84f7f8764c792b5 : SUCCESS in 44m 17s\n- swift-dsvm-functional https://zuul.opendev.org/t/openstack/build/74a668a841d44af086efa548cd455ee7 : SUCCESS in 40m 21s\n- swift-dsvm-functional-ipv6 https://zuul.opendev.org/t/openstack/build/0b04e69d022e4110ace07488651dd9b0 : SUCCESS in 37m 35s\n- swift-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/b2fa563c637348cdb40c7d9373252ba8 : SUCCESS in 13m 36s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/49426dc9af3d4f2f84f7ae6d971a2495 : SUCCESS in 5m 13s\n- swift-multinode-rolling-upgrade https://zuul.opendev.org/t/openstack/build/380b09cc8d7d43ffad278b6afc136afa : FAILURE in 10m 11s (non-voting)\n- tempest-ipv6-only https://zuul.opendev.org/t/openstack/build/058e0836d36248549f07f644f96a2eec : SUCCESS in 1h 01m 23s","accounts_in_message":[],"_revision_number":14},{"id":"7dbeee77f916a96f599fd3dec3e0d707244f4161","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-02-26 17:17:46.000000000","message":"Patch Set 14: -Verified\n\nStarting gate jobs.","accounts_in_message":[],"_revision_number":14},{"id":"98847e780c1236496be747f1856dc0bf65aae0a0","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-02-26 18:42:31.000000000","message":"Patch Set 14: Verified+2\n\nBuild succeeded (gate pipeline).\n\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/433dcc63647040608ec2581ee1dcdde4 : SUCCESS in 10m 13s\n- grenade https://zuul.opendev.org/t/openstack/build/43acacf824fb4552b94363433409aa94 : SUCCESS in 1h 01m 55s\n- tempest-integrated-object-storage https://zuul.opendev.org/t/openstack/build/9715b159eb4840bd88af8d37dc05c542 : SUCCESS in 39m 32s\n- swift-upload-image https://zuul.opendev.org/t/openstack/build/d3823ad4eb1e47799adcd8408f010807 : SUCCESS in 11m 20s (non-voting)\n- swift-upload-image-py3 https://zuul.opendev.org/t/openstack/build/e990e30014394418ad6af47206117723 : SUCCESS in 6m 11s (non-voting)\n- swift-tox-py27 https://zuul.opendev.org/t/openstack/build/5619ebe220134914a514b008509dc4f2 : SUCCESS in 14m 11s\n- swift-tox-py36 https://zuul.opendev.org/t/openstack/build/36b187a47c744e9ca584a6573ee537b7 : SUCCESS in 13m 02s\n- swift-tox-py37 https://zuul.opendev.org/t/openstack/build/3333b48500c143f2927a61c62ba62914 : SUCCESS in 14m 50s\n- swift-tox-py38 https://zuul.opendev.org/t/openstack/build/3988ca1a87154ddb8205ddf08d60d2d6 : SUCCESS in 14m 25s\n- swift-tox-py39 https://zuul.opendev.org/t/openstack/build/d0bbe60f6b4e44978c083a8f2dd8d765 : SUCCESS in 20m 00s\n- swift-tox-func-py27 https://zuul.opendev.org/t/openstack/build/69a94d5c20a648ef9e9a4925bd623f92 : SUCCESS in 22m 18s\n- swift-tox-func-encryption-py27 https://zuul.opendev.org/t/openstack/build/fb12618b3f6d420aa63ad928d4babb00 : SUCCESS in 17m 33s\n- swift-tox-func-ec-py27 https://zuul.opendev.org/t/openstack/build/03952395ad4040b1be395ce310d9efba : SUCCESS in 18m 26s\n- swift-tox-func-py37 https://zuul.opendev.org/t/openstack/build/343cefb18f9348b881eeee8281b50add : SUCCESS in 42m 33s\n- swift-tox-func-encryption-py37 https://zuul.opendev.org/t/openstack/build/a35284c7584a4d9cacb98fbd350721c9 : SUCCESS in 17m 33s\n- swift-tox-func-ec-py37 https://zuul.opendev.org/t/openstack/build/3f22eb4e433f425f83cf62d9c605f869 : SUCCESS in 28m 30s\n- swift-probetests-centos-7 https://zuul.opendev.org/t/openstack/build/9b86565c945e478d941fd6d7bf5815b6 : SUCCESS in 49m 51s\n- swift-probetests-centos-8 https://zuul.opendev.org/t/openstack/build/89fa7a21348147ffa6344b2d86237dde : SUCCESS in 42m 44s\n- swift-dsvm-functional https://zuul.opendev.org/t/openstack/build/59049ea0b30e41b6af155222aa783b99 : SUCCESS in 38m 52s\n- swift-dsvm-functional-ipv6 https://zuul.opendev.org/t/openstack/build/c0acf2d10b2248ad9633aa85d2baa007 : SUCCESS in 57m 40s\n- swift-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/ed4b02f9b4044f8fa058460b91745dae : SUCCESS in 14m 37s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/07b7d2b107154a1090e4c312e5bd9101 : SUCCESS in 7m 58s\n- tempest-ipv6-only https://zuul.opendev.org/t/openstack/build/8c8f4d5f5ca14313abec5ffa0256952b : SUCCESS in 1h 13m 18s","accounts_in_message":[],"_revision_number":14},{"id":"bbb9dd62f97c7a9b26b5281828f0cb98c6dc354d","tag":"autogenerated:gerrit:merged","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-02-26 18:42:36.000000000","message":"Change has been successfully merged by Zuul","accounts_in_message":[],"_revision_number":14},{"id":"ef3e3983d4d4ac3d0ad974bbe4192cc1c64e79fc","tag":"autogenerated:zuul:promote","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-02-26 18:45:51.000000000","message":"Patch Set 14:\n\nBuild succeeded (promote pipeline).\n\n- promote-openstack-tox-docs https://zuul.opendev.org/t/openstack/build/51c894c62cd743548b64db946e39ed82 : SUCCESS in 1m 08s\n- swift-promote-image https://zuul.opendev.org/t/openstack/build/4b4ee69f15b04ceaa73d9d628b07bc3b : SUCCESS in 43s (non-voting)\n- swift-promote-image-py3 https://zuul.opendev.org/t/openstack/build/29cd3e889c69491eb730dd4055553eca : SUCCESS in 47s (non-voting)","accounts_in_message":[],"_revision_number":14}],"current_revision_number":14,"current_revision":"1de98348167f9923710d1a6d634ef6d8008eea5d","revisions":{"5ce8cb89eb4068477baaf431f42e4ac8095bbd20":{"kind":"REWORK","_number":1,"created":"2021-02-08 06:12:10.000000000","uploader":{"_account_id":7233,"name":"Matthew Oliver","email":"matt@oliver.net.au","username":"mattoliverau"},"ref":"refs/changes/93/774393/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/swift","ref":"refs/changes/93/774393/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/swift refs/changes/93/774393/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/swift refs/changes/93/774393/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/swift refs/changes/93/774393/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/swift refs/changes/93/774393/1"}}},"commit":{"parents":[{"commit":"1d34f321ac5d092ec93e7a35bc8406da1eaeda69","subject":"Merge \"Enable shard ranges to be manually shrunk to root container\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/swift/commit/1d34f321ac5d092ec93e7a35bc8406da1eaeda69"}]}],"author":{"name":"Matthew Oliver","email":"matt@oliver.net.au","date":"2021-02-08 06:01:23.000000000","tz":660},"committer":{"name":"Matthew Oliver","email":"matt@oliver.net.au","date":"2021-02-08 06:01:23.000000000","tz":660},"subject":"Report final in_progress when sharding is complete","message":"Report final in_progress when sharding is complete\n\nOn every sharder cycle up update in progress recon stats for each sharding\ncontainer. However, we tend to not run it one final time once sharding\nis complete because the DB state is changed to SHARDED and therefore the\nin_progress stats never get their final update.\nFor those collecting this data to monitor, this makes sharding/cleaving shards\nnever complete.\n\nThis patch, adds a final call to `_record_sharding_progress()` when the\nsharder completes the sharding of a container. This allows the dump to\ncontainer update information of the current state of all shard ranges\nand the current state of the own_shard_range.\n\nThe patch also adds SHRUNK to the in_progress ShardRange state check so\na final shrinking cleave (treated as cleaving) can also update the\ncurrent cleaving states.\n\nChange-Id: I5fa39d41f9cd3b211e45d2012fd709f4135f595e\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/swift/commit/5ce8cb89eb4068477baaf431f42e4ac8095bbd20"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/swift/commit/5ce8cb89eb4068477baaf431f42e4ac8095bbd20"}]},"branch":"refs/heads/master"},"32d976b110e22536893fc7cb366d1ed237994b7c":{"kind":"REWORK","_number":2,"created":"2021-02-18 10:01:59.000000000","uploader":{"_account_id":7233,"name":"Matthew Oliver","email":"matt@oliver.net.au","username":"mattoliverau"},"ref":"refs/changes/93/774393/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/swift","ref":"refs/changes/93/774393/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/swift refs/changes/93/774393/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/swift refs/changes/93/774393/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/swift refs/changes/93/774393/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/swift refs/changes/93/774393/2"}}},"commit":{"parents":[{"commit":"1d34f321ac5d092ec93e7a35bc8406da1eaeda69","subject":"Merge \"Enable shard ranges to be manually shrunk to root container\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/swift/commit/1d34f321ac5d092ec93e7a35bc8406da1eaeda69"}]}],"author":{"name":"Matthew Oliver","email":"matt@oliver.net.au","date":"2021-02-08 06:01:23.000000000","tz":660},"committer":{"name":"Matthew Oliver","email":"matt@oliver.net.au","date":"2021-02-18 10:00:24.000000000","tz":660},"subject":"Report final in_progress when sharding is complete","message":"Report final in_progress when sharding is complete\n\nOn every sharder cycle up update in progress recon stats for each sharding\ncontainer. However, we tend to not run it one final time once sharding\nis complete because the DB state is changed to SHARDED and therefore the\nin_progress stats never get their final update.\nFor those collecting this data to monitor, this makes sharding/cleaving shards\nnever complete.\n\nThis patch, adds a final call to `_record_sharding_progress()` when the\nsharder completes the sharding of a container. This allows the dump to\ncontainer update information of the current state of all shard ranges\nand the current state of the own_shard_range.\n\nTo help in testing, this patch also plumbs sharding stats into the recon\nmiddleware. So now your sharding recon stats are only a REST call away.\n\nChange-Id: I5fa39d41f9cd3b211e45d2012fd709f4135f595e\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/swift/commit/32d976b110e22536893fc7cb366d1ed237994b7c"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/swift/commit/32d976b110e22536893fc7cb366d1ed237994b7c"}]},"branch":"refs/heads/master"},"5d30fa8caf82bc47c699245560adafe3fe408416":{"kind":"REWORK","_number":3,"created":"2021-02-18 10:04:16.000000000","uploader":{"_account_id":7233,"name":"Matthew Oliver","email":"matt@oliver.net.au","username":"mattoliverau"},"ref":"refs/changes/93/774393/3","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/swift","ref":"refs/changes/93/774393/3","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/swift refs/changes/93/774393/3 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/swift refs/changes/93/774393/3 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/swift refs/changes/93/774393/3 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/swift refs/changes/93/774393/3"}}},"commit":{"parents":[{"commit":"1d34f321ac5d092ec93e7a35bc8406da1eaeda69","subject":"Merge \"Enable shard ranges to be manually shrunk to root container\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/swift/commit/1d34f321ac5d092ec93e7a35bc8406da1eaeda69"}]}],"author":{"name":"Matthew Oliver","email":"matt@oliver.net.au","date":"2021-02-08 06:01:23.000000000","tz":660},"committer":{"name":"Matthew Oliver","email":"matt@oliver.net.au","date":"2021-02-18 10:04:02.000000000","tz":660},"subject":"Report final in_progress when sharding is complete","message":"Report final in_progress when sharding is complete\n\nOn every sharder cycle up update in progress recon stats for each sharding\ncontainer. However, we tend to not run it one final time once sharding\nis complete because the DB state is changed to SHARDED and therefore the\nin_progress stats never get their final update.\nFor those collecting this data to monitor, this makes sharding/cleaving shards\nnever complete.\n\nThis patch, adds a final call to `_record_sharding_progress()` when the\nsharder completes the sharding of a container. This allows the dump to\ncontainer update information of the current state of all shard ranges\nand the current state of the own_shard_range.\n\nTo help in testing, this patch also plumbs sharding stats into the recon\nmiddleware. So now your sharding recon stats are only a REST call away.\n\nChange-Id: I5fa39d41f9cd3b211e45d2012fd709f4135f595e\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/swift/commit/5d30fa8caf82bc47c699245560adafe3fe408416"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/swift/commit/5d30fa8caf82bc47c699245560adafe3fe408416"}]},"branch":"refs/heads/master"},"9f461f9699b211e69e88cb26a7c2a77e7352072f":{"kind":"REWORK","_number":4,"created":"2021-02-19 05:12:12.000000000","uploader":{"_account_id":7233,"name":"Matthew Oliver","email":"matt@oliver.net.au","username":"mattoliverau"},"ref":"refs/changes/93/774393/4","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/swift","ref":"refs/changes/93/774393/4","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/swift refs/changes/93/774393/4 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/swift refs/changes/93/774393/4 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/swift refs/changes/93/774393/4 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/swift refs/changes/93/774393/4"}}},"commit":{"parents":[{"commit":"1d34f321ac5d092ec93e7a35bc8406da1eaeda69","subject":"Merge \"Enable shard ranges to be manually shrunk to root container\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/swift/commit/1d34f321ac5d092ec93e7a35bc8406da1eaeda69"}]}],"author":{"name":"Matthew Oliver","email":"matt@oliver.net.au","date":"2021-02-08 06:01:23.000000000","tz":660},"committer":{"name":"Matthew Oliver","email":"matt@oliver.net.au","date":"2021-02-19 05:11:52.000000000","tz":660},"subject":"Report final in_progress when sharding is complete","message":"Report final in_progress when sharding is complete\n\nOn every sharder cycle up update in progress recon stats for each sharding\ncontainer. However, we tend to not run it one final time once sharding\nis complete because the DB state is changed to SHARDED and therefore the\nin_progress stats never get their final update.\nFor those collecting this data to monitor, this makes sharding/cleaving shards\nnever complete.\n\nThis patch, adds a final call to `_record_sharding_progress()` when the\nsharder completes the sharding of a container. This allows the dump to\ncontainer update information of the current state of all shard ranges\nand the current state of the own_shard_range.\n\nTo help in testing, this patch also plumbs sharding stats into the recon\nmiddleware. So now your sharding recon stats are only a REST call away.\n\nChange-Id: I5fa39d41f9cd3b211e45d2012fd709f4135f595e\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/swift/commit/9f461f9699b211e69e88cb26a7c2a77e7352072f"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/swift/commit/9f461f9699b211e69e88cb26a7c2a77e7352072f"}]},"branch":"refs/heads/master"},"85ca6d999d392efe9f48b118e9a5895dfab2929e":{"kind":"TRIVIAL_REBASE_WITH_MESSAGE_UPDATE","_number":5,"created":"2021-02-24 05:03:25.000000000","uploader":{"_account_id":7233,"name":"Matthew Oliver","email":"matt@oliver.net.au","username":"mattoliverau"},"ref":"refs/changes/93/774393/5","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/swift","ref":"refs/changes/93/774393/5","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/swift refs/changes/93/774393/5 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/swift refs/changes/93/774393/5 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/swift refs/changes/93/774393/5 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/swift refs/changes/93/774393/5"}}},"commit":{"parents":[{"commit":"01643a339dae1c50b68a0e15f3c26405ddf529ff","subject":"Plumb sharding stats though recon middleware","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/swift/commit/01643a339dae1c50b68a0e15f3c26405ddf529ff"}]}],"author":{"name":"Matthew Oliver","email":"matt@oliver.net.au","date":"2021-02-24 04:06:08.000000000","tz":660},"committer":{"name":"Matthew Oliver","email":"matt@oliver.net.au","date":"2021-02-24 05:02:24.000000000","tz":660},"subject":"Report final in_progress when sharding is complete","message":"Report final in_progress when sharding is complete\n\nOn every sharder cycle up update in progress recon stats for each sharding\ncontainer. However, we tend to not run it one final time once sharding\nis complete because the DB state is changed to SHARDED and therefore the\nin_progress stats never get their final update.\nFor those collecting this data to monitor, this makes sharding/cleaving shards\nnever complete.\n\nThis patch, adds a final call to `_record_sharding_progress()` when the\nsharder completes the sharding of a container. This allows the dump to\ncontainer update information of the current state of all shard ranges\nand the current state of the own_shard_range.\n\nChange-Id: I5fa39d41f9cd3b211e45d2012fd709f4135f595e\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/swift/commit/85ca6d999d392efe9f48b118e9a5895dfab2929e"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/swift/commit/85ca6d999d392efe9f48b118e9a5895dfab2929e"}]},"branch":"refs/heads/master"},"e8079d55953883936bfb34530e846c1e3853da66":{"kind":"REWORK","_number":6,"created":"2021-02-24 05:22:36.000000000","uploader":{"_account_id":7233,"name":"Matthew Oliver","email":"matt@oliver.net.au","username":"mattoliverau"},"ref":"refs/changes/93/774393/6","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/swift","ref":"refs/changes/93/774393/6","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/swift refs/changes/93/774393/6 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/swift refs/changes/93/774393/6 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/swift refs/changes/93/774393/6 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/swift refs/changes/93/774393/6"}}},"commit":{"parents":[{"commit":"01643a339dae1c50b68a0e15f3c26405ddf529ff","subject":"Plumb sharding stats though recon middleware","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/swift/commit/01643a339dae1c50b68a0e15f3c26405ddf529ff"}]}],"author":{"name":"Matthew Oliver","email":"matt@oliver.net.au","date":"2021-02-24 04:06:08.000000000","tz":660},"committer":{"name":"Matthew Oliver","email":"matt@oliver.net.au","date":"2021-02-24 05:21:56.000000000","tz":660},"subject":"Report final in_progress when sharding is complete","message":"Report final in_progress when sharding is complete\n\nOn every sharder cycle up update in progress recon stats for each sharding\ncontainer. However, we tend to not run it one final time once sharding\nis complete because the DB state is changed to SHARDED and therefore the\nin_progress stats never get their final update.\nFor those collecting this data to monitor, this makes sharding/cleaving shards\nnever complete.\n\nThis patch, adds a final call to `_record_sharding_progress()` when the\nsharder completes the sharding of a container. This allows the dump to\ncontainer update information of the current state of all shard ranges\nand the current state of the own_shard_range.\n\nChange-Id: I5fa39d41f9cd3b211e45d2012fd709f4135f595e\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/swift/commit/e8079d55953883936bfb34530e846c1e3853da66"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/swift/commit/e8079d55953883936bfb34530e846c1e3853da66"}]},"branch":"refs/heads/master"},"f64277bdc10b60f43535d54c29eb43c2a098ff57":{"kind":"REWORK","_number":7,"created":"2021-02-25 04:51:27.000000000","uploader":{"_account_id":7233,"name":"Matthew Oliver","email":"matt@oliver.net.au","username":"mattoliverau"},"ref":"refs/changes/93/774393/7","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/swift","ref":"refs/changes/93/774393/7","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/swift refs/changes/93/774393/7 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/swift refs/changes/93/774393/7 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/swift refs/changes/93/774393/7 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/swift refs/changes/93/774393/7"}}},"commit":{"parents":[{"commit":"b31eea938d5acd9c6c8023a6afc8e0dfe4997581","subject":"Plumb sharding stats though recon middleware","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/swift/commit/b31eea938d5acd9c6c8023a6afc8e0dfe4997581"}]}],"author":{"name":"Matthew Oliver","email":"matt@oliver.net.au","date":"2021-02-24 04:06:08.000000000","tz":660},"committer":{"name":"Matthew Oliver","email":"matt@oliver.net.au","date":"2021-02-25 04:49:07.000000000","tz":660},"subject":"Report final in_progress when sharding is complete","message":"Report final in_progress when sharding is complete\n\nOn every sharder cycle up update in progress recon stats for each sharding\ncontainer. However, we tend to not run it one final time once sharding\nis complete because the DB state is changed to SHARDED and therefore the\nin_progress stats never get their final update.\nFor those collecting this data to monitor, this makes sharding/cleaving shards\nnever complete.\n\nThis patch, adds a new option `recon_shared_timeout` which will now\nallow sharded containers to be processed by `_record_sharding_progress()`\nafter they\u0027ve finished sharding for an amount of time.\n\nChange-Id: I5fa39d41f9cd3b211e45d2012fd709f4135f595e\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/swift/commit/f64277bdc10b60f43535d54c29eb43c2a098ff57"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/swift/commit/f64277bdc10b60f43535d54c29eb43c2a098ff57"}]},"branch":"refs/heads/master"},"e27f74c6b3844c9c74cc7534791be763340a43bc":{"kind":"TRIVIAL_REBASE","_number":8,"created":"2021-02-25 05:36:06.000000000","uploader":{"_account_id":7233,"name":"Matthew Oliver","email":"matt@oliver.net.au","username":"mattoliverau"},"ref":"refs/changes/93/774393/8","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/swift","ref":"refs/changes/93/774393/8","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/swift refs/changes/93/774393/8 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/swift refs/changes/93/774393/8 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/swift refs/changes/93/774393/8 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/swift refs/changes/93/774393/8"}}},"commit":{"parents":[{"commit":"27ca59bf48abac5130b806a423f82d521b82038a","subject":"Plumb sharding stats though recon middleware","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/swift/commit/27ca59bf48abac5130b806a423f82d521b82038a"}]}],"author":{"name":"Matthew Oliver","email":"matt@oliver.net.au","date":"2021-02-24 04:06:08.000000000","tz":660},"committer":{"name":"Matthew Oliver","email":"matt@oliver.net.au","date":"2021-02-25 05:35:42.000000000","tz":660},"subject":"Report final in_progress when sharding is complete","message":"Report final in_progress when sharding is complete\n\nOn every sharder cycle up update in progress recon stats for each sharding\ncontainer. However, we tend to not run it one final time once sharding\nis complete because the DB state is changed to SHARDED and therefore the\nin_progress stats never get their final update.\nFor those collecting this data to monitor, this makes sharding/cleaving shards\nnever complete.\n\nThis patch, adds a new option `recon_shared_timeout` which will now\nallow sharded containers to be processed by `_record_sharding_progress()`\nafter they\u0027ve finished sharding for an amount of time.\n\nChange-Id: I5fa39d41f9cd3b211e45d2012fd709f4135f595e\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/swift/commit/e27f74c6b3844c9c74cc7534791be763340a43bc"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/swift/commit/e27f74c6b3844c9c74cc7534791be763340a43bc"}]},"branch":"refs/heads/master"},"95d1244e020abd25242fd37745c93694837b552a":{"kind":"REWORK","_number":9,"created":"2021-02-25 05:55:37.000000000","uploader":{"_account_id":7233,"name":"Matthew Oliver","email":"matt@oliver.net.au","username":"mattoliverau"},"ref":"refs/changes/93/774393/9","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/swift","ref":"refs/changes/93/774393/9","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/swift refs/changes/93/774393/9 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/swift refs/changes/93/774393/9 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/swift refs/changes/93/774393/9 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/swift refs/changes/93/774393/9"}}},"commit":{"parents":[{"commit":"27ca59bf48abac5130b806a423f82d521b82038a","subject":"Plumb sharding stats though recon middleware","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/swift/commit/27ca59bf48abac5130b806a423f82d521b82038a"}]}],"author":{"name":"Matthew Oliver","email":"matt@oliver.net.au","date":"2021-02-24 04:06:08.000000000","tz":660},"committer":{"name":"Matthew Oliver","email":"matt@oliver.net.au","date":"2021-02-25 05:55:12.000000000","tz":660},"subject":"Report final in_progress when sharding is complete","message":"Report final in_progress when sharding is complete\n\nOn every sharder cycle up update in progress recon stats for each sharding\ncontainer. However, we tend to not run it one final time once sharding\nis complete because the DB state is changed to SHARDED and therefore the\nin_progress stats never get their final update.\nFor those collecting this data to monitor, this makes sharding/cleaving shards\nnever complete.\n\nThis patch, adds a new option `recon_shared_timeout` which will now\nallow sharded containers to be processed by `_record_sharding_progress()`\nafter they\u0027ve finished sharding for an amount of time.\n\nChange-Id: I5fa39d41f9cd3b211e45d2012fd709f4135f595e\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/swift/commit/95d1244e020abd25242fd37745c93694837b552a"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/swift/commit/95d1244e020abd25242fd37745c93694837b552a"}]},"branch":"refs/heads/master"},"898e9ca4ed9a3f341153a31b1dbd89add9badac4":{"kind":"REWORK","_number":10,"created":"2021-02-26 01:03:50.000000000","uploader":{"_account_id":7233,"name":"Matthew Oliver","email":"matt@oliver.net.au","username":"mattoliverau"},"ref":"refs/changes/93/774393/10","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/swift","ref":"refs/changes/93/774393/10","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/swift refs/changes/93/774393/10 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/swift refs/changes/93/774393/10 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/swift refs/changes/93/774393/10 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/swift refs/changes/93/774393/10"}}},"commit":{"parents":[{"commit":"27ca59bf48abac5130b806a423f82d521b82038a","subject":"Plumb sharding stats though recon middleware","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/swift/commit/27ca59bf48abac5130b806a423f82d521b82038a"}]}],"author":{"name":"Matthew Oliver","email":"matt@oliver.net.au","date":"2021-02-24 04:06:08.000000000","tz":660},"committer":{"name":"Matthew Oliver","email":"matt@oliver.net.au","date":"2021-02-26 01:02:40.000000000","tz":660},"subject":"Report final in_progress when sharding is complete","message":"Report final in_progress when sharding is complete\n\nOn every sharder cycle up update in progress recon stats for each sharding\ncontainer. However, we tend to not run it one final time once sharding\nis complete because the DB state is changed to SHARDED and therefore the\nin_progress stats never get their final update.\nFor those collecting this data to monitor, this makes sharding/cleaving shards\nnever complete.\n\nThis patch, adds a new option `recon_shared_timeout` which will now\nallow sharded containers to be processed by `_record_sharding_progress()`\nafter they\u0027ve finished sharding for an amount of time.\n\nChange-Id: I5fa39d41f9cd3b211e45d2012fd709f4135f595e\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/swift/commit/898e9ca4ed9a3f341153a31b1dbd89add9badac4"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/swift/commit/898e9ca4ed9a3f341153a31b1dbd89add9badac4"}]},"branch":"refs/heads/master"},"7f96e517a71ac9a6cacdbb57af21b5fcba200854":{"kind":"REWORK","_number":11,"created":"2021-02-26 01:35:55.000000000","uploader":{"_account_id":7233,"name":"Matthew Oliver","email":"matt@oliver.net.au","username":"mattoliverau"},"ref":"refs/changes/93/774393/11","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/swift","ref":"refs/changes/93/774393/11","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/swift refs/changes/93/774393/11 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/swift refs/changes/93/774393/11 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/swift refs/changes/93/774393/11 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/swift refs/changes/93/774393/11"}}},"commit":{"parents":[{"commit":"27ca59bf48abac5130b806a423f82d521b82038a","subject":"Plumb sharding stats though recon middleware","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/swift/commit/27ca59bf48abac5130b806a423f82d521b82038a"}]}],"author":{"name":"Matthew Oliver","email":"matt@oliver.net.au","date":"2021-02-24 04:06:08.000000000","tz":660},"committer":{"name":"Matthew Oliver","email":"matt@oliver.net.au","date":"2021-02-26 01:35:33.000000000","tz":660},"subject":"Report final in_progress when sharding is complete","message":"Report final in_progress when sharding is complete\n\nOn every sharder cycle up update in progress recon stats for each sharding\ncontainer. However, we tend to not run it one final time once sharding\nis complete because the DB state is changed to SHARDED and therefore the\nin_progress stats never get their final update.\nFor those collecting this data to monitor, this makes sharding/cleaving shards\nnever complete.\n\nThis patch, adds a new option `recon_shared_timeout` which will now\nallow sharded containers to be processed by `_record_sharding_progress()`\nafter they\u0027ve finished sharding for an amount of time.\n\nChange-Id: I5fa39d41f9cd3b211e45d2012fd709f4135f595e\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/swift/commit/7f96e517a71ac9a6cacdbb57af21b5fcba200854"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/swift/commit/7f96e517a71ac9a6cacdbb57af21b5fcba200854"}]},"branch":"refs/heads/master"},"ae1f5c55149562668416f85f79cfd3b91b1daef0":{"kind":"TRIVIAL_REBASE","_number":12,"created":"2021-02-26 01:40:47.000000000","uploader":{"_account_id":7233,"name":"Matthew Oliver","email":"matt@oliver.net.au","username":"mattoliverau"},"ref":"refs/changes/93/774393/12","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/swift","ref":"refs/changes/93/774393/12","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/swift refs/changes/93/774393/12 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/swift refs/changes/93/774393/12 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/swift refs/changes/93/774393/12 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/swift refs/changes/93/774393/12"}}},"commit":{"parents":[{"commit":"228423ef27e1a7756281fc8bdeffc34d7c631e1a","subject":"Plumb sharding stats though recon middleware","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/swift/commit/228423ef27e1a7756281fc8bdeffc34d7c631e1a"}]}],"author":{"name":"Matthew Oliver","email":"matt@oliver.net.au","date":"2021-02-24 04:06:08.000000000","tz":660},"committer":{"name":"Matthew Oliver","email":"matt@oliver.net.au","date":"2021-02-26 01:40:18.000000000","tz":660},"subject":"Report final in_progress when sharding is complete","message":"Report final in_progress when sharding is complete\n\nOn every sharder cycle up update in progress recon stats for each sharding\ncontainer. However, we tend to not run it one final time once sharding\nis complete because the DB state is changed to SHARDED and therefore the\nin_progress stats never get their final update.\nFor those collecting this data to monitor, this makes sharding/cleaving shards\nnever complete.\n\nThis patch, adds a new option `recon_shared_timeout` which will now\nallow sharded containers to be processed by `_record_sharding_progress()`\nafter they\u0027ve finished sharding for an amount of time.\n\nChange-Id: I5fa39d41f9cd3b211e45d2012fd709f4135f595e\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/swift/commit/ae1f5c55149562668416f85f79cfd3b91b1daef0"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/swift/commit/ae1f5c55149562668416f85f79cfd3b91b1daef0"}]},"branch":"refs/heads/master"},"1d8b4101dd64d76352246a2a03909d8b4a586821":{"kind":"REWORK","_number":13,"created":"2021-02-26 05:13:19.000000000","uploader":{"_account_id":7233,"name":"Matthew Oliver","email":"matt@oliver.net.au","username":"mattoliverau"},"ref":"refs/changes/93/774393/13","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/swift","ref":"refs/changes/93/774393/13","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/swift refs/changes/93/774393/13 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/swift refs/changes/93/774393/13 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/swift refs/changes/93/774393/13 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/swift refs/changes/93/774393/13"}}},"commit":{"parents":[{"commit":"228423ef27e1a7756281fc8bdeffc34d7c631e1a","subject":"Plumb sharding stats though recon middleware","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/swift/commit/228423ef27e1a7756281fc8bdeffc34d7c631e1a"}]}],"author":{"name":"Matthew Oliver","email":"matt@oliver.net.au","date":"2021-02-24 04:06:08.000000000","tz":660},"committer":{"name":"Matthew Oliver","email":"matt@oliver.net.au","date":"2021-02-26 05:13:01.000000000","tz":660},"subject":"Report final in_progress when sharding is complete","message":"Report final in_progress when sharding is complete\n\nOn every sharder cycle up update in progress recon stats for each sharding\ncontainer. However, we tend to not run it one final time once sharding\nis complete because the DB state is changed to SHARDED and therefore the\nin_progress stats never get their final update.\nFor those collecting this data to monitor, this makes sharding/cleaving shards\nnever complete.\n\nThis patch, adds a new option `recon_shared_timeout` which will now\nallow sharded containers to be processed by `_record_sharding_progress()`\nafter they\u0027ve finished sharding for an amount of time.\n\nChange-Id: I5fa39d41f9cd3b211e45d2012fd709f4135f595e\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/swift/commit/1d8b4101dd64d76352246a2a03909d8b4a586821"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/swift/commit/1d8b4101dd64d76352246a2a03909d8b4a586821"}]},"branch":"refs/heads/master"},"1de98348167f9923710d1a6d634ef6d8008eea5d":{"kind":"REWORK","_number":14,"created":"2021-02-26 15:56:46.000000000","uploader":{"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},"ref":"refs/changes/93/774393/14","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/swift","ref":"refs/changes/93/774393/14","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/swift refs/changes/93/774393/14 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/swift refs/changes/93/774393/14 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/swift refs/changes/93/774393/14 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/swift refs/changes/93/774393/14"}}},"commit":{"parents":[{"commit":"b1309c95e5d1219a9241bdf1243d9dbdd17cb2d0","subject":"Plumb sharding stats though recon middleware","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/swift/commit/b1309c95e5d1219a9241bdf1243d9dbdd17cb2d0"}]}],"author":{"name":"Matthew Oliver","email":"matt@oliver.net.au","date":"2021-02-24 04:06:08.000000000","tz":660},"committer":{"name":"Alistair Coles","email":"alistairncoles@gmail.com","date":"2021-02-26 15:56:30.000000000","tz":0},"subject":"Report final in_progress when sharding is complete","message":"Report final in_progress when sharding is complete\n\nOn every sharder cycle up update in progress recon stats for each sharding\ncontainer. However, we tend to not run it one final time once sharding\nis complete because the DB state is changed to SHARDED and therefore the\nin_progress stats never get their final update.\nFor those collecting this data to monitor, this makes sharding/cleaving shards\nnever complete.\n\nThis patch, adds a new option `recon_shared_timeout` which will now\nallow sharded containers to be processed by `_record_sharding_progress()`\nafter they\u0027ve finished sharding for an amount of time.\n\nChange-Id: I5fa39d41f9cd3b211e45d2012fd709f4135f595e\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/swift/commit/1de98348167f9923710d1a6d634ef6d8008eea5d"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/swift/commit/1de98348167f9923710d1a6d634ef6d8008eea5d"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[{"status":"CLOSED","labels":[{"label":"Verified","status":"OK","applied_by":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}},{"label":"Code-Review","status":"OK","applied_by":{"_account_id":1179,"name":"Clay Gerrard","email":"clay.gerrard@gmail.com","username":"clay-gerrard"}},{"label":"Workflow","status":"OK","applied_by":{"_account_id":1179,"name":"Clay Gerrard","email":"clay.gerrard@gmail.com","username":"clay-gerrard"}}]}],"submit_requirements":[{"name":"Verified","status":"SATISFIED","is_legacy":true,"submittability_expression_result":{"expression":"label:Verified\u003dCustom-Rule","fulfilled":true,"status":"PASS","passing_atoms":["label:Verified\u003dCustom-Rule"],"failing_atoms":[],"atom_explanations":{}}},{"name":"Workflow","status":"SATISFIED","is_legacy":true,"submittability_expression_result":{"expression":"label:Workflow\u003dCustom-Rule","fulfilled":true,"status":"PASS","passing_atoms":["label:Workflow\u003dCustom-Rule"],"failing_atoms":[],"atom_explanations":{}}},{"name":"Code-Review","status":"SATISFIED","is_legacy":true,"submittability_expression_result":{"expression":"label:Code-Review\u003dCustom-Rule","fulfilled":true,"status":"PASS","passing_atoms":["label:Code-Review\u003dCustom-Rule"],"failing_atoms":[],"atom_explanations":{}}}]}