)]}'
{"id":"openstack%2Fswift~771908","triplet_id":"openstack%2Fswift~master~Ie69265e5229598e3c2634a1d4190a4c75412647a","project":"openstack/swift","branch":"master","topic":"remove_sharder_push_sr_to_shrinking_shards","hashtags":[],"change_id":"Ie69265e5229598e3c2634a1d4190a4c75412647a","subject":"Stop pushing acceptor and donor SR to shrinking shards","status":"NEW","created":"2021-01-22 05:55:14.000000000","updated":"2021-03-24 05:40:46.000000000","submit_type":"MERGE_IF_NECESSARY","mergeable":false,"submittable":false,"total_comment_count":1,"unresolved_comment_count":1,"has_review_started":true,"meta_rev_id":"d55751092519cfcb0aead47f30f3225625e024ec","_number":771908,"virtual_id_number":771908,"owner":{"_account_id":7233,"name":"Matthew Oliver","email":"matt@oliver.net.au","username":"mattoliverau"},"actions":{},"labels":{"Verified":{"disliked":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"_account_id":7233,"name":"Matthew Oliver","email":"matt@oliver.net.au","username":"mattoliverau"},{"tag":"autogenerated:zuul:check","value":-1,"date":"2021-03-24 05:40:46.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":"","value":-1,"default_value":0,"optional":true},"Code-Review":{"all":[{"value":0,"permitted_voting_range":{"min":-2,"max":2},"_account_id":7233,"name":"Matthew Oliver","email":"matt@oliver.net.au","username":"mattoliverau"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_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":{"all":[{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":7233,"name":"Matthew Oliver","email":"matt@oliver.net.au","username":"mattoliverau"},{"_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":{"CC":[{"_account_id":1179,"name":"Clay Gerrard","email":"clay.gerrard@gmail.com","username":"clay-gerrard"},{"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"}],"REVIEWER":[{"_account_id":7233,"name":"Matthew Oliver","email":"matt@oliver.net.au","username":"mattoliverau"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2021-01-22 07:08:20.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-01-22 15:27:37.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":"CC"},{"updated":"2021-02-18 20:01:44.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"}],"messages":[{"id":"480a3199462b06c096d2698115c01bc2bf5e2cca","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":7233,"name":"Matthew Oliver","email":"matt@oliver.net.au","username":"mattoliverau"},"date":"2021-01-22 05:55:14.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"d396ea43ad1dcb8fd1a3ced8316218156d87e2f7","author":{"_account_id":7233,"name":"Matthew Oliver","email":"matt@oliver.net.au","username":"mattoliverau"},"date":"2021-01-22 06:00:09.000000000","message":"Patch Set 1:\n\nThis is an initial version, might also take this time to have a look through the sharder code and see if we can refactor the _send_shard_ranges code out, as I\u0027m hoping we may only need the send shards to root container now. But there seems to be a few other calls to _send_shard_ranges in the sharder, and I haven\u0027t had time to go through them yet.","accounts_in_message":[],"_revision_number":1},{"id":"4b790372f388bf5da7d67ba36615c2f49b702acd","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-01-22 07:08:20.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/200e78fe8f484e25bf7062c2b0ef2ed0 : SUCCESS in 8m 50s\n- grenade https://zuul.opendev.org/t/openstack/build/53cb64ab78ba4d819648c04e63080af4 : SUCCESS in 1h 04m 05s\n- tempest-integrated-object-storage https://zuul.opendev.org/t/openstack/build/15e8d233cec741b1ac18afc5e62677b9 : SUCCESS in 48m 42s\n- swift-build-image https://zuul.opendev.org/t/openstack/build/8ac163233912478c9ec020b501634d6c : SUCCESS in 9m 27s (non-voting)\n- swift-build-image-py3 https://zuul.opendev.org/t/openstack/build/a7bf2bf91c594a6a8c012436c5aab8b5 : SUCCESS in 9m 14s (non-voting)\n- swift-tox-py27 https://zuul.opendev.org/t/openstack/build/84196357697a4c3e84b4cebe284fcb36 : SUCCESS in 15m 06s\n- swift-tox-py36 https://zuul.opendev.org/t/openstack/build/bafc29a23f194644b2197712d392d2b8 : SUCCESS in 14m 17s\n- swift-tox-py37 https://zuul.opendev.org/t/openstack/build/1ef71634d7b549e4a31dd1c0912e704f : SUCCESS in 14m 33s\n- swift-tox-py38 https://zuul.opendev.org/t/openstack/build/03f5a1c69d17445989c4216ecf4c6856 : SUCCESS in 13m 17s\n- swift-tox-func-py27 https://zuul.opendev.org/t/openstack/build/f71366fb4cd646a69ade47426cec670c : SUCCESS in 17m 15s\n- swift-tox-func-encryption-py27 https://zuul.opendev.org/t/openstack/build/0b682aa021d843da9147f424a031d338 : SUCCESS in 18m 20s\n- swift-tox-func-ec-py27 https://zuul.opendev.org/t/openstack/build/feb84feefbcf4424a4537c49d9c95201 : SUCCESS in 18m 15s\n- swift-tox-func-py37 https://zuul.opendev.org/t/openstack/build/676d06ef985a426c977763f1274c26f6 : SUCCESS in 16m 04s\n- swift-tox-func-encryption-py37 https://zuul.opendev.org/t/openstack/build/ba1d54cc1476433ba9c93d934a5fab6d : SUCCESS in 18m 25s\n- swift-tox-func-ec-py37 https://zuul.opendev.org/t/openstack/build/14e948f5afc04fec998a1cb0fd42f020 : SUCCESS in 16m 59s\n- swift-tox-func-s3api-ceph-s3tests-tempauth https://zuul.opendev.org/t/openstack/build/cb902df9877b432fbcca7460382dd958 : SUCCESS in 21m 34s (non-voting)\n- swift-probetests-centos-7 https://zuul.opendev.org/t/openstack/build/383235c2d6b342bba0a1d5331e334d89 : FAILURE in 51m 26s\n- swift-probetests-centos-8 https://zuul.opendev.org/t/openstack/build/e9458c54ecde4f42971266e739ec9ed6 : FAILURE in 47m 44s\n- swift-dsvm-functional https://zuul.opendev.org/t/openstack/build/8f0dd617e7f54266b4a24ba66f7b71bb : SUCCESS in 44m 19s\n- swift-dsvm-functional-ipv6 https://zuul.opendev.org/t/openstack/build/ee959026ae4141a698efd5e84cac790b : SUCCESS in 39m 39s\n- swift-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/63521043168442c4a10155ae5dde3621 : SUCCESS in 14m 03s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/1d9be697e7194c4ea0bcc0fcb9e71b83 : SUCCESS in 6m 06s\n- swift-multinode-rolling-upgrade https://zuul.opendev.org/t/openstack/build/895a82ef460b44479ed615208709caa0 : FAILURE in 9m 15s (non-voting)\n- tempest-ipv6-only https://zuul.opendev.org/t/openstack/build/22d22dcca71340f09c45864b3a7fe24a : SUCCESS in 59m 05s","accounts_in_message":[],"_revision_number":1},{"id":"61adbc76edf94ba0fe44385edaa608cf1f2d4474","author":{"_account_id":1179,"name":"Clay Gerrard","email":"clay.gerrard@gmail.com","username":"clay-gerrard"},"date":"2021-01-22 15:27:37.000000000","message":"Patch Set 1:\n\n(1 comment)\n\nprobe test failures look kind of legit maybe\n\nhow do we know if we want to review this one or https://review.opendev.org/c/openstack/swift/+/771913","accounts_in_message":[],"_revision_number":1},{"id":"f263c2914067e1b7449c397cd9ebf1a3adbee80a","author":{"_account_id":7233,"name":"Matthew Oliver","email":"matt@oliver.net.au","username":"mattoliverau"},"date":"2021-01-25 08:45:40.000000000","message":"Patch Set 1: Workflow-1\n\nYup, sorry, didn\u0027t run the other probetests after some hacking. I\u0027ve gone and fixed it in https://review.opendev.org/c/openstack/swift/+/771913 which is almost the same patch but next to the patch that make this possible.\n\nThe reason for the two is this is a version that is post collapse changes (as that makes quite a shange to the shrinking code).\n\nCan rebase this if the other doesn\u0027t land before collapse, so will mark it WIP.","accounts_in_message":[],"_revision_number":1},{"id":"839d8b914d9ad1234cffe6a293acbfe06dbbac34","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":7233,"name":"Matthew Oliver","email":"matt@oliver.net.au","username":"mattoliverau"},"date":"2021-02-18 03:21:14.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"240691b1a5a88b1289971cb110869d6c5a08aa85","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-02-18 04:26:47.000000000","message":"Patch Set 2: 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/ef2df157317541d5a13b136823f97aa7 : SUCCESS in 8m 55s\n- grenade https://zuul.opendev.org/t/openstack/build/a5f1a0ddd788450f89c5aa80d12b0187 : SUCCESS in 56m 10s\n- tempest-integrated-object-storage https://zuul.opendev.org/t/openstack/build/8b8f4153418b4176a363e5fb5cdabdbb : SUCCESS in 40m 09s\n- swift-build-image https://zuul.opendev.org/t/openstack/build/d9ad9144c5c844f3b06079db7273279e : FAILURE in 3m 51s (non-voting)\n- swift-build-image-py3 https://zuul.opendev.org/t/openstack/build/ef675ef7b62f418cbba82c9709fda035 : SUCCESS in 7m 28s (non-voting)\n- swift-tox-py27 https://zuul.opendev.org/t/openstack/build/00446f17bd1c462e9456a564f86f36d0 : SUCCESS in 16m 56s\n- swift-tox-py36 https://zuul.opendev.org/t/openstack/build/60fdbef3db914775a96c7c2206507a98 : SUCCESS in 14m 42s\n- swift-tox-py37 https://zuul.opendev.org/t/openstack/build/b718663293df49f7b8c9e0e0ef610641 : SUCCESS in 16m 36s\n- swift-tox-py38 https://zuul.opendev.org/t/openstack/build/6b5f7cc1b82541bba7a38bb7b9751ecd : SUCCESS in 14m 29s\n- swift-tox-py39 https://zuul.opendev.org/t/openstack/build/5488267ba4284d00a4d6e669538f27cf : SUCCESS in 17m 40s\n- swift-tox-func-py27 https://zuul.opendev.org/t/openstack/build/455f1b008ee242a49ca5932837f6c583 : SUCCESS in 16m 18s\n- swift-tox-func-encryption-py27 https://zuul.opendev.org/t/openstack/build/d50e00ca112040569195e6b5525e1d39 : SUCCESS in 27m 50s\n- swift-tox-func-ec-py27 https://zuul.opendev.org/t/openstack/build/ebda23ca25f64c9e8ca87d7396d91e05 : SUCCESS in 19m 27s\n- swift-tox-func-py37 https://zuul.opendev.org/t/openstack/build/0b1493f9521c420b86012f91238a5854 : SUCCESS in 18m 30s\n- swift-tox-func-encryption-py37 https://zuul.opendev.org/t/openstack/build/fe8c4d70c34e49ccb9c1c3b42a29c7a1 : SUCCESS in 25m 22s\n- swift-tox-func-ec-py37 https://zuul.opendev.org/t/openstack/build/9832237255bc44c69552ae6dd2bacb6c : SUCCESS in 21m 45s\n- swift-tox-func-s3api-ceph-s3tests-tempauth https://zuul.opendev.org/t/openstack/build/8f96b44e94ff47b0a8a8e381d20149ad : FAILURE in 29m 39s (non-voting)\n- swift-probetests-centos-7 https://zuul.opendev.org/t/openstack/build/886c8002c71b49999c79645bb880cd36 : FAILURE in 49m 30s\n- swift-probetests-centos-8 https://zuul.opendev.org/t/openstack/build/e2461ac64c07475cb91c20256e584430 : FAILURE in 57m 52s\n- swift-dsvm-functional https://zuul.opendev.org/t/openstack/build/e44d4bdf107e4f008939b726657a95c0 : SUCCESS in 41m 44s\n- swift-dsvm-functional-ipv6 https://zuul.opendev.org/t/openstack/build/b11ac7d1feab4c20b911bea47972da0d : SUCCESS in 37m 15s\n- swift-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/a21a8c378eee402d8e1c68995f4997ee : SUCCESS in 13m 30s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/c6cb0232484b403c8d36ef29b1e60864 : SUCCESS in 7m 09s\n- swift-multinode-rolling-upgrade https://zuul.opendev.org/t/openstack/build/7a6fe47f88bf44239e200fcc10e20807 : FAILURE in 10m 00s (non-voting)\n- tempest-ipv6-only https://zuul.opendev.org/t/openstack/build/0de76c3e425d46c88f97ff5c0c199dec : SUCCESS in 45m 25s","accounts_in_message":[],"_revision_number":2},{"id":"856df1d6ed0836d2807d2d5dfd93b93e59919a48","author":{"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},"date":"2021-02-18 20:01:44.000000000","message":"Patch Set 2:\n\nmakes sense in principle but obviously disturbed probe tests. \n\nShall we merge some of the patches that already have +2 and conflict with this to save repeat work on those? Then get this rebased and merged. Particularly if the probe tests need fixing here, might be best to have other changes on master.","accounts_in_message":[],"_revision_number":2},{"id":"bb2f978bf149ecebee77488e54b77a15a54f232e","author":{"_account_id":7233,"name":"Matthew Oliver","email":"matt@oliver.net.au","username":"mattoliverau"},"date":"2021-02-19 03:27:17.000000000","message":"Patch Set 2:\n\noh yeah, happy to wait on others to land first. This just really needed a rebase and made current again.","accounts_in_message":[],"_revision_number":2},{"id":"d8b346268778f2d27aeafbb0e14df8702d9e9d1a","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":7233,"name":"Matthew Oliver","email":"matt@oliver.net.au","username":"mattoliverau"},"date":"2021-03-24 04:41:06.000000000","message":"Uploaded patch set 3.","accounts_in_message":[],"_revision_number":3},{"id":"d55751092519cfcb0aead47f30f3225625e024ec","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-03-24 05:40:46.000000000","message":"Patch Set 3: Verified-1\n\n(1 comment)\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/ec749ff698474f1a9fc75f85d7698a54 : SUCCESS in 9m 55s\n- grenade https://zuul.opendev.org/t/openstack/build/10b307749372436eb96990e4809375a5 : SUCCESS in 56m 33s\n- tempest-integrated-object-storage https://zuul.opendev.org/t/openstack/build/d9f061474f66431a87fe29fd033bbde0 : SUCCESS in 42m 58s\n- swift-build-image https://zuul.opendev.org/t/openstack/build/d02b967ab5aa4a5dbdd4206f54bc9762 : SUCCESS in 10m 27s (non-voting)\n- swift-build-image-py3 https://zuul.opendev.org/t/openstack/build/9e50e7f0906b446e89af9207e39eaeb2 : SUCCESS in 10m 32s (non-voting)\n- swift-tox-py27 https://zuul.opendev.org/t/openstack/build/93d9b2bef2644df0a880c0e6c2cc508e : SUCCESS in 19m 22s\n- swift-tox-py36 https://zuul.opendev.org/t/openstack/build/2364b5bef93a4dffa78645b98672b614 : SUCCESS in 18m 02s\n- swift-tox-py37 https://zuul.opendev.org/t/openstack/build/9163795387d04961929b4ff9564598dd : SUCCESS in 17m 45s\n- swift-tox-py38 https://zuul.opendev.org/t/openstack/build/9ec121d153d34a2f801200de654cd604 : SUCCESS in 18m 19s\n- swift-tox-py39 https://zuul.opendev.org/t/openstack/build/9f2b57743b8949f7adf099a0eb539bcf : SUCCESS in 19m 02s\n- swift-tox-func-py27 https://zuul.opendev.org/t/openstack/build/005993e6d471459b9e4c2c2c4aa04378 : SUCCESS in 20m 40s\n- swift-tox-func-encryption-py27 https://zuul.opendev.org/t/openstack/build/709db13fa63248b79bd07a5b9017c830 : SUCCESS in 19m 50s\n- swift-tox-func-ec-py27 https://zuul.opendev.org/t/openstack/build/47b8e13c428143e98f4cfcab368a5605 : SUCCESS in 21m 34s\n- swift-tox-func-py37 https://zuul.opendev.org/t/openstack/build/84f14b833e274b92b72a2f1956affab4 : SUCCESS in 19m 33s\n- swift-tox-func-encryption-py37 https://zuul.opendev.org/t/openstack/build/ad0f70bbeb2f404c9cd575275216f7b9 : SUCCESS in 21m 11s\n- swift-tox-func-ec-py37 https://zuul.opendev.org/t/openstack/build/fbd3f391da9c448ea3f6c63957465ea1 : SUCCESS in 20m 14s\n- swift-func-cors https://zuul.opendev.org/t/openstack/build/06dd67abe5bc430496e1f33e4097c933 : SUCCESS in 6m 53s\n- swift-tox-func-s3api-ceph-s3tests-tempauth https://zuul.opendev.org/t/openstack/build/67d2a2474f0a4f7a88c2e2ad08cc103a : SUCCESS in 24m 48s (non-voting)\n- swift-probetests-centos-7 https://zuul.opendev.org/t/openstack/build/816b0221a8e64514bc3fc947dae3537b : FAILURE in 55m 22s\n- swift-probetests-centos-8 https://zuul.opendev.org/t/openstack/build/12d71d8d89b3406987bc47426ae3732d : FAILURE in 55m 57s\n- swift-dsvm-functional https://zuul.opendev.org/t/openstack/build/91755b97da644e0bbd451d1efb5538c1 : SUCCESS in 40m 21s\n- swift-dsvm-functional-ipv6 https://zuul.opendev.org/t/openstack/build/d55f8b870e804efc8632b2bbc0118730 : SUCCESS in 41m 01s\n- swift-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/97fb5c2cecbc49cea2fd3489c0352c3d : SUCCESS in 16m 02s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/ec0fbffaab914ec8862021b3d830fe32 : FAILURE in 6m 13s\n- swift-multinode-rolling-upgrade https://zuul.opendev.org/t/openstack/build/0686a2d6a8d1499197dd4be0db25b40e : SUCCESS in 30m 22s\n- tempest-ipv6-only https://zuul.opendev.org/t/openstack/build/9cc749cae6414a659fc46bcbe7eab1fc : SUCCESS in 54m 44s","accounts_in_message":[],"_revision_number":3}],"current_revision_number":3,"current_revision":"aa33ff7aa48d4859053cfed36fb960da3010a377","revisions":{"e58f1f920a944f18b3403bd37e5cda0aadde40b8":{"kind":"REWORK","_number":1,"created":"2021-01-22 05:55:14.000000000","uploader":{"_account_id":7233,"name":"Matthew Oliver","email":"matt@oliver.net.au","username":"mattoliverau"},"ref":"refs/changes/08/771908/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/swift","ref":"refs/changes/08/771908/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/swift refs/changes/08/771908/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/swift refs/changes/08/771908/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/swift refs/changes/08/771908/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/swift refs/changes/08/771908/1"}}},"commit":{"parents":[{"commit":"db75962152b745b0951535c372420fb5bd134592","subject":"swift-manage-shard-ranges: add \u0027compact\u0027 command","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/swift/commit/db75962152b745b0951535c372420fb5bd134592"}]}],"author":{"name":"Matthew Oliver","email":"matt@oliver.net.au","date":"2021-01-22 05:45:48.000000000","tz":660},"committer":{"name":"Matthew Oliver","email":"matt@oliver.net.au","date":"2021-01-22 05:45:48.000000000","tz":660},"subject":"Stop pushing acceptor and donor SR to shrinking shards","message":"Stop pushing acceptor and donor SR to shrinking shards\n\nWe\u0027ve currently added the ability for shards that are shrinking to pull\ndown overlapping ranges in the shard audit. This allows the shards that\nare shrinking, the donor, to pull in the acceptor node from root.\n\nWe\u0027ve also now got the final step, where the container backend can\ninclude the root acceptor when required so the root container can pass\nitself as an acceptor during a shards audit when the shard needs to\nshrink/collapse back into root.\n\nWith that last piece of the puzzle we can now remove the sharder code\nthat pushes the acceptor SR to the donor shard replicas. This not only\nstops a bunch of extra requests but also gets use to full root driven\nshrinking.\n\nThis patch removes said code and updates the test_shrinking sharder\nprobe test to use the root driven approach.\n\nChange-Id: Ie69265e5229598e3c2634a1d4190a4c75412647a\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/swift/commit/e58f1f920a944f18b3403bd37e5cda0aadde40b8"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/swift/commit/e58f1f920a944f18b3403bd37e5cda0aadde40b8"}]},"branch":"refs/heads/master"},"b164e357a0b118d5b2ae7603bf272c46e5c66c72":{"kind":"REWORK","_number":2,"created":"2021-02-18 03:21:14.000000000","uploader":{"_account_id":7233,"name":"Matthew Oliver","email":"matt@oliver.net.au","username":"mattoliverau"},"ref":"refs/changes/08/771908/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/swift","ref":"refs/changes/08/771908/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/swift refs/changes/08/771908/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/swift refs/changes/08/771908/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/swift refs/changes/08/771908/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/swift refs/changes/08/771908/2"}}},"commit":{"parents":[{"commit":"6fca356be9cf92525c5133a1e31efb6074387e46","subject":"Merge \"Correct spelling of \u0027compactable\u0027 to \u0027compactible\u0027\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/swift/commit/6fca356be9cf92525c5133a1e31efb6074387e46"}]}],"author":{"name":"Matthew Oliver","email":"matt@oliver.net.au","date":"2021-02-18 03:18:30.000000000","tz":660},"committer":{"name":"Matthew Oliver","email":"matt@oliver.net.au","date":"2021-02-18 03:18:30.000000000","tz":660},"subject":"Stop pushing acceptor and donor SR to shrinking shards","message":"Stop pushing acceptor and donor SR to shrinking shards\n\nWe\u0027ve recently added the ability for shards that are shrinking to pull\ndown overlapping ranges in the shard audit. This allows the shards that\nare shrinking, the donor, to pull in the acceptor node from root.\n\nWe\u0027ve also now got the final step, where the container backend can\ninclude the root acceptor when required so the root container can pass\nitself as an acceptor during a shards audit when the shard needs to\nshrink/collapse back into root.\n\nWith that last piece of the puzzle we can now remove the sharder code\nthat pushes the acceptor SR to the donor shard replicas. This not only\nstops a bunch of extra requests but also gets us to full root driven\nshrinking.\n\nThis patch removes said code and updates the test_shrinking sharder\nprobe test to use the root driven approach.\n\nChange-Id: Ie69265e5229598e3c2634a1d4190a4c75412647a\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/swift/commit/b164e357a0b118d5b2ae7603bf272c46e5c66c72"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/swift/commit/b164e357a0b118d5b2ae7603bf272c46e5c66c72"}]},"branch":"refs/heads/master"},"aa33ff7aa48d4859053cfed36fb960da3010a377":{"kind":"REWORK","_number":3,"created":"2021-03-24 04:41:06.000000000","uploader":{"_account_id":7233,"name":"Matthew Oliver","email":"matt@oliver.net.au","username":"mattoliverau"},"ref":"refs/changes/08/771908/3","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/swift","ref":"refs/changes/08/771908/3","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/swift refs/changes/08/771908/3 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/swift refs/changes/08/771908/3 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/swift refs/changes/08/771908/3 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/swift refs/changes/08/771908/3"}}},"commit":{"parents":[{"commit":"3a41cbe67027e0abe2816a7b659a8fd01f95fd14","subject":"relinker: Allow multiple policies to be specified","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/swift/commit/3a41cbe67027e0abe2816a7b659a8fd01f95fd14"}]}],"author":{"name":"Matthew Oliver","email":"matt@oliver.net.au","date":"2021-02-18 03:18:30.000000000","tz":660},"committer":{"name":"Matthew Oliver","email":"matt@oliver.net.au","date":"2021-03-24 04:15:25.000000000","tz":660},"subject":"Stop pushing acceptor and donor SR to shrinking shards","message":"Stop pushing acceptor and donor SR to shrinking shards\n\nWe\u0027ve recently added the ability for shards that are shrinking to pull\ndown overlapping ranges in the shard audit. This allows the shards that\nare shrinking, the donor, to pull in the acceptor node from root.\n\nWe\u0027ve also now got the final step, where the container backend can\ninclude the root acceptor when required so the root container can pass\nitself as an acceptor during a shards audit when the shard needs to\nshrink/collapse back into root.\n\nWith that last piece of the puzzle we can now remove the sharder code\nthat pushes the acceptor SR to the donor shard replicas. This not only\nstops a bunch of extra requests but also gets us to full root driven\nshrinking.\n\nThis patch removes said code and updates the test_shrinking sharder\nprobe test to use the root driven approach.\n\nChange-Id: Ie69265e5229598e3c2634a1d4190a4c75412647a\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/swift/commit/aa33ff7aa48d4859053cfed36fb960da3010a377"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/swift/commit/aa33ff7aa48d4859053cfed36fb960da3010a377"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[{"rule_name":"gerrit~DefaultSubmitRule","status":"OK","labels":[{"label":"Verified","status":"MAY","applied_by":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}},{"label":"Code-Review","status":"MAY"},{"label":"Workflow","status":"MAY"}]}],"submit_requirements":[{"name":"Verified","description":"Verified in gate by CI","status":"UNSATISFIED","is_legacy":false,"submittability_expression_result":{"expression":"label:Verified\u003dMAX AND -label:Verified\u003dMIN","fulfilled":false,"status":"FAIL","passing_atoms":[],"failing_atoms":["label:Verified\u003dMAX","label:Verified\u003dMIN"],"atom_explanations":{"label:Verified\u003dMAX":"","label:Verified\u003dMIN":""}}},{"name":"Code-Review","description":"Code reviewed by core reviewer","status":"UNSATISFIED","is_legacy":false,"submittability_expression_result":{"expression":"label:Code-Review\u003dMAX AND -label:Code-Review\u003dMIN","fulfilled":false,"status":"FAIL","passing_atoms":[],"failing_atoms":["label:Code-Review\u003dMAX","label:Code-Review\u003dMIN"],"atom_explanations":{"label:Code-Review\u003dMAX":"","label:Code-Review\u003dMIN":""}}},{"name":"Workflow","description":"Approved for gate by core reviewer","status":"UNSATISFIED","is_legacy":false,"submittability_expression_result":{"expression":"label:Workflow\u003dMAX AND -label:Workflow\u003dMIN","fulfilled":false,"status":"FAIL","passing_atoms":[],"failing_atoms":["label:Workflow\u003dMAX","label:Workflow\u003dMIN"],"atom_explanations":{"label:Workflow\u003dMAX":"","label:Workflow\u003dMIN":""}}}]}
