)]}'
{"specs/mitaka/approved/inject-nmi-ironic.rst":[{"author":{"_account_id":14131,"name":"shaohef","email":"shaohe.feng@intel.com","username":"shaohefeng"},"change_message_id":"4f0273f9db7f82456966a58cda0dc5a3ffbba132","unresolved":false,"context_lines":[{"line_number":20,"context_line":"example, in the mission critical area, it is necessary to trigger the kernel"},{"line_number":21,"context_line":"crash dump mechanism."},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"When blueprint https://blueprints.launchpad.net/nova/+spec/inject-nmi is"},{"line_number":24,"context_line":"implemented, there will be an API to inject NMI to instance in nova. But ironic"},{"line_number":25,"context_line":"driver does not have any interface corresponding to that API yet. So this spec"},{"line_number":26,"context_line":"aims at adding an API to ironic driver so that NMI can be injected to instances"}],"source_content_type":"text/x-rst","patch_set":1,"id":"bae84128_2c8c4ed9","line":23,"range":{"start_line":23,"start_character":15,"end_line":23,"end_character":70},"updated":"2016-02-23 08:50:49.000000000","message":"I did not find this BP.","commit_id":"5c1fafddda3bd50a45b1c8fa3620269dc88644c3"},{"author":{"_account_id":14937,"name":"Tang Chen","email":"imtangchen@163.com","username":"tangchen"},"change_message_id":"f139466b8bd0953014cfad310bea65e711da545c","unresolved":false,"context_lines":[{"line_number":20,"context_line":"example, in the mission critical area, it is necessary to trigger the kernel"},{"line_number":21,"context_line":"crash dump mechanism."},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"When blueprint https://blueprints.launchpad.net/nova/+spec/inject-nmi is"},{"line_number":24,"context_line":"implemented, there will be an API to inject NMI to instance in nova. But ironic"},{"line_number":25,"context_line":"driver does not have any interface corresponding to that API yet. So this spec"},{"line_number":26,"context_line":"aims at adding an API to ironic driver so that NMI can be injected to instances"}],"source_content_type":"text/x-rst","patch_set":1,"id":"bae84128_18442048","line":23,"range":{"start_line":23,"start_character":15,"end_line":23,"end_character":70},"in_reply_to":"bae84128_2c8c4ed9","updated":"2016-02-23 09:56:12.000000000","message":"Hi Feng, I just restored it. Please wait for a spec for Newton. :)\n\nThanks.","commit_id":"5c1fafddda3bd50a45b1c8fa3620269dc88644c3"},{"author":{"_account_id":12898,"name":"Tony Breeds","email":"tony@bakeyournoodle.com","username":"tonyb"},"change_message_id":"58f336cdb6d0450ecb65512c2f8f097ce8c2957f","unresolved":false,"context_lines":[{"line_number":24,"context_line":"implemented, there will be an API to inject NMI to instance in nova. But ironic"},{"line_number":25,"context_line":"driver does not have any interface corresponding to that API yet. So this spec"},{"line_number":26,"context_line":"aims at adding an API to ironic driver so that NMI can be injected to instances"},{"line_number":27,"context_line":"managed by ironic."},{"line_number":28,"context_line":""},{"line_number":29,"context_line":"Use Cases"},{"line_number":30,"context_line":"----------"}],"source_content_type":"text/x-rst","patch_set":1,"id":"3a29b11f_34211efc","line":27,"updated":"2015-10-20 05:14:40.000000000","message":"The Ironic spec: https://review.openstack.org/#/c/186700 isn\u0027t approved yet.","commit_id":"5c1fafddda3bd50a45b1c8fa3620269dc88644c3"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"a5f00cbd35a86fbda8bd4fcc45a313e5a740f32b","unresolved":false,"context_lines":[{"line_number":24,"context_line":"implemented, there will be an API to inject NMI to instance in nova. But ironic"},{"line_number":25,"context_line":"driver does not have any interface corresponding to that API yet. So this spec"},{"line_number":26,"context_line":"aims at adding an API to ironic driver so that NMI can be injected to instances"},{"line_number":27,"context_line":"managed by ironic."},{"line_number":28,"context_line":""},{"line_number":29,"context_line":"Use Cases"},{"line_number":30,"context_line":"----------"}],"source_content_type":"text/x-rst","patch_set":1,"id":"9a8ffd7b_55e75c12","line":27,"in_reply_to":"3a29b11f_34211efc","updated":"2015-11-24 11:37:38.000000000","message":"we should add a depends-on: relationship with this spec.","commit_id":"5c1fafddda3bd50a45b1c8fa3620269dc88644c3"},{"author":{"_account_id":12898,"name":"Tony Breeds","email":"tony@bakeyournoodle.com","username":"tonyb"},"change_message_id":"58f336cdb6d0450ecb65512c2f8f097ce8c2957f","unresolved":false,"context_lines":[{"line_number":51,"context_line":""},{"line_number":52,"context_line":"REST API impact"},{"line_number":53,"context_line":"---------------"},{"line_number":54,"context_line":"None"},{"line_number":55,"context_line":""},{"line_number":56,"context_line":"Security impact"},{"line_number":57,"context_line":"---------------"}],"source_content_type":"text/x-rst","patch_set":1,"id":"3a29b11f_94691228","line":54,"updated":"2015-10-20 05:14:40.000000000","message":"Well you\u0027re going to need to expose it via REST.","commit_id":"5c1fafddda3bd50a45b1c8fa3620269dc88644c3"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"a5f00cbd35a86fbda8bd4fcc45a313e5a740f32b","unresolved":false,"context_lines":[{"line_number":51,"context_line":""},{"line_number":52,"context_line":"REST API impact"},{"line_number":53,"context_line":"---------------"},{"line_number":54,"context_line":"None"},{"line_number":55,"context_line":""},{"line_number":56,"context_line":"Security impact"},{"line_number":57,"context_line":"---------------"}],"source_content_type":"text/x-rst","patch_set":1,"id":"9a8ffd7b_15c8f473","line":54,"in_reply_to":"3a29b11f_94691228","updated":"2015-11-24 11:37:38.000000000","message":"I guess you mean use the existing API for this, you should really add a dependency on that spec? Can\u0027t find the link right now.","commit_id":"5c1fafddda3bd50a45b1c8fa3620269dc88644c3"},{"author":{"_account_id":12898,"name":"Tony Breeds","email":"tony@bakeyournoodle.com","username":"tonyb"},"change_message_id":"58f336cdb6d0450ecb65512c2f8f097ce8c2957f","unresolved":false,"context_lines":[{"line_number":75,"context_line":""},{"line_number":76,"context_line":"Developer impact"},{"line_number":77,"context_line":"----------------"},{"line_number":78,"context_line":"This spec will implement the new API in nova ironic driver."},{"line_number":79,"context_line":""},{"line_number":80,"context_line":"Implementation"},{"line_number":81,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":1,"id":"3a29b11f_344fdebe","line":78,"updated":"2015-10-20 05:14:40.000000000","message":"other hypervisors also have this ability so there will need to be collaboration on the design to ensure it works well.","commit_id":"5c1fafddda3bd50a45b1c8fa3620269dc88644c3"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"a5f00cbd35a86fbda8bd4fcc45a313e5a740f32b","unresolved":false,"context_lines":[{"line_number":75,"context_line":""},{"line_number":76,"context_line":"Developer impact"},{"line_number":77,"context_line":"----------------"},{"line_number":78,"context_line":"This spec will implement the new API in nova ironic driver."},{"line_number":79,"context_line":""},{"line_number":80,"context_line":"Implementation"},{"line_number":81,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":1,"id":"9a8ffd7b_55951c3e","line":78,"in_reply_to":"3a29b11f_344fdebe","updated":"2015-11-24 11:37:38.000000000","message":"its already proposed somewhere for other hypervisors.","commit_id":"5c1fafddda3bd50a45b1c8fa3620269dc88644c3"}],"specs/newton/approved/inject-nmi-ironic.rst":[{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"92392ecd2f15d5bb2c93a6ffaba3243210514eff","unresolved":false,"context_lines":[{"line_number":107,"context_line":""},{"line_number":108,"context_line":"Testing"},{"line_number":109,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":110,"context_line":"Unit tests will be added."},{"line_number":111,"context_line":""},{"line_number":112,"context_line":"Documentation Impact"},{"line_number":113,"context_line":"\u003d\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":2,"id":"9abb7d3a_6fd7f2bf","line":110,"updated":"2016-06-02 16:04:26.000000000","message":"I would actually expect a Tempest integration test for this, since it\u0027s going to be based on microversions in Ironic to use the code in the Ironic driver in Nova to see if this is even possible. And we have a CI job for Ironic that runs on Nova changes (it\u0027s non-voting today), but I\u0027d think a test could be built into that.","commit_id":"782dc0a111f3c2fbff58763fd95dc4453662c1fa"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"92392ecd2f15d5bb2c93a6ffaba3243210514eff","unresolved":false,"context_lines":[{"line_number":111,"context_line":""},{"line_number":112,"context_line":"Documentation Impact"},{"line_number":113,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":114,"context_line":"The hypervisor support matrix will be fixed with this implementation."},{"line_number":115,"context_line":""},{"line_number":116,"context_line":"References"},{"line_number":117,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":2,"id":"9abb7d3a_cffb7e46","line":114,"updated":"2016-06-02 16:04:26.000000000","message":"Sure, but it\u0027s totally conditional on the version of Ironic you have available in the deployment.","commit_id":"782dc0a111f3c2fbff58763fd95dc4453662c1fa"}]}
