)]}'
{"id":"openstack%2Fswift~241978","triplet_id":"openstack%2Fswift~master~I02967ca8d7c790595e5ee551581196aa64552eea","project":"openstack/swift","branch":"master","topic":"multiple-storage-policy-names","hashtags":[],"change_id":"I02967ca8d7c790595e5ee551581196aa64552eea","subject":"Add support for storage policies to have more than one name","status":"MERGED","created":"2015-11-05 10:38:06.000000000","updated":"2015-12-23 00:45:59.000000000","submitted":"2015-12-23 00:45:59.000000000","submitter":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"total_comment_count":18,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"680f82610681ad66169700505debfab41fa2328e","_number":241978,"virtual_id_number":241978,"owner":{"_account_id":18978,"name":"Catherine Northcott","email":"catherine@northcott.nz","username":"Zyric"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"value":0,"_account_id":7233,"name":"Matthew Oliver","email":"matt@oliver.net.au","username":"mattoliverau"},{"value":0,"_account_id":18978,"name":"Catherine Northcott","email":"catherine@northcott.nz","username":"Zyric"},{"value":2,"date":"2015-12-23 00:45:59.000000000","post_submit":true,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":10068,"name":"Welcome, new contributor!","username":"welcome-message"},{"value":0,"_account_id":12279,"name":"Janie Richling","email":"jrichli@us.ibm.com","username":"jrichli"},{"value":0,"_account_id":9625,"name":"Thiago da Silva","email":"thiagodasilva@gmail.com","username":"thiago"},{"value":0,"_account_id":330,"name":"John Dickinson","email":"me@not.mn","username":"notmyname"},{"value":0,"_account_id":7002,"name":"Yatin Kumbhare","email":"yatinkumbhare@gmail.com","username":"yatin.kumbhare"},{"value":0,"_account_id":13052,"name":"SwiftStack Cluster CI","email":"openstack-ci@swiftstack.com","username":"swiftstack-cluster-ci","tags":["SERVICE_USER"]},{"value":0,"_account_id":2622,"name":"Samuel Merritt","email":"spam+launchpad@andcheese.org","username":"torgomatic"},{"value":0,"_account_id":18015,"name":"Peter Lisák","email":"peter.lisak@gmail.com","username":"peter.lisak"}],"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":7233,"name":"Matthew Oliver","email":"matt@oliver.net.au","username":"mattoliverau"},"all":[{"value":2,"date":"2015-12-22 23:43:57.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":7233,"name":"Matthew Oliver","email":"matt@oliver.net.au","username":"mattoliverau"},{"value":0,"_account_id":18978,"name":"Catherine Northcott","email":"catherine@northcott.nz","username":"Zyric"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":10068,"name":"Welcome, new contributor!","username":"welcome-message"},{"value":0,"_account_id":12279,"name":"Janie Richling","email":"jrichli@us.ibm.com","username":"jrichli"},{"value":0,"_account_id":9625,"name":"Thiago da Silva","email":"thiagodasilva@gmail.com","username":"thiago"},{"value":2,"date":"2015-12-18 23:24:18.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":330,"name":"John Dickinson","email":"me@not.mn","username":"notmyname"},{"value":0,"_account_id":7002,"name":"Yatin Kumbhare","email":"yatinkumbhare@gmail.com","username":"yatin.kumbhare"},{"value":0,"_account_id":13052,"name":"SwiftStack Cluster CI","email":"openstack-ci@swiftstack.com","username":"swiftstack-cluster-ci","tags":["SERVICE_USER"]},{"value":0,"_account_id":2622,"name":"Samuel Merritt","email":"spam+launchpad@andcheese.org","username":"torgomatic"},{"value":0,"_account_id":18015,"name":"Peter Lisák","email":"peter.lisak@gmail.com","username":"peter.lisak"}],"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":7233,"name":"Matthew Oliver","email":"matt@oliver.net.au","username":"mattoliverau"},"all":[{"value":1,"date":"2015-12-22 23:43:57.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":7233,"name":"Matthew Oliver","email":"matt@oliver.net.au","username":"mattoliverau"},{"value":0,"_account_id":18978,"name":"Catherine Northcott","email":"catherine@northcott.nz","username":"Zyric"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":10068,"name":"Welcome, new contributor!","username":"welcome-message"},{"value":0,"_account_id":12279,"name":"Janie Richling","email":"jrichli@us.ibm.com","username":"jrichli"},{"value":0,"_account_id":9625,"name":"Thiago da Silva","email":"thiagodasilva@gmail.com","username":"thiago"},{"value":0,"_account_id":330,"name":"John Dickinson","email":"me@not.mn","username":"notmyname"},{"value":0,"_account_id":7002,"name":"Yatin Kumbhare","email":"yatinkumbhare@gmail.com","username":"yatin.kumbhare"},{"value":0,"_account_id":13052,"name":"SwiftStack Cluster CI","email":"openstack-ci@swiftstack.com","username":"swiftstack-cluster-ci","tags":["SERVICE_USER"]},{"value":0,"_account_id":2622,"name":"Samuel Merritt","email":"spam+launchpad@andcheese.org","username":"torgomatic"},{"value":0,"_account_id":18015,"name":"Peter Lisák","email":"peter.lisak@gmail.com","username":"peter.lisak"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":330,"name":"John Dickinson","email":"me@not.mn","username":"notmyname"},{"_account_id":2622,"name":"Samuel Merritt","email":"spam+launchpad@andcheese.org","username":"torgomatic"},{"_account_id":7002,"name":"Yatin Kumbhare","email":"yatinkumbhare@gmail.com","username":"yatin.kumbhare"},{"_account_id":7233,"name":"Matthew Oliver","email":"matt@oliver.net.au","username":"mattoliverau"},{"_account_id":9625,"name":"Thiago da Silva","email":"thiagodasilva@gmail.com","username":"thiago"},{"_account_id":10068,"name":"Welcome, new contributor!","username":"welcome-message"},{"_account_id":12279,"name":"Janie Richling","email":"jrichli@us.ibm.com","username":"jrichli"},{"_account_id":13052,"name":"SwiftStack Cluster CI","email":"openstack-ci@swiftstack.com","username":"swiftstack-cluster-ci","tags":["SERVICE_USER"]},{"_account_id":18015,"name":"Peter Lisák","email":"peter.lisak@gmail.com","username":"peter.lisak"},{"_account_id":18978,"name":"Catherine Northcott","email":"catherine@northcott.nz","username":"Zyric"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2015-11-05 10:38:16.000000000","updated_by":{"_account_id":10068,"name":"Welcome, new contributor!","username":"welcome-message"},"reviewer":{"_account_id":10068,"name":"Welcome, new contributor!","username":"welcome-message"},"state":"REVIEWER"},{"updated":"2015-11-05 10:58:42.000000000","updated_by":{"_account_id":18015,"name":"Peter Lisák","email":"peter.lisak@gmail.com","username":"peter.lisak"},"reviewer":{"_account_id":18015,"name":"Peter Lisák","email":"peter.lisak@gmail.com","username":"peter.lisak"},"state":"REVIEWER"},{"updated":"2015-11-05 22:56:41.000000000","updated_by":{"_account_id":2622,"name":"Samuel Merritt","email":"spam+launchpad@andcheese.org","username":"torgomatic"},"reviewer":{"_account_id":2622,"name":"Samuel Merritt","email":"spam+launchpad@andcheese.org","username":"torgomatic"},"state":"REVIEWER"},{"updated":"2015-11-24 05:09:25.000000000","updated_by":{"_account_id":7002,"name":"Yatin Kumbhare","email":"yatinkumbhare@gmail.com","username":"yatin.kumbhare"},"reviewer":{"_account_id":7002,"name":"Yatin Kumbhare","email":"yatinkumbhare@gmail.com","username":"yatin.kumbhare"},"state":"REVIEWER"},{"updated":"2015-11-25 04:09:00.000000000","updated_by":{"_account_id":12279,"name":"Janie Richling","email":"jrichli@us.ibm.com","username":"jrichli"},"reviewer":{"_account_id":12279,"name":"Janie Richling","email":"jrichli@us.ibm.com","username":"jrichli"},"state":"REVIEWER"},{"updated":"2015-12-09 01:45:29.000000000","updated_by":{"_account_id":13052,"name":"SwiftStack Cluster CI","email":"openstack-ci@swiftstack.com","username":"swiftstack-cluster-ci","tags":["SERVICE_USER"]},"reviewer":{"_account_id":13052,"name":"SwiftStack Cluster CI","email":"openstack-ci@swiftstack.com","username":"swiftstack-cluster-ci","tags":["SERVICE_USER"]},"state":"REVIEWER"},{"updated":"2015-12-09 14:59:22.000000000","updated_by":{"_account_id":9625,"name":"Thiago da Silva","email":"thiagodasilva@gmail.com","username":"thiago"},"reviewer":{"_account_id":9625,"name":"Thiago da Silva","email":"thiagodasilva@gmail.com","username":"thiago"},"state":"REVIEWER"},{"updated":"2015-12-18 23:24:18.000000000","updated_by":{"_account_id":330,"name":"John Dickinson","email":"me@not.mn","username":"notmyname"},"reviewer":{"_account_id":330,"name":"John Dickinson","email":"me@not.mn","username":"notmyname"},"state":"REVIEWER"},{"updated":"2015-12-22 23:43:57.000000000","updated_by":{"_account_id":7233,"name":"Matthew Oliver","email":"matt@oliver.net.au","username":"mattoliverau"},"reviewer":{"_account_id":7233,"name":"Matthew Oliver","email":"matt@oliver.net.au","username":"mattoliverau"},"state":"REVIEWER"},{"updated":"2015-12-23 00:45:59.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"}],"messages":[{"id":"e56329f433a248dc2d644bea9575fe9cd9f84c09","author":{"_account_id":18978,"name":"Catherine Northcott","email":"catherine@northcott.nz","username":"Zyric"},"date":"2015-11-05 10:38:06.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"bf2748ba7ae703257db325a9f9be450e9d90dc9b","author":{"_account_id":10068,"name":"Welcome, new contributor!","username":"welcome-message"},"date":"2015-11-05 10:38:16.000000000","message":"Patch Set 1:\n\nThank you for your first contribution to OpenStack.\n\n    Your patch will now be tested automatically by OpenStack testing frameworks\n    and once the automatic tests pass, it will be reviewed by other friendly\n    developers. They will give you feedback and may require you to refine it.\n\n    People seldom get their patch approved on the first try, so don\u0027t be\n    concerned if requested to make corrections. Feel free to modify your patch\n    and resubmit a new change-set.\n\n    Patches usually take 3 to 7 days to be reviewed so be patient and be\n    available on IRC to ask and answer questions about your work. Also it\n    takes generally at least a couple of weeks for cores to get around to\n    reviewing code. The more you participate in the community the more\n    rewarding it is for you. You may also notice that the more you get to know\n    people and get to be known, the faster your patches will be reviewed and\n    eventually approved. Get to know others and become known by doing code\n    reviews: anybody can do it, and it\u0027s a great way to learn the code base.\n\n    Thanks again for supporting OpenStack, we look forward to working with you.\n\n    IRC: https://wiki.openstack.org/wiki/IRC\n    Workflow: http://docs.openstack.org/infra/manual/developers.html\n    Commit Messages: https://wiki.openstack.org/wiki/GitCommitMessages","accounts_in_message":[],"_revision_number":1},{"id":"d69e05fad85804fb3f9eb42912f5a546de6d9e50","author":{"_account_id":18015,"name":"Peter Lisák","email":"peter.lisak@gmail.com","username":"peter.lisak"},"date":"2015-11-05 10:58:42.000000000","message":"Patch Set 1: Code-Review-1\n\nAt first sight I see some pep8 issues (trailing whitespace, ...).\n\nYou should run flake8 to check it.","accounts_in_message":[],"_revision_number":1},{"id":"66f136308f7a9454430a68112a3fe736c0b0bd9e","author":{"_account_id":13052,"name":"SwiftStack Cluster CI","email":"openstack-ci@swiftstack.com","username":"swiftstack-cluster-ci","tags":["SERVICE_USER"]},"date":"2015-11-05 11:25:26.000000000","message":"Patch Set 1:\n\nBuild succeeded \n\n* cluster-swift-tox-func https://8b86aea46fb38e6450f2-0e5f4c086da474abc1df58826577db2f.ssl.cf1.rackcdn.com/241978/2971/ : SUCCESS\n\n* cluster-swift-tox-func-ec https://8b86aea46fb38e6450f2-0e5f4c086da474abc1df58826577db2f.ssl.cf1.rackcdn.com/241978/1015/ : SUCCESS\n\n* cluster-swift-ssbench https://8b86aea46fb38e6450f2-0e5f4c086da474abc1df58826577db2f.ssl.cf1.rackcdn.com/241978/1612/ssbench : SUCCESS\n\n* vm-saio-probe https://8b86aea46fb38e6450f2-0e5f4c086da474abc1df58826577db2f.ssl.cf1.rackcdn.com/241978/728/probetests/ : SUCCESS","accounts_in_message":[],"_revision_number":1},{"id":"652e07f9a29b9063b7c8bf867fd9c5cb35e1b7e8","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-11-05 11:53:07.000000000","message":"Patch Set 1: Verified-1\n\nBuild failed (check pipeline).  For information on how to proceed, see http://docs.openstack.org/infra/manual/developers.html#automated-testing\n\n- gate-swift-pep8 http://logs.openstack.org/78/241978/1/check/gate-swift-pep8/1392349/ : FAILURE in 1m 50s\n- gate-swift-docs http://docs-draft.openstack.org/78/241978/1/check/gate-swift-docs/5b9f67b//doc/build/html/ : SUCCESS in 4m 11s\n- gate-swift-python27 http://logs.openstack.org/78/241978/1/check/gate-swift-python27/de65b7b/ : SUCCESS in 4m 31s\n- gate-swift-tox-func http://logs.openstack.org/78/241978/1/check/gate-swift-tox-func/a301de1/ : SUCCESS in 5m 19s\n- gate-swift-python34 http://logs.openstack.org/78/241978/1/check/gate-swift-python34/62781ee/ : FAILURE in 2m 54s (non-voting)\n- gate-tempest-dsvm-full http://logs.openstack.org/78/241978/1/check/gate-tempest-dsvm-full/e86678c/ : SUCCESS in 58m 32s\n- gate-grenade-dsvm http://logs.openstack.org/78/241978/1/check/gate-grenade-dsvm/b235f0a/ : SUCCESS in 56m 10s\n- gate-swift-dsvm-functional http://logs.openstack.org/78/241978/1/check/gate-swift-dsvm-functional/9abf5d1/ : SUCCESS in 33m 38s","accounts_in_message":[],"_revision_number":1},{"id":"30429e363efd7991977e7d0598c88856a2e615d2","author":{"_account_id":330,"name":"John Dickinson","email":"me@not.mn","username":"notmyname"},"date":"2015-11-05 20:10:08.000000000","message":"Patch Set 1: Code-Review-1\n\n(4 comments)\n\nLooks like a great start!\n\nAside from the pep8 errors that need to be cleaned up, I left a few more comments and questions.\n\nOne thing that should be added is that the new names (aliases) should be exposed int he /info endpoint. Look around line 219 in swift/proxy/server.py for where this is set.\n\nIs there a reason to distinguish between the first one in the list and any \"aliases\"? They\u0027re all equally valid to be used, but the one that\u0027s first in the list is the one that\u0027s reported when you get container info or account info. How do you think that should be documented or changed for the end-user?\n\nI\u0027m not sure if any existing container reconciler probe tests should be updated.","accounts_in_message":[],"_revision_number":1},{"id":"8b66d7a6752f6480c3b32872c6d7d3cb24678ae3","author":{"_account_id":2622,"name":"Samuel Merritt","email":"spam+launchpad@andcheese.org","username":"torgomatic"},"date":"2015-11-05 22:56:41.000000000","message":"Patch Set 1: Code-Review-1\n\n(1 comment)\n\nTo add on, I\u0027d prefer to see things called something like \u0027primary_name\u0027 and \u0027aliases\u0027; right now, a storage policy has `.name` and `.names`, and I know I\u0027ll screw that up about half the time.","accounts_in_message":[],"_revision_number":1},{"id":"f87e5c0b1e82774312f2717118106fd929655cfd","author":{"_account_id":18978,"name":"Catherine Northcott","email":"catherine@northcott.nz","username":"Zyric"},"date":"2015-11-10 03:01:39.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"d9bfb01909e0278f082f545277779378dc316bf7","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-11-10 04:12:38.000000000","message":"Patch Set 2: Verified-1\n\nBuild failed (check pipeline).  For information on how to proceed, see http://docs.openstack.org/infra/manual/developers.html#automated-testing\n\n- gate-swift-pep8 http://logs.openstack.org/78/241978/2/check/gate-swift-pep8/8c30117/ : FAILURE in 1m 51s\n- gate-swift-docs http://docs-draft.openstack.org/78/241978/2/check/gate-swift-docs/e85e814//doc/build/html/ : SUCCESS in 3m 28s\n- gate-swift-python27 http://logs.openstack.org/78/241978/2/check/gate-swift-python27/86270d1/ : FAILURE in 5m 06s\n- gate-swift-tox-func http://logs.openstack.org/78/241978/2/check/gate-swift-tox-func/8c5e675/ : SUCCESS in 5m 32s\n- gate-swift-python34 http://logs.openstack.org/78/241978/2/check/gate-swift-python34/08c654b/ : FAILURE in 4m 15s (non-voting)\n- gate-tempest-dsvm-full http://logs.openstack.org/78/241978/2/check/gate-tempest-dsvm-full/863a703/ : SUCCESS in 1h 10m 47s\n- gate-grenade-dsvm http://logs.openstack.org/78/241978/2/check/gate-grenade-dsvm/558d6b1/ : SUCCESS in 1h 05m 28s\n- gate-swift-dsvm-functional http://logs.openstack.org/78/241978/2/check/gate-swift-dsvm-functional/efe5aa8/ : SUCCESS in 38m 12s","accounts_in_message":[],"_revision_number":2},{"id":"051d595e3b86346e3bc25b4e4679bc414ebc5834","author":{"_account_id":13052,"name":"SwiftStack Cluster CI","email":"openstack-ci@swiftstack.com","username":"swiftstack-cluster-ci","tags":["SERVICE_USER"]},"date":"2015-11-10 04:17:10.000000000","message":"Patch Set 2:\n\nBuild succeeded \n\n* cluster-swift-tox-func https://8b86aea46fb38e6450f2-0e5f4c086da474abc1df58826577db2f.ssl.cf1.rackcdn.com/241978/3033/ : SUCCESS\n\n* cluster-swift-tox-func-ec https://8b86aea46fb38e6450f2-0e5f4c086da474abc1df58826577db2f.ssl.cf1.rackcdn.com/241978/1077/ : SUCCESS\n\n* cluster-swift-ssbench https://8b86aea46fb38e6450f2-0e5f4c086da474abc1df58826577db2f.ssl.cf1.rackcdn.com/241978/1674/ssbench : SUCCESS\n\n* vm-saio-probe https://8b86aea46fb38e6450f2-0e5f4c086da474abc1df58826577db2f.ssl.cf1.rackcdn.com/241978/790/probetests/ : SUCCESS","accounts_in_message":[],"_revision_number":2},{"id":"a4cc6a681f39761557f1f08ff6d08ee3e312fe32","author":{"_account_id":330,"name":"John Dickinson","email":"me@not.mn","username":"notmyname"},"date":"2015-11-10 05:29:47.000000000","message":"Patch Set 2: Code-Review-1\n\nPlease fix the pep8 errors in your next patch. You can use `tox -epep8` to run the tests and see what fails. It\u0027s impossible for your patch to land without passing the pep8 checks.\n\nThe same with the unit test errors. You\u0027ve messed up the name property (getter) and you need to validate it is returning what you expect. Adding a test for it wouldn\u0027t go amiss.\n\nPlease make sure you run the tests and that they pass locally in your SAIO before you push a patch.\n\nYou should probably also modify one or more of the existing tests to use a storage policy configured to have aliases. This will help find any places where names and aliases are used incorrectly and prevent any regressions there.\n\nOn the output of a /info request, I\u0027d prefer to see \"name\" and \"aliases\" instead of everything in the \"name\" attribute. If a user looks at that field as a string and sends the whole value as a storage policy, they\u0027ll rightly get an error. We shouldn\u0027t hand the users a foot-gun like this. Instead, if \"name\" is the primary name and the aliases are in a new attribute, it becomes more clear what should be used. Implementing this will more than likely mean you need to rewrite the name property.","accounts_in_message":[],"_revision_number":2},{"id":"1b74a7097e3a187c2b8194df73dec8d204cf9356","author":{"_account_id":18978,"name":"Catherine Northcott","email":"catherine@northcott.nz","username":"Zyric"},"date":"2015-11-10 09:23:06.000000000","message":"Uploaded patch set 3.","accounts_in_message":[],"_revision_number":3},{"id":"d99203e2471150ec59d92216d651019f9304cfea","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-11-10 10:35:27.000000000","message":"Patch Set 3: Verified-1\n\nBuild failed (check pipeline).  For information on how to proceed, see http://docs.openstack.org/infra/manual/developers.html#automated-testing\n\n- gate-swift-pep8 http://logs.openstack.org/78/241978/3/check/gate-swift-pep8/f9ba614/ : SUCCESS in 1m 55s\n- gate-swift-docs http://docs-draft.openstack.org/78/241978/3/check/gate-swift-docs/a164b13//doc/build/html/ : SUCCESS in 3m 22s\n- gate-swift-python27 http://logs.openstack.org/78/241978/3/check/gate-swift-python27/8b606e1/ : SUCCESS in 4m 52s\n- gate-swift-tox-func http://logs.openstack.org/78/241978/3/check/gate-swift-tox-func/e0994e6/ : FAILURE in 2m 08s\n- gate-swift-python34 http://logs.openstack.org/78/241978/3/check/gate-swift-python34/d317a53/ : FAILURE in 3m 21s (non-voting)\n- gate-tempest-dsvm-full http://logs.openstack.org/78/241978/3/check/gate-tempest-dsvm-full/fb0711f/ : SUCCESS in 56m 01s\n- gate-grenade-dsvm http://logs.openstack.org/78/241978/3/check/gate-grenade-dsvm/4a90e60/ : SUCCESS in 1h 09m 21s\n- gate-swift-dsvm-functional http://logs.openstack.org/78/241978/3/check/gate-swift-dsvm-functional/f137c39/ : SUCCESS in 26m 48s","accounts_in_message":[],"_revision_number":3},{"id":"b598aa3a169e1f4bed8011c9f73637c27d8ed256","author":{"_account_id":13052,"name":"SwiftStack Cluster CI","email":"openstack-ci@swiftstack.com","username":"swiftstack-cluster-ci","tags":["SERVICE_USER"]},"date":"2015-11-10 10:53:38.000000000","message":"Patch Set 3:\n\nBuild succeeded \n\n* cluster-swift-tox-func https://8b86aea46fb38e6450f2-0e5f4c086da474abc1df58826577db2f.ssl.cf1.rackcdn.com/241978/3040/ : SUCCESS\n\n* cluster-swift-tox-func-ec https://8b86aea46fb38e6450f2-0e5f4c086da474abc1df58826577db2f.ssl.cf1.rackcdn.com/241978/1084/ : SUCCESS\n\n* cluster-swift-ssbench https://8b86aea46fb38e6450f2-0e5f4c086da474abc1df58826577db2f.ssl.cf1.rackcdn.com/241978/1681/ssbench : SUCCESS\n\n* vm-saio-probe https://8b86aea46fb38e6450f2-0e5f4c086da474abc1df58826577db2f.ssl.cf1.rackcdn.com/241978/801/probetests/ : SUCCESS","accounts_in_message":[],"_revision_number":3},{"id":"6dae3b9f779795f27e89039354e33636ad800f34","author":{"_account_id":330,"name":"John Dickinson","email":"me@not.mn","username":"notmyname"},"date":"2015-11-18 17:08:06.000000000","message":"Patch Set 3:\n\nAre you still looking into the test failure here?","accounts_in_message":[],"_revision_number":3},{"id":"7ab35f194f6fd8b772dfb36cbb2db0b8af755224","author":{"_account_id":330,"name":"John Dickinson","email":"me@not.mn","username":"notmyname"},"date":"2015-11-24 00:15:22.000000000","message":"Patch Set 3:\n\ngate issues can be duplicated with\n\n`SWIFT_TEST_IN_PROCESS\u003dtrue SWIFT_TEST_IN_PROCESS_CONF_DIR\u003d/tmp tox -efunc`","accounts_in_message":[],"_revision_number":3},{"id":"49a920358d64c271a9d0ec0cdb4d4180f4c1c572","author":{"_account_id":18978,"name":"Catherine Northcott","email":"catherine@northcott.nz","username":"Zyric"},"date":"2015-11-24 06:35:05.000000000","message":"Uploaded patch set 4.","accounts_in_message":[],"_revision_number":4},{"id":"92a2097f11962da966e1f24ed8e1902bb967cf34","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-11-24 07:21:37.000000000","message":"Patch Set 4: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-swift-pep8 http://logs.openstack.org/78/241978/4/check/gate-swift-pep8/55d9754/ : SUCCESS in 2m 02s\n- gate-swift-docs http://docs-draft.openstack.org/78/241978/4/check/gate-swift-docs/e2374e9//doc/build/html/ : SUCCESS in 3m 23s\n- gate-swift-python27 http://logs.openstack.org/78/241978/4/check/gate-swift-python27/8169402/ : SUCCESS in 4m 37s\n- gate-swift-tox-func http://logs.openstack.org/78/241978/4/check/gate-swift-tox-func/87f3e1a/ : SUCCESS in 6m 50s\n- gate-swift-python34 http://logs.openstack.org/78/241978/4/check/gate-swift-python34/a8127e3/ : FAILURE in 2m 54s (non-voting)\n- gate-tempest-dsvm-full http://logs.openstack.org/78/241978/4/check/gate-tempest-dsvm-full/117844a/ : SUCCESS in 43m 41s\n- gate-grenade-dsvm http://logs.openstack.org/78/241978/4/check/gate-grenade-dsvm/ffa39cd/ : SUCCESS in 40m 49s\n- gate-swift-dsvm-functional http://logs.openstack.org/78/241978/4/check/gate-swift-dsvm-functional/892a00c/ : SUCCESS in 27m 08s","accounts_in_message":[],"_revision_number":4},{"id":"82b52ef664d2b5447072a110f656c4fa2c021679","author":{"_account_id":13052,"name":"SwiftStack Cluster CI","email":"openstack-ci@swiftstack.com","username":"swiftstack-cluster-ci","tags":["SERVICE_USER"]},"date":"2015-11-24 07:22:16.000000000","message":"Patch Set 4:\n\nBuild Failed \n\n* cluster-swift-tox-func https://8b86aea46fb38e6450f2-0e5f4c086da474abc1df58826577db2f.ssl.cf1.rackcdn.com/241978/3167/ : FAILURE\n\n* cluster-swift-tox-func-ec https://8b86aea46fb38e6450f2-0e5f4c086da474abc1df58826577db2f.ssl.cf1.rackcdn.com/241978/1211/ : FAILURE\n\n* cluster-swift-ssbench https://8b86aea46fb38e6450f2-0e5f4c086da474abc1df58826577db2f.ssl.cf1.rackcdn.com/241978/1808/ssbench : SUCCESS\n\n* vm-saio-probe https://8b86aea46fb38e6450f2-0e5f4c086da474abc1df58826577db2f.ssl.cf1.rackcdn.com/241978/928/probetests/ : SUCCESS","accounts_in_message":[],"_revision_number":4},{"id":"d966e1f99262cc59033275833c84b948fdca017f","author":{"_account_id":330,"name":"John Dickinson","email":"me@not.mn","username":"notmyname"},"date":"2015-11-24 19:10:01.000000000","message":"Patch Set 4:\n\n(3 comments)\n\ncouple of inline comments.\n\nThis patch set doesn\u0027t pass aliases on to the super class in the EC policy constructor so EC policies can\u0027t have aliases. I\u0027ll push a fix over this one in a few minutes.","accounts_in_message":[],"_revision_number":4},{"id":"87f1d5b1fa1edc3a2cfbf1130c750dfe5ce6ae05","author":{"_account_id":330,"name":"John Dickinson","email":"me@not.mn","username":"notmyname"},"date":"2015-11-24 19:10:25.000000000","message":"Patch Set 4: Code-Review-1","accounts_in_message":[],"_revision_number":4},{"id":"b697e52f7feb66b332a38ced16604c5b6a5f5f55","author":{"_account_id":330,"name":"John Dickinson","email":"me@not.mn","username":"notmyname"},"date":"2015-11-24 19:24:51.000000000","message":"Uploaded patch set 5.","accounts_in_message":[],"_revision_number":5},{"id":"459859d8941a855ea72c8e5aa7783c48c2ee5e64","author":{"_account_id":330,"name":"John Dickinson","email":"me@not.mn","username":"notmyname"},"date":"2015-11-24 19:25:46.000000000","message":"Patch Set 5: Code-Review+2\n\nno major changes except for plumbing this through to EC policies","accounts_in_message":[],"_revision_number":5},{"id":"cf5213e7b0242dd204d00ecc3b60e58a20f37edd","author":{"_account_id":12279,"name":"Janie Richling","email":"jrichli@us.ibm.com","username":"jrichli"},"date":"2015-11-24 20:10:46.000000000","message":"Patch Set 5: Code-Review+1","accounts_in_message":[],"_revision_number":5},{"id":"f1413100130273e127ea7a5191af51d2b70a1337","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-11-24 20:29:28.000000000","message":"Patch Set 5: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-swift-pep8 http://logs.openstack.org/78/241978/5/check/gate-swift-pep8/b40de60/ : SUCCESS in 1m 52s\n- gate-swift-docs http://docs-draft.openstack.org/78/241978/5/check/gate-swift-docs/f3f33b5//doc/build/html/ : SUCCESS in 3m 16s\n- gate-swift-python27 http://logs.openstack.org/78/241978/5/check/gate-swift-python27/6624c79/ : SUCCESS in 5m 13s\n- gate-swift-tox-func http://logs.openstack.org/78/241978/5/check/gate-swift-tox-func/09de2bf/ : SUCCESS in 6m 47s\n- gate-swift-python34 http://logs.openstack.org/78/241978/5/check/gate-swift-python34/9c174ab/ : FAILURE in 3m 12s (non-voting)\n- gate-tempest-dsvm-full http://logs.openstack.org/78/241978/5/check/gate-tempest-dsvm-full/9992b4c/ : SUCCESS in 52m 25s\n- gate-grenade-dsvm http://logs.openstack.org/78/241978/5/check/gate-grenade-dsvm/3000b6a/ : SUCCESS in 42m 46s\n- gate-swift-dsvm-functional http://logs.openstack.org/78/241978/5/check/gate-swift-dsvm-functional/eb5ec00/ : SUCCESS in 26m 34s","accounts_in_message":[],"_revision_number":5},{"id":"209f5cc37ad5441ae6565fd17534972823b4a868","author":{"_account_id":13052,"name":"SwiftStack Cluster CI","email":"openstack-ci@swiftstack.com","username":"swiftstack-cluster-ci","tags":["SERVICE_USER"]},"date":"2015-11-24 22:40:59.000000000","message":"Patch Set 5:\n\nBuild Failed \n\n* cluster-swift-tox-func https://8b86aea46fb38e6450f2-0e5f4c086da474abc1df58826577db2f.ssl.cf1.rackcdn.com/241978/3179/ : FAILURE\n\n* cluster-swift-tox-func-ec https://8b86aea46fb38e6450f2-0e5f4c086da474abc1df58826577db2f.ssl.cf1.rackcdn.com/241978/1223/ : FAILURE\n\n* cluster-swift-ssbench https://8b86aea46fb38e6450f2-0e5f4c086da474abc1df58826577db2f.ssl.cf1.rackcdn.com/241978/1820/ssbench : SUCCESS\n\n* vm-saio-probe https://8b86aea46fb38e6450f2-0e5f4c086da474abc1df58826577db2f.ssl.cf1.rackcdn.com/241978/940/probetests/ : SUCCESS","accounts_in_message":[],"_revision_number":5},{"id":"b49a935f2bdf512e14fb5f68227080513835d8bc","author":{"_account_id":7233,"name":"Matthew Oliver","email":"matt@oliver.net.au","username":"mattoliverau"},"date":"2015-11-25 02:36:43.000000000","message":"Patch Set 5: Code-Review-1\n\n(1 comment)\n\nAwesome work on this Catherine, just have a question about storing the .aliases string, but then never updating or using it. And why not just use the alias_list to handle aliases seeing as you use it to handle .name anyway. (see inline).\n\nI know John already +2\u0027ed it, and if people feel I\u0027m just nitpicking then I\u0027m happy to change my score and merge it :)","accounts_in_message":[],"_revision_number":5},{"id":"4da42351e5ce3575f6f6c247bce793f10b56f497","author":{"_account_id":12279,"name":"Janie Richling","email":"jrichli@us.ibm.com","username":"jrichli"},"date":"2015-11-25 04:09:00.000000000","message":"Patch Set 5: -Code-Review\n\n(1 comment)","accounts_in_message":[],"_revision_number":5},{"id":"65568a8289f19a09930eb011a8835e88bcaa300c","author":{"_account_id":18978,"name":"Catherine Northcott","email":"catherine@northcott.nz","username":"Zyric"},"date":"2015-11-26 06:16:57.000000000","message":"Patch Set 5:\n\nMatthew: That\u0027s a good point and I like your suggestion of using only one data structure for all name related functions. I\u0027ll make the changes and submit a revision shortly.\n\nJanie: The .name attribute is supported mainly for the current codebase which uses .name extensively. The .name attribute acts as a default primary name value since a storage policy must have at least one name but won\u0027t necessarily have more names / any aliases. I\u0027m now attempting to keep the name data itself contained using the aliases_list attribute, the reason for having an .aliases too is to display the aliases_list in a human-readable format for info calls. So hopefully, while we have a range of methods for accessing and manipulating name data, all of it is actually only in .aliases_list","accounts_in_message":[],"_revision_number":5},{"id":"e7d5ba056f0e80729bf87f67dd411362b7dd0bfa","author":{"_account_id":18978,"name":"Catherine Northcott","email":"catherine@northcott.nz","username":"Zyric"},"date":"2015-11-26 06:18:08.000000000","message":"Uploaded patch set 6.","accounts_in_message":[],"_revision_number":6},{"id":"cdbe7fe858087c1f680eb128b01e28736900bcf1","author":{"_account_id":13052,"name":"SwiftStack Cluster CI","email":"openstack-ci@swiftstack.com","username":"swiftstack-cluster-ci","tags":["SERVICE_USER"]},"date":"2015-11-26 07:19:40.000000000","message":"Patch Set 6:\n\nBuild Failed \n\n* cluster-swift-tox-func https://8b86aea46fb38e6450f2-0e5f4c086da474abc1df58826577db2f.ssl.cf1.rackcdn.com/241978/3203/ : FAILURE\n\n* cluster-swift-tox-func-ec https://8b86aea46fb38e6450f2-0e5f4c086da474abc1df58826577db2f.ssl.cf1.rackcdn.com/241978/1247/ : FAILURE\n\n* cluster-swift-ssbench https://8b86aea46fb38e6450f2-0e5f4c086da474abc1df58826577db2f.ssl.cf1.rackcdn.com/241978/1844/ssbench : SUCCESS\n\n* vm-saio-probe https://8b86aea46fb38e6450f2-0e5f4c086da474abc1df58826577db2f.ssl.cf1.rackcdn.com/241978/964/probetests/ : SUCCESS","accounts_in_message":[],"_revision_number":6},{"id":"f871a45c78411e8d73b91179a58842b424aa7eda","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-11-26 07:42:57.000000000","message":"Patch Set 6: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-swift-pep8 http://logs.openstack.org/78/241978/6/check/gate-swift-pep8/99047b9/ : SUCCESS in 1m 50s\n- gate-swift-docs http://docs-draft.openstack.org/78/241978/6/check/gate-swift-docs/3a05354//doc/build/html/ : SUCCESS in 3m 32s\n- gate-swift-python27 http://logs.openstack.org/78/241978/6/check/gate-swift-python27/7d5e98c/ : SUCCESS in 6m 56s\n- gate-swift-tox-func http://logs.openstack.org/78/241978/6/check/gate-swift-tox-func/bdfb8e5/ : SUCCESS in 5m 13s\n- gate-swift-python34 http://logs.openstack.org/78/241978/6/check/gate-swift-python34/927455d/ : FAILURE in 3m 01s (non-voting)\n- gate-tempest-dsvm-full http://logs.openstack.org/78/241978/6/check/gate-tempest-dsvm-full/e4b53f2/ : SUCCESS in 1h 21m 20s\n- gate-grenade-dsvm http://logs.openstack.org/78/241978/6/check/gate-grenade-dsvm/65fd386/ : SUCCESS in 45m 05s\n- gate-swift-dsvm-functional http://logs.openstack.org/78/241978/6/check/gate-swift-dsvm-functional/e2b7d7c/ : SUCCESS in 33m 57s","accounts_in_message":[],"_revision_number":6},{"id":"eeb6981db8487d70afac1e6ebd4981ed0e87bc9b","author":{"_account_id":330,"name":"John Dickinson","email":"me@not.mn","username":"notmyname"},"date":"2015-12-08 04:32:32.000000000","message":"Patch Set 6:\n\nrecheck","accounts_in_message":[],"_revision_number":6},{"id":"c0533c4eda9879f23513ce6f6b40a3747bbea01f","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-12-08 05:43:57.000000000","message":"Patch Set 6:\n\nBuild succeeded (check pipeline).\n\n- gate-swift-pep8 http://logs.openstack.org/78/241978/6/check/gate-swift-pep8/94e2832/ : SUCCESS in 1m 16s\n- gate-swift-docs http://docs-draft.openstack.org/78/241978/6/check/gate-swift-docs/4a50d90//doc/build/html/ : SUCCESS in 2m 19s\n- gate-swift-python27 http://logs.openstack.org/78/241978/6/check/gate-swift-python27/be3fc73/ : SUCCESS in 3m 39s\n- gate-swift-tox-func http://logs.openstack.org/78/241978/6/check/gate-swift-tox-func/cdccbba/ : SUCCESS in 4m 56s\n- gate-swift-python34 http://logs.openstack.org/78/241978/6/check/gate-swift-python34/48d1988/ : SUCCESS in 2m 16s (non-voting)\n- gate-tempest-dsvm-full http://logs.openstack.org/78/241978/6/check/gate-tempest-dsvm-full/d7bb317/ : SUCCESS in 1h 00m 45s\n- gate-grenade-dsvm http://logs.openstack.org/78/241978/6/check/gate-grenade-dsvm/ce01f6a/ : SUCCESS in 56m 16s\n- gate-swift-dsvm-functional http://logs.openstack.org/78/241978/6/check/gate-swift-dsvm-functional/e7274c5/ : SUCCESS in 35m 34s","accounts_in_message":[],"_revision_number":6},{"id":"546d20c8638b327c32601a340181f84db071b5f7","author":{"_account_id":330,"name":"John Dickinson","email":"me@not.mn","username":"notmyname"},"date":"2015-12-08 05:57:45.000000000","message":"Patch Set 6: Code-Review-1\n\n(3 comments)\n\nLooks like good progress. Matt\u0027s suggestions were, of course, right on. Just a little more cleanup, I think.","accounts_in_message":[],"_revision_number":6},{"id":"d16996cc3e9c59aed48ef2d0653f9faad89282e2","author":{"_account_id":18978,"name":"Catherine Northcott","email":"catherine@northcott.nz","username":"Zyric"},"date":"2015-12-08 23:06:07.000000000","message":"Uploaded patch set 7.","accounts_in_message":[],"_revision_number":7},{"id":"ad42a4f7a02d142503b9965e8bde761f73dc6380","author":{"_account_id":18978,"name":"Catherine Northcott","email":"catherine@northcott.nz","username":"Zyric"},"date":"2015-12-08 23:07:11.000000000","message":"Patch Set 7:\n\nJohn: I\u0027ve changed the break statement to a continue one in the policy constructor - no good reason to halt processing there. Because of how the automated testing works I\u0027ve opted to ignore re-occurrences of the \u0027main name\u0027 set under aliases on construction but this won\u0027t be ignored if attempting to add existing names via add_policy_alias(). There is a test for that case included in \"test_add_remove_names\" (adding an alias which already belongs to a policy to that policy). There wasn\u0027t a test for repeated aliases on-construction, though I\u0027ve added one now for thoroughness as well as a set of tests for multiple name configuration for EC Policies.","accounts_in_message":[],"_revision_number":7},{"id":"7b1a091c0b5e417bfd6d06376d79076847964ec4","author":{"_account_id":13052,"name":"SwiftStack Cluster CI","email":"openstack-ci@swiftstack.com","username":"swiftstack-cluster-ci","tags":["SERVICE_USER"]},"date":"2015-12-09 01:45:29.000000000","message":"Patch Set 7:\n\nBuild succeeded \n\n* vm-saio-probe https://8b86aea46fb38e6450f2-0e5f4c086da474abc1df58826577db2f.ssl.cf1.rackcdn.com/241978/1103/probetests/ : SUCCESS\n\n* cluster-swift-tox-func-ec https://8b86aea46fb38e6450f2-0e5f4c086da474abc1df58826577db2f.ssl.cf1.rackcdn.com/241978/1385/ : SUCCESS\n\n* cluster-swift-ssbench https://8b86aea46fb38e6450f2-0e5f4c086da474abc1df58826577db2f.ssl.cf1.rackcdn.com/241978/1981/ssbench : SUCCESS\n\n* cluster-swift-tox-func https://8b86aea46fb38e6450f2-0e5f4c086da474abc1df58826577db2f.ssl.cf1.rackcdn.com/241978/3346/ : SUCCESS","accounts_in_message":[],"_revision_number":7},{"id":"f6e753f064a9d2d669e4d9ae185e038625e47c13","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-12-09 02:47:34.000000000","message":"Patch Set 7: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-swift-pep8 http://logs.openstack.org/78/241978/7/check/gate-swift-pep8/8186cb8/ : SUCCESS in 1m 04s\n- gate-swift-docs http://docs-draft.openstack.org/78/241978/7/check/gate-swift-docs/710b1c4//doc/build/html/ : SUCCESS in 2m 20s\n- gate-swift-python27 http://logs.openstack.org/78/241978/7/check/gate-swift-python27/4ff23f2/ : SUCCESS in 4m 01s\n- gate-swift-python34 http://logs.openstack.org/78/241978/7/check/gate-swift-python34/5b96ee4/ : SUCCESS in 2m 08s\n- gate-swift-tox-func http://logs.openstack.org/78/241978/7/check/gate-swift-tox-func/6aebc26/ : SUCCESS in 5m 22s\n- gate-tempest-dsvm-full http://logs.openstack.org/78/241978/7/check/gate-tempest-dsvm-full/1c8e0b1/ : SUCCESS in 48m 57s\n- gate-grenade-dsvm http://logs.openstack.org/78/241978/7/check/gate-grenade-dsvm/f699226/ : SUCCESS in 49m 29s\n- gate-swift-dsvm-functional http://logs.openstack.org/78/241978/7/check/gate-swift-dsvm-functional/8a175ba/ : SUCCESS in 28m 04s","accounts_in_message":[],"_revision_number":7},{"id":"e592fdac0796957dfd1a43f0eab7deba8a392afe","author":{"_account_id":330,"name":"John Dickinson","email":"me@not.mn","username":"notmyname"},"date":"2015-12-09 04:16:26.000000000","message":"Patch Set 7: Code-Review+2\n\ngreat work!","accounts_in_message":[],"_revision_number":7},{"id":"5de405aaf2b3814f0fa78d134e76664644828e22","author":{"_account_id":9625,"name":"Thiago da Silva","email":"thiagodasilva@gmail.com","username":"thiago"},"date":"2015-12-09 14:59:22.000000000","message":"Patch Set 7: Code-Review-1\n\n(5 comments)\n\nnice patch! I put a -1 for now, but it\u0027s a soft -1 :)\nMostly I have questions regarding adding and removing alias. I was trying to understand the two new methods in the \n\nCollections class and how would they be used.\n\nThere\u0027s also some small suggested updates to the docs.\n\nIf there\u0027s no real issues with the questions I raised, I\u0027d be happy to change to a +2 and we could update the docs in a separate patch.","accounts_in_message":[],"_revision_number":7},{"id":"2dc874cf7f7db414d9228f06c86439954e638a9b","author":{"_account_id":9625,"name":"Thiago da Silva","email":"thiagodasilva@gmail.com","username":"thiago"},"date":"2015-12-09 15:12:50.000000000","message":"Patch Set 7:\n\none more question. I noticed that the user can use an alias when creating a container, but on a HEAD request the primary name is returned, it seems counter intuitive. is that the desired behavior?","accounts_in_message":[],"_revision_number":7},{"id":"07ece5617bc10a2207f29c5af3c3d0cad0f85a52","author":{"_account_id":330,"name":"John Dickinson","email":"me@not.mn","username":"notmyname"},"date":"2015-12-09 17:58:01.000000000","message":"Patch Set 7:\n\nThiago, I think it\u0027s expected that the reported policy for a container and the name used in the account per-policy aggregate stats are the primary name, even if initially set with an alias.","accounts_in_message":[],"_revision_number":7},{"id":"2b3dc330a5574e3573c6356cb4458aab8110f2a8","author":{"_account_id":330,"name":"John Dickinson","email":"me@not.mn","username":"notmyname"},"date":"2015-12-09 18:13:40.000000000","message":"Patch Set 7:\n\nThiago, the way I read the code, you\u0027re right that removing the name will rotate in the first alias as the new name. While that\u0027s (very!) impactful for end-users, it\u0027s not really different from changing the name before, right? We already caution against that. Do you think we need some more words written about that?\n\nCatherine, what do you think?","accounts_in_message":[],"_revision_number":7},{"id":"747c424f37e94b12d744d0ec0a8797ad4bcc14cd","author":{"_account_id":18978,"name":"Catherine Northcott","email":"catherine@northcott.nz","username":"Zyric"},"date":"2015-12-09 23:29:38.000000000","message":"Patch Set 7: Workflow-1\n\nThaigo, John: Yes, removing the first alias which acts as the primary/default name will switch out the default name to whichever is next available in the alias_list. This is the behaviour I expected, but definitely could be better documented to warn unsuspecting users. Would you expect this warning in the docstrings, the manual style docs, probably both?\n\nIt does bring into light a feature I had overlooked though - the ability to change the default name of a policy to a new name or a different alias belonging to the policy. I think this should be included. All names/aliases assigned to a policy should still resolve to said policy regardless of whether they are considered the primary name or not though.","accounts_in_message":[],"_revision_number":7},{"id":"956594747816cbe2a31ed0f34f49685fefaabe3e","author":{"_account_id":18978,"name":"Catherine Northcott","email":"catherine@northcott.nz","username":"Zyric"},"date":"2015-12-10 01:52:45.000000000","message":"Uploaded patch set 8.","accounts_in_message":[],"_revision_number":8},{"id":"7f12cee5a4d2efe81d9771d2396e26ff9a105975","author":{"_account_id":18978,"name":"Catherine Northcott","email":"catherine@northcott.nz","username":"Zyric"},"date":"2015-12-10 01:53:24.000000000","message":"Patch Set 8:\n\nAlight, I\u0027ve updated the docs with some hopefully clearer language and warnings about what will happen when removing removing primary/default names using the remove aliases function in the code.\n\nAlso added change_default_name(name) for policies and change_policy_default_name(index, name) for collections, which does exactly that. Tests have now been added for removing a default name, changing default name to a new name, changing default name to a bad new name, changing default name to an existing alias, and changing default name to a name/alias belonging to another policy.","accounts_in_message":[],"_revision_number":8},{"id":"9162d076fb3e0d2f94f481f7af1300d745057401","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-12-10 02:57:17.000000000","message":"Patch Set 8: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-swift-pep8 http://logs.openstack.org/78/241978/8/check/gate-swift-pep8/7e1032c/ : SUCCESS in 1m 10s\n- gate-swift-docs http://docs-draft.openstack.org/78/241978/8/check/gate-swift-docs/6783388//doc/build/html/ : SUCCESS in 2m 37s\n- gate-swift-python27 http://logs.openstack.org/78/241978/8/check/gate-swift-python27/0e036cf/ : SUCCESS in 3m 52s\n- gate-swift-python34 http://logs.openstack.org/78/241978/8/check/gate-swift-python34/3175068/ : SUCCESS in 1m 45s\n- gate-swift-tox-func http://logs.openstack.org/78/241978/8/check/gate-swift-tox-func/44f3737/ : SUCCESS in 6m 10s\n- gate-tempest-dsvm-full http://logs.openstack.org/78/241978/8/check/gate-tempest-dsvm-full/2697a68/ : SUCCESS in 54m 50s\n- gate-grenade-dsvm http://logs.openstack.org/78/241978/8/check/gate-grenade-dsvm/d524bd6/ : SUCCESS in 49m 49s\n- gate-swift-dsvm-functional http://logs.openstack.org/78/241978/8/check/gate-swift-dsvm-functional/f7d8368/ : SUCCESS in 40m 20s","accounts_in_message":[],"_revision_number":8},{"id":"776f303736c36c795e16355fa09516f52f832369","author":{"_account_id":13052,"name":"SwiftStack Cluster CI","email":"openstack-ci@swiftstack.com","username":"swiftstack-cluster-ci","tags":["SERVICE_USER"]},"date":"2015-12-10 08:26:44.000000000","message":"Patch Set 8:\n\nBuild Failed \n\n* vm-saio-probe https://8b86aea46fb38e6450f2-0e5f4c086da474abc1df58826577db2f.ssl.cf1.rackcdn.com/241978/1121/probetests/ : SUCCESS\n\n* cluster-swift-tox-func-ec https://8b86aea46fb38e6450f2-0e5f4c086da474abc1df58826577db2f.ssl.cf1.rackcdn.com/241978/1403/ : FAILURE\n\n* cluster-swift-ssbench https://8b86aea46fb38e6450f2-0e5f4c086da474abc1df58826577db2f.ssl.cf1.rackcdn.com/241978/1999/ssbench : FAILURE\n\n* cluster-swift-tox-func https://8b86aea46fb38e6450f2-0e5f4c086da474abc1df58826577db2f.ssl.cf1.rackcdn.com/241978/3364/ : FAILURE","accounts_in_message":[],"_revision_number":8},{"id":"d4daf4667276c4ad7f3780ef3e9713207e8e076c","author":{"_account_id":9625,"name":"Thiago da Silva","email":"thiagodasilva@gmail.com","username":"thiago"},"date":"2015-12-10 13:37:07.000000000","message":"Patch Set 8:\n\n@John, @Catherine: The new patch looks good. Still I\u0027m just struggling to understand the real use case for this patch. Having aliases sound nice, but if the user needs to know the alias and the primary name what\u0027s the point of having the alias?\n\nIf on a PUT I can use the alias, but when doing a GET or HEAD I get the primary name, it seems that we are missing something in terms of user experience.\n\nThoughts?","accounts_in_message":[],"_revision_number":8},{"id":"c4574fbd744db01f13343e6091f6b58e9a904658","author":{"_account_id":2622,"name":"Samuel Merritt","email":"spam+launchpad@andcheese.org","username":"torgomatic"},"date":"2015-12-10 21:31:06.000000000","message":"Patch Set 8:\n\nAs an administrator, I create a new storage cluster with 3 storage policies of various speeds, called \"bronze\", \"silver\", and \"glod\". Users start writing clients and putting stuff in with various requests.\n\nThen, someone pulls me aside and says \"hey, you misspelled \u0027gold\u0027; fix it.\"\n\nCurrently, I can\u0027t change \"glod\" to \"gold\" without breaking users\u0027 PUT requests, so it\u0027s a real pain.\n\nWith this patch, I can make the name \"gold\" and add \"glod\" as an alias, keeping more stuff working while fixing my typos.","accounts_in_message":[],"_revision_number":8},{"id":"8258128ee6f62b7512d408cc4fc6a5b0abe1562f","author":{"_account_id":330,"name":"John Dickinson","email":"me@not.mn","username":"notmyname"},"date":"2015-12-18 23:24:18.000000000","message":"Patch Set 8: Code-Review+2","accounts_in_message":[],"_revision_number":8},{"id":"13cc0ee18560fb5ad4fe9cf380c979fca928007a","author":{"_account_id":330,"name":"John Dickinson","email":"me@not.mn","username":"notmyname"},"date":"2015-12-18 23:26:41.000000000","message":"Patch Set 8:\n\nrecheck","accounts_in_message":[],"_revision_number":8},{"id":"9bcf874f1fe24f8759f23fc61cf3b89b11b62478","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-12-19 00:20:59.000000000","message":"Patch Set 8:\n\nBuild succeeded (check pipeline).\n\n- gate-swift-pep8 http://logs.openstack.org/78/241978/8/check/gate-swift-pep8/6f53e22/ : SUCCESS in 1m 23s\n- gate-swift-docs http://docs-draft.openstack.org/78/241978/8/check/gate-swift-docs/cb18ee2//doc/build/html/ : SUCCESS in 2m 23s\n- gate-swift-python27 http://logs.openstack.org/78/241978/8/check/gate-swift-python27/aad30df/ : SUCCESS in 3m 50s\n- gate-swift-python34 http://logs.openstack.org/78/241978/8/check/gate-swift-python34/e2cdfe9/ : SUCCESS in 1m 53s\n- gate-swift-tox-func http://logs.openstack.org/78/241978/8/check/gate-swift-tox-func/52f6813/ : SUCCESS in 4m 25s\n- gate-tempest-dsvm-full http://logs.openstack.org/78/241978/8/check/gate-tempest-dsvm-full/57ebe17/ : SUCCESS in 54m 07s\n- gate-grenade-dsvm http://logs.openstack.org/78/241978/8/check/gate-grenade-dsvm/9b0d938/ : SUCCESS in 40m 56s\n- gate-swift-dsvm-functional http://logs.openstack.org/78/241978/8/check/gate-swift-dsvm-functional/84178d3/ : SUCCESS in 36m 28s","accounts_in_message":[],"_revision_number":8},{"id":"eca675d1c17028d48176f60e11f639d4198140b5","author":{"_account_id":7233,"name":"Matthew Oliver","email":"matt@oliver.net.au","username":"mattoliverau"},"date":"2015-12-22 23:43:57.000000000","message":"Patch Set 8: Code-Review+2 Workflow+1\n\nAwesome work, Catherine. Awesome new feature and thanks for working on it!","accounts_in_message":[],"_revision_number":8},{"id":"b0a3876b2f2fb6217b786a084b169787c830e094","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-12-22 23:44:02.000000000","message":"Patch Set 8: -Verified\n\nStarting gate jobs.\nhttp://status.openstack.org/zuul/","accounts_in_message":[],"_revision_number":8},{"id":"7ef42d67205617e79a51b6940a1e7b738913dd0c","date":"2015-12-23 00:45:59.000000000","message":"Change has been successfully merged into the git repository by Jenkins","accounts_in_message":[],"_revision_number":8},{"id":"0c999c0b7b0812912847877c594fb499c0abc148","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-12-23 00:45:59.000000000","message":"Patch Set 8: Verified+2\n\nBuild succeeded (gate pipeline).\n\n- gate-swift-docs http://docs-draft.openstack.org/78/241978/8/gate/gate-swift-docs/5af336d//doc/build/html/ : SUCCESS in 2m 49s\n- gate-swift-pep8 http://logs.openstack.org/78/241978/8/gate/gate-swift-pep8/3d13d91/ : SUCCESS in 1m 20s\n- gate-swift-python27 http://logs.openstack.org/78/241978/8/gate/gate-swift-python27/1f57b6b/ : SUCCESS in 4m 43s\n- gate-swift-python34 http://logs.openstack.org/78/241978/8/gate/gate-swift-python34/6bece28/ : SUCCESS in 2m 24s\n- gate-swift-tox-func http://logs.openstack.org/78/241978/8/gate/gate-swift-tox-func/a8ff7c4/ : SUCCESS in 4m 20s\n- gate-tempest-dsvm-full http://logs.openstack.org/78/241978/8/gate/gate-tempest-dsvm-full/c100fb0/ : SUCCESS in 52m 50s\n- gate-grenade-dsvm http://logs.openstack.org/78/241978/8/gate/gate-grenade-dsvm/10391be/ : SUCCESS in 54m 19s\n- gate-swift-dsvm-functional http://logs.openstack.org/78/241978/8/gate/gate-swift-dsvm-functional/29086c0/ : SUCCESS in 25m 43s","accounts_in_message":[],"_revision_number":8}],"current_revision_number":8,"current_revision":"211758f8cb02298fe16e59bf2954a146c6b24b83","revisions":{"3eaa142976ce6fa7cbf68d0a006b9669cd2a798c":{"kind":"REWORK","_number":1,"created":"2015-11-05 10:38:06.000000000","uploader":{"_account_id":18978,"name":"Catherine Northcott","email":"catherine@northcott.nz","username":"Zyric"},"ref":"refs/changes/78/241978/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/swift","ref":"refs/changes/78/241978/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/swift refs/changes/78/241978/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/swift refs/changes/78/241978/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/swift refs/changes/78/241978/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/swift refs/changes/78/241978/1"}}},"commit":{"parents":[{"commit":"9a7795086ccaf9463657389092677153380f6955","subject":"Merge \"Add verification socket close causes StopIteration\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/swift/commit/9a7795086ccaf9463657389092677153380f6955"}]}],"author":{"name":"Catherine Northcott","email":"catherine@northcott.nz","date":"2015-11-05 10:04:14.000000000","tz":780},"committer":{"name":"Catherine Northcott","email":"catherine@northcott.nz","date":"2015-11-05 10:36:31.000000000","tz":780},"subject":"Add support for storage policies to have more than one name","message":"Add support for storage policies to have more than one name\n\nThis patch alters storage_policy.py to allow storage policies\nto have multiple names. Now users are able to add a number of\nhuman-readable aliases for storage policies.\n\nThis is my first patch to OpenStack, your feedback is very\nmuch appriciated, thank you!\n\nChange-Id: I02967ca8d7c790595e5ee551581196aa64552eea\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/swift/commit/3eaa142976ce6fa7cbf68d0a006b9669cd2a798c"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/swift/commit/3eaa142976ce6fa7cbf68d0a006b9669cd2a798c"}]},"branch":"refs/heads/master"},"083bd5d1e2725855f0201cbd5be48b3d54b95816":{"kind":"REWORK","_number":2,"created":"2015-11-10 03:01:39.000000000","uploader":{"_account_id":18978,"name":"Catherine Northcott","email":"catherine@northcott.nz","username":"Zyric"},"ref":"refs/changes/78/241978/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/swift","ref":"refs/changes/78/241978/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/swift refs/changes/78/241978/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/swift refs/changes/78/241978/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/swift refs/changes/78/241978/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/swift refs/changes/78/241978/2"}}},"commit":{"parents":[{"commit":"9a7795086ccaf9463657389092677153380f6955","subject":"Merge \"Add verification socket close causes StopIteration\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/swift/commit/9a7795086ccaf9463657389092677153380f6955"}]}],"author":{"name":"Catherine Northcott","email":"catherine@northcott.nz","date":"2015-11-05 10:04:14.000000000","tz":780},"committer":{"name":"Catherine Northcott","email":"catherine@northcott.nz","date":"2015-11-10 02:42:36.000000000","tz":780},"subject":"Add support for storage policies to have more than one name","message":"Add support for storage policies to have more than one name\n\nThis patch alters storage_policy.py to allow storage policies\nto have multiple names. Now users are able to add a number of\nhuman-readable aliases for storage policies.\n\nChanges:\n* Policies now have a .primary_name, .aliases, and (for\n  compatibility) a .name which simply outputs a comma separated\n  string of all names for the /info endpoint. If it would be\n  preferable to do an interface overhaul instead let me know.\n  Since this would require changes to many tests and possibly\n  force users to alter their config files I\u0027ve opted to err\n  on the side of caution and try a more legacy compatibile\n  approach first.\n* Changed docstrings and function names for easier readability\n  and understandability\n* Fixed whitespace issues for PEP8\n\nThis is my first patch to OpenStack, your feedback is very\nmuch appriciated, thank you!\n\nChange-Id: I02967ca8d7c790595e5ee551581196aa64552eea\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/swift/commit/083bd5d1e2725855f0201cbd5be48b3d54b95816"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/swift/commit/083bd5d1e2725855f0201cbd5be48b3d54b95816"}]},"branch":"refs/heads/master"},"ff021c7ed4f5281b6991f8f27b5fab74950b6eef":{"kind":"REWORK","_number":3,"created":"2015-11-10 09:23:06.000000000","uploader":{"_account_id":18978,"name":"Catherine Northcott","email":"catherine@northcott.nz","username":"Zyric"},"ref":"refs/changes/78/241978/3","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/swift","ref":"refs/changes/78/241978/3","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/swift refs/changes/78/241978/3 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/swift refs/changes/78/241978/3 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/swift refs/changes/78/241978/3 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/swift refs/changes/78/241978/3"}}},"commit":{"parents":[{"commit":"9a7795086ccaf9463657389092677153380f6955","subject":"Merge \"Add verification socket close causes StopIteration\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/swift/commit/9a7795086ccaf9463657389092677153380f6955"}]}],"author":{"name":"Catherine Northcott","email":"catherine@northcott.nz","date":"2015-11-05 10:04:14.000000000","tz":780},"committer":{"name":"Catherine Northcott","email":"catherine@northcott.nz","date":"2015-11-10 09:22:14.000000000","tz":780},"subject":"Add support for storage policies to have more than one name","message":"Add support for storage policies to have more than one name\n\nThis patch alters storage_policy.py to allow storage policies\nto have multiple names. Now users are able to add a number of\nhuman-readable aliases for storage policies.\n\nChanges (Patch 2):\n* Policies have a .primary_name, .aliases, and (for\n  compatibility) a .name which simply outputs a comma separated\n  string of all names for the /info endpoint. If it would be\n  preferable to do an interface overhaul instead let me know.\n  Since this would require changes to many tests and possibly\n  force users to alter their config files I\u0027ve opted to err\n  on the side of caution and try a more legacy compatibile\n  approach first.\n* Changed docstrings and function names for easier readability\n  and understandability\n* Fixed whitespace issues for PEP8\n\nChanges (Patch 3):\n* Policies have a .name and an .aliases attirbute.\n* Policies will always have their default .name value included\n  in aliases, so will always have an aliases value of at least\n  aliases\u003d[\u0027name\u0027]. This is because .aliases acts as storage\n  for all human readable names attached to a policy.\n* Updated docs and tests to reflect changes.\n* Fixed remaining hanging indentation and whitespace issues\n  for PEP8\n* Changed policy.get_policy_info to display the name\n  and aliases\n\nThis is my first patch to OpenStack, your feedback is very\nmuch appriciated, thank you!\n\nChange-Id: I02967ca8d7c790595e5ee551581196aa64552eea\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/swift/commit/ff021c7ed4f5281b6991f8f27b5fab74950b6eef"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/swift/commit/ff021c7ed4f5281b6991f8f27b5fab74950b6eef"}]},"branch":"refs/heads/master"},"595ce6611b9b780a0797226a38234bdd4219bef1":{"kind":"REWORK","_number":4,"created":"2015-11-24 06:35:05.000000000","uploader":{"_account_id":18978,"name":"Catherine Northcott","email":"catherine@northcott.nz","username":"Zyric"},"ref":"refs/changes/78/241978/4","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/swift","ref":"refs/changes/78/241978/4","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/swift refs/changes/78/241978/4 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/swift refs/changes/78/241978/4 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/swift refs/changes/78/241978/4 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/swift refs/changes/78/241978/4"}}},"commit":{"parents":[{"commit":"09133f5bd988df202b10d3569868cb93aba399ca","subject":"Merge \"remove pbr from requirements.txt. It\u0027s not a run-time dependency\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/swift/commit/09133f5bd988df202b10d3569868cb93aba399ca"}]}],"author":{"name":"Catherine Northcott","email":"catherine@northcott.nz","date":"2015-11-05 10:04:14.000000000","tz":780},"committer":{"name":"Catherine Northcott","email":"catherine@northcott.nz","date":"2015-11-24 06:34:36.000000000","tz":780},"subject":"Add support for storage policies to have more than one name","message":"Add support for storage policies to have more than one name\n\nThis patch alters storage_policy.py to allow storage policies\nto have multiple names. Now users are able to add a number of\nhuman-readable aliases for storage policies. Policies now have\na .name (the default name), .aliases (a string of comma\nseperated aliases), and .aliases_list (a list of all human\nreadable names). Policies will always have an .aliases value\nif no aliases are set it will be \u0027\u0027. The policy docs and\ntests have been updated to reflect changes and\npolicy.get_policy_info has been altered to display the name\nand aliases\n\nChange-Id: I02967ca8d7c790595e5ee551581196aa64552eea\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/swift/commit/595ce6611b9b780a0797226a38234bdd4219bef1"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/swift/commit/595ce6611b9b780a0797226a38234bdd4219bef1"}]},"branch":"refs/heads/master"},"a7d18f98ee9bd7aa47ac4b60e8ccdc0a9fde7004":{"kind":"REWORK","_number":5,"created":"2015-11-24 19:24:51.000000000","uploader":{"_account_id":330,"name":"John Dickinson","email":"me@not.mn","username":"notmyname"},"ref":"refs/changes/78/241978/5","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/swift","ref":"refs/changes/78/241978/5","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/swift refs/changes/78/241978/5 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/swift refs/changes/78/241978/5 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/swift refs/changes/78/241978/5 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/swift refs/changes/78/241978/5"}}},"commit":{"parents":[{"commit":"f48c6f508e9970eba299eeef32f617608cbe4e1e","subject":"Merge \"Add container and account reverse listings\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/swift/commit/f48c6f508e9970eba299eeef32f617608cbe4e1e"}]}],"author":{"name":"Catherine Northcott","email":"catherine@northcott.nz","date":"2015-11-05 10:04:14.000000000","tz":780},"committer":{"name":"John Dickinson","email":"me@not.mn","date":"2015-11-24 19:00:20.000000000","tz":-480},"subject":"Add support for storage policies to have more than one name","message":"Add support for storage policies to have more than one name\n\nThis patch alters storage_policy.py to allow storage policies\nto have multiple names. Now users are able to add a number of\nhuman-readable aliases for storage policies. Policies now have\na .name (the default name), .aliases (a string of comma\nseperated aliases), and .aliases_list (a list of all human\nreadable names). Policies will always have an .aliases value\nif no aliases are set it will be \u0027\u0027. The policy docs and\ntests have been updated to reflect changes and\npolicy.get_policy_info has been altered to display the name\nand aliases\n\nChange-Id: I02967ca8d7c790595e5ee551581196aa64552eea\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/swift/commit/a7d18f98ee9bd7aa47ac4b60e8ccdc0a9fde7004"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/swift/commit/a7d18f98ee9bd7aa47ac4b60e8ccdc0a9fde7004"}]},"branch":"refs/heads/master"},"6012aef8f595d587a907db87e9d4c29e46e64730":{"kind":"REWORK","_number":6,"created":"2015-11-26 06:18:08.000000000","uploader":{"_account_id":18978,"name":"Catherine Northcott","email":"catherine@northcott.nz","username":"Zyric"},"ref":"refs/changes/78/241978/6","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/swift","ref":"refs/changes/78/241978/6","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/swift refs/changes/78/241978/6 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/swift refs/changes/78/241978/6 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/swift refs/changes/78/241978/6 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/swift refs/changes/78/241978/6"}}},"commit":{"parents":[{"commit":"09133f5bd988df202b10d3569868cb93aba399ca","subject":"Merge \"remove pbr from requirements.txt. It\u0027s not a run-time dependency\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/swift/commit/09133f5bd988df202b10d3569868cb93aba399ca"}]}],"author":{"name":"Catherine Northcott","email":"catherine@northcott.nz","date":"2015-11-05 10:04:14.000000000","tz":780},"committer":{"name":"Catherine Northcott","email":"catherine@northcott.nz","date":"2015-11-26 06:13:17.000000000","tz":780},"subject":"Add support for storage policies to have more than one name","message":"Add support for storage policies to have more than one name\n\nThis patch alters storage_policy.py to allow storage policies\nto have multiple names. Now users are able to add a number of\nhuman-readable aliases for storage policies. Policies now have\na .name (the default name), .aliases (a string of comma\nseperated aliases), and .aliases_list (a list of all human\nreadable names). Policies will always have an .aliases value\nif no aliases are set it will contain the default name.\nThe policy docs and tests have been updated to reflect changes\nand policy.get_policy_info has been altered to display the\nname and aliases\n\nChange-Id: I02967ca8d7c790595e5ee551581196aa64552eea\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/swift/commit/6012aef8f595d587a907db87e9d4c29e46e64730"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/swift/commit/6012aef8f595d587a907db87e9d4c29e46e64730"}]},"branch":"refs/heads/master"},"84d289e9eadc7654f9ece1cd46ca2bef6e4f0c65":{"kind":"REWORK","_number":7,"created":"2015-12-08 23:06:07.000000000","uploader":{"_account_id":18978,"name":"Catherine Northcott","email":"catherine@northcott.nz","username":"Zyric"},"ref":"refs/changes/78/241978/7","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/swift","ref":"refs/changes/78/241978/7","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/swift refs/changes/78/241978/7 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/swift refs/changes/78/241978/7 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/swift refs/changes/78/241978/7 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/swift refs/changes/78/241978/7"}}},"commit":{"parents":[{"commit":"09133f5bd988df202b10d3569868cb93aba399ca","subject":"Merge \"remove pbr from requirements.txt. It\u0027s not a run-time dependency\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/swift/commit/09133f5bd988df202b10d3569868cb93aba399ca"}]}],"author":{"name":"Catherine Northcott","email":"catherine@northcott.nz","date":"2015-11-05 10:04:14.000000000","tz":780},"committer":{"name":"Catherine Northcott","email":"catherine@northcott.nz","date":"2015-12-08 22:56:26.000000000","tz":780},"subject":"Add support for storage policies to have more than one name","message":"Add support for storage policies to have more than one name\n\nThis patch alters storage_policy.py to allow storage policies\nto have multiple names. Now users are able to add a number of\nhuman-readable aliases for storage policies. Policies now have\na .name (the default name), .aliases (a string of comma\nseperated aliases), and .aliases_list (a list of all human\nreadable names). Policies will always have an .aliases value\nif no aliases are set it will contain the default name.\nThe policy docs and tests have been updated to reflect changes\nand policy.get_policy_info has been altered to display the\nname and aliases\n\nChange-Id: I02967ca8d7c790595e5ee551581196aa64552eea\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/swift/commit/84d289e9eadc7654f9ece1cd46ca2bef6e4f0c65"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/swift/commit/84d289e9eadc7654f9ece1cd46ca2bef6e4f0c65"}]},"branch":"refs/heads/master"},"211758f8cb02298fe16e59bf2954a146c6b24b83":{"kind":"REWORK","_number":8,"created":"2015-12-10 01:52:45.000000000","uploader":{"_account_id":18978,"name":"Catherine Northcott","email":"catherine@northcott.nz","username":"Zyric"},"ref":"refs/changes/78/241978/8","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/swift","ref":"refs/changes/78/241978/8","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/swift refs/changes/78/241978/8 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/swift refs/changes/78/241978/8 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/swift refs/changes/78/241978/8 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/swift refs/changes/78/241978/8"}}},"commit":{"parents":[{"commit":"09133f5bd988df202b10d3569868cb93aba399ca","subject":"Merge \"remove pbr from requirements.txt. It\u0027s not a run-time dependency\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/swift/commit/09133f5bd988df202b10d3569868cb93aba399ca"}]}],"author":{"name":"Catherine Northcott","email":"catherine@northcott.nz","date":"2015-11-05 10:04:14.000000000","tz":780},"committer":{"name":"Catherine Northcott","email":"catherine@northcott.nz","date":"2015-12-10 01:47:23.000000000","tz":780},"subject":"Add support for storage policies to have more than one name","message":"Add support for storage policies to have more than one name\n\nThis patch alters storage_policy.py to allow storage policies\nto have multiple names. Now users are able to add a number of\nhuman-readable aliases for storage policies. Policies now have\na .name (the default name), .aliases (a string of comma\nseperated aliases), and .aliases_list (a list of all human\nreadable names). Policies will always have an .aliases value\nif no aliases are set it will contain the default name.\nThe policy docs and tests have been updated to reflect changes\nand policy.get_policy_info has been altered to display the\nname and aliases\n\nChange-Id: I02967ca8d7c790595e5ee551581196aa64552eea\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/swift/commit/211758f8cb02298fe16e59bf2954a146c6b24b83"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/swift/commit/211758f8cb02298fe16e59bf2954a146c6b24b83"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}