)]}'
{"specs/liberty/split-ceilometer-alarming.rst":[{"author":{"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},"change_message_id":"474ee4ea4e1c9f4f622a154ff8eadaa8671ce43e","unresolved":false,"context_lines":[{"line_number":121,"context_line":"2. Create a new Python package and repository with that isolated codebase."},{"line_number":122,"context_line":"3. Publish this repository as openstack/\u003cprojectname\u003e"},{"line_number":123,"context_line":"4. Deprecate alarming code from Ceilometer"},{"line_number":124,"context_line":"5. After Kilo: remove alarming code from Ceilometer"},{"line_number":125,"context_line":""},{"line_number":126,"context_line":""},{"line_number":127,"context_line":"Future lifecycle"}],"source_content_type":"text/x-rst","patch_set":1,"id":"7a016987_931fb86b","line":124,"updated":"2015-05-19 21:18:23.000000000","message":"Liberty","commit_id":"55041e30f3b4cc1d8d641e4e713690195ac02977"},{"author":{"_account_id":10608,"name":"Matthew Edmonds","email":"edmondsw@us.ibm.com","username":"edmondsw"},"change_message_id":"7cf6e79bbb53fe1dc9a89a42f08e7ec2954e1343","unresolved":false,"context_lines":[{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Ceilometer evolved from a a simple meter gathering component to a lot of"},{"line_number":14,"context_line":"different component doing different things. The storage layer has been"},{"line_number":15,"context_line":"abstracted during Juno and Kilo a being now is going to be handled by Gnocchi."},{"line_number":16,"context_line":"This spec proposes that the work continues so that the alarming subsystem of"},{"line_number":17,"context_line":"Ceilometer is split out as an autonomous component connected to the rest of"},{"line_number":18,"context_line":"Ceilometer and Gnocchi."}],"source_content_type":"text/x-rst","patch_set":3,"id":"7a016987_54675525","line":15,"updated":"2015-05-26 19:46:58.000000000","message":"\"a being now\" ?","commit_id":"1155e13e6108ad509ce341ffa55a4830f347691f"},{"author":{"_account_id":1669,"name":"Julien Danjou","display_name":"jd","email":"julien@danjou.info","username":"jdanjou"},"change_message_id":"db78e1f386db39f28bd685c085b42a950c99eed9","unresolved":false,"context_lines":[{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Ceilometer evolved from a a simple meter gathering component to a lot of"},{"line_number":14,"context_line":"different component doing different things. The storage layer has been"},{"line_number":15,"context_line":"abstracted during Juno and Kilo a being now is going to be handled by Gnocchi."},{"line_number":16,"context_line":"This spec proposes that the work continues so that the alarming subsystem of"},{"line_number":17,"context_line":"Ceilometer is split out as an autonomous component connected to the rest of"},{"line_number":18,"context_line":"Ceilometer and Gnocchi."}],"source_content_type":"text/x-rst","patch_set":3,"id":"7a016987_42f9c5c7","line":15,"in_reply_to":"7a016987_54675525","updated":"2015-05-27 06:27:50.000000000","message":"Done","commit_id":"1155e13e6108ad509ce341ffa55a4830f347691f"},{"author":{"_account_id":10608,"name":"Matthew Edmonds","email":"edmondsw@us.ibm.com","username":"edmondsw"},"change_message_id":"7cf6e79bbb53fe1dc9a89a42f08e7ec2954e1343","unresolved":false,"context_lines":[{"line_number":18,"context_line":"Ceilometer and Gnocchi."},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"This should help with code review, maintenance, deployment, upgrade… as having"},{"line_number":21,"context_line":"several small service-oriented components to manage is going to be easier that"},{"line_number":22,"context_line":"a single monolithic project."},{"line_number":23,"context_line":""},{"line_number":24,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"7a016987_947dcdb3","line":21,"updated":"2015-05-26 19:46:58.000000000","message":"s/that/than/","commit_id":"1155e13e6108ad509ce341ffa55a4830f347691f"},{"author":{"_account_id":1669,"name":"Julien Danjou","display_name":"jd","email":"julien@danjou.info","username":"jdanjou"},"change_message_id":"db78e1f386db39f28bd685c085b42a950c99eed9","unresolved":false,"context_lines":[{"line_number":18,"context_line":"Ceilometer and Gnocchi."},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"This should help with code review, maintenance, deployment, upgrade… as having"},{"line_number":21,"context_line":"several small service-oriented components to manage is going to be easier that"},{"line_number":22,"context_line":"a single monolithic project."},{"line_number":23,"context_line":""},{"line_number":24,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"7a016987_0203cdb9","line":21,"in_reply_to":"7a016987_947dcdb3","updated":"2015-05-27 06:27:50.000000000","message":"Done","commit_id":"1155e13e6108ad509ce341ffa55a4830f347691f"},{"author":{"_account_id":10608,"name":"Matthew Edmonds","email":"edmondsw@us.ibm.com","username":"edmondsw"},"change_message_id":"7cf6e79bbb53fe1dc9a89a42f08e7ec2954e1343","unresolved":false,"context_lines":[{"line_number":50,"context_line":"* Mezzelune https://en.wikipedia.org/wiki/Mezzelune"},{"line_number":51,"context_line":"* Ravioli https://en.wikipedia.org/wiki/Ravioli"},{"line_number":52,"context_line":"* Spätzle https://en.wikipedia.org/wiki/Sp%C3%A4tzle"},{"line_number":53,"context_line":"* Something else someone will come up in this review"},{"line_number":54,"context_line":""},{"line_number":55,"context_line":""},{"line_number":56,"context_line":"Alternatives"}],"source_content_type":"text/x-rst","patch_set":3,"id":"7a016987_c792759f","line":53,"updated":"2015-05-26 19:46:58.000000000","message":"Campanelle is a bell-shaped pasta. Alarms, bells...","commit_id":"1155e13e6108ad509ce341ffa55a4830f347691f"},{"author":{"_account_id":1669,"name":"Julien Danjou","display_name":"jd","email":"julien@danjou.info","username":"jdanjou"},"change_message_id":"db78e1f386db39f28bd685c085b42a950c99eed9","unresolved":false,"context_lines":[{"line_number":50,"context_line":"* Mezzelune https://en.wikipedia.org/wiki/Mezzelune"},{"line_number":51,"context_line":"* Ravioli https://en.wikipedia.org/wiki/Ravioli"},{"line_number":52,"context_line":"* Spätzle https://en.wikipedia.org/wiki/Sp%C3%A4tzle"},{"line_number":53,"context_line":"* Something else someone will come up in this review"},{"line_number":54,"context_line":""},{"line_number":55,"context_line":""},{"line_number":56,"context_line":"Alternatives"}],"source_content_type":"text/x-rst","patch_set":3,"id":"7a016987_622a812e","line":53,"in_reply_to":"7a016987_c792759f","updated":"2015-05-27 06:27:50.000000000","message":"Like it, added","commit_id":"1155e13e6108ad509ce341ffa55a4830f347691f"},{"author":{"_account_id":10608,"name":"Matthew Edmonds","email":"edmondsw@us.ibm.com","username":"edmondsw"},"change_message_id":"7cf6e79bbb53fe1dc9a89a42f08e7ec2954e1343","unresolved":false,"context_lines":[{"line_number":139,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":140,"context_line":""},{"line_number":141,"context_line":"The question that still remains is whether we want this new component to be"},{"line_number":142,"context_line":"integrated or not. There is not clear path to have it integrated, and the fact"},{"line_number":143,"context_line":"that Gnocchi was not integrated can weight in the balance."},{"line_number":144,"context_line":""},{"line_number":145,"context_line":"Work items 4 can be removed if the project agrees to not having a deprecation"}],"source_content_type":"text/x-rst","patch_set":3,"id":"7a016987_a719b924","line":142,"updated":"2015-05-26 19:46:58.000000000","message":"what do we mean by \"integrated\" here?","commit_id":"1155e13e6108ad509ce341ffa55a4830f347691f"},{"author":{"_account_id":1669,"name":"Julien Danjou","display_name":"jd","email":"julien@danjou.info","username":"jdanjou"},"change_message_id":"db78e1f386db39f28bd685c085b42a950c99eed9","unresolved":false,"context_lines":[{"line_number":139,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":140,"context_line":""},{"line_number":141,"context_line":"The question that still remains is whether we want this new component to be"},{"line_number":142,"context_line":"integrated or not. There is not clear path to have it integrated, and the fact"},{"line_number":143,"context_line":"that Gnocchi was not integrated can weight in the balance."},{"line_number":144,"context_line":""},{"line_number":145,"context_line":"Work items 4 can be removed if the project agrees to not having a deprecation"}],"source_content_type":"text/x-rst","patch_set":3,"id":"7a016987_4220a549","line":142,"in_reply_to":"7a016987_a719b924","updated":"2015-05-27 06:27:50.000000000","message":"\"integrated release\"","commit_id":"1155e13e6108ad509ce341ffa55a4830f347691f"},{"author":{"_account_id":10608,"name":"Matthew Edmonds","email":"edmondsw@us.ibm.com","username":"edmondsw"},"change_message_id":"7cf6e79bbb53fe1dc9a89a42f08e7ec2954e1343","unresolved":false,"context_lines":[{"line_number":143,"context_line":"that Gnocchi was not integrated can weight in the balance."},{"line_number":144,"context_line":""},{"line_number":145,"context_line":"Work items 4 can be removed if the project agrees to not having a deprecation"},{"line_number":146,"context_line":"period. Otherwise, core reviewer will have to be careful that the no code"},{"line_number":147,"context_line":"changes/fixes are merged into Ceilometer rather than the new repository."},{"line_number":148,"context_line":""},{"line_number":149,"context_line":"Dependencies"}],"source_content_type":"text/x-rst","patch_set":3,"id":"7a016987_2798296a","line":146,"updated":"2015-05-26 19:46:58.000000000","message":"Would we even consider not having a deprecation period? That surprises me.","commit_id":"1155e13e6108ad509ce341ffa55a4830f347691f"},{"author":{"_account_id":1669,"name":"Julien Danjou","display_name":"jd","email":"julien@danjou.info","username":"jdanjou"},"change_message_id":"db78e1f386db39f28bd685c085b42a950c99eed9","unresolved":false,"context_lines":[{"line_number":143,"context_line":"that Gnocchi was not integrated can weight in the balance."},{"line_number":144,"context_line":""},{"line_number":145,"context_line":"Work items 4 can be removed if the project agrees to not having a deprecation"},{"line_number":146,"context_line":"period. Otherwise, core reviewer will have to be careful that the no code"},{"line_number":147,"context_line":"changes/fixes are merged into Ceilometer rather than the new repository."},{"line_number":148,"context_line":""},{"line_number":149,"context_line":"Dependencies"}],"source_content_type":"text/x-rst","patch_set":3,"id":"7a016987_42e78512","line":146,"in_reply_to":"7a016987_2798296a","updated":"2015-05-27 06:27:50.000000000","message":"We could but I\u0027d be ok with no. :)","commit_id":"1155e13e6108ad509ce341ffa55a4830f347691f"},{"author":{"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},"change_message_id":"3ea26ed15e2afc11aeb1f408580a35643c6d7b3b","unresolved":false,"context_lines":[{"line_number":28,"context_line":"Currently, Ceilometer features can be grouped in different components, mainly:"},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"* Metric polling (Ceilometer polling agents and collector)"},{"line_number":31,"context_line":"* Event storage (Ceilometer notification agent and collector)"},{"line_number":32,"context_line":"* Alarm evaluation and triggering"},{"line_number":33,"context_line":""},{"line_number":34,"context_line":"These all 3 components are currently merged in one code base and repository,"}],"source_content_type":"text/x-rst","patch_set":4,"id":"7a016987_73fcf971","line":31,"updated":"2015-06-01 12:52:11.000000000","message":"I think making a correspondence between polling and metrics and notifications and events is bad mojo. We want to be able to get both metrics and events from notifications because ideally everything would come from notifications (_event_ually).\n\nNot that that is particularly relevant to alarming, I just don\u0027t want to see us reinforcing incorrect mental models in these specs.","commit_id":"1fd953bbe9dc0befe349be99c5f030d15de4f42e"},{"author":{"_account_id":10608,"name":"Matthew Edmonds","email":"edmondsw@us.ibm.com","username":"edmondsw"},"change_message_id":"2506849be6c1d9b6c7c32317e0a1317b87a448f6","unresolved":false,"context_lines":[{"line_number":28,"context_line":"Currently, Ceilometer features can be grouped in different components, mainly:"},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"* Metric polling (Ceilometer polling agents and collector)"},{"line_number":31,"context_line":"* Event storage (Ceilometer notification agent and collector)"},{"line_number":32,"context_line":"* Alarm evaluation and triggering"},{"line_number":33,"context_line":""},{"line_number":34,"context_line":"These all 3 components are currently merged in one code base and repository,"}],"source_content_type":"text/x-rst","patch_set":4,"id":"7a016987_8cccf235","line":31,"in_reply_to":"7a016987_73fcf971","updated":"2015-06-01 14:34:41.000000000","message":"this is a good point. To take the opposite example, one could identify events via polling. That polling may create a notification which then becomes an event, but it doesn\u0027t seem like we should preclude polling agents that create an event directly (reducing load on the notifications system).","commit_id":"1fd953bbe9dc0befe349be99c5f030d15de4f42e"},{"author":{"_account_id":1669,"name":"Julien Danjou","display_name":"jd","email":"julien@danjou.info","username":"jdanjou"},"change_message_id":"dd565d2983e9f53292adb63c2a9730da4956605b","unresolved":false,"context_lines":[{"line_number":28,"context_line":"Currently, Ceilometer features can be grouped in different components, mainly:"},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"* Metric polling (Ceilometer polling agents and collector)"},{"line_number":31,"context_line":"* Event storage (Ceilometer notification agent and collector)"},{"line_number":32,"context_line":"* Alarm evaluation and triggering"},{"line_number":33,"context_line":""},{"line_number":34,"context_line":"These all 3 components are currently merged in one code base and repository,"}],"source_content_type":"text/x-rst","patch_set":4,"id":"5afe65bd_e5035675","line":31,"in_reply_to":"7a016987_8cccf235","updated":"2015-06-01 15:45:24.000000000","message":"I\u0027ve merged them and added Gnocchi.","commit_id":"1fd953bbe9dc0befe349be99c5f030d15de4f42e"},{"author":{"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},"change_message_id":"3ea26ed15e2afc11aeb1f408580a35643c6d7b3b","unresolved":false,"context_lines":[{"line_number":45,"context_line":""},{"line_number":46,"context_line":"The new repository and project will be called either:"},{"line_number":47,"context_line":""},{"line_number":48,"context_line":"* aodh https://en.wikipedia.org/wiki/Aodh_(given_name)"},{"line_number":49,"context_line":"* Lanterne https://en.wikipedia.org/wiki/Lanterne_(pasta)"},{"line_number":50,"context_line":"* Mezzelune https://en.wikipedia.org/wiki/Mezzelune"},{"line_number":51,"context_line":"* Ravioli https://en.wikipedia.org/wiki/Ravioli"}],"source_content_type":"text/x-rst","patch_set":4,"id":"7a016987_730ab97a","line":48,"updated":"2015-06-01 12:52:11.000000000","message":"Alarm Over Data Heuristically/Helpfully/Hopefully","commit_id":"1fd953bbe9dc0befe349be99c5f030d15de4f42e"},{"author":{"_account_id":1669,"name":"Julien Danjou","display_name":"jd","email":"julien@danjou.info","username":"jdanjou"},"change_message_id":"dd565d2983e9f53292adb63c2a9730da4956605b","unresolved":false,"context_lines":[{"line_number":45,"context_line":""},{"line_number":46,"context_line":"The new repository and project will be called either:"},{"line_number":47,"context_line":""},{"line_number":48,"context_line":"* aodh https://en.wikipedia.org/wiki/Aodh_(given_name)"},{"line_number":49,"context_line":"* Lanterne https://en.wikipedia.org/wiki/Lanterne_(pasta)"},{"line_number":50,"context_line":"* Mezzelune https://en.wikipedia.org/wiki/Mezzelune"},{"line_number":51,"context_line":"* Ravioli https://en.wikipedia.org/wiki/Ravioli"}],"source_content_type":"text/x-rst","patch_set":4,"id":"5afe65bd_c593ba4e","line":48,"in_reply_to":"7a016987_730ab97a","updated":"2015-06-01 15:45:24.000000000","message":"+1","commit_id":"1fd953bbe9dc0befe349be99c5f030d15de4f42e"},{"author":{"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},"change_message_id":"3ea26ed15e2afc11aeb1f408580a35643c6d7b3b","unresolved":false,"context_lines":[{"line_number":71,"context_line":"The alarming API should not change and continues to be contained in"},{"line_number":72,"context_line":"`/v2/alarms`. However, the `ceilometer-api` endpoint and daemons is going to be"},{"line_number":73,"context_line":"replace by a specific REST API daemon only implementing the alarming subsystem."},{"line_number":74,"context_line":"This means a new Keystone endpoint will need to be created and registered."},{"line_number":75,"context_line":""},{"line_number":76,"context_line":"Security impact"},{"line_number":77,"context_line":"---------------"}],"source_content_type":"text/x-rst","patch_set":4,"id":"7a016987_36a66fae","line":74,"updated":"2015-06-01 12:52:11.000000000","message":"I hate to suggest this, but should we consider a proxy or documentation that would explain configuration of apache so that /v2/alarms would go to the new code under the old port?","commit_id":"1fd953bbe9dc0befe349be99c5f030d15de4f42e"},{"author":{"_account_id":1669,"name":"Julien Danjou","display_name":"jd","email":"julien@danjou.info","username":"jdanjou"},"change_message_id":"dd565d2983e9f53292adb63c2a9730da4956605b","unresolved":false,"context_lines":[{"line_number":71,"context_line":"The alarming API should not change and continues to be contained in"},{"line_number":72,"context_line":"`/v2/alarms`. However, the `ceilometer-api` endpoint and daemons is going to be"},{"line_number":73,"context_line":"replace by a specific REST API daemon only implementing the alarming subsystem."},{"line_number":74,"context_line":"This means a new Keystone endpoint will need to be created and registered."},{"line_number":75,"context_line":""},{"line_number":76,"context_line":"Security impact"},{"line_number":77,"context_line":"---------------"}],"source_content_type":"text/x-rst","patch_set":4,"id":"5afe65bd_a5aafe83","line":74,"in_reply_to":"7a016987_36a66fae","updated":"2015-06-01 15:45:24.000000000","message":"Done","commit_id":"1fd953bbe9dc0befe349be99c5f030d15de4f42e"},{"author":{"_account_id":10608,"name":"Matthew Edmonds","email":"edmondsw@us.ibm.com","username":"edmondsw"},"change_message_id":"2506849be6c1d9b6c7c32317e0a1317b87a448f6","unresolved":false,"context_lines":[{"line_number":76,"context_line":"Security impact"},{"line_number":77,"context_line":"---------------"},{"line_number":78,"context_line":""},{"line_number":79,"context_line":"None"},{"line_number":80,"context_line":""},{"line_number":81,"context_line":"Pipeline impact"},{"line_number":82,"context_line":"---------------"}],"source_content_type":"text/x-rst","patch_set":4,"id":"7a016987_7dd5e240","line":79,"updated":"2015-06-01 14:34:41.000000000","message":"I\u0027d like to see something here. At least saying that security-relevant things like policy, auditing, etc. will be no worse than they were in ceilometer.","commit_id":"1fd953bbe9dc0befe349be99c5f030d15de4f42e"},{"author":{"_account_id":1669,"name":"Julien Danjou","display_name":"jd","email":"julien@danjou.info","username":"jdanjou"},"change_message_id":"dd565d2983e9f53292adb63c2a9730da4956605b","unresolved":false,"context_lines":[{"line_number":76,"context_line":"Security impact"},{"line_number":77,"context_line":"---------------"},{"line_number":78,"context_line":""},{"line_number":79,"context_line":"None"},{"line_number":80,"context_line":""},{"line_number":81,"context_line":"Pipeline impact"},{"line_number":82,"context_line":"---------------"}],"source_content_type":"text/x-rst","patch_set":4,"id":"5afe65bd_657c0621","line":79,"in_reply_to":"7a016987_7dd5e240","updated":"2015-06-01 15:45:24.000000000","message":"That\u0027s what impact \u003d None is saying, no?","commit_id":"1fd953bbe9dc0befe349be99c5f030d15de4f42e"},{"author":{"_account_id":7075,"name":"Swann Croiset","email":"scroiset@mirantis.com","username":"swann"},"change_message_id":"f10a9fa70f89d084950443f907db93dd5727deec","unresolved":false,"context_lines":[{"line_number":130,"context_line":"----------"},{"line_number":131,"context_line":""},{"line_number":132,"context_line":"1. Isolate Ceilometer alarming codebase from the other part of Ceilometer."},{"line_number":133,"context_line":"2. Create a new Python package and repository with that isolated codebase."},{"line_number":134,"context_line":"3. Publish this repository as openstack/\u003cprojectname\u003e"},{"line_number":135,"context_line":"4. Deprecate alarming code from Ceilometer"},{"line_number":136,"context_line":"5. After Liberty: remove alarming code from Ceilometer"}],"source_content_type":"text/x-rst","patch_set":4,"id":"7a016987_585bf82e","line":133,"range":{"start_line":133,"start_character":3,"end_line":133,"end_character":74},"updated":"2015-05-27 07:48:04.000000000","message":"what about client side? will be also split ? I would say yes","commit_id":"1fd953bbe9dc0befe349be99c5f030d15de4f42e"},{"author":{"_account_id":7075,"name":"Swann Croiset","email":"scroiset@mirantis.com","username":"swann"},"change_message_id":"2ae10e35a5c8dff5b1686251321d20ad31a0f616","unresolved":false,"context_lines":[{"line_number":130,"context_line":"----------"},{"line_number":131,"context_line":""},{"line_number":132,"context_line":"1. Isolate Ceilometer alarming codebase from the other part of Ceilometer."},{"line_number":133,"context_line":"2. Create a new Python package and repository with that isolated codebase."},{"line_number":134,"context_line":"3. Publish this repository as openstack/\u003cprojectname\u003e"},{"line_number":135,"context_line":"4. Deprecate alarming code from Ceilometer"},{"line_number":136,"context_line":"5. After Liberty: remove alarming code from Ceilometer"}],"source_content_type":"text/x-rst","patch_set":4,"id":"7a016987_2975f062","line":133,"in_reply_to":"7a016987_3629cfe6","updated":"2015-06-01 15:35:26.000000000","message":"I mean for the cleanliness","commit_id":"1fd953bbe9dc0befe349be99c5f030d15de4f42e"},{"author":{"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},"change_message_id":"3ea26ed15e2afc11aeb1f408580a35643c6d7b3b","unresolved":false,"context_lines":[{"line_number":130,"context_line":"----------"},{"line_number":131,"context_line":""},{"line_number":132,"context_line":"1. Isolate Ceilometer alarming codebase from the other part of Ceilometer."},{"line_number":133,"context_line":"2. Create a new Python package and repository with that isolated codebase."},{"line_number":134,"context_line":"3. Publish this repository as openstack/\u003cprojectname\u003e"},{"line_number":135,"context_line":"4. Deprecate alarming code from Ceilometer"},{"line_number":136,"context_line":"5. After Liberty: remove alarming code from Ceilometer"}],"source_content_type":"text/x-rst","patch_set":4,"id":"7a016987_3629cfe6","line":133,"in_reply_to":"7a016987_585bf82e","updated":"2015-06-01 12:52:11.000000000","message":"What\u0027s the benefit of splitting out the client? The same cleanliness being discussed here or something else?","commit_id":"1fd953bbe9dc0befe349be99c5f030d15de4f42e"},{"author":{"_account_id":7075,"name":"Swann Croiset","email":"scroiset@mirantis.com","username":"swann"},"change_message_id":"f10a9fa70f89d084950443f907db93dd5727deec","unresolved":false,"context_lines":[{"line_number":151,"context_line":"Dependencies"},{"line_number":152,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":153,"context_line":""},{"line_number":154,"context_line":"None"},{"line_number":155,"context_line":""},{"line_number":156,"context_line":"Testing"},{"line_number":157,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":4,"id":"7a016987_18f6a0d7","line":154,"range":{"start_line":154,"start_character":0,"end_line":154,"end_character":4},"updated":"2015-05-27 07:48:04.000000000","message":"Heat has a direct dependency on ceilometer for alarms (maybe even in gate?), depending of deprecation period chosen here, the same politic must be follow by Heat.\nThat said, I don\u0027t know how to manage coordination between projects and if you should write down this on the spec.","commit_id":"1fd953bbe9dc0befe349be99c5f030d15de4f42e"},{"author":{"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},"change_message_id":"3ea26ed15e2afc11aeb1f408580a35643c6d7b3b","unresolved":false,"context_lines":[{"line_number":168,"context_line":"References"},{"line_number":169,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":170,"context_line":""},{"line_number":171,"context_line":"* Kilo Summit Etherpad https://etherpad.openstack.org/p/ceilo-multi-identity"}],"source_content_type":"text/x-rst","patch_set":4,"id":"7a016987_166b731f","line":171,"updated":"2015-06-01 12:52:11.000000000","message":"s/Kilo/Liberty/","commit_id":"1fd953bbe9dc0befe349be99c5f030d15de4f42e"},{"author":{"_account_id":1669,"name":"Julien Danjou","display_name":"jd","email":"julien@danjou.info","username":"jdanjou"},"change_message_id":"dd565d2983e9f53292adb63c2a9730da4956605b","unresolved":false,"context_lines":[{"line_number":168,"context_line":"References"},{"line_number":169,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":170,"context_line":""},{"line_number":171,"context_line":"* Kilo Summit Etherpad https://etherpad.openstack.org/p/ceilo-multi-identity"}],"source_content_type":"text/x-rst","patch_set":4,"id":"5afe65bd_e5d23634","line":171,"in_reply_to":"7a016987_166b731f","updated":"2015-06-01 15:45:24.000000000","message":"Done","commit_id":"1fd953bbe9dc0befe349be99c5f030d15de4f42e"},{"author":{"_account_id":2284,"name":"Eoghan Glynn","email":"eglynn@redhat.com","username":"eglynn"},"change_message_id":"f41caaeea089bd406b1349f842ebbf88a6775af9","unresolved":false,"context_lines":[{"line_number":46,"context_line":"The new repository and project will be called either:"},{"line_number":47,"context_line":""},{"line_number":48,"context_line":"* aodh https://en.wikipedia.org/wiki/Aodh_(given_name)"},{"line_number":49,"context_line":"  (Alarm Over Data Heuristically/Helpfully/Hopefully)"},{"line_number":50,"context_line":"* Lanterne https://en.wikipedia.org/wiki/Lanterne_(pasta)"},{"line_number":51,"context_line":"* Mezzelune https://en.wikipedia.org/wiki/Mezzelune"},{"line_number":52,"context_line":"* Ravioli https://en.wikipedia.org/wiki/Ravioli"}],"source_content_type":"text/x-rst","patch_set":5,"id":"5afe65bd_a4a9d7e1","line":49,"updated":"2015-06-01 19:52:34.000000000","message":"I\u0027m loving Aodh for this :)","commit_id":"8b43014536baf4baa25791c85d4176638b9df3f8"},{"author":{"_account_id":7052,"name":"Fabio Giannetti","email":"fgiannet@cisco.com","username":"fabgia"},"change_message_id":"ea54437d433ecbe27c74c1dcf00188bd75490273","unresolved":false,"context_lines":[{"line_number":53,"context_line":"* Spätzle https://en.wikipedia.org/wiki/Sp%C3%A4tzle"},{"line_number":54,"context_line":"* Campanelle https://en.wikipedia.org/wiki/Campanelle (a bell-shaped pasta. Alarms, bells…)"},{"line_number":55,"context_line":"* Something else someone will come up in this review"},{"line_number":56,"context_line":""},{"line_number":57,"context_line":""},{"line_number":58,"context_line":"Alternatives"},{"line_number":59,"context_line":"------------"}],"source_content_type":"text/x-rst","patch_set":5,"id":"5afe65bd_882a772b","line":56,"updated":"2015-06-01 16:02:21.000000000","message":"If we are splitting Ceilometer into components, I believe we should all prefix them with ceilometer.\nThe reason is that people know Ceilometer and we may risk to get a lot of people still contribute to the alarms in ceilometer not realizing that this functionality has now moved to a specific dedicated package. So, IMHO we should call any new package ceilometer-\u003cfunc\u003e.","commit_id":"8b43014536baf4baa25791c85d4176638b9df3f8"},{"author":{"_account_id":1894,"name":"Ryota MIBU","email":"r-mibu@cq.jp.nec.com","username":"r-mibu"},"change_message_id":"c816a24d7380f0503430ba0287ca1b2075d1f994","unresolved":false,"context_lines":[{"line_number":53,"context_line":"* Spätzle https://en.wikipedia.org/wiki/Sp%C3%A4tzle"},{"line_number":54,"context_line":"* Campanelle https://en.wikipedia.org/wiki/Campanelle (a bell-shaped pasta. Alarms, bells…)"},{"line_number":55,"context_line":"* Something else someone will come up in this review"},{"line_number":56,"context_line":""},{"line_number":57,"context_line":""},{"line_number":58,"context_line":"Alternatives"},{"line_number":59,"context_line":"------------"}],"source_content_type":"text/x-rst","patch_set":5,"id":"5afe65bd_2f2e52a5","line":56,"in_reply_to":"5afe65bd_882a772b","updated":"2015-06-04 17:34:52.000000000","message":"+1\n\nI prefer \"ceilometer-alarm\", since we are not creating something new but just splitting code. It should be nice to express that there is no change in functionality and API.","commit_id":"8b43014536baf4baa25791c85d4176638b9df3f8"},{"author":{"_account_id":2284,"name":"Eoghan Glynn","email":"eglynn@redhat.com","username":"eglynn"},"change_message_id":"f41caaeea089bd406b1349f842ebbf88a6775af9","unresolved":false,"context_lines":[{"line_number":76,"context_line":""},{"line_number":77,"context_line":"We\u0027ll also provide a documentation about how to support configuring a proxy"},{"line_number":78,"context_line":"that can route requests to the Ceilometer API endpoint or the new alarm API"},{"line_number":79,"context_line":"endpoint."},{"line_number":80,"context_line":""},{"line_number":81,"context_line":"Security impact"},{"line_number":82,"context_line":"---------------"}],"source_content_type":"text/x-rst","patch_set":5,"id":"5afe65bd_04e9cb1a","line":79,"updated":"2015-06-01 19:52:34.000000000","message":"For simple (e.g. AIO) deployments, would we still support all APIs collocated in a single process?","commit_id":"8b43014536baf4baa25791c85d4176638b9df3f8"},{"author":{"_account_id":2284,"name":"Eoghan Glynn","email":"eglynn@redhat.com","username":"eglynn"},"change_message_id":"f41caaeea089bd406b1349f842ebbf88a6775af9","unresolved":false,"context_lines":[{"line_number":92,"context_line":"---------------------"},{"line_number":93,"context_line":""},{"line_number":94,"context_line":"As stated above, this will require a new Keystone endpoint to manipulate"},{"line_number":95,"context_line":"alarms."},{"line_number":96,"context_line":""},{"line_number":97,"context_line":"Performance/Scalability Impacts"},{"line_number":98,"context_line":"-------------------------------"}],"source_content_type":"text/x-rst","patch_set":5,"id":"5afe65bd_44e7c340","line":95,"updated":"2015-06-01 19:52:34.000000000","message":"Need to split the client side also?\n\n(Probably not, I\u0027d guess)","commit_id":"8b43014536baf4baa25791c85d4176638b9df3f8"},{"author":{"_account_id":2284,"name":"Eoghan Glynn","email":"eglynn@redhat.com","username":"eglynn"},"change_message_id":"f41caaeea089bd406b1349f842ebbf88a6775af9","unresolved":false,"context_lines":[{"line_number":99,"context_line":""},{"line_number":100,"context_line":"It\u0027s easy to envision a better scalability if anything. It\u0027ll be easier to"},{"line_number":101,"context_line":"scale the API endpoint for alarming only. Also, performances might be a little"},{"line_number":102,"context_line":"improved as the alarming codebase could be smaller."},{"line_number":103,"context_line":""},{"line_number":104,"context_line":"Other deployer impact"},{"line_number":105,"context_line":"---------------------"}],"source_content_type":"text/x-rst","patch_set":5,"id":"5afe65bd_04378bb1","line":102,"updated":"2015-06-01 19:52:34.000000000","message":"I don\u0027t see much or any impact on the performance of polling the v2 statistics or gnocchi APIs.","commit_id":"8b43014536baf4baa25791c85d4176638b9df3f8"},{"author":{"_account_id":2284,"name":"Eoghan Glynn","email":"eglynn@redhat.com","username":"eglynn"},"change_message_id":"f41caaeea089bd406b1349f842ebbf88a6775af9","unresolved":false,"context_lines":[{"line_number":107,"context_line":"The configuration file used by the alarming subsystem when it\u0027ll be part of its"},{"line_number":108,"context_line":"own project/repository should be split out of `ceilometer.conf`. That means a"},{"line_number":109,"context_line":"new configuration file, but close to the current `ceilometer.conf` file, should"},{"line_number":110,"context_line":"be created."},{"line_number":111,"context_line":""},{"line_number":112,"context_line":"Other than that, no option should be added or removed."},{"line_number":113,"context_line":""}],"source_content_type":"text/x-rst","patch_set":5,"id":"5afe65bd_646f5f93","line":110,"updated":"2015-06-01 19:52:34.000000000","message":"One thing to bare in mind is the implicit coupling between the alarm evaluation interval and the coresponding polling interval in the ceilo pipeline.yaml.\n\n(the one should be a whole number multiple of the other to avoid spurious INSUFFICIENT_DATA transitions)","commit_id":"8b43014536baf4baa25791c85d4176638b9df3f8"},{"author":{"_account_id":2284,"name":"Eoghan Glynn","email":"eglynn@redhat.com","username":"eglynn"},"change_message_id":"f41caaeea089bd406b1349f842ebbf88a6775af9","unresolved":false,"context_lines":[{"line_number":138,"context_line":"2. Create a new Python package and repository with that isolated codebase."},{"line_number":139,"context_line":"3. Publish this repository as openstack/\u003cprojectname\u003e"},{"line_number":140,"context_line":"4. Deprecate alarming code from Ceilometer"},{"line_number":141,"context_line":"5. After Liberty: remove alarming code from Ceilometer"},{"line_number":142,"context_line":""},{"line_number":143,"context_line":""},{"line_number":144,"context_line":"Future lifecycle"}],"source_content_type":"text/x-rst","patch_set":5,"id":"5afe65bd_24c5e75a","line":141,"updated":"2015-06-01 19:52:34.000000000","message":"6. Contribute to the a downstream switch-over effort to migrate existing Fedora/Deb packages and puppet modules.","commit_id":"8b43014536baf4baa25791c85d4176638b9df3f8"},{"author":{"_account_id":2284,"name":"Eoghan Glynn","email":"eglynn@redhat.com","username":"eglynn"},"change_message_id":"f41caaeea089bd406b1349f842ebbf88a6775af9","unresolved":false,"context_lines":[{"line_number":145,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":146,"context_line":""},{"line_number":147,"context_line":"The question that still remains is whether we want this new component to be"},{"line_number":148,"context_line":"integrated as part of the \"integrated release\" or not. There is not clear path"},{"line_number":149,"context_line":"to have it integrated, and the fact that Gnocchi was not integrated can weight"},{"line_number":150,"context_line":"in the balance."},{"line_number":151,"context_line":""}],"source_content_type":"text/x-rst","patch_set":5,"id":"5afe65bd_a4e1d700","line":148,"updated":"2015-06-01 19:52:34.000000000","message":"Surely there will not be much meaning remaining in the integrated release concept, in the post-kilo big tent world?\n\nWill the concept even exist, beyond the presence of a legacy tag?\n\nAnd even then, that tag is applied to projects (such as ceilo) IIUC, as opposed to individual repos (such as openstack/aodh or openstack/gnocchi).","commit_id":"8b43014536baf4baa25791c85d4176638b9df3f8"},{"author":{"_account_id":6537,"name":"gordon chung","email":"gord@live.ca","username":"chungg"},"change_message_id":"5cec20d5ca330b90fc67380b44c7581180b5003b","unresolved":false,"context_lines":[{"line_number":112,"context_line":"Other than that, no option should be added or removed."},{"line_number":113,"context_line":""},{"line_number":114,"context_line":"The daemon will remain the same and the API will be deployed in the same way –"},{"line_number":115,"context_line":"either with a (new) daemon either through the recommended WSGI interface."},{"line_number":116,"context_line":""},{"line_number":117,"context_line":"Developer impact"},{"line_number":118,"context_line":"----------------"}],"source_content_type":"text/x-rst","patch_set":6,"id":"3afb71cf_deedc526","line":115,"updated":"2015-06-04 21:58:02.000000000","message":"can we add a blurb on how we will manage versioning/release of each lib to ensure compatibility? ie. how aodh  will work with current ceilometer base","commit_id":"2bbf315895add7aa2354977876d202bfc24f3600"},{"author":{"_account_id":1669,"name":"Julien Danjou","display_name":"jd","email":"julien@danjou.info","username":"jdanjou"},"change_message_id":"20e399afe6ce37dab2aedc14df326c783db90c92","unresolved":false,"context_lines":[{"line_number":112,"context_line":"Other than that, no option should be added or removed."},{"line_number":113,"context_line":""},{"line_number":114,"context_line":"The daemon will remain the same and the API will be deployed in the same way –"},{"line_number":115,"context_line":"either with a (new) daemon either through the recommended WSGI interface."},{"line_number":116,"context_line":""},{"line_number":117,"context_line":"Developer impact"},{"line_number":118,"context_line":"----------------"}],"source_content_type":"text/x-rst","patch_set":6,"id":"3afb71cf_3c74d79a","line":115,"in_reply_to":"3afb71cf_deedc526","updated":"2015-06-05 06:58:34.000000000","message":"Done","commit_id":"2bbf315895add7aa2354977876d202bfc24f3600"},{"author":{"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},"change_message_id":"260c1a7ff6b49cd1a6e167a0670b2f55f9e5e729","unresolved":false,"context_lines":[{"line_number":136,"context_line":""},{"line_number":137,"context_line":"1. Isolate Ceilometer alarming codebase from the other part of Ceilometer."},{"line_number":138,"context_line":"2. Create a new Python package and repository with that isolated codebase."},{"line_number":139,"context_line":"3. Publish this repository as openstack/\u003cprojectname\u003e"},{"line_number":140,"context_line":"4. Deprecate alarming code from Ceilometer"},{"line_number":141,"context_line":"5. After Liberty: remove alarming code from Ceilometer"},{"line_number":142,"context_line":"6. Contribute to the a downstream switch-over effort to migrate existing"}],"source_content_type":"text/x-rst","patch_set":6,"id":"5afe65bd_496ea5ca","line":139,"updated":"2015-06-02 09:47:31.000000000","message":"There\u0027s a step or two in here about addressing devstack and grenade\u0027s handling of the alarming service. How that\u0027s done will depend on whether this change is complete before or after the migration of ceilometer to using devstack and grenade plugins. It seems quite likely this will be done before that.","commit_id":"2bbf315895add7aa2354977876d202bfc24f3600"},{"author":{"_account_id":1669,"name":"Julien Danjou","display_name":"jd","email":"julien@danjou.info","username":"jdanjou"},"change_message_id":"20e399afe6ce37dab2aedc14df326c783db90c92","unresolved":false,"context_lines":[{"line_number":136,"context_line":""},{"line_number":137,"context_line":"1. Isolate Ceilometer alarming codebase from the other part of Ceilometer."},{"line_number":138,"context_line":"2. Create a new Python package and repository with that isolated codebase."},{"line_number":139,"context_line":"3. Publish this repository as openstack/\u003cprojectname\u003e"},{"line_number":140,"context_line":"4. Deprecate alarming code from Ceilometer"},{"line_number":141,"context_line":"5. After Liberty: remove alarming code from Ceilometer"},{"line_number":142,"context_line":"6. Contribute to the a downstream switch-over effort to migrate existing"}],"source_content_type":"text/x-rst","patch_set":6,"id":"3afb71cf_bc68c7bb","line":139,"in_reply_to":"5afe65bd_496ea5ca","updated":"2015-06-05 06:58:34.000000000","message":"Done","commit_id":"2bbf315895add7aa2354977876d202bfc24f3600"},{"author":{"_account_id":6537,"name":"gordon chung","email":"gord@live.ca","username":"chungg"},"change_message_id":"5640d9c1550aaaab6319a30b6940b8bc105c1241","unresolved":false,"context_lines":[{"line_number":43,"context_line":"the alarming subsystem out of Ceilometer main repository and having it living"},{"line_number":44,"context_line":"its own life in its own repository."},{"line_number":45,"context_line":""},{"line_number":46,"context_line":"The new repository and project will be called either *aodh*."},{"line_number":47,"context_line":""},{"line_number":48,"context_line":"Alternatives"},{"line_number":49,"context_line":"------------"}],"source_content_type":"text/x-rst","patch_set":7,"id":"3afb71cf_a2f8394a","line":46,"updated":"2015-06-09 04:08:40.000000000","message":"s/called either *aodh*/called: *aodh*/","commit_id":"756cab3e087f0dd2dd8c9a9bfaa248525a21d29c"},{"author":{"_account_id":1894,"name":"Ryota MIBU","email":"r-mibu@cq.jp.nec.com","username":"r-mibu"},"change_message_id":"7c7aa1fdec79ad620a1f9af2a05d31cedb94ac0d","unresolved":false,"context_lines":[{"line_number":43,"context_line":"the alarming subsystem out of Ceilometer main repository and having it living"},{"line_number":44,"context_line":"its own life in its own repository."},{"line_number":45,"context_line":""},{"line_number":46,"context_line":"The new repository and project will be called either *aodh*."},{"line_number":47,"context_line":""},{"line_number":48,"context_line":"Alternatives"},{"line_number":49,"context_line":"------------"}],"source_content_type":"text/x-rst","patch_set":7,"id":"3afb71cf_5dd72623","line":46,"in_reply_to":"3afb71cf_a2f8394a","updated":"2015-06-09 04:41:34.000000000","message":"Why don\u0027t we leave the description?\n\n    https://en.wikipedia.org/wiki/Aodh_(given_name)\n    Alarm Over Data Heuristically/Helpfully/Hopefully","commit_id":"756cab3e087f0dd2dd8c9a9bfaa248525a21d29c"}]}