)]}'
{"id":"openstack%2Fswift~759502","triplet_id":"openstack%2Fswift~master~I6bdab4210a42fcec84d7c288a8437d5ab81318c4","project":"openstack/swift","branch":"master","topic":"p-memcache-error-limit-strategy","hashtags":[],"change_id":"I6bdab4210a42fcec84d7c288a8437d5ab81318c4","subject":"memcache: change strategy for always available servers","status":"ABANDONED","created":"2020-10-23 18:07:18.000000000","updated":"2020-11-06 10:40:59.000000000","total_comment_count":10,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"5b1feeac308b7ec6b070d32963e36bd20d66db55","_number":759502,"virtual_id_number":759502,"owner":{"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},"actions":{},"labels":{"Verified":{"disliked":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},{"tag":"autogenerated:zuul:check","value":-1,"date":"2020-10-26 13:37:00.000000000","permitted_voting_range":{"min":-2,"max":2},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"_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":"","value":-1,"default_value":0,"optional":true},"Code-Review":{"all":[{"value":0,"permitted_voting_range":{"min":-2,"max":2},"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"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":{"all":[{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_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":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":"2020-10-23 18:35:15.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":"2020-10-26 13:37:00.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"}],"messages":[{"id":"87569c2295d42ccc08feec8764882f6840427c9f","author":{"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},"date":"2020-10-23 18:07:18.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"3098ae20657bf75242db95e3f51fd8ef83a2c453","author":{"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"},"date":"2020-10-23 18:35:15.000000000","message":"Patch Set 1: Code-Review+1\n\n(4 comments)\n\nYeah, this seems reasonable and likely better.","accounts_in_message":[],"_revision_number":1},{"id":"bd2cf589c9282cac276c6000e723628673e4bc7f","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-10-23 20:19:24.000000000","message":"Patch Set 1: Verified-1\n\n(2 comments)\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/65525427e1994c889011f28a3a8c7617 : SUCCESS in 9m 29s\n- grenade https://zuul.opendev.org/t/openstack/build/2d0d50eaf4074f0f8c0d83bdba0f6d13 : FAILURE in 58m 28s\n- tempest-integrated-object-storage https://zuul.opendev.org/t/openstack/build/6ccde465957a499d957dcaf961451d50 : SUCCESS in 49m 51s\n- swift-build-image https://zuul.opendev.org/t/openstack/build/5811b0541e8f46c6ae4c732c3dcf8a88 : SUCCESS in 9m 29s (non-voting)\n- swift-build-image-py3 https://zuul.opendev.org/t/openstack/build/74ac08c484b3415994dd0238accfacfe : SUCCESS in 10m 56s (non-voting)\n- swift-tox-py27 https://zuul.opendev.org/t/openstack/build/f67b4056a7d8477c9f3f001d79711ab1 : SUCCESS in 20m 10s\n- swift-tox-py36 https://zuul.opendev.org/t/openstack/build/f5c0bd67433745b29025424729585490 : SUCCESS in 17m 33s\n- swift-tox-py37 https://zuul.opendev.org/t/openstack/build/5128304782034344aac8d756d5d48efe : SUCCESS in 20m 42s\n- swift-tox-py38 https://zuul.opendev.org/t/openstack/build/f9440a00127d46078a18adcbee9e7973 : SUCCESS in 18m 15s\n- swift-tox-func-py27 https://zuul.opendev.org/t/openstack/build/238712ca580e41858958d01563cec91e : SUCCESS in 17m 16s\n- swift-tox-func-encryption-py27 https://zuul.opendev.org/t/openstack/build/8610401d74174bdea46ed1b6b3df0f74 : SUCCESS in 20m 40s\n- swift-tox-func-ec-py27 https://zuul.opendev.org/t/openstack/build/4c57c5328d934d03ad295bd9e7c7d384 : SUCCESS in 19m 09s\n- swift-tox-func-py37 https://zuul.opendev.org/t/openstack/build/3db83653ff494b6b84a767d81275b453 : SUCCESS in 23m 44s\n- swift-tox-func-encryption-py37 https://zuul.opendev.org/t/openstack/build/41db641efde64d12b183a787b7ca0fb0 : SUCCESS in 18m 25s\n- swift-tox-func-ec-py37 https://zuul.opendev.org/t/openstack/build/8c7ee99a61c842dbbc0009b2d55c93e2 : SUCCESS in 31m 04s\n- swift-tox-func-s3api-ceph-s3tests-tempauth https://zuul.opendev.org/t/openstack/build/54d5718763474aef94659c89d765caaf : SUCCESS in 20m 34s (non-voting)\n- swift-probetests-centos-7 https://zuul.opendev.org/t/openstack/build/3258a34e0341459dbb1b6312362bf04d : SUCCESS in 46m 09s\n- swift-dsvm-functional https://zuul.opendev.org/t/openstack/build/1bf1db1c35f7462c9ea11de0133030c0 : SUCCESS in 50m 37s\n- swift-dsvm-functional-ipv6 https://zuul.opendev.org/t/openstack/build/2fc9f56828ac4700bf26d7482ef3b532 : SUCCESS in 46m 06s\n- swift-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/3e1fdb2ca6c944c29d42a5572cbd73af : SUCCESS in 15m 00s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/02fd1d45694046fc8c75d258581cff60 : FAILURE in 7m 39s\n- swift-multinode-rolling-upgrade https://zuul.opendev.org/t/openstack/build/5e20c0d657ed4e44b5dba60d1234115a : SUCCESS in 21m 41s\n- tempest-ipv6-only https://zuul.opendev.org/t/openstack/build/5ba593a1b5504a648482d8dfc1dbf474 : SUCCESS in 1h 01m 54s","accounts_in_message":[],"_revision_number":1},{"id":"2953d186834b43a52a14e01e76fe69239c41e22f","author":{"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},"date":"2020-10-26 12:28:04.000000000","message":"Patch Set 1:\n\n(4 comments)","accounts_in_message":[],"_revision_number":1},{"id":"4cb604d8eb12f38c77ce505df1838278deccb121","author":{"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},"date":"2020-10-26 12:28:42.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"99d4a0bcf761d7dd071d4dee5bee9004c25d424a","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-10-26 13:37:00.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/49d5f393fda14613b26bd07eb9ae09be : SUCCESS in 9m 16s\n- grenade https://zuul.opendev.org/t/openstack/build/5d7e1b0e73f94c66bd8b443642ff252e : FAILURE in 53m 35s\n- tempest-integrated-object-storage https://zuul.opendev.org/t/openstack/build/7c4771eff12f4f888964770c1d2d045c : SUCCESS in 52m 42s\n- swift-build-image https://zuul.opendev.org/t/openstack/build/908326b52b474d0ca73f609467033e4c : SUCCESS in 8m 04s (non-voting)\n- swift-build-image-py3 https://zuul.opendev.org/t/openstack/build/94557645229b40caa97fefe0b6af5958 : SUCCESS in 9m 09s (non-voting)\n- swift-tox-py27 https://zuul.opendev.org/t/openstack/build/981db0188b6841c1b377266ce9d7e850 : SUCCESS in 13m 24s\n- swift-tox-py36 https://zuul.opendev.org/t/openstack/build/6088beb2f5cf4933b2e7493d6ea477e9 : SUCCESS in 14m 09s\n- swift-tox-py37 https://zuul.opendev.org/t/openstack/build/30cb066f1e0d4ea08a40900297e9b1c6 : SUCCESS in 12m 43s\n- swift-tox-py38 https://zuul.opendev.org/t/openstack/build/b9113d76f53f480a921f591a7590e895 : SUCCESS in 13m 00s\n- swift-tox-func-py27 https://zuul.opendev.org/t/openstack/build/9134a86a7b254ea8b6be2c5a76b47e59 : SUCCESS in 17m 51s\n- swift-tox-func-encryption-py27 https://zuul.opendev.org/t/openstack/build/a77ac7b0c859403890f00db12bb2383b : SUCCESS in 17m 31s\n- swift-tox-func-ec-py27 https://zuul.opendev.org/t/openstack/build/1183724b792a4db9b15b4b6645b45eba : SUCCESS in 18m 00s\n- swift-tox-func-py37 https://zuul.opendev.org/t/openstack/build/8272fdad3d6748c5952118d64fea67f2 : SUCCESS in 16m 05s\n- swift-tox-func-encryption-py37 https://zuul.opendev.org/t/openstack/build/57914f36559f4c6f9d1631d909140173 : SUCCESS in 22m 47s\n- swift-tox-func-ec-py37 https://zuul.opendev.org/t/openstack/build/e314ebd458784d06820348cfeaa0406d : SUCCESS in 40m 43s\n- swift-tox-func-s3api-ceph-s3tests-tempauth https://zuul.opendev.org/t/openstack/build/e13e8e3cf3524f1da5a2c85209f6ac9f : SUCCESS in 22m 11s (non-voting)\n- swift-probetests-centos-7 https://zuul.opendev.org/t/openstack/build/b29dc659b6e24aa4ba4bf119a137727a : SUCCESS in 49m 56s\n- swift-dsvm-functional https://zuul.opendev.org/t/openstack/build/0cff59900a1c43f3a2b62e0a9244f1e5 : SUCCESS in 44m 51s\n- swift-dsvm-functional-ipv6 https://zuul.opendev.org/t/openstack/build/aad30bc85d4948738748ef235421d418 : SUCCESS in 41m 57s\n- swift-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/e31c9df024814171b06cb1aefeac814a : SUCCESS in 13m 12s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/034459e4ae854d4a9a6c902c0f8078f1 : SUCCESS in 6m 11s\n- swift-multinode-rolling-upgrade https://zuul.opendev.org/t/openstack/build/3ccaea4532594a7facfcc646229f42d6 : SUCCESS in 19m 31s\n- tempest-ipv6-only https://zuul.opendev.org/t/openstack/build/c156ba7f08e0442cbc270753dd210123 : SUCCESS in 1h 00m 50s","accounts_in_message":[],"_revision_number":2},{"id":"2fcf646fcd9227f9fa9a053dbcdb884cf08b125a","author":{"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},"date":"2020-11-06 10:40:59.000000000","message":"Abandoned\n\nsuperseded by https://review.opendev.org/#/c/761029/","accounts_in_message":[],"_revision_number":2}],"current_revision_number":2,"current_revision":"0c771bae70dc791697588c5286bd2976530754b5","revisions":{"50552a7cc9de362ddc77e095db2defcc3d856c6f":{"kind":"REWORK","_number":1,"created":"2020-10-23 18:07:18.000000000","uploader":{"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},"ref":"refs/changes/02/759502/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/swift","ref":"refs/changes/02/759502/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/swift refs/changes/02/759502/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/swift refs/changes/02/759502/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/swift refs/changes/02/759502/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/swift refs/changes/02/759502/1"}}},"commit":{"parents":[{"commit":"686fde31b9e779c8f8a0e2ad232725a0317060b4","subject":"memcache: make minimum number of servers configurable","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/swift/commit/686fde31b9e779c8f8a0e2ad232725a0317060b4"}]}],"author":{"name":"Alistair Coles","email":"alistairncoles@gmail.com","date":"2020-10-23 12:03:47.000000000","tz":60},"committer":{"name":"Alistair Coles","email":"alistairncoles@gmail.com","date":"2020-10-23 18:03:48.000000000","tz":60},"subject":"memcache: change strategy for always available servers","message":"memcache: change strategy for always available servers\n\nPreviously, when all but a configurable minimum number of memcache\nservers have been error limited, remaining servers are never error\nlimited. Consequently the server that has most recently been observed\nto fail remains available. With this change, one of the remaining\nservers may be error limited, but if that happens the server that is\nclosest in time to being available again (i.e. the server that was\nleast recently error limited) is made prematurely available in order\nto maintain the required minimum number of available servers.\n\nChange-Id: I6bdab4210a42fcec84d7c288a8437d5ab81318c4\nRelated-Change: Ibeaf6bdcdfb32addb277858f7095c9f0035fd5f3\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/swift/commit/50552a7cc9de362ddc77e095db2defcc3d856c6f"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/swift/commit/50552a7cc9de362ddc77e095db2defcc3d856c6f"}]},"branch":"refs/heads/master"},"0c771bae70dc791697588c5286bd2976530754b5":{"kind":"REWORK","_number":2,"created":"2020-10-26 12:28:42.000000000","uploader":{"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},"ref":"refs/changes/02/759502/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/swift","ref":"refs/changes/02/759502/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/swift refs/changes/02/759502/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/swift refs/changes/02/759502/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/swift refs/changes/02/759502/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/swift refs/changes/02/759502/2"}}},"commit":{"parents":[{"commit":"e10b4e2457061ffb3235dae978ae9182d045bf55","subject":"memcache: make minimum number of servers configurable","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/swift/commit/e10b4e2457061ffb3235dae978ae9182d045bf55"}]}],"author":{"name":"Alistair Coles","email":"alistairncoles@gmail.com","date":"2020-10-23 12:03:47.000000000","tz":60},"committer":{"name":"Alistair Coles","email":"alistairncoles@gmail.com","date":"2020-10-26 12:25:47.000000000","tz":0},"subject":"memcache: change strategy for always available servers","message":"memcache: change strategy for always available servers\n\nPreviously, when all but a configurable minimum number of memcache\nservers have been error limited, remaining servers are never error\nlimited. Consequently the server that has most recently been observed\nto fail remains available. With this change, one of the remaining\nservers may be error limited, but if that happens the server that is\nclosest in time to being available again (i.e. the server that was\nleast recently error limited) is made prematurely available in order\nto maintain the required minimum number of available servers.\n\nChange-Id: I6bdab4210a42fcec84d7c288a8437d5ab81318c4\nRelated-Change: Ibeaf6bdcdfb32addb277858f7095c9f0035fd5f3\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/swift/commit/0c771bae70dc791697588c5286bd2976530754b5"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/swift/commit/0c771bae70dc791697588c5286bd2976530754b5"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}
