)]}'
{"specs/kilo/approved/pass-flavor-capabilities-to-ironic-virt-driver.rst":[{"author":{"_account_id":12898,"name":"Tony Breeds","email":"tony@bakeyournoodle.com","username":"tonyb"},"change_message_id":"4656d3ed8a1606bfdb8e094e214b391192d07154","unresolved":false,"context_lines":[{"line_number":43,"context_line":"node in desired power state."},{"line_number":44,"context_line":"This is applicable for all the hardware capabilities which requires"},{"line_number":45,"context_line":"some action from the driver as per the requested capability in the"},{"line_number":46,"context_line":"flavor extra-spec key."},{"line_number":47,"context_line":""},{"line_number":48,"context_line":"Project Priority"},{"line_number":49,"context_line":"-----------------"}],"source_content_type":"text/x-rst","patch_set":1,"id":"5a890539_62f5e9c9","line":46,"updated":"2014-12-04 06:12:01.000000000","message":"The flavor ID is in the instance info, why can\u0027t ironic reference that and look it up directly?","commit_id":"7e927472527b68f5a3ff18eed12d3859427cea61"},{"author":{"_account_id":2889,"name":"Aeva Black","email":"aeva.online@gmail.com","username":"tenbrae"},"change_message_id":"d20f2a3735004581c83efabce0f7c42b7f891604","unresolved":false,"context_lines":[{"line_number":43,"context_line":"node in desired power state."},{"line_number":44,"context_line":"This is applicable for all the hardware capabilities which requires"},{"line_number":45,"context_line":"some action from the driver as per the requested capability in the"},{"line_number":46,"context_line":"flavor extra-spec key."},{"line_number":47,"context_line":""},{"line_number":48,"context_line":"Project Priority"},{"line_number":49,"context_line":"-----------------"}],"source_content_type":"text/x-rst","patch_set":1,"id":"5a890539_8600dcac","line":46,"in_reply_to":"5a890539_62f5e9c9","updated":"2014-12-05 23:30:36.000000000","message":"Since Ironic has the Nova instance_uuid, it would be possible for Ironic to request this information from Nova, assuming Ironic has sufficient privileges to do so. That might require additional keystone round trip to generate such a token, or we might be able to re-use the token that Nova passed to Ironic in the first place.\n\nThat said, Nova is _already_ passing a fair bit of metadata to Ironic in the node.instance_info field. I think that taking a small amount of metadata, which Nova already has at this point in spawn(), and adding it to the existing Ironic request to save instance_info is a smaller and more efficient change.","commit_id":"7e927472527b68f5a3ff18eed12d3859427cea61"},{"author":{"_account_id":12898,"name":"Tony Breeds","email":"tony@bakeyournoodle.com","username":"tonyb"},"change_message_id":"3b172593902eb23ea2f8bb32112f7ff65e2ff4d0","unresolved":false,"context_lines":[{"line_number":43,"context_line":"node in desired power state."},{"line_number":44,"context_line":"This is applicable for all the hardware capabilities which requires"},{"line_number":45,"context_line":"some action from the driver as per the requested capability in the"},{"line_number":46,"context_line":"flavor extra-spec key."},{"line_number":47,"context_line":""},{"line_number":48,"context_line":"Project Priority"},{"line_number":49,"context_line":"-----------------"}],"source_content_type":"text/x-rst","patch_set":1,"id":"5a890539_d0b37998","line":46,"in_reply_to":"5a890539_8600dcac","updated":"2014-12-06 04:19:11.000000000","message":"Ok.  I think that should be captured in the spec as a valid but sub-optimal alternative.","commit_id":"7e927472527b68f5a3ff18eed12d3859427cea61"},{"author":{"_account_id":2271,"name":"Michael Still","email":"mikal@stillhq.com","username":"mikalstill"},"change_message_id":"9eead0a818b20aa0e83230c6efbff3f7b48fcb2d","unresolved":false,"context_lines":[{"line_number":43,"context_line":"node in desired power state."},{"line_number":44,"context_line":"This is applicable for all the hardware capabilities which requires"},{"line_number":45,"context_line":"some action from the driver as per the requested capability in the"},{"line_number":46,"context_line":"flavor extra-spec key."},{"line_number":47,"context_line":""},{"line_number":48,"context_line":"Project Priority"},{"line_number":49,"context_line":"-----------------"}],"source_content_type":"text/x-rst","patch_set":1,"id":"3a961159_ecfb6045","line":46,"in_reply_to":"5a890539_d0b37998","updated":"2014-12-07 23:18:52.000000000","message":"Done","commit_id":"7e927472527b68f5a3ff18eed12d3859427cea61"},{"author":{"_account_id":2271,"name":"Michael Still","email":"mikal@stillhq.com","username":"mikalstill"},"change_message_id":"24b6309b313bb94d9a0818496f69f134d23e6cce","unresolved":false,"context_lines":[{"line_number":49,"context_line":"-----------------"},{"line_number":50,"context_line":""},{"line_number":51,"context_line":"The kilo priorities list is currently undefined, But this partly falls under"},{"line_number":52,"context_line":"the cross project requirement."},{"line_number":53,"context_line":""},{"line_number":54,"context_line":"Proposed change"},{"line_number":55,"context_line":"\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":"5a890539_e2f2b96d","line":52,"updated":"2014-12-04 06:54:51.000000000","message":"This is no longer true, we have defined them. The correct answer here is \"None\" now.","commit_id":"7e927472527b68f5a3ff18eed12d3859427cea61"},{"author":{"_account_id":2271,"name":"Michael Still","email":"mikal@stillhq.com","username":"mikalstill"},"change_message_id":"9eead0a818b20aa0e83230c6efbff3f7b48fcb2d","unresolved":false,"context_lines":[{"line_number":49,"context_line":"-----------------"},{"line_number":50,"context_line":""},{"line_number":51,"context_line":"The kilo priorities list is currently undefined, But this partly falls under"},{"line_number":52,"context_line":"the cross project requirement."},{"line_number":53,"context_line":""},{"line_number":54,"context_line":"Proposed change"},{"line_number":55,"context_line":"\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":"3a961159_2c9568e4","line":52,"in_reply_to":"5a890539_e2f2b96d","updated":"2014-12-07 23:18:52.000000000","message":"Done","commit_id":"7e927472527b68f5a3ff18eed12d3859427cea61"}]}