)]}'
{"id":"openstack%2Fpython-troveclient~530021","triplet_id":"openstack%2Fpython-troveclient~master~Ic6a943b4d5980017915dd2c0361636877376c5cb","project":"openstack/python-troveclient","branch":"master","topic":"bug/1740015","hashtags":[],"change_id":"Ic6a943b4d5980017915dd2c0361636877376c5cb","subject":"Fix show instance with integer name NotFound","status":"MERGED","created":"2017-12-25 10:39:05.000000000","updated":"2018-01-17 04:16:08.000000000","submitted":"2018-01-17 04:16:08.000000000","submitter":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"total_comment_count":28,"unresolved_comment_count":0,"has_review_started":true,"submission_id":"530021-1516162568384-c16e5cb7","meta_rev_id":"def0fc22c00d308b6c2cffad46b8414dd811b24e","_number":530021,"virtual_id_number":530021,"owner":{"_account_id":8037,"name":"renminmin","email":"rmm0811@gmail.com","username":"renminmin"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"value":0,"_account_id":22608,"name":"zhanggang","email":"zhanggang@cmss.chinamobile.com","username":"zhanggang"},{"value":0,"_account_id":24739,"name":"Yao Wang","email":"wangyao@cmss.chinamobile.com","username":"wangyao"},{"value":0,"_account_id":13157,"name":"Zhao Chao","email":"zhaochao1984@gmail.com","username":"zhaochao"},{"value":0,"_account_id":8037,"name":"renminmin","email":"rmm0811@gmail.com","username":"renminmin"},{"value":0,"_account_id":22694,"name":"jian.song","email":"jiansong19941206@gmail.com","username":"jiansong"},{"value":0,"_account_id":25113,"name":"Fan Zhang","email":"zh.f@outlook.com","username":"fanzhang"},{"value":2,"date":"2018-01-17 04:16:08.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"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":25113,"name":"Fan Zhang","email":"zh.f@outlook.com","username":"fanzhang"},"recommended":{"_account_id":13157,"name":"Zhao Chao","email":"zhaochao1984@gmail.com","username":"zhaochao"},"all":[{"value":1,"date":"2018-01-17 01:35:53.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":22608,"name":"zhanggang","email":"zhanggang@cmss.chinamobile.com","username":"zhanggang"},{"value":0,"_account_id":24739,"name":"Yao Wang","email":"wangyao@cmss.chinamobile.com","username":"wangyao"},{"value":1,"date":"2018-01-17 01:59:45.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":13157,"name":"Zhao Chao","email":"zhaochao1984@gmail.com","username":"zhaochao"},{"value":0,"_account_id":8037,"name":"renminmin","email":"rmm0811@gmail.com","username":"renminmin"},{"value":0,"_account_id":22694,"name":"jian.song","email":"jiansong19941206@gmail.com","username":"jiansong"},{"value":2,"date":"2018-01-17 02:27:33.000000000","_account_id":25113,"name":"Fan Zhang","email":"zh.f@outlook.com","username":"fanzhang"},{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"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":25113,"name":"Fan Zhang","email":"zh.f@outlook.com","username":"fanzhang"},"all":[{"value":0,"_account_id":22608,"name":"zhanggang","email":"zhanggang@cmss.chinamobile.com","username":"zhanggang"},{"value":0,"_account_id":24739,"name":"Yao Wang","email":"wangyao@cmss.chinamobile.com","username":"wangyao"},{"value":0,"_account_id":13157,"name":"Zhao Chao","email":"zhaochao1984@gmail.com","username":"zhaochao"},{"value":0,"_account_id":8037,"name":"renminmin","email":"rmm0811@gmail.com","username":"renminmin"},{"value":0,"_account_id":22694,"name":"jian.song","email":"jiansong19941206@gmail.com","username":"jiansong"},{"value":1,"date":"2018-01-17 02:27:35.000000000","_account_id":25113,"name":"Fan Zhang","email":"zh.f@outlook.com","username":"fanzhang"},{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true},"Backport-Candidate":{"all":[{"value":0,"_account_id":22608,"name":"zhanggang","email":"zhanggang@cmss.chinamobile.com","username":"zhanggang"},{"value":0,"_account_id":24739,"name":"Yao Wang","email":"wangyao@cmss.chinamobile.com","username":"wangyao"},{"value":0,"_account_id":13157,"name":"Zhao Chao","email":"zhaochao1984@gmail.com","username":"zhaochao"},{"value":0,"_account_id":8037,"name":"renminmin","email":"rmm0811@gmail.com","username":"renminmin"},{"value":0,"_account_id":22694,"name":"jian.song","email":"jiansong19941206@gmail.com","username":"jiansong"},{"value":0,"_account_id":25113,"name":"Fan Zhang","email":"zh.f@outlook.com","username":"fanzhang"},{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"values":{"-1":"Not A Backport Candidate"," 0":"Backport Review Needed","+1":"Proposed Backport"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":8037,"name":"renminmin","email":"rmm0811@gmail.com","username":"renminmin"},{"_account_id":13157,"name":"Zhao Chao","email":"zhaochao1984@gmail.com","username":"zhaochao"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"_account_id":22608,"name":"zhanggang","email":"zhanggang@cmss.chinamobile.com","username":"zhanggang"},{"_account_id":22694,"name":"jian.song","email":"jiansong19941206@gmail.com","username":"jiansong"},{"_account_id":24739,"name":"Yao Wang","email":"wangyao@cmss.chinamobile.com","username":"wangyao"},{"_account_id":25113,"name":"Fan Zhang","email":"zh.f@outlook.com","username":"fanzhang"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2018-01-04 08:41:48.000000000","updated_by":{"_account_id":22694,"name":"jian.song","email":"jiansong19941206@gmail.com","username":"jiansong"},"reviewer":{"_account_id":22694,"name":"jian.song","email":"jiansong19941206@gmail.com","username":"jiansong"},"state":"REVIEWER"},{"updated":"2018-01-06 09:35:40.000000000","updated_by":{"_account_id":24739,"name":"Yao Wang","email":"wangyao@cmss.chinamobile.com","username":"wangyao"},"reviewer":{"_account_id":24739,"name":"Yao Wang","email":"wangyao@cmss.chinamobile.com","username":"wangyao"},"state":"REVIEWER"},{"updated":"2018-01-17 01:35:53.000000000","updated_by":{"_account_id":22608,"name":"zhanggang","email":"zhanggang@cmss.chinamobile.com","username":"zhanggang"},"reviewer":{"_account_id":22608,"name":"zhanggang","email":"zhanggang@cmss.chinamobile.com","username":"zhanggang"},"state":"REVIEWER"},{"updated":"2018-01-17 01:59:45.000000000","updated_by":{"_account_id":13157,"name":"Zhao Chao","email":"zhaochao1984@gmail.com","username":"zhaochao"},"reviewer":{"_account_id":13157,"name":"Zhao Chao","email":"zhaochao1984@gmail.com","username":"zhaochao"},"state":"REVIEWER"},{"updated":"2018-01-17 02:27:35.000000000","updated_by":{"_account_id":25113,"name":"Fan Zhang","email":"zh.f@outlook.com","username":"fanzhang"},"reviewer":{"_account_id":25113,"name":"Fan Zhang","email":"zh.f@outlook.com","username":"fanzhang"},"state":"REVIEWER"},{"updated":"2018-01-17 04:16:08.000000000","updated_by":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"reviewer":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"state":"REVIEWER"}],"messages":[{"id":"6a7ca839966f3403cc622e685d5075601efbf1c3","author":{"_account_id":8037,"name":"renminmin","email":"rmm0811@gmail.com","username":"renminmin"},"date":"2017-12-25 10:39:05.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"e87cba1fb682b85cd116943ef273273b827a7fec","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2017-12-25 15:34:21.000000000","message":"Patch Set 1: Verified+1\n\nBuild succeeded (check pipeline).\n\n- build-openstack-sphinx-docs http://logs.openstack.org/21/530021/1/check/build-openstack-sphinx-docs/9ea42a0/html/ : SUCCESS in 6m 25s\n- openstack-tox-pep8 http://logs.openstack.org/21/530021/1/check/openstack-tox-pep8/65c1e55/ : SUCCESS in 4m 48s\n- openstack-tox-py27 http://logs.openstack.org/21/530021/1/check/openstack-tox-py27/564bbec/ : SUCCESS in 6m 39s\n- openstack-tox-py35 http://logs.openstack.org/21/530021/1/check/openstack-tox-py35/b779b47/ : SUCCESS in 5m 21s\n- openstackclient-check-plugins http://logs.openstack.org/21/530021/1/check/openstackclient-check-plugins/678fa1a/ : SUCCESS in 7m 12s (non-voting)\n- legacy-trove-functional-dsvm-mysql http://logs.openstack.org/21/530021/1/check/legacy-trove-functional-dsvm-mysql/3fd5866/ : SUCCESS in 2h 11m 34s\n- tempest-neutron-src-python-troveclient http://logs.openstack.org/21/530021/1/check/tempest-neutron-src-python-troveclient/dfa2d48/ : SUCCESS in 1h 28m 25s","accounts_in_message":[],"_revision_number":1},{"id":"d47e35d378fa44aa01d8c86fd7c896d9d2e83c77","author":{"_account_id":22694,"name":"jian.song","email":"jiansong19941206@gmail.com","username":"jiansong"},"date":"2018-01-04 08:41:48.000000000","message":"Patch Set 1:\n\nThere should be no problem overall modification, need to increase the test, I noticed that the test did not simulate this situation.","accounts_in_message":[],"_revision_number":1},{"id":"52c95fc40517a65a7bf51a3edd2540c523dfe2a8","author":{"_account_id":25113,"name":"Fan Zhang","email":"zh.f@outlook.com","username":"fanzhang"},"date":"2018-01-05 02:01:08.000000000","message":"Patch Set 1: Code-Review-1\n\n\u003e There should be no problem overall modification, need to increase\n \u003e the test, I noticed that the test did not simulate this situation.\n\nAgreed. Unit test(s) is necessary for such a change. I am not sure I\u0027ve ever hit this bug, so an unit test definitely matters.","accounts_in_message":[],"_revision_number":1},{"id":"0ac7928442f40f1fd4fec92b534e47cd3f2482ad","author":{"_account_id":25113,"name":"Fan Zhang","email":"zh.f@outlook.com","username":"fanzhang"},"date":"2018-01-05 02:07:07.000000000","message":"Patch Set 1: -Code-Review","accounts_in_message":[],"_revision_number":1},{"id":"2ecedc4f6d4fde089a93f46a02001aca7e465dff","author":{"_account_id":24739,"name":"Yao Wang","email":"wangyao@cmss.chinamobile.com","username":"wangyao"},"date":"2018-01-06 09:35:40.000000000","message":"Patch Set 1:\n\n(1 comment)\n\nThe problem exists. Agree with fan and jian you may add a unit tests.","accounts_in_message":[],"_revision_number":1},{"id":"9ae820acdcddebdfa31d04972eef829776747456","author":{"_account_id":13157,"name":"Zhao Chao","email":"zhaochao1984@gmail.com","username":"zhaochao"},"date":"2018-01-08 15:08:05.000000000","message":"Patch Set 1: Code-Review-1\n\n(1 comment)\n\nAgree with new unittests should be added.","accounts_in_message":[],"_revision_number":1},{"id":"b700ea6aaa2a8278a4494cae84ed75e41e17326e","author":{"_account_id":8037,"name":"renminmin","email":"rmm0811@gmail.com","username":"renminmin"},"date":"2018-01-09 02:40:21.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"ab87c237f12fa371b9731cba7eee446c633a9f3f","author":{"_account_id":25113,"name":"Fan Zhang","email":"zh.f@outlook.com","username":"fanzhang"},"date":"2018-01-09 02:55:43.000000000","message":"Patch Set 2:\n\n(1 comment)","accounts_in_message":[],"_revision_number":2},{"id":"4f01a59a94ac2c6677627911acaf68f54f72206b","author":{"_account_id":13157,"name":"Zhao Chao","email":"zhaochao1984@gmail.com","username":"zhaochao"},"date":"2018-01-09 02:57:35.000000000","message":"Patch Set 2: Code-Review-1\n\n(2 comments)","accounts_in_message":[],"_revision_number":2},{"id":"acbf89d734d2cd26ce017014b5e3ea7060bb8b59","author":{"_account_id":8037,"name":"renminmin","email":"rmm0811@gmail.com","username":"renminmin"},"date":"2018-01-09 03:24:33.000000000","message":"Patch Set 2:\n\n(2 comments)","accounts_in_message":[],"_revision_number":2},{"id":"3ea4ea20759003d76c0c09299d4c1753ea6fd243","author":{"_account_id":13157,"name":"Zhao Chao","email":"zhaochao1984@gmail.com","username":"zhaochao"},"date":"2018-01-09 03:45:33.000000000","message":"Patch Set 2:\n\n(1 comment)","accounts_in_message":[],"_revision_number":2},{"id":"9e86966f17b60653cdb30e2ccf60edaaf3cda815","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2018-01-09 05:32:52.000000000","message":"Patch Set 2: Verified-1\n\nBuild failed (check pipeline).  For information on how to proceed, see\nhttp://docs.openstack.org/infra/manual/developers.html#automated-testing\n\n\n- build-openstack-sphinx-docs http://logs.openstack.org/21/530021/2/check/build-openstack-sphinx-docs/debdf31/html/ : SUCCESS in 7m 32s\n- openstack-tox-pep8 http://logs.openstack.org/21/530021/2/check/openstack-tox-pep8/af885f6/ : SUCCESS in 3m 37s\n- openstack-tox-py27 http://logs.openstack.org/21/530021/2/check/openstack-tox-py27/049bdd9/ : SUCCESS in 3m 24s\n- openstack-tox-py35 http://logs.openstack.org/21/530021/2/check/openstack-tox-py35/1ea540f/ : SUCCESS in 7m 17s\n- openstackclient-check-plugins http://logs.openstack.org/21/530021/2/check/openstackclient-check-plugins/4f07946/ : SUCCESS in 5m 26s (non-voting)\n- legacy-trove-functional-dsvm-mysql http://logs.openstack.org/21/530021/2/check/legacy-trove-functional-dsvm-mysql/9cd16f3/ : FAILURE in 2h 50m 39s\n- tempest-neutron-src-python-troveclient http://logs.openstack.org/21/530021/2/check/tempest-neutron-src-python-troveclient/a5a3585/ : SUCCESS in 1h 16m 06s","accounts_in_message":[],"_revision_number":2},{"id":"32b7a196ce7c3cf537a0ec13e4319fdd6df4db56","author":{"_account_id":8037,"name":"renminmin","email":"rmm0811@gmail.com","username":"renminmin"},"date":"2018-01-09 06:01:58.000000000","message":"Patch Set 2:\n\n(1 comment)","accounts_in_message":[],"_revision_number":2},{"id":"2502fcd7a469b60d624a8abdcd7326612b9feed6","author":{"_account_id":8037,"name":"renminmin","email":"rmm0811@gmail.com","username":"renminmin"},"date":"2018-01-09 08:02:11.000000000","message":"Uploaded patch set 3.","accounts_in_message":[],"_revision_number":3},{"id":"38dedd8dffebd9e72d3ce91bc2e3b2af9a18a0da","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2018-01-09 10:35:08.000000000","message":"Patch Set 3: Verified-1\n\nBuild failed (check pipeline).  For information on how to proceed, see\nhttp://docs.openstack.org/infra/manual/developers.html#automated-testing\n\n\n- build-openstack-sphinx-docs http://logs.openstack.org/21/530021/3/check/build-openstack-sphinx-docs/ca36baf/html/ : SUCCESS in 5m 19s\n- openstack-tox-pep8 http://logs.openstack.org/21/530021/3/check/openstack-tox-pep8/a2a163c/ : FAILURE in 4m 00s\n- openstack-tox-py27 http://logs.openstack.org/21/530021/3/check/openstack-tox-py27/db2c556/ : SUCCESS in 5m 00s\n- openstack-tox-py35 http://logs.openstack.org/21/530021/3/check/openstack-tox-py35/bac4546/ : SUCCESS in 4m 19s\n- openstackclient-check-plugins http://logs.openstack.org/21/530021/3/check/openstackclient-check-plugins/870e49b/ : SUCCESS in 7m 59s (non-voting)\n- legacy-trove-functional-dsvm-mysql http://logs.openstack.org/21/530021/3/check/legacy-trove-functional-dsvm-mysql/a2ab4fa/ : SUCCESS in 2h 30m 40s\n- tempest-neutron-src-python-troveclient http://logs.openstack.org/21/530021/3/check/tempest-neutron-src-python-troveclient/5150406/ : FAILURE in 1h 10m 07s","accounts_in_message":[],"_revision_number":3},{"id":"a98808b5aede5cd300da219ecd7491a05c6eda4c","author":{"_account_id":8037,"name":"renminmin","email":"rmm0811@gmail.com","username":"renminmin"},"date":"2018-01-10 01:13:26.000000000","message":"Uploaded patch set 4.","accounts_in_message":[],"_revision_number":4},{"id":"4341d05a342ea59feb64ec8952195121f2461948","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2018-01-10 03:43:16.000000000","message":"Patch Set 4: Verified-1\n\nBuild failed (check pipeline).  For information on how to proceed, see\nhttp://docs.openstack.org/infra/manual/developers.html#automated-testing\n\n\n- build-openstack-sphinx-docs http://logs.openstack.org/21/530021/4/check/build-openstack-sphinx-docs/d410e63/html/ : SUCCESS in 4m 40s\n- openstack-tox-pep8 http://logs.openstack.org/21/530021/4/check/openstack-tox-pep8/5b307ab/ : FAILURE in 4m 39s\n- openstack-tox-py27 http://logs.openstack.org/21/530021/4/check/openstack-tox-py27/1961b85/ : FAILURE in 17m 01s\n- openstack-tox-py35 http://logs.openstack.org/21/530021/4/check/openstack-tox-py35/9bcf708/ : FAILURE in 6m 59s\n- openstackclient-check-plugins http://logs.openstack.org/21/530021/4/check/openstackclient-check-plugins/d153963/ : FAILURE in 5m 41s (non-voting)\n- legacy-trove-functional-dsvm-mysql http://logs.openstack.org/21/530021/4/check/legacy-trove-functional-dsvm-mysql/6612b52/ : SUCCESS in 1h 54m 29s\n- tempest-neutron-src-python-troveclient http://logs.openstack.org/21/530021/4/check/tempest-neutron-src-python-troveclient/4aa93ea/ : SUCCESS in 1h 19m 01s","accounts_in_message":[],"_revision_number":4},{"id":"cd496462726c69f6a2091f87d4aea9e0a69c173d","author":{"_account_id":22608,"name":"zhanggang","email":"zhanggang@cmss.chinamobile.com","username":"zhanggang"},"date":"2018-01-12 02:41:37.000000000","message":"Patch Set 4: Code-Review-1\n\n(3 comments)","accounts_in_message":[],"_revision_number":4},{"id":"f7fe966a4d2ae604dea47269d714f974400fb55e","author":{"_account_id":22608,"name":"zhanggang","email":"zhanggang@cmss.chinamobile.com","username":"zhanggang"},"date":"2018-01-12 02:53:29.000000000","message":"Patch Set 4:\n\nBesides the test_case, aggree with you guys that the change works. And I have some doubt about method `find_resource` itself. \nThe bug appears is because find_resource treat pure-number-name as type int.\nhttps://github.com/openstack/python-troveclient/blob/master/troveclient/utils.py#L218\nI don\u0027t know why should do this thing. How about just remove the \"if...else\" code and  just retain \"name_or_id \u003d encodeutils.safe_decode(name_or_id)\"","accounts_in_message":[],"_revision_number":4},{"id":"b7278fca41cd7f534bb0c6eec7c49963ce3c20eb","author":{"_account_id":13157,"name":"Zhao Chao","email":"zhaochao1984@gmail.com","username":"zhaochao"},"date":"2018-01-12 02:59:35.000000000","message":"Patch Set 4:\n\n\u003e Besides the test_case, aggree with you guys that the change works.\n \u003e And I have some doubt about method `find_resource` itself.\n \u003e The bug appears is because find_resource treat pure-number-name as\n \u003e type int.\n \u003e https://github.com/openstack/python-troveclient/blob/master/troveclient/utils.py#L218\n \u003e I don\u0027t know why should do this thing. How about just remove the\n \u003e \"if...else\" code and  just retain \"name_or_id \u003d encodeutils.safe_decode(name_or_id)\"\n\nSee my previous comments, I also agree to just use name_or_id as string only. However safe_decode cannot deal with int, for this case we can use str() instead.","accounts_in_message":[],"_revision_number":4},{"id":"e85ee195aeacfe3cab0cb9cfd6bb292d5796348b","author":{"_account_id":22608,"name":"zhanggang","email":"zhanggang@cmss.chinamobile.com","username":"zhanggang"},"date":"2018-01-12 03:14:06.000000000","message":"Patch Set 4:\n\n\u003e \u003e Besides the test_case, aggree with you guys that the change\n \u003e works.\n \u003e \u003e And I have some doubt about method `find_resource` itself.\n \u003e \u003e The bug appears is because find_resource treat pure-number-name\n \u003e as\n \u003e \u003e type int.\n \u003e \u003e https://github.com/openstack/python-troveclient/blob/master/troveclient/utils.py#L218\n \u003e \u003e I don\u0027t know why should do this thing. How about just remove the\n \u003e \u003e \"if...else\" code and  just retain \"name_or_id \u003d encodeutils.safe_decode(name_or_id)\"\n \u003e \n \u003e See my previous comments, I also agree to just use name_or_id as\n \u003e string only. However safe_decode cannot deal with int, for this\n \u003e case we can use str() instead.\n\nI think the arg parser treat the parameter as string, and \"safe_decode(name_or_id)\" works for me. Or use \"six.text_type(name_or_id)\" for safe, this is what the author do.","accounts_in_message":[],"_revision_number":4},{"id":"ac2fdabd7fa79f379c79a07f4f17b119bab3ab57","author":{"_account_id":13157,"name":"Zhao Chao","email":"zhaochao1984@gmail.com","username":"zhaochao"},"date":"2018-01-12 03:23:12.000000000","message":"Patch Set 4:\n\n\u003e \u003e \u003e Besides the test_case, aggree with you guys that the change\n \u003e \u003e works.\n \u003e \u003e \u003e And I have some doubt about method `find_resource` itself.\n \u003e \u003e \u003e The bug appears is because find_resource treat pure-number-name\n \u003e \u003e as\n \u003e \u003e \u003e type int.\n \u003e \u003e \u003e https://github.com/openstack/python-troveclient/blob/master/troveclient/utils.py#L218\n \u003e \u003e \u003e I don\u0027t know why should do this thing. How about just remove\n \u003e the\n \u003e \u003e \u003e \"if...else\" code and  just retain \"name_or_id \u003d\n \u003e encodeutils.safe_decode(name_or_id)\"\n \u003e \u003e\n \u003e \u003e See my previous comments, I also agree to just use name_or_id as\n \u003e \u003e string only. However safe_decode cannot deal with int, for this\n \u003e \u003e case we can use str() instead.\n \u003e \n \u003e I think the arg parser treat the parameter as string, and\n \u003e \"safe_decode(name_or_id)\" works for me. Or use \"six.text_type(name_or_id)\"\n \u003e for safe, this is what the author do.\n\nYes, six.text_type may be better solution for int. In fact troveclient is not only used for command line, and also a client library, so \"name_or_id\" may be int if a client application acts that way.","accounts_in_message":[],"_revision_number":4},{"id":"213ac38fe7a7c9222749d4ac0d4705a22d0fdf2e","author":{"_account_id":8037,"name":"renminmin","email":"rmm0811@gmail.com","username":"renminmin"},"date":"2018-01-15 02:00:02.000000000","message":"Patch Set 4:\n\n(1 comment)","accounts_in_message":[],"_revision_number":4},{"id":"ae46736d1128ed72b5a1cf849b5ac038d3d56fcb","author":{"_account_id":8037,"name":"renminmin","email":"rmm0811@gmail.com","username":"renminmin"},"date":"2018-01-15 02:17:44.000000000","message":"Patch Set 4:\n\n\u003e \u003e \u003e \u003e Besides the test_case, aggree with you guys that the change\n \u003e \u003e \u003e works.\n \u003e \u003e \u003e \u003e And I have some doubt about method `find_resource` itself.\n \u003e \u003e \u003e \u003e The bug appears is because find_resource treat\n \u003e pure-number-name\n \u003e \u003e \u003e as\n \u003e \u003e \u003e \u003e type int.\n \u003e \u003e \u003e \u003e https://github.com/openstack/python-troveclient/blob/master/troveclient/utils.py#L218\n \u003e \u003e \u003e \u003e I don\u0027t know why should do this thing. How about just remove\n \u003e \u003e the\n \u003e \u003e \u003e \u003e \"if...else\" code and  just retain \"name_or_id \u003d\n \u003e \u003e encodeutils.safe_decode(name_or_id)\"\n \u003e \u003e \u003e\n \u003e \u003e \u003e See my previous comments, I also agree to just use name_or_id\n \u003e as\n \u003e \u003e \u003e string only. However safe_decode cannot deal with int, for this\n \u003e \u003e \u003e case we can use str() instead.\n \u003e \u003e\n \u003e \u003e I think the arg parser treat the parameter as string, and\n \u003e \u003e \"safe_decode(name_or_id)\" works for me. Or use \"six.text_type(name_or_id)\"\n \u003e \u003e for safe, this is what the author do.\n \u003e \n \u003e Yes, six.text_type may be better solution for int. In fact\n \u003e troveclient is not only used for command line, and also a client\n \u003e library, so \"name_or_id\" may be int if a client application acts\n \u003e that way.\n\nfind_resource is common function not only to find instance with name. \nIt cloud be find by flavor id/name and type is int or isdigit() \u003d\u003d True.","accounts_in_message":[],"_revision_number":4},{"id":"cae1edbf40acd8808d3b77a41fa1734f13e11554","author":{"_account_id":8037,"name":"renminmin","email":"rmm0811@gmail.com","username":"renminmin"},"date":"2018-01-15 02:31:19.000000000","message":"Uploaded patch set 5.","accounts_in_message":[],"_revision_number":5},{"id":"685f48c5d14125b853ee097a230a0a1522e9163b","author":{"_account_id":13157,"name":"Zhao Chao","email":"zhaochao1984@gmail.com","username":"zhaochao"},"date":"2018-01-15 02:51:40.000000000","message":"Patch Set 5:\n\n\u003e \u003e \u003e \u003e \u003e Besides the test_case, aggree with you guys that the change\n \u003e \u003e \u003e \u003e works.\n \u003e \u003e \u003e \u003e \u003e And I have some doubt about method `find_resource` itself.\n \u003e \u003e \u003e \u003e \u003e The bug appears is because find_resource treat\n \u003e \u003e pure-number-name\n \u003e \u003e \u003e \u003e as\n \u003e \u003e \u003e \u003e \u003e type int.\n \u003e \u003e \u003e \u003e \u003e https://github.com/openstack/python-troveclient/blob/master/troveclient/utils.py#L218\n \u003e \u003e \u003e \u003e \u003e I don\u0027t know why should do this thing. How about just\n \u003e remove\n \u003e \u003e \u003e the\n \u003e \u003e \u003e \u003e \u003e \"if...else\" code and  just retain \"name_or_id \u003d\n \u003e \u003e \u003e encodeutils.safe_decode(name_or_id)\"\n \u003e \u003e \u003e \u003e\n \u003e \u003e \u003e \u003e See my previous comments, I also agree to just use name_or_id\n \u003e \u003e as\n \u003e \u003e \u003e \u003e string only. However safe_decode cannot deal with int, for\n \u003e this\n \u003e \u003e \u003e \u003e case we can use str() instead.\n \u003e \u003e \u003e\n \u003e \u003e \u003e I think the arg parser treat the parameter as string, and\n \u003e \u003e \u003e \"safe_decode(name_or_id)\" works for me. Or use\n \u003e \"six.text_type(name_or_id)\"\n \u003e \u003e \u003e for safe, this is what the author do.\n \u003e \u003e\n \u003e \u003e Yes, six.text_type may be better solution for int. In fact\n \u003e \u003e troveclient is not only used for command line, and also a client\n \u003e \u003e library, so \"name_or_id\" may be int if a client application acts\n \u003e \u003e that way.\n \u003e \n \u003e find_resource is common function not only to find instance with\n \u003e name.\n \u003e It cloud be find by flavor id/name and type is int or isdigit() \u003d\u003d\n \u003e True.\n\nHi, renminmin, is there any reason that we cannot just modify the conversion lines at troveclient/utils.py:L218-L222? I think We could always convert name_or_id to a str at the begin of find_resource.","accounts_in_message":[],"_revision_number":5},{"id":"32d85bb30407ec318848b21a7c10322ad7b952a0","author":{"_account_id":8037,"name":"renminmin","email":"rmm0811@gmail.com","username":"renminmin"},"date":"2018-01-15 03:33:16.000000000","message":"Patch Set 5:\n\n\u003e \u003e \u003e \u003e \u003e \u003e Besides the test_case, aggree with you guys that the\n \u003e change\n \u003e \u003e \u003e \u003e \u003e works.\n \u003e \u003e \u003e \u003e \u003e \u003e And I have some doubt about method `find_resource`\n \u003e itself.\n \u003e \u003e \u003e \u003e \u003e \u003e The bug appears is because find_resource treat\n \u003e \u003e \u003e pure-number-name\n \u003e \u003e \u003e \u003e \u003e as\n \u003e \u003e \u003e \u003e \u003e \u003e type int.\n \u003e \u003e \u003e \u003e \u003e \u003e https://github.com/openstack/python-troveclient/blob/master/troveclient/utils.py#L218\n \u003e \u003e \u003e \u003e \u003e \u003e I don\u0027t know why should do this thing. How about just\n \u003e \u003e remove\n \u003e \u003e \u003e \u003e the\n \u003e \u003e \u003e \u003e \u003e \u003e \"if...else\" code and  just retain \"name_or_id \u003d\n \u003e \u003e \u003e \u003e encodeutils.safe_decode(name_or_id)\"\n \u003e \u003e \u003e \u003e \u003e\n \u003e \u003e \u003e \u003e \u003e See my previous comments, I also agree to just use\n \u003e name_or_id\n \u003e \u003e \u003e as\n \u003e \u003e \u003e \u003e \u003e string only. However safe_decode cannot deal with int, for\n \u003e \u003e this\n \u003e \u003e \u003e \u003e \u003e case we can use str() instead.\n \u003e \u003e \u003e \u003e\n \u003e \u003e \u003e \u003e I think the arg parser treat the parameter as string, and\n \u003e \u003e \u003e \u003e \"safe_decode(name_or_id)\" works for me. Or use\n \u003e \u003e \"six.text_type(name_or_id)\"\n \u003e \u003e \u003e \u003e for safe, this is what the author do.\n \u003e \u003e \u003e\n \u003e \u003e \u003e Yes, six.text_type may be better solution for int. In fact\n \u003e \u003e \u003e troveclient is not only used for command line, and also a\n \u003e client\n \u003e \u003e \u003e library, so \"name_or_id\" may be int if a client application\n \u003e acts\n \u003e \u003e \u003e that way.\n \u003e \u003e\n \u003e \u003e find_resource is common function not only to find instance with\n \u003e \u003e name.\n \u003e \u003e It cloud be find by flavor id/name and type is int or isdigit()\n \u003e \u003d\u003d\n \u003e \u003e True.\n \u003e \n \u003e Hi, renminmin, is there any reason that we cannot just modify the\n \u003e conversion lines at troveclient/utils.py:L218-L222? I think We\n \u003e could always convert name_or_id to a str at the begin of\n \u003e find_resource.\n\nYou\u0027re right. I will update PS","accounts_in_message":[],"_revision_number":5},{"id":"00070147df99853ef38b91d2a9826a8910746bb5","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2018-01-15 04:20:28.000000000","message":"Patch Set 5: Verified-1\n\nBuild failed (check pipeline).  For information on how to proceed, see\nhttp://docs.openstack.org/infra/manual/developers.html#automated-testing\n\n\n- build-openstack-sphinx-docs finger://ze06.openstack.org/dfbcad6c6f954fea92cd9e07313e3ae7 : POST_FAILURE in 5m 41s\n- openstack-tox-pep8 finger://ze01.openstack.org/d6d0848a020d470b84433f8cee231546 : POST_FAILURE in 4m 25s\n- openstack-tox-py27 finger://ze05.openstack.org/e1c26e6e579b4a5183d82f5bc85b84d3 : POST_FAILURE in 4m 02s\n- openstack-tox-py35 finger://ze08.openstack.org/30ba8c474d1146f1a0c99f324710c599 : POST_FAILURE in 5m 39s\n- openstackclient-check-plugins finger://ze10.openstack.org/c27c1aacd8c148ba8123e0d02f260399 : POST_FAILURE in 6m 46s (non-voting)\n- legacy-trove-functional-dsvm-mysql http://logs.openstack.org/21/530021/5/check/legacy-trove-functional-dsvm-mysql/539c2a9/ : SUCCESS in 1h 45m 54s\n- tempest-neutron-src-python-troveclient http://logs.openstack.org/21/530021/5/check/tempest-neutron-src-python-troveclient/c66cbbb/ : SUCCESS in 1h 37m 52s","accounts_in_message":[],"_revision_number":5},{"id":"69d12da9a6283e1d92379fbfce286a68e5eb3493","author":{"_account_id":8037,"name":"renminmin","email":"rmm0811@gmail.com","username":"renminmin"},"date":"2018-01-15 06:51:25.000000000","message":"Uploaded patch set 6.","accounts_in_message":[],"_revision_number":6},{"id":"2610b88203d1bc04430c41fca644901a730d916a","author":{"_account_id":13157,"name":"Zhao Chao","email":"zhaochao1984@gmail.com","username":"zhaochao"},"date":"2018-01-15 07:05:16.000000000","message":"Patch Set 6: Code-Review-1\n\n(3 comments)","accounts_in_message":[],"_revision_number":6},{"id":"f60173667b09c120eaab806da4439f151013f9b2","author":{"_account_id":8037,"name":"renminmin","email":"rmm0811@gmail.com","username":"renminmin"},"date":"2018-01-15 07:33:20.000000000","message":"Patch Set 6:\n\n(1 comment)","accounts_in_message":[],"_revision_number":6},{"id":"a1748b25d3dc3a77e532c7ee57424db4f2abb956","author":{"_account_id":8037,"name":"renminmin","email":"rmm0811@gmail.com","username":"renminmin"},"date":"2018-01-15 07:39:32.000000000","message":"Patch Set 6:\n\n(1 comment)","accounts_in_message":[],"_revision_number":6},{"id":"2b7c91e3d50b63bd8716369787debbb7d93807b7","author":{"_account_id":13157,"name":"Zhao Chao","email":"zhaochao1984@gmail.com","username":"zhaochao"},"date":"2018-01-15 07:50:20.000000000","message":"Patch Set 6:\n\n(2 comments)","accounts_in_message":[],"_revision_number":6},{"id":"48d2f04043638a7aaf246e31ed5a809011bb1407","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2018-01-15 08:39:57.000000000","message":"Patch Set 6: Verified+1\n\nBuild succeeded (check pipeline).\n\n- build-openstack-sphinx-docs http://logs.openstack.org/21/530021/6/check/build-openstack-sphinx-docs/114b96f/html/ : SUCCESS in 5m 14s\n- openstack-tox-pep8 http://logs.openstack.org/21/530021/6/check/openstack-tox-pep8/28c7aff/ : SUCCESS in 3m 55s\n- openstack-tox-py27 http://logs.openstack.org/21/530021/6/check/openstack-tox-py27/b28845d/ : SUCCESS in 4m 55s\n- openstack-tox-py35 http://logs.openstack.org/21/530021/6/check/openstack-tox-py35/5d10c95/ : SUCCESS in 4m 04s\n- openstackclient-check-plugins http://logs.openstack.org/21/530021/6/check/openstackclient-check-plugins/187cdd9/ : SUCCESS in 6m 02s (non-voting)\n- legacy-trove-functional-dsvm-mysql http://logs.openstack.org/21/530021/6/check/legacy-trove-functional-dsvm-mysql/03a0c78/ : SUCCESS in 1h 42m 57s\n- tempest-neutron-src-python-troveclient http://logs.openstack.org/21/530021/6/check/tempest-neutron-src-python-troveclient/5650628/ : SUCCESS in 1h 16m 24s","accounts_in_message":[],"_revision_number":6},{"id":"2fe4c2012bbb1e7d6c27712a2db357ef0f53b1c4","author":{"_account_id":22608,"name":"zhanggang","email":"zhanggang@cmss.chinamobile.com","username":"zhanggang"},"date":"2018-01-15 09:36:33.000000000","message":"Patch Set 6: Code-Review-1\n\n(2 comments)","accounts_in_message":[],"_revision_number":6},{"id":"9f6e4ca36d5089390e6486ec0e4e905c4cb19d58","author":{"_account_id":13157,"name":"Zhao Chao","email":"zhaochao1984@gmail.com","username":"zhaochao"},"date":"2018-01-15 10:04:03.000000000","message":"Patch Set 6:\n\n(2 comments)","accounts_in_message":[],"_revision_number":6},{"id":"c943410d32779540523b3bc6eb6d41674bff3adc","author":{"_account_id":8037,"name":"renminmin","email":"rmm0811@gmail.com","username":"renminmin"},"date":"2018-01-15 12:19:03.000000000","message":"Patch Set 6:\n\n(2 comments)","accounts_in_message":[],"_revision_number":6},{"id":"c1c39d9aa8acf4dc6be75853a3cf5a25b423a78b","author":{"_account_id":22608,"name":"zhanggang","email":"zhanggang@cmss.chinamobile.com","username":"zhanggang"},"date":"2018-01-15 12:49:16.000000000","message":"Patch Set 6: -Code-Review\n\n(2 comments)\n\nAnd remove my -1.","accounts_in_message":[],"_revision_number":6},{"id":"3a520cb792ce4769335cfc45a73e932865668f25","author":{"_account_id":8037,"name":"renminmin","email":"rmm0811@gmail.com","username":"renminmin"},"date":"2018-01-16 08:02:32.000000000","message":"Uploaded patch set 7.","accounts_in_message":[],"_revision_number":7},{"id":"0a86a72aead8c98dd3831f8a152b8c03e6bfe6c0","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2018-01-16 10:40:34.000000000","message":"Patch Set 7: Verified+1\n\nBuild succeeded (check pipeline).\n\n- build-openstack-sphinx-docs http://logs.openstack.org/21/530021/7/check/build-openstack-sphinx-docs/3a5d052/html/ : SUCCESS in 6m 33s\n- openstack-tox-pep8 http://logs.openstack.org/21/530021/7/check/openstack-tox-pep8/d030441/ : SUCCESS in 4m 46s\n- openstack-tox-py27 http://logs.openstack.org/21/530021/7/check/openstack-tox-py27/aaa667c/ : SUCCESS in 4m 14s\n- openstack-tox-py35 http://logs.openstack.org/21/530021/7/check/openstack-tox-py35/ca66625/ : SUCCESS in 4m 46s\n- openstackclient-check-plugins http://logs.openstack.org/21/530021/7/check/openstackclient-check-plugins/d153f9f/ : SUCCESS in 8m 23s (non-voting)\n- legacy-trove-functional-dsvm-mysql http://logs.openstack.org/21/530021/7/check/legacy-trove-functional-dsvm-mysql/00f172c/ : SUCCESS in 1h 55m 32s\n- tempest-neutron-src-python-troveclient http://logs.openstack.org/21/530021/7/check/tempest-neutron-src-python-troveclient/953c67c/ : SUCCESS in 1h 31m 40s","accounts_in_message":[],"_revision_number":7},{"id":"d80f6f9ab4ae64d2eda92c7c9ed96d6b1074ec5e","author":{"_account_id":22608,"name":"zhanggang","email":"zhanggang@cmss.chinamobile.com","username":"zhanggang"},"date":"2018-01-17 01:35:53.000000000","message":"Patch Set 7: Code-Review+1\n\nthe new update looks good to me, waiting for advise from others.","accounts_in_message":[],"_revision_number":7},{"id":"4873712f86d5de573a57a006bb34a45865f017e2","author":{"_account_id":13157,"name":"Zhao Chao","email":"zhaochao1984@gmail.com","username":"zhaochao"},"date":"2018-01-17 01:59:45.000000000","message":"Patch Set 7: Code-Review+1\n\nI\u0027m not insisting on the unittests, so let\u0027s make thing progress.\n\nThanks for working on this.","accounts_in_message":[],"_revision_number":7},{"id":"c1058d217071cb7fc3b837ddff644cdf4739b1dd","author":{"_account_id":25113,"name":"Fan Zhang","email":"zh.f@outlook.com","username":"fanzhang"},"date":"2018-01-17 02:27:33.000000000","message":"Patch Set 7: Code-Review+2\n\nLGTM, thanks.","accounts_in_message":[],"_revision_number":7},{"id":"34d54bfe9890c7372130a4c3c5a3f3afdd53962a","author":{"_account_id":25113,"name":"Fan Zhang","email":"zh.f@outlook.com","username":"fanzhang"},"date":"2018-01-17 02:27:35.000000000","message":"Patch Set 7: Workflow+1","accounts_in_message":[],"_revision_number":7},{"id":"1ee7d0cd055bcac7cad60a60ea6e520d4fc6cd34","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2018-01-17 02:27:47.000000000","message":"Patch Set 7: -Verified\n\nStarting gate jobs.","accounts_in_message":[],"_revision_number":7},{"id":"d46bfe72572a23f932b3ff4e71b38d66e31c0ff6","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2018-01-17 04:16:08.000000000","message":"Patch Set 7: Verified+2\n\nBuild succeeded (gate pipeline).\n\n- build-openstack-sphinx-docs http://logs.openstack.org/21/530021/7/gate/build-openstack-sphinx-docs/c4bd6bb/html/ : SUCCESS in 7m 58s\n- openstack-tox-pep8 http://logs.openstack.org/21/530021/7/gate/openstack-tox-pep8/4e94ee9/ : SUCCESS in 4m 13s\n- openstack-tox-py27 http://logs.openstack.org/21/530021/7/gate/openstack-tox-py27/890e777/ : SUCCESS in 5m 04s\n- openstack-tox-py35 http://logs.openstack.org/21/530021/7/gate/openstack-tox-py35/533592b/ : SUCCESS in 5m 53s\n- legacy-trove-functional-dsvm-mysql http://logs.openstack.org/21/530021/7/gate/legacy-trove-functional-dsvm-mysql/ba99be2/ : SUCCESS in 1h 42m 24s\n- tempest-neutron-src-python-troveclient http://logs.openstack.org/21/530021/7/gate/tempest-neutron-src-python-troveclient/97e6899/ : SUCCESS in 1h 28m 34s","accounts_in_message":[],"_revision_number":7},{"id":"575b8131dc326edd5253745b34e73bc5df3a4a62","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2018-01-17 04:16:08.000000000","message":"Change has been successfully merged by Zuul","accounts_in_message":[],"_revision_number":7}],"current_revision_number":7,"current_revision":"c346c7be7138edf4437ace2c7efdeb320d814332","revisions":{"13939e99db5c7e71983833b4decf6919c2583d6a":{"kind":"REWORK","_number":1,"created":"2017-12-25 10:39:05.000000000","uploader":{"_account_id":8037,"name":"renminmin","email":"rmm0811@gmail.com","username":"renminmin"},"ref":"refs/changes/21/530021/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-troveclient","ref":"refs/changes/21/530021/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-troveclient refs/changes/21/530021/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-troveclient refs/changes/21/530021/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-troveclient refs/changes/21/530021/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-troveclient refs/changes/21/530021/1"}}},"commit":{"parents":[{"commit":"6d8a3aa8c8ed9a77146fa94ca601129745b919b3","subject":"Merge \"Add user access related to OSC\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-troveclient/commit/6d8a3aa8c8ed9a77146fa94ca601129745b919b3"}]}],"author":{"name":"renminmin","email":"mm.ren@haihangyun.com","date":"2017-12-25 10:30:29.000000000","tz":480},"committer":{"name":"renminmin","email":"mm.ren@haihangyun.com","date":"2017-12-25 10:37:39.000000000","tz":480},"subject":"Fix show instance with integer name NotFound","message":"Fix show instance with integer name NotFound\n\nFix trove cli \u0027show\u0027 instance with integer name error\n\nChange-Id: Ic6a943b4d5980017915dd2c0361636877376c5cb\nCloses-bug: #1740015\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-troveclient/commit/13939e99db5c7e71983833b4decf6919c2583d6a"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-troveclient/commit/13939e99db5c7e71983833b4decf6919c2583d6a"}]},"branch":"refs/heads/master"},"1f9f5cd02c4fb040ea68c5e88bd3b89750396358":{"kind":"REWORK","_number":2,"created":"2018-01-09 02:40:21.000000000","uploader":{"_account_id":8037,"name":"renminmin","email":"rmm0811@gmail.com","username":"renminmin"},"ref":"refs/changes/21/530021/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-troveclient","ref":"refs/changes/21/530021/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-troveclient refs/changes/21/530021/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-troveclient refs/changes/21/530021/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-troveclient refs/changes/21/530021/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-troveclient refs/changes/21/530021/2"}}},"commit":{"parents":[{"commit":"6d8a3aa8c8ed9a77146fa94ca601129745b919b3","subject":"Merge \"Add user access related to OSC\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-troveclient/commit/6d8a3aa8c8ed9a77146fa94ca601129745b919b3"}]}],"author":{"name":"renminmin","email":"mm.ren@haihangyun.com","date":"2017-12-25 10:30:29.000000000","tz":480},"committer":{"name":"renminmin","email":"mm.ren@haihangyun.com","date":"2018-01-09 02:40:01.000000000","tz":480},"subject":"Fix show instance with integer name NotFound","message":"Fix show instance with integer name NotFound\n\nFix trove cli \u0027show\u0027 instance with integer name error\n\nChange-Id: Ic6a943b4d5980017915dd2c0361636877376c5cb\nCloses-bug: #1740015\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-troveclient/commit/1f9f5cd02c4fb040ea68c5e88bd3b89750396358"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-troveclient/commit/1f9f5cd02c4fb040ea68c5e88bd3b89750396358"}]},"branch":"refs/heads/master"},"f7489c49a618f34626cd63f136a558af4dd0bc38":{"kind":"REWORK","_number":3,"created":"2018-01-09 08:02:11.000000000","uploader":{"_account_id":8037,"name":"renminmin","email":"rmm0811@gmail.com","username":"renminmin"},"ref":"refs/changes/21/530021/3","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-troveclient","ref":"refs/changes/21/530021/3","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-troveclient refs/changes/21/530021/3 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-troveclient refs/changes/21/530021/3 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-troveclient refs/changes/21/530021/3 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-troveclient refs/changes/21/530021/3"}}},"commit":{"parents":[{"commit":"6d8a3aa8c8ed9a77146fa94ca601129745b919b3","subject":"Merge \"Add user access related to OSC\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-troveclient/commit/6d8a3aa8c8ed9a77146fa94ca601129745b919b3"}]}],"author":{"name":"renminmin","email":"mm.ren@haihangyun.com","date":"2017-12-25 10:30:29.000000000","tz":480},"committer":{"name":"renminmin","email":"mm.ren@haihangyun.com","date":"2018-01-09 08:01:48.000000000","tz":480},"subject":"Fix show instance with integer name NotFound","message":"Fix show instance with integer name NotFound\n\nFix trove cli \u0027show\u0027 instance with integer name error\n\nChange-Id: Ic6a943b4d5980017915dd2c0361636877376c5cb\nCloses-bug: #1740015\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-troveclient/commit/f7489c49a618f34626cd63f136a558af4dd0bc38"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-troveclient/commit/f7489c49a618f34626cd63f136a558af4dd0bc38"}]},"branch":"refs/heads/master"},"b6e46589e151ea7f4e80a9a2d1bb44cf9b05e087":{"kind":"REWORK","_number":4,"created":"2018-01-10 01:13:26.000000000","uploader":{"_account_id":8037,"name":"renminmin","email":"rmm0811@gmail.com","username":"renminmin"},"ref":"refs/changes/21/530021/4","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-troveclient","ref":"refs/changes/21/530021/4","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-troveclient refs/changes/21/530021/4 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-troveclient refs/changes/21/530021/4 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-troveclient refs/changes/21/530021/4 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-troveclient refs/changes/21/530021/4"}}},"commit":{"parents":[{"commit":"6d8a3aa8c8ed9a77146fa94ca601129745b919b3","subject":"Merge \"Add user access related to OSC\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-troveclient/commit/6d8a3aa8c8ed9a77146fa94ca601129745b919b3"}]}],"author":{"name":"renminmin","email":"mm.ren@haihangyun.com","date":"2017-12-25 10:30:29.000000000","tz":480},"committer":{"name":"renminmin","email":"mm.ren@haihangyun.com","date":"2018-01-10 01:13:00.000000000","tz":480},"subject":"Fix show instance with integer name NotFound","message":"Fix show instance with integer name NotFound\n\nFix trove cli \u0027show\u0027 instance with integer name error\n\nChange-Id: Ic6a943b4d5980017915dd2c0361636877376c5cb\nCloses-bug: #1740015\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-troveclient/commit/b6e46589e151ea7f4e80a9a2d1bb44cf9b05e087"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-troveclient/commit/b6e46589e151ea7f4e80a9a2d1bb44cf9b05e087"}]},"branch":"refs/heads/master"},"df44558943475db78125318d6ef1b30519b33ea1":{"kind":"REWORK","_number":5,"created":"2018-01-15 02:31:19.000000000","uploader":{"_account_id":8037,"name":"renminmin","email":"rmm0811@gmail.com","username":"renminmin"},"ref":"refs/changes/21/530021/5","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-troveclient","ref":"refs/changes/21/530021/5","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-troveclient refs/changes/21/530021/5 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-troveclient refs/changes/21/530021/5 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-troveclient refs/changes/21/530021/5 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-troveclient refs/changes/21/530021/5"}}},"commit":{"parents":[{"commit":"6d8a3aa8c8ed9a77146fa94ca601129745b919b3","subject":"Merge \"Add user access related to OSC\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-troveclient/commit/6d8a3aa8c8ed9a77146fa94ca601129745b919b3"}]}],"author":{"name":"renminmin","email":"mm.ren@haihangyun.com","date":"2017-12-25 10:30:29.000000000","tz":480},"committer":{"name":"renminmin","email":"mm.ren@haihangyun.com","date":"2018-01-15 02:30:55.000000000","tz":480},"subject":"Fix show instance with integer name NotFound","message":"Fix show instance with integer name NotFound\n\nFix trove cli \u0027show\u0027 instance with integer name error\n\nChange-Id: Ic6a943b4d5980017915dd2c0361636877376c5cb\nCloses-bug: #1740015\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-troveclient/commit/df44558943475db78125318d6ef1b30519b33ea1"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-troveclient/commit/df44558943475db78125318d6ef1b30519b33ea1"}]},"branch":"refs/heads/master"},"3a23d19f8c87797aec7adf2ac8e802ee83d82094":{"kind":"REWORK","_number":6,"created":"2018-01-15 06:51:25.000000000","uploader":{"_account_id":8037,"name":"renminmin","email":"rmm0811@gmail.com","username":"renminmin"},"ref":"refs/changes/21/530021/6","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-troveclient","ref":"refs/changes/21/530021/6","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-troveclient refs/changes/21/530021/6 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-troveclient refs/changes/21/530021/6 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-troveclient refs/changes/21/530021/6 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-troveclient refs/changes/21/530021/6"}}},"commit":{"parents":[{"commit":"8faed19f1ca7172a03ac2d79bb12fae86c204eef","subject":"Merge \"Add user-update-attributes to OSC\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-troveclient/commit/8faed19f1ca7172a03ac2d79bb12fae86c204eef"}]}],"author":{"name":"renminmin","email":"mm.ren@haihangyun.com","date":"2017-12-25 10:30:29.000000000","tz":480},"committer":{"name":"renminmin","email":"mm.ren@haihangyun.com","date":"2018-01-15 06:49:38.000000000","tz":480},"subject":"Fix show instance with integer name NotFound","message":"Fix show instance with integer name NotFound\n\nFix trove cli \u0027show\u0027 instance with integer name error\n\nChange-Id: Ic6a943b4d5980017915dd2c0361636877376c5cb\nCloses-bug: #1740015\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-troveclient/commit/3a23d19f8c87797aec7adf2ac8e802ee83d82094"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-troveclient/commit/3a23d19f8c87797aec7adf2ac8e802ee83d82094"}]},"branch":"refs/heads/master"},"c346c7be7138edf4437ace2c7efdeb320d814332":{"kind":"REWORK","_number":7,"created":"2018-01-16 08:02:32.000000000","uploader":{"_account_id":8037,"name":"renminmin","email":"rmm0811@gmail.com","username":"renminmin"},"ref":"refs/changes/21/530021/7","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-troveclient","ref":"refs/changes/21/530021/7","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-troveclient refs/changes/21/530021/7 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-troveclient refs/changes/21/530021/7 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-troveclient refs/changes/21/530021/7 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-troveclient refs/changes/21/530021/7"}}},"commit":{"parents":[{"commit":"8faed19f1ca7172a03ac2d79bb12fae86c204eef","subject":"Merge \"Add user-update-attributes to OSC\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-troveclient/commit/8faed19f1ca7172a03ac2d79bb12fae86c204eef"}]}],"author":{"name":"renminmin","email":"mm.ren@haihangyun.com","date":"2017-12-25 10:30:29.000000000","tz":480},"committer":{"name":"renminmin","email":"mm.ren@haihangyun.com","date":"2018-01-16 08:00:29.000000000","tz":480},"subject":"Fix show instance with integer name NotFound","message":"Fix show instance with integer name NotFound\n\nFix trove cli \u0027show\u0027 instance with integer name error\n\nChange-Id: Ic6a943b4d5980017915dd2c0361636877376c5cb\nCloses-bug: #1740015\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-troveclient/commit/c346c7be7138edf4437ace2c7efdeb320d814332"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-troveclient/commit/c346c7be7138edf4437ace2c7efdeb320d814332"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}