)]}'
{"id":"x%2Fpackstack~258636","triplet_id":"x%2Fpackstack~master~I5608691776566700386cc379ce296fb510358d50","project":"x/packstack","branch":"master","topic":"fixsahara","hashtags":[],"change_id":"I5608691776566700386cc379ce296fb510358d50","subject":"Fix Sahara templates to work with latest puppet-sahara","status":"MERGED","created":"2015-12-16 16:59:25.000000000","updated":"2016-01-12 09:10:08.000000000","submitted":"2016-01-12 09:10:08.000000000","submitter":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"9a062475188675b9f1944a61f92429c56d0264ad","_number":258636,"virtual_id_number":258636,"owner":{"_account_id":13294,"name":"Javier Peña","email":"jpena@redhat.com","username":"jpena"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"value":0,"_account_id":11166,"name":"Lukas Bezdicka","email":"social@v3.sk","username":"xbezdick"},{"value":0,"_account_id":5241,"name":"Martin Magr","email":"mmagr@redhat.com","username":"mmagr"},{"value":0,"_account_id":7462,"name":"Ivan Chavero","email":"ichavero@redhat.com","username":"ichavero"},{"value":2,"date":"2016-01-12 09:10:07.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":"","default_value":0,"optional":true},"Code-Review":{"approved":{"_account_id":7462,"name":"Ivan Chavero","email":"ichavero@redhat.com","username":"ichavero"},"all":[{"value":0,"_account_id":11166,"name":"Lukas Bezdicka","email":"social@v3.sk","username":"xbezdick"},{"value":0,"_account_id":5241,"name":"Martin Magr","email":"mmagr@redhat.com","username":"mmagr"},{"value":2,"date":"2016-01-12 09:06:48.000000000","_account_id":7462,"name":"Ivan Chavero","email":"ichavero@redhat.com","username":"ichavero"},{"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":7462,"name":"Ivan Chavero","email":"ichavero@redhat.com","username":"ichavero"},"all":[{"value":0,"_account_id":11166,"name":"Lukas Bezdicka","email":"social@v3.sk","username":"xbezdick"},{"value":0,"_account_id":5241,"name":"Martin Magr","email":"mmagr@redhat.com","username":"mmagr"},{"value":1,"date":"2016-01-12 09:06:48.000000000","_account_id":7462,"name":"Ivan Chavero","email":"ichavero@redhat.com","username":"ichavero"},{"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":5241,"name":"Martin Magr","email":"mmagr@redhat.com","username":"mmagr"},{"_account_id":7462,"name":"Ivan Chavero","email":"ichavero@redhat.com","username":"ichavero"},{"_account_id":11166,"name":"Lukas Bezdicka","email":"social@v3.sk","username":"xbezdick"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2015-12-22 17:11:29.000000000","updated_by":{"_account_id":5241,"name":"Martin Magr","email":"mmagr@redhat.com","username":"mmagr"},"reviewer":{"_account_id":5241,"name":"Martin Magr","email":"mmagr@redhat.com","username":"mmagr"},"state":"REVIEWER"},{"updated":"2015-12-22 17:11:37.000000000","updated_by":{"_account_id":11166,"name":"Lukas Bezdicka","email":"social@v3.sk","username":"xbezdick"},"reviewer":{"_account_id":11166,"name":"Lukas Bezdicka","email":"social@v3.sk","username":"xbezdick"},"state":"REVIEWER"},{"updated":"2016-01-12 09:06:48.000000000","updated_by":{"_account_id":7462,"name":"Ivan Chavero","email":"ichavero@redhat.com","username":"ichavero"},"reviewer":{"_account_id":7462,"name":"Ivan Chavero","email":"ichavero@redhat.com","username":"ichavero"},"state":"REVIEWER"},{"updated":"2016-01-12 09:10:07.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"}],"messages":[{"id":"ed5252617d646b3647b6de1ae2cfb6c63349d647","author":{"_account_id":13294,"name":"Javier Peña","email":"jpena@redhat.com","username":"jpena"},"date":"2015-12-16 16:59:25.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"13446fe090f1f37e31f762e39f575ae1aef7af25","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-12-16 21:54:21.000000000","message":"Patch Set 1: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-packstack-pep8 http://logs.openstack.org/36/258636/1/check/gate-packstack-pep8/88698d1/ : SUCCESS in 1m 04s\n- gate-packstack-python27 http://logs.openstack.org/36/258636/1/check/gate-packstack-python27/d6ae016/ : SUCCESS in 1m 10s","accounts_in_message":[],"_revision_number":1},{"id":"18620d555f3d70af6c246045950fb7355cab1e9a","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-12-29 22:29:11.000000000","message":"Patch Set 1: Verified-1\n\nMerge Failed.\n\nThis change or one of its cross-repo dependencies was unable to be automatically merged with the current state of its repository. Please rebase the change and upload a new patchset.","accounts_in_message":[],"_revision_number":1},{"id":"8d8a0d86f616ab5f2886b3cae96d6f82f22ab540","author":{"_account_id":13294,"name":"Javier Peña","email":"jpena@redhat.com","username":"jpena"},"date":"2016-01-07 11:55:47.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"a1b98a135f2ef9b29160088882545ee4c616dea2","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-01-07 11:58:05.000000000","message":"Patch Set 2: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-packstack-pep8 http://logs.openstack.org/36/258636/2/check/gate-packstack-pep8/348128c/ : SUCCESS in 1m 22s\n- gate-packstack-python27 http://logs.openstack.org/36/258636/2/check/gate-packstack-python27/c076318/ : SUCCESS in 1m 18s","accounts_in_message":[],"_revision_number":2},{"id":"9926bcc0de28b2ba6a68f21b85d359c076db2995","author":{"_account_id":7462,"name":"Ivan Chavero","email":"ichavero@redhat.com","username":"ichavero"},"date":"2016-01-12 08:46:09.000000000","message":"Patch Set 2: Code-Review+1\n\ncode looks good, testing","accounts_in_message":[],"_revision_number":2},{"id":"9c9344f6b34d0ee494ba8007bcda9759ebf951c4","author":{"_account_id":7462,"name":"Ivan Chavero","email":"ichavero@redhat.com","username":"ichavero"},"date":"2016-01-12 09:06:48.000000000","message":"Patch Set 2: Code-Review+2 Workflow+1\n\nworks ok","accounts_in_message":[],"_revision_number":2},{"id":"35e1bfa13bbadef17ccdf9c7eb9a01cf6a9d54ff","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-01-12 09:06:54.000000000","message":"Patch Set 2: -Verified\n\nStarting gate jobs.\nhttp://status.openstack.org/zuul/","accounts_in_message":[],"_revision_number":2},{"id":"06c987c0306c5b764b2ecd9688686bad278b7359","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-01-12 09:10:07.000000000","message":"Patch Set 2: Verified+2\n\nBuild succeeded (gate pipeline).\n\n- gate-packstack-pep8 http://logs.openstack.org/36/258636/2/gate/gate-packstack-pep8/f978a01/ : SUCCESS in 1m 09s\n- gate-packstack-python27 http://logs.openstack.org/36/258636/2/gate/gate-packstack-python27/7f49b32/ : SUCCESS in 1m 07s","accounts_in_message":[],"_revision_number":2},{"id":"9465c8a2ecb52547cbb02cbd1413d53c19cbf067","date":"2016-01-12 09:10:08.000000000","message":"Change has been successfully merged into the git repository by Jenkins","accounts_in_message":[],"_revision_number":2}],"current_revision_number":2,"current_revision":"3e727b5f1cdff3e0e80f40b3b9b70ce06caba7a9","revisions":{"3fba65004ead29d4a17261020a90a8ab32ddf065":{"kind":"REWORK","_number":1,"created":"2015-12-16 16:59:25.000000000","uploader":{"_account_id":13294,"name":"Javier Peña","email":"jpena@redhat.com","username":"jpena"},"ref":"refs/changes/36/258636/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/x/packstack","ref":"refs/changes/36/258636/1","commands":{"Checkout":"git fetch https://review.opendev.org/x/packstack refs/changes/36/258636/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/x/packstack refs/changes/36/258636/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/x/packstack refs/changes/36/258636/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/x/packstack refs/changes/36/258636/1"}}},"commit":{"parents":[{"commit":"0ea7f7955929d0a5ce236633f7147f5e2713bc69","subject":"Merge \"Fix Nova notifications for Neutron\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/x/packstack/commit/0ea7f7955929d0a5ce236633f7147f5e2713bc69"}]}],"author":{"name":"Javier Pena","email":"jpena@redhat.com","date":"2015-12-16 16:49:21.000000000","tz":60},"committer":{"name":"Javier Pena","email":"jpena@redhat.com","date":"2015-12-16 16:49:21.000000000","tz":60},"subject":"Fix Sahara templates to work with latest puppet-sahara","message":"Fix Sahara templates to work with latest puppet-sahara\n\nThe latest puppet-sahara module introduces a few incompatibilities:\n\n- It enables $::os_service_default (https://review.openstack.org/245178),\n  which requires changes in the way kombu_ssl_* parameters are\n  handled.\n- It removes sahara::notify::rabbitmq and sahara::notify::qpid\n  (https://review.openstack.org/253099), so we need to configure\n  the rpc backend in a different way.\n- It also removes all qpid functionality\n  (https://review.openstack.org/253105). It is kept for now, so it\n  can be properly removed with a single review.\n\nChange-Id: I5608691776566700386cc379ce296fb510358d50\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/x/packstack/commit/3fba65004ead29d4a17261020a90a8ab32ddf065"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/x/packstack/commit/3fba65004ead29d4a17261020a90a8ab32ddf065"}]},"branch":"refs/heads/master"},"3e727b5f1cdff3e0e80f40b3b9b70ce06caba7a9":{"kind":"REWORK","_number":2,"created":"2016-01-07 11:55:47.000000000","uploader":{"_account_id":13294,"name":"Javier Peña","email":"jpena@redhat.com","username":"jpena"},"ref":"refs/changes/36/258636/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/x/packstack","ref":"refs/changes/36/258636/2","commands":{"Checkout":"git fetch https://review.opendev.org/x/packstack refs/changes/36/258636/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/x/packstack refs/changes/36/258636/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/x/packstack refs/changes/36/258636/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/x/packstack refs/changes/36/258636/2"}}},"commit":{"parents":[{"commit":"28e74215ad11842edab1a882d8fd2250d065c2fe","subject":"Merge \"Deprecated tox -downloadcache option removed\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/x/packstack/commit/28e74215ad11842edab1a882d8fd2250d065c2fe"}]}],"author":{"name":"Javier Pena","email":"jpena@redhat.com","date":"2015-12-16 16:49:21.000000000","tz":60},"committer":{"name":"Javier Pena","email":"jpena@redhat.com","date":"2016-01-07 11:55:18.000000000","tz":60},"subject":"Fix Sahara templates to work with latest puppet-sahara","message":"Fix Sahara templates to work with latest puppet-sahara\n\nThe latest puppet-sahara module introduces a few incompatibilities:\n\n- It enables $::os_service_default (https://review.openstack.org/245178),\n  which requires changes in the way kombu_ssl_* parameters are\n  handled.\n- It removes sahara::notify::rabbitmq and sahara::notify::qpid\n  (https://review.openstack.org/253099), so we need to configure\n  the rpc backend in a different way.\n- It also removes all qpid functionality\n  (https://review.openstack.org/253105). It is kept for now, so it\n  can be properly removed with a single review.\n\nChange-Id: I5608691776566700386cc379ce296fb510358d50\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/x/packstack/commit/3e727b5f1cdff3e0e80f40b3b9b70ce06caba7a9"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/x/packstack/commit/3e727b5f1cdff3e0e80f40b3b9b70ce06caba7a9"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}