)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"76c4441011e569d2b7e39c98bd92bcabf0f30365","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Kevin_Zheng \u003czhengzhenyu@huawei.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2015-11-09 11:06:12 +0800"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Add timestamp filter support for os-isntance-action API"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Currently, os-instance-actions API does not support filter the migration"},{"line_number":10,"context_line":"record by last updated time. As for production deployment, an instance"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"ba8a016a_8cc84a76","line":7,"updated":"2015-11-16 16:47:00.000000000","message":"s/isntance/instance/","commit_id":"cde3414cd108568aa9e8e8f2431a1fc34f1dc30f"},{"author":{"_account_id":2271,"name":"Michael Still","email":"mikal@stillhq.com","username":"mikalstill"},"change_message_id":"16d2afac93d15ae7b52c32b7974799aec6ff4183","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Zhenyu Zheng \u003czhengzhenyu@huawei.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2016-05-27 13:57:47 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Add pagination and changes since filter support for os-isntance-action API"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Currently, os-instance-actions  API does not support pagination."},{"line_number":10,"context_line":"As in production deployment the number of instance action records"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":10,"id":"9abb7d3a_9e131366","line":7,"range":{"start_line":7,"start_character":55,"end_line":7,"end_character":63},"updated":"2016-05-31 04:22:57.000000000","message":"instance","commit_id":"7ac28698c66bf93290294a95b9dfbdaa0438ab8d"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"41ee2fd3ac43648875811943ab7f3f62328ce179","unresolved":false,"context_lines":[{"line_number":18,"context_line":"last update time."},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"APIImpact"},{"line_number":21,"context_line":"DocImpact"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"Change-Id: I9ac4d73c120292fbe9e075d5da2549bdf6a6feaf"},{"line_number":24,"context_line":"Implements: blueprint pagination-add-changes-since-for-instance-action-list"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":10,"id":"9abb7d3a_22925c4c","line":21,"range":{"start_line":21,"start_character":0,"end_line":21,"end_character":9},"updated":"2016-06-01 18:47:15.000000000","message":"You don\u0027t need DocImpact on a spec.","commit_id":"7ac28698c66bf93290294a95b9dfbdaa0438ab8d"}],"specs/mitaka/approved/add-timestamp-support-for-instance-action-list.rst":[{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"76c4441011e569d2b7e39c98bd92bcabf0f30365","unresolved":false,"context_lines":[{"line_number":5,"context_line":" http://creativecommons.org/licenses/by/3.0/legalcode"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":8,"context_line":"Add pagination and changes-since filter support for os-migrations API"},{"line_number":9,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"https://blueprints.launchpad.net/nova/+spec/pagination-add-changes"}],"source_content_type":"text/x-rst","patch_set":3,"id":"ba8a016a_ec923657","line":8,"updated":"2015-11-16 16:47:00.000000000","message":"This is the wrong API - looks like you just copied this spec from another one of yours.","commit_id":"cde3414cd108568aa9e8e8f2431a1fc34f1dc30f"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"76c4441011e569d2b7e39c98bd92bcabf0f30365","unresolved":false,"context_lines":[{"line_number":35,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":36,"context_line":""},{"line_number":37,"context_line":"Add an API microversion that allows to get several migrations using"},{"line_number":38,"context_line":"filter with the help of `changes-since` optional parameter to"},{"line_number":39,"context_line":"GET /os-instance-actions  request."},{"line_number":40,"context_line":""},{"line_number":41,"context_line":"Alternatives"}],"source_content_type":"text/x-rst","patch_set":3,"id":"ba8a016a_2c87de10","line":38,"updated":"2015-11-16 16:47:00.000000000","message":"This doesn\u0027t talk about pagination at all.","commit_id":"cde3414cd108568aa9e8e8f2431a1fc34f1dc30f"}],"specs/newton/approved/pagination-and-changes-since-for-instance-action-list.rst":[{"author":{"_account_id":5441,"name":"Andrew Laski","email":"andrew@lascii.com","username":"alaski"},"change_message_id":"67fd1e0c7e92a7660980878ea15003f57b78e1b2","unresolved":false,"context_lines":[{"line_number":16,"context_line":""},{"line_number":17,"context_line":"This blueprint also add `changes-since` optional parameters to"},{"line_number":18,"context_line":"GET /os-instance-actions request to support filtering response data by"},{"line_number":19,"context_line":"updated time."},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"Problem description"},{"line_number":22,"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":5,"id":"7a5de9d1_6282a102","line":19,"updated":"2016-01-28 09:36:49.000000000","message":"limit and marker would be useful to have.  The other parameters do not seem at all useful and I think there needs to be a more compelling reason to add them.  What keys would you like to sort on other than updated_at?  The default sorting is currently updated_at so changes-since is not necessary because the list will already return the last changed instance-actions.\n\nUnless there is a very good reason to do otherwise this spec should be limited to just limit and marker.","commit_id":"64d129b54a1cb1738dd6db547eaeb5ba17c1b464"},{"author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"change_message_id":"3143d0b6811d508297fbcb5b52d54628cff4a44d","unresolved":false,"context_lines":[{"line_number":16,"context_line":""},{"line_number":17,"context_line":"This blueprint also add `changes-since` optional parameters to"},{"line_number":18,"context_line":"GET /os-instance-actions request to support filtering response data by"},{"line_number":19,"context_line":"updated time."},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"Problem description"},{"line_number":22,"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":5,"id":"fa0719c6_de4e9cc0","line":19,"in_reply_to":"7a5de9d1_5e6e2e02","updated":"2016-03-21 12:48:36.000000000","message":"I think changes-since is pretty useful because changes-since lets you build an event poll where you are getting everything changed since your last poll. \n\nI do agree that sort_key isn\u0027t useful for this table. And sort_dir is dubious (especially if changes-since is implemented)","commit_id":"64d129b54a1cb1738dd6db547eaeb5ba17c1b464"},{"author":{"_account_id":15888,"name":"Zhenyu Zheng","email":"zheng.zhenyu@outlook.com","username":"Kevin_Zheng"},"change_message_id":"0d2d723771c542628b495795997c5e80514897bf","unresolved":false,"context_lines":[{"line_number":16,"context_line":""},{"line_number":17,"context_line":"This blueprint also add `changes-since` optional parameters to"},{"line_number":18,"context_line":"GET /os-instance-actions request to support filtering response data by"},{"line_number":19,"context_line":"updated time."},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"Problem description"},{"line_number":22,"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":5,"id":"7a5de9d1_5e6e2e02","line":19,"in_reply_to":"7a5de9d1_6282a102","updated":"2016-01-29 03:02:29.000000000","message":"Hi, thanks for your comments. For a large and stable production deployment of OpenStack, Public Cloud and NFV for example, we can have large number of instances running for a very long time, over years, for example, during these instances\u0027 lifetime, numbers of instance actions can happen, for change-since feature, we can list only the instance-actions in the period of time that we are interested, and as for sort_key I agree with you, but for sort_dir, it will be also useful, we can both list it in dicrease order or increase order, it will be very good for user experiance.","commit_id":"64d129b54a1cb1738dd6db547eaeb5ba17c1b464"},{"author":{"_account_id":5441,"name":"Andrew Laski","email":"andrew@lascii.com","username":"alaski"},"change_message_id":"b509d21eff0b3eca71a945d5e2ffd1b40714fcd5","unresolved":false,"context_lines":[{"line_number":16,"context_line":""},{"line_number":17,"context_line":"This blueprint also add `changes-since` optional parameters to"},{"line_number":18,"context_line":"GET /os-instance-actions request to support filtering response data by"},{"line_number":19,"context_line":"updated time."},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"Problem description"},{"line_number":22,"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":5,"id":"fa0719c6_149593e3","line":19,"in_reply_to":"fa0719c6_de4e9cc0","updated":"2016-03-21 13:23:36.000000000","message":"I think changes-since is a quite poor implementation for a valid use case. I would rather not add it in any more places and instead focus on a way to expose notifications so polling is no longer necessary. Or pass that off to Searchlight to implement.","commit_id":"64d129b54a1cb1738dd6db547eaeb5ba17c1b464"},{"author":{"_account_id":5441,"name":"Andrew Laski","email":"andrew@lascii.com","username":"alaski"},"change_message_id":"d0e45cfd5e736791f76c15fb21b4ad0eeb0425fe","unresolved":false,"context_lines":[{"line_number":43,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":44,"context_line":""},{"line_number":45,"context_line":"Add an API microversion that allows to get several migrations using"},{"line_number":46,"context_line":"general pagination mechanism with the help of `limit`, `sort_key`,"},{"line_number":47,"context_line":"`sort_dir` optional parameters to GET /os-instance-actions  request."},{"line_number":48,"context_line":"And add filter with the help of `changes-since` optional parameter to"},{"line_number":49,"context_line":"GET /os-instance-actions  request."},{"line_number":50,"context_line":""}],"source_content_type":"text/x-rst","patch_set":9,"id":"9abb7d3a_7026bc17","line":47,"range":{"start_line":46,"start_character":55,"end_line":47,"end_character":10},"updated":"2016-05-27 13:39:38.000000000","message":"These should be removed.","commit_id":"1dcdc04494d89d848cb6a4cb6ba5f6ef0090c2f9"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"254e6aa26eb734779bbac5552bd977b347466fa3","unresolved":false,"context_lines":[{"line_number":236,"context_line":"Work Items"},{"line_number":237,"context_line":"----------"},{"line_number":238,"context_line":""},{"line_number":239,"context_line":"Create a new API microversion for getting several migrations using general"},{"line_number":240,"context_line":"pagination mechanism and time stamp filtering."},{"line_number":241,"context_line":""},{"line_number":242,"context_line":"Dependencies"}],"source_content_type":"text/x-rst","patch_set":10,"id":"9abb7d3a_028e382d","line":239,"range":{"start_line":239,"start_character":50,"end_line":239,"end_character":60},"updated":"2016-06-01 18:46:44.000000000","message":"The spec repeatedly talks about getting migrations, but this spec is only for adding limit/marker and sorting on changes-since by the os-instance-actions API, so I\u0027m not sure why migrations keeps getting mentioned in here. I\u0027m assuming it\u0027s an artifact of copying this initially from a similar spec to add limit/marker/sort support to the os-migrations API?","commit_id":"7ac28698c66bf93290294a95b9dfbdaa0438ab8d"},{"author":{"_account_id":15888,"name":"Zhenyu Zheng","email":"zheng.zhenyu@outlook.com","username":"Kevin_Zheng"},"change_message_id":"0f42e9c8b67ee278cc8235a13c9298a940abe6e1","unresolved":false,"context_lines":[{"line_number":236,"context_line":"Work Items"},{"line_number":237,"context_line":"----------"},{"line_number":238,"context_line":""},{"line_number":239,"context_line":"Create a new API microversion for getting several migrations using general"},{"line_number":240,"context_line":"pagination mechanism and time stamp filtering."},{"line_number":241,"context_line":""},{"line_number":242,"context_line":"Dependencies"}],"source_content_type":"text/x-rst","patch_set":10,"id":"9abb7d3a_0fd92584","line":239,"range":{"start_line":239,"start_character":50,"end_line":239,"end_character":60},"in_reply_to":"9abb7d3a_028e382d","updated":"2016-06-02 07:40:36.000000000","message":"Yes, should I correct this?","commit_id":"7ac28698c66bf93290294a95b9dfbdaa0438ab8d"}]}