)]}'
{"id":"openstack%2Fswift~803423","triplet_id":"openstack%2Fswift~master~I026b699fc5f0fba619cf524093632d67ca38d32f","project":"openstack/swift","branch":"master","topic":"container-server-return-specified-api-objs","hashtags":[],"change_id":"I026b699fc5f0fba619cf524093632d67ca38d32f","subject":"container-server: return objects of a given policy","status":"MERGED","created":"2021-08-04 07:21:46.000000000","updated":"2021-08-16 23:02:50.000000000","submitted":"2021-08-16 23:01:05.000000000","submitter":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"total_comment_count":55,"unresolved_comment_count":35,"has_review_started":true,"submission_id":"803423-container-server-return-specified-api-objs","meta_rev_id":"e545971ee1082d29eabc469eeb941752d0135cf4","_number":803423,"virtual_id_number":803423,"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":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-08-16 23:01:04.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"}],"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":1179,"name":"Clay Gerrard","email":"clay.gerrard@gmail.com","username":"clay-gerrard"},"all":[{"value":0,"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},{"value":2,"date":"2021-08-16 21:33:17.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"]},{"value":2,"date":"2021-08-16 18:30:14.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"}],"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":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},{"value":1,"date":"2021-08-16 21:33:17.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"]},{"value":0,"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"}],"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":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},{"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2021-08-04 08:28:44.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-08-05 11:57:11.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-08-06 10:48:04.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-08-09 20:30:53.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-08-09 21:59:55.000000000","updated_by":{"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"},"reviewer":{"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"},"state":"REVIEWER"},{"updated":"2021-08-09 22:16:35.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":"497a8b94d50e8ac4083679c821a2a07a88c5e2c8","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":7233,"name":"Matthew Oliver","email":"matt@oliver.net.au","username":"mattoliverau"},"date":"2021-08-04 07:21:46.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"9b60915479afdea4e4a3ea8c80881a05a18dedb9","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-08-04 08:28:44.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/f527251909d6485da875049bfba005be : SUCCESS in 9m 42s\n- grenade https://zuul.opendev.org/t/openstack/build/679994b7cdb141af89e24cefd8ba44fc : SUCCESS in 1h 02m 52s\n- tempest-integrated-object-storage https://zuul.opendev.org/t/openstack/build/f3abfa63d0d641f08f2b348e896669d3 : SUCCESS in 44m 45s\n- swift-build-image https://zuul.opendev.org/t/openstack/build/198e02478fea44edad34e42cddb61eb5 : SUCCESS in 8m 14s (non-voting)\n- swift-build-image-py3 https://zuul.opendev.org/t/openstack/build/738cb70e0f1b4e50ba4679435fece2c1 : SUCCESS in 8m 45s (non-voting)\n- swift-tox-py27 https://zuul.opendev.org/t/openstack/build/716d9b1cd1fb403e886ed25ee75fb93b : FAILURE in 18m 08s\n- swift-tox-py36 https://zuul.opendev.org/t/openstack/build/75cc2c15c8df4923b6e0fda298f40be5 : FAILURE in 17m 02s\n- swift-tox-py37 https://zuul.opendev.org/t/openstack/build/d2a9c0db3e26410ba5d929f0f9c30d31 : FAILURE in 17m 47s\n- swift-tox-py38 https://zuul.opendev.org/t/openstack/build/5bee4584b4fe4ebc8555a1dddfabd600 : FAILURE in 16m 28s\n- swift-tox-py39 https://zuul.opendev.org/t/openstack/build/53b4c36f21f64cc78106614dc351521e : FAILURE in 16m 47s\n- swift-tox-func-py27 https://zuul.opendev.org/t/openstack/build/eab1d421809f4ffea8f9cf69bb4255b1 : SUCCESS in 16m 43s\n- swift-tox-func-encryption-py27 https://zuul.opendev.org/t/openstack/build/784ef6d8b0b849a2893e826f22a4742a : SUCCESS in 16m 47s\n- swift-tox-func-ec-py27 https://zuul.opendev.org/t/openstack/build/baf1e4176be145819d11ead1b2a1b763 : SUCCESS in 17m 52s\n- swift-tox-func-py38 https://zuul.opendev.org/t/openstack/build/c156899bd214490ea189e87f7bd7a322 : SUCCESS in 19m 58s\n- swift-tox-func-encryption-py38 https://zuul.opendev.org/t/openstack/build/bb8ede2d69ff48948764e4b0a636f160 : SUCCESS in 17m 16s\n- swift-tox-func-ec-py38 https://zuul.opendev.org/t/openstack/build/a87ef043782e419c88aecfec9db5f142 : SUCCESS in 17m 32s\n- swift-func-cors https://zuul.opendev.org/t/openstack/build/de52c048b3954173bcdaa0fe4494a087 : SUCCESS in 6m 49s\n- swift-tox-func-s3api-ceph-s3tests-tempauth https://zuul.opendev.org/t/openstack/build/0d45bc50c77b4ceb8597dec84bce17d5 : SUCCESS in 20m 08s (non-voting)\n- swift-probetests-centos-7 https://zuul.opendev.org/t/openstack/build/58bf42c7ca014402853a7b4f9bb3861a : FAILURE in 50m 52s\n- swift-probetests-centos-8 https://zuul.opendev.org/t/openstack/build/5d613a148d1e49ba869d0655b480ab9f : SUCCESS in 53m 45s\n- swift-dsvm-functional https://zuul.opendev.org/t/openstack/build/652f8d2e6fc749468040aa1d14ef9cb2 : SUCCESS in 39m 38s\n- swift-dsvm-functional-ipv6 https://zuul.opendev.org/t/openstack/build/30012aa14c2648b283104aa651b71d68 : SUCCESS in 40m 17s\n- swift-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/6864441209ba479cb2c8973749474f38 : FAILURE in 16m 49s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/a35fde356fd241388976d4ad66493977 : SUCCESS in 6m 22s\n- swift-multinode-rolling-upgrade https://zuul.opendev.org/t/openstack/build/d2e2553973524a9697265bae2443cdf6 : SUCCESS in 28m 34s\n- tempest-ipv6-only https://zuul.opendev.org/t/openstack/build/fb0830040b0e4a63bd4604c0b29cc8bd : SUCCESS in 43m 43s","accounts_in_message":[],"_revision_number":1},{"id":"416c1bbd7453eab9856d1170eed6e95a32af83a1","tag":"autogenerated:zuul:check-arm64","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-08-04 09:00:48.000000000","message":"Patch Set 1:\n\nBuild succeeded (ARM64 pipeline).\n\n- swift-tox-py38-arm64 https://zuul.opendev.org/t/openstack/build/451945865bd541e1b8ab06ade04eacb4 : FAILURE in 37m 53s (non-voting)\n- swift-tox-py39-arm64 https://zuul.opendev.org/t/openstack/build/4b7b15c9ad7146b1b516c092286b816b : FAILURE in 37m 57s (non-voting)\n- swift-probetests-centos-8-arm64 https://zuul.opendev.org/t/openstack/build/aa4d56d002664ac09333ad9aecbf52b6 : FAILURE in 1h 33m 25s (non-voting)\n- swift-tox-func-encryption-py38-arm64 https://zuul.opendev.org/t/openstack/build/bdb1b6541da4440888c421a432b79a7d : SUCCESS in 46m 48s (non-voting)\n- swift-tox-func-py38-arm64 https://zuul.opendev.org/t/openstack/build/53a56121109445a38c67ac1bf663c600 : SUCCESS in 40m 27s (non-voting)","accounts_in_message":[],"_revision_number":1},{"id":"4d1776132fbe59206ac508d36c5677406227eca6","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":7233,"name":"Matthew Oliver","email":"matt@oliver.net.au","username":"mattoliverau"},"date":"2021-08-05 03:27:50.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"cbd74f0070713c6def8122ee344c7e1af80b76d0","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-08-05 04:41:48.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/c3f2949f1cdb4e1b91d3c84e152aea5c : SUCCESS in 14m 16s\n- grenade https://zuul.opendev.org/t/openstack/build/ff2f40bd4c6c4fe48aee56ab2e23c633 : SUCCESS in 1h 02m 00s\n- tempest-integrated-object-storage https://zuul.opendev.org/t/openstack/build/dcd0514be429432e8ff31c087ad7e445 : FAILURE in 1h 04m 44s\n- swift-build-image https://zuul.opendev.org/t/openstack/build/1d8660ef165f4e30bf629974710e33d8 : SUCCESS in 7m 53s (non-voting)\n- swift-build-image-py3 https://zuul.opendev.org/t/openstack/build/1e29e90169f1424f8eefec7d404cd792 : SUCCESS in 10m 11s (non-voting)\n- swift-tox-py27 https://zuul.opendev.org/t/openstack/build/e0b7d4bb1eff4944a6b539cfe1b50f2e : FAILURE in 17m 26s\n- swift-tox-py36 https://zuul.opendev.org/t/openstack/build/d754918bb9aa4da6b27286da512c5d05 : FAILURE in 17m 11s\n- swift-tox-py37 https://zuul.opendev.org/t/openstack/build/c40310c994d54aa9ac4aca5cb38b4bb9 : FAILURE in 20m 27s\n- swift-tox-py38 https://zuul.opendev.org/t/openstack/build/a40a496f8c1d4d2ea118685fa8d78f80 : FAILURE in 16m 32s\n- swift-tox-py39 https://zuul.opendev.org/t/openstack/build/de358159816247fba06baff0c92440b3 : FAILURE in 14m 50s\n- swift-tox-func-py27 https://zuul.opendev.org/t/openstack/build/e836e254ce3241da82ad6a44c9271e39 : SUCCESS in 18m 01s\n- swift-tox-func-encryption-py27 https://zuul.opendev.org/t/openstack/build/2e2e1a74a2114dd28af655421b5c9516 : SUCCESS in 16m 40s\n- swift-tox-func-ec-py27 https://zuul.opendev.org/t/openstack/build/6f8ec6ecfb6b472eb01a6f7fd0186052 : SUCCESS in 18m 12s\n- swift-tox-func-py38 https://zuul.opendev.org/t/openstack/build/65fb1604388049ccb253fcc2910c4507 : SUCCESS in 20m 49s\n- swift-tox-func-encryption-py38 https://zuul.opendev.org/t/openstack/build/ab5d5af8c27e4362adc7fa896e21e367 : SUCCESS in 17m 01s\n- swift-tox-func-ec-py38 https://zuul.opendev.org/t/openstack/build/da42df2bd50c4d5cbf51f0efd04eeb41 : SUCCESS in 17m 11s\n- swift-func-cors https://zuul.opendev.org/t/openstack/build/da03ee89e33f45d292d3ecb4f17fee01 : SUCCESS in 6m 37s\n- swift-tox-func-s3api-ceph-s3tests-tempauth https://zuul.opendev.org/t/openstack/build/8b64059469cf4160b56e804d362248c7 : SUCCESS in 22m 23s (non-voting)\n- swift-probetests-centos-7 https://zuul.opendev.org/t/openstack/build/4b39a49e604f47fbbaefc44d3133b0f4 : SUCCESS in 52m 37s\n- swift-probetests-centos-8 https://zuul.opendev.org/t/openstack/build/1ba39774a6644f6fbefa4ed083f2a8c3 : SUCCESS in 50m 51s\n- swift-dsvm-functional https://zuul.opendev.org/t/openstack/build/d4bba963c0af421093fe3955fb4ad32a : SUCCESS in 39m 31s\n- swift-dsvm-functional-ipv6 https://zuul.opendev.org/t/openstack/build/84b4f26df0b14d9a834c33085893d955 : SUCCESS in 38m 11s\n- swift-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/6ca4339676e34433ac0dfb3b27dd4768 : FAILURE in 16m 32s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/d79221904b814bf9a4017e67fb61152f : SUCCESS in 6m 13s\n- swift-multinode-rolling-upgrade https://zuul.opendev.org/t/openstack/build/50469b466e8545db8d139ff6ba525c42 : SUCCESS in 23m 10s\n- tempest-ipv6-only https://zuul.opendev.org/t/openstack/build/4e53011915de49229bf9a16b223217b9 : SUCCESS in 52m 11s","accounts_in_message":[],"_revision_number":2},{"id":"d1ce6637decfe1de27ae381d2f94fb36355d6cc4","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":7233,"name":"Matthew Oliver","email":"matt@oliver.net.au","username":"mattoliverau"},"date":"2021-08-05 04:55:17.000000000","message":"Uploaded patch set 3.","accounts_in_message":[],"_revision_number":3},{"id":"99748e3c70127e56d3640af1c45eeb1d4de258ec","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-08-05 06:07:33.000000000","message":"Patch Set 3: Verified+1\n\nBuild succeeded (check pipeline).\n\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/e5534f8d7caa41c9b82d0a82b8417a38 : SUCCESS in 9m 47s\n- grenade https://zuul.opendev.org/t/openstack/build/594d23eb47d845558fb8ce78a958f9f2 : SUCCESS in 1h 07m 46s\n- tempest-integrated-object-storage https://zuul.opendev.org/t/openstack/build/2cb52a705a464702852a811101bd2a1e : SUCCESS in 37m 45s\n- swift-build-image https://zuul.opendev.org/t/openstack/build/a995b26663994e3798db54e0b232cac6 : SUCCESS in 7m 58s (non-voting)\n- swift-build-image-py3 https://zuul.opendev.org/t/openstack/build/32a3bb138d0b463bb35e84e1fa781f6a : SUCCESS in 8m 31s (non-voting)\n- swift-tox-py27 https://zuul.opendev.org/t/openstack/build/7c862437739d46b09135283766c5bf76 : SUCCESS in 20m 18s\n- swift-tox-py36 https://zuul.opendev.org/t/openstack/build/8c88f51db23a471397f6cdfff1f57479 : SUCCESS in 21m 40s\n- swift-tox-py37 https://zuul.opendev.org/t/openstack/build/43bfefa6109748bfb352cb456370a257 : SUCCESS in 21m 12s\n- swift-tox-py38 https://zuul.opendev.org/t/openstack/build/7406e1bd72074da797193ed0797ab2a2 : SUCCESS in 20m 43s\n- swift-tox-py39 https://zuul.opendev.org/t/openstack/build/18f84fa15e8c4edd97efcf9c2262bbab : SUCCESS in 20m 46s\n- swift-tox-func-py27 https://zuul.opendev.org/t/openstack/build/8c9831fa5b8948699002e9461d8d2806 : SUCCESS in 20m 19s\n- swift-tox-func-encryption-py27 https://zuul.opendev.org/t/openstack/build/9e6072e38d5440b2b45d3d8cafa61eaf : SUCCESS in 21m 10s\n- swift-tox-func-ec-py27 https://zuul.opendev.org/t/openstack/build/ac31f0e2ddb841e8b826d945707469f7 : SUCCESS in 21m 17s\n- swift-tox-func-py38 https://zuul.opendev.org/t/openstack/build/327fe23a0e4f44bfb8f33df5c28146a4 : SUCCESS in 20m 42s\n- swift-tox-func-encryption-py38 https://zuul.opendev.org/t/openstack/build/74fdc3c8ac184728bc2cdb3ee8258683 : SUCCESS in 21m 04s\n- swift-tox-func-ec-py38 https://zuul.opendev.org/t/openstack/build/4a825787f1594bfd8dc848db4e489fb4 : SUCCESS in 23m 06s\n- swift-func-cors https://zuul.opendev.org/t/openstack/build/b659eed136ab45468380dd8b7fdd7095 : SUCCESS in 7m 02s\n- swift-tox-func-s3api-ceph-s3tests-tempauth https://zuul.opendev.org/t/openstack/build/6314d3604289419b84d83617b183535b : SUCCESS in 24m 28s (non-voting)\n- swift-probetests-centos-7 https://zuul.opendev.org/t/openstack/build/2eb624ec7ab441aa994351b02aedfd20 : SUCCESS in 56m 12s\n- swift-probetests-centos-8 https://zuul.opendev.org/t/openstack/build/628dba0ae8a04f0792d1af70812aba0e : SUCCESS in 49m 36s\n- swift-dsvm-functional https://zuul.opendev.org/t/openstack/build/7e59927fb88a41f88840992b6af32c3c : SUCCESS in 41m 07s\n- swift-dsvm-functional-ipv6 https://zuul.opendev.org/t/openstack/build/ebd6d09388fa435da550082b37b3bcb9 : SUCCESS in 37m 09s\n- swift-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/18bc50cfdc2d4819be263c75e6f6a1dd : SUCCESS in 13m 26s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/2cf6b4a7b6714c5c813c857d321c2631 : SUCCESS in 6m 02s\n- swift-multinode-rolling-upgrade https://zuul.opendev.org/t/openstack/build/dcb7fd0da9ec4be9ab5e32646b962964 : SUCCESS in 31m 24s\n- tempest-ipv6-only https://zuul.opendev.org/t/openstack/build/9ec82d50c6844645aa46e8c2ec1962f2 : SUCCESS in 46m 24s","accounts_in_message":[],"_revision_number":3},{"id":"9776049c262e21c1b7a85f245990a28489a52afd","tag":"autogenerated:zuul:check-arm64","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-08-05 06:58:48.000000000","message":"Patch Set 3:\n\nBuild succeeded (ARM64 pipeline).\n\n- swift-tox-py38-arm64 https://zuul.opendev.org/t/openstack/build/27d20314b34e49c0adf8079c6ff6c22d : SUCCESS in 42m 58s (non-voting)\n- swift-tox-py39-arm64 https://zuul.opendev.org/t/openstack/build/28f6ae071aa14516a7caa8c07fd72043 : SUCCESS in 31m 30s (non-voting)\n- swift-probetests-centos-8-arm64 https://zuul.opendev.org/t/openstack/build/f100b596ea2c4b60ad214eeaa089441e : TIMED_OUT in 2h 01m 17s (non-voting)\n- swift-tox-func-encryption-py38-arm64 https://zuul.opendev.org/t/openstack/build/4122176e405f4bb4a31ac02016616067 : SUCCESS in 31m 59s (non-voting)\n- swift-tox-func-py38-arm64 https://zuul.opendev.org/t/openstack/build/5bcad2c1952742a39924ba8678e46edc : TIMED_OUT in 1h 31m 26s (non-voting)","accounts_in_message":[],"_revision_number":3},{"id":"3609c69cfdbaa62322195225ce79676210baa5aa","author":{"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},"date":"2021-08-05 11:57:11.000000000","message":"Patch Set 3:\n\n(12 comments)\n\n-1 because I think we should validate the policy index at backend otherwise looks good and a great improvement. The commit message could probably do with some cleanup given that this is independent of any policy migration changes.\n\nSome of the suggested changes are in this follow-up, feel free to squash if you like it:\n\nhttps://review.opendev.org/c/openstack/swift/+/803589","accounts_in_message":[],"_revision_number":3},{"id":"e0399e9f2b35982ec17badb4ac384ceef2822d9f","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":7233,"name":"Matthew Oliver","email":"matt@oliver.net.au","username":"mattoliverau"},"date":"2021-08-06 00:12:40.000000000","message":"Uploaded patch set 4.","accounts_in_message":[],"_revision_number":4},{"id":"7810c7a055ea2218b70c9f86a37d5b3d4402ca40","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-08-06 01:12:12.000000000","message":"Patch Set 4: Verified+1\n\nBuild succeeded (check pipeline).\n\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/b14cd16b184348d38bdf070c871af751 : SUCCESS in 13m 00s\n- grenade https://zuul.opendev.org/t/openstack/build/4ba5e182b426450d813445a7f92fec42 : SUCCESS in 56m 28s\n- tempest-integrated-object-storage https://zuul.opendev.org/t/openstack/build/0f0b1e09bd344772966cbc3719fccdaf : SUCCESS in 37m 19s\n- swift-build-image https://zuul.opendev.org/t/openstack/build/695d49b8bcd148219acb37c839b55dc1 : SUCCESS in 8m 55s (non-voting)\n- swift-build-image-py3 https://zuul.opendev.org/t/openstack/build/5e709e4e442d4e19ad036d91c0521b80 : SUCCESS in 9m 26s (non-voting)\n- swift-tox-py27 https://zuul.opendev.org/t/openstack/build/d4993677c46149d0959bb9022835a0ab : SUCCESS in 15m 11s\n- swift-tox-py36 https://zuul.opendev.org/t/openstack/build/5de72d88b81c40e4aa308238840e52bf : SUCCESS in 16m 44s\n- swift-tox-py37 https://zuul.opendev.org/t/openstack/build/7d74880b1dfa4bef96873f518ef92f12 : SUCCESS in 16m 09s\n- swift-tox-py38 https://zuul.opendev.org/t/openstack/build/2a0dbe006ce246ac8fef2dfb66fe76fa : SUCCESS in 15m 54s\n- swift-tox-py39 https://zuul.opendev.org/t/openstack/build/8ea2fb46403e46db92cf2c691a538bdd : SUCCESS in 16m 22s\n- swift-tox-func-py27 https://zuul.opendev.org/t/openstack/build/1c69ae281b3143ff9aef69a26c1997d5 : SUCCESS in 15m 51s\n- swift-tox-func-encryption-py27 https://zuul.opendev.org/t/openstack/build/3f12d689eba448c88af3f91edc87c0ea : SUCCESS in 17m 01s\n- swift-tox-func-ec-py27 https://zuul.opendev.org/t/openstack/build/fae2c03479ac47f28412360c73847251 : SUCCESS in 17m 47s\n- swift-tox-func-py38 https://zuul.opendev.org/t/openstack/build/e7485affccac4eb4919f6e9b5cb89511 : SUCCESS in 16m 41s\n- swift-tox-func-encryption-py38 https://zuul.opendev.org/t/openstack/build/84caf071f32c42aab2bb117708e6891e : SUCCESS in 17m 30s\n- swift-tox-func-ec-py38 https://zuul.opendev.org/t/openstack/build/4cc9e64987344689b1456c80ea644848 : SUCCESS in 17m 28s\n- swift-func-cors https://zuul.opendev.org/t/openstack/build/6b5e44cd0dd34a2caf95dcd72ac573cc : SUCCESS in 7m 49s\n- swift-tox-func-s3api-ceph-s3tests-tempauth https://zuul.opendev.org/t/openstack/build/fa97bb9789324baf891467018bf3c534 : SUCCESS in 22m 07s (non-voting)\n- swift-probetests-centos-7 https://zuul.opendev.org/t/openstack/build/f1dd28339bfb4e81931e0e8e3b75868f : SUCCESS in 48m 53s\n- swift-probetests-centos-8 https://zuul.opendev.org/t/openstack/build/f09f832c596e4db68829a0cc1f1bea8e : SUCCESS in 52m 19s\n- swift-dsvm-functional https://zuul.opendev.org/t/openstack/build/674806f578ce4002b65c661e1e2cd7f6 : SUCCESS in 40m 01s\n- swift-dsvm-functional-ipv6 https://zuul.opendev.org/t/openstack/build/740dd5bd1fd64b98ab49f779f330ac72 : SUCCESS in 39m 58s\n- swift-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/baaba73e7163473dbaf90ede7f0a81b6 : SUCCESS in 15m 45s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/3f793e325cd04c6baf44d08296987002 : SUCCESS in 6m 40s\n- swift-multinode-rolling-upgrade https://zuul.opendev.org/t/openstack/build/7770c77fd3594b8c93910e1e27a6703a : SUCCESS in 27m 19s\n- tempest-ipv6-only https://zuul.opendev.org/t/openstack/build/3eb51fdb0fde454c841e597640d308e8 : SUCCESS in 41m 34s","accounts_in_message":[],"_revision_number":4},{"id":"d5af6224fdd21fcf54fa7b3b1932e010b3b34368","tag":"autogenerated:zuul:check-arm64","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-08-06 01:48:10.000000000","message":"Patch Set 4:\n\nBuild succeeded (ARM64 pipeline).\n\n- swift-tox-py38-arm64 https://zuul.opendev.org/t/openstack/build/e4b1e3686e2345ba826f023963187691 : SUCCESS in 41m 59s (non-voting)\n- swift-tox-py39-arm64 https://zuul.opendev.org/t/openstack/build/f47976e78cd043d38b45ff15cfdb50b5 : SUCCESS in 50m 59s (non-voting)\n- swift-probetests-centos-8-arm64 https://zuul.opendev.org/t/openstack/build/77594aa1042c49e380b1b746d1784d61 : SUCCESS in 1h 29m 10s (non-voting)\n- swift-tox-func-encryption-py38-arm64 https://zuul.opendev.org/t/openstack/build/b33febf7205c41cc9f314cba702db45e : SUCCESS in 30m 32s (non-voting)\n- swift-tox-func-py38-arm64 https://zuul.opendev.org/t/openstack/build/80eecbd591a143c8b95420d9283aa826 : SUCCESS in 32m 35s (non-voting)","accounts_in_message":[],"_revision_number":4},{"id":"463c88e4e31e270f3b445f926bb18c975392f25d","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},"date":"2021-08-06 10:48:04.000000000","message":"Uploaded patch set 5: Commit message was updated.","accounts_in_message":[],"_revision_number":5},{"id":"f30ddef51081b0296f14c15a8f31f4674dfc23d9","author":{"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},"date":"2021-08-06 10:55:25.000000000","message":"Patch Set 5: Code-Review-1\n\n(1 comment)\n\nI think we ought to preserve the header when recursively listing (see follow up for squashing https://review.opendev.org/c/openstack/swift/+/803729):\n\n- Sharded root container has all objects deleted\n\n- root container is deleted and revived with new policy index\n\n- shard ranges still have old policy\n\n- shard range start to shard itself: this is very unlikely with autosharding because new object put in the shard will have root\u0027s policy, not shard\u0027s policy, so shard will report itself as having 0 objects to root; but with manual sharding this could happen.\n\n- sub-shards have old policy index\n\n- listing root will redirect to original shards, then sharding shard redirects to sub-shards","accounts_in_message":[],"_revision_number":5},{"id":"dc704b677a1f865b9f5a8a86846f7cfc5c9aa3f7","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-08-06 11:53:03.000000000","message":"Patch Set 5: Verified+1\n\nBuild succeeded (check pipeline).\n\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/ef4ee45134f34126927a6fd5fa772927 : SUCCESS in 8m 34s\n- grenade https://zuul.opendev.org/t/openstack/build/d14a352723e84db0b8d5a5af92ec08cf : SUCCESS in 58m 00s\n- tempest-integrated-object-storage https://zuul.opendev.org/t/openstack/build/a1fb577368424a9392bc733020aab6d3 : SUCCESS in 37m 00s\n- swift-build-image https://zuul.opendev.org/t/openstack/build/dcc318e689dd4aeb918d37f4a386422c : SUCCESS in 9m 04s (non-voting)\n- swift-build-image-py3 https://zuul.opendev.org/t/openstack/build/79bf45473a6e4c50a362cf027cce7529 : SUCCESS in 10m 00s (non-voting)\n- swift-tox-py27 https://zuul.opendev.org/t/openstack/build/8acb6bce701f49b583d535cd6447c46a : SUCCESS in 17m 13s\n- swift-tox-py36 https://zuul.opendev.org/t/openstack/build/e404edebd6814690a9e713e7eec90ba1 : SUCCESS in 17m 33s\n- swift-tox-py37 https://zuul.opendev.org/t/openstack/build/7dd8506b28c44ea0af3028392ceca2a1 : SUCCESS in 17m 35s\n- swift-tox-py38 https://zuul.opendev.org/t/openstack/build/9a91877e9f4945d8a605922afe320ae3 : SUCCESS in 17m 22s\n- swift-tox-py39 https://zuul.opendev.org/t/openstack/build/7415ed2cf4944f81ba3b2994e636b010 : SUCCESS in 17m 05s\n- swift-tox-func-py27 https://zuul.opendev.org/t/openstack/build/fd135474e94b4419b9361d90f61f6a2b : SUCCESS in 17m 46s\n- swift-tox-func-encryption-py27 https://zuul.opendev.org/t/openstack/build/7121ca59617d46309b9beaa853451147 : SUCCESS in 17m 25s\n- swift-tox-func-ec-py27 https://zuul.opendev.org/t/openstack/build/41cc84070891490baaab0942e0274113 : SUCCESS in 16m 40s\n- swift-tox-func-py38 https://zuul.opendev.org/t/openstack/build/ce87b2bed3fd4d369e3c6dd294a9d03b : SUCCESS in 17m 05s\n- swift-tox-func-encryption-py38 https://zuul.opendev.org/t/openstack/build/79b1ac74dae340d9a07962fb2bac3f74 : SUCCESS in 18m 13s\n- swift-tox-func-ec-py38 https://zuul.opendev.org/t/openstack/build/e584a9dc1acd4112bf8c526a8a32b55f : SUCCESS in 17m 11s\n- swift-func-cors https://zuul.opendev.org/t/openstack/build/32a4ede02eb54ee9814c3632442d45b2 : SUCCESS in 6m 50s\n- swift-tox-func-s3api-ceph-s3tests-tempauth https://zuul.opendev.org/t/openstack/build/6098fc45e5354ce6bec6b1cc1b76ad83 : SUCCESS in 19m 24s (non-voting)\n- swift-probetests-centos-7 https://zuul.opendev.org/t/openstack/build/0093e26866a242c3aca552c16d6a8c88 : SUCCESS in 53m 32s\n- swift-probetests-centos-8 https://zuul.opendev.org/t/openstack/build/5dd9f20daf544c4bb2692a7d6a0a34bc : SUCCESS in 50m 34s\n- swift-dsvm-functional https://zuul.opendev.org/t/openstack/build/8817a9c9aaef4c3db30b9afa14bec640 : SUCCESS in 49m 26s\n- swift-dsvm-functional-ipv6 https://zuul.opendev.org/t/openstack/build/ee114d4d44cf4145a3fbd94a93453101 : SUCCESS in 37m 16s\n- swift-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/3e65d380ca0c44c3963a52dd1b28ca56 : SUCCESS in 20m 01s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/c6e107864aa54885a7584af11ddf98f1 : SUCCESS in 6m 03s\n- swift-multinode-rolling-upgrade https://zuul.opendev.org/t/openstack/build/31e6ef58a2504fbf87bb72655f7544e0 : SUCCESS in 28m 24s\n- tempest-ipv6-only https://zuul.opendev.org/t/openstack/build/2a963c7576a74d9f91268a7bd4071ada : SUCCESS in 44m 19s","accounts_in_message":[],"_revision_number":5},{"id":"a3640cb7dd930794e9fca0a900eff18872291071","tag":"autogenerated:zuul:check-arm64","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-08-06 12:52:18.000000000","message":"Patch Set 5:\n\nBuild succeeded (ARM64 pipeline).\n\n- swift-tox-py38-arm64 https://zuul.opendev.org/t/openstack/build/4038914f752d47a587ff4fa2814876c8 : SUCCESS in 47m 38s (non-voting)\n- swift-tox-py39-arm64 https://zuul.opendev.org/t/openstack/build/98475c3be71f49bf8133f82d1ff5044f : SUCCESS in 41m 47s (non-voting)\n- swift-probetests-centos-8-arm64 https://zuul.opendev.org/t/openstack/build/88724e1fa70045f0b57afa2ac2659858 : TIMED_OUT in 2h 01m 41s (non-voting)\n- swift-tox-func-encryption-py38-arm64 https://zuul.opendev.org/t/openstack/build/7db1c507fa5f48dda9ae5d3700e22df1 : SUCCESS in 1h 18m 47s (non-voting)\n- swift-tox-func-py38-arm64 https://zuul.opendev.org/t/openstack/build/020311807f724cacb26c6c3743dc6d91 : SUCCESS in 1h 14m 04s (non-voting)","accounts_in_message":[],"_revision_number":5},{"id":"4e1e5737d4448c4e7e63fb3a448b387aa8e43f3a","author":{"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},"date":"2021-08-06 15:12:43.000000000","message":"Patch Set 5:\n\nI tweaked the commit message slightly.","accounts_in_message":[],"_revision_number":5},{"id":"b4da7ff7b64eb9f01d5aeeefc392da10f72213b2","author":{"_account_id":7233,"name":"Matthew Oliver","email":"matt@oliver.net.au","username":"mattoliverau"},"date":"2021-08-09 00:56:54.000000000","message":"Patch Set 5:\n\n\u003e Patch Set 5: Code-Review-1\n\u003e \n\u003e (1 comment)\n\u003e \n\u003e I think we ought to preserve the header when recursively listing (see follow up for squashing https://review.opendev.org/c/openstack/swift/+/803729):\n\u003e \n\u003e - Sharded root container has all objects deleted\n\u003e \n\u003e - root container is deleted and revived with new policy index\n\u003e \n\u003e - shard ranges still have old policy\n\u003e \n\u003e - shard range start to shard itself: this is very unlikely with autosharding because new object put in the shard will have root\u0027s policy, not shard\u0027s policy, so shard will report itself as having 0 objects to root; but with manual sharding this could happen.\n\u003e \n\u003e - sub-shards have old policy index\n\u003e \n\u003e - listing root will redirect to original shards, then sharding shard redirects to sub-shards\n\nI did think about this when writing the patch, (I have a note on my notepad) but then obviously promptly forgot to loop back and think about it some more. Thanks for bringing it up! .. and potentially fixing it (haven\u0027t looked at the follow up yet, will now).","accounts_in_message":[],"_revision_number":5},{"id":"7701804594d781bc7544c7d8a83ffa727ec80047","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":7233,"name":"Matthew Oliver","email":"matt@oliver.net.au","username":"mattoliverau"},"date":"2021-08-09 05:12:41.000000000","message":"Uploaded patch set 6.","accounts_in_message":[],"_revision_number":6},{"id":"3b9a74839f780c1c911e9de84a66fd8642ec3866","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-08-09 06:23:31.000000000","message":"Patch Set 6: Verified+1\n\nBuild succeeded (check pipeline).\n\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/1d7f839e723e4842833f56acb2ef5ce1 : SUCCESS in 9m 57s\n- grenade https://zuul.opendev.org/t/openstack/build/2b7f953ccec54742a5ba1f1636432688 : SUCCESS in 1h 06m 00s\n- tempest-integrated-object-storage https://zuul.opendev.org/t/openstack/build/95d88791599548a29e4856395c7b2e25 : SUCCESS in 36m 56s\n- swift-build-image https://zuul.opendev.org/t/openstack/build/bec7000c19be4e69b57aadcdc3697f67 : SUCCESS in 9m 28s (non-voting)\n- swift-build-image-py3 https://zuul.opendev.org/t/openstack/build/55402f13357e466ebe637d245e31e832 : SUCCESS in 8m 16s (non-voting)\n- swift-tox-py27 https://zuul.opendev.org/t/openstack/build/ebd89c68ef4745559aa73d0b70db7e09 : SUCCESS in 15m 53s\n- swift-tox-py36 https://zuul.opendev.org/t/openstack/build/19ecd35414f84be5a7af4330a48eaa36 : SUCCESS in 16m 37s\n- swift-tox-py37 https://zuul.opendev.org/t/openstack/build/a6fd087c1e9f444da380edc7e2a5815e : SUCCESS in 16m 14s\n- swift-tox-py38 https://zuul.opendev.org/t/openstack/build/49746b9257904964942ea83f56c3fa1a : SUCCESS in 16m 10s\n- swift-tox-py39 https://zuul.opendev.org/t/openstack/build/9e2b9f5a98be44e2a15bbfd7ee4158cb : SUCCESS in 23m 05s\n- swift-tox-func-py27 https://zuul.opendev.org/t/openstack/build/615919bebb9e47f3855e1f55c4dd4938 : SUCCESS in 16m 10s\n- swift-tox-func-encryption-py27 https://zuul.opendev.org/t/openstack/build/96a2384f7aff42f8bdb3614bf9fbff2f : SUCCESS in 16m 06s\n- swift-tox-func-ec-py27 https://zuul.opendev.org/t/openstack/build/a60a536ef85b4823aa7552851fbb2dac : SUCCESS in 17m 08s\n- swift-tox-func-py38 https://zuul.opendev.org/t/openstack/build/5cdf4c50e86044e2b65151224788a0d2 : SUCCESS in 16m 41s\n- swift-tox-func-encryption-py38 https://zuul.opendev.org/t/openstack/build/71ebdc605c7b4956838e88fa548de70b : SUCCESS in 16m 53s\n- swift-tox-func-ec-py38 https://zuul.opendev.org/t/openstack/build/986d8812e3ba41f9a6e6300655b2c130 : SUCCESS in 17m 17s\n- swift-func-cors https://zuul.opendev.org/t/openstack/build/338ae3cac1b449cea0aa6e9376a239b9 : SUCCESS in 6m 50s\n- swift-tox-func-s3api-ceph-s3tests-tempauth https://zuul.opendev.org/t/openstack/build/10e281f1b89540949dba3585e3058d65 : SUCCESS in 24m 19s (non-voting)\n- swift-probetests-centos-7 https://zuul.opendev.org/t/openstack/build/58c799cfb204491aa1aa2d6b957afb41 : SUCCESS in 51m 54s\n- swift-probetests-centos-8 https://zuul.opendev.org/t/openstack/build/4031f2301e17459aa698ea9fb0ea49f9 : SUCCESS in 51m 38s\n- swift-dsvm-functional https://zuul.opendev.org/t/openstack/build/713e93f493e248c4ab78f595d3a80cea : SUCCESS in 36m 58s\n- swift-dsvm-functional-ipv6 https://zuul.opendev.org/t/openstack/build/c7ea0a6710be4ff39209423a99925f81 : SUCCESS in 36m 11s\n- swift-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/a2856d366bdf418a964264e01e3ea017 : SUCCESS in 17m 47s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/27b11f39988047e8a67f3c4eaadbbcde : SUCCESS in 6m 21s\n- swift-multinode-rolling-upgrade https://zuul.opendev.org/t/openstack/build/5bdb1d61068942d48acee655ecba70be : SUCCESS in 24m 53s\n- tempest-ipv6-only https://zuul.opendev.org/t/openstack/build/c3e047ea8e554688ba6423e396736503 : SUCCESS in 43m 33s","accounts_in_message":[],"_revision_number":6},{"id":"58b1cbf716cda2524623b929e4d4722cfcb1f1cc","tag":"autogenerated:zuul:check-arm64","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-08-09 06:54:32.000000000","message":"Patch Set 6:\n\nBuild succeeded (ARM64 pipeline).\n\n- swift-tox-py38-arm64 https://zuul.opendev.org/t/openstack/build/edfc2bbeeae44ae3b14ac3179ce051f0 : SUCCESS in 34m 38s (non-voting)\n- swift-tox-py39-arm64 https://zuul.opendev.org/t/openstack/build/459250050d0b429da1d2fe00b4b887dd : SUCCESS in 35m 25s (non-voting)\n- swift-probetests-centos-8-arm64 https://zuul.opendev.org/t/openstack/build/494ffe656619445faac60471021a0215 : FAILURE in 1h 39m 56s (non-voting)\n- swift-tox-func-encryption-py38-arm64 https://zuul.opendev.org/t/openstack/build/07fc75ed886545c0b1ed7811060882dc : SUCCESS in 37m 11s (non-voting)\n- swift-tox-func-py38-arm64 https://zuul.opendev.org/t/openstack/build/29dc9daadddf4254922be83060a70476 : SUCCESS in 36m 24s (non-voting)","accounts_in_message":[],"_revision_number":6},{"id":"9ae18076932e9fd15837731c797d93807c53f82d","author":{"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},"date":"2021-08-09 11:41:23.000000000","message":"Patch Set 6: Code-Review+2\n\nI\u0027m co-author so leaving for someone else to +A","accounts_in_message":[],"_revision_number":6},{"id":"29f7df80fca03401afc0c01b16eb731af880b999","author":{"_account_id":1179,"name":"Clay Gerrard","email":"clay.gerrard@gmail.com","username":"clay-gerrard"},"date":"2021-08-09 20:30:53.000000000","message":"Patch Set 6:\n\n(10 comments)\n\nI don\u0027t understand yet how this interacts with caching - maybe i just don\u0027t grok the change in the contaier controller - i\u0027ll keep playing with it","accounts_in_message":[],"_revision_number":6},{"id":"c42c8cf20d7b8d5b0262a44451bfb699ed856d7d","author":{"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"},"date":"2021-08-09 21:59:55.000000000","message":"Patch Set 6: Code-Review+1\n\n(3 comments)\n\nThe\n\n policy_key not in req.headers\n\nclause is still weirding me out a little -- but especially the lack of something like\n\n headers[policy_key] \u003d req.headers[policy_key]\n\nwhen it *is* present. I wouldn\u0027t block it over this, but I want to think about that path some more (including whether it should ever actually come up in practice).","accounts_in_message":[],"_revision_number":6},{"id":"0cc4075b0bcdcf2994305e6563e529c3e43989f4","author":{"_account_id":1179,"name":"Clay Gerrard","email":"clay.gerrard@gmail.com","username":"clay-gerrard"},"date":"2021-08-09 22:16:35.000000000","message":"Patch Set 6: Code-Review+1\n\n(4 comments)\n\nok, i can see how this works with caching - but I\u0027m wondering if the controller changes in the proxy need more time to bake","accounts_in_message":[],"_revision_number":6},{"id":"7ef2eea745880a2cec6be986767f63e9aa00bd47","author":{"_account_id":7233,"name":"Matthew Oliver","email":"matt@oliver.net.au","username":"mattoliverau"},"date":"2021-08-10 02:25:26.000000000","message":"Patch Set 6:\n\n(4 comments)","accounts_in_message":[],"_revision_number":6},{"id":"950cba2212faf0bb94d16d35cc9e064cc822ae38","author":{"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},"date":"2021-08-10 09:07:30.000000000","message":"Patch Set 6:\n\n(6 comments)","accounts_in_message":[],"_revision_number":6},{"id":"9effafb970bfac39d1c0a909a2b7b4e32b4e27b6","author":{"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"},"date":"2021-08-10 18:16:37.000000000","message":"Patch Set 6: -Code-Review\n\n(8 comments)\n\nI\u0027m getting a little worried about re-using the same response header that\u0027s previously meant \"the storage policy for this container\" to now mean \"the storage policy for this set of objects\".","accounts_in_message":[],"_revision_number":6},{"id":"5a1fee27d4d6fc006dea754e3266df76bff48081","author":{"_account_id":7233,"name":"Matthew Oliver","email":"matt@oliver.net.au","username":"mattoliverau"},"date":"2021-08-11 06:16:24.000000000","message":"Patch Set 6:\n\n(2 comments)","accounts_in_message":[],"_revision_number":6},{"id":"0490cb434fedbfce1594f00a53e5dcc9987f85bf","author":{"_account_id":7233,"name":"Matthew Oliver","email":"matt@oliver.net.au","username":"mattoliverau"},"date":"2021-08-11 07:14:21.000000000","message":"Patch Set 6:\n\n(1 comment)","accounts_in_message":[],"_revision_number":6},{"id":"95252f7657ac6f6e879c427451626cd5087dec68","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":7233,"name":"Matthew Oliver","email":"matt@oliver.net.au","username":"mattoliverau"},"date":"2021-08-12 01:52:23.000000000","message":"Uploaded patch set 7.","accounts_in_message":[],"_revision_number":7},{"id":"bbf23275286eb5990b3bd57b08c705d181d3e879","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-08-12 02:58:56.000000000","message":"Patch Set 7: Verified+1\n\nBuild succeeded (check pipeline).\n\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/15baa8f2e6f94cf18568c47e0a0224ad : SUCCESS in 8m 55s\n- grenade https://zuul.opendev.org/t/openstack/build/5b6fc63680cc4cdea3aa365b6e425d83 : SUCCESS in 1h 05m 16s\n- tempest-integrated-object-storage https://zuul.opendev.org/t/openstack/build/a3ed840e0b754e448fcd3273a3dbb0dd : SUCCESS in 41m 25s\n- swift-build-image https://zuul.opendev.org/t/openstack/build/1db63732180d4d448bfc678611bfb07b : SUCCESS in 10m 41s (non-voting)\n- swift-build-image-py3 https://zuul.opendev.org/t/openstack/build/04d8be63e2fe427e963a6e72c0ba2805 : SUCCESS in 9m 31s (non-voting)\n- swift-tox-py27 https://zuul.opendev.org/t/openstack/build/89dcd4be0c084ebabbe6310b7d898d9f : SUCCESS in 21m 14s\n- swift-tox-py36 https://zuul.opendev.org/t/openstack/build/fc2a8b27ea4e4ca8a7f0b9e341bbbd9f : SUCCESS in 20m 44s\n- swift-tox-py37 https://zuul.opendev.org/t/openstack/build/32b20c92834b4583a8f53976cbec059b : SUCCESS in 18m 17s\n- swift-tox-py38 https://zuul.opendev.org/t/openstack/build/0fdb41114784471fa65e591d64c82be8 : SUCCESS in 20m 13s\n- swift-tox-py39 https://zuul.opendev.org/t/openstack/build/73375e77ae67408183859ade893f2c6f : SUCCESS in 19m 44s\n- swift-tox-func-py27 https://zuul.opendev.org/t/openstack/build/1ab7ad238bf54684afdf85a69bc6fec1 : SUCCESS in 14m 40s\n- swift-tox-func-encryption-py27 https://zuul.opendev.org/t/openstack/build/dcaf364ab85b411d97363daeba33f1c4 : SUCCESS in 20m 07s\n- swift-tox-func-ec-py27 https://zuul.opendev.org/t/openstack/build/abc98ca2e4d04581b4ce7724b36a8397 : SUCCESS in 18m 46s\n- swift-tox-func-py38 https://zuul.opendev.org/t/openstack/build/700a51abe3664e9c8d9753db571d7021 : SUCCESS in 17m 07s\n- swift-tox-func-encryption-py38 https://zuul.opendev.org/t/openstack/build/0eaa9fb209904308bc80e9d48b541913 : SUCCESS in 30m 36s\n- swift-tox-func-ec-py38 https://zuul.opendev.org/t/openstack/build/44ee6f8913d142ae8cda8b4699a2072e : SUCCESS in 20m 53s\n- swift-func-cors https://zuul.opendev.org/t/openstack/build/1610db903e514ea784c63e07d9f0941c : SUCCESS in 7m 00s\n- swift-tox-func-s3api-ceph-s3tests-tempauth https://zuul.opendev.org/t/openstack/build/a54e519d7e3548748eac370a11ac8f76 : SUCCESS in 22m 01s (non-voting)\n- swift-probetests-centos-7 https://zuul.opendev.org/t/openstack/build/58d0b2c70b1c4932953e6b3e66902e8c : SUCCESS in 49m 23s\n- swift-probetests-centos-8 https://zuul.opendev.org/t/openstack/build/5f01c33b27294445892608d62577cb85 : SUCCESS in 49m 15s\n- swift-dsvm-functional https://zuul.opendev.org/t/openstack/build/d41b870348dc4705ac57105484fcbdb1 : SUCCESS in 37m 30s\n- swift-dsvm-functional-ipv6 https://zuul.opendev.org/t/openstack/build/8c5c21a91ca04991a4f81414d0b13e41 : SUCCESS in 41m 05s\n- swift-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/06767c2a7c274bd3a695b78d99dfb37b : SUCCESS in 16m 39s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/6b485a4a02024230adce3b057f9c6189 : SUCCESS in 6m 48s\n- swift-multinode-rolling-upgrade https://zuul.opendev.org/t/openstack/build/b68271e5cfb5426a9fa73b7e3a1a59dd : SUCCESS in 24m 11s\n- tempest-ipv6-only https://zuul.opendev.org/t/openstack/build/f427a7b7894b4c46b9c001d378dd191b : SUCCESS in 45m 07s","accounts_in_message":[],"_revision_number":7},{"id":"1cf4a29a55b131943fee46aa046729a269ebb87a","tag":"autogenerated:zuul:check-arm64","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-08-12 03:26:07.000000000","message":"Patch Set 7:\n\nBuild succeeded (ARM64 pipeline).\n\n- swift-tox-py38-arm64 https://zuul.opendev.org/t/openstack/build/101e263a40ce49fcaee9681007201c7f : SUCCESS in 40m 13s (non-voting)\n- swift-tox-py39-arm64 https://zuul.opendev.org/t/openstack/build/6bc778b86f81477bb21ffca4d55a186f : SUCCESS in 43m 34s (non-voting)\n- swift-probetests-centos-8-arm64 https://zuul.opendev.org/t/openstack/build/0216608263114d0289862ed171f5ed91 : FAILURE in 1h 25m 39s (non-voting)\n- swift-tox-func-encryption-py38-arm64 https://zuul.opendev.org/t/openstack/build/83b82b2992c64f6aa3fd39542d7eb6d5 : SUCCESS in 34m 59s (non-voting)\n- swift-tox-func-py38-arm64 https://zuul.opendev.org/t/openstack/build/4ba09a6dce6a4087b2ee6adf8f9d7905 : SUCCESS in 33m 20s (non-voting)","accounts_in_message":[],"_revision_number":7},{"id":"39abc089c44492d66cdf902ed60ba572156c1f4f","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},"date":"2021-08-12 12:57:20.000000000","message":"Uploaded patch set 8.","accounts_in_message":[],"_revision_number":8},{"id":"eb5c486f76d21eac70c5a3c04e5975e55c21bef5","author":{"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},"date":"2021-08-12 12:59:08.000000000","message":"Patch Set 8: Code-Review+2\n\n(1 comment)\n\nI\u0027ve squashed in the change to how we set the header in the root request from https://review.opendev.org/c/openstack/swift/+/804234/3","accounts_in_message":[],"_revision_number":8},{"id":"bb376a32360ff50fb69463b5563469a66eecbc84","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-08-12 14:06:14.000000000","message":"Patch Set 8: Verified+1\n\nBuild succeeded (check pipeline).\n\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/6146071d0db543b08e07b4fac8ab2858 : SUCCESS in 9m 21s\n- grenade https://zuul.opendev.org/t/openstack/build/d6803cc8418a4b27ac5f55dc838a07e5 : SUCCESS in 55m 00s\n- tempest-integrated-object-storage https://zuul.opendev.org/t/openstack/build/ab354cd13e2d4297a410993bff2bb8be : SUCCESS in 41m 24s\n- swift-build-image https://zuul.opendev.org/t/openstack/build/33e3d0266f0b4fb4a8a8f9cb662dfb75 : SUCCESS in 9m 37s (non-voting)\n- swift-build-image-py3 https://zuul.opendev.org/t/openstack/build/f732359dadc14a38bb9daf7b511e8749 : SUCCESS in 9m 11s (non-voting)\n- swift-tox-py27 https://zuul.opendev.org/t/openstack/build/e06b6d64aac44ec78c86e66b750b72dc : SUCCESS in 17m 54s\n- swift-tox-py36 https://zuul.opendev.org/t/openstack/build/80ae2b405e59488aa0c67149d0fb5695 : SUCCESS in 19m 15s\n- swift-tox-py37 https://zuul.opendev.org/t/openstack/build/6bd810c200624b918d6dd0108fe006bc : SUCCESS in 21m 11s\n- swift-tox-py38 https://zuul.opendev.org/t/openstack/build/e37b697e66dc4584bb6ef0184dd514d4 : SUCCESS in 21m 20s\n- swift-tox-py39 https://zuul.opendev.org/t/openstack/build/1473f8f9beb04d0b8b9b067ed454104b : SUCCESS in 16m 37s\n- swift-tox-func-py27 https://zuul.opendev.org/t/openstack/build/17dcbb2ccc4e44bf9abe946cb7ef1c5c : SUCCESS in 19m 21s\n- swift-tox-func-encryption-py27 https://zuul.opendev.org/t/openstack/build/4721f7919fde45d198fdcfd8126b45db : SUCCESS in 21m 29s\n- swift-tox-func-ec-py27 https://zuul.opendev.org/t/openstack/build/463a910922c341a29d005f497c2cc0e5 : SUCCESS in 21m 35s\n- swift-tox-func-py38 https://zuul.opendev.org/t/openstack/build/1124319f758c4e80839f083dd532aeac : SUCCESS in 16m 02s\n- swift-tox-func-encryption-py38 https://zuul.opendev.org/t/openstack/build/3856ed662b2f478796ebe70b59c470a0 : SUCCESS in 18m 56s\n- swift-tox-func-ec-py38 https://zuul.opendev.org/t/openstack/build/12c861f8871e49b7859496c54745a33d : SUCCESS in 22m 36s\n- swift-func-cors https://zuul.opendev.org/t/openstack/build/d8dd2801c3904a83beddfb1adab1a91a : SUCCESS in 10m 16s\n- swift-tox-func-s3api-ceph-s3tests-tempauth https://zuul.opendev.org/t/openstack/build/b37a620d642b4b8ba5dec4938afc212d : SUCCESS in 20m 04s (non-voting)\n- swift-probetests-centos-7 https://zuul.opendev.org/t/openstack/build/7e3224844f484df69c0b40b3077951ce : SUCCESS in 54m 30s\n- swift-probetests-centos-8 https://zuul.opendev.org/t/openstack/build/8e437f5f7c6a47be964c3092f0ce0416 : SUCCESS in 57m 19s\n- swift-dsvm-functional https://zuul.opendev.org/t/openstack/build/60c48c7ded124c5da7bf0aca16219644 : SUCCESS in 39m 06s\n- swift-dsvm-functional-ipv6 https://zuul.opendev.org/t/openstack/build/c214319358e641a497e29d5726d52055 : SUCCESS in 41m 10s\n- swift-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/d925c6560dda4ed28819b81e4be5115a : SUCCESS in 17m 55s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/f446a88a37b643848b5ef8e7f4b28c54 : SUCCESS in 7m 14s\n- swift-multinode-rolling-upgrade https://zuul.opendev.org/t/openstack/build/20e06033ed8042a3b2f1fceb240098d7 : SUCCESS in 24m 02s\n- tempest-ipv6-only https://zuul.opendev.org/t/openstack/build/edfc6a557e99413891ebfc73724f5edd : SUCCESS in 1h 03m 16s","accounts_in_message":[],"_revision_number":8},{"id":"442c859e970ccb98c80fb346c4ed13b8f58bcc00","tag":"autogenerated:zuul:check-arm64","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-08-12 14:27:21.000000000","message":"Patch Set 8:\n\nBuild succeeded (ARM64 pipeline).\n\n- swift-tox-py38-arm64 https://zuul.opendev.org/t/openstack/build/d80a802e2e1942dda8fdac210964f579 : SUCCESS in 41m 37s (non-voting)\n- swift-tox-py39-arm64 https://zuul.opendev.org/t/openstack/build/c5555df70bcf41eebaf2b72b57ecdefb : SUCCESS in 44m 13s (non-voting)\n- swift-probetests-centos-8-arm64 https://zuul.opendev.org/t/openstack/build/dc2250f0340249f3a8aaedc1e5c5313b : SUCCESS in 1h 24m 35s (non-voting)\n- swift-tox-func-encryption-py38-arm64 https://zuul.opendev.org/t/openstack/build/1ff49d8a0a364e118774780fbeb2337b : SUCCESS in 39m 28s (non-voting)\n- swift-tox-func-py38-arm64 https://zuul.opendev.org/t/openstack/build/cd09fd611d39480fa12faa0884b904c2 : SUCCESS in 42m 20s (non-voting)","accounts_in_message":[],"_revision_number":8},{"id":"7e163fb2c72c1752be71727c07e500938034cdf6","author":{"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"},"date":"2021-08-13 21:04:35.000000000","message":"Patch Set 8: Code-Review+1\n\n(2 comments)","accounts_in_message":[],"_revision_number":8},{"id":"7bb5b680864de67c123e92b6ab1cd5cd68891447","author":{"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},"date":"2021-08-16 07:07:04.000000000","message":"Patch Set 8:\n\n(1 comment)","accounts_in_message":[],"_revision_number":8},{"id":"10003666ac38771892ac1217d9468f7d4aa3b7c6","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},"date":"2021-08-16 10:35:00.000000000","message":"Uploaded patch set 9.","accounts_in_message":[],"_revision_number":9},{"id":"b5b307a20cc0199d349595f0ca7cae4565778868","author":{"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},"date":"2021-08-16 10:38:37.000000000","message":"Patch Set 9:\n\nI put this in a follow up patch https://review.opendev.org/c/openstack/swift/+/804695\n\nSince we are already carrying the current patch I wanted to consider the change from returning an empty (most likely) listing to a 503 separately.","accounts_in_message":[],"_revision_number":9},{"id":"51cb5a69fb5ab5188e6125b58eb11e5d7387af4f","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},"date":"2021-08-16 10:57:06.000000000","message":"Uploaded patch set 10: New patch set was added with same tree, parent, and commit message as Patch Set 9.","accounts_in_message":[],"_revision_number":10},{"id":"7f56836604faee98ed30175cc79b993e19f39e3b","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-08-16 11:57:52.000000000","message":"Patch Set 10: Verified+1\n\nBuild succeeded (check pipeline).\n\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/68ffd78970524615bcc5f87b4d8b84ec : SUCCESS in 9m 16s\n- grenade https://zuul.opendev.org/t/openstack/build/ce15a88ebbeb44a1b9509f1966944b4d : SUCCESS in 55m 49s\n- tempest-integrated-object-storage https://zuul.opendev.org/t/openstack/build/bf3e60b052694c199e2165c19a521e7a : SUCCESS in 37m 36s\n- swift-build-image https://zuul.opendev.org/t/openstack/build/fc27de6ce3ef4f558cc470d899f99815 : SUCCESS in 10m 07s (non-voting)\n- swift-build-image-py3 https://zuul.opendev.org/t/openstack/build/b6e2b7288a1044009e0697ca496627cf : SUCCESS in 8m 37s (non-voting)\n- swift-tox-py27 https://zuul.opendev.org/t/openstack/build/7be0949bd3f04c8ab7ce569f6d0faa45 : SUCCESS in 18m 25s\n- swift-tox-py36 https://zuul.opendev.org/t/openstack/build/2aa39f3515624b87b4bf488729233de3 : SUCCESS in 16m 17s\n- swift-tox-py37 https://zuul.opendev.org/t/openstack/build/3f5dc868f3ed4639806f67e3a757c5ed : SUCCESS in 20m 52s\n- swift-tox-py38 https://zuul.opendev.org/t/openstack/build/3d703358aa9f48118e2c0bc3dae1620c : SUCCESS in 15m 54s\n- swift-tox-py39 https://zuul.opendev.org/t/openstack/build/8d1945b3850d4d07b50b7c8d057012f4 : SUCCESS in 17m 03s\n- swift-tox-func-py27 https://zuul.opendev.org/t/openstack/build/0832bc62e9d3406db942d4580fc4b12c : SUCCESS in 16m 59s\n- swift-tox-func-encryption-py27 https://zuul.opendev.org/t/openstack/build/298d0562e4c94df1911b45c82080aef1 : SUCCESS in 16m 58s\n- swift-tox-func-ec-py27 https://zuul.opendev.org/t/openstack/build/6dce8b57940742e5be5635e2d463f82c : SUCCESS in 17m 14s\n- swift-tox-func-py38 https://zuul.opendev.org/t/openstack/build/1b9f466c0af84eb2a848f720792bc3fe : SUCCESS in 17m 52s\n- swift-tox-func-encryption-py38 https://zuul.opendev.org/t/openstack/build/1278ff8a951543d5bdf9410d61c82825 : SUCCESS in 16m 30s\n- swift-tox-func-ec-py38 https://zuul.opendev.org/t/openstack/build/cb24a52f6b8b44508b1800b0477c1df3 : SUCCESS in 20m 56s\n- swift-func-cors https://zuul.opendev.org/t/openstack/build/5d0715553d454cdc80ee67560f23ee02 : SUCCESS in 6m 55s\n- swift-tox-func-s3api-ceph-s3tests-tempauth https://zuul.opendev.org/t/openstack/build/9d4319844a77495f97a32061c2c40ede : SUCCESS in 25m 46s (non-voting)\n- swift-probetests-centos-7 https://zuul.opendev.org/t/openstack/build/105e022327b84a39875d92263826714d : SUCCESS in 56m 37s\n- swift-probetests-centos-8 https://zuul.opendev.org/t/openstack/build/6c502b3594964cc3946471530430ec6f : SUCCESS in 52m 48s\n- swift-dsvm-functional https://zuul.opendev.org/t/openstack/build/9dc8b1aa9a4c4180984cc0b4b167241f : SUCCESS in 40m 59s\n- swift-dsvm-functional-ipv6 https://zuul.opendev.org/t/openstack/build/cd488952e0174206886e9fa9e3379ec3 : SUCCESS in 41m 44s\n- swift-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/d8d67a77ea9b4fd69470b327395ad4de : SUCCESS in 23m 32s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/106db6bea5594563824d1f0f327d1eaa : SUCCESS in 7m 16s\n- swift-multinode-rolling-upgrade https://zuul.opendev.org/t/openstack/build/3c2616a3f643450595a7a7f37ebbd7a2 : SUCCESS in 32m 17s\n- tempest-ipv6-only https://zuul.opendev.org/t/openstack/build/57f242bec1cd48258a0136a062ee3eb8 : SUCCESS in 51m 30s","accounts_in_message":[],"_revision_number":10},{"id":"15a110883cf98a7bd1a521e9ff6572a843dafe50","tag":"autogenerated:zuul:check-arm64","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-08-16 12:48:13.000000000","message":"Patch Set 10:\n\nBuild succeeded (ARM64 pipeline).\n\n- swift-tox-py38-arm64 https://zuul.opendev.org/t/openstack/build/df5ec192c34e4fc99614aa6826fa4bcc : SUCCESS in 36m 11s (non-voting)\n- swift-tox-py39-arm64 https://zuul.opendev.org/t/openstack/build/c11e1117f052487d809731cd9f3d1504 : SUCCESS in 41m 07s (non-voting)\n- swift-probetests-centos-8-arm64 https://zuul.opendev.org/t/openstack/build/88fca03c03254c02b70aafb460e8092b : FAILURE in 1h 47m 06s (non-voting)\n- swift-tox-func-encryption-py38-arm64 https://zuul.opendev.org/t/openstack/build/f8cff816d53e47b692e0654bc4d78056 : SUCCESS in 53m 09s (non-voting)\n- swift-tox-func-py38-arm64 https://zuul.opendev.org/t/openstack/build/34eb5dc2fc3144d8bfecf06ad2156428 : SUCCESS in 34m 16s (non-voting)","accounts_in_message":[],"_revision_number":10},{"id":"a01f8b27cd89d4666a61495e7b261b3cc041b2f8","author":{"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"},"date":"2021-08-16 18:30:14.000000000","message":"Patch Set 10: Code-Review+2","accounts_in_message":[],"_revision_number":10},{"id":"4d5e835204998a5969e2c95fabc3c9bca2a2def0","author":{"_account_id":1179,"name":"Clay Gerrard","email":"clay.gerrard@gmail.com","username":"clay-gerrard"},"date":"2021-08-16 21:33:17.000000000","message":"Patch Set 10: Code-Review+2 Workflow+1\n\ni like how this got polished up","accounts_in_message":[],"_revision_number":10},{"id":"1dcd34784a8efae20f7cb410871421d2b540704c","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-08-16 21:33:49.000000000","message":"Patch Set 10: -Verified\n\nStarting gate jobs.","accounts_in_message":[],"_revision_number":10},{"id":"0bb1875e543fc191bee40b424c84a8b6a3b6b806","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-08-16 23:01:04.000000000","message":"Patch Set 10: Verified+2\n\nBuild succeeded (gate pipeline).\n\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/8f0e97ea9dce4633bbb924011cf41487 : SUCCESS in 10m 08s\n- grenade https://zuul.opendev.org/t/openstack/build/34afdbe31d624f558ad886ff1fc531a0 : SUCCESS in 1h 04m 17s\n- tempest-integrated-object-storage https://zuul.opendev.org/t/openstack/build/a7612bff1bf54b0ca029eb3a7dfa9a2c : SUCCESS in 48m 57s\n- swift-upload-image https://zuul.opendev.org/t/openstack/build/06adc08475294931a55344b9fc07254c : SUCCESS in 10m 12s (non-voting)\n- swift-upload-image-py3 https://zuul.opendev.org/t/openstack/build/406f0ca1c8604096b1b4f7a4f9371f78 : SUCCESS in 9m 43s (non-voting)\n- swift-tox-py27 https://zuul.opendev.org/t/openstack/build/14079dec25744773ac828e0c703a1a75 : SUCCESS in 16m 04s\n- swift-tox-py36 https://zuul.opendev.org/t/openstack/build/d221986d19b94fe0a199d7fa946e7854 : SUCCESS in 18m 21s\n- swift-tox-py37 https://zuul.opendev.org/t/openstack/build/86742877437b44ed80223f0bab11a20d : SUCCESS in 17m 58s\n- swift-tox-py38 https://zuul.opendev.org/t/openstack/build/98b4ebdd63d5479188f90e796597f97e : SUCCESS in 21m 16s\n- swift-tox-py39 https://zuul.opendev.org/t/openstack/build/32b7e455c41c45e1b6ef3dbca80bfff9 : SUCCESS in 17m 17s\n- swift-tox-func-py27 https://zuul.opendev.org/t/openstack/build/8475773735c54b6385002d99b9a81531 : SUCCESS in 16m 17s\n- swift-tox-func-encryption-py27 https://zuul.opendev.org/t/openstack/build/8e262fc08fac498eaeeae14aee01ef97 : SUCCESS in 18m 08s\n- swift-tox-func-ec-py27 https://zuul.opendev.org/t/openstack/build/5b4c77fc5c6044888d2f67aee45f9236 : SUCCESS in 23m 30s\n- swift-tox-func-py38 https://zuul.opendev.org/t/openstack/build/14a7408869074c2e9bb30b7924d5ac75 : SUCCESS in 21m 36s\n- swift-tox-func-encryption-py38 https://zuul.opendev.org/t/openstack/build/b1b3b0ca10274300957724afe6d98f78 : SUCCESS in 17m 13s\n- swift-tox-func-ec-py38 https://zuul.opendev.org/t/openstack/build/dc21a35de6f54a7087318daaca60fadc : SUCCESS in 18m 00s\n- swift-func-cors https://zuul.opendev.org/t/openstack/build/b439c9106fec41c780aa8d6e3bcfe91e : SUCCESS in 6m 39s\n- swift-probetests-centos-7 https://zuul.opendev.org/t/openstack/build/58ef7c5e244e4c4e82aecb551801492f : SUCCESS in 48m 11s\n- swift-probetests-centos-8 https://zuul.opendev.org/t/openstack/build/a2257747ebf84a699c739192434a4a4e : SUCCESS in 45m 39s\n- swift-dsvm-functional https://zuul.opendev.org/t/openstack/build/18eb323577f24c7da3a6133ff0e3213b : SUCCESS in 37m 17s\n- swift-dsvm-functional-ipv6 https://zuul.opendev.org/t/openstack/build/0e6d736346ba4502bf19df59800b6256 : SUCCESS in 36m 51s\n- swift-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/197fbc886df44c6a86d7d0a31804e758 : SUCCESS in 15m 24s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/5aadb8aee0da4321a4416935919c65fe : SUCCESS in 6m 08s\n- swift-multinode-rolling-upgrade https://zuul.opendev.org/t/openstack/build/b91c47395922427ea975395ae913a609 : SUCCESS in 22m 07s\n- tempest-ipv6-only https://zuul.opendev.org/t/openstack/build/e3c492d3efe94dc5aad01c993ca996c8 : SUCCESS in 46m 28s","accounts_in_message":[],"_revision_number":10},{"id":"0dce0ff6444c8e6c7e1faa774c668cf24fb77474","tag":"autogenerated:gerrit:merged","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-08-16 23:01:05.000000000","message":"Change has been successfully merged by Zuul","accounts_in_message":[],"_revision_number":10},{"id":"e545971ee1082d29eabc469eeb941752d0135cf4","tag":"autogenerated:zuul:promote","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-08-16 23:02:50.000000000","message":"Patch Set 10:\n\nBuild succeeded (promote pipeline).\n\n- promote-openstack-tox-docs https://zuul.opendev.org/t/openstack/build/2cfefcd312cd487e88fa2e5a0c457ee4 : SUCCESS in 1m 08s\n- swift-promote-image https://zuul.opendev.org/t/openstack/build/cb395ddf7eea46818fc857d52d751520 : SUCCESS in 47s (non-voting)\n- swift-promote-image-py3 https://zuul.opendev.org/t/openstack/build/a758703eb6ea4f518e0bf1d24b17ae6b : SUCCESS in 1m 01s (non-voting)","accounts_in_message":[],"_revision_number":10}],"current_revision_number":10,"current_revision":"56510ab3c31070a1b8720a33d26c8a220e6ce5c9","revisions":{"8d257c8998192afb393c72889f6349cc1581951b":{"kind":"REWORK","_number":1,"created":"2021-08-04 07:21:46.000000000","uploader":{"_account_id":7233,"name":"Matthew Oliver","email":"matt@oliver.net.au","username":"mattoliverau"},"ref":"refs/changes/23/803423/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/swift","ref":"refs/changes/23/803423/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/swift refs/changes/23/803423/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/swift refs/changes/23/803423/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/swift refs/changes/23/803423/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/swift refs/changes/23/803423/1"}}},"commit":{"parents":[{"commit":"6053a65c63340cafa661fa4379ae85d035615f53","subject":"sharder: update shard storage_policy_index if roots changes","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/swift/commit/6053a65c63340cafa661fa4379ae85d035615f53"}]}],"author":{"name":"Matthew Oliver","email":"matt@oliver.net.au","date":"2021-08-04 07:07:19.000000000","tz":600},"committer":{"name":"Matthew Oliver","email":"matt@oliver.net.au","date":"2021-08-04 07:07:19.000000000","tz":600},"subject":"conatiner-server: return objects of a given policy","message":"conatiner-server: return objects of a given policy\n\nThere is a tight coupling between a root container and it\u0027s shards and\nthere can be a tail before a roots storage policy index is migrated to\nit\u0027s shards. Especially in the case of a delete and creating the new\ncontainer again with a new policy.\nWhen we PUT objects in to a root container, it\u0027ll redirect them, with\nthe roots policy to the shards. And the shards are happy to take them.\nBut when it comes to GETs, the root forwards the GET onto it\u0027s shards\nwhom currently wont respond with objects (which they probably took) becuase\nthey are of a different policy. When getting objects from the container\nserver, the policy used is always the brokers policy.\n\nThis patch corrects this behaviour by allowing the ability to override\nthe policy index to use. If the request to the container server\ncontains an \u0027X-Backend-Storage-Policy-Index\u0027 header it\u0027ll be used\ninstead of the policy index stored in the broker. This header isn\u0027t\nadded between the Proxy -\u003e Container servers by default.\n\nThis patch adds the root containers policy as this header to the proxy\ncontainer controllers `_get_from_shards` method which is used to build up\nthe GET response for a shard container.\n\nOn a plus side this new container policy API allows us a way to check\ncontainers for object listing is other policies. So might come in handy\nfor OPs/SREs.\n\nChange-Id: I026b699fc5f0fba619cf524093632d67ca38d32f\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/swift/commit/8d257c8998192afb393c72889f6349cc1581951b"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/swift/commit/8d257c8998192afb393c72889f6349cc1581951b"}]},"branch":"refs/heads/master"},"9852ac9263a98403a8667e5b74ca842ed0943977":{"kind":"REWORK","_number":2,"created":"2021-08-05 03:27:50.000000000","uploader":{"_account_id":7233,"name":"Matthew Oliver","email":"matt@oliver.net.au","username":"mattoliverau"},"ref":"refs/changes/23/803423/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/swift","ref":"refs/changes/23/803423/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/swift refs/changes/23/803423/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/swift refs/changes/23/803423/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/swift refs/changes/23/803423/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/swift refs/changes/23/803423/2"}}},"commit":{"parents":[{"commit":"b3c5a9a9e01aafa2a0819636b2eb20640cf7b529","subject":"Merge \"s3api: Fix (async) multi-delete of MPUs\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/swift/commit/b3c5a9a9e01aafa2a0819636b2eb20640cf7b529"}]}],"author":{"name":"Matthew Oliver","email":"matt@oliver.net.au","date":"2021-08-04 07:07:19.000000000","tz":600},"committer":{"name":"Matthew Oliver","email":"matt@oliver.net.au","date":"2021-08-05 03:25:51.000000000","tz":600},"subject":"conatiner-server: return objects of a given policy","message":"conatiner-server: return objects of a given policy\n\nThere is a tight coupling between a root container and it\u0027s shards and\nthere can be a tail before a roots storage policy index is migrated to\nit\u0027s shards. Especially in the case of a delete and creating the new\ncontainer again with a new policy.\nWhen we PUT objects in to a root container, it\u0027ll redirect them, with\nthe roots policy to the shards. And the shards are happy to take them.\nBut when it comes to GETs, the root forwards the GET onto it\u0027s shards\nwhom currently wont respond with objects (which they probably took) becuase\nthey are of a different policy. When getting objects from the container\nserver, the policy used is always the brokers policy.\n\nThis patch corrects this behaviour by allowing the ability to override\nthe policy index to use. If the request to the container server\ncontains an \u0027X-Backend-Storage-Policy-Index\u0027 header it\u0027ll be used\ninstead of the policy index stored in the broker. This header isn\u0027t\nadded between the Proxy -\u003e Container servers by default.\n\nThis patch adds the root containers policy as this header to the proxy\ncontainer controllers `_get_from_shards` method which is used to build up\nthe GET response for a shard container.\n\nOn a plus side this new container policy API allows us a way to check\ncontainers for object listing is other policies. So might come in handy\nfor OPs/SREs.\n\nChange-Id: I026b699fc5f0fba619cf524093632d67ca38d32f\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/swift/commit/9852ac9263a98403a8667e5b74ca842ed0943977"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/swift/commit/9852ac9263a98403a8667e5b74ca842ed0943977"}]},"branch":"refs/heads/master"},"b54844b0bb1beca29b85bc5c666df018b49198cb":{"kind":"REWORK","_number":3,"created":"2021-08-05 04:55:17.000000000","uploader":{"_account_id":7233,"name":"Matthew Oliver","email":"matt@oliver.net.au","username":"mattoliverau"},"ref":"refs/changes/23/803423/3","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/swift","ref":"refs/changes/23/803423/3","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/swift refs/changes/23/803423/3 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/swift refs/changes/23/803423/3 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/swift refs/changes/23/803423/3 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/swift refs/changes/23/803423/3"}}},"commit":{"parents":[{"commit":"b3c5a9a9e01aafa2a0819636b2eb20640cf7b529","subject":"Merge \"s3api: Fix (async) multi-delete of MPUs\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/swift/commit/b3c5a9a9e01aafa2a0819636b2eb20640cf7b529"}]}],"author":{"name":"Matthew Oliver","email":"matt@oliver.net.au","date":"2021-08-04 07:07:19.000000000","tz":600},"committer":{"name":"Matthew Oliver","email":"matt@oliver.net.au","date":"2021-08-05 04:55:04.000000000","tz":600},"subject":"conatiner-server: return objects of a given policy","message":"conatiner-server: return objects of a given policy\n\nThere is a tight coupling between a root container and it\u0027s shards and\nthere can be a tail before a roots storage policy index is migrated to\nit\u0027s shards. Especially in the case of a delete and creating the new\ncontainer again with a new policy.\nWhen we PUT objects in to a root container, it\u0027ll redirect them, with\nthe roots policy to the shards. And the shards are happy to take them.\nBut when it comes to GETs, the root forwards the GET onto it\u0027s shards\nwhom currently wont respond with objects (which they probably took) becuase\nthey are of a different policy. When getting objects from the container\nserver, the policy used is always the brokers policy.\n\nThis patch corrects this behaviour by allowing the ability to override\nthe policy index to use. If the request to the container server\ncontains an \u0027X-Backend-Storage-Policy-Index\u0027 header it\u0027ll be used\ninstead of the policy index stored in the broker. This header isn\u0027t\nadded between the Proxy -\u003e Container servers by default.\n\nThis patch adds the root containers policy as this header to the proxy\ncontainer controllers `_get_from_shards` method which is used to build up\nthe GET response for a shard container.\n\nOn a plus side this new container policy API allows us a way to check\ncontainers for object listing is other policies. So might come in handy\nfor OPs/SREs.\n\nChange-Id: I026b699fc5f0fba619cf524093632d67ca38d32f\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/swift/commit/b54844b0bb1beca29b85bc5c666df018b49198cb"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/swift/commit/b54844b0bb1beca29b85bc5c666df018b49198cb"}]},"branch":"refs/heads/master"},"57242c1ab314791a1df7a2d7ff4e3ed411aef5b3":{"kind":"REWORK","_number":4,"created":"2021-08-06 00:12:40.000000000","uploader":{"_account_id":7233,"name":"Matthew Oliver","email":"matt@oliver.net.au","username":"mattoliverau"},"ref":"refs/changes/23/803423/4","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/swift","ref":"refs/changes/23/803423/4","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/swift refs/changes/23/803423/4 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/swift refs/changes/23/803423/4 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/swift refs/changes/23/803423/4 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/swift refs/changes/23/803423/4"}}},"commit":{"parents":[{"commit":"b3c5a9a9e01aafa2a0819636b2eb20640cf7b529","subject":"Merge \"s3api: Fix (async) multi-delete of MPUs\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/swift/commit/b3c5a9a9e01aafa2a0819636b2eb20640cf7b529"}]}],"author":{"name":"Matthew Oliver","email":"matt@oliver.net.au","date":"2021-08-04 07:07:19.000000000","tz":600},"committer":{"name":"Matthew Oliver","email":"matt@oliver.net.au","date":"2021-08-05 23:45:30.000000000","tz":600},"subject":"container-server: return objects of a given policy","message":"container-server: return objects of a given policy\n\nThere is a tight coupling between a root container and it\u0027s shards the\nshards hold the object metadata for the root container, so are really an\nextension of the root.\nWhen we PUT objects in to a root container, it\u0027ll redirect them, with\nthe roots policy, to the shards. And the shards are happy to take them.\nBut when it comes to GETs, the root redirects the GET onto it\u0027s shards\nwhom currently wont respond with objects (which they probably took) because\nthey are of a different policy. When getting objects from the container\nserver, the policy used is always the brokers policy.\n\nThis patch corrects this behaviour by allowing the ability to override\nthe policy index to use. If the request to the container server\ncontains an \u0027X-Backend-Storage-Policy-Index\u0027 header it\u0027ll be used\ninstead of the policy index stored in the broker.\n\nThis patch adds the root containers policy as this header to the proxy\ncontainer controllers `_get_from_shards` method which is used by the proxy\nto redirect a GET to a root to it\u0027s shards.\n\nOn a plus side this new container policy API allows us a way to check\ncontainers for object listing is other policies. So might come in handy\nfor OPs/SREs.\n\nChange-Id: I026b699fc5f0fba619cf524093632d67ca38d32f\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/swift/commit/57242c1ab314791a1df7a2d7ff4e3ed411aef5b3"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/swift/commit/57242c1ab314791a1df7a2d7ff4e3ed411aef5b3"}]},"branch":"refs/heads/master"},"1e13e3fd0321f13bca2dd0be5bef5c917cb65e7e":{"kind":"NO_CODE_CHANGE","_number":5,"created":"2021-08-06 10:48:04.000000000","uploader":{"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},"ref":"refs/changes/23/803423/5","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/swift","ref":"refs/changes/23/803423/5","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/swift refs/changes/23/803423/5 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/swift refs/changes/23/803423/5 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/swift refs/changes/23/803423/5 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/swift refs/changes/23/803423/5"}}},"commit":{"parents":[{"commit":"b3c5a9a9e01aafa2a0819636b2eb20640cf7b529","subject":"Merge \"s3api: Fix (async) multi-delete of MPUs\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/swift/commit/b3c5a9a9e01aafa2a0819636b2eb20640cf7b529"}]}],"author":{"name":"Matthew Oliver","email":"matt@oliver.net.au","date":"2021-08-04 07:07:19.000000000","tz":600},"committer":{"name":"Alistair Coles","email":"alistairncoles@gmail.com","date":"2021-08-06 08:47:37.000000000","tz":60},"subject":"container-server: return objects of a given policy","message":"container-server: return objects of a given policy\n\nThere is a tight coupling between a root container and it\u0027s shards the\nshards hold the object metadata for the root container, so are really an\nextension of the root.\nWhen we PUT objects in to a root container, it\u0027ll redirect them, with\nthe root\u0027s policy, to the shards. And the shards are happy to take them,\neven if the shard\u0027s policy is different to the root\u0027s.\nBut when it comes to GETs, the root redirects the GET onto it\u0027s shards\nwhom currently wont respond with objects (which they probably took) because\nthey are of a different policy. When getting objects from the container\nserver, the policy used is always the brokers policy.\n\nThis patch corrects this behaviour by allowing the ability to override\nthe policy index to use. If the request to the container server\ncontains an \u0027X-Backend-Storage-Policy-Index\u0027 header it\u0027ll be used\ninstead of the policy index stored in the broker.\n\nThis patch adds the root containers policy as this header in the proxy\ncontainer controllers `_get_from_shards` method which is used by the proxy\nto redirect a GET to a root to it\u0027s shards.\n\nOn a plus side this new container policy API allows us a way to check\ncontainers for object listing is other policies. So might come in handy\nfor OPs/SREs.\n\nChange-Id: I026b699fc5f0fba619cf524093632d67ca38d32f\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/swift/commit/1e13e3fd0321f13bca2dd0be5bef5c917cb65e7e"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/swift/commit/1e13e3fd0321f13bca2dd0be5bef5c917cb65e7e"}]},"branch":"refs/heads/master"},"63c6933f5d938bea47c365407ab7cea0b015b033":{"kind":"REWORK","_number":6,"created":"2021-08-09 05:12:41.000000000","uploader":{"_account_id":7233,"name":"Matthew Oliver","email":"matt@oliver.net.au","username":"mattoliverau"},"ref":"refs/changes/23/803423/6","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/swift","ref":"refs/changes/23/803423/6","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/swift refs/changes/23/803423/6 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/swift refs/changes/23/803423/6 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/swift refs/changes/23/803423/6 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/swift refs/changes/23/803423/6"}}},"commit":{"parents":[{"commit":"b3c5a9a9e01aafa2a0819636b2eb20640cf7b529","subject":"Merge \"s3api: Fix (async) multi-delete of MPUs\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/swift/commit/b3c5a9a9e01aafa2a0819636b2eb20640cf7b529"}]}],"author":{"name":"Matthew Oliver","email":"matt@oliver.net.au","date":"2021-08-04 07:07:19.000000000","tz":600},"committer":{"name":"Matthew Oliver","email":"matt@oliver.net.au","date":"2021-08-09 04:30:19.000000000","tz":600},"subject":"container-server: return objects of a given policy","message":"container-server: return objects of a given policy\n\nThere is a tight coupling between a root container and it\u0027s shards the\nshards hold the object metadata for the root container, so are really an\nextension of the root.\nWhen we PUT objects in to a root container, it\u0027ll redirect them, with\nthe root\u0027s policy, to the shards. And the shards are happy to take them,\neven if the shard\u0027s policy is different to the root\u0027s.\nBut when it comes to GETs, the root redirects the GET onto it\u0027s shards\nwhom currently wont respond with objects (which they probably took) because\nthey are of a different policy. When getting objects from the container\nserver, the policy used is always the brokers policy.\n\nThis patch corrects this behaviour by allowing the ability to override\nthe policy index to use. If the request to the container server\ncontains an \u0027X-Backend-Storage-Policy-Index\u0027 header it\u0027ll be used\ninstead of the policy index stored in the broker.\n\nThis patch adds the root containers policy as this header in the proxy\ncontainer controllers `_get_from_shards` method which is used by the proxy\nto redirect a GET to a root to it\u0027s shards.\n\nOn a plus side this new container policy API allows us a way to check\ncontainers for object listing is other policies. So might come in handy\nfor OPs/SREs.\n\nCo-Authored-By: Alistair Coles \u003calistairncoles@gmail.com\u003e\nChange-Id: I026b699fc5f0fba619cf524093632d67ca38d32f\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/swift/commit/63c6933f5d938bea47c365407ab7cea0b015b033"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/swift/commit/63c6933f5d938bea47c365407ab7cea0b015b033"}]},"branch":"refs/heads/master"},"5f519176ed2bad028cb272203886ae2c758cef93":{"kind":"REWORK","_number":7,"created":"2021-08-12 01:52:23.000000000","uploader":{"_account_id":7233,"name":"Matthew Oliver","email":"matt@oliver.net.au","username":"mattoliverau"},"ref":"refs/changes/23/803423/7","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/swift","ref":"refs/changes/23/803423/7","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/swift refs/changes/23/803423/7 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/swift refs/changes/23/803423/7 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/swift refs/changes/23/803423/7 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/swift refs/changes/23/803423/7"}}},"commit":{"parents":[{"commit":"b3c5a9a9e01aafa2a0819636b2eb20640cf7b529","subject":"Merge \"s3api: Fix (async) multi-delete of MPUs\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/swift/commit/b3c5a9a9e01aafa2a0819636b2eb20640cf7b529"}]}],"author":{"name":"Matthew Oliver","email":"matt@oliver.net.au","date":"2021-08-04 07:07:19.000000000","tz":600},"committer":{"name":"Matthew Oliver","email":"matt@oliver.net.au","date":"2021-08-12 01:48:17.000000000","tz":600},"subject":"container-server: return objects of a given policy","message":"container-server: return objects of a given policy\n\nThere is a tight coupling between a root container and it\u0027s shards the\nshards hold the object metadata for the root container, so are really an\nextension of the root.\nWhen we PUT objects in to a root container, it\u0027ll redirect them, with\nthe root\u0027s policy, to the shards. And the shards are happy to take them,\neven if the shard\u0027s policy is different to the root\u0027s.\nBut when it comes to GETs, the root redirects the GET onto it\u0027s shards\nwhom currently wont respond with objects (which they probably took) because\nthey are of a different policy. When getting objects from the container\nserver, the policy used is always the brokers policy.\n\nThis patch corrects this behaviour by allowing the ability to override\nthe policy index to use. If the request to the container server\ncontains an \u0027X-Backend-Storage-Policy-Index\u0027 header it\u0027ll be used\ninstead of the policy index stored in the broker.\n\nThis patch adds the root containers policy as this header in the proxy\ncontainer controllers `_get_from_shards` method which is used by the proxy\nto redirect a GET to a root to it\u0027s shards.\n\nFurther, a new header has been added, if the container resp contains an\n`X-Backend-Record-Type: object` header, then it means the response if a\nresponse with objects in it. In this case this patch also adds a\n`X-Backend-Record-Storage-Policy-Index  header so the spi of the given\nobjects is known, as X-Backend-Storage-Policy-Index in the response\n_always_ represents the spi of the container itself.\n\nOn a plus side this new container policy API allows us a way to check\ncontainers for object listing is other policies. So might come in handy\nfor OPs/SREs.\n\nCo-Authored-By: Alistair Coles \u003calistairncoles@gmail.com\u003e\nChange-Id: I026b699fc5f0fba619cf524093632d67ca38d32f\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/swift/commit/5f519176ed2bad028cb272203886ae2c758cef93"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/swift/commit/5f519176ed2bad028cb272203886ae2c758cef93"}]},"branch":"refs/heads/master"},"91e1f3047bd616db61a75e8947bc26b7365a3552":{"kind":"REWORK","_number":8,"created":"2021-08-12 12:57:20.000000000","uploader":{"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},"ref":"refs/changes/23/803423/8","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/swift","ref":"refs/changes/23/803423/8","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/swift refs/changes/23/803423/8 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/swift refs/changes/23/803423/8 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/swift refs/changes/23/803423/8 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/swift refs/changes/23/803423/8"}}},"commit":{"parents":[{"commit":"b3c5a9a9e01aafa2a0819636b2eb20640cf7b529","subject":"Merge \"s3api: Fix (async) multi-delete of MPUs\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/swift/commit/b3c5a9a9e01aafa2a0819636b2eb20640cf7b529"}]}],"author":{"name":"Matthew Oliver","email":"matt@oliver.net.au","date":"2021-08-04 07:07:19.000000000","tz":600},"committer":{"name":"Alistair Coles","email":"alistairncoles@gmail.com","date":"2021-08-12 10:19:10.000000000","tz":60},"subject":"container-server: return objects of a given policy","message":"container-server: return objects of a given policy\n\nThere is a tight coupling between a root container and its shards: the\nshards hold the object metadata for the root container, so are really\nan extension of the root.  When we PUT objects in to a root container,\nit\u0027ll redirect them, with the root\u0027s policy, to the shards. And the\nshards are happy to take them, even if the shard\u0027s policy is different\nto the root\u0027s.  But when it comes to GETs, the root redirects the GET\nonto it\u0027s shards whom currently wont respond with objects (which they\nprobably took) because they are of a different policy. Currently, when\ngetting objects from the container server, the policy used is always\nthe broker\u0027s policy.\n\nThis patch corrects this behaviour by allowing the ability to override\nthe policy index to use. If the request to the container server\ncontains an \u0027X-Backend-Storage-Policy-Index\u0027 header it\u0027ll be used\ninstead of the policy index stored in the broker.\n\nThis patch adds the root container\u0027s policy as this header in the\nproxy container controller\u0027s `_get_from_shards` method which is used\nby the proxy to redirect a GET to a root to its shards.\n\nFurther, a new backend response header has been added. If the\ncontainer response contains an `X-Backend-Record-Type: object` header,\nthen it means the response is a response with objects in it. In this\ncase this patch also adds a `X-Backend-Record-Storage-Policy-Index`\nheader so the policy index of the given objects is known, as\nX-Backend-Storage-Policy-Index in the response _always_ represents the\npolicy index of the container itself.\n\nOn a plus side this new container policy API allows us a way to check\ncontainers for object listing is other policies. So might come in handy\nfor OPs/SREs.\n\nCo-Authored-By: Alistair Coles \u003calistairncoles@gmail.com\u003e\nChange-Id: I026b699fc5f0fba619cf524093632d67ca38d32f\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/swift/commit/91e1f3047bd616db61a75e8947bc26b7365a3552"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/swift/commit/91e1f3047bd616db61a75e8947bc26b7365a3552"}]},"branch":"refs/heads/master"},"89945a6ae9929eeae91946f212751e6482e52565":{"kind":"REWORK","_number":9,"created":"2021-08-16 10:35:00.000000000","uploader":{"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},"ref":"refs/changes/23/803423/9","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/swift","ref":"refs/changes/23/803423/9","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/swift refs/changes/23/803423/9 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/swift refs/changes/23/803423/9 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/swift refs/changes/23/803423/9 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/swift refs/changes/23/803423/9"}}},"commit":{"parents":[{"commit":"b3c5a9a9e01aafa2a0819636b2eb20640cf7b529","subject":"Merge \"s3api: Fix (async) multi-delete of MPUs\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/swift/commit/b3c5a9a9e01aafa2a0819636b2eb20640cf7b529"}]}],"author":{"name":"Matthew Oliver","email":"matt@oliver.net.au","date":"2021-08-04 07:07:19.000000000","tz":600},"committer":{"name":"Alistair Coles","email":"alistairncoles@gmail.com","date":"2021-08-16 10:34:44.000000000","tz":60},"subject":"container-server: return objects of a given policy","message":"container-server: return objects of a given policy\n\nThere is a tight coupling between a root container and its shards: the\nshards hold the object metadata for the root container, so are really\nan extension of the root.  When we PUT objects in to a root container,\nit\u0027ll redirect them, with the root\u0027s policy, to the shards. And the\nshards are happy to take them, even if the shard\u0027s policy is different\nto the root\u0027s.  But when it comes to GETs, the root redirects the GET\nonto it\u0027s shards whom currently wont respond with objects (which they\nprobably took) because they are of a different policy. Currently, when\ngetting objects from the container server, the policy used is always\nthe broker\u0027s policy.\n\nThis patch corrects this behaviour by allowing the ability to override\nthe policy index to use. If the request to the container server\ncontains an \u0027X-Backend-Storage-Policy-Index\u0027 header it\u0027ll be used\ninstead of the policy index stored in the broker.\n\nThis patch adds the root container\u0027s policy as this header in the\nproxy container controller\u0027s `_get_from_shards` method which is used\nby the proxy to redirect a GET to a root to its shards.\n\nFurther, a new backend response header has been added. If the\ncontainer response contains an `X-Backend-Record-Type: object` header,\nthen it means the response is a response with objects in it. In this\ncase this patch also adds a `X-Backend-Record-Storage-Policy-Index`\nheader so the policy index of the given objects is known, as\nX-Backend-Storage-Policy-Index in the response _always_ represents the\npolicy index of the container itself.\n\nOn a plus side this new container policy API allows us a way to check\ncontainers for object listing is other policies. So might come in handy\nfor OPs/SREs.\n\nCo-Authored-By: Alistair Coles \u003calistairncoles@gmail.com\u003e\nChange-Id: I026b699fc5f0fba619cf524093632d67ca38d32f\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/swift/commit/89945a6ae9929eeae91946f212751e6482e52565"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/swift/commit/89945a6ae9929eeae91946f212751e6482e52565"}]},"branch":"refs/heads/master"},"56510ab3c31070a1b8720a33d26c8a220e6ce5c9":{"kind":"NO_CHANGE","_number":10,"created":"2021-08-16 10:57:06.000000000","uploader":{"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},"ref":"refs/changes/23/803423/10","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/swift","ref":"refs/changes/23/803423/10","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/swift refs/changes/23/803423/10 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/swift refs/changes/23/803423/10 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/swift refs/changes/23/803423/10 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/swift refs/changes/23/803423/10"}}},"commit":{"parents":[{"commit":"b3c5a9a9e01aafa2a0819636b2eb20640cf7b529","subject":"Merge \"s3api: Fix (async) multi-delete of MPUs\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/swift/commit/b3c5a9a9e01aafa2a0819636b2eb20640cf7b529"}]}],"author":{"name":"Matthew Oliver","email":"matt@oliver.net.au","date":"2021-08-04 07:07:19.000000000","tz":600},"committer":{"name":"Alistair Coles","email":"alistairncoles@gmail.com","date":"2021-08-16 10:56:54.000000000","tz":60},"subject":"container-server: return objects of a given policy","message":"container-server: return objects of a given policy\n\nThere is a tight coupling between a root container and its shards: the\nshards hold the object metadata for the root container, so are really\nan extension of the root.  When we PUT objects in to a root container,\nit\u0027ll redirect them, with the root\u0027s policy, to the shards. And the\nshards are happy to take them, even if the shard\u0027s policy is different\nto the root\u0027s.  But when it comes to GETs, the root redirects the GET\nonto it\u0027s shards whom currently wont respond with objects (which they\nprobably took) because they are of a different policy. Currently, when\ngetting objects from the container server, the policy used is always\nthe broker\u0027s policy.\n\nThis patch corrects this behaviour by allowing the ability to override\nthe policy index to use. If the request to the container server\ncontains an \u0027X-Backend-Storage-Policy-Index\u0027 header it\u0027ll be used\ninstead of the policy index stored in the broker.\n\nThis patch adds the root container\u0027s policy as this header in the\nproxy container controller\u0027s `_get_from_shards` method which is used\nby the proxy to redirect a GET to a root to its shards.\n\nFurther, a new backend response header has been added. If the\ncontainer response contains an `X-Backend-Record-Type: object` header,\nthen it means the response is a response with objects in it. In this\ncase this patch also adds a `X-Backend-Record-Storage-Policy-Index`\nheader so the policy index of the given objects is known, as\nX-Backend-Storage-Policy-Index in the response _always_ represents the\npolicy index of the container itself.\n\nOn a plus side this new container policy API allows us a way to check\ncontainers for object listing is other policies. So might come in handy\nfor OPs/SREs.\n\nCo-Authored-By: Alistair Coles \u003calistairncoles@gmail.com\u003e\nChange-Id: I026b699fc5f0fba619cf524093632d67ca38d32f\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/swift/commit/56510ab3c31070a1b8720a33d26c8a220e6ce5c9"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/swift/commit/56510ab3c31070a1b8720a33d26c8a220e6ce5c9"}]},"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":{}}}]}