)]}'
{"id":"openstack%2Fopenstack-ansible-os_ceilometer~463811","triplet_id":"openstack%2Fopenstack-ansible-os_ceilometer~master~Iae293bf96e31b04cb89a0f12963602e6030fe697","project":"openstack/openstack-ansible-os_ceilometer","branch":"master","topic":"venv_checksum_fix","hashtags":[],"change_id":"Iae293bf96e31b04cb89a0f12963602e6030fe697","subject":"Fix for lookup not detecting errors and happening from wrong place","status":"ABANDONED","created":"2017-05-10 15:41:00.000000000","updated":"2017-06-18 13:53:12.000000000","total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"9df3fe90c0d86f9074e708e22d62b6a3ff41d5f3","_number":463811,"virtual_id_number":463811,"owner":{"_account_id":25022,"name":"Rick Box","email":"box.rick@gmail.com","username":"rickbox"},"actions":{},"labels":{"Verified":{"disliked":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"},{"value":-1,"date":"2017-05-10 16:21:05.000000000","_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":"","value":-1,"default_value":0,"optional":true},"Code-Review":{"disliked":{"_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"},"all":[{"value":-1,"date":"2017-05-15 07:45:24.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_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":"","value":-1,"default_value":0,"optional":true},"Workflow":{"all":[{"_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"},{"_account_id":3,"name":"Jenkins","username":"jenkins"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true},"Backport-Candidate":{"all":[{"_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"},{"_account_id":3,"name":"Jenkins","username":"jenkins"}],"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":3,"name":"Jenkins","username":"jenkins"},{"_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2017-05-10 16:21:05.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"},{"updated":"2017-05-15 07:45:24.000000000","updated_by":{"_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"},"reviewer":{"_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"},"state":"REVIEWER"}],"messages":[{"id":"3f79f67837f8b9d4415d593e170570a2d01a30a3","author":{"_account_id":25022,"name":"Rick Box","email":"box.rick@gmail.com","username":"rickbox"},"date":"2017-05-10 15:41:00.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"784176b5fc7cc3b208c48397da54bddb97e172cb","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2017-05-10 16:21:05.000000000","message":"Patch Set 1: Verified-1\n\nBuild failed (check pipeline).  For information on how to proceed, see http://docs.openstack.org/infra/manual/developers.html#automated-testing\n\n- gate-openstack-ansible-os_ceilometer-docs-ubuntu-xenial http://docs-draft.openstack.org/11/463811/1/check/gate-openstack-ansible-os_ceilometer-docs-ubuntu-xenial/118353e//doc/build/html/ : SUCCESS in 1m 58s\n- gate-openstack-ansible-os_ceilometer-linters-ubuntu-xenial http://logs.openstack.org/11/463811/1/check/gate-openstack-ansible-os_ceilometer-linters-ubuntu-xenial/cc398d4/ : SUCCESS in 1m 09s\n- gate-openstack-ansible-os_ceilometer-ansible-func-centos-7 http://logs.openstack.org/11/463811/1/check/gate-openstack-ansible-os_ceilometer-ansible-func-centos-7/176e11a/ : FAILURE in 24m 42s\n- gate-openstack-ansible-os_ceilometer-ansible-func-ubuntu-xenial http://logs.openstack.org/11/463811/1/check/gate-openstack-ansible-os_ceilometer-ansible-func-ubuntu-xenial/032f688/ : FAILURE in 16m 11s\n- gate-openstack-ansible-os_ceilometer-ansible-gnocchi-ubuntu-trusty-nv http://logs.openstack.org/11/463811/1/check/gate-openstack-ansible-os_ceilometer-ansible-gnocchi-ubuntu-trusty-nv/9f0e252/ : FAILURE in 53s (non-voting)\n- gate-openstack-ansible-os_ceilometer-ansible-gnocchi-ubuntu-xenial-nv http://logs.openstack.org/11/463811/1/check/gate-openstack-ansible-os_ceilometer-ansible-gnocchi-ubuntu-xenial-nv/ddf3a73/ : FAILURE in 19m 17s (non-voting)\n- gate-openstack-ansible-os_ceilometer-ansible-gnocchi-centos-7-nv http://logs.openstack.org/11/463811/1/check/gate-openstack-ansible-os_ceilometer-ansible-gnocchi-centos-7-nv/47677d8/ : FAILURE in 39m 44s (non-voting)","accounts_in_message":[],"_revision_number":1},{"id":"3fc1b0fcf04563c1682e15c99817c62cb0703f73","author":{"_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"},"date":"2017-05-11 10:08:06.000000000","message":"Topic changed from ceilometer_lookup_fix to venv_checksum_fix","accounts_in_message":[],"_revision_number":1},{"id":"23afb97259a6137c0dbee714152b5720f2b2b2af","author":{"_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"},"date":"2017-05-15 07:45:24.000000000","message":"Patch Set 1: Code-Review-1\n\nI\u0027ve implemented https://review.openstack.org/463988 which solves the same problem in a consistent way with the keystone role\u0027s implementation of git-sourced config. It also adds the feature of being able to use a file verbatim from the deployment host if it exists.\n\nAs such I don\u0027t think this patch is required and can be abandoned.","accounts_in_message":[],"_revision_number":1},{"id":"9df3fe90c0d86f9074e708e22d62b6a3ff41d5f3","author":{"_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"},"date":"2017-06-18 13:53:12.000000000","message":"Abandoned\n\nThis has been taken care of in another patch","accounts_in_message":[],"_revision_number":1}],"current_revision_number":1,"current_revision":"8f26ebdf804cca29789bb5332f7e8dd791ca965f","revisions":{"8f26ebdf804cca29789bb5332f7e8dd791ca965f":{"kind":"REWORK","_number":1,"created":"2017-05-10 15:41:00.000000000","uploader":{"_account_id":25022,"name":"Rick Box","email":"box.rick@gmail.com","username":"rickbox"},"ref":"refs/changes/11/463811/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/openstack-ansible-os_ceilometer","ref":"refs/changes/11/463811/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/openstack-ansible-os_ceilometer refs/changes/11/463811/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/openstack-ansible-os_ceilometer refs/changes/11/463811/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/openstack-ansible-os_ceilometer refs/changes/11/463811/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/openstack-ansible-os_ceilometer refs/changes/11/463811/1"}}},"commit":{"parents":[{"commit":"ce46b7b16c0b0533a982758a90c205d30ca19919","subject":"Fix for lookup and get_url occuring in different places","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-ansible-os_ceilometer/commit/ce46b7b16c0b0533a982758a90c205d30ca19919"}]}],"author":{"name":"Rick Box","email":"rick.box@bbc.co.uk","date":"2017-05-10 15:36:09.000000000","tz":60},"committer":{"name":"Rick Box","email":"rick.box@bbc.co.uk","date":"2017-05-10 15:36:09.000000000","tz":60},"subject":"Fix for lookup not detecting errors and happening from wrong place","message":"Fix for lookup not detecting errors and happening from wrong place\n\nThe lookup here occurs from the deployment host and not the container\nhost itself this means that if the deployment host doesn\u0027t have direct\naccess to the network it fails.\n\nSecondly because lookups don\u0027t have any real error checking, in our case\nafter the proxy timeout it simply takes the contents of this failed page\nand templates it out.\n\nThe following fix allows for some error checking and it will occur from\nthe correct host.\n\nChange-Id: Iae293bf96e31b04cb89a0f12963602e6030fe697\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-ansible-os_ceilometer/commit/8f26ebdf804cca29789bb5332f7e8dd791ca965f"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-ansible-os_ceilometer/commit/8f26ebdf804cca29789bb5332f7e8dd791ca965f"}]},"parents_data":[{"branch_name":"refs/heads/master","commit_id":"ce46b7b16c0b0533a982758a90c205d30ca19919","is_merged_in_target_branch":true,"change_id":"Ifee1c551ce596c50b9b9af7b5f390e995a6c7b67","change_number":463707,"patch_set_number":1,"change_status":"MERGED"}],"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}
