)]}'
{"id":"openstack%2Ftripleo-validations~463726","triplet_id":"openstack%2Ftripleo-validations~stable%2Focata~I751400d414d17a39715aa33d0a8b323ed1211643","project":"openstack/tripleo-validations","branch":"stable/ocata","topic":"bug/1689789","hashtags":[],"change_id":"I751400d414d17a39715aa33d0a8b323ed1211643","subject":"Improve performance when fetching outputs","status":"MERGED","created":"2017-05-10 11:43:15.000000000","updated":"2017-05-10 13:58:09.000000000","submitted":"2017-05-10 13:58:09.000000000","submitter":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"caf13d745647114774b03b1d0aac5c34167f0f16","_number":463726,"virtual_id_number":463726,"owner":{"_account_id":17888,"name":"Florian Fuchs","email":"f@florianfuchs.com","username":"flfuchs"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"value":0,"_account_id":20172,"name":"Michele Baldessari","email":"michele@acksyn.org","username":"michele"},{"value":0,"_account_id":13039,"name":"Martin André","email":"m.andre@redhat.com","username":"mandre"},{"value":0,"_account_id":10873,"name":"Juan Antonio Osorio Robles","email":"jaosorior@redhat.com","username":"ejuaoso"},{"value":0,"_account_id":17888,"name":"Florian Fuchs","email":"f@florianfuchs.com","username":"flfuchs"},{"value":2,"date":"2017-05-10 13:58:09.000000000","post_submit":true,"_account_id":3,"name":"Jenkins","username":"jenkins"}],"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":20172,"name":"Michele Baldessari","email":"michele@acksyn.org","username":"michele"},"all":[{"value":2,"date":"2017-05-10 13:36:00.000000000","_account_id":20172,"name":"Michele Baldessari","email":"michele@acksyn.org","username":"michele"},{"value":2,"date":"2017-05-10 13:54:42.000000000","_account_id":13039,"name":"Martin André","email":"m.andre@redhat.com","username":"mandre"},{"value":0,"_account_id":10873,"name":"Juan Antonio Osorio Robles","email":"jaosorior@redhat.com","username":"ejuaoso"},{"value":0,"_account_id":17888,"name":"Florian Fuchs","email":"f@florianfuchs.com","username":"flfuchs"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"}],"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":13039,"name":"Martin André","email":"m.andre@redhat.com","username":"mandre"},"all":[{"value":0,"_account_id":20172,"name":"Michele Baldessari","email":"michele@acksyn.org","username":"michele"},{"value":1,"date":"2017-05-10 13:54:42.000000000","_account_id":13039,"name":"Martin André","email":"m.andre@redhat.com","username":"mandre"},{"value":0,"_account_id":10873,"name":"Juan Antonio Osorio Robles","email":"jaosorior@redhat.com","username":"ejuaoso"},{"value":0,"date":"2017-05-10 13:36:58.000000000","_account_id":17888,"name":"Florian Fuchs","email":"f@florianfuchs.com","username":"flfuchs"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":10873,"name":"Juan Antonio Osorio Robles","email":"jaosorior@redhat.com","username":"ejuaoso"},{"_account_id":13039,"name":"Martin André","email":"m.andre@redhat.com","username":"mandre"},{"_account_id":17888,"name":"Florian Fuchs","email":"f@florianfuchs.com","username":"flfuchs"},{"_account_id":20172,"name":"Michele Baldessari","email":"michele@acksyn.org","username":"michele"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2017-05-10 13:06:04.000000000","updated_by":{"_account_id":10873,"name":"Juan Antonio Osorio Robles","email":"jaosorior@redhat.com","username":"ejuaoso"},"reviewer":{"_account_id":10873,"name":"Juan Antonio Osorio Robles","email":"jaosorior@redhat.com","username":"ejuaoso"},"state":"REVIEWER"},{"updated":"2017-05-10 13:36:00.000000000","updated_by":{"_account_id":20172,"name":"Michele Baldessari","email":"michele@acksyn.org","username":"michele"},"reviewer":{"_account_id":20172,"name":"Michele Baldessari","email":"michele@acksyn.org","username":"michele"},"state":"REVIEWER"},{"updated":"2017-05-10 13:54:42.000000000","updated_by":{"_account_id":13039,"name":"Martin André","email":"m.andre@redhat.com","username":"mandre"},"reviewer":{"_account_id":13039,"name":"Martin André","email":"m.andre@redhat.com","username":"mandre"},"state":"REVIEWER"},{"updated":"2017-05-10 13:58:09.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"}],"messages":[{"id":"2ceeebebb275cf0b96ab3a8b326bb9c903ccf669","author":{"_account_id":17888,"name":"Florian Fuchs","email":"f@florianfuchs.com","username":"flfuchs"},"date":"2017-05-10 11:43:15.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"db6f85f77089659beffa598949703c257e96a6db","author":{"_account_id":17888,"name":"Florian Fuchs","email":"f@florianfuchs.com","username":"flfuchs"},"date":"2017-05-10 11:43:52.000000000","message":"Patch Set 1: Workflow-1\n\nNot ready to merge, since it hasn\u0027t been tested in an ocata environment yet.","accounts_in_message":[],"_revision_number":1},{"id":"bbcfc65a4a3b1c52f4177567973ef3e642dbfc36","author":{"_account_id":17888,"name":"Florian Fuchs","email":"f@florianfuchs.com","username":"flfuchs"},"date":"2017-05-10 11:45:31.000000000","message":"Topic changed from stable-ocata to inventory-performance","accounts_in_message":[],"_revision_number":1},{"id":"ee16fa703d9f215886e5e09ceafd5245f26064bf","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2017-05-10 11:47:45.000000000","message":"Patch Set 1: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-tripleo-validations-docs-ubuntu-xenial http://docs-draft.openstack.org/26/463726/1/check/gate-tripleo-validations-docs-ubuntu-xenial/7856630//doc/build/html/ : SUCCESS in 2m 23s\n- gate-tripleo-validations-pep8-ubuntu-xenial http://logs.openstack.org/26/463726/1/check/gate-tripleo-validations-pep8-ubuntu-xenial/26b4fac/ : SUCCESS in 3m 44s\n- gate-tripleo-validations-python27-ubuntu-xenial http://logs.openstack.org/26/463726/1/check/gate-tripleo-validations-python27-ubuntu-xenial/5dbcaf2/ : SUCCESS in 3m 45s","accounts_in_message":[],"_revision_number":1},{"id":"24e109045f1a1c53947b8541f5f962b505841dbf","author":{"_account_id":20172,"name":"Michele Baldessari","email":"michele@acksyn.org","username":"michele"},"date":"2017-05-10 13:01:46.000000000","message":"Patch Set 1: Code-Review+1\n\n# Before the patch\n\n(undercloud) [stack@undercloud ~]$ time tripleo-ansible-inventory --list                                                                                                                      \nn{\"overcloud\": {\"children\": [\"controller\", \"compute\"], \"vars\": {\"ansible_ssh_user\": \"heat-admin\"}}, \"controller\": [\"192.168.24.14\", \"192.168.24.18\", \"192.168.24.12\"], \"undercloud\": {\"hosts\": [\"localhost\"], \"vars\": {\"overcloud_horizon_url\": \"http://[2001:db8:fd00:1000::19]:80/dashboard\", \"overcloud_admin_password\": \"sPCZZYxzJv6XTCfWRGdavJbB9\", \"ansible_connection\": \"local\"}}, \"compute\": [\"192.168.24.10\"]}\n\nreal    1m3.639s\nuser    0m1.868s\nsys     0m0.112s\n\n# After the patch\n(undercloud) [stack@undercloud ~]$ time tripleo-ansible-inventory --list\n{\"overcloud\": {\"children\": [\"controller\", \"compute\"], \"vars\": {\"ansible_ssh_user\": \"heat-admin\"}}, \"controller\": [\"192.168.24.14\", \"192.168.24.18\", \"192.168.24.12\"], \"undercloud\": {\"hosts\": [\"localhost\"], \"vars\": {\"overcloud_horizon_url\": \"http://[2001:db8:fd00:1000::19]:80/dashboard\", \"overcloud_admin_password\": \"sPCZZYxzJv6XTCfWRGdavJbB9\", \"ansible_connection\": \"local\"}}, \"compute\": [\"192.168.24.10\"]}\n\nreal    0m26.892s\nuser    0m1.885s\nsys     0m0.101s\n\n\nVery nice improvement ;)","accounts_in_message":[],"_revision_number":1},{"id":"9e973f95892775e0359f981b3f72eaf2046c3a2d","author":{"_account_id":10873,"name":"Juan Antonio Osorio Robles","email":"jaosorior@redhat.com","username":"ejuaoso"},"date":"2017-05-10 13:06:04.000000000","message":"Patch Set 1: Code-Review-1\n\nLets add https://bugs.launchpad.net/tripleo/+bug/1689789 as a bug here","accounts_in_message":[],"_revision_number":1},{"id":"2acede9de6f8bd9c05a4a4625032a191ad4028ca","author":{"_account_id":20172,"name":"Michele Baldessari","email":"michele@acksyn.org","username":"michele"},"date":"2017-05-10 13:12:17.000000000","message":"Patch Set 1:\n\n\u003e Lets add https://bugs.launchpad.net/tripleo/+bug/1689789 as a bug\n \u003e here\n\nMaybe as a Partial-Bug? I think having an option to dump the inventory as yaml still has some merit","accounts_in_message":[],"_revision_number":1},{"id":"74065ab75ea381abcf1e38696700a943c47c24e6","author":{"_account_id":17888,"name":"Florian Fuchs","email":"f@florianfuchs.com","username":"flfuchs"},"date":"2017-05-10 13:18:56.000000000","message":"Uploaded patch set 2: Commit message was updated.","accounts_in_message":[],"_revision_number":2},{"id":"938987a7280d09ea582694c89b7b2acabc68074b","author":{"_account_id":17888,"name":"Florian Fuchs","email":"f@florianfuchs.com","username":"flfuchs"},"date":"2017-05-10 13:19:15.000000000","message":"Patch Set 2: Workflow-1","accounts_in_message":[],"_revision_number":2},{"id":"a8fc2f872e09674597e28bcfe99c240cefba96d1","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2017-05-10 13:23:26.000000000","message":"Patch Set 2: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-tripleo-validations-docs-ubuntu-xenial http://docs-draft.openstack.org/26/463726/2/check/gate-tripleo-validations-docs-ubuntu-xenial/7b6bbe5//doc/build/html/ : SUCCESS in 1m 51s\n- gate-tripleo-validations-pep8-ubuntu-xenial http://logs.openstack.org/26/463726/2/check/gate-tripleo-validations-pep8-ubuntu-xenial/988fa0a/ : SUCCESS in 3m 51s\n- gate-tripleo-validations-python27-ubuntu-xenial http://logs.openstack.org/26/463726/2/check/gate-tripleo-validations-python27-ubuntu-xenial/c7ab20a/ : SUCCESS in 2m 46s","accounts_in_message":[],"_revision_number":2},{"id":"b1186cd6609cee5b1431d10f2755be584239fce6","author":{"_account_id":20172,"name":"Michele Baldessari","email":"michele@acksyn.org","username":"michele"},"date":"2017-05-10 13:36:00.000000000","message":"Patch Set 2: Code-Review+2\n\nTested on ocata as well:\n[stack@hac-01 ~]$ time tripleo-ansible-inventory --list\n{\"overcloud\": {\"children\": [\"controller\", \"compute\"], \"vars\": {\"ansible_ssh_user\": \"heat-admin\"}}, \"controller\": [\"192.168.24.16\", \"192.168.24.17\", \"192.168.24.19\"], \"undercloud\": {\"hosts\": [\"localhost\"], \"vars\": {\"overcloud_admin_password\": \"pTFs6Z7DNFVHZeDucU6huF899\", \"ansible_connection\": \"local\"}}, \"compute\": [\"192.168.24.14\", \"192.168.24.6\"]}\n\nreal\t0m27.724s\nuser\t0m0.467s\nsys\t0m0.051s\n\n\nPreviously the time as ~1minute","accounts_in_message":[],"_revision_number":2},{"id":"9320cdbd30d4f7fdc1023b55c3a7e86a4ba815bc","author":{"_account_id":17888,"name":"Florian Fuchs","email":"f@florianfuchs.com","username":"flfuchs"},"date":"2017-05-10 13:36:58.000000000","message":"Patch Set 2: -Workflow","accounts_in_message":[],"_revision_number":2},{"id":"529d83ca3784e3530cc7e2e2b959c26083ad1c62","author":{"_account_id":13039,"name":"Martin André","email":"m.andre@redhat.com","username":"mandre"},"date":"2017-05-10 13:54:42.000000000","message":"Patch Set 2: Code-Review+2 Workflow+1","accounts_in_message":[],"_revision_number":2},{"id":"e4edf597f5fe18fde079fefd11b77776feebbc36","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2017-05-10 13:54:52.000000000","message":"Patch Set 2: -Verified\n\nStarting gate jobs.\nhttp://status.openstack.org/zuul/","accounts_in_message":[],"_revision_number":2},{"id":"9be62fad1409a11dc20927aec15f59950b8fa407","date":"2017-05-10 13:58:09.000000000","message":"Change has been successfully merged into the git repository by Jenkins","accounts_in_message":[],"_revision_number":2},{"id":"3b20bba70e0646878651e83efb56e94623c422b1","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2017-05-10 13:58:09.000000000","message":"Patch Set 2: Verified+2\n\nBuild succeeded (gate pipeline).\n\n- gate-tripleo-validations-docs-ubuntu-xenial http://docs-draft.openstack.org/26/463726/2/gate/gate-tripleo-validations-docs-ubuntu-xenial/8e1e16a//doc/build/html/ : SUCCESS in 2m 32s\n- gate-tripleo-validations-pep8-ubuntu-xenial http://logs.openstack.org/26/463726/2/gate/gate-tripleo-validations-pep8-ubuntu-xenial/833b45e/ : SUCCESS in 2m 10s\n- gate-tripleo-validations-python27-ubuntu-xenial http://logs.openstack.org/26/463726/2/gate/gate-tripleo-validations-python27-ubuntu-xenial/80c4d1c/ : SUCCESS in 3m 09s","accounts_in_message":[],"_revision_number":2}],"current_revision_number":2,"current_revision":"54d7ff11d99939933bbd41e9c6e7fffd39a8930b","revisions":{"cf37d6ece1b92201ca9974dfb82662301b8a10d4":{"kind":"REWORK","_number":1,"created":"2017-05-10 11:43:15.000000000","uploader":{"_account_id":17888,"name":"Florian Fuchs","email":"f@florianfuchs.com","username":"flfuchs"},"ref":"refs/changes/26/463726/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/tripleo-validations","ref":"refs/changes/26/463726/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/tripleo-validations refs/changes/26/463726/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/tripleo-validations refs/changes/26/463726/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/tripleo-validations refs/changes/26/463726/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/tripleo-validations refs/changes/26/463726/1"}}},"commit":{"parents":[{"commit":"3ddf186f7a68e67f4448b934d6cbd6350a68417c","subject":"Add \u0027become: true\u0027 to haproxy validation","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/tripleo-validations/commit/3ddf186f7a68e67f4448b934d6cbd6350a68417c"}]}],"author":{"name":"Florian Fuchs","email":"flfuchs@redhat.com","date":"2017-05-10 11:39:16.000000000","tz":0},"committer":{"name":"Florian Fuchs","email":"flfuchs@redhat.com","date":"2017-05-10 11:39:16.000000000","tz":0},"subject":"Improve performance when fetching outputs","message":"Improve performance when fetching outputs\n\nStack outputs where fetched via `heat_client.stacks.get(...)` which\nproved to be fairly slow.\n\nThis patch changes output fetching to use\n`heat_client.stacks.output_show(...)` instead.\n\nChange-Id: I751400d414d17a39715aa33d0a8b323ed1211643\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/tripleo-validations/commit/cf37d6ece1b92201ca9974dfb82662301b8a10d4"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/tripleo-validations/commit/cf37d6ece1b92201ca9974dfb82662301b8a10d4"}]},"branch":"refs/heads/stable/ocata"},"54d7ff11d99939933bbd41e9c6e7fffd39a8930b":{"kind":"NO_CODE_CHANGE","_number":2,"created":"2017-05-10 13:18:56.000000000","uploader":{"_account_id":17888,"name":"Florian Fuchs","email":"f@florianfuchs.com","username":"flfuchs"},"ref":"refs/changes/26/463726/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/tripleo-validations","ref":"refs/changes/26/463726/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/tripleo-validations refs/changes/26/463726/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/tripleo-validations refs/changes/26/463726/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/tripleo-validations refs/changes/26/463726/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/tripleo-validations refs/changes/26/463726/2"}}},"commit":{"parents":[{"commit":"3ddf186f7a68e67f4448b934d6cbd6350a68417c","subject":"Add \u0027become: true\u0027 to haproxy validation","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/tripleo-validations/commit/3ddf186f7a68e67f4448b934d6cbd6350a68417c"}]}],"author":{"name":"Florian Fuchs","email":"flfuchs@redhat.com","date":"2017-05-10 11:39:16.000000000","tz":0},"committer":{"name":"Florian Fuchs","email":"flfuchs@redhat.com","date":"2017-05-10 13:18:27.000000000","tz":0},"subject":"Improve performance when fetching outputs","message":"Improve performance when fetching outputs\n\nStack outputs where fetched via `heat_client.stacks.get(...)` which\nproved to be fairly slow.\n\nThis patch changes output fetching to use\n`heat_client.stacks.output_show(...)` instead.\n\nPartial-Bug: #1689789\nChange-Id: I751400d414d17a39715aa33d0a8b323ed1211643\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/tripleo-validations/commit/54d7ff11d99939933bbd41e9c6e7fffd39a8930b"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/tripleo-validations/commit/54d7ff11d99939933bbd41e9c6e7fffd39a8930b"}]},"branch":"refs/heads/stable/ocata"}},"requirements":[],"submit_records":[],"submit_requirements":[]}