)]}'
{"id":"openstack%2Fpython-cinderclient~159814","triplet_id":"openstack%2Fpython-cinderclient~master~I195ccc37fe96dbb54a0460527fabf55146170bc7","project":"openstack/python-cinderclient","branch":"master","topic":"bug/1422244","hashtags":[],"change_id":"I195ccc37fe96dbb54a0460527fabf55146170bc7","subject":"cinder list now prints dash \u0027-\u0027 when data is None","status":"MERGED","created":"2015-02-27 12:18:26.000000000","updated":"2015-03-16 21:36:03.000000000","submitted":"2015-03-16 21:36:02.000000000","submitter":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"total_comment_count":1,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"91a6f7804a7c60ef06b6566c3d9f7b9dc0b5c0da","_number":159814,"virtual_id_number":159814,"owner":{"_account_id":11793,"name":"Kanchan Gupta","email":"kanchan.gupta1@tcs.com","username":"Kanchan"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"value":0,"_account_id":11793,"name":"Kanchan Gupta","email":"kanchan.gupta1@tcs.com","username":"Kanchan"},{"value":2,"date":"2015-03-16 21:36:02.000000000","_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":2243,"name":"John Griffith","email":"john.griffith8@gmail.com","username":"john-griffith"},{"value":0,"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},{"value":0,"_account_id":1207,"name":"Duncan Thomas","email":"duncan.thomas@gmail.com","username":"duncan-thomas"},{"value":0,"_account_id":170,"name":"Mike Perez","email":"thingee@gmail.com","username":"thingee"},{"value":0,"_account_id":14271,"name":"Kallebe Monteiro","email":"kallebe.monteiro@lsbd.ufc.br","username":"kallebe"}],"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":2243,"name":"John Griffith","email":"john.griffith8@gmail.com","username":"john-griffith"},"all":[{"value":0,"_account_id":11793,"name":"Kanchan Gupta","email":"kanchan.gupta1@tcs.com","username":"Kanchan"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":2,"date":"2015-03-16 19:52:44.000000000","_account_id":2243,"name":"John Griffith","email":"john.griffith8@gmail.com","username":"john-griffith"},{"value":0,"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},{"value":0,"_account_id":1207,"name":"Duncan Thomas","email":"duncan.thomas@gmail.com","username":"duncan-thomas"},{"value":2,"date":"2015-03-16 20:39:21.000000000","_account_id":170,"name":"Mike Perez","email":"thingee@gmail.com","username":"thingee"},{"value":1,"date":"2015-03-16 19:46:47.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":14271,"name":"Kallebe Monteiro","email":"kallebe.monteiro@lsbd.ufc.br","username":"kallebe"}],"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":170,"name":"Mike Perez","email":"thingee@gmail.com","username":"thingee"},"all":[{"value":0,"_account_id":11793,"name":"Kanchan Gupta","email":"kanchan.gupta1@tcs.com","username":"Kanchan"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":2243,"name":"John Griffith","email":"john.griffith8@gmail.com","username":"john-griffith"},{"value":0,"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},{"value":0,"_account_id":1207,"name":"Duncan Thomas","email":"duncan.thomas@gmail.com","username":"duncan-thomas"},{"value":1,"date":"2015-03-16 20:39:21.000000000","_account_id":170,"name":"Mike Perez","email":"thingee@gmail.com","username":"thingee"},{"value":0,"_account_id":14271,"name":"Kallebe Monteiro","email":"kallebe.monteiro@lsbd.ufc.br","username":"kallebe"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true},"Review-Priority":{"all":[{"value":0,"_account_id":11793,"name":"Kanchan Gupta","email":"kanchan.gupta1@tcs.com","username":"Kanchan"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":2243,"name":"John Griffith","email":"john.griffith8@gmail.com","username":"john-griffith"},{"value":0,"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},{"value":0,"_account_id":1207,"name":"Duncan Thomas","email":"duncan.thomas@gmail.com","username":"duncan-thomas"},{"value":0,"_account_id":170,"name":"Mike Perez","email":"thingee@gmail.com","username":"thingee"},{"value":0,"_account_id":14271,"name":"Kallebe Monteiro","email":"kallebe.monteiro@lsbd.ufc.br","username":"kallebe"}],"values":{"-1":"Branch Freeze"," 0":"No Priority","+1":"Important Change","+2":"Gate Blocker Fix / Urgent Change"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":170,"name":"Mike Perez","email":"thingee@gmail.com","username":"thingee"},{"_account_id":1207,"name":"Duncan Thomas","email":"duncan.thomas@gmail.com","username":"duncan-thomas"},{"_account_id":2243,"name":"John Griffith","email":"john.griffith8@gmail.com","username":"john-griffith"},{"_account_id":11793,"name":"Kanchan Gupta","email":"kanchan.gupta1@tcs.com","username":"Kanchan"},{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},{"_account_id":14271,"name":"Kallebe Monteiro","email":"kallebe.monteiro@lsbd.ufc.br","username":"kallebe"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2015-03-01 17:04:29.000000000","updated_by":{"_account_id":1207,"name":"Duncan Thomas","email":"duncan.thomas@gmail.com","username":"duncan-thomas"},"reviewer":{"_account_id":1207,"name":"Duncan Thomas","email":"duncan.thomas@gmail.com","username":"duncan-thomas"},"state":"REVIEWER"},{"updated":"2015-03-02 17:46:11.000000000","updated_by":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"reviewer":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"state":"REVIEWER"},{"updated":"2015-03-16 19:46:47.000000000","updated_by":{"_account_id":14271,"name":"Kallebe Monteiro","email":"kallebe.monteiro@lsbd.ufc.br","username":"kallebe"},"reviewer":{"_account_id":14271,"name":"Kallebe Monteiro","email":"kallebe.monteiro@lsbd.ufc.br","username":"kallebe"},"state":"REVIEWER"},{"updated":"2015-03-16 19:52:44.000000000","updated_by":{"_account_id":2243,"name":"John Griffith","email":"john.griffith8@gmail.com","username":"john-griffith"},"reviewer":{"_account_id":2243,"name":"John Griffith","email":"john.griffith8@gmail.com","username":"john-griffith"},"state":"REVIEWER"},{"updated":"2015-03-16 20:39:21.000000000","updated_by":{"_account_id":170,"name":"Mike Perez","email":"thingee@gmail.com","username":"thingee"},"reviewer":{"_account_id":170,"name":"Mike Perez","email":"thingee@gmail.com","username":"thingee"},"state":"REVIEWER"},{"updated":"2015-03-16 21:36:02.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"}],"messages":[{"id":"16fb0c8eb417d3a5db3cd91226d0c736de208553","author":{"_account_id":11793,"name":"Kanchan Gupta","email":"kanchan.gupta1@tcs.com","username":"Kanchan"},"date":"2015-02-27 12:18:26.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"a7bbf324471bacac33be9245085b97a373f81a5f","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-02-27 13:28:10.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-python-cinderclient-python26 http://logs.openstack.org/14/159814/1/check/gate-python-cinderclient-python26/1ab7b2a/ : SUCCESS in 59s\n- gate-python-cinderclient-pep8 http://logs.openstack.org/14/159814/1/check/gate-python-cinderclient-pep8/1adb8ad/ : SUCCESS in 47s\n- gate-python-cinderclient-docs http://docs-draft.openstack.org/14/159814/1/check/gate-python-cinderclient-docs/15c3f6c//doc/build/html/ : SUCCESS in 47s\n- gate-python-cinderclient-python27 http://logs.openstack.org/14/159814/1/check/gate-python-cinderclient-python27/4d336a3/ : SUCCESS in 43s\n- gate-python-cinderclient-python34 http://logs.openstack.org/14/159814/1/check/gate-python-cinderclient-python34/14fadb2/ : SUCCESS in 1m 47s\n- gate-tempest-dsvm-neutron-src-python-cinderclient http://logs.openstack.org/14/159814/1/check/gate-tempest-dsvm-neutron-src-python-cinderclient/54cb946/ : SUCCESS in 1h 09m 40s\n- gate-tempest-dsvm-neutron-src-python-cinderclient-icehouse http://logs.openstack.org/14/159814/1/check/gate-tempest-dsvm-neutron-src-python-cinderclient-icehouse/c36bb59/ : SUCCESS in 24m 48s\n- gate-tempest-dsvm-neutron-src-python-cinderclient-juno http://logs.openstack.org/14/159814/1/check/gate-tempest-dsvm-neutron-src-python-cinderclient-juno/20ef9bc/ : FAILURE in 54m 56s\n- gate-python-cinderclient-pypy http://logs.openstack.org/14/159814/1/check/gate-python-cinderclient-pypy/3597c29/ : SUCCESS in 1m 26s","accounts_in_message":[],"_revision_number":1},{"id":"d824d769ada6654649f9cc4adc9f454952496254","author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"date":"2015-02-27 14:27:26.000000000","message":"Patch Set 1: Code-Review-1\n\nYou again took the bug wrongly. You should allow volume to be created without display_name. But when volume is printed you should display \"\" or \"-\" instead of None. As None here is ambiguous whether it is None or \"None\".","accounts_in_message":[],"_revision_number":1},{"id":"394961ce0c0ef911bc81ba7b46a40d82f706271f","author":{"_account_id":14271,"name":"Kallebe Monteiro","email":"kallebe.monteiro@lsbd.ufc.br","username":"kallebe"},"date":"2015-02-27 19:02:09.000000000","message":"Patch Set 1: Code-Review-1\n\nI agree with yatin that it is better the way the suggested.\nAlso, I am not sure but I think you should change v2/shell, since v1 is going to be deprecated and the focus is on v2 now.","accounts_in_message":[],"_revision_number":1},{"id":"9473f3a77b2b2a5bf7a22e081209c94f961be5b7","author":{"_account_id":14271,"name":"Kallebe Monteiro","email":"kallebe.monteiro@lsbd.ufc.br","username":"kallebe"},"date":"2015-02-27 19:13:11.000000000","message":"Patch Set 1:\n\nAlso, consider the fact that when a string has value \"\" for all test condition purpose it is the same as None because it evaluates to False.\n\ndef test(s):\n    if s:\n        print \u0027s passes as non-empty\u0027\n    else:\n        print \u0027s is as if were None\u0027\n\ntest(\u0027\u0027) # prints the second option\n\nSo, if the user does not pass a value, I think the display name should be \"-\" to avoid future hard to find unexpected results.","accounts_in_message":[],"_revision_number":1},{"id":"71d26a1925ae72a57ba99a744768b1fa01b84858","author":{"_account_id":1207,"name":"Duncan Thomas","email":"duncan.thomas@gmail.com","username":"duncan-thomas"},"date":"2015-03-01 17:04:29.000000000","message":"Patch Set 1: Code-Review-1\n\nIf the user passes None, then None is what we have to store in the DB - that is what we already have in the DB for any existing deployment. The client can convert this to \"\", \"-\" or indeed anything else that seems appropriate, cinder however should not mangle what it is passed.","accounts_in_message":[],"_revision_number":1},{"id":"ceefcfc686ff8b114a52539dccbce5249e2965d9","author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"date":"2015-03-02 17:45:03.000000000","message":"Patch Set 1: -Code-Review\n\n@Kallebe Monteiro\n\nBoth of the following are different:\n\ns\u003d\"\" (not None in this case) and s\u003dNone\n\nif s: -\u003e returns False  (First case)\n\nif s is None -\u003e returns True  (Second case)","accounts_in_message":[],"_revision_number":1},{"id":"38bb0b9c26f3d67df5112084288b2d90190b738b","author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"date":"2015-03-02 17:46:11.000000000","message":"Patch Set 1: Code-Review-1\n\n@Kallebe Monteiro\n\nBoth of the following are different:\n\ns\u003d\"\" (not None in this case) and s\u003dNone\n\nif s: -\u003e returns False (First case)\n\nif s is None -\u003e returns True (Second case)","accounts_in_message":[],"_revision_number":1},{"id":"75d5adf2e8e5b68712938bdeffb10b6e06e04ca2","author":{"_account_id":14271,"name":"Kallebe Monteiro","email":"kallebe.monteiro@lsbd.ufc.br","username":"kallebe"},"date":"2015-03-02 17:56:37.000000000","message":"Patch Set 1:\n\n@yatin\n\nYes, that is true, but I was talking about tests without None.\n\nIf someone in the future tests only for\n\"if s:\" instead of checking against None, the test would not determine if the value was provided or if it was \"\".\n\nLike in my test function, both results:\n\ntest(\u0027\u0027)\ntest(None)\n\ngive the same result, which would make debugging harder.","accounts_in_message":[],"_revision_number":1},{"id":"3716de33e3e9c399bcea6eeb37e88fa537499ce6","author":{"_account_id":14271,"name":"Kallebe Monteiro","email":"kallebe.monteiro@lsbd.ufc.br","username":"kallebe"},"date":"2015-03-02 17:58:54.000000000","message":"Patch Set 1:\n\n@yatin\nYes, that is true, but I was talking about tests without None.\nIf someone in the future tests only for \"if s:\" instead of checking against None, the test would not determine if the value was provided or if it was \"\".\nLike in my test function, both results:\n\ntest(\u0027\u0027)\n\ntest(None)\n\n\ngive the same result, which would make debugging harder.","accounts_in_message":[],"_revision_number":1},{"id":"d65c012759143b410e7aed504e044c0f6968bef2","author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"date":"2015-03-03 05:08:20.000000000","message":"Patch Set 1:\n\n@Kallebe Monteiro\n\nYeh, that\u0027s right, while writing tests this need to be taken care off. If some patch works on both \"\" and None, Then tests for both cases have to be written.","accounts_in_message":[],"_revision_number":1},{"id":"3354910ac509ad9dfeac7aa467fbe300656f6fce","author":{"_account_id":11793,"name":"Kanchan Gupta","email":"kanchan.gupta1@tcs.com","username":"Kanchan"},"date":"2015-03-03 05:34:26.000000000","message":"Patch Set 1:\n\nHi Yatin,\n\nI understand what you are saying that the changes need to be done while displaying but the issue is that when we create a image using \"cinder create 1\" a summary of the image-created is displayed and in this summary the \"display_name\" printed is None and if we do changes for only printing then won\u0027t there be inconsistency in data as while creating display_name is None and while printing it will show \"\".\n\nPlease clarify and hence I will do the changes.","accounts_in_message":[],"_revision_number":1},{"id":"b490f8bff4b8c0d634e37b4d74d28ecc9905d6cd","author":{"_account_id":1207,"name":"Duncan Thomas","email":"duncan.thomas@gmail.com","username":"duncan-thomas"},"date":"2015-03-03 08:13:44.000000000","message":"Patch Set 1:\n\n@Kanchan Gupta you can not change what goes in the cinder database for this. The changes *must* be done in python-cinderclient display code or not at all. python-cinderclient is not the only thing creating volume, curl or some other library might be used, so None as an entry in the database is always an option. The printing is the only place this can be fixed up","accounts_in_message":[],"_revision_number":1},{"id":"c4aaa267d39645022bfcd3ad4a39025d24fdbbfe","author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"date":"2015-03-04 04:54:05.000000000","message":"Patch Set 1:\n\nHi Kanchan,\n\nAs @Duncan mentioned that you should not change display_name while creating volume, as None display_name can be provided by user. Here the bug corresponds to display, \nAlso, cinder list and cinder create uses same utility print function (utils.print_dict) to print the created and available images. You only have to change around this. Also go through the bug description once again. For your reference you can try running \"nova volume-list\" to know what i really mean. As in this case also cinder database consist of None display_name but while printing it shows \"-\". I want same in cinderclient, either \"\" or \"-\", both will be good. \nHope you get my point.\n\nThanks","accounts_in_message":[],"_revision_number":1},{"id":"e322c4f9eccd009ef1077c92206537bf695967d6","author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"date":"2015-03-16 18:00:17.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"446d12873cfd0fbeded11404b66103863c30f877","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-03-16 19:43:26.000000000","message":"Patch Set 2: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-python-cinderclient-python26 http://logs.openstack.org/14/159814/2/check/gate-python-cinderclient-python26/d9f7dba/ : SUCCESS in 56s\n- gate-python-cinderclient-pep8 http://logs.openstack.org/14/159814/2/check/gate-python-cinderclient-pep8/7c7351e/ : SUCCESS in 1m 14s\n- gate-python-cinderclient-docs http://docs-draft.openstack.org/14/159814/2/check/gate-python-cinderclient-docs/5a7d902//doc/build/html/ : SUCCESS in 49s\n- gate-python-cinderclient-python27 http://logs.openstack.org/14/159814/2/check/gate-python-cinderclient-python27/711ca94/ : SUCCESS in 53s\n- gate-python-cinderclient-python34 http://logs.openstack.org/14/159814/2/check/gate-python-cinderclient-python34/747a082/ : SUCCESS in 2m 25s\n- gate-tempest-dsvm-neutron-src-python-cinderclient http://logs.openstack.org/14/159814/2/check/gate-tempest-dsvm-neutron-src-python-cinderclient/2875695/ : SUCCESS in 56m 50s\n- gate-tempest-dsvm-neutron-src-python-cinderclient-icehouse http://logs.openstack.org/14/159814/2/check/gate-tempest-dsvm-neutron-src-python-cinderclient-icehouse/5acb345/ : SUCCESS in 30m 52s\n- gate-tempest-dsvm-neutron-src-python-cinderclient-juno http://logs.openstack.org/14/159814/2/check/gate-tempest-dsvm-neutron-src-python-cinderclient-juno/0b35fe4/ : SUCCESS in 48m 13s\n- gate-python-cinderclient-pypy http://logs.openstack.org/14/159814/2/check/gate-python-cinderclient-pypy/43bbafd/ : SUCCESS in 1m 38s","accounts_in_message":[],"_revision_number":2},{"id":"bfdc6ffa99b89c8e6286e39770c565683fdf4c46","author":{"_account_id":14271,"name":"Kallebe Monteiro","email":"kallebe.monteiro@lsbd.ufc.br","username":"kallebe"},"date":"2015-03-16 19:46:47.000000000","message":"Patch Set 2: Code-Review+1","accounts_in_message":[],"_revision_number":2},{"id":"e681ed10e8c3999f36540ac172a603bb09a790ee","author":{"_account_id":2243,"name":"John Griffith","email":"john.griffith8@gmail.com","username":"john-griffith"},"date":"2015-03-16 19:52:44.000000000","message":"Patch Set 2: Code-Review+2\n\n(1 comment)\n\nNote inline about the second set of if/else (don\u0027t think we need it).  As far as the add itself I like it, thanks!","accounts_in_message":[],"_revision_number":2},{"id":"e3a2d71a539e3ca36ae9dd4343ac27d1210e313c","author":{"_account_id":170,"name":"Mike Perez","email":"thingee@gmail.com","username":"thingee"},"date":"2015-03-16 20:39:21.000000000","message":"Patch Set 2: Code-Review+2 Workflow+1","accounts_in_message":[],"_revision_number":2},{"id":"d63241a70b35cafce4eb68994457ca8f16b926cf","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-03-16 20:39:22.000000000","message":"Patch Set 2: -Verified\n\nStarting gate jobs.\nhttp://status.openstack.org/zuul/","accounts_in_message":[],"_revision_number":2},{"id":"1ee472f2d7d787744a531a852ddb03fa8ee9b911","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-03-16 21:36:02.000000000","message":"Patch Set 2: Verified+2\n\nBuild succeeded (gate pipeline).\n\n- gate-python-cinderclient-python26 http://logs.openstack.org/14/159814/2/gate/gate-python-cinderclient-python26/14b6d3f/ : SUCCESS in 54s\n- gate-python-cinderclient-docs http://docs-draft.openstack.org/14/159814/2/gate/gate-python-cinderclient-docs/f1ab52e//doc/build/html/ : SUCCESS in 47s\n- gate-python-cinderclient-pep8 http://logs.openstack.org/14/159814/2/gate/gate-python-cinderclient-pep8/3f6f4c3/ : SUCCESS in 59s\n- gate-python-cinderclient-python27 http://logs.openstack.org/14/159814/2/gate/gate-python-cinderclient-python27/d48eb46/ : SUCCESS in 59s\n- gate-python-cinderclient-python34 http://logs.openstack.org/14/159814/2/gate/gate-python-cinderclient-python34/e40565c/ : SUCCESS in 2m 06s\n- gate-tempest-dsvm-neutron-src-python-cinderclient http://logs.openstack.org/14/159814/2/gate/gate-tempest-dsvm-neutron-src-python-cinderclient/acc817a/ : SUCCESS in 46m 30s\n- gate-python-cinderclient-pypy http://logs.openstack.org/14/159814/2/gate/gate-python-cinderclient-pypy/79d6552/ : SUCCESS in 1m 30s","accounts_in_message":[],"_revision_number":2},{"id":"7f445070632b0395ffcd743f7f225e73e3c1b699","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-03-16 21:36:03.000000000","message":"Change has been successfully merged into the git repository.","accounts_in_message":[],"_revision_number":2}],"current_revision_number":2,"current_revision":"0f73c5fb8ab593da7e332ade273e7da31ea8e303","revisions":{"26d846507855fa88d346ef76230903429972e760":{"kind":"REWORK","_number":1,"created":"2015-02-27 12:18:26.000000000","uploader":{"_account_id":11793,"name":"Kanchan Gupta","email":"kanchan.gupta1@tcs.com","username":"Kanchan"},"ref":"refs/changes/14/159814/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-cinderclient","ref":"refs/changes/14/159814/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/14/159814/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/14/159814/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/14/159814/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-cinderclient refs/changes/14/159814/1"}}},"commit":{"parents":[{"commit":"c01529fa538d28af3a9f0f7cdc1a96a9df5ed221","subject":"Add missing all-tenants option to transfer-list","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/c01529fa538d28af3a9f0f7cdc1a96a9df5ed221"}]}],"author":{"name":"Kanchan Gupta","email":"kanchan.gupta1@tcs.com","date":"2015-02-27 12:15:42.000000000","tz":330},"committer":{"name":"Kanchan Gupta","email":"kanchan.gupta1@tcs.com","date":"2015-02-27 12:18:13.000000000","tz":330},"subject":"Checking name for volume creation in cinder","message":"Checking name for volume creation in cinder\n\nPreviously the display_name was not checked whether its blank\nor none. Added the checking parameter for the display name.\n\nChange-Id: I195ccc37fe96dbb54a0460527fabf55146170bc7\nCloses-Bug: #1422244\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/26d846507855fa88d346ef76230903429972e760"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/26d846507855fa88d346ef76230903429972e760"}]},"branch":"refs/heads/master"},"0f73c5fb8ab593da7e332ade273e7da31ea8e303":{"kind":"REWORK","_number":2,"created":"2015-03-16 18:00:17.000000000","uploader":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"ref":"refs/changes/14/159814/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-cinderclient","ref":"refs/changes/14/159814/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/14/159814/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/14/159814/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/14/159814/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-cinderclient refs/changes/14/159814/2"}}},"commit":{"parents":[{"commit":"f47f973d9b15599983224c05c1408ee723236ea2","subject":"Updated help on cinder reset-state cli","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/f47f973d9b15599983224c05c1408ee723236ea2"}]}],"author":{"name":"yatin karel","email":"yatin.karel@nectechnologies.in","date":"2015-03-16 17:56:12.000000000","tz":330},"committer":{"name":"yatin karel","email":"yatin.karel@nectechnologies.in","date":"2015-03-16 17:56:22.000000000","tz":330},"subject":"cinder list now prints dash \u0027-\u0027 when data is None","message":"cinder list now prints dash \u0027-\u0027 when data is None\n\ncinder list used to print None when volume was created without name.\nNow it prints \u0027-\u0027 dash when display_name is None\n\nCloses-Bug: #1422244\nChange-Id: I195ccc37fe96dbb54a0460527fabf55146170bc7\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/0f73c5fb8ab593da7e332ade273e7da31ea8e303"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/0f73c5fb8ab593da7e332ade273e7da31ea8e303"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}