)]}'
{"id":"openstack%2Fdevstack-gate~522241","triplet_id":"openstack%2Fdevstack-gate~master~I2fd2afd06b73cab68e427c94e4d2064fe727702f","project":"openstack/devstack-gate","branch":"master","hashtags":[],"change_id":"I2fd2afd06b73cab68e427c94e4d2064fe727702f","subject":"Add a new hook ``collect_artifacts_hook``","status":"ABANDONED","created":"2017-11-22 13:26:04.000000000","updated":"2017-11-24 10:45:03.000000000","total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"6de1285aa467377a2a05eedcd4e39ae907e217ac","_number":522241,"virtual_id_number":522241,"owner":{"_account_id":23804,"name":"Daniel Alvarez","email":"dalvarez@redhat.com","username":"dalvarez"},"actions":{},"labels":{"Verified":{"recommended":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"date":"2017-11-22 13:35:49.000000000","_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},{"value":1,"date":"2017-11-22 15:06:24.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":"","value":1,"default_value":0,"optional":true},"Code-Review":{"all":[{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},{"value":0,"date":"2017-11-22 13:39:54.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_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":{"all":[{"_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},{"_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":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2017-11-22 13:35:49.000000000","updated_by":{"_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},"reviewer":{"_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},"state":"REVIEWER"},{"updated":"2017-11-22 13:39:54.000000000","updated_by":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"reviewer":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"state":"REVIEWER"},{"updated":"2017-11-22 15:06:24.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":"b273125608636ecd7e5f8134244746395fd81105","author":{"_account_id":23804,"name":"Daniel Alvarez","email":"dalvarez@redhat.com","username":"dalvarez"},"date":"2017-11-22 13:26:04.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"aeeccbdbb25236a6ff4cf80c06634095c638a724","author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"date":"2017-11-22 13:30:27.000000000","message":"Patch Set 1: Code-Review+1\n\n+1 for the idea.\n\nIf you look at the cleanup_host() function we can see many project-specific conditionals copying logs/artifacts to the \"devstack/logs\" directory that will then be shown in gate. So, I think it makes sense to have that hook which projects could, in their own tree, take care of selecting the artifacts that makes sense to be exposed in tree for their jobs run.","accounts_in_message":[],"_revision_number":1},{"id":"27a56ece0eefb4c5790365fa8879e8f263aab673","author":{"_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},"date":"2017-11-22 13:35:49.000000000","message":"Patch Set 1:\n\npost-run playbooks are run unconditionally after the job and are a great place to collect things. The long-term vision for devstack-gate is to retire it in favor of the devstack base job for devstack jobs and the multinode base job for multi-node jobs that don\u0027t actually need devstack itself.\n\nI don\u0027t see any problems with this patch, but I wanted to nudge towards making use of zuul v3 native features instead if possible.","accounts_in_message":[],"_revision_number":1},{"id":"e1456821bdef70ddbfcbf692ba31ac015cc6d1dc","author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"date":"2017-11-22 13:39:54.000000000","message":"Patch Set 1: -Code-Review\n\nThanks for the pointer Monty!\n\nWe\u0027ve already migrated the playbooks to the networking-ovn tree (project which we are trying to collect the logs) and I think we can do it in the post-run then.","accounts_in_message":[],"_revision_number":1},{"id":"d19bc8b20cf0d48245a85f1b0ecde222b547d402","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2017-11-22 15:06:24.000000000","message":"Patch Set 1: Verified+1\n\nBuild succeeded (check pipeline).\n\n- legacy-tempest-dsvm-py35 http://logs.openstack.org/41/522241/1/check/legacy-tempest-dsvm-py35/afa8351/ : SUCCESS in 1h 10m 52s\n- legacy-devstack-gate-tox-run-tests http://logs.openstack.org/41/522241/1/check/legacy-devstack-gate-tox-run-tests/0516d31/ : SUCCESS in 4m 08s\n- legacy-devstack-gate-tox-py3-run-tests http://logs.openstack.org/41/522241/1/check/legacy-devstack-gate-tox-py3-run-tests/9d58f95/ : SUCCESS in 3m 56s\n- openstack-tox-bashate http://logs.openstack.org/41/522241/1/check/openstack-tox-bashate/0f3776d/ : SUCCESS in 2m 59s\n- legacy-dg-hooks-dsvm http://logs.openstack.org/41/522241/1/check/legacy-dg-hooks-dsvm/795d877/ : SUCCESS in 7m 27s\n- legacy-grenade-dsvm-neutron http://logs.openstack.org/41/522241/1/check/legacy-grenade-dsvm-neutron/41ffb3a/ : SUCCESS in 43m 18s\n- legacy-grenade-dsvm-neutron-multinode http://logs.openstack.org/41/522241/1/check/legacy-grenade-dsvm-neutron-multinode/128bca7/ : SUCCESS in 54m 05s\n- legacy-grenade-dsvm-neutron-dvr-multinode http://logs.openstack.org/41/522241/1/check/legacy-grenade-dsvm-neutron-dvr-multinode/451c6ca/ : SUCCESS in 1h 00m 38s (non-voting)\n- legacy-neutron-dsvm-functional http://logs.openstack.org/41/522241/1/check/legacy-neutron-dsvm-functional/fe78e4f/ : SUCCESS in 28m 41s (non-voting)\n- legacy-neutron-dsvm-api http://logs.openstack.org/41/522241/1/check/legacy-neutron-dsvm-api/0db5d8b/ : SUCCESS in 36m 56s (non-voting)\n- legacy-swift-dsvm-functional http://logs.openstack.org/41/522241/1/check/legacy-swift-dsvm-functional/0678976/ : SUCCESS in 19m 15s (non-voting)\n- legacy-tempest-dsvm-neutron-full-newton http://logs.openstack.org/41/522241/1/check/legacy-tempest-dsvm-neutron-full-newton/21a242c/ : SUCCESS in 1h 17m 02s (non-voting)\n- legacy-tempest-dsvm-neutron-full-ocata http://logs.openstack.org/41/522241/1/check/legacy-tempest-dsvm-neutron-full-ocata/67c4392/ : SUCCESS in 1h 23m 59s (non-voting)\n- legacy-tempest-dsvm-neutron-full-pike http://logs.openstack.org/41/522241/1/check/legacy-tempest-dsvm-neutron-full-pike/d9f9e6d/ : SUCCESS in 1h 32m 57s (non-voting)\n- legacy-tempest-dsvm-neutron-full-centos-7 http://logs.openstack.org/41/522241/1/check/legacy-tempest-dsvm-neutron-full-centos-7/365fb3f/ : SUCCESS in 1h 37m 01s (non-voting)\n- legacy-tempest-dsvm-multinode-full http://logs.openstack.org/41/522241/1/check/legacy-tempest-dsvm-multinode-full/2586417/ : SUCCESS in 1h 11m 45s (non-voting)\n- legacy-tempest-dsvm-neutron-multinode-full http://logs.openstack.org/41/522241/1/check/legacy-tempest-dsvm-neutron-multinode-full/0fba0ce/ : SUCCESS in 1h 26m 33s (non-voting)\n- legacy-tempest-dsvm-neutron-full-opensuse-423 http://logs.openstack.org/41/522241/1/check/legacy-tempest-dsvm-neutron-full-opensuse-423/0542550/ : SUCCESS in 1h 23m 22s (non-voting)\n- legacy-tempest-dsvm-neutron-full http://logs.openstack.org/41/522241/1/check/legacy-tempest-dsvm-neutron-full/f41abd9/ : SUCCESS in 1h 29m 45s","accounts_in_message":[],"_revision_number":1},{"id":"6de1285aa467377a2a05eedcd4e39ae907e217ac","author":{"_account_id":23804,"name":"Daniel Alvarez","email":"dalvarez@redhat.com","username":"dalvarez"},"date":"2017-11-24 10:45:03.000000000","message":"Abandoned\n\nIn favor of https://review.openstack.org/#/c/520609/","accounts_in_message":[],"_revision_number":1}],"current_revision_number":1,"current_revision":"590949ac892a60caeeb36dca41524f71187ece95","revisions":{"590949ac892a60caeeb36dca41524f71187ece95":{"kind":"REWORK","_number":1,"created":"2017-11-22 13:26:04.000000000","uploader":{"_account_id":23804,"name":"Daniel Alvarez","email":"dalvarez@redhat.com","username":"dalvarez"},"ref":"refs/changes/41/522241/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack-gate","ref":"refs/changes/41/522241/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack-gate refs/changes/41/522241/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack-gate refs/changes/41/522241/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack-gate refs/changes/41/522241/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack-gate refs/changes/41/522241/1"}}},"commit":{"parents":[{"commit":"b8db7d219e02086ffd2282c0aa75c48bcf4ef2bf","subject":"Checkout from -eol tag if stable branch does not exist","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack-gate/commit/b8db7d219e02086ffd2282c0aa75c48bcf4ef2bf"}]}],"author":{"name":"Daniel Alvarez","email":"dalvarez@redhat.com","date":"2017-11-22 13:22:39.000000000","tz":60},"committer":{"name":"Daniel Alvarez","email":"dalvarez@redhat.com","date":"2017-11-22 13:26:01.000000000","tz":60},"subject":"Add a new hook ``collect_artifacts_hook``","message":"Add a new hook ``collect_artifacts_hook``\n\nRight now, there\u0027s a lot of project specific code in the cleanup_host\nfunction. This patch adds a generic way for projects to collect logs,\nconfiguration files, etc. regardless of the job exit code.\n\nThis hook will be called unconditionally as the last step before\ncleanup.\n\nChange-Id: I2fd2afd06b73cab68e427c94e4d2064fe727702f\nSigned-off-by: Daniel Alvarez \u003cdalvarez@redhat.com\u003e\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack-gate/commit/590949ac892a60caeeb36dca41524f71187ece95"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack-gate/commit/590949ac892a60caeeb36dca41524f71187ece95"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}