)]}'
{"id":"openstack%2Fplacement~675060","triplet_id":"openstack%2Fplacement~master~Ic39d9c65c1c15401da81725491090149d3c430ec","project":"openstack/placement","branch":"master","topic":"trivial","hashtags":[],"change_id":"Ic39d9c65c1c15401da81725491090149d3c430ec","subject":"Trivial: Remove duplicate usage of db context","status":"MERGED","created":"2019-08-07 08:49:15.000000000","updated":"2019-08-07 17:41:17.000000000","submitted":"2019-08-07 17:41:17.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":"675060-1565199677890-a273883a","meta_rev_id":"849ade1bd0cfa36899ecb954ed1a4a829f369f46","_number":675060,"virtual_id_number":675060,"owner":{"_account_id":25625,"name":"Tetsuro Nakamura","email":"tetsuro.nakamura.bc@hco.ntt.co.jp","username":"tetsuro0907"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"value":0,"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},{"value":0,"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},{"value":2,"date":"2019-08-07 17:41:17.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":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"all":[{"value":2,"date":"2019-08-07 16:05:49.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},{"value":2,"date":"2019-08-07 08:58:50.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},{"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":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"all":[{"value":1,"date":"2019-08-07 16:05:49.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},{"value":0,"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},{"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},"Review-Priority":{"all":[{"value":0,"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},{"value":0,"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"values":{" 0":"Default Priority","+1":"Contributor Review Promise","+2":"Core Review Promise"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2019-08-07 08:58:50.000000000","updated_by":{"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},"reviewer":{"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},"state":"REVIEWER"},{"updated":"2019-08-07 16:05:49.000000000","updated_by":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"reviewer":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"state":"REVIEWER"},{"updated":"2019-08-07 17:41:17.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":"79a9d8cf0a6434ab637ff4c85cd9818325761349","author":{"_account_id":25625,"name":"Tetsuro Nakamura","email":"tetsuro.nakamura.bc@hco.ntt.co.jp","username":"tetsuro0907"},"date":"2019-08-07 08:49:15.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"17dd5c6bc4063d67ac371135588647471f9fa29f","author":{"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},"date":"2019-08-07 08:58:50.000000000","message":"Patch Set 1: Code-Review+2\n\nNice, thanks for getting this.","accounts_in_message":[],"_revision_number":1},{"id":"7a0c5f4288f92cae6d46d54316556265e178d500","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-08-07 10:59:55.000000000","message":"Patch Set 1: Verified+1\n\nBuild succeeded (check pipeline).\n\n- grenade-py3 https://logs.opendev.org/60/675060/1/check/grenade-py3/7a07d47/ : SUCCESS in 1h 00m 23s\n- tempest-integrated-placement https://logs.opendev.org/60/675060/1/check/tempest-integrated-placement/207d212/ : SUCCESS in 1h 22m 44s\n- openstack-tox-docs https://logs.opendev.org/60/675060/1/check/openstack-tox-docs/0a270b4/html/ : SUCCESS in 3m 54s\n- openstack-tox-cover https://logs.opendev.org/60/675060/1/check/openstack-tox-cover/66cb1b8/cover/ : SUCCESS in 5m 45s\n- openstack-tox-lower-constraints https://logs.opendev.org/60/675060/1/check/openstack-tox-lower-constraints/aa31001/ : SUCCESS in 6m 41s\n- openstack-tox-pep8 https://logs.opendev.org/60/675060/1/check/openstack-tox-pep8/25000e9/ : SUCCESS in 5m 23s\n- openstack-tox-py27 https://logs.opendev.org/60/675060/1/check/openstack-tox-py27/65cf3bb/ : SUCCESS in 7m 24s\n- openstack-tox-py36 https://logs.opendev.org/60/675060/1/check/openstack-tox-py36/9679ecc/ : SUCCESS in 6m 58s\n- openstack-tox-py37 https://logs.opendev.org/60/675060/1/check/openstack-tox-py37/33f86bd/ : SUCCESS in 5m 07s\n- openstack-tox-functional https://logs.opendev.org/60/675060/1/check/openstack-tox-functional/624b6eb/ : SUCCESS in 4m 59s\n- openstack-tox-functional-py36 https://logs.opendev.org/60/675060/1/check/openstack-tox-functional-py36/809f8cf/ : SUCCESS in 7m 36s\n- openstack-tox-functional-py37 https://logs.opendev.org/60/675060/1/check/openstack-tox-functional-py37/9e5b338/ : SUCCESS in 6m 15s\n- placement-nova-tox-functional-py37 https://logs.opendev.org/60/675060/1/check/placement-nova-tox-functional-py37/4496e25/ : SUCCESS in 13m 26s\n- placement-nested-perfload https://logs.opendev.org/60/675060/1/check/placement-nested-perfload/ca8ead3/ : SUCCESS in 18m 35s (non-voting)\n- placement-perfload https://logs.opendev.org/60/675060/1/check/placement-perfload/193820b/ : SUCCESS in 8m 21s (non-voting)\n- tempest-ipv6-only https://logs.opendev.org/60/675060/1/check/tempest-ipv6-only/3b6e316/ : SUCCESS in 1h 04m 27s","accounts_in_message":[],"_revision_number":1},{"id":"713259d7da55fbf44d0a3752aa1c0e43fd57cfce","author":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"date":"2019-08-07 16:05:49.000000000","message":"Patch Set 1: Code-Review+2 Workflow+1\n\nyup","accounts_in_message":[],"_revision_number":1},{"id":"033c722d0d1e8a5d23116496d73ea06cc077e19e","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-08-07 16:06:10.000000000","message":"Patch Set 1: -Verified\n\nStarting gate jobs.","accounts_in_message":[],"_revision_number":1},{"id":"b04cf384a5cd1f79da5682d9f5bb74e1bb9fbf99","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-08-07 17:41:17.000000000","message":"Patch Set 1: Verified+2\n\nBuild succeeded (gate pipeline).\n\n- grenade-py3 https://logs.opendev.org/60/675060/1/gate/grenade-py3/c6cf4ba/ : SUCCESS in 52m 41s\n- tempest-integrated-placement https://logs.opendev.org/60/675060/1/gate/tempest-integrated-placement/6bbdcc9/ : SUCCESS in 1h 24m 33s\n- openstack-tox-docs https://logs.opendev.org/60/675060/1/gate/openstack-tox-docs/e18b73b/html/ : SUCCESS in 4m 29s\n- openstack-tox-lower-constraints https://logs.opendev.org/60/675060/1/gate/openstack-tox-lower-constraints/73cbde5/ : SUCCESS in 8m 02s\n- openstack-tox-pep8 https://logs.opendev.org/60/675060/1/gate/openstack-tox-pep8/6b195eb/ : SUCCESS in 4m 29s\n- openstack-tox-py27 https://logs.opendev.org/60/675060/1/gate/openstack-tox-py27/248cb1d/ : SUCCESS in 5m 43s\n- openstack-tox-py36 https://logs.opendev.org/60/675060/1/gate/openstack-tox-py36/9b75a5c/ : SUCCESS in 4m 59s\n- openstack-tox-py37 https://logs.opendev.org/60/675060/1/gate/openstack-tox-py37/cb33f21/ : SUCCESS in 6m 56s\n- openstack-tox-functional https://logs.opendev.org/60/675060/1/gate/openstack-tox-functional/0337fa2/ : SUCCESS in 7m 24s\n- openstack-tox-functional-py36 https://logs.opendev.org/60/675060/1/gate/openstack-tox-functional-py36/4596963/ : SUCCESS in 8m 23s\n- openstack-tox-functional-py37 https://logs.opendev.org/60/675060/1/gate/openstack-tox-functional-py37/2e82bb6/ : SUCCESS in 6m 10s\n- placement-nova-tox-functional-py37 https://logs.opendev.org/60/675060/1/gate/placement-nova-tox-functional-py37/9d283fc/ : SUCCESS in 15m 01s\n- tempest-ipv6-only https://logs.opendev.org/60/675060/1/gate/tempest-ipv6-only/0523d05/ : SUCCESS in 1h 03m 36s","accounts_in_message":[],"_revision_number":1},{"id":"ac7e61f6f82784f2c3946ccc25a7ac09f33a6233","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-08-07 17:41:17.000000000","message":"Change has been successfully merged by Zuul","accounts_in_message":[],"_revision_number":1}],"current_revision_number":1,"current_revision":"d6c151f277c061506f450594b5b13ad16a2e9672","revisions":{"d6c151f277c061506f450594b5b13ad16a2e9672":{"kind":"REWORK","_number":1,"created":"2019-08-07 08:49:15.000000000","uploader":{"_account_id":25625,"name":"Tetsuro Nakamura","email":"tetsuro.nakamura.bc@hco.ntt.co.jp","username":"tetsuro0907"},"ref":"refs/changes/60/675060/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/placement","ref":"refs/changes/60/675060/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/placement refs/changes/60/675060/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/placement refs/changes/60/675060/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/placement refs/changes/60/675060/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/placement refs/changes/60/675060/1"}}},"commit":{"parents":[{"commit":"5f6393312bedd5aa9f5866c83f7bcc61ccddbd54","subject":"Merge \"Use TraitCache for Trait.get_by_name\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/placement/commit/5f6393312bedd5aa9f5866c83f7bcc61ccddbd54"}]}],"author":{"name":"Tetsuro Nakamura","email":"tetsuro.nakamura.bc@hco.ntt.co.jp","date":"2019-08-06 08:54:32.000000000","tz":0},"committer":{"name":"Tetsuro Nakamura","email":"tetsuro.nakamura.bc@hco.ntt.co.jp","date":"2019-08-06 08:54:32.000000000","tz":0},"subject":"Trivial: Remove duplicate usage of db context","message":"Trivial: Remove duplicate usage of db context\n\nThe change to use cache in Trait object [1] moved the raw database\nquery to deeper layer from ids_from_names() function, but it was\nstill decorated with the db context manager, so this patch removes it.\n\n[1] Ia19ea2b4ecdde25323579edf60ad6269d05e75a2\n\nChange-Id: Ic39d9c65c1c15401da81725491090149d3c430ec\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/placement/commit/d6c151f277c061506f450594b5b13ad16a2e9672"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/placement/commit/d6c151f277c061506f450594b5b13ad16a2e9672"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}