)]}'
{"id":"openstack%2Fpython-swiftclient~263259","triplet_id":"openstack%2Fpython-swiftclient~master~I575e58aa12adcf71cdaa70d025a0ea5c63f46903","project":"openstack/python-swiftclient","branch":"master","topic":"bug/1478210","hashtags":[],"change_id":"I575e58aa12adcf71cdaa70d025a0ea5c63f46903","subject":"Fix upload to pseudo-dir passed by \u003ccontainer\u003e arg","status":"MERGED","created":"2016-01-04 13:43:26.000000000","updated":"2016-01-11 23:58:41.000000000","submitted":"2016-01-11 23:58:41.000000000","submitter":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"total_comment_count":3,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"711029ef1902763aca14209c1baf559864e4b56a","_number":263259,"virtual_id_number":263259,"owner":{"_account_id":10786,"name":"James Nzomo","email":"james@tdt.rocks","username":"moje"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"value":2,"date":"2016-01-11 23:58:41.000000000","post_submit":true,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":10789,"name":"Mauro Stettler","email":"mauro.stettler@gmail.com","username":"replay"},{"value":0,"_account_id":1179,"name":"Clay Gerrard","email":"clay.gerrard@gmail.com","username":"clay-gerrard"},{"value":0,"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"}],"values":{"-2":"Fails","-1":"Doesn\u0027t seem to work"," 0":"No score","+1":"Works for me","+2":"Verified"},"description":"","default_value":0,"optional":true},"Code-Review":{"approved":{"_account_id":1179,"name":"Clay Gerrard","email":"clay.gerrard@gmail.com","username":"clay-gerrard"},"recommended":{"_account_id":10789,"name":"Mauro Stettler","email":"mauro.stettler@gmail.com","username":"replay"},"all":[{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":1,"date":"2016-01-06 14:29:33.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":10789,"name":"Mauro Stettler","email":"mauro.stettler@gmail.com","username":"replay"},{"value":2,"date":"2016-01-11 22:50:48.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":1179,"name":"Clay Gerrard","email":"clay.gerrard@gmail.com","username":"clay-gerrard"},{"value":2,"date":"2016-01-05 19:07:57.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"}],"values":{"-2":"Do not merge","-1":"This patch needs further work before it can be merged"," 0":"No score","+1":"Looks good to me, but someone else must approve","+2":"Looks good to me (core reviewer)"},"description":"","value":1,"default_value":0,"optional":true},"Workflow":{"approved":{"_account_id":1179,"name":"Clay Gerrard","email":"clay.gerrard@gmail.com","username":"clay-gerrard"},"all":[{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":10789,"name":"Mauro Stettler","email":"mauro.stettler@gmail.com","username":"replay"},{"value":1,"date":"2016-01-11 22:50:48.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":1179,"name":"Clay Gerrard","email":"clay.gerrard@gmail.com","username":"clay-gerrard"},{"value":0,"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":1179,"name":"Clay Gerrard","email":"clay.gerrard@gmail.com","username":"clay-gerrard"},{"_account_id":10789,"name":"Mauro Stettler","email":"mauro.stettler@gmail.com","username":"replay"},{"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2016-01-05 19:07:57.000000000","updated_by":{"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"},"reviewer":{"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"},"state":"REVIEWER"},{"updated":"2016-01-06 14:29:33.000000000","updated_by":{"_account_id":10789,"name":"Mauro Stettler","email":"mauro.stettler@gmail.com","username":"replay"},"reviewer":{"_account_id":10789,"name":"Mauro Stettler","email":"mauro.stettler@gmail.com","username":"replay"},"state":"REVIEWER"},{"updated":"2016-01-11 22:50:48.000000000","updated_by":{"_account_id":1179,"name":"Clay Gerrard","email":"clay.gerrard@gmail.com","username":"clay-gerrard"},"reviewer":{"_account_id":1179,"name":"Clay Gerrard","email":"clay.gerrard@gmail.com","username":"clay-gerrard"},"state":"REVIEWER"},{"updated":"2016-01-11 23:58:41.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"}],"messages":[{"id":"5abdea11eff651c0e782fede2ed474c0ccc50d3c","author":{"_account_id":10786,"name":"James Nzomo","email":"james@tdt.rocks","username":"moje"},"date":"2016-01-04 13:43:26.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"d1913d2cee0f6ec7b3accc415e9ab487f72d34c3","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-01-04 14:54:04.000000000","message":"Patch Set 1: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-python-swiftclient-pep8 http://logs.openstack.org/59/263259/1/check/gate-python-swiftclient-pep8/f73bde5/ : SUCCESS in 45s\n- gate-python-swiftclient-docs http://docs-draft.openstack.org/59/263259/1/check/gate-python-swiftclient-docs/829965d//doc/build/html/ : SUCCESS in 57s\n- gate-python-swiftclient-python27 http://logs.openstack.org/59/263259/1/check/gate-python-swiftclient-python27/4ddbee7/ : SUCCESS in 1m 02s\n- gate-python-swiftclient-python34 http://logs.openstack.org/59/263259/1/check/gate-python-swiftclient-python34/1ce9e18/ : SUCCESS in 1m 38s\n- gate-python-swiftclient-pypy http://logs.openstack.org/59/263259/1/check/gate-python-swiftclient-pypy/64aabf8/ : SUCCESS in 1m 42s (non-voting)\n- gate-tempest-dsvm-neutron-src-python-swiftclient http://logs.openstack.org/59/263259/1/check/gate-tempest-dsvm-neutron-src-python-swiftclient/d33f9f3/ : SUCCESS in 1h 01m 41s\n- gate-swift-dsvm-functional http://logs.openstack.org/59/263259/1/check/gate-swift-dsvm-functional/f1991a4/ : SUCCESS in 33m 12s\n- gate-swiftclient-dsvm-functional http://logs.openstack.org/59/263259/1/check/gate-swiftclient-dsvm-functional/21c9012/ : SUCCESS in 21m 38s","accounts_in_message":[],"_revision_number":1},{"id":"663fa018f03e79a1cc295016d876ce7ab7c00c60","author":{"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"},"date":"2016-01-04 19:05:08.000000000","message":"Patch Set 1: Code-Review+2\n\n(2 comments)\n\nCouple small points, but this looks good to me.","accounts_in_message":[],"_revision_number":1},{"id":"283b611a0be01569ee83a1984ed6922a319c2932","author":{"_account_id":10786,"name":"James Nzomo","email":"james@tdt.rocks","username":"moje"},"date":"2016-01-04 19:54:58.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"749f696163cf5f762976d5e27a401b5990575a68","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-01-04 23:45:40.000000000","message":"Patch Set 2: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-python-swiftclient-pep8 http://logs.openstack.org/59/263259/2/check/gate-python-swiftclient-pep8/c74238d/ : SUCCESS in 50s\n- gate-python-swiftclient-docs http://docs-draft.openstack.org/59/263259/2/check/gate-python-swiftclient-docs/1e8d970//doc/build/html/ : SUCCESS in 52s\n- gate-python-swiftclient-python27 http://logs.openstack.org/59/263259/2/check/gate-python-swiftclient-python27/46b407a/ : SUCCESS in 58s\n- gate-python-swiftclient-python34 http://logs.openstack.org/59/263259/2/check/gate-python-swiftclient-python34/7e2dcfd/ : SUCCESS in 1m 57s\n- gate-python-swiftclient-pypy http://logs.openstack.org/59/263259/2/check/gate-python-swiftclient-pypy/f53ab64/ : SUCCESS in 1m 22s (non-voting)\n- gate-tempest-dsvm-neutron-src-python-swiftclient http://logs.openstack.org/59/263259/2/check/gate-tempest-dsvm-neutron-src-python-swiftclient/005bb59/ : SUCCESS in 1h 21m 46s\n- gate-swift-dsvm-functional http://logs.openstack.org/59/263259/2/check/gate-swift-dsvm-functional/74de092/ : SUCCESS in 31m 58s\n- gate-swiftclient-dsvm-functional http://logs.openstack.org/59/263259/2/check/gate-swiftclient-dsvm-functional/2a7d857/ : SUCCESS in 24m 29s","accounts_in_message":[],"_revision_number":2},{"id":"342dfbff74e915ca8258c18ed412433d8d180c9a","author":{"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"},"date":"2016-01-05 19:07:57.000000000","message":"Patch Set 2: Code-Review+2","accounts_in_message":[],"_revision_number":2},{"id":"31f531ad414336fb90dad3522aa917bf80cce2e3","author":{"_account_id":10789,"name":"Mauro Stettler","email":"mauro.stettler@gmail.com","username":"replay"},"date":"2016-01-06 14:29:33.000000000","message":"Patch Set 2: Code-Review+1\n\nSeems to make sense to me","accounts_in_message":[],"_revision_number":2},{"id":"ac24bea88adf47d0f2ea4fa977790f3ffc87b68e","author":{"_account_id":1179,"name":"Clay Gerrard","email":"clay.gerrard@gmail.com","username":"clay-gerrard"},"date":"2016-01-11 22:29:17.000000000","message":"Patch Set 2: Code-Review+1\n\n(1 comment)\n\nIt\u0027s sorta weird for me that the name of the obj reported as being created doesn\u0027t include the new pseudo directory path:\n\nvagrant@saio:~$ swift upload test/folder deleteme \ndeleteme\n\n^ expected folder/deleteme maybe?  Not sure.  Consider similar confusion with lp but #1412425\n\nIn fact I\u0027m not really sure that psudeo folder \"support\" was intentional or if obj_put with a container name that includes a slash just *accidentally* ends up doing something that wasn\u0027t obviously wrong.\n\nI wanna pause right quick and consider if we should support this use case and open bugs for uploading segmented objects to pseudo folders etc or if we should just validate container names given on the command line don\u0027t include /\u0027s and call the existing behavior a mis-feature?  Maybe it\u0027s to late for that...","accounts_in_message":[],"_revision_number":2},{"id":"07e7948c57119ab51b6dcdbed5ec6de14a036c80","author":{"_account_id":1179,"name":"Clay Gerrard","email":"clay.gerrard@gmail.com","username":"clay-gerrard"},"date":"2016-01-11 22:50:48.000000000","message":"Patch Set 2: Code-Review+2 Workflow+1\n\nok, I think there may be some good reasons to try and support the syntax?  It\u0027d probably be hard to justify breaking it anyway.\n\nI filed lp bug #1532981 for the pseudo-folder segmented upload","accounts_in_message":[],"_revision_number":2},{"id":"ad8d297f084935a0e8edc3dae396f5875d765839","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-01-11 22:50:53.000000000","message":"Patch Set 2: -Verified\n\nStarting gate jobs.\nhttp://status.openstack.org/zuul/","accounts_in_message":[],"_revision_number":2},{"id":"25e54748057bad96d642dcb86f9e258b999393fe","date":"2016-01-11 23:58:41.000000000","message":"Change has been successfully merged into the git repository by Jenkins","accounts_in_message":[],"_revision_number":2},{"id":"d395b072ad4fb72192b9e364106fd2cec33f2cd1","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-01-11 23:58:41.000000000","message":"Patch Set 2: Verified+2\n\nBuild succeeded (gate pipeline).\n\n- gate-python-swiftclient-docs http://docs-draft.openstack.org/59/263259/2/gate/gate-python-swiftclient-docs/757a53e//doc/build/html/ : SUCCESS in 50s\n- gate-python-swiftclient-pep8 http://logs.openstack.org/59/263259/2/gate/gate-python-swiftclient-pep8/9f4be94/ : SUCCESS in 45s\n- gate-python-swiftclient-python27 http://logs.openstack.org/59/263259/2/gate/gate-python-swiftclient-python27/de1a59b/ : SUCCESS in 53s\n- gate-python-swiftclient-python34 http://logs.openstack.org/59/263259/2/gate/gate-python-swiftclient-python34/e81df84/ : SUCCESS in 1m 54s\n- gate-tempest-dsvm-neutron-src-python-swiftclient http://logs.openstack.org/59/263259/2/gate/gate-tempest-dsvm-neutron-src-python-swiftclient/6f15869/ : SUCCESS in 1h 07m 29s\n- gate-swift-dsvm-functional http://logs.openstack.org/59/263259/2/gate/gate-swift-dsvm-functional/050bfd9/ : SUCCESS in 31m 38s\n- gate-swiftclient-dsvm-functional http://logs.openstack.org/59/263259/2/gate/gate-swiftclient-dsvm-functional/9cdb7b1/ : SUCCESS in 22m 34s","accounts_in_message":[],"_revision_number":2}],"current_revision_number":2,"current_revision":"2c6f367035bd09978d0fbf8c959d03210e421a9b","revisions":{"ef168449fa71ff8bb1f6c5895d8cfc78c05aa63c":{"kind":"REWORK","_number":1,"created":"2016-01-04 13:43:26.000000000","uploader":{"_account_id":10786,"name":"James Nzomo","email":"james@tdt.rocks","username":"moje"},"ref":"refs/changes/59/263259/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-swiftclient","ref":"refs/changes/59/263259/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-swiftclient refs/changes/59/263259/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-swiftclient refs/changes/59/263259/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-swiftclient refs/changes/59/263259/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-swiftclient refs/changes/59/263259/1"}}},"commit":{"parents":[{"commit":"62bfe10f58ce777a1b70549a641528b88f94f246","subject":"Fix some typos","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-swiftclient/commit/62bfe10f58ce777a1b70549a641528b88f94f246"}]}],"author":{"name":"James Nzomo","email":"kazikubwa@gmail.com","date":"2016-01-04 13:09:29.000000000","tz":180},"committer":{"name":"James Nzomo","email":"kazikubwa@gmail.com","date":"2016-01-04 13:37:35.000000000","tz":180},"subject":"Fix upload to pseudo-dir passed by \u003ccontainer\u003e arg","message":"Fix upload to pseudo-dir passed by \u003ccontainer\u003e arg\n\nThis fix makes it possible to upload objects to pseudo-folders by\npassing the upload paths via \u003ccontainer\u003e arg regardless of whether the\ncontainer or folder path exist or not.\n\nChange-Id: I575e58aa12adcf71cdaa70d025a0ea5c63f46903\nCloses-Bug: #1478210\nPartial-Bug: #1432734\nRelated-Bug: #1432734\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-swiftclient/commit/ef168449fa71ff8bb1f6c5895d8cfc78c05aa63c"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-swiftclient/commit/ef168449fa71ff8bb1f6c5895d8cfc78c05aa63c"}]},"branch":"refs/heads/master"},"2c6f367035bd09978d0fbf8c959d03210e421a9b":{"kind":"REWORK","_number":2,"created":"2016-01-04 19:54:58.000000000","uploader":{"_account_id":10786,"name":"James Nzomo","email":"james@tdt.rocks","username":"moje"},"ref":"refs/changes/59/263259/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-swiftclient","ref":"refs/changes/59/263259/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-swiftclient refs/changes/59/263259/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-swiftclient refs/changes/59/263259/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-swiftclient refs/changes/59/263259/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-swiftclient refs/changes/59/263259/2"}}},"commit":{"parents":[{"commit":"62bfe10f58ce777a1b70549a641528b88f94f246","subject":"Fix some typos","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-swiftclient/commit/62bfe10f58ce777a1b70549a641528b88f94f246"}]}],"author":{"name":"James Nzomo","email":"kazikubwa@gmail.com","date":"2016-01-04 13:09:29.000000000","tz":180},"committer":{"name":"James Nzomo","email":"james@tdt.rocks","date":"2016-01-04 19:48:07.000000000","tz":180},"subject":"Fix upload to pseudo-dir passed by \u003ccontainer\u003e arg","message":"Fix upload to pseudo-dir passed by \u003ccontainer\u003e arg\n\nThis fix makes it possible to upload objects to pseudo-folders by\npassing the upload paths via \u003ccontainer\u003e arg regardless of whether the\ncontainer or folder path exist or not.\n\nChange-Id: I575e58aa12adcf71cdaa70d025a0ea5c63f46903\nCloses-Bug: #1478210\nPartial-Bug: #1432734\nRelated-Bug: #1432734\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-swiftclient/commit/2c6f367035bd09978d0fbf8c959d03210e421a9b"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-swiftclient/commit/2c6f367035bd09978d0fbf8c959d03210e421a9b"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}