)]}'
{"specs/newton/approved/xenapi-independent-nova.rst":[{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"5a02a28773899e4e008bf65928a17deebe3a9d4d","unresolved":false,"context_lines":[{"line_number":114,"context_line":"|                              | import as a disk                          |"},{"line_number":115,"context_line":"+------------------------------+-------------------------------------------+"},{"line_number":116,"context_line":"| vm_utils._fetch_disk_image   | Replace with the download approach in     |"},{"line_number":117,"context_line":"|                              | spec xenapi-image-streaming               |"},{"line_number":118,"context_line":"+------------------------------+-------------------------------------------+"},{"line_number":119,"context_line":"| vm_utils.preconfigure        | CONF.flat_injected must be disabled if    |"},{"line_number":120,"context_line":"|          instance            | independent_compute[3]                    |"}],"source_content_type":"text/x-rst","patch_set":2,"id":"dab17558_e984f4d3","line":117,"updated":"2016-05-16 15:53:11.000000000","message":"isn\u0027t this independent of the change here? or is this ram disk and kernel stuff?","commit_id":"b71a68891a8b504dcb91c2bbc591ce4062d05ea2"},{"author":{"_account_id":6735,"name":"Bob Ball","email":"bob.ball@citrix.com","username":"bob-ball"},"change_message_id":"e33f801aa1a588e2b52114e622819673aa4acd66","unresolved":false,"context_lines":[{"line_number":114,"context_line":"|                              | import as a disk                          |"},{"line_number":115,"context_line":"+------------------------------+-------------------------------------------+"},{"line_number":116,"context_line":"| vm_utils._fetch_disk_image   | Replace with the download approach in     |"},{"line_number":117,"context_line":"|                              | spec xenapi-image-streaming               |"},{"line_number":118,"context_line":"+------------------------------+-------------------------------------------+"},{"line_number":119,"context_line":"| vm_utils.preconfigure        | CONF.flat_injected must be disabled if    |"},{"line_number":120,"context_line":"|          instance            | independent_compute[3]                    |"}],"source_content_type":"text/x-rst","patch_set":2,"id":"dab17558_bbb20992","line":117,"in_reply_to":"dab17558_e984f4d3","updated":"2016-05-17 11:01:26.000000000","message":"Looking at the callers yes, it seems only used in the ramdisk/kernel code so perhaps we can just fail if we\u0027re using this independent_compute mode rather than depend on xenapi-image-streaming.","commit_id":"b71a68891a8b504dcb91c2bbc591ce4062d05ea2"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"5a02a28773899e4e008bf65928a17deebe3a9d4d","unresolved":false,"context_lines":[{"line_number":216,"context_line":"The primary deployment use case is not being changed, and will"},{"line_number":217,"context_line":"continue to be tested with the Citrix XenServer CI.  It is anticipated"},{"line_number":218,"context_line":"that developers working on XenAPI will use the independent compute"},{"line_number":219,"context_line":"approach to simplify the development environment."},{"line_number":220,"context_line":""},{"line_number":221,"context_line":"Documentation Impact"},{"line_number":222,"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":"dab17558_c904907f","line":219,"updated":"2016-05-16 15:53:11.000000000","message":"Hmm, we really need to look into the test implications of this direction.\n\nGiven the separate download path, it does feel like we need some way to CI this approach.\n\nCan we talk about what the differences would be in the gate-tested code path between independent and non-independent compute?","commit_id":"b71a68891a8b504dcb91c2bbc591ce4062d05ea2"},{"author":{"_account_id":6735,"name":"Bob Ball","email":"bob.ball@citrix.com","username":"bob-ball"},"change_message_id":"e33f801aa1a588e2b52114e622819673aa4acd66","unresolved":false,"context_lines":[{"line_number":216,"context_line":"The primary deployment use case is not being changed, and will"},{"line_number":217,"context_line":"continue to be tested with the Citrix XenServer CI.  It is anticipated"},{"line_number":218,"context_line":"that developers working on XenAPI will use the independent compute"},{"line_number":219,"context_line":"approach to simplify the development environment."},{"line_number":220,"context_line":""},{"line_number":221,"context_line":"Documentation Impact"},{"line_number":222,"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":"dab17558_d6440448","line":219,"in_reply_to":"dab17558_c904907f","updated":"2016-05-17 11:01:26.000000000","message":"It\u0027s assumed that deployment scenarios will continue (in the short term) to use a integrated compute VM.  \nWhat do you mean by a separate download path?  Are you meaning for Glance, where we will continue to use the plugin?\n\nThe main change that would impact both use cases is the way we generate the config drive.  The proposal here is not to detect whether we are using an independent compute and just to assume that the config drive needs to be streamed to a VDI.\n\nI think the main runtime differences between the independent and integrated compute will just be the checking for unsupported options","commit_id":"b71a68891a8b504dcb91c2bbc591ce4062d05ea2"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"1605bb22e8159327655fbafa8c5af70f5ef7e84a","unresolved":false,"context_lines":[{"line_number":209,"context_line":"----------"},{"line_number":210,"context_line":""},{"line_number":211,"context_line":"* \u0027partition_utils\u0027 plugin: Add plugin to move disk-focused"},{"line_number":212,"context_line":"  functionality from DomU to Dom0"},{"line_number":213,"context_line":""},{"line_number":214,"context_line":"* Import config drive: Build config drive in DomU as a RAW disk then"},{"line_number":215,"context_line":"  use API to import the raw into a VDI"}],"source_content_type":"text/x-rst","patch_set":3,"id":"bab6814e_901dd768","line":212,"updated":"2016-05-25 09:56:33.000000000","message":"We should probably have some protection here, we need to be careful about using too many Dom0 resources and impacting how other guests are running. Using nice might help reduce the concerns.","commit_id":"f6bb773ac412f85e78c413208d78cb8db2e3332d"},{"author":{"_account_id":5441,"name":"Andrew Laski","email":"andrew@lascii.com","username":"alaski"},"change_message_id":"321bd43796f42361546257f7e9da5471dd1014f8","unresolved":false,"context_lines":[{"line_number":35,"context_line":"* Developer wishes to test the XenAPI driver in their existing Linux"},{"line_number":36,"context_line":"  environment, by running the nova compute services using DevStack but"},{"line_number":37,"context_line":"  connecting to a XenServer hypervisor running as a virtual machine"},{"line_number":38,"context_line":"  inside VirtualBox to deploy the tenant VMs."},{"line_number":39,"context_line":""},{"line_number":40,"context_line":"* Deployer wishes to consolidate the Nova compute services to enable"},{"line_number":41,"context_line":"  specialised Nova compute environments with individual VMs running"}],"source_content_type":"text/x-rst","patch_set":4,"id":"bab6814e_7d66f5ed","line":38,"updated":"2016-05-26 16:55:56.000000000","message":"This would be really nice.","commit_id":"9d2196b29b86a7c82f07b8328c5ac2c4e7d8b6d9"},{"author":{"_account_id":5441,"name":"Andrew Laski","email":"andrew@lascii.com","username":"alaski"},"change_message_id":"321bd43796f42361546257f7e9da5471dd1014f8","unresolved":false,"context_lines":[{"line_number":40,"context_line":"* Deployer wishes to consolidate the Nova compute services to enable"},{"line_number":41,"context_line":"  specialised Nova compute environments with individual VMs running"},{"line_number":42,"context_line":"  the services independently of the hypervisor (1:1 relationship"},{"line_number":43,"context_line":"  between nova compute process and hypervisor)"},{"line_number":44,"context_line":""},{"line_number":45,"context_line":"* Deployer wishes to consolidate the Nova compute services into docker"},{"line_number":46,"context_line":"  containers running on a physical host, independently of the"}],"source_content_type":"text/x-rst","patch_set":4,"id":"bab6814e_5df051fd","line":43,"updated":"2016-05-26 16:55:56.000000000","message":"Thanks for including this note. Changing this relationship is a big concern until we have better ways to handle it. As we\u0027ve seen with Ironic there are unresolved issues with doing it.","commit_id":"9d2196b29b86a7c82f07b8328c5ac2c4e7d8b6d9"},{"author":{"_account_id":5441,"name":"Andrew Laski","email":"andrew@lascii.com","username":"alaski"},"change_message_id":"321bd43796f42361546257f7e9da5471dd1014f8","unresolved":false,"context_lines":[{"line_number":61,"context_line":"nova independently of the XenServer host, therefore if"},{"line_number":62,"context_line":"get_this_vm_uuid detects that we are not able to identify a VM we are"},{"line_number":63,"context_line":"running on (so are running indepdenently) it will check for"},{"line_number":64,"context_line":"configuration options that cannot be supported and fail at start of"},{"line_number":65,"context_line":"day if any are present.  To simplify the changes, a variable"},{"line_number":66,"context_line":"\"independent_compute\" will be added to XenAPI\u0027s driver and set at"},{"line_number":67,"context_line":"start of day."},{"line_number":68,"context_line":""}],"source_content_type":"text/x-rst","patch_set":4,"id":"bab6814e_9d73d97f","line":65,"range":{"start_line":64,"start_character":59,"end_line":65,"end_character":3},"updated":"2016-05-26 16:55:56.000000000","message":"\"start of day\" must be an idiom I am not familiar with. I\u0027m assuming this means at startup time?","commit_id":"9d2196b29b86a7c82f07b8328c5ac2c4e7d8b6d9"},{"author":{"_account_id":5441,"name":"Andrew Laski","email":"andrew@lascii.com","username":"alaski"},"change_message_id":"49e67346cd4c673c549f03b521deab70ca25b9ca","unresolved":false,"context_lines":[{"line_number":62,"context_line":"get_this_vm_uuid detects that we are not able to identify a VM we are"},{"line_number":63,"context_line":"running on (so are running indepdenently) it will check for"},{"line_number":64,"context_line":"configuration options that cannot be supported and fail at start of"},{"line_number":65,"context_line":"day if any are present.  To simplify the changes, a variable"},{"line_number":66,"context_line":"\"independent_compute\" will be added to XenAPI\u0027s driver and set at"},{"line_number":67,"context_line":"start of day."},{"line_number":68,"context_line":""}],"source_content_type":"text/x-rst","patch_set":4,"id":"9abb7d3a_b09f845e","line":65,"in_reply_to":"bab6814e_5865f542","updated":"2016-05-27 13:35:32.000000000","message":"Heh, it was fine to leave it. It was just a new one for me :)","commit_id":"9d2196b29b86a7c82f07b8328c5ac2c4e7d8b6d9"},{"author":{"_account_id":6735,"name":"Bob Ball","email":"bob.ball@citrix.com","username":"bob-ball"},"change_message_id":"c6d05c1b8ad8c10e8cc399c2d61e6d7e006b01a2","unresolved":false,"context_lines":[{"line_number":62,"context_line":"get_this_vm_uuid detects that we are not able to identify a VM we are"},{"line_number":63,"context_line":"running on (so are running indepdenently) it will check for"},{"line_number":64,"context_line":"configuration options that cannot be supported and fail at start of"},{"line_number":65,"context_line":"day if any are present.  To simplify the changes, a variable"},{"line_number":66,"context_line":"\"independent_compute\" will be added to XenAPI\u0027s driver and set at"},{"line_number":67,"context_line":"start of day."},{"line_number":68,"context_line":""}],"source_content_type":"text/x-rst","patch_set":4,"id":"bab6814e_5865f542","line":65,"in_reply_to":"bab6814e_9d73d97f","updated":"2016-05-26 20:56:15.000000000","message":"Yes, sorry, it does.  I mean when the Nova services are started and makes a connection to the hypervisor.","commit_id":"9d2196b29b86a7c82f07b8328c5ac2c4e7d8b6d9"},{"author":{"_account_id":5441,"name":"Andrew Laski","email":"andrew@lascii.com","username":"alaski"},"change_message_id":"321bd43796f42361546257f7e9da5471dd1014f8","unresolved":false,"context_lines":[{"line_number":73,"context_line":"+------------------------------+-------------------------------------------+"},{"line_number":74,"context_line":"|Function name                 | Proposed resolution                       |"},{"line_number":75,"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\u003d\u003d\u003d\u003d+"},{"line_number":76,"context_line":"| vm_utils.get_this_vm_uuid    | Fix callers:vm_utils.get_this_vm_ref      |"},{"line_number":77,"context_line":"|                              | pool._create_slave_info                   |"},{"line_number":78,"context_line":"|                              | vm_utils.ensure_correct_host              |"},{"line_number":79,"context_line":"+------------------------------+-------------------------------------------+"}],"source_content_type":"text/x-rst","patch_set":4,"id":"bab6814e_1d7a49f1","line":76,"range":{"start_line":76,"start_character":33,"end_line":76,"end_character":44},"updated":"2016-05-26 16:55:56.000000000","message":"What does this mean exactly?","commit_id":"9d2196b29b86a7c82f07b8328c5ac2c4e7d8b6d9"},{"author":{"_account_id":6735,"name":"Bob Ball","email":"bob.ball@citrix.com","username":"bob-ball"},"change_message_id":"c6d05c1b8ad8c10e8cc399c2d61e6d7e006b01a2","unresolved":false,"context_lines":[{"line_number":73,"context_line":"+------------------------------+-------------------------------------------+"},{"line_number":74,"context_line":"|Function name                 | Proposed resolution                       |"},{"line_number":75,"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\u003d\u003d\u003d\u003d+"},{"line_number":76,"context_line":"| vm_utils.get_this_vm_uuid    | Fix callers:vm_utils.get_this_vm_ref      |"},{"line_number":77,"context_line":"|                              | pool._create_slave_info                   |"},{"line_number":78,"context_line":"|                              | vm_utils.ensure_correct_host              |"},{"line_number":79,"context_line":"+------------------------------+-------------------------------------------+"}],"source_content_type":"text/x-rst","patch_set":4,"id":"bab6814e_98e91d82","line":76,"in_reply_to":"bab6814e_1d7a49f1","updated":"2016-05-26 20:56:15.000000000","message":"I was trying to show the chain of calls that would need to be fixed.  On the left we have the offending function, and the right is how I intend to fix it.\nIn this case, each of the callers of get_this_vm_uuid are represented below, with their own resolution, hence get_this_vm_uuid will be left unchanged.","commit_id":"9d2196b29b86a7c82f07b8328c5ac2c4e7d8b6d9"},{"author":{"_account_id":5441,"name":"Andrew Laski","email":"andrew@lascii.com","username":"alaski"},"change_message_id":"49e67346cd4c673c549f03b521deab70ca25b9ca","unresolved":false,"context_lines":[{"line_number":73,"context_line":"+------------------------------+-------------------------------------------+"},{"line_number":74,"context_line":"|Function name                 | Proposed resolution                       |"},{"line_number":75,"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\u003d\u003d\u003d\u003d+"},{"line_number":76,"context_line":"| vm_utils.get_this_vm_uuid    | Fix callers:vm_utils.get_this_vm_ref      |"},{"line_number":77,"context_line":"|                              | pool._create_slave_info                   |"},{"line_number":78,"context_line":"|                              | vm_utils.ensure_correct_host              |"},{"line_number":79,"context_line":"+------------------------------+-------------------------------------------+"}],"source_content_type":"text/x-rst","patch_set":4,"id":"9abb7d3a_f0874c10","line":76,"in_reply_to":"bab6814e_98e91d82","updated":"2016-05-27 13:35:32.000000000","message":"Ahh, I completely missed the connection that these were referenced below.","commit_id":"9d2196b29b86a7c82f07b8328c5ac2c4e7d8b6d9"},{"author":{"_account_id":5441,"name":"Andrew Laski","email":"andrew@lascii.com","username":"alaski"},"change_message_id":"321bd43796f42361546257f7e9da5471dd1014f8","unresolved":false,"context_lines":[{"line_number":128,"context_line":"|          instance            | independent_compute[4]                    |"},{"line_number":129,"context_line":"+------------------------------+-------------------------------------------+"},{"line_number":130,"context_line":""},{"line_number":131,"context_line":"Alternatives"},{"line_number":132,"context_line":"------------"},{"line_number":133,"context_line":""},{"line_number":134,"context_line":"[1] Pool host aggregates currently depend on restarting the VM running"}],"source_content_type":"text/x-rst","patch_set":4,"id":"bab6814e_9dd2b9fe","line":131,"updated":"2016-05-26 16:55:56.000000000","message":"These are not alternatives in this section but footnotes. The alternatives section should go below.","commit_id":"9d2196b29b86a7c82f07b8328c5ac2c4e7d8b6d9"},{"author":{"_account_id":6735,"name":"Bob Ball","email":"bob.ball@citrix.com","username":"bob-ball"},"change_message_id":"c6d05c1b8ad8c10e8cc399c2d61e6d7e006b01a2","unresolved":false,"context_lines":[{"line_number":128,"context_line":"|          instance            | independent_compute[4]                    |"},{"line_number":129,"context_line":"+------------------------------+-------------------------------------------+"},{"line_number":130,"context_line":""},{"line_number":131,"context_line":"Alternatives"},{"line_number":132,"context_line":"------------"},{"line_number":133,"context_line":""},{"line_number":134,"context_line":"[1] Pool host aggregates currently depend on restarting the VM running"}],"source_content_type":"text/x-rst","patch_set":4,"id":"bab6814e_589a1515","line":131,"in_reply_to":"bab6814e_9dd2b9fe","updated":"2016-05-26 20:56:15.000000000","message":"Done","commit_id":"9d2196b29b86a7c82f07b8328c5ac2c4e7d8b6d9"},{"author":{"_account_id":5441,"name":"Andrew Laski","email":"andrew@lascii.com","username":"alaski"},"change_message_id":"321bd43796f42361546257f7e9da5471dd1014f8","unresolved":false,"context_lines":[{"line_number":226,"context_line":"Dependencies"},{"line_number":227,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":228,"context_line":""},{"line_number":229,"context_line":"* Depends on xenapi-image-streaming to download from glance into a VDI"},{"line_number":230,"context_line":"  without attaching the VDI to DomU"},{"line_number":231,"context_line":""},{"line_number":232,"context_line":"Testing"}],"source_content_type":"text/x-rst","patch_set":4,"id":"bab6814e_1d96c9ef","line":229,"range":{"start_line":229,"start_character":13,"end_line":229,"end_character":35},"updated":"2016-05-26 16:55:56.000000000","message":"Can you include a link to this in references. I don\u0027t see that spec merged, is it going to make it this cycle?","commit_id":"9d2196b29b86a7c82f07b8328c5ac2c4e7d8b6d9"},{"author":{"_account_id":6735,"name":"Bob Ball","email":"bob.ball@citrix.com","username":"bob-ball"},"change_message_id":"c6d05c1b8ad8c10e8cc399c2d61e6d7e006b01a2","unresolved":false,"context_lines":[{"line_number":226,"context_line":"Dependencies"},{"line_number":227,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":228,"context_line":""},{"line_number":229,"context_line":"* Depends on xenapi-image-streaming to download from glance into a VDI"},{"line_number":230,"context_line":"  without attaching the VDI to DomU"},{"line_number":231,"context_line":""},{"line_number":232,"context_line":"Testing"}],"source_content_type":"text/x-rst","patch_set":4,"id":"bab6814e_58c175e3","line":229,"in_reply_to":"bab6814e_1d96c9ef","updated":"2016-05-26 20:56:15.000000000","message":"This was a strict dependency in an earlier version of this spec, but was removed in the latest change.  We removed the dependency by dropping support for 3-part images in the isolated compute case.","commit_id":"9d2196b29b86a7c82f07b8328c5ac2c4e7d8b6d9"}]}