)]}'
{"id":"openstack%2Fceilometer~579473","triplet_id":"openstack%2Fceilometer~master~Ide5abb964efbe14d1f2cbaa822a699afc1055786","project":"openstack/ceilometer","branch":"master","topic":"notification-pool","hashtags":[],"change_id":"Ide5abb964efbe14d1f2cbaa822a699afc1055786","subject":"Use notification listener pools","status":"ABANDONED","created":"2018-07-02 07:24:31.000000000","updated":"2025-09-09 07:20:05.000000000","total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"df48714fef3faa9af261671626a2673e2fef86b8","_number":579473,"virtual_id_number":579473,"owner":{"_account_id":9542,"name":"Pavlo Shchelokovskyy","email":"pshchelokovskyy@mirantis.com","username":"pshchelo"},"actions":{},"labels":{"Verified":{"recommended":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"value":1,"date":"2018-07-02 08:16:05.000000000","permitted_voting_range":{"min":-2,"max":2},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"_account_id":2813,"name":"Mehdi Abaakouk","email":"sileht@sileht.net","username":"sileht"},{"date":"2018-07-02 07:37:04.000000000","_account_id":15843,"name":"Intel Hardware-Meters CI","email":"ipmi-ci@intel.com","username":"ipmici"}],"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":2813,"name":"Mehdi Abaakouk","email":"sileht@sileht.net","username":"sileht"},"all":[{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":-1,"date":"2018-07-02 08:29:30.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":2813,"name":"Mehdi Abaakouk","email":"sileht@sileht.net","username":"sileht"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":15843,"name":"Intel Hardware-Meters CI","email":"ipmi-ci@intel.com","username":"ipmici"}],"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":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"_account_id":2813,"name":"Mehdi Abaakouk","email":"sileht@sileht.net","username":"sileht"},{"_account_id":15843,"name":"Intel Hardware-Meters CI","email":"ipmi-ci@intel.com","username":"ipmici"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":2813,"name":"Mehdi Abaakouk","email":"sileht@sileht.net","username":"sileht"},{"_account_id":15843,"name":"Intel Hardware-Meters CI","email":"ipmi-ci@intel.com","username":"ipmici"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2018-07-02 07:37:04.000000000","updated_by":{"_account_id":15843,"name":"Intel Hardware-Meters CI","email":"ipmi-ci@intel.com","username":"ipmici"},"reviewer":{"_account_id":15843,"name":"Intel Hardware-Meters CI","email":"ipmi-ci@intel.com","username":"ipmici"},"state":"REVIEWER"},{"updated":"2018-07-02 08:16:05.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"},{"updated":"2018-07-02 08:29:30.000000000","updated_by":{"_account_id":2813,"name":"Mehdi Abaakouk","email":"sileht@sileht.net","username":"sileht"},"reviewer":{"_account_id":2813,"name":"Mehdi Abaakouk","email":"sileht@sileht.net","username":"sileht"},"state":"REVIEWER"}],"messages":[{"id":"6bfe204dfe0c6e6ffde1ce1d71187d37fecc00ae","author":{"_account_id":9542,"name":"Pavlo Shchelokovskyy","email":"pshchelokovskyy@mirantis.com","username":"pshchelo"},"date":"2018-07-02 07:24:31.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"3fdf76cd246ef3fbe68dbb742b5fbc075616334c","author":{"_account_id":15843,"name":"Intel Hardware-Meters CI","email":"ipmi-ci@intel.com","username":"ipmici"},"date":"2018-07-02 07:37:04.000000000","message":"Patch Set 1:\n\n* ipmi-test http://52.27.155.124/ipmi/579473/1 : SUCCESS","accounts_in_message":[],"_revision_number":1},{"id":"fc95b96031467267217c60816bdbfd46459fb24b","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2018-07-02 08:16:05.000000000","message":"Patch Set 1: Verified+1\n\nBuild succeeded (check pipeline).\n\n- openstack-tox-pep8 http://logs.openstack.org/73/579473/1/check/openstack-tox-pep8/99ee473/ : SUCCESS in 3m 46s\n- openstack-tox-py27 http://logs.openstack.org/73/579473/1/check/openstack-tox-py27/02befca/ : SUCCESS in 4m 25s\n- openstack-tox-py35 http://logs.openstack.org/73/579473/1/check/openstack-tox-py35/11487a2/ : SUCCESS in 4m 16s\n- build-openstack-sphinx-docs http://logs.openstack.org/73/579473/1/check/build-openstack-sphinx-docs/040c158/html/ : SUCCESS in 4m 33s\n- grenade-dsvm-ceilometer http://logs.openstack.org/73/579473/1/check/grenade-dsvm-ceilometer/d877d47/ : SUCCESS in 48m 58s\n- telemetry-dsvm-integration http://logs.openstack.org/73/579473/1/check/telemetry-dsvm-integration/67af495/ : SUCCESS in 42m 40s","accounts_in_message":[],"_revision_number":1},{"id":"af8ff85639c86191a601d78378d72da74578a160","author":{"_account_id":2813,"name":"Mehdi Abaakouk","email":"sileht@sileht.net","username":"sileht"},"date":"2018-07-02 08:29:30.000000000","message":"Patch Set 1: Code-Review-1\n\nI agree, it\u0027s better to set it. \n\nBut hardcoding it will break rolling upgrade, and oslo.messaging will still continue to push things on default pool anyways. If no consumer listen for the default pool the queue will just grow infinitely.\n\nAbout configuration, I think you\u0027re wrong, the configuration is parsed at this stage.","accounts_in_message":[],"_revision_number":1},{"id":"df48714fef3faa9af261671626a2673e2fef86b8","tag":"autogenerated:gerrit:abandon","author":{"_account_id":32968,"name":"Juan Larriba","email":"jlarriba@redhat.com","username":"jlarriba"},"date":"2025-09-09 07:20:05.000000000","message":"Abandoned\n\nThis has been here for years. If you are still interested in getting this merged, please send in a new patch for review.","accounts_in_message":[],"_revision_number":1}],"current_revision_number":1,"current_revision":"d00f9f46b760d8ab4423e0ffb3921c3019599cfc","revisions":{"d00f9f46b760d8ab4423e0ffb3921c3019599cfc":{"kind":"REWORK","_number":1,"created":"2018-07-02 07:24:31.000000000","uploader":{"_account_id":9542,"name":"Pavlo Shchelokovskyy","email":"pshchelokovskyy@mirantis.com","username":"pshchelo"},"ref":"refs/changes/73/579473/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/ceilometer","ref":"refs/changes/73/579473/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/ceilometer refs/changes/73/579473/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/ceilometer refs/changes/73/579473/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/ceilometer refs/changes/73/579473/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/ceilometer refs/changes/73/579473/1"}}},"commit":{"parents":[{"commit":"4ae919c96e4116ab83e5d83f2b726ed44d165278","subject":"Imported Translations from Zanata","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ceilometer/commit/4ae919c96e4116ab83e5d83f2b726ed44d165278"}]}],"author":{"name":"Pavlo Shchelokovskyy","email":"shchelokovskyy@gmail.com","date":"2018-07-02 06:57:22.000000000","tz":0},"committer":{"name":"Pavlo Shchelokovskyy","email":"shchelokovskyy@gmail.com","date":"2018-07-02 06:57:22.000000000","tz":0},"subject":"Use notification listener pools","message":"Use notification listener pools\n\nspecifying pools for notification listeners is an approach alternative\nto specifying multiple topics for notification emitters (and supscribing\nto those in notification consumers). It allows easier splitting of\nnotification listening services and ensuring notification delivery to\nevery service without changing configuration of notification emitting\nservices.\n\nUnfortunately we can not specify the pool name in config (as it is not\nparsed yet at that stage), so we hardcode the listener pool name to\n\u0027ceilometer-agent-notification\u0027 and use try/except to deal with those\noslo.messaging drivers that do not support pools for notification\nlisteners (currently only \u0027amqp1\u0027 does not support it, while\n\u0027rabbit\u0027/\u0027kombu\u0027 and \u0027kafka\u0027 do support it).\n\nChange-Id: Ide5abb964efbe14d1f2cbaa822a699afc1055786\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ceilometer/commit/d00f9f46b760d8ab4423e0ffb3921c3019599cfc"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ceilometer/commit/d00f9f46b760d8ab4423e0ffb3921c3019599cfc"}]},"parents_data":[{"branch_name":"refs/heads/master","commit_id":"4ae919c96e4116ab83e5d83f2b726ed44d165278","is_merged_in_target_branch":true,"change_id":"I95df9f07b61d2e3cda69717ce692b0d39f1ef936","change_number":564977,"patch_set_number":8,"change_status":"MERGED"}],"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[{"name":"Verified","description":"Verified in gate by CI","status":"UNSATISFIED","is_legacy":false,"submittability_expression_result":{"expression":"label:Verified\u003dMAX AND -label:Verified\u003dMIN","fulfilled":false,"status":"FAIL","passing_atoms":[],"failing_atoms":["label:Verified\u003dMAX","label:Verified\u003dMIN"],"atom_explanations":{}}},{"name":"Code-Review","description":"Code reviewed by core reviewer","status":"UNSATISFIED","is_legacy":false,"submittability_expression_result":{"expression":"label:Code-Review\u003dMAX AND -label:Code-Review\u003dMIN","fulfilled":false,"status":"FAIL","passing_atoms":[],"failing_atoms":["label:Code-Review\u003dMAX","label:Code-Review\u003dMIN"],"atom_explanations":{}}},{"name":"Workflow","description":"Approved for gate by core reviewer","status":"UNSATISFIED","is_legacy":false,"submittability_expression_result":{"expression":"label:Workflow\u003dMAX AND -label:Workflow\u003dMIN","fulfilled":false,"status":"FAIL","passing_atoms":[],"failing_atoms":["label:Workflow\u003dMAX","label:Workflow\u003dMIN"],"atom_explanations":{}}}]}
