)]}'
{"specs/juno/delete-alarmhistory.rst":[{"author":{"_account_id":6676,"name":"zqfan","email":"aji.zqfan@gmail.com","username":"aji-zqfan"},"change_message_id":"3264d671605feba290a2ea027facfcfeaa2f0ecc","unresolved":false,"context_lines":[{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Currently the ceilometer-expirer doesn\u0027t delete expired AlarmChanges."},{"line_number":14,"context_line":"Remained AlarmChanges would be cause of wasted disk-space and slow response."},{"line_number":15,"context_line":"This blueprint add the function of deleting alarm history."},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"Problem description"},{"line_number":18,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":1,"id":"baada198_814cd943","line":15,"updated":"2014-07-04 08:33:55.000000000","message":"nit: s/add/adds/","commit_id":"9bf2ef1cdeacca86f2b9a436c412816c8df95b69"},{"author":{"_account_id":8988,"name":"Rikimaru Honjo","email":"honjo.rikimaru@ntt-tx.co.jp","username":"rikimaru.honjo","inactive":true},"change_message_id":"3cb9ddf88db03a0dbda36b63e11eed5f233590c3","unresolved":false,"context_lines":[{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Currently the ceilometer-expirer doesn\u0027t delete expired AlarmChanges."},{"line_number":14,"context_line":"Remained AlarmChanges would be cause of wasted disk-space and slow response."},{"line_number":15,"context_line":"This blueprint add the function of deleting alarm history."},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"Problem description"},{"line_number":18,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":1,"id":"baada198_fb517230","line":15,"in_reply_to":"baada198_814cd943","updated":"2014-07-08 08:02:25.000000000","message":"will do.","commit_id":"9bf2ef1cdeacca86f2b9a436c412816c8df95b69"},{"author":{"_account_id":6676,"name":"zqfan","email":"aji.zqfan@gmail.com","username":"aji-zqfan"},"change_message_id":"3264d671605feba290a2ea027facfcfeaa2f0ecc","unresolved":false,"context_lines":[{"line_number":82,"context_line":"Other deployer impact"},{"line_number":83,"context_line":"---------------------"},{"line_number":84,"context_line":""},{"line_number":85,"context_line":"None"},{"line_number":86,"context_line":""},{"line_number":87,"context_line":"Developer impact"},{"line_number":88,"context_line":"----------------"}],"source_content_type":"text/x-rst","patch_set":1,"id":"baada198_61298536","line":85,"updated":"2014-07-04 08:33:55.000000000","message":"nit: can have since there are a new option and a renamed option, and ceilometer-expirer has new functionality","commit_id":"9bf2ef1cdeacca86f2b9a436c412816c8df95b69"},{"author":{"_account_id":8988,"name":"Rikimaru Honjo","email":"honjo.rikimaru@ntt-tx.co.jp","username":"rikimaru.honjo","inactive":true},"change_message_id":"3cb9ddf88db03a0dbda36b63e11eed5f233590c3","unresolved":false,"context_lines":[{"line_number":82,"context_line":"Other deployer impact"},{"line_number":83,"context_line":"---------------------"},{"line_number":84,"context_line":""},{"line_number":85,"context_line":"None"},{"line_number":86,"context_line":""},{"line_number":87,"context_line":"Developer impact"},{"line_number":88,"context_line":"----------------"}],"source_content_type":"text/x-rst","patch_set":1,"id":"baada198_1b3466fa","line":85,"in_reply_to":"baada198_61298536","updated":"2014-07-08 08:02:25.000000000","message":"will do.","commit_id":"9bf2ef1cdeacca86f2b9a436c412816c8df95b69"},{"author":{"_account_id":6676,"name":"zqfan","email":"aji.zqfan@gmail.com","username":"aji-zqfan"},"change_message_id":"3264d671605feba290a2ea027facfcfeaa2f0ecc","unresolved":false,"context_lines":[{"line_number":87,"context_line":"Developer impact"},{"line_number":88,"context_line":"----------------"},{"line_number":89,"context_line":""},{"line_number":90,"context_line":"None"},{"line_number":91,"context_line":""},{"line_number":92,"context_line":""},{"line_number":93,"context_line":"Implementation"}],"source_content_type":"text/x-rst","patch_set":1,"id":"baada198_01d5291b","line":90,"updated":"2014-07-04 08:33:55.000000000","message":"nit: can have since one of storage api has been changed","commit_id":"9bf2ef1cdeacca86f2b9a436c412816c8df95b69"},{"author":{"_account_id":8988,"name":"Rikimaru Honjo","email":"honjo.rikimaru@ntt-tx.co.jp","username":"rikimaru.honjo","inactive":true},"change_message_id":"3cb9ddf88db03a0dbda36b63e11eed5f233590c3","unresolved":false,"context_lines":[{"line_number":87,"context_line":"Developer impact"},{"line_number":88,"context_line":"----------------"},{"line_number":89,"context_line":""},{"line_number":90,"context_line":"None"},{"line_number":91,"context_line":""},{"line_number":92,"context_line":""},{"line_number":93,"context_line":"Implementation"}],"source_content_type":"text/x-rst","patch_set":1,"id":"baada198_3b376a00","line":90,"in_reply_to":"baada198_01d5291b","updated":"2014-07-08 08:02:25.000000000","message":"will do.","commit_id":"9bf2ef1cdeacca86f2b9a436c412816c8df95b69"},{"author":{"_account_id":6676,"name":"zqfan","email":"aji.zqfan@gmail.com","username":"aji-zqfan"},"change_message_id":"3264d671605feba290a2ea027facfcfeaa2f0ecc","unresolved":false,"context_lines":[{"line_number":134,"context_line":"Documentation Impact"},{"line_number":135,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":136,"context_line":""},{"line_number":137,"context_line":"None"},{"line_number":138,"context_line":""},{"line_number":139,"context_line":"References"},{"line_number":140,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":1,"id":"baada198_e1b83513","line":137,"updated":"2014-07-04 08:33:55.000000000","message":"-1: there is doc impact, since you try to rename an existent option and add a new option (which means doc/source/configuration should be updated, notify openstack-manuals about this change will be nicer), then change a command\u0027s original behavior which will clean an additional table now, even modify the storage api (that can be handled via doc string)","commit_id":"9bf2ef1cdeacca86f2b9a436c412816c8df95b69"},{"author":{"_account_id":8988,"name":"Rikimaru Honjo","email":"honjo.rikimaru@ntt-tx.co.jp","username":"rikimaru.honjo","inactive":true},"change_message_id":"3cb9ddf88db03a0dbda36b63e11eed5f233590c3","unresolved":false,"context_lines":[{"line_number":134,"context_line":"Documentation Impact"},{"line_number":135,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":136,"context_line":""},{"line_number":137,"context_line":"None"},{"line_number":138,"context_line":""},{"line_number":139,"context_line":"References"},{"line_number":140,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":1,"id":"baada198_1b1b8681","line":137,"in_reply_to":"baada198_e1b83513","updated":"2014-07-08 08:02:25.000000000","message":"Will not do, since there are no database section in configuration.rst (and expire/TTL documentation does not exists). It is possible to document TTLs only, though, it will be preferable to write the database section, which should be treated as a separate BP.","commit_id":"9bf2ef1cdeacca86f2b9a436c412816c8df95b69"},{"author":{"_account_id":4491,"name":"Lianhao Lu","email":"llh_misc@outlook.com","username":"lianhao-lu"},"change_message_id":"3c5d102d31f86fc492e188f5226c66e101818fd8","unresolved":false,"context_lines":[{"line_number":109,"context_line":"* Code the implementation of alarm_history_time_to_live."},{"line_number":110,"context_line":"* Code the implementation of deleting alarm history in impl_sqlalchemy."},{"line_number":111,"context_line":"* Code the implementation of deleting alarm history in impl_mongodb."},{"line_number":112,"context_line":"* Code the implementation of deleting alarm history in impl_log."},{"line_number":113,"context_line":"* Code the implementation of deleting alarm history in impl_hbase, impl_db2."},{"line_number":114,"context_line":""},{"line_number":115,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"baada198_e24dde3c","line":112,"updated":"2014-07-31 05:55:38.000000000","message":"is it doable in iml_log? I don\u0027t think so","commit_id":"2e981209fa6c9d8a2c3c100bfbaf48ab432b4efe"},{"author":{"_account_id":8988,"name":"Rikimaru Honjo","email":"honjo.rikimaru@ntt-tx.co.jp","username":"rikimaru.honjo","inactive":true},"change_message_id":"261738aef86952b4779456967195f75e04b74be9","unresolved":false,"context_lines":[{"line_number":109,"context_line":"* Code the implementation of alarm_history_time_to_live."},{"line_number":110,"context_line":"* Code the implementation of deleting alarm history in impl_sqlalchemy."},{"line_number":111,"context_line":"* Code the implementation of deleting alarm history in impl_mongodb."},{"line_number":112,"context_line":"* Code the implementation of deleting alarm history in impl_log."},{"line_number":113,"context_line":"* Code the implementation of deleting alarm history in impl_hbase, impl_db2."},{"line_number":114,"context_line":""},{"line_number":115,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"baada198_ff4ef36c","line":112,"in_reply_to":"baada198_e24dde3c","updated":"2014-08-11 13:14:07.000000000","message":"I\u0027ve fixed.","commit_id":"2e981209fa6c9d8a2c3c100bfbaf48ab432b4efe"},{"author":{"_account_id":4491,"name":"Lianhao Lu","email":"llh_misc@outlook.com","username":"lianhao-lu"},"change_message_id":"3c5d102d31f86fc492e188f5226c66e101818fd8","unresolved":false,"context_lines":[{"line_number":123,"context_line":""},{"line_number":124,"context_line":"* It is preferable to merge this functionality after we split database to"},{"line_number":125,"context_line":"  add support for multiple backends by the following blueprint."},{"line_number":126,"context_line":"  https://blueprints.launchpad.net/ceilometer/+spec/dedicated-alarm-database"},{"line_number":127,"context_line":""},{"line_number":128,"context_line":""},{"line_number":129,"context_line":"Testing"}],"source_content_type":"text/x-rst","patch_set":2,"id":"baada198_42568acf","line":126,"updated":"2014-07-31 05:55:38.000000000","message":"looks like the bp is already completed, would you please update the relevant information in the above sections?","commit_id":"2e981209fa6c9d8a2c3c100bfbaf48ab432b4efe"},{"author":{"_account_id":8988,"name":"Rikimaru Honjo","email":"honjo.rikimaru@ntt-tx.co.jp","username":"rikimaru.honjo","inactive":true},"change_message_id":"261738aef86952b4779456967195f75e04b74be9","unresolved":false,"context_lines":[{"line_number":123,"context_line":""},{"line_number":124,"context_line":"* It is preferable to merge this functionality after we split database to"},{"line_number":125,"context_line":"  add support for multiple backends by the following blueprint."},{"line_number":126,"context_line":"  https://blueprints.launchpad.net/ceilometer/+spec/dedicated-alarm-database"},{"line_number":127,"context_line":""},{"line_number":128,"context_line":""},{"line_number":129,"context_line":"Testing"}],"source_content_type":"text/x-rst","patch_set":2,"id":"baada198_df4bef5c","line":126,"in_reply_to":"baada198_42568acf","updated":"2014-08-11 13:14:07.000000000","message":"I\u0027ve fixed.","commit_id":"2e981209fa6c9d8a2c3c100bfbaf48ab432b4efe"},{"author":{"_account_id":3012,"name":"Dina Belova","email":"dbelova@mirantis.com","username":"dbelova"},"change_message_id":"7a16753fc5d50ae41b937b55f6a25373d3226d29","unresolved":false,"context_lines":[{"line_number":140,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":141,"context_line":""},{"line_number":142,"context_line":"None"},{"line_number":143,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"baada198_11c850a7","line":143,"updated":"2014-07-31 13:33:15.000000000","message":"no estimation date?","commit_id":"2e981209fa6c9d8a2c3c100bfbaf48ab432b4efe"},{"author":{"_account_id":8988,"name":"Rikimaru Honjo","email":"honjo.rikimaru@ntt-tx.co.jp","username":"rikimaru.honjo","inactive":true},"change_message_id":"261738aef86952b4779456967195f75e04b74be9","unresolved":false,"context_lines":[{"line_number":140,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":141,"context_line":""},{"line_number":142,"context_line":"None"},{"line_number":143,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"baada198_1f1ba76f","line":143,"in_reply_to":"baada198_11c850a7","updated":"2014-08-11 13:14:07.000000000","message":"Thank you. I wrote references.\n\nI\u0027d like to realize this at juno-3.","commit_id":"2e981209fa6c9d8a2c3c100bfbaf48ab432b4efe"},{"author":{"_account_id":7052,"name":"Fabio Giannetti","email":"fgiannet@cisco.com","username":"fabgia"},"change_message_id":"f68b99728d65146e6cfcc5b569ad81e626107576","unresolved":false,"context_lines":[{"line_number":106,"context_line":"Work Items"},{"line_number":107,"context_line":"----------"},{"line_number":108,"context_line":""},{"line_number":109,"context_line":"* \"alarm_history_time_to_live\" is added as a config option."},{"line_number":110,"context_line":"* \"time_to_live\" is renamed to sample_time_to_live."},{"line_number":111,"context_line":"* Deprecated \"time_to_live\"."},{"line_number":112,"context_line":"* Code the implementation of deleting alarm history in"}],"source_content_type":"text/x-rst","patch_set":4,"id":"5a890539_6db215e7","line":109,"updated":"2014-12-05 03:43:45.000000000","message":"Actually I think a better option is to have this added in the [alarm] section of the config file:\n\n[alarm]\nrecord_history\u003dtrue\ntime_to_live_history\u003dNNs\n\n[database]\ntime_to_leave \u003d MMs\n\nThis will achieve your goal keeping 100% backward compatibility.","commit_id":"d4205e19404f67956c1aff5dbc5793207e1bd8ac"},{"author":{"_account_id":6676,"name":"zqfan","email":"aji.zqfan@gmail.com","username":"aji-zqfan"},"change_message_id":"59283cd71849786dc0da054b9e77babc70ce5a80","unresolved":false,"context_lines":[{"line_number":106,"context_line":"Work Items"},{"line_number":107,"context_line":"----------"},{"line_number":108,"context_line":""},{"line_number":109,"context_line":"* \"alarm_history_time_to_live\" is added as a config option."},{"line_number":110,"context_line":"* \"time_to_live\" is renamed to sample_time_to_live."},{"line_number":111,"context_line":"* Deprecated \"time_to_live\"."},{"line_number":112,"context_line":"* Code the implementation of deleting alarm history in"}],"source_content_type":"text/x-rst","patch_set":4,"id":"3a961159_c4a0fb44","line":109,"in_reply_to":"5a890539_6db215e7","updated":"2015-01-15 01:23:02.000000000","message":"config option can use deprecated tag for backward compati issue, see the code in my patch of event-ttl https://review.openstack.org/146367\n\nwe use metering_connection, alarm_connection, event_connection in database section, and time_to_live is in database section too, so I think maybe the event_time_to_live, alarm_history_time_to_live can be there too\n\nrecord_history is in alarm section, but record is generated at api module, time_to_live is a database feature, so it either can be in alarm section or database, I think database may be better, configure a database connection then set its time_to_live sounds continuous","commit_id":"d4205e19404f67956c1aff5dbc5793207e1bd8ac"},{"author":{"_account_id":7052,"name":"Fabio Giannetti","email":"fgiannet@cisco.com","username":"fabgia"},"change_message_id":"57cc69045f88a4d1b50fdcc6a8ec1506a0b55ad4","unresolved":false,"context_lines":[{"line_number":106,"context_line":"Work Items"},{"line_number":107,"context_line":"----------"},{"line_number":108,"context_line":""},{"line_number":109,"context_line":"* \"alarm_history_time_to_live\" is added as a config option."},{"line_number":110,"context_line":"* \"time_to_live\" is renamed to sample_time_to_live."},{"line_number":111,"context_line":"* Deprecated \"time_to_live\"."},{"line_number":112,"context_line":"* Code the implementation of deleting alarm history in"}],"source_content_type":"text/x-rst","patch_set":4,"id":"5a890539_2d62cd3d","line":109,"in_reply_to":"5a890539_6db215e7","updated":"2014-12-05 03:45:15.000000000","message":"sorry the comment was badly indented:\n\n[alarm]\n\nrecord_history\u003dtrue\n\ntime_to_live_history\u003dNNs\n\n[database]\n\ntime_to_live \u003d MMs","commit_id":"d4205e19404f67956c1aff5dbc5793207e1bd8ac"},{"author":{"_account_id":7052,"name":"Fabio Giannetti","email":"fgiannet@cisco.com","username":"fabgia"},"change_message_id":"f68b99728d65146e6cfcc5b569ad81e626107576","unresolved":false,"context_lines":[{"line_number":108,"context_line":""},{"line_number":109,"context_line":"* \"alarm_history_time_to_live\" is added as a config option."},{"line_number":110,"context_line":"* \"time_to_live\" is renamed to sample_time_to_live."},{"line_number":111,"context_line":"* Deprecated \"time_to_live\"."},{"line_number":112,"context_line":"* Code the implementation of deleting alarm history in"},{"line_number":113,"context_line":"  ceilometer/alarm/storage/impl_sqlalchemy, impl_mondodb."},{"line_number":114,"context_line":"* Code the implementation of deleting alarm history in"}],"source_content_type":"text/x-rst","patch_set":4,"id":"5a890539_cdc0692e","line":111,"updated":"2014-12-05 03:43:45.000000000","message":"Please see comment above. No need to deprecate.\nAnd in the future you can add a similar thing to the [event] section to handle events deletion.","commit_id":"d4205e19404f67956c1aff5dbc5793207e1bd8ac"},{"author":{"_account_id":6676,"name":"zqfan","email":"aji.zqfan@gmail.com","username":"aji-zqfan"},"change_message_id":"59283cd71849786dc0da054b9e77babc70ce5a80","unresolved":false,"context_lines":[{"line_number":108,"context_line":""},{"line_number":109,"context_line":"* \"alarm_history_time_to_live\" is added as a config option."},{"line_number":110,"context_line":"* \"time_to_live\" is renamed to sample_time_to_live."},{"line_number":111,"context_line":"* Deprecated \"time_to_live\"."},{"line_number":112,"context_line":"* Code the implementation of deleting alarm history in"},{"line_number":113,"context_line":"  ceilometer/alarm/storage/impl_sqlalchemy, impl_mondodb."},{"line_number":114,"context_line":"* Code the implementation of deleting alarm history in"}],"source_content_type":"text/x-rst","patch_set":4,"id":"3a961159_55cd492f","line":111,"in_reply_to":"5a890539_cdc0692e","updated":"2015-01-15 01:23:02.000000000","message":"see above response","commit_id":"d4205e19404f67956c1aff5dbc5793207e1bd8ac"}]}