)]}'
{"id":"openstack%2Fpython-magnumclient~255736","triplet_id":"openstack%2Fpython-magnumclient~master~I7160e114d6379462f8d1e700fc76e396409767c1","project":"openstack/python-magnumclient","branch":"master","topic":"bp/add-bay-column-to-container","hashtags":[],"change_id":"I7160e114d6379462f8d1e700fc76e396409767c1","subject":"Add bay_uuid to container-list output","status":"ABANDONED","created":"2015-12-10 08:47:44.000000000","updated":"2015-12-18 07:57:36.000000000","total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"b275c29f74335b702ed220916cdf78857e50d35a","_number":255736,"virtual_id_number":255736,"owner":{"_account_id":12175,"name":"Eli Qiao","email":"qiaoliyong@gmail.com","username":"Eli"},"actions":{},"labels":{"Verified":{"disliked":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"_account_id":11536,"name":"hongbin","email":"hongbin034@gmail.com","username":"hongbin"},{"value":-1,"date":"2015-12-17 19:45:50.000000000","_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":10263,"name":"Vilobh Meshram","email":"vilobhmm@yahoo-inc.com","username":"vilobhmm"},{"date":"2015-12-11 05:59:34.000000000","_account_id":12175,"name":"Eli Qiao","email":"qiaoliyong@gmail.com","username":"Eli"},{"date":"2015-12-11 05:54:14.000000000","_account_id":8143,"name":"Egor Guz","email":"egor.guz@gmail.com","username":"eghobo"}],"values":{"-2":"Fails","-1":"Doesn\u0027t seem to work"," 0":"No score","+1":"Works for me","+2":"Verified"},"description":"","value":-1,"default_value":0,"optional":true},"Code-Review":{"approved":{"_account_id":11536,"name":"hongbin","email":"hongbin034@gmail.com","username":"hongbin"},"all":[{"value":2,"date":"2015-12-17 19:13:33.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":11536,"name":"hongbin","email":"hongbin034@gmail.com","username":"hongbin"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":-1,"date":"2015-12-18 01:28:21.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":10263,"name":"Vilobh Meshram","email":"vilobhmm@yahoo-inc.com","username":"vilobhmm"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":12175,"name":"Eli Qiao","email":"qiaoliyong@gmail.com","username":"Eli"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":8143,"name":"Egor Guz","email":"egor.guz@gmail.com","username":"eghobo"}],"values":{"-2":"Do not merge","-1":"This patch needs further work before it can be merged"," 0":"No score","+1":"Looks good to me, but someone else must approve","+2":"Looks good to me (core reviewer)"},"description":"","default_value":0,"optional":true},"Workflow":{"all":[{"_account_id":11536,"name":"hongbin","email":"hongbin034@gmail.com","username":"hongbin"},{"_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":10263,"name":"Vilobh Meshram","email":"vilobhmm@yahoo-inc.com","username":"vilobhmm"},{"value":0,"permitted_voting_range":{"min":-1,"max":0},"_account_id":12175,"name":"Eli Qiao","email":"qiaoliyong@gmail.com","username":"Eli"},{"_account_id":8143,"name":"Egor Guz","email":"egor.guz@gmail.com","username":"eghobo"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true},"Backport-Candidate":{"all":[{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":11536,"name":"hongbin","email":"hongbin034@gmail.com","username":"hongbin"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":10263,"name":"Vilobh Meshram","email":"vilobhmm@yahoo-inc.com","username":"vilobhmm"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":12175,"name":"Eli Qiao","email":"qiaoliyong@gmail.com","username":"Eli"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":8143,"name":"Egor Guz","email":"egor.guz@gmail.com","username":"eghobo"}],"values":{"-2":"Do Not Backport","-1":"Not A Backport Candidate"," 0":"Backport Review Needed","+1":"Proposed Backport","+2":"Should Backport"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":8143,"name":"Egor Guz","email":"egor.guz@gmail.com","username":"eghobo"},{"_account_id":10263,"name":"Vilobh Meshram","email":"vilobhmm@yahoo-inc.com","username":"vilobhmm"},{"_account_id":11536,"name":"hongbin","email":"hongbin034@gmail.com","username":"hongbin"},{"_account_id":12175,"name":"Eli Qiao","email":"qiaoliyong@gmail.com","username":"Eli"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2015-12-11 05:54:14.000000000","updated_by":{"_account_id":8143,"name":"Egor Guz","email":"egor.guz@gmail.com","username":"eghobo"},"reviewer":{"_account_id":8143,"name":"Egor Guz","email":"egor.guz@gmail.com","username":"eghobo"},"state":"REVIEWER"},{"updated":"2015-12-17 19:13:33.000000000","updated_by":{"_account_id":11536,"name":"hongbin","email":"hongbin034@gmail.com","username":"hongbin"},"reviewer":{"_account_id":11536,"name":"hongbin","email":"hongbin034@gmail.com","username":"hongbin"},"state":"REVIEWER"},{"updated":"2015-12-17 19:45:50.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"},{"updated":"2015-12-18 01:28:21.000000000","updated_by":{"_account_id":10263,"name":"Vilobh Meshram","email":"vilobhmm@yahoo-inc.com","username":"vilobhmm"},"reviewer":{"_account_id":10263,"name":"Vilobh Meshram","email":"vilobhmm@yahoo-inc.com","username":"vilobhmm"},"state":"REVIEWER"}],"messages":[{"id":"e3baa443734f1e49aa5e24cee05066aa4441695d","author":{"_account_id":12175,"name":"Eli Qiao","email":"qiaoliyong@gmail.com","username":"Eli"},"date":"2015-12-10 08:47:44.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"12be46cddeb37579f79d7baa42927eb81e6ae942","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-12-10 08:51:19.000000000","message":"Patch Set 1: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-python-magnumclient-pep8 http://logs.openstack.org/36/255736/1/check/gate-python-magnumclient-pep8/f039fd1/ : SUCCESS in 48s\n- gate-python-magnumclient-docs http://docs-draft.openstack.org/36/255736/1/check/gate-python-magnumclient-docs/d072d1f//doc/build/html/ : SUCCESS in 47s\n- gate-python-magnumclient-python27 http://logs.openstack.org/36/255736/1/check/gate-python-magnumclient-python27/ad8623f/ : SUCCESS in 1m 53s\n- gate-python-magnumclient-tox-bandit http://logs.openstack.org/36/255736/1/check/gate-python-magnumclient-tox-bandit/cf4b096/ : SUCCESS in 55s (non-voting)","accounts_in_message":[],"_revision_number":1},{"id":"43573bced1d5d31837bb5b6814ffb9be86fb3ed6","author":{"_account_id":8143,"name":"Egor Guz","email":"egor.guz@gmail.com","username":"eghobo"},"date":"2015-12-11 05:54:14.000000000","message":"Patch Set 1:\n\n@Eli: I thought that bay uid/name is required parameter for any container related operation, Vilobh? And user shouldn\u0027t see multiply containers from different bays.","accounts_in_message":[],"_revision_number":1},{"id":"c6465d667aaab205021ebb64452506a247a7411a","author":{"_account_id":12175,"name":"Eli Qiao","email":"qiaoliyong@gmail.com","username":"Eli"},"date":"2015-12-11 05:59:34.000000000","message":"Patch Set 1:\n\ncurrently ,container-list  will return all containers for a user.\n\nbut now I am working on add --bay to container-list, but it is still optional.","accounts_in_message":[],"_revision_number":1},{"id":"13b1c1903bd0bec20be48b704dc132c644ff898c","author":{"_account_id":11536,"name":"hongbin","email":"hongbin034@gmail.com","username":"hongbin"},"date":"2015-12-17 19:13:33.000000000","message":"Patch Set 1: Code-Review+2","accounts_in_message":[],"_revision_number":1},{"id":"99ce8a88d29005fb0bb1e7513f67e17f896991d4","author":{"_account_id":10263,"name":"Vilobh Meshram","email":"vilobhmm@yahoo-inc.com","username":"vilobhmm"},"date":"2015-12-17 19:16:58.000000000","message":"Patch Set 1: Code-Review-1\n\n@Eli : Isn\u0027t this missing some tests ? \n\n\nWhy add bay_uuid and not bay_ident (bay_identifier) as it can be name/uuid both ? Or we can even suffice with just bay_uuid here ?","accounts_in_message":[],"_revision_number":1},{"id":"c2957efc8fdf13171e523d0907df33e188230a62","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-12-17 19:45:50.000000000","message":"Patch Set 1: Verified-1\n\nMerge Failed.\n\nThis change or one of its cross-repo dependencies was unable to be automatically merged with the current state of its repository. Please rebase the change and upload a new patchset.","accounts_in_message":[],"_revision_number":1},{"id":"88d4eafecd3e826fcadc07de858993453830f06f","author":{"_account_id":12175,"name":"Eli Qiao","email":"qiaoliyong@gmail.com","username":"Eli"},"date":"2015-12-18 01:17:33.000000000","message":"Patch Set 1:\n\n@Vilobh Meshram\ncan you tell, how can I get  bay_ident (bay_identifier) in client side ?\ncurrently, this is an example of server return.\n\n            {\n                \"status\": \"Unknown\",\n                \"uuid\": \"acac3371-9543-4d66-8204-d92c3ca87d38\",\n                \"links\": [\n                    {\n                        \"href\": \"http://10.238.157.49:9511/v1/containers/acac3371-9543-4d66-8204-d92c3ca87d38\",\n                        \"rel\": \"self\"\n                    },\n                    {\n                        \"href\": \"http://10.238.157.49:9511/containers/acac3371-9543-4d66-8204-d92c3ca87d38\",\n                        \"rel\": \"bookmark\"\n                    }\n                ],\n                \"image\": \"docker.io/cirros\",\n                \"environment\": null,\n                \"command\": \"ping -c 100 10.248.2.5\",\n                \"memory\": null,\n                \"bay_uuid\": \"66a8f326-72ef-49e7-a923-20c2cbe52d07\",\n                \"name\": \"test_ping\"\n            },\n\nI would say please do some investigation before you made -1 comment.","accounts_in_message":[],"_revision_number":1},{"id":"85f8fc5e5ee3c6a52facec82cdfb28d294a575b1","author":{"_account_id":10263,"name":"Vilobh Meshram","email":"vilobhmm@yahoo-inc.com","username":"vilobhmm"},"date":"2015-12-18 01:27:38.000000000","message":"Patch Set 1: Code-Review+1\n\n@Eli : It was a question for you to verify whether its even a possibility and we are not missing out on it. Good that you verified it.","accounts_in_message":[],"_revision_number":1},{"id":"86f5916a0304a5b126af532fd0c98c1f55a58bfb","author":{"_account_id":10263,"name":"Vilobh Meshram","email":"vilobhmm@yahoo-inc.com","username":"vilobhmm"},"date":"2015-12-18 01:28:21.000000000","message":"Patch Set 1: Code-Review-1\n\nNeeds rebase. -1 for rebase.","accounts_in_message":[],"_revision_number":1},{"id":"3ec523d98551b7fbb2c0b0d1b8b2999ea7a2703b","author":{"_account_id":12175,"name":"Eli Qiao","email":"qiaoliyong@gmail.com","username":"Eli"},"date":"2015-12-18 07:57:36.000000000","message":"Abandoned\n\nI4ae74afc543f841434ba0242cfe42c727afb9d7d has included this change.","accounts_in_message":[],"_revision_number":1}],"current_revision_number":1,"current_revision":"044c2f5b837d659ea85bb7132521764094a487b0","revisions":{"044c2f5b837d659ea85bb7132521764094a487b0":{"kind":"REWORK","_number":1,"created":"2015-12-10 08:47:44.000000000","uploader":{"_account_id":12175,"name":"Eli Qiao","email":"qiaoliyong@gmail.com","username":"Eli"},"ref":"refs/changes/36/255736/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-magnumclient","ref":"refs/changes/36/255736/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-magnumclient refs/changes/36/255736/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-magnumclient refs/changes/36/255736/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-magnumclient refs/changes/36/255736/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-magnumclient refs/changes/36/255736/1"}}},"commit":{"parents":[{"commit":"c8810bef54d12487d06449676f6b49ab1f6970c7","subject":"Improve client master_count validation","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-magnumclient/commit/c8810bef54d12487d06449676f6b49ab1f6970c7"}]}],"author":{"name":"Eli Qiao","email":"liyong.qiao@intel.com","date":"2015-12-10 08:44:39.000000000","tz":480},"committer":{"name":"Eli Qiao","email":"liyong.qiao@intel.com","date":"2015-12-10 08:44:41.000000000","tz":480},"subject":"Add bay_uuid to container-list output","message":"Add bay_uuid to container-list output\n\nThis patch adds bay_uuid to container-list output.\n\nFor a user has create multiple bays, and create containers with same name,\nuser can not identify where is the container running on without doing\ncontianer-show.\n\nPartially implements: blueprint add-bay-column-to-container\nChange-Id: I7160e114d6379462f8d1e700fc76e396409767c1\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-magnumclient/commit/044c2f5b837d659ea85bb7132521764094a487b0"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-magnumclient/commit/044c2f5b837d659ea85bb7132521764094a487b0"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}