)]}'
{"specs/mitaka/approved/vmware-opaque-network-support.rst":[{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"b73c4255235e81d9d550b27e75bd3b4fb5b58dea","unresolved":false,"context_lines":[{"line_number":41,"context_line":"Project Priority"},{"line_number":42,"context_line":"-----------------"},{"line_number":43,"context_line":""},{"line_number":44,"context_line":"Without this support the Neutron plugin cannot work."},{"line_number":45,"context_line":""},{"line_number":46,"context_line":"Proposed change"},{"line_number":47,"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":"7a2fa921_5373db15","line":44,"updated":"2015-10-12 09:09:02.000000000","message":"This section should be removed please, see latest template.","commit_id":"e5a71738fc69710dfe761375add3c98d84076905"},{"author":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"change_message_id":"d5958a0c179059b576578472f4a7364d58efe2b4","unresolved":false,"context_lines":[{"line_number":41,"context_line":"Project Priority"},{"line_number":42,"context_line":"-----------------"},{"line_number":43,"context_line":""},{"line_number":44,"context_line":"Without this support the Neutron plugin cannot work."},{"line_number":45,"context_line":""},{"line_number":46,"context_line":"Proposed change"},{"line_number":47,"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":"7a2fa921_353631b5","line":44,"in_reply_to":"7a2fa921_5373db15","updated":"2015-10-12 17:05:33.000000000","message":"Done","commit_id":"e5a71738fc69710dfe761375add3c98d84076905"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"b73c4255235e81d9d550b27e75bd3b4fb5b58dea","unresolved":false,"context_lines":[{"line_number":48,"context_line":""},{"line_number":49,"context_line":"The change is self contained within the VMware driver code and just related to"},{"line_number":50,"context_line":"how the ethernet device backing is configured. This is only when the Neutron"},{"line_number":51,"context_line":"virtual port is of the type \u0027ovs\u0027."},{"line_number":52,"context_line":""},{"line_number":53,"context_line":"Alternatives"},{"line_number":54,"context_line":"------------"}],"source_content_type":"text/x-rst","patch_set":1,"id":"7a2fa921_f3c5470a","line":51,"updated":"2015-10-12 09:09:02.000000000","message":"When Neutron has a virtual port type of ovs, it would be good if you could cover how the two cases work, the existing and the new case, and how the configuration plays into that.","commit_id":"e5a71738fc69710dfe761375add3c98d84076905"},{"author":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"change_message_id":"d5958a0c179059b576578472f4a7364d58efe2b4","unresolved":false,"context_lines":[{"line_number":48,"context_line":""},{"line_number":49,"context_line":"The change is self contained within the VMware driver code and just related to"},{"line_number":50,"context_line":"how the ethernet device backing is configured. This is only when the Neutron"},{"line_number":51,"context_line":"virtual port is of the type \u0027ovs\u0027."},{"line_number":52,"context_line":""},{"line_number":53,"context_line":"Alternatives"},{"line_number":54,"context_line":"------------"}],"source_content_type":"text/x-rst","patch_set":1,"id":"7a2fa921_1086a313","line":51,"in_reply_to":"7a2fa921_f3c5470a","updated":"2015-10-12 17:05:33.000000000","message":"Done","commit_id":"e5a71738fc69710dfe761375add3c98d84076905"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"b73c4255235e81d9d550b27e75bd3b4fb5b58dea","unresolved":false,"context_lines":[{"line_number":80,"context_line":""},{"line_number":81,"context_line":"The NSXv3 installation will be greenfield. So there is no upgrade impact."},{"line_number":82,"context_line":"For existing NSX-MH platforms the code is backwards compatible and those"},{"line_number":83,"context_line":"platforms will continue to work as they are today."},{"line_number":84,"context_line":""},{"line_number":85,"context_line":"Performance Impact"},{"line_number":86,"context_line":"------------------"}],"source_content_type":"text/x-rst","patch_set":1,"id":"7a2fa921_93cdf35f","line":83,"updated":"2015-10-12 09:09:02.000000000","message":"This is not really end user impact (talks about CLI changes, etc, usually), this seems like operator impact?\n\nNone is fine here.","commit_id":"e5a71738fc69710dfe761375add3c98d84076905"},{"author":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"change_message_id":"d5958a0c179059b576578472f4a7364d58efe2b4","unresolved":false,"context_lines":[{"line_number":80,"context_line":""},{"line_number":81,"context_line":"The NSXv3 installation will be greenfield. So there is no upgrade impact."},{"line_number":82,"context_line":"For existing NSX-MH platforms the code is backwards compatible and those"},{"line_number":83,"context_line":"platforms will continue to work as they are today."},{"line_number":84,"context_line":""},{"line_number":85,"context_line":"Performance Impact"},{"line_number":86,"context_line":"------------------"}],"source_content_type":"text/x-rst","patch_set":1,"id":"7a2fa921_5a2439ae","line":83,"in_reply_to":"7a2fa921_93cdf35f","updated":"2015-10-12 17:05:33.000000000","message":"Done","commit_id":"e5a71738fc69710dfe761375add3c98d84076905"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"b73c4255235e81d9d550b27e75bd3b4fb5b58dea","unresolved":false,"context_lines":[{"line_number":91,"context_line":"---------------------"},{"line_number":92,"context_line":""},{"line_number":93,"context_line":"When using the NSXv3 plugion the deployer should NOT configure the"},{"line_number":94,"context_line":"\u0027integration_bridge\u0027 configuration variable in the \u0027vmware\u0027 section."},{"line_number":95,"context_line":""},{"line_number":96,"context_line":"Developer impact"},{"line_number":97,"context_line":"----------------"}],"source_content_type":"text/x-rst","patch_set":1,"id":"7a2fa921_736cd736","line":94,"updated":"2015-10-12 09:09:02.000000000","message":"It seems like the code changes the default of \u0027br-int\u0027 to None.\n\nThis feels like quite an upgrade impact, and technically a backwards incompatible change.\n\nIs there not a way to get the neutron plugin to send info so Nova knows if it should ignore the integration_bridge, or am I missing out a use case here?","commit_id":"e5a71738fc69710dfe761375add3c98d84076905"},{"author":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"change_message_id":"d5958a0c179059b576578472f4a7364d58efe2b4","unresolved":false,"context_lines":[{"line_number":91,"context_line":"---------------------"},{"line_number":92,"context_line":""},{"line_number":93,"context_line":"When using the NSXv3 plugion the deployer should NOT configure the"},{"line_number":94,"context_line":"\u0027integration_bridge\u0027 configuration variable in the \u0027vmware\u0027 section."},{"line_number":95,"context_line":""},{"line_number":96,"context_line":"Developer impact"},{"line_number":97,"context_line":"----------------"}],"source_content_type":"text/x-rst","patch_set":1,"id":"7a2fa921_3a561d07","line":94,"in_reply_to":"7a2fa921_736cd736","updated":"2015-10-12 17:05:33.000000000","message":"At the moment we are unable to get this information from neutron. The API only has extensions and the same extensions are supported. It is something that can be looked at but is not in the context of this work.\nWe will document what administrators should do.","commit_id":"e5a71738fc69710dfe761375add3c98d84076905"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"630bf8e0658cf5d83db2a4367cc6c2001ed761b9","unresolved":false,"context_lines":[{"line_number":96,"context_line":""},{"line_number":97,"context_line":"The NSXv3 installation will be greenfield. So there is no upgrade impact."},{"line_number":98,"context_line":"For existing NSX-MH platforms the code is backwards compatible and those"},{"line_number":99,"context_line":"platforms will continue to work as they are today."},{"line_number":100,"context_line":""},{"line_number":101,"context_line":"Developer impact"},{"line_number":102,"context_line":"----------------"}],"source_content_type":"text/x-rst","patch_set":2,"id":"5a2ca52d_e92b1d32","line":99,"updated":"2015-10-13 10:53:38.000000000","message":"I think we should note that if you use the default integration bridge with NSX-MH, you need to update the configuration, due to how the code changes the default of \u0027br-int\u0027 to None. Unless I am miss understanding something.\n\nThis technically violates our upgrade commitments, but adding an extra configuration seems like a bad option.","commit_id":"9bc7dee46ea498ccc9a0ec4d4bb99302bd1dd519"},{"author":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"change_message_id":"2bd40838634e8684042b62f7cb66e6d0a4e90181","unresolved":false,"context_lines":[{"line_number":96,"context_line":""},{"line_number":97,"context_line":"The NSXv3 installation will be greenfield. So there is no upgrade impact."},{"line_number":98,"context_line":"For existing NSX-MH platforms the code is backwards compatible and those"},{"line_number":99,"context_line":"platforms will continue to work as they are today."},{"line_number":100,"context_line":""},{"line_number":101,"context_line":"Developer impact"},{"line_number":102,"context_line":"----------------"}],"source_content_type":"text/x-rst","patch_set":2,"id":"5a2ca52d_9e373ce3","line":99,"in_reply_to":"5a2ca52d_e92b1d32","updated":"2015-10-13 12:03:36.000000000","message":"Done\n\nI think that moving forwards we would like to use the NSXv3 and the NSX-MH will be deprecated. So it would be better moving forwards to change the default.","commit_id":"9bc7dee46ea498ccc9a0ec4d4bb99302bd1dd519"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"6d9e2ebcde22f60589fa07b1279067e4f078a6cf","unresolved":false,"context_lines":[{"line_number":10,"context_line":""},{"line_number":11,"context_line":"https://blueprints.launchpad.net/nova/+spec/vmware-expand-opaque-support"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"An opaque network was introduced in the vSphere API in version 5.5. This is"},{"line_number":14,"context_line":"a network that is managed by a control plane outside of vSphere. The identifier"},{"line_number":15,"context_line":"and name of this network is made know to vSphere so that a host and virtual"},{"line_number":16,"context_line":"machine ethernet device can be connected to them."}],"source_content_type":"text/x-rst","patch_set":3,"id":"1a26ad4f_2d9185f1","line":13,"updated":"2015-11-02 15:14:19.000000000","message":"Current nova min VC version is 5.1.0, so would there be checking around that if you\u0027re trying to use NSXv3?","commit_id":"9c8ff331bc0dfb5f0c96436f4073497713c2f1f6"},{"author":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"change_message_id":"222cbe401e0915f51a02fa2e2e18057d3074b709","unresolved":false,"context_lines":[{"line_number":10,"context_line":""},{"line_number":11,"context_line":"https://blueprints.launchpad.net/nova/+spec/vmware-expand-opaque-support"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"An opaque network was introduced in the vSphere API in version 5.5. This is"},{"line_number":14,"context_line":"a network that is managed by a control plane outside of vSphere. The identifier"},{"line_number":15,"context_line":"and name of this network is made know to vSphere so that a host and virtual"},{"line_number":16,"context_line":"machine ethernet device can be connected to them."}],"source_content_type":"text/x-rst","patch_set":3,"id":"fa80f949_b571eae2","line":13,"in_reply_to":"1a26ad4f_2d9185f1","updated":"2015-11-03 10:51:44.000000000","message":"At the moment the NSX-MH is support from VC 5.5 onward. The same here. This is something that we have documented. In the code we can add a check to provide logging that may help an admin trouble shoot.","commit_id":"9c8ff331bc0dfb5f0c96436f4073497713c2f1f6"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"348a5513d5cfffa4bc2f35fa298ee16c763ea916","unresolved":false,"context_lines":[{"line_number":10,"context_line":""},{"line_number":11,"context_line":"https://blueprints.launchpad.net/nova/+spec/vmware-expand-opaque-support"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"An opaque network was introduced in the vSphere API in version 5.5. This is"},{"line_number":14,"context_line":"a network that is managed by a control plane outside of vSphere. The identifier"},{"line_number":15,"context_line":"and name of this network is made know to vSphere so that a host and virtual"},{"line_number":16,"context_line":"machine ethernet device can be connected to them."}],"source_content_type":"text/x-rst","patch_set":3,"id":"fa80f949_527e89af","line":13,"in_reply_to":"fa80f949_b571eae2","updated":"2015-11-03 15:01:40.000000000","message":"OK, sounds like logging that would be orthogonal if we can detect it in the code.\n\nAs discussed in IRC, there are 4 supported neutron plugin options for vmware:\n\n1. NSX-MH, 2,NSXv 3. Simple DVS and 4. NSXv3 (which will deprecate the NSX-MH)\n\nIf you have vCenter \u003c 5.5, you can only use 2 and 3. 3 has limited neutron API support, 2 supports full neutron stack, and 4 replaces all of them.","commit_id":"9c8ff331bc0dfb5f0c96436f4073497713c2f1f6"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"6d9e2ebcde22f60589fa07b1279067e4f078a6cf","unresolved":false,"context_lines":[{"line_number":12,"context_line":""},{"line_number":13,"context_line":"An opaque network was introduced in the vSphere API in version 5.5. This is"},{"line_number":14,"context_line":"a network that is managed by a control plane outside of vSphere. The identifier"},{"line_number":15,"context_line":"and name of this network is made know to vSphere so that a host and virtual"},{"line_number":16,"context_line":"machine ethernet device can be connected to them."},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"The initial code was added to support the NSX-MH (multi hypervisor) Neutron"}],"source_content_type":"text/x-rst","patch_set":3,"id":"1a26ad4f_2de3c5a5","line":15,"updated":"2015-11-02 15:14:19.000000000","message":"s/know/known/","commit_id":"9c8ff331bc0dfb5f0c96436f4073497713c2f1f6"},{"author":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"change_message_id":"222cbe401e0915f51a02fa2e2e18057d3074b709","unresolved":false,"context_lines":[{"line_number":12,"context_line":""},{"line_number":13,"context_line":"An opaque network was introduced in the vSphere API in version 5.5. This is"},{"line_number":14,"context_line":"a network that is managed by a control plane outside of vSphere. The identifier"},{"line_number":15,"context_line":"and name of this network is made know to vSphere so that a host and virtual"},{"line_number":16,"context_line":"machine ethernet device can be connected to them."},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"The initial code was added to support the NSX-MH (multi hypervisor) Neutron"}],"source_content_type":"text/x-rst","patch_set":3,"id":"fa80f949_7574e2ce","line":15,"in_reply_to":"1a26ad4f_2de3c5a5","updated":"2015-11-03 10:51:44.000000000","message":"Done","commit_id":"9c8ff331bc0dfb5f0c96436f4073497713c2f1f6"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"6d9e2ebcde22f60589fa07b1279067e4f078a6cf","unresolved":false,"context_lines":[{"line_number":15,"context_line":"and name of this network is made know to vSphere so that a host and virtual"},{"line_number":16,"context_line":"machine ethernet device can be connected to them."},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"The initial code was added to support the NSX-MH (multi hypervisor) Neutron"},{"line_number":19,"context_line":"plugin. This was in commit 2d7520264a4610068630d7664eeff70fb5e8c681. That"},{"line_number":20,"context_line":"support would require the configuration of a global integration bridge and"},{"line_number":21,"context_line":"ensuring that the network was connected to that bridge."}],"source_content_type":"text/x-rst","patch_set":3,"id":"1a26ad4f_90d0eef7","line":18,"updated":"2015-11-02 15:14:19.000000000","message":"Is that this?\n\nhttps://github.com/openstack/vmware-nsx\n\nI get confused by all of the different neutron plugins.  As far as I know, there is only nsx and dvSwitch for vmware plugins in neutron, is that right?  Does this impact the usage of the neutron dvswitch plugin at all?","commit_id":"9c8ff331bc0dfb5f0c96436f4073497713c2f1f6"},{"author":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"change_message_id":"222cbe401e0915f51a02fa2e2e18057d3074b709","unresolved":false,"context_lines":[{"line_number":15,"context_line":"and name of this network is made know to vSphere so that a host and virtual"},{"line_number":16,"context_line":"machine ethernet device can be connected to them."},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"The initial code was added to support the NSX-MH (multi hypervisor) Neutron"},{"line_number":19,"context_line":"plugin. This was in commit 2d7520264a4610068630d7664eeff70fb5e8c681. That"},{"line_number":20,"context_line":"support would require the configuration of a global integration bridge and"},{"line_number":21,"context_line":"ensuring that the network was connected to that bridge."}],"source_content_type":"text/x-rst","patch_set":3,"id":"fa80f949_381ad16f","line":18,"in_reply_to":"1a26ad4f_90d0eef7","updated":"2015-11-03 10:51:44.000000000","message":"No. The plugin is https://github.com/openstack/neutron/blob/stable/juno/neutron/plugins/vmware/plugin.py#L21 (that was later moved to the vmware_nsx repo).\n\nThe NSX-MH and the NSX_V3 do not use the DVS (distributed vSwitches. They make use of a new switch called an opaque switch)","commit_id":"9c8ff331bc0dfb5f0c96436f4073497713c2f1f6"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"6d9e2ebcde22f60589fa07b1279067e4f078a6cf","unresolved":false,"context_lines":[{"line_number":20,"context_line":"support would require the configuration of a global integration bridge and"},{"line_number":21,"context_line":"ensuring that the network was connected to that bridge."},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"In the Liberty cycle support for the next generation Neutron plugin, NSXv3,"},{"line_number":24,"context_line":"was added. The compute integration for this plugin needs to make use of the"},{"line_number":25,"context_line":"Neutron network ID and not the integration brdige."},{"line_number":26,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"1a26ad4f_7098b2c0","line":23,"updated":"2015-11-02 15:14:19.000000000","message":"Support was added in Neutron, not Nova, right? If Nova, is there a commit to reference?","commit_id":"9c8ff331bc0dfb5f0c96436f4073497713c2f1f6"},{"author":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"change_message_id":"222cbe401e0915f51a02fa2e2e18057d3074b709","unresolved":false,"context_lines":[{"line_number":20,"context_line":"support would require the configuration of a global integration bridge and"},{"line_number":21,"context_line":"ensuring that the network was connected to that bridge."},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"In the Liberty cycle support for the next generation Neutron plugin, NSXv3,"},{"line_number":24,"context_line":"was added. The compute integration for this plugin needs to make use of the"},{"line_number":25,"context_line":"Neutron network ID and not the integration brdige."},{"line_number":26,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"fa80f949_7837f9e3","line":23,"in_reply_to":"1a26ad4f_7098b2c0","updated":"2015-11-03 10:51:44.000000000","message":"This was add in Neutron and not Nova.","commit_id":"9c8ff331bc0dfb5f0c96436f4073497713c2f1f6"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"6d9e2ebcde22f60589fa07b1279067e4f078a6cf","unresolved":false,"context_lines":[{"line_number":22,"context_line":""},{"line_number":23,"context_line":"In the Liberty cycle support for the next generation Neutron plugin, NSXv3,"},{"line_number":24,"context_line":"was added. The compute integration for this plugin needs to make use of the"},{"line_number":25,"context_line":"Neutron network ID and not the integration brdige."},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"This spec will deal with that integration."},{"line_number":28,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"1a26ad4f_0d9cc90d","line":25,"updated":"2015-11-02 15:14:19.000000000","message":"s/brdige/bridge/","commit_id":"9c8ff331bc0dfb5f0c96436f4073497713c2f1f6"},{"author":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"change_message_id":"222cbe401e0915f51a02fa2e2e18057d3074b709","unresolved":false,"context_lines":[{"line_number":22,"context_line":""},{"line_number":23,"context_line":"In the Liberty cycle support for the next generation Neutron plugin, NSXv3,"},{"line_number":24,"context_line":"was added. The compute integration for this plugin needs to make use of the"},{"line_number":25,"context_line":"Neutron network ID and not the integration brdige."},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"This spec will deal with that integration."},{"line_number":28,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"fa80f949_987bbd7b","line":25,"in_reply_to":"1a26ad4f_0d9cc90d","updated":"2015-11-03 10:51:44.000000000","message":"Done","commit_id":"9c8ff331bc0dfb5f0c96436f4073497713c2f1f6"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"6d9e2ebcde22f60589fa07b1279067e4f078a6cf","unresolved":false,"context_lines":[{"line_number":46,"context_line":"virtual port is of the type \u0027ovs\u0027. When the type is \u0027ovs\u0027 there are two"},{"line_number":47,"context_line":"different flows:"},{"line_number":48,"context_line":""},{"line_number":49,"context_line":"* If the configuration flag \u0027integration_bridge\u0027 is set. This is for the"},{"line_number":50,"context_line":"  NSX-MH plugin. This requires that the backing type opaqueNetworkId be set"},{"line_number":51,"context_line":"  as the \u0027integration_bridge\u0027; the backing type opaqueNetworkType be set as"},{"line_number":52,"context_line":"  \u0027opaque\u0027."}],"source_content_type":"text/x-rst","patch_set":3,"id":"1a26ad4f_1065beb3","line":49,"updated":"2015-11-02 15:14:19.000000000","message":"This is set to br-int by default:\n\nhttps://github.com/openstack/nova/blob/master/nova/virt/vmwareapi/vif.py#L36\n\nShould we have been deprecating that option or adding more help text to it to explain in what cases it\u0027s used and what the values should be, including how None is handled?","commit_id":"9c8ff331bc0dfb5f0c96436f4073497713c2f1f6"},{"author":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"change_message_id":"222cbe401e0915f51a02fa2e2e18057d3074b709","unresolved":false,"context_lines":[{"line_number":46,"context_line":"virtual port is of the type \u0027ovs\u0027. When the type is \u0027ovs\u0027 there are two"},{"line_number":47,"context_line":"different flows:"},{"line_number":48,"context_line":""},{"line_number":49,"context_line":"* If the configuration flag \u0027integration_bridge\u0027 is set. This is for the"},{"line_number":50,"context_line":"  NSX-MH plugin. This requires that the backing type opaqueNetworkId be set"},{"line_number":51,"context_line":"  as the \u0027integration_bridge\u0027; the backing type opaqueNetworkType be set as"},{"line_number":52,"context_line":"  \u0027opaque\u0027."}],"source_content_type":"text/x-rst","patch_set":3,"id":"fa80f949_38e4113a","line":49,"in_reply_to":"1a26ad4f_1065beb3","updated":"2015-11-03 10:51:44.000000000","message":"This option can and should only be deprecated when the NSX-MH plugin is deprecated. The help for that configuration value should be updated.","commit_id":"9c8ff331bc0dfb5f0c96436f4073497713c2f1f6"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"348a5513d5cfffa4bc2f35fa298ee16c763ea916","unresolved":false,"context_lines":[{"line_number":46,"context_line":"virtual port is of the type \u0027ovs\u0027. When the type is \u0027ovs\u0027 there are two"},{"line_number":47,"context_line":"different flows:"},{"line_number":48,"context_line":""},{"line_number":49,"context_line":"* If the configuration flag \u0027integration_bridge\u0027 is set. This is for the"},{"line_number":50,"context_line":"  NSX-MH plugin. This requires that the backing type opaqueNetworkId be set"},{"line_number":51,"context_line":"  as the \u0027integration_bridge\u0027; the backing type opaqueNetworkType be set as"},{"line_number":52,"context_line":"  \u0027opaque\u0027."}],"source_content_type":"text/x-rst","patch_set":3,"id":"fa80f949_088f7024","line":49,"in_reply_to":"fa80f949_38e4113a","updated":"2015-11-03 15:01:40.000000000","message":"When is the NSX-MH plugin deprecated?  As part of this change to add support for NSXv3?","commit_id":"9c8ff331bc0dfb5f0c96436f4073497713c2f1f6"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"6d9e2ebcde22f60589fa07b1279067e4f078a6cf","unresolved":false,"context_lines":[{"line_number":51,"context_line":"  as the \u0027integration_bridge\u0027; the backing type opaqueNetworkType be set as"},{"line_number":52,"context_line":"  \u0027opaque\u0027."},{"line_number":53,"context_line":""},{"line_number":54,"context_line":"* If the flag is not set then this is the the NSXv3 plugin. This requires that"},{"line_number":55,"context_line":"  the backing value opaqueNetworkId be set as the neutron network UUID; the"},{"line_number":56,"context_line":"  backing type opaqueNetworkType will have value \u0027nsx.LogicalSwitch\u0027; and the"},{"line_number":57,"context_line":"  backing externalId  has the the neutron port UUID."}],"source_content_type":"text/x-rst","patch_set":3,"id":"1a26ad4f_10f05efd","line":54,"updated":"2015-11-02 15:14:19.000000000","message":"s/the the/the/","commit_id":"9c8ff331bc0dfb5f0c96436f4073497713c2f1f6"},{"author":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"change_message_id":"222cbe401e0915f51a02fa2e2e18057d3074b709","unresolved":false,"context_lines":[{"line_number":51,"context_line":"  as the \u0027integration_bridge\u0027; the backing type opaqueNetworkType be set as"},{"line_number":52,"context_line":"  \u0027opaque\u0027."},{"line_number":53,"context_line":""},{"line_number":54,"context_line":"* If the flag is not set then this is the the NSXv3 plugin. This requires that"},{"line_number":55,"context_line":"  the backing value opaqueNetworkId be set as the neutron network UUID; the"},{"line_number":56,"context_line":"  backing type opaqueNetworkType will have value \u0027nsx.LogicalSwitch\u0027; and the"},{"line_number":57,"context_line":"  backing externalId  has the the neutron port UUID."}],"source_content_type":"text/x-rst","patch_set":3,"id":"fa80f949_f809a9de","line":54,"in_reply_to":"1a26ad4f_10f05efd","updated":"2015-11-03 10:51:44.000000000","message":"Done","commit_id":"9c8ff331bc0dfb5f0c96436f4073497713c2f1f6"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"6d9e2ebcde22f60589fa07b1279067e4f078a6cf","unresolved":false,"context_lines":[{"line_number":96,"context_line":""},{"line_number":97,"context_line":"The NSXv3 installation will be greenfield. So there is no upgrade impact."},{"line_number":98,"context_line":"Due to the fact that the NSX-MH will be deprecated and replaced by the NSXv3"},{"line_number":99,"context_line":"we will set the default \u0027integration_bridge\u0027 value as None. This means that"},{"line_number":100,"context_line":"a user running the existing NSX-MH will need to make sure that this value"},{"line_number":101,"context_line":"is set. This is something that will be clearly documented."},{"line_number":102,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"1a26ad4f_f02da246","line":99,"updated":"2015-11-02 15:14:19.000000000","message":"OK, this is what I was wondering about above.\n\nDoes this impact usage of the neutron vmware dvswitch plugin at all?  If not, maybe we should call that out somewhere in the spec.","commit_id":"9c8ff331bc0dfb5f0c96436f4073497713c2f1f6"},{"author":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"change_message_id":"222cbe401e0915f51a02fa2e2e18057d3074b709","unresolved":false,"context_lines":[{"line_number":96,"context_line":""},{"line_number":97,"context_line":"The NSXv3 installation will be greenfield. So there is no upgrade impact."},{"line_number":98,"context_line":"Due to the fact that the NSX-MH will be deprecated and replaced by the NSXv3"},{"line_number":99,"context_line":"we will set the default \u0027integration_bridge\u0027 value as None. This means that"},{"line_number":100,"context_line":"a user running the existing NSX-MH will need to make sure that this value"},{"line_number":101,"context_line":"is set. This is something that will be clearly documented."},{"line_number":102,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"fa80f949_18db4d1d","line":99,"in_reply_to":"1a26ad4f_f02da246","updated":"2015-11-03 10:51:44.000000000","message":"No. This does not have any impact on that. In fact and admin can use the same VC for multiple different Neutron plugins if they wish.","commit_id":"9c8ff331bc0dfb5f0c96436f4073497713c2f1f6"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"6d9e2ebcde22f60589fa07b1279067e4f078a6cf","unresolved":false,"context_lines":[{"line_number":125,"context_line":"Dependencies"},{"line_number":126,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":127,"context_line":""},{"line_number":128,"context_line":"This code depends on the Neutron driver NSXv3 added in the Liberty cycle."},{"line_number":129,"context_line":""},{"line_number":130,"context_line":""},{"line_number":131,"context_line":"Testing"}],"source_content_type":"text/x-rst","patch_set":3,"id":"1a26ad4f_10da9e4c","line":128,"updated":"2015-11-02 15:14:19.000000000","message":"Is there a link to this in a separate git repo as part of the \u0027neutron stadium\u0027?  Might be good to reference the repo for this if it\u0027s not in-tree neutron.","commit_id":"9c8ff331bc0dfb5f0c96436f4073497713c2f1f6"},{"author":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"change_message_id":"222cbe401e0915f51a02fa2e2e18057d3074b709","unresolved":false,"context_lines":[{"line_number":125,"context_line":"Dependencies"},{"line_number":126,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":127,"context_line":""},{"line_number":128,"context_line":"This code depends on the Neutron driver NSXv3 added in the Liberty cycle."},{"line_number":129,"context_line":""},{"line_number":130,"context_line":""},{"line_number":131,"context_line":"Testing"}],"source_content_type":"text/x-rst","patch_set":3,"id":"fa80f949_387411fc","line":128,"in_reply_to":"1a26ad4f_10da9e4c","updated":"2015-11-03 10:51:44.000000000","message":"Done","commit_id":"9c8ff331bc0dfb5f0c96436f4073497713c2f1f6"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"6d9e2ebcde22f60589fa07b1279067e4f078a6cf","unresolved":false,"context_lines":[{"line_number":131,"context_line":"Testing"},{"line_number":132,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":133,"context_line":""},{"line_number":134,"context_line":"The code is tested as part of the Neutron CI testing."},{"line_number":135,"context_line":""},{"line_number":136,"context_line":""},{"line_number":137,"context_line":"Documentation Impact"}],"source_content_type":"text/x-rst","patch_set":3,"id":"1a26ad4f_d0df4638","line":134,"updated":"2015-11-02 15:14:19.000000000","message":"Neutron NSX CI testing, right?","commit_id":"9c8ff331bc0dfb5f0c96436f4073497713c2f1f6"},{"author":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"change_message_id":"222cbe401e0915f51a02fa2e2e18057d3074b709","unresolved":false,"context_lines":[{"line_number":131,"context_line":"Testing"},{"line_number":132,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":133,"context_line":""},{"line_number":134,"context_line":"The code is tested as part of the Neutron CI testing."},{"line_number":135,"context_line":""},{"line_number":136,"context_line":""},{"line_number":137,"context_line":"Documentation Impact"}],"source_content_type":"text/x-rst","patch_set":3,"id":"fa80f949_98597d91","line":134,"in_reply_to":"1a26ad4f_d0df4638","updated":"2015-11-03 10:51:44.000000000","message":"This is run with each VMware NSX patch posted","commit_id":"9c8ff331bc0dfb5f0c96436f4073497713c2f1f6"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"6d9e2ebcde22f60589fa07b1279067e4f078a6cf","unresolved":false,"context_lines":[{"line_number":137,"context_line":"Documentation Impact"},{"line_number":138,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":139,"context_line":""},{"line_number":140,"context_line":"A deployer is only affected by this."},{"line_number":141,"context_line":""},{"line_number":142,"context_line":"References"},{"line_number":143,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":3,"id":"1a26ad4f_30c6ba68","line":140,"updated":"2015-11-02 15:14:19.000000000","message":"This is kind of an odd statement.  I assume you mean the deployer that\u0027s using NSX-MH or wants to use NSXv3 is affected.  There would be doc changes to the help text of the CONF.vmware.integration_bridge config option and there would be a release note impact for nova in mitaka.","commit_id":"9c8ff331bc0dfb5f0c96436f4073497713c2f1f6"},{"author":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"change_message_id":"222cbe401e0915f51a02fa2e2e18057d3074b709","unresolved":false,"context_lines":[{"line_number":137,"context_line":"Documentation Impact"},{"line_number":138,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":139,"context_line":""},{"line_number":140,"context_line":"A deployer is only affected by this."},{"line_number":141,"context_line":""},{"line_number":142,"context_line":"References"},{"line_number":143,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":3,"id":"fa80f949_b8a3214f","line":140,"in_reply_to":"1a26ad4f_30c6ba68","updated":"2015-11-03 10:51:44.000000000","message":"Done","commit_id":"9c8ff331bc0dfb5f0c96436f4073497713c2f1f6"},{"author":{"_account_id":13438,"name":"Kobi Samoray","email":"kobi.samoray@broadcom.com","username":"ksamoray"},"change_message_id":"b7c189c2ae64483dd4540653e84ab4f0037e40fb","unresolved":false,"context_lines":[{"line_number":127,"context_line":"----------"},{"line_number":128,"context_line":""},{"line_number":129,"context_line":"The code has been implemented in the Liberty cycle and can be seen at"},{"line_number":130,"context_line":"https://review.openstack.org/#/c/165750/."},{"line_number":131,"context_line":""},{"line_number":132,"context_line":""},{"line_number":133,"context_line":"Dependencies"}],"source_content_type":"text/x-rst","patch_set":4,"id":"fa80f949_39589056","line":130,"updated":"2015-11-03 13:36:20.000000000","message":"That won\u0027t fall in during Liberty cycle","commit_id":"3b80ceb9cd5e09fd4bf8da670323c1b6e4e33213"},{"author":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"change_message_id":"acfe5dd4d83fe38ad4b70dc59e15ff85c333f32f","unresolved":false,"context_lines":[{"line_number":127,"context_line":"----------"},{"line_number":128,"context_line":""},{"line_number":129,"context_line":"The code has been implemented in the Liberty cycle and can be seen at"},{"line_number":130,"context_line":"https://review.openstack.org/#/c/165750/."},{"line_number":131,"context_line":""},{"line_number":132,"context_line":""},{"line_number":133,"context_line":"Dependencies"}],"source_content_type":"text/x-rst","patch_set":4,"id":"fa80f949_5c1a2270","line":130,"in_reply_to":"fa80f949_39589056","updated":"2015-11-03 13:39:51.000000000","message":"Done","commit_id":"3b80ceb9cd5e09fd4bf8da670323c1b6e4e33213"},{"author":{"_account_id":261,"name":"Salvatore Orlando","email":"salv.orlando@gmail.com","username":"salvatore-orlando"},"change_message_id":"bb2c106f72a4f835e5e6c8c471baadfed2ee4a17","unresolved":false,"context_lines":[{"line_number":18,"context_line":"The initial code was added to support the NSX-MH (multi hypervisor) Neutron"},{"line_number":19,"context_line":"plugin. This was in commit 2d7520264a4610068630d7664eeff70fb5e8c681. That"},{"line_number":20,"context_line":"support would require the configuration of a global integration bridge and"},{"line_number":21,"context_line":"ensuring that the network was connected to that bridge."},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"In the Liberty cycle support for the next generation Neutron plugin, NSXv3,"},{"line_number":24,"context_line":"was added to the vmware_nsx repository. The compute integration for this plugin"}],"source_content_type":"text/x-rst","patch_set":5,"id":"fa80f949_12e241d6","line":21,"updated":"2015-11-03 14:38:15.000000000","message":"If you wish, you can add, for the sake of non vmware experts, that this approach is very similar to libvirt VIF driver","commit_id":"27cd7039d72daeaa9ce6a786239a7184a269de38"},{"author":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"change_message_id":"a7912086a035e0e8554d8e661b2cac63ccc13524","unresolved":false,"context_lines":[{"line_number":18,"context_line":"The initial code was added to support the NSX-MH (multi hypervisor) Neutron"},{"line_number":19,"context_line":"plugin. This was in commit 2d7520264a4610068630d7664eeff70fb5e8c681. That"},{"line_number":20,"context_line":"support would require the configuration of a global integration bridge and"},{"line_number":21,"context_line":"ensuring that the network was connected to that bridge."},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"In the Liberty cycle support for the next generation Neutron plugin, NSXv3,"},{"line_number":24,"context_line":"was added to the vmware_nsx repository. The compute integration for this plugin"}],"source_content_type":"text/x-rst","patch_set":5,"id":"fa80f949_bac52a5c","line":21,"in_reply_to":"fa80f949_12e241d6","updated":"2015-11-04 09:28:26.000000000","message":"Done","commit_id":"27cd7039d72daeaa9ce6a786239a7184a269de38"},{"author":{"_account_id":261,"name":"Salvatore Orlando","email":"salv.orlando@gmail.com","username":"salvatore-orlando"},"change_message_id":"bb2c106f72a4f835e5e6c8c471baadfed2ee4a17","unresolved":false,"context_lines":[{"line_number":20,"context_line":"support would require the configuration of a global integration bridge and"},{"line_number":21,"context_line":"ensuring that the network was connected to that bridge."},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"In the Liberty cycle support for the next generation Neutron plugin, NSXv3,"},{"line_number":24,"context_line":"was added to the vmware_nsx repository. The compute integration for this plugin"},{"line_number":25,"context_line":"needs to make use of the Neutron network ID and not the integration bridge."},{"line_number":26,"context_line":""}],"source_content_type":"text/x-rst","patch_set":5,"id":"fa80f949_324985d1","line":23,"updated":"2015-11-03 14:38:15.000000000","message":"avoid confusion (and buzzwords). In liberty a plugin for a new NSX backend was added to openstack/vmware-nsx - that\u0027s what you are trying to communicate imho.","commit_id":"27cd7039d72daeaa9ce6a786239a7184a269de38"},{"author":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"change_message_id":"a7912086a035e0e8554d8e661b2cac63ccc13524","unresolved":false,"context_lines":[{"line_number":20,"context_line":"support would require the configuration of a global integration bridge and"},{"line_number":21,"context_line":"ensuring that the network was connected to that bridge."},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"In the Liberty cycle support for the next generation Neutron plugin, NSXv3,"},{"line_number":24,"context_line":"was added to the vmware_nsx repository. The compute integration for this plugin"},{"line_number":25,"context_line":"needs to make use of the Neutron network ID and not the integration bridge."},{"line_number":26,"context_line":""}],"source_content_type":"text/x-rst","patch_set":5,"id":"fa80f949_9aaaa618","line":23,"in_reply_to":"fa80f949_324985d1","updated":"2015-11-04 09:28:26.000000000","message":"Done","commit_id":"27cd7039d72daeaa9ce6a786239a7184a269de38"},{"author":{"_account_id":261,"name":"Salvatore Orlando","email":"salv.orlando@gmail.com","username":"salvatore-orlando"},"change_message_id":"bb2c106f72a4f835e5e6c8c471baadfed2ee4a17","unresolved":false,"context_lines":[{"line_number":22,"context_line":""},{"line_number":23,"context_line":"In the Liberty cycle support for the next generation Neutron plugin, NSXv3,"},{"line_number":24,"context_line":"was added to the vmware_nsx repository. The compute integration for this plugin"},{"line_number":25,"context_line":"needs to make use of the Neutron network ID and not the integration bridge."},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"This spec will deal with that integration."},{"line_number":28,"context_line":""}],"source_content_type":"text/x-rst","patch_set":5,"id":"fa80f949_d2a3594e","line":25,"updated":"2015-11-03 14:38:15.000000000","message":"compute integration \u003d\u003d vif driver, right?","commit_id":"27cd7039d72daeaa9ce6a786239a7184a269de38"},{"author":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"change_message_id":"a7912086a035e0e8554d8e661b2cac63ccc13524","unresolved":false,"context_lines":[{"line_number":22,"context_line":""},{"line_number":23,"context_line":"In the Liberty cycle support for the next generation Neutron plugin, NSXv3,"},{"line_number":24,"context_line":"was added to the vmware_nsx repository. The compute integration for this plugin"},{"line_number":25,"context_line":"needs to make use of the Neutron network ID and not the integration bridge."},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"This spec will deal with that integration."},{"line_number":28,"context_line":""}],"source_content_type":"text/x-rst","patch_set":5,"id":"fa80f949_1a6d5604","line":25,"in_reply_to":"fa80f949_d2a3594e","updated":"2015-11-04 09:28:26.000000000","message":"No. The nova vmware code does not have a vif driver","commit_id":"27cd7039d72daeaa9ce6a786239a7184a269de38"},{"author":{"_account_id":16929,"name":"Giridhar Jayavelu","email":"gjayavelu@vmware.com","username":"gjayavelu"},"change_message_id":"7679e76ede018aa5ca3b37343d6d622e39e301a9","unresolved":false,"context_lines":[{"line_number":43,"context_line":""},{"line_number":44,"context_line":"The change is self contained within the VMware driver code and just related to"},{"line_number":45,"context_line":"how the ethernet device backing is configured. This is only when the Neutron"},{"line_number":46,"context_line":"virtual port is of the type \u0027ovs\u0027. When the type is \u0027ovs\u0027 there are two"},{"line_number":47,"context_line":"different flows:"},{"line_number":48,"context_line":""},{"line_number":49,"context_line":"* If the configuration flag \u0027integration_bridge\u0027 is set. This is for the"}],"source_content_type":"text/x-rst","patch_set":5,"id":"fa80f949_dab5a77e","line":46,"updated":"2015-11-03 17:30:21.000000000","message":"would be good to mention explicitly that \u0027ovs\u0027 is the port type in NSXv3 even for ESX.","commit_id":"27cd7039d72daeaa9ce6a786239a7184a269de38"},{"author":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"change_message_id":"a7912086a035e0e8554d8e661b2cac63ccc13524","unresolved":false,"context_lines":[{"line_number":43,"context_line":""},{"line_number":44,"context_line":"The change is self contained within the VMware driver code and just related to"},{"line_number":45,"context_line":"how the ethernet device backing is configured. This is only when the Neutron"},{"line_number":46,"context_line":"virtual port is of the type \u0027ovs\u0027. When the type is \u0027ovs\u0027 there are two"},{"line_number":47,"context_line":"different flows:"},{"line_number":48,"context_line":""},{"line_number":49,"context_line":"* If the configuration flag \u0027integration_bridge\u0027 is set. This is for the"}],"source_content_type":"text/x-rst","patch_set":5,"id":"fa80f949_fa097232","line":46,"in_reply_to":"fa80f949_dab5a77e","updated":"2015-11-04 09:28:26.000000000","message":"Done","commit_id":"27cd7039d72daeaa9ce6a786239a7184a269de38"},{"author":{"_account_id":16929,"name":"Giridhar Jayavelu","email":"gjayavelu@vmware.com","username":"gjayavelu"},"change_message_id":"7679e76ede018aa5ca3b37343d6d622e39e301a9","unresolved":false,"context_lines":[{"line_number":54,"context_line":"* If the flag is not set then this is the NSXv3 plugin. This requires that"},{"line_number":55,"context_line":"  the backing value opaqueNetworkId be set as the neutron network UUID; the"},{"line_number":56,"context_line":"  backing type opaqueNetworkType will have value \u0027nsx.LogicalSwitch\u0027; and the"},{"line_number":57,"context_line":"  backing externalId  has the the neutron port UUID."},{"line_number":58,"context_line":""},{"line_number":59,"context_line":"NOTES:"},{"line_number":60,"context_line":""}],"source_content_type":"text/x-rst","patch_set":5,"id":"fa80f949_3a6873f5","line":57,"updated":"2015-11-03 17:30:21.000000000","message":"s/externalId  has the the/externalId has the/","commit_id":"27cd7039d72daeaa9ce6a786239a7184a269de38"},{"author":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"change_message_id":"a7912086a035e0e8554d8e661b2cac63ccc13524","unresolved":false,"context_lines":[{"line_number":54,"context_line":"* If the flag is not set then this is the NSXv3 plugin. This requires that"},{"line_number":55,"context_line":"  the backing value opaqueNetworkId be set as the neutron network UUID; the"},{"line_number":56,"context_line":"  backing type opaqueNetworkType will have value \u0027nsx.LogicalSwitch\u0027; and the"},{"line_number":57,"context_line":"  backing externalId  has the the neutron port UUID."},{"line_number":58,"context_line":""},{"line_number":59,"context_line":"NOTES:"},{"line_number":60,"context_line":""}],"source_content_type":"text/x-rst","patch_set":5,"id":"fa80f949_7c09ff2b","line":57,"in_reply_to":"fa80f949_3a6873f5","updated":"2015-11-04 09:28:26.000000000","message":"Done","commit_id":"27cd7039d72daeaa9ce6a786239a7184a269de38"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"8c508efdead37c29f3ba43639ee608383f3510a2","unresolved":false,"context_lines":[{"line_number":56,"context_line":"  backing type opaqueNetworkType will have value \u0027nsx.LogicalSwitch\u0027; and the"},{"line_number":57,"context_line":"  backing externalId  has the the neutron port UUID."},{"line_number":58,"context_line":""},{"line_number":59,"context_line":"NOTES:"},{"line_number":60,"context_line":""},{"line_number":61,"context_line":"* The help for the configuration option \u0027integration_bridge\u0027 will be updated"},{"line_number":62,"context_line":"  to reflect the values for the different plugins."}],"source_content_type":"text/x-rst","patch_set":5,"id":"fa80f949_afdae376","line":59,"updated":"2015-11-03 17:04:37.000000000","message":"nit: could use .. note:: syntax here for nicer rendering.","commit_id":"27cd7039d72daeaa9ce6a786239a7184a269de38"},{"author":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"change_message_id":"a7912086a035e0e8554d8e661b2cac63ccc13524","unresolved":false,"context_lines":[{"line_number":56,"context_line":"  backing type opaqueNetworkType will have value \u0027nsx.LogicalSwitch\u0027; and the"},{"line_number":57,"context_line":"  backing externalId  has the the neutron port UUID."},{"line_number":58,"context_line":""},{"line_number":59,"context_line":"NOTES:"},{"line_number":60,"context_line":""},{"line_number":61,"context_line":"* The help for the configuration option \u0027integration_bridge\u0027 will be updated"},{"line_number":62,"context_line":"  to reflect the values for the different plugins."}],"source_content_type":"text/x-rst","patch_set":5,"id":"fa80f949_dc2b4b81","line":59,"in_reply_to":"fa80f949_afdae376","updated":"2015-11-04 09:28:26.000000000","message":"Done","commit_id":"27cd7039d72daeaa9ce6a786239a7184a269de38"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"8c508efdead37c29f3ba43639ee608383f3510a2","unresolved":false,"context_lines":[{"line_number":58,"context_line":""},{"line_number":59,"context_line":"NOTES:"},{"line_number":60,"context_line":""},{"line_number":61,"context_line":"* The help for the configuration option \u0027integration_bridge\u0027 will be updated"},{"line_number":62,"context_line":"  to reflect the values for the different plugins."},{"line_number":63,"context_line":""},{"line_number":64,"context_line":"* A log warning will appear if the invalid VC version is used."}],"source_content_type":"text/x-rst","patch_set":5,"id":"fa80f949_ef454b17","line":61,"updated":"2015-11-03 17:04:37.000000000","message":"Are these two things something we could/should do regardless of the nsxv3 support for this spec?","commit_id":"27cd7039d72daeaa9ce6a786239a7184a269de38"},{"author":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"change_message_id":"a7912086a035e0e8554d8e661b2cac63ccc13524","unresolved":false,"context_lines":[{"line_number":58,"context_line":""},{"line_number":59,"context_line":"NOTES:"},{"line_number":60,"context_line":""},{"line_number":61,"context_line":"* The help for the configuration option \u0027integration_bridge\u0027 will be updated"},{"line_number":62,"context_line":"  to reflect the values for the different plugins."},{"line_number":63,"context_line":""},{"line_number":64,"context_line":"* A log warning will appear if the invalid VC version is used."}],"source_content_type":"text/x-rst","patch_set":5,"id":"fa80f949_fc340fdf","line":61,"in_reply_to":"fa80f949_ef454b17","updated":"2015-11-04 09:28:26.000000000","message":"Yes","commit_id":"27cd7039d72daeaa9ce6a786239a7184a269de38"},{"author":{"_account_id":261,"name":"Salvatore Orlando","email":"salv.orlando@gmail.com","username":"salvatore-orlando"},"change_message_id":"bb2c106f72a4f835e5e6c8c471baadfed2ee4a17","unresolved":false,"context_lines":[{"line_number":104,"context_line":""},{"line_number":105,"context_line":"The NSXv3 installation will be greenfield. So there is no upgrade impact."},{"line_number":106,"context_line":"Due to the fact that the NSX-MH will be deprecated and replaced by the NSXv3"},{"line_number":107,"context_line":"we will set the default \u0027integration_bridge\u0027 value as None. This means that"},{"line_number":108,"context_line":"a user running the existing NSX-MH will need to make sure that this value"},{"line_number":109,"context_line":"is set. This is something that will be clearly documented."},{"line_number":110,"context_line":""}],"source_content_type":"text/x-rst","patch_set":5,"id":"fa80f949_8d41323c","line":107,"updated":"2015-11-03 14:38:15.000000000","message":"for the sake of readers you should note that this variable is specific to the VMware driver and that you believe there is no reason for going through a deprecation process for this configuration variable.\n\nAlso, this is just a pedant note about your wording... but \"deprecated and replaced\" conflicts a bit with the fact that all the installations are greenfield ;)","commit_id":"27cd7039d72daeaa9ce6a786239a7184a269de38"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"8c508efdead37c29f3ba43639ee608383f3510a2","unresolved":false,"context_lines":[{"line_number":104,"context_line":""},{"line_number":105,"context_line":"The NSXv3 installation will be greenfield. So there is no upgrade impact."},{"line_number":106,"context_line":"Due to the fact that the NSX-MH will be deprecated and replaced by the NSXv3"},{"line_number":107,"context_line":"we will set the default \u0027integration_bridge\u0027 value as None. This means that"},{"line_number":108,"context_line":"a user running the existing NSX-MH will need to make sure that this value"},{"line_number":109,"context_line":"is set. This is something that will be clearly documented."},{"line_number":110,"context_line":""}],"source_content_type":"text/x-rst","patch_set":5,"id":"fa80f949_cf25e75c","line":107,"in_reply_to":"fa80f949_49d57741","updated":"2015-11-03 17:04:37.000000000","message":"Or say \"The NSXv3 support will be greenfield.\"\n\nThe NSX-MH is not though, and that is going to be deprecated, so there is deployer impact if you were using that, and noting that the vmware.integration_bridge config option in nova is going to be deprecated is something to call out here.\n\nTo Salvatore\u0027s point, \u0027deprecated and replaced\u0027 could be \u0027deprecated in favor of NSXv3\u0027.","commit_id":"27cd7039d72daeaa9ce6a786239a7184a269de38"},{"author":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"change_message_id":"a7912086a035e0e8554d8e661b2cac63ccc13524","unresolved":false,"context_lines":[{"line_number":104,"context_line":""},{"line_number":105,"context_line":"The NSXv3 installation will be greenfield. So there is no upgrade impact."},{"line_number":106,"context_line":"Due to the fact that the NSX-MH will be deprecated and replaced by the NSXv3"},{"line_number":107,"context_line":"we will set the default \u0027integration_bridge\u0027 value as None. This means that"},{"line_number":108,"context_line":"a user running the existing NSX-MH will need to make sure that this value"},{"line_number":109,"context_line":"is set. This is something that will be clearly documented."},{"line_number":110,"context_line":""}],"source_content_type":"text/x-rst","patch_set":5,"id":"fa80f949_9ce2a302","line":107,"in_reply_to":"fa80f949_8d41323c","updated":"2015-11-04 09:28:26.000000000","message":"Done","commit_id":"27cd7039d72daeaa9ce6a786239a7184a269de38"},{"author":{"_account_id":4395,"name":"Aaron Rosen","email":"aaronorosen@gmail.com","username":"arosen"},"change_message_id":"5289c80e46a98666ef0719b2d95dc8c4a424f780","unresolved":false,"context_lines":[{"line_number":104,"context_line":""},{"line_number":105,"context_line":"The NSXv3 installation will be greenfield. So there is no upgrade impact."},{"line_number":106,"context_line":"Due to the fact that the NSX-MH will be deprecated and replaced by the NSXv3"},{"line_number":107,"context_line":"we will set the default \u0027integration_bridge\u0027 value as None. This means that"},{"line_number":108,"context_line":"a user running the existing NSX-MH will need to make sure that this value"},{"line_number":109,"context_line":"is set. This is something that will be clearly documented."},{"line_number":110,"context_line":""}],"source_content_type":"text/x-rst","patch_set":5,"id":"fa80f949_49d57741","line":107,"in_reply_to":"fa80f949_8d41323c","updated":"2015-11-03 16:28:39.000000000","message":"I\u0027d maybe even just set the section to none ;)","commit_id":"27cd7039d72daeaa9ce6a786239a7184a269de38"},{"author":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"change_message_id":"a7912086a035e0e8554d8e661b2cac63ccc13524","unresolved":false,"context_lines":[{"line_number":104,"context_line":""},{"line_number":105,"context_line":"The NSXv3 installation will be greenfield. So there is no upgrade impact."},{"line_number":106,"context_line":"Due to the fact that the NSX-MH will be deprecated and replaced by the NSXv3"},{"line_number":107,"context_line":"we will set the default \u0027integration_bridge\u0027 value as None. This means that"},{"line_number":108,"context_line":"a user running the existing NSX-MH will need to make sure that this value"},{"line_number":109,"context_line":"is set. This is something that will be clearly documented."},{"line_number":110,"context_line":""}],"source_content_type":"text/x-rst","patch_set":5,"id":"fa80f949_dcd82b56","line":107,"in_reply_to":"fa80f949_cf25e75c","updated":"2015-11-04 09:28:26.000000000","message":"Done","commit_id":"27cd7039d72daeaa9ce6a786239a7184a269de38"},{"author":{"_account_id":9172,"name":"Radoslav Gerganov","email":"rgerganov@vmware.com","username":"rgerganov"},"change_message_id":"f2f21150df8a4852f7b0e17cb9727b5abbf28c3a","unresolved":false,"context_lines":[{"line_number":160,"context_line":"References"},{"line_number":161,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":162,"context_line":""},{"line_number":163,"context_line":"* https://www.vmware.com/support/developer/converter-sdk/conv55_apireference/vim.OpaqueNetwork.html"},{"line_number":164,"context_line":""},{"line_number":165,"context_line":"* https://review.openstack.org/#/c/165750/"},{"line_number":166,"context_line":""}],"source_content_type":"text/x-rst","patch_set":6,"id":"fa80f949_f6c8a676","line":163,"updated":"2015-11-04 12:06:19.000000000","message":"http://pubs.vmware.com/vsphere-60/topic/com.vmware.wssdk.apiref.doc/vim.OpaqueNetwork.html is a better reference","commit_id":"de4e9d7c647f7b2ca92302f7ff74501d435749eb"}]}