)]}'
{"id":"openstack%2Fpython-openstackclient~519385","triplet_id":"openstack%2Fpython-openstackclient~master~Idd8c15255f024bba7079d3a9a29545dec0c91b58","project":"openstack/python-openstackclient","branch":"master","hashtags":[],"change_id":"Idd8c15255f024bba7079d3a9a29545dec0c91b58","subject":"Replace %r with %s on printing string variable","status":"MERGED","created":"2017-11-13 15:24:05.000000000","updated":"2017-11-28 21:35:55.000000000","submitted":"2017-11-28 21:35:55.000000000","submitter":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"submission_id":"519385-1511904955008-9becd347","meta_rev_id":"9a896c6ae70aeb8f17ac37b2cc95267960ad8e7f","_number":519385,"virtual_id_number":519385,"owner":{"_account_id":11536,"name":"hongbin","email":"hongbin034@gmail.com","username":"hongbin"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"value":0,"date":"2017-11-14 00:19:06.000000000","_account_id":11536,"name":"hongbin","email":"hongbin034@gmail.com","username":"hongbin"},{"value":0,"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},{"value":0,"_account_id":170,"name":"Mike Perez","email":"thingee@gmail.com","username":"thingee"},{"value":0,"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"value":2,"date":"2017-11-28 21:35:54.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":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"all":[{"value":0,"_account_id":11536,"name":"hongbin","email":"hongbin034@gmail.com","username":"hongbin"},{"value":2,"date":"2017-11-14 05:59:51.000000000","_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},{"value":1,"date":"2017-11-23 00:17:19.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":170,"name":"Mike Perez","email":"thingee@gmail.com","username":"thingee"},{"value":2,"date":"2017-11-28 20:06:06.000000000","_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"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":"","default_value":0,"optional":true},"Workflow":{"approved":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"all":[{"value":0,"_account_id":11536,"name":"hongbin","email":"hongbin034@gmail.com","username":"hongbin"},{"value":0,"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},{"value":0,"_account_id":170,"name":"Mike Perez","email":"thingee@gmail.com","username":"thingee"},{"value":1,"date":"2017-11-28 20:06:06.000000000","_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"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}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":170,"name":"Mike Perez","email":"thingee@gmail.com","username":"thingee"},{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},{"_account_id":11536,"name":"hongbin","email":"hongbin034@gmail.com","username":"hongbin"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2017-11-14 05:59:51.000000000","updated_by":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"reviewer":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"state":"REVIEWER"},{"updated":"2017-11-23 00:17:19.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":"2017-11-28 20:06:06.000000000","updated_by":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"reviewer":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"state":"REVIEWER"},{"updated":"2017-11-28 21:35:54.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":"722e998a66d021bb85b4467b2e80daab03479882","author":{"_account_id":11536,"name":"hongbin","email":"hongbin034@gmail.com","username":"hongbin"},"date":"2017-11-13 15:24:05.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"92b8b94c8e188aaab22f6820e5e9c180a4120fd6","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2017-11-13 18:08:04.000000000","message":"Patch Set 1: Verified+1\n\nBuild succeeded (check pipeline).\n\n- openstack-tox-pep8 http://logs.openstack.org/85/519385/1/check/openstack-tox-pep8/8d6df02/ : SUCCESS in 8m 19s\n- openstack-tox-py27 http://logs.openstack.org/85/519385/1/check/openstack-tox-py27/ac91002/ : SUCCESS in 5m 38s\n- openstack-tox-py35 http://logs.openstack.org/85/519385/1/check/openstack-tox-py35/5d70180/ : SUCCESS in 5m 31s\n- build-openstack-sphinx-docs http://logs.openstack.org/85/519385/1/check/build-openstack-sphinx-docs/09b75f8/html/ : SUCCESS in 6m 53s\n- legacy-tempest-dsvm-neutron-src http://logs.openstack.org/85/519385/1/check/legacy-tempest-dsvm-neutron-src/4d49436/ : SUCCESS in 1h 12m 15s\n- legacy-tripleo-ci-centos-7-nonha-multinode-oooq http://logs.openstack.org/85/519385/1/check/legacy-tripleo-ci-centos-7-nonha-multinode-oooq/34ba929/ : SUCCESS in 2h 34m 41s (non-voting)\n- openstackclient-check-plugins http://logs.openstack.org/85/519385/1/check/openstackclient-check-plugins/da1098d/ : SUCCESS in 11m 10s (non-voting)\n- osc-tox-unit-tips http://logs.openstack.org/85/519385/1/check/osc-tox-unit-tips/d4964ee/ : SUCCESS in 6m 39s\n- osc-functional-devstack http://logs.openstack.org/85/519385/1/check/osc-functional-devstack/4d901ff/ : SUCCESS in 37m 55s\n- osc-functional-devstack-tips http://logs.openstack.org/85/519385/1/check/osc-functional-devstack-tips/3436a4e/ : SUCCESS in 46m 23s (non-voting)","accounts_in_message":[],"_revision_number":1},{"id":"2fb3db7f5a55c1b928b1961c6c7a578534f45d46","author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"date":"2017-11-13 22:09:40.000000000","message":"Patch Set 1: Code-Review+2","accounts_in_message":[],"_revision_number":1},{"id":"839c9dac22919311b91004eb3e25bd5f91aec962","author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"date":"2017-11-13 22:10:40.000000000","message":"Patch Set 1: -Code-Review\n\nactually, we may want to use %r here, since it will give more data https://stackoverflow.com/questions/6005159/when-to-use-r-instead-of-s-in-python","accounts_in_message":[],"_revision_number":1},{"id":"293b17652b14f6ee403af339a82cd04dcb940f1f","author":{"_account_id":11536,"name":"hongbin","email":"hongbin034@gmail.com","username":"hongbin"},"date":"2017-11-14 00:19:06.000000000","message":"Patch Set 1:\n\n\u003e actually, we may want to use %r here, since it will give more data\n \u003e https://stackoverflow.com/questions/6005159/when-to-use-r-instead-of-s-in-python\n\nAs pointed out by reviewer in another patch (https://review.openstack.org/#/c/510308/), this variable is the name of an entry point which is a string so I think using %r won\u0027t give more data in here.","accounts_in_message":[],"_revision_number":1},{"id":"a2c88ca17ff637afb4086f93419959d7376ed3f1","author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"date":"2017-11-14 05:59:51.000000000","message":"Patch Set 1: Code-Review+2\n\n\u003e \u003e actually, we may want to use %r here, since it will give more\n \u003e data\n \u003e \u003e https://stackoverflow.com/questions/6005159/when-to-use-r-instead-of-s-in-python\n \u003e \n \u003e As pointed out by reviewer in another patch (https://review.openstack.org/#/c/510308/),\n \u003e this variable is the name of an entry point which is a string so I\n \u003e think using %r won\u0027t give more data in here.\n\nthanks for the explanation!","accounts_in_message":[],"_revision_number":1},{"id":"aaa245879457d0f95087b2361090d4f67ae0ccf3","author":{"_account_id":170,"name":"Mike Perez","email":"thingee@gmail.com","username":"thingee"},"date":"2017-11-23 00:17:19.000000000","message":"Patch Set 1: Code-Review+1","accounts_in_message":[],"_revision_number":1},{"id":"29756711901285138d84d29002dce19d094ad4a0","author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"date":"2017-11-28 20:06:06.000000000","message":"Patch Set 1: Workflow+1 Code-Review+2","accounts_in_message":[],"_revision_number":1},{"id":"7d7f74b243a6827222fdacad9f3a8b96e6ddd37a","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2017-11-28 20:06:18.000000000","message":"Patch Set 1: -Verified\n\nStarting gate jobs.","accounts_in_message":[],"_revision_number":1},{"id":"87ade237f0bdbdba6e8fce458ab8f9c287de190c","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2017-11-28 21:35:54.000000000","message":"Patch Set 1: Verified+2\n\nBuild succeeded (gate pipeline).\n\n- openstack-tox-pep8 http://logs.openstack.org/85/519385/1/gate/openstack-tox-pep8/2f1d8e5/ : SUCCESS in 5m 01s\n- openstack-tox-py27 http://logs.openstack.org/85/519385/1/gate/openstack-tox-py27/3bdb702/ : SUCCESS in 4m 35s\n- openstack-tox-py35 http://logs.openstack.org/85/519385/1/gate/openstack-tox-py35/38a90c6/ : SUCCESS in 5m 11s\n- build-openstack-sphinx-docs http://logs.openstack.org/85/519385/1/gate/build-openstack-sphinx-docs/0e36aea/html/ : SUCCESS in 5m 26s\n- legacy-tempest-dsvm-neutron-src http://logs.openstack.org/85/519385/1/gate/legacy-tempest-dsvm-neutron-src/d4f118d/ : SUCCESS in 1h 22m 07s\n- osc-functional-devstack http://logs.openstack.org/85/519385/1/gate/osc-functional-devstack/79bca39/ : SUCCESS in 31m 53s","accounts_in_message":[],"_revision_number":1},{"id":"9a896c6ae70aeb8f17ac37b2cc95267960ad8e7f","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2017-11-28 21:35:55.000000000","message":"Change has been successfully merged by Zuul","accounts_in_message":[],"_revision_number":1}],"current_revision_number":1,"current_revision":"7ef2867ff61d6e4d468b80a9284fdb0bd5bdb6c9","revisions":{"7ef2867ff61d6e4d468b80a9284fdb0bd5bdb6c9":{"kind":"REWORK","_number":1,"created":"2017-11-13 15:24:05.000000000","uploader":{"_account_id":11536,"name":"hongbin","email":"hongbin034@gmail.com","username":"hongbin"},"ref":"refs/changes/85/519385/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-openstackclient","ref":"refs/changes/85/519385/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-openstackclient refs/changes/85/519385/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-openstackclient refs/changes/85/519385/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-openstackclient refs/changes/85/519385/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-openstackclient refs/changes/85/519385/1"}}},"commit":{"parents":[{"commit":"4742d4df7089cd10d03635a1b3dbca9e7e80b1cc","subject":"Updated from global requirements","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-openstackclient/commit/4742d4df7089cd10d03635a1b3dbca9e7e80b1cc"}]}],"author":{"name":"Hongbin Lu","email":"hongbin.lu@huawei.com","date":"2017-11-13 15:23:16.000000000","tz":0},"committer":{"name":"Hongbin Lu","email":"hongbin.lu@huawei.com","date":"2017-11-13 15:23:16.000000000","tz":0},"subject":"Replace %r with %s on printing string variable","message":"Replace %r with %s on printing string variable\n\nChange-Id: Idd8c15255f024bba7079d3a9a29545dec0c91b58\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-openstackclient/commit/7ef2867ff61d6e4d468b80a9284fdb0bd5bdb6c9"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-openstackclient/commit/7ef2867ff61d6e4d468b80a9284fdb0bd5bdb6c9"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}