)]}'
{"defaults/main.yml":[{"author":{"_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"},"change_message_id":"a7bc3b21e0b8f97521912b7f52ab62bc88a16adf","unresolved":false,"context_lines":[{"line_number":105,"context_line":"keystone_ceilometer_enabled: false"},{"line_number":106,"context_line":""},{"line_number":107,"context_line":"# Defaults to True if the service is deployed."},{"line_number":108,"context_line":"ceilometer_aodh_enabled: groups[\u0027aodh_all\u0027] is defined and groups[\u0027aodh_all\u0027] | length \u003e 0"},{"line_number":109,"context_line":""},{"line_number":110,"context_line":""},{"line_number":111,"context_line":"## Keystone authentication middleware"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"7aa08908_782fce88","line":108,"range":{"start_line":108,"start_character":25,"end_line":108,"end_character":90},"updated":"2016-06-15 10:06:18.000000000","message":"I wonder whether this is better done in group_vars instead of here, and this should instead just default to False?","commit_id":"1ac1875f7803b365ec53fbe960fbff7c4f5efb85"},{"author":{"_account_id":12807,"name":"Steve Lewis (stevelle)","email":"stevelle@gmail.com","username":"stevelle"},"change_message_id":"b9f5352209ed6b7b3080048a3e3bca8622ca216b","unresolved":false,"context_lines":[{"line_number":105,"context_line":"keystone_ceilometer_enabled: false"},{"line_number":106,"context_line":""},{"line_number":107,"context_line":"# Defaults to True if the service is deployed."},{"line_number":108,"context_line":"ceilometer_aodh_enabled: groups[\u0027aodh_all\u0027] is defined and groups[\u0027aodh_all\u0027] | length \u003e 0"},{"line_number":109,"context_line":""},{"line_number":110,"context_line":""},{"line_number":111,"context_line":"## Keystone authentication middleware"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"7aa08908_664b9260","line":108,"range":{"start_line":108,"start_character":25,"end_line":108,"end_character":90},"in_reply_to":"7aa08908_782fce88","updated":"2016-06-15 18:36:36.000000000","message":"That\u0027s a pretty good solution for all the concerns here. I\u0027ll make this change depend on that one.","commit_id":"1ac1875f7803b365ec53fbe960fbff7c4f5efb85"}],"releasenotes/notes/update-aodh-integration-fd2a27e8864bd8ff.yaml":[{"author":{"_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"},"change_message_id":"a7bc3b21e0b8f97521912b7f52ab62bc88a16adf","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"other:"},{"line_number":3,"context_line":"  - Ceilometer no longer manages alarm storage when Aodh"},{"line_number":4,"context_line":"    is enabled. It now redirects alarm-related requests to"},{"line_number":5,"context_line":"    the Aodh API. This is now auto-enabled when Aodh is"},{"line_number":6,"context_line":"    deployed."},{"line_number":7,"context_line":"upgrade:"},{"line_number":8,"context_line":"  - Overrides for ceilometer aodh_connection_string will"},{"line_number":9,"context_line":"    no longer work."}],"source_content_type":"text/x-yaml","patch_set":1,"id":"7aa08908_752a3792","line":6,"range":{"start_line":3,"start_character":0,"end_line":6,"end_character":13},"updated":"2016-06-15 10:06:18.000000000","message":"I think that this item is better placed in the \u0027upgrade\u0027 section, as this is something that only matters to people who\u0027re upgrading.","commit_id":"1ac1875f7803b365ec53fbe960fbff7c4f5efb85"},{"author":{"_account_id":12807,"name":"Steve Lewis (stevelle)","email":"stevelle@gmail.com","username":"stevelle"},"change_message_id":"b9f5352209ed6b7b3080048a3e3bca8622ca216b","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"other:"},{"line_number":3,"context_line":"  - Ceilometer no longer manages alarm storage when Aodh"},{"line_number":4,"context_line":"    is enabled. It now redirects alarm-related requests to"},{"line_number":5,"context_line":"    the Aodh API. This is now auto-enabled when Aodh is"},{"line_number":6,"context_line":"    deployed."},{"line_number":7,"context_line":"upgrade:"},{"line_number":8,"context_line":"  - Overrides for ceilometer aodh_connection_string will"},{"line_number":9,"context_line":"    no longer work."}],"source_content_type":"text/x-yaml","patch_set":1,"id":"7aa08908_41cac01b","line":6,"range":{"start_line":3,"start_character":0,"end_line":6,"end_character":13},"in_reply_to":"7aa08908_752a3792","updated":"2016-06-15 18:36:36.000000000","message":"Done","commit_id":"1ac1875f7803b365ec53fbe960fbff7c4f5efb85"},{"author":{"_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"},"change_message_id":"a7bc3b21e0b8f97521912b7f52ab62bc88a16adf","unresolved":false,"context_lines":[{"line_number":5,"context_line":"    the Aodh API. This is now auto-enabled when Aodh is"},{"line_number":6,"context_line":"    deployed."},{"line_number":7,"context_line":"upgrade:"},{"line_number":8,"context_line":"  - Overrides for ceilometer aodh_connection_string will"},{"line_number":9,"context_line":"    no longer work."},{"line_number":10,"context_line":"fixes:"},{"line_number":11,"context_line":"  - Specifying an Aodh connection string in Ceilometer"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"7aa08908_75019706","line":8,"range":{"start_line":8,"start_character":29,"end_line":8,"end_character":51},"updated":"2016-06-15 10:06:18.000000000","message":"This needs to be ``aodh_connection_string``","commit_id":"1ac1875f7803b365ec53fbe960fbff7c4f5efb85"},{"author":{"_account_id":12807,"name":"Steve Lewis (stevelle)","email":"stevelle@gmail.com","username":"stevelle"},"change_message_id":"b9f5352209ed6b7b3080048a3e3bca8622ca216b","unresolved":false,"context_lines":[{"line_number":5,"context_line":"    the Aodh API. This is now auto-enabled when Aodh is"},{"line_number":6,"context_line":"    deployed."},{"line_number":7,"context_line":"upgrade:"},{"line_number":8,"context_line":"  - Overrides for ceilometer aodh_connection_string will"},{"line_number":9,"context_line":"    no longer work."},{"line_number":10,"context_line":"fixes:"},{"line_number":11,"context_line":"  - Specifying an Aodh connection string in Ceilometer"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"7aa08908_e1d92cf6","line":8,"range":{"start_line":8,"start_character":29,"end_line":8,"end_character":51},"in_reply_to":"7aa08908_75019706","updated":"2016-06-15 18:36:36.000000000","message":"Done","commit_id":"1ac1875f7803b365ec53fbe960fbff7c4f5efb85"},{"author":{"_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"},"change_message_id":"a7bc3b21e0b8f97521912b7f52ab62bc88a16adf","unresolved":false,"context_lines":[{"line_number":8,"context_line":"  - Overrides for ceilometer aodh_connection_string will"},{"line_number":9,"context_line":"    no longer work."},{"line_number":10,"context_line":"fixes:"},{"line_number":11,"context_line":"  - Specifying an Aodh connection string in Ceilometer"},{"line_number":12,"context_line":"    was deprecated within Ceilometer in a prior release"},{"line_number":13,"context_line":"    so this option has been removed."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":""},{"line_number":16,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":1,"id":"7aa08908_550a9b28","line":13,"range":{"start_line":11,"start_character":0,"end_line":13,"end_character":36},"updated":"2016-06-15 10:06:18.000000000","message":"I think perhaps this is an explanation for the note about \u0027aodh_connection_string\u0027 so perhaps this should just be a follow-on sentence for that item?","commit_id":"1ac1875f7803b365ec53fbe960fbff7c4f5efb85"},{"author":{"_account_id":12807,"name":"Steve Lewis (stevelle)","email":"stevelle@gmail.com","username":"stevelle"},"change_message_id":"b9f5352209ed6b7b3080048a3e3bca8622ca216b","unresolved":false,"context_lines":[{"line_number":8,"context_line":"  - Overrides for ceilometer aodh_connection_string will"},{"line_number":9,"context_line":"    no longer work."},{"line_number":10,"context_line":"fixes:"},{"line_number":11,"context_line":"  - Specifying an Aodh connection string in Ceilometer"},{"line_number":12,"context_line":"    was deprecated within Ceilometer in a prior release"},{"line_number":13,"context_line":"    so this option has been removed."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":""},{"line_number":16,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":1,"id":"7aa08908_c1d6b0c2","line":13,"range":{"start_line":11,"start_character":0,"end_line":13,"end_character":36},"in_reply_to":"7aa08908_550a9b28","updated":"2016-06-15 18:36:36.000000000","message":"Done","commit_id":"1ac1875f7803b365ec53fbe960fbff7c4f5efb85"},{"author":{"_account_id":538,"name":"Major Hayden","email":"major@mhtx.net","username":"rackerhacker"},"change_message_id":"80a38cee814bd1d836da6d492b8ca778b474edba","unresolved":false,"context_lines":[{"line_number":4,"context_line":"    is enabled. It now redirects alarm-related requests to"},{"line_number":5,"context_line":"    the Aodh API. This is now auto-enabled when Aodh is"},{"line_number":6,"context_line":"    deployed."},{"line_number":7,"context_line":"  - Overrides for ceilometer aodh_connection_string will"},{"line_number":8,"context_line":"    no longer work. Specifying an Aodh connection string"},{"line_number":9,"context_line":"    in Ceilometer was deprecated within Ceilometer in a"},{"line_number":10,"context_line":"    prior release so this option has been removed."}],"source_content_type":"text/x-yaml","patch_set":3,"id":"7aa08908_273344f8","line":7,"updated":"2016-06-15 19:01:30.000000000","message":"Minor nit: Jesse suggested wrapping this to make it monospaced -\u003e ``aodh_connection_string``","commit_id":"15033fadd95b91b7d4874f94b3d607c24ef740d8"},{"author":{"_account_id":12807,"name":"Steve Lewis (stevelle)","email":"stevelle@gmail.com","username":"stevelle"},"change_message_id":"7b357d050a0a99bd91ad9b22e83b21f305e92501","unresolved":false,"context_lines":[{"line_number":4,"context_line":"    is enabled. It now redirects alarm-related requests to"},{"line_number":5,"context_line":"    the Aodh API. This is now auto-enabled when Aodh is"},{"line_number":6,"context_line":"    deployed."},{"line_number":7,"context_line":"  - Overrides for ceilometer aodh_connection_string will"},{"line_number":8,"context_line":"    no longer work. Specifying an Aodh connection string"},{"line_number":9,"context_line":"    in Ceilometer was deprecated within Ceilometer in a"},{"line_number":10,"context_line":"    prior release so this option has been removed."}],"source_content_type":"text/x-yaml","patch_set":3,"id":"7aa08908_ef7f52a9","line":7,"in_reply_to":"7aa08908_273344f8","updated":"2016-06-15 20:31:49.000000000","message":"oops, missed that. fixed","commit_id":"15033fadd95b91b7d4874f94b3d607c24ef740d8"}],"templates/ceilometer.conf.j2":[{"author":{"_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"},"change_message_id":"76989603eb25f6db9239a8dc66d7063587edcc91","unresolved":false,"context_lines":[{"line_number":8,"context_line":"auth_strategy \u003d keystone"},{"line_number":9,"context_line":"notification_topics \u003d notifications"},{"line_number":10,"context_line":"rpc_backend \u003d rabbit"},{"line_number":11,"context_line":"{% if ceilometer_aodh_enabled | bool %}"},{"line_number":12,"context_line":"aodh_is_enabled \u003d True"},{"line_number":13,"context_line":"{% endif %}"},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"[oslo_policy]"},{"line_number":16,"context_line":"policy_file \u003d /etc/ceilometer/policy.json"}],"source_content_type":"text/x-jinja2","patch_set":4,"id":"7aa08908_5d08d2d9","line":13,"range":{"start_line":11,"start_character":0,"end_line":13,"end_character":11},"updated":"2016-06-16 06:34:51.000000000","message":"nit: I suppose this could be more concisely expressed as:\n\naodh_is_enabled \u003d ceilometer_aodh_enabled | bool","commit_id":"d7feaf703c8505cbe0ac1f805780b923441d3814"},{"author":{"_account_id":12807,"name":"Steve Lewis (stevelle)","email":"stevelle@gmail.com","username":"stevelle"},"change_message_id":"8a7e1ae565e7f84c852d4d69c3519f7850762c34","unresolved":false,"context_lines":[{"line_number":8,"context_line":"auth_strategy \u003d keystone"},{"line_number":9,"context_line":"notification_topics \u003d notifications"},{"line_number":10,"context_line":"rpc_backend \u003d rabbit"},{"line_number":11,"context_line":"{% if ceilometer_aodh_enabled | bool %}"},{"line_number":12,"context_line":"aodh_is_enabled \u003d True"},{"line_number":13,"context_line":"{% endif %}"},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"[oslo_policy]"},{"line_number":16,"context_line":"policy_file \u003d /etc/ceilometer/policy.json"}],"source_content_type":"text/x-jinja2","patch_set":4,"id":"7aa08908_5d4d7298","line":13,"range":{"start_line":11,"start_character":0,"end_line":13,"end_character":11},"in_reply_to":"7aa08908_5d08d2d9","updated":"2016-06-16 06:55:39.000000000","message":"Sure, I prefer that but it isn\u0027t the common pattern and I auto-piloted.","commit_id":"d7feaf703c8505cbe0ac1f805780b923441d3814"}]}