)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":13995,"name":"Nate Johnston","email":"nate.johnston@redhat.com","username":"natejohnston"},"change_message_id":"972e468b464553e3f3bf5be8bae66f7ceb692766","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Extend api for HPB"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This spec introduces an Extend filed in Network API for"},{"line_number":10,"context_line":"Hierarchical Port Binding."},{"line_number":11,"context_line":"Implements: blueprint extend-api-for-hpb"},{"line_number":12,"context_line":"Change-Id: I9a8afc0f59c3129eb550862a080d6a72e4ca8186"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":13,"id":"3f79a3b5_7d62993f","line":9,"updated":"2018-09-13 15:56:14.000000000","message":"If I understand this correctly, I think you should say \"introduces an extension for Hierarchical Port Binding\".","commit_id":"fd9364dd488391eb49a26276bfa78037d710a26d"},{"author":{"_account_id":13995,"name":"Nate Johnston","email":"nate.johnston@redhat.com","username":"natejohnston"},"change_message_id":"972e468b464553e3f3bf5be8bae66f7ceb692766","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This spec introduces an Extend filed in Network API for"},{"line_number":10,"context_line":"Hierarchical Port Binding."},{"line_number":11,"context_line":"Implements: blueprint extend-api-for-hpb"},{"line_number":12,"context_line":"Change-Id: I9a8afc0f59c3129eb550862a080d6a72e4ca8186"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":13,"id":"3f79a3b5_1d59258d","line":11,"updated":"2018-09-13 15:56:14.000000000","message":"nit: please add a newline after the paragraph and before \"Implements\".","commit_id":"fd9364dd488391eb49a26276bfa78037d710a26d"}],"specs/stein/extend-api-for-hpb.rst":[{"author":{"_account_id":27815,"name":"Huang Cheng","email":"hcn518@gmail.com","username":"pedh"},"change_message_id":"6b0e0078cb6825675a83a97cb7b1dca480bf705e","unresolved":false,"context_lines":[{"line_number":16,"context_line":"Problem Description"},{"line_number":17,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"At present, in many scenarios such as finance or government, customers use port "},{"line_number":20,"context_line":"hierarchical binding to break through 4K network. In addition, for security and "},{"line_number":21,"context_line":"privacy needs, customers need to specify the provider of the underlying segment "},{"line_number":22,"context_line":"of the network, such as physnet1 for business network or physnet2 for storage "}],"source_content_type":"text/x-rst","patch_set":6,"id":"3f79a3b5_150a9c26","line":19,"range":{"start_line":19,"start_character":79,"end_line":19,"end_character":80},"updated":"2018-09-11 02:05:39.000000000","message":"some trailing whitespaces","commit_id":"61fdb426e593a267f9398ee244a0db83d1bf269c"},{"author":{"_account_id":27815,"name":"Huang Cheng","email":"hcn518@gmail.com","username":"pedh"},"change_message_id":"6b0e0078cb6825675a83a97cb7b1dca480bf705e","unresolved":false,"context_lines":[{"line_number":48,"context_line":"            \"provider:physical_network\": null,"},{"line_number":49,"context_line":"            \"provider:segmentation_id\": 599,"},{"line_number":50,"context_line":"            \"provider:network_type\": \"vxlan\"，"},{"line_number":51,"context_line":"            \"bottom_segment_provider\":\"physnet1\""},{"line_number":52,"context_line":"      }"},{"line_number":53,"context_line":"    }"},{"line_number":54,"context_line":""}],"source_content_type":"text/x-rst","patch_set":6,"id":"3f79a3b5_20d99883","line":51,"range":{"start_line":51,"start_character":12,"end_line":51,"end_character":48},"updated":"2018-09-11 02:05:39.000000000","message":"Not seems to be able to meet the demand of specifying different physical network for each binding host.","commit_id":"61fdb426e593a267f9398ee244a0db83d1bf269c"},{"author":{"_account_id":13995,"name":"Nate Johnston","email":"nate.johnston@redhat.com","username":"natejohnston"},"change_message_id":"b773d871e347a8cda67861ea7ce9cb618dbbbe83","unresolved":false,"context_lines":[{"line_number":17,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"At present, in many scenarios such as finance or government, customers use hierarchical port"},{"line_number":20,"context_line":"binding to break through 4K network. In addition, for security and privacy needs, customers"},{"line_number":21,"context_line":"need different providers for underlying segments of the networks, such asbussiness00 for"},{"line_number":22,"context_line":"business network or storage01 for storage network."},{"line_number":23,"context_line":"The current neutron API for the network does not support specify the provider of"}],"source_content_type":"text/x-rst","patch_set":10,"id":"3f79a3b5_dffbd012","line":20,"updated":"2018-09-11 20:42:21.000000000","message":"When you say \"4K network\" it is clear to me that you mean the limitation on having 4095 VLANs.  But it might be better for you to be precise and refer to that by name.","commit_id":"4ccff195ebf0c141e0922ec1a9ad880625358c1b"},{"author":{"_account_id":28056,"name":"Liushy","email":"liuxie_11@163.com","username":"liuxie_11"},"change_message_id":"549e4c2798bd675e0aef8756b18fd9a55a82b535","unresolved":false,"context_lines":[{"line_number":17,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"At present, in many scenarios such as finance or government, customers use hierarchical port"},{"line_number":20,"context_line":"binding to break through 4K network. In addition, for security and privacy needs, customers"},{"line_number":21,"context_line":"need different providers for underlying segments of the networks, such asbussiness00 for"},{"line_number":22,"context_line":"business network or storage01 for storage network."},{"line_number":23,"context_line":"The current neutron API for the network does not support specify the provider of"}],"source_content_type":"text/x-rst","patch_set":10,"id":"3f79a3b5_3359d434","line":20,"in_reply_to":"3f79a3b5_dffbd012","updated":"2018-09-12 07:51:50.000000000","message":"I will modify this description.","commit_id":"4ccff195ebf0c141e0922ec1a9ad880625358c1b"},{"author":{"_account_id":13995,"name":"Nate Johnston","email":"nate.johnston@redhat.com","username":"natejohnston"},"change_message_id":"b773d871e347a8cda67861ea7ce9cb618dbbbe83","unresolved":false,"context_lines":[{"line_number":18,"context_line":""},{"line_number":19,"context_line":"At present, in many scenarios such as finance or government, customers use hierarchical port"},{"line_number":20,"context_line":"binding to break through 4K network. In addition, for security and privacy needs, customers"},{"line_number":21,"context_line":"need different providers for underlying segments of the networks, such asbussiness00 for"},{"line_number":22,"context_line":"business network or storage01 for storage network."},{"line_number":23,"context_line":"The current neutron API for the network does not support specify the provider of"},{"line_number":24,"context_line":"the underlying segment[1]. For this requirement, we need to extend the network API"}],"source_content_type":"text/x-rst","patch_set":10,"id":"3f79a3b5_ff000c03","line":21,"updated":"2018-09-11 20:42:21.000000000","message":"What do you mean by \"different providers\" here - do you mean different ML2 mechanism drivers?  If so then you should use that nomenclature; if not, then can you give an example?","commit_id":"4ccff195ebf0c141e0922ec1a9ad880625358c1b"},{"author":{"_account_id":28056,"name":"Liushy","email":"liuxie_11@163.com","username":"liuxie_11"},"change_message_id":"549e4c2798bd675e0aef8756b18fd9a55a82b535","unresolved":false,"context_lines":[{"line_number":18,"context_line":""},{"line_number":19,"context_line":"At present, in many scenarios such as finance or government, customers use hierarchical port"},{"line_number":20,"context_line":"binding to break through 4K network. In addition, for security and privacy needs, customers"},{"line_number":21,"context_line":"need different providers for underlying segments of the networks, such asbussiness00 for"},{"line_number":22,"context_line":"business network or storage01 for storage network."},{"line_number":23,"context_line":"The current neutron API for the network does not support specify the provider of"},{"line_number":24,"context_line":"the underlying segment[1]. For this requirement, we need to extend the network API"}],"source_content_type":"text/x-rst","patch_set":10,"id":"3f79a3b5_306f8afd","line":21,"in_reply_to":"3f79a3b5_ff000c03","updated":"2018-09-12 07:51:50.000000000","message":"Sorry, this description is wrong. I mean different physical network names.For example,physnet1 or physnet2.I will modify this later.","commit_id":"4ccff195ebf0c141e0922ec1a9ad880625358c1b"},{"author":{"_account_id":13995,"name":"Nate Johnston","email":"nate.johnston@redhat.com","username":"natejohnston"},"change_message_id":"b773d871e347a8cda67861ea7ce9cb618dbbbe83","unresolved":false,"context_lines":[{"line_number":33,"context_line":"bottom-level driver, it only need to dynamically assign VLANID to the mached physnetworks."},{"line_number":34,"context_line":"For example:"},{"line_number":35,"context_line":"ml2_conf.ini conf like these:"},{"line_number":36,"context_line":"cluster1: network_vlan_ranges\u003dstorage00:1:2000,busness00:2001:4094"},{"line_number":37,"context_line":"cluster2: network_vlan_ranges\u003dstorage01:2001:4094,busness01:1:2000"},{"line_number":38,"context_line":"We can use the field to specify the provider prefix like \"storage\" for"},{"line_number":39,"context_line":"the bottom segment of storage-network."}],"source_content_type":"text/x-rst","patch_set":10,"id":"3f79a3b5_9f055813","line":36,"updated":"2018-09-11 20:42:21.000000000","message":"What is a \"cluster\"?  This appears to be undefined in this spec.","commit_id":"4ccff195ebf0c141e0922ec1a9ad880625358c1b"},{"author":{"_account_id":28056,"name":"Liushy","email":"liuxie_11@163.com","username":"liuxie_11"},"change_message_id":"549e4c2798bd675e0aef8756b18fd9a55a82b535","unresolved":false,"context_lines":[{"line_number":33,"context_line":"bottom-level driver, it only need to dynamically assign VLANID to the mached physnetworks."},{"line_number":34,"context_line":"For example:"},{"line_number":35,"context_line":"ml2_conf.ini conf like these:"},{"line_number":36,"context_line":"cluster1: network_vlan_ranges\u003dstorage00:1:2000,busness00:2001:4094"},{"line_number":37,"context_line":"cluster2: network_vlan_ranges\u003dstorage01:2001:4094,busness01:1:2000"},{"line_number":38,"context_line":"We can use the field to specify the provider prefix like \"storage\" for"},{"line_number":39,"context_line":"the bottom segment of storage-network."}],"source_content_type":"text/x-rst","patch_set":10,"id":"3f79a3b5_d073b6e7","line":36,"in_reply_to":"3f79a3b5_9f055813","updated":"2018-09-12 07:51:50.000000000","message":"A cluster is a set of hosts with the same bridge_mappings and network_vlan_ranges.","commit_id":"4ccff195ebf0c141e0922ec1a9ad880625358c1b"},{"author":{"_account_id":13995,"name":"Nate Johnston","email":"nate.johnston@redhat.com","username":"natejohnston"},"change_message_id":"b773d871e347a8cda67861ea7ce9cb618dbbbe83","unresolved":false,"context_lines":[{"line_number":34,"context_line":"For example:"},{"line_number":35,"context_line":"ml2_conf.ini conf like these:"},{"line_number":36,"context_line":"cluster1: network_vlan_ranges\u003dstorage00:1:2000,busness00:2001:4094"},{"line_number":37,"context_line":"cluster2: network_vlan_ranges\u003dstorage01:2001:4094,busness01:1:2000"},{"line_number":38,"context_line":"We can use the field to specify the provider prefix like \"storage\" for"},{"line_number":39,"context_line":"the bottom segment of storage-network."},{"line_number":40,"context_line":"POST /v2.0/networks"}],"source_content_type":"text/x-rst","patch_set":10,"id":"3f79a3b5_bf0a941d","line":37,"updated":"2018-09-11 20:42:21.000000000","message":"In your example REST below, \"bottom_segment_provider\" is \"storage\" but here it looks like you are using \"storage00\".  I would prefer it if neutron would not need to split strings this way, without a divider token.\n\nAlso this configuration is very difficult to read.  A more verbose declaration syntax might allow for this to be more easily understandable by operators.\n\nFinally, is this configuration going to be relatively static, or is it likely that a very large scale region might need to add on to it?  If so, adding a set of CRUD actions to manage the vlan to provider to \"cluster\" mapping and then storing the map in the database might be more straightforward.","commit_id":"4ccff195ebf0c141e0922ec1a9ad880625358c1b"},{"author":{"_account_id":13995,"name":"Nate Johnston","email":"nate.johnston@redhat.com","username":"natejohnston"},"change_message_id":"e3da52f1ec50c766b2de3b432fe46f5bc93f387e","unresolved":false,"context_lines":[{"line_number":34,"context_line":"For example:"},{"line_number":35,"context_line":"ml2_conf.ini conf like these:"},{"line_number":36,"context_line":"cluster1: network_vlan_ranges\u003dstorage00:1:2000,busness00:2001:4094"},{"line_number":37,"context_line":"cluster2: network_vlan_ranges\u003dstorage01:2001:4094,busness01:1:2000"},{"line_number":38,"context_line":"We can use the field to specify the provider prefix like \"storage\" for"},{"line_number":39,"context_line":"the bottom segment of storage-network."},{"line_number":40,"context_line":"POST /v2.0/networks"}],"source_content_type":"text/x-rst","patch_set":10,"id":"3f79a3b5_9de1b59e","line":37,"updated":"2018-09-13 15:56:37.000000000","message":"| \"adding a set of CRUD actions to manage the vlan\" - do you mean use the api of \"segments\" ?\n\nI mean the data you have assigned in ml2_conf.ini specifically.  Thinking like an operator, if I have an OpenStack deployment that is years old and has evolved and grown, I may need to adjust these mappings.","commit_id":"4ccff195ebf0c141e0922ec1a9ad880625358c1b"},{"author":{"_account_id":28056,"name":"Liushy","email":"liuxie_11@163.com","username":"liuxie_11"},"change_message_id":"549e4c2798bd675e0aef8756b18fd9a55a82b535","unresolved":false,"context_lines":[{"line_number":34,"context_line":"For example:"},{"line_number":35,"context_line":"ml2_conf.ini conf like these:"},{"line_number":36,"context_line":"cluster1: network_vlan_ranges\u003dstorage00:1:2000,busness00:2001:4094"},{"line_number":37,"context_line":"cluster2: network_vlan_ranges\u003dstorage01:2001:4094,busness01:1:2000"},{"line_number":38,"context_line":"We can use the field to specify the provider prefix like \"storage\" for"},{"line_number":39,"context_line":"the bottom segment of storage-network."},{"line_number":40,"context_line":"POST /v2.0/networks"}],"source_content_type":"text/x-rst","patch_set":10,"id":"3f79a3b5_b07abab7","line":37,"in_reply_to":"3f79a3b5_bf0a941d","updated":"2018-09-12 07:51:50.000000000","message":"What we consider is that every cluster has two physical networks which one for business and another for storage. Their business networks (or storage networks) have different physical network names but the same prefix.\nFor example, \"business00\" and \"business01\". They have the same prefix is  \"business\".The user specifies the prefix like \"business\" when create a vxlan network. Top-level driver dynamically assign VLANID to the physical network that maches the prefix when bind_port.\n\"adding a set of CRUD actions to manage the vlan\" - do you mean use the api of \"segments\" ?","commit_id":"4ccff195ebf0c141e0922ec1a9ad880625358c1b"},{"author":{"_account_id":13995,"name":"Nate Johnston","email":"nate.johnston@redhat.com","username":"natejohnston"},"change_message_id":"b773d871e347a8cda67861ea7ce9cb618dbbbe83","unresolved":false,"context_lines":[{"line_number":47,"context_line":"            \"provider:physical_network\": null,"},{"line_number":48,"context_line":"            \"provider:segmentation_id\": 599,"},{"line_number":49,"context_line":"            \"provider:network_type\": \"vxlan\"，"},{"line_number":50,"context_line":"            \"bottom_segment_provider\":\"storage\""},{"line_number":51,"context_line":"      }"},{"line_number":52,"context_line":"    }"},{"line_number":53,"context_line":""}],"source_content_type":"text/x-rst","patch_set":10,"id":"3f79a3b5_5f0fc02e","line":50,"updated":"2018-09-11 20:42:21.000000000","message":"In the","commit_id":"4ccff195ebf0c141e0922ec1a9ad880625358c1b"},{"author":{"_account_id":13995,"name":"Nate Johnston","email":"nate.johnston@redhat.com","username":"natejohnston"},"change_message_id":"b773d871e347a8cda67861ea7ce9cb618dbbbe83","unresolved":false,"context_lines":[{"line_number":68,"context_line":"              \u0027is_visible\u0027: True, \u0027default\u0027:None},"},{"line_number":69,"context_line":"      },"},{"line_number":70,"context_line":"    }"},{"line_number":71,"context_line":""},{"line_number":72,"context_line":""},{"line_number":73,"context_line":"References"},{"line_number":74,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":10,"id":"3f79a3b5_7f0cfc38","line":71,"updated":"2018-09-11 20:42:21.000000000","message":"I think it is necessary to at least provide an API endpoint so that the mapping of bottom segment provider to vlan to cluster can be retrieved for later reference.  Because of separation-of-concerns in an organization, the person issuing this command might not be able to inspect the ml2_conf.ini to see what the range of available values is.","commit_id":"4ccff195ebf0c141e0922ec1a9ad880625358c1b"},{"author":{"_account_id":28056,"name":"Liushy","email":"liuxie_11@163.com","username":"liuxie_11"},"change_message_id":"549e4c2798bd675e0aef8756b18fd9a55a82b535","unresolved":false,"context_lines":[{"line_number":68,"context_line":"              \u0027is_visible\u0027: True, \u0027default\u0027:None},"},{"line_number":69,"context_line":"      },"},{"line_number":70,"context_line":"    }"},{"line_number":71,"context_line":""},{"line_number":72,"context_line":""},{"line_number":73,"context_line":"References"},{"line_number":74,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":10,"id":"3f79a3b5_5049e68b","line":71,"in_reply_to":"3f79a3b5_7f0cfc38","updated":"2018-09-12 07:51:50.000000000","message":"yeah,i agree with you.","commit_id":"4ccff195ebf0c141e0922ec1a9ad880625358c1b"},{"author":{"_account_id":13995,"name":"Nate Johnston","email":"nate.johnston@redhat.com","username":"natejohnston"},"change_message_id":"972e468b464553e3f3bf5be8bae66f7ceb692766","unresolved":false,"context_lines":[{"line_number":19,"context_line":"At present, in many scenarios such as finance or government, customers use hierarchical port"},{"line_number":20,"context_line":"binding to break through the limitation of 4K vlans for network. In addition, for security"},{"line_number":21,"context_line":"and privacy needs, customers need different physical_network for underlying segments of the networks,"},{"line_number":22,"context_line":"such asbussiness00 for business network or storage01 for storage network."},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"The current neutron API for the network does not support specify the physical_network name of"},{"line_number":25,"context_line":"the underlying segment[1]. For this requirement, we need to extend the network API of neutron"}],"source_content_type":"text/x-rst","patch_set":13,"id":"3f79a3b5_3d5c217c","line":22,"updated":"2018-09-13 15:56:14.000000000","message":"s/asbusiness00/as business00/","commit_id":"fd9364dd488391eb49a26276bfa78037d710a26d"},{"author":{"_account_id":13995,"name":"Nate Johnston","email":"nate.johnston@redhat.com","username":"natejohnston"},"change_message_id":"972e468b464553e3f3bf5be8bae66f7ceb692766","unresolved":false,"context_lines":[{"line_number":30,"context_line":""},{"line_number":31,"context_line":"Introduce an attribute \"bottom_segment_physnet\" to network API."},{"line_number":32,"context_line":"The user specifies the prefix of physical_network name of the bottom segment by"},{"line_number":33,"context_line":"\"bottom_segment_physnet\" When creating a network. When top-level driver assign VLAN to"},{"line_number":34,"context_line":"bottom-level driver, it only need to dynamically assign VLANID to the mached physnetworks."},{"line_number":35,"context_line":""},{"line_number":36,"context_line":"::"}],"source_content_type":"text/x-rst","patch_set":13,"id":"3f79a3b5_dd52cd68","line":33,"updated":"2018-09-13 15:56:14.000000000","message":"s/When/when/","commit_id":"fd9364dd488391eb49a26276bfa78037d710a26d"},{"author":{"_account_id":13995,"name":"Nate Johnston","email":"nate.johnston@redhat.com","username":"natejohnston"},"change_message_id":"972e468b464553e3f3bf5be8bae66f7ceb692766","unresolved":false,"context_lines":[{"line_number":31,"context_line":"Introduce an attribute \"bottom_segment_physnet\" to network API."},{"line_number":32,"context_line":"The user specifies the prefix of physical_network name of the bottom segment by"},{"line_number":33,"context_line":"\"bottom_segment_physnet\" When creating a network. When top-level driver assign VLAN to"},{"line_number":34,"context_line":"bottom-level driver, it only need to dynamically assign VLANID to the mached physnetworks."},{"line_number":35,"context_line":""},{"line_number":36,"context_line":"::"},{"line_number":37,"context_line":""}],"source_content_type":"text/x-rst","patch_set":13,"id":"3f79a3b5_fd4d49c5","line":34,"updated":"2018-09-13 15:56:14.000000000","message":"s/mached physnetworks/physical_networks that match/","commit_id":"fd9364dd488391eb49a26276bfa78037d710a26d"},{"author":{"_account_id":27815,"name":"Huang Cheng","email":"hcn518@gmail.com","username":"pedh"},"change_message_id":"82b78eaee8ccf2fe78275f9fed28e7d23ccac6e8","unresolved":false,"context_lines":[{"line_number":47,"context_line":"           |            |                 |            |"},{"line_number":48,"context_line":"           | ToR Switch |                 | ToR Switch |"},{"line_number":49,"context_line":"           |            |                 |            |"},{"line_number":50,"context_line":"           +-+-+--+-+---+                 +---+-+--+-+-+"},{"line_number":51,"context_line":"       vlan  | |  | |   vlan          vlan    | |  | |   vlan"},{"line_number":52,"context_line":"    +--------+ |  | +------+          +-------+ |  | +-------+"},{"line_number":53,"context_line":"    |    +-----+  +---+    |          |    +----+  +-----+   |"},{"line_number":54,"context_line":"    |    |            |    |          |    |             |   |"},{"line_number":55,"context_line":"    |    |  cluster1  |    |          |    |   cluster2  |   |"},{"line_number":56,"context_line":"  +---------+      +---------+      +---------+      +---------+"},{"line_number":57,"context_line":"  |         |      |         |      |         |      |         |"},{"line_number":58,"context_line":"  |  Host0  |      |  Host1  |      |  Host2  |      |  Host3  |"},{"line_number":59,"context_line":"  |         |      |         |      |         |      |         |"},{"line_number":60,"context_line":"  +---------+      +---------+      +---------+      +---------+"},{"line_number":61,"context_line":""},{"line_number":62,"context_line":"A cluster is a set of hosts with the same physical networks."},{"line_number":63,"context_line":"For example:"}],"source_content_type":"text/x-rst","patch_set":13,"id":"3f79a3b5_ee3b5936","line":60,"range":{"start_line":50,"start_character":0,"end_line":60,"end_character":64},"updated":"2018-09-13 11:46:47.000000000","message":"The ascii picture is not comprehensible, you could check http://www.ascii-art.de/tools/ for some efficient ascii art tools.","commit_id":"fd9364dd488391eb49a26276bfa78037d710a26d"},{"author":{"_account_id":27815,"name":"Huang Cheng","email":"hcn518@gmail.com","username":"pedh"},"change_message_id":"82b78eaee8ccf2fe78275f9fed28e7d23ccac6e8","unresolved":false,"context_lines":[{"line_number":62,"context_line":"A cluster is a set of hosts with the same physical networks."},{"line_number":63,"context_line":"For example:"},{"line_number":64,"context_line":"ml2_conf.ini conf like these:"},{"line_number":65,"context_line":"cluster1: network_vlan_ranges\u003dstorage00:1:2000,busness00:2001:4094"},{"line_number":66,"context_line":"cluster2: network_vlan_ranges\u003dstorage01:2001:4094,busness01:1:2000"},{"line_number":67,"context_line":"What we consider is that every cluster has two physical networks which one for business and"},{"line_number":68,"context_line":"another for storage. Their business networks (or storage networks) have different physical"}],"source_content_type":"text/x-rst","patch_set":13,"id":"3f79a3b5_ae4561b6","line":65,"range":{"start_line":65,"start_character":47,"end_line":65,"end_character":56},"updated":"2018-09-13 11:46:47.000000000","message":"typo","commit_id":"fd9364dd488391eb49a26276bfa78037d710a26d"},{"author":{"_account_id":13995,"name":"Nate Johnston","email":"nate.johnston@redhat.com","username":"natejohnston"},"change_message_id":"972e468b464553e3f3bf5be8bae66f7ceb692766","unresolved":false,"context_lines":[{"line_number":63,"context_line":"For example:"},{"line_number":64,"context_line":"ml2_conf.ini conf like these:"},{"line_number":65,"context_line":"cluster1: network_vlan_ranges\u003dstorage00:1:2000,busness00:2001:4094"},{"line_number":66,"context_line":"cluster2: network_vlan_ranges\u003dstorage01:2001:4094,busness01:1:2000"},{"line_number":67,"context_line":"What we consider is that every cluster has two physical networks which one for business and"},{"line_number":68,"context_line":"another for storage. Their business networks (or storage networks) have different physical"},{"line_number":69,"context_line":"network names but the same prefix.For example, \"business00\" and \"business01\". They have the"}],"source_content_type":"text/x-rst","patch_set":13,"id":"3f79a3b5_9d4c55c9","line":66,"updated":"2018-09-13 15:56:14.000000000","message":"Please put these 2 lines in an RST literal block\n\nhttp://docutils.sourceforge.net/docs/ref/rst/restructuredtext.html#literal-blocks","commit_id":"fd9364dd488391eb49a26276bfa78037d710a26d"},{"author":{"_account_id":13995,"name":"Nate Johnston","email":"nate.johnston@redhat.com","username":"natejohnston"},"change_message_id":"972e468b464553e3f3bf5be8bae66f7ceb692766","unresolved":false,"context_lines":[{"line_number":65,"context_line":"cluster1: network_vlan_ranges\u003dstorage00:1:2000,busness00:2001:4094"},{"line_number":66,"context_line":"cluster2: network_vlan_ranges\u003dstorage01:2001:4094,busness01:1:2000"},{"line_number":67,"context_line":"What we consider is that every cluster has two physical networks which one for business and"},{"line_number":68,"context_line":"another for storage. Their business networks (or storage networks) have different physical"},{"line_number":69,"context_line":"network names but the same prefix.For example, \"business00\" and \"business01\". They have the"},{"line_number":70,"context_line":"same prefix is \"business\".The user specifies the prefix like \"business\" when create a vxlan network."},{"line_number":71,"context_line":"Top-level driver dynamically assign VLANID to the physical network that maches the prefix when bind_port"}],"source_content_type":"text/x-rst","patch_set":13,"id":"3f79a3b5_bd47d1a3","line":68,"updated":"2018-09-13 15:56:14.000000000","message":"There\u0027s no reason this could be limited to two types of network, right?  I want to make sure that I could have seven network types named for colors of the rainbow and it does not affect the details of how this will work.","commit_id":"fd9364dd488391eb49a26276bfa78037d710a26d"},{"author":{"_account_id":13995,"name":"Nate Johnston","email":"nate.johnston@redhat.com","username":"natejohnston"},"change_message_id":"972e468b464553e3f3bf5be8bae66f7ceb692766","unresolved":false,"context_lines":[{"line_number":66,"context_line":"cluster2: network_vlan_ranges\u003dstorage01:2001:4094,busness01:1:2000"},{"line_number":67,"context_line":"What we consider is that every cluster has two physical networks which one for business and"},{"line_number":68,"context_line":"another for storage. Their business networks (or storage networks) have different physical"},{"line_number":69,"context_line":"network names but the same prefix.For example, \"business00\" and \"business01\". They have the"},{"line_number":70,"context_line":"same prefix is \"business\".The user specifies the prefix like \"business\" when create a vxlan network."},{"line_number":71,"context_line":"Top-level driver dynamically assign VLANID to the physical network that maches the prefix when bind_port"},{"line_number":72,"context_line":"POST /v2.0/networks"}],"source_content_type":"text/x-rst","patch_set":13,"id":"3f79a3b5_5dc47d0e","line":69,"updated":"2018-09-13 15:56:14.000000000","message":"I repeat my objection to having Neutron decipher strings bt separating text and integers without some syntactic sugar like a proper delimiter character.","commit_id":"fd9364dd488391eb49a26276bfa78037d710a26d"},{"author":{"_account_id":27815,"name":"Huang Cheng","email":"hcn518@gmail.com","username":"pedh"},"change_message_id":"82b78eaee8ccf2fe78275f9fed28e7d23ccac6e8","unresolved":false,"context_lines":[{"line_number":66,"context_line":"cluster2: network_vlan_ranges\u003dstorage01:2001:4094,busness01:1:2000"},{"line_number":67,"context_line":"What we consider is that every cluster has two physical networks which one for business and"},{"line_number":68,"context_line":"another for storage. Their business networks (or storage networks) have different physical"},{"line_number":69,"context_line":"network names but the same prefix.For example, \"business00\" and \"business01\". They have the"},{"line_number":70,"context_line":"same prefix is \"business\".The user specifies the prefix like \"business\" when create a vxlan network."},{"line_number":71,"context_line":"Top-level driver dynamically assign VLANID to the physical network that maches the prefix when bind_port"},{"line_number":72,"context_line":"POST /v2.0/networks"}],"source_content_type":"text/x-rst","patch_set":13,"id":"3f79a3b5_ce40dda5","line":69,"range":{"start_line":69,"start_character":27,"end_line":69,"end_character":33},"updated":"2018-09-13 11:46:47.000000000","message":"Using \"prefix\" is not a good solution for general purpose.","commit_id":"fd9364dd488391eb49a26276bfa78037d710a26d"}]}