)]}'
{"multinode_diagram.txt":[{"author":{"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"},"change_message_id":"72190fb0f784b990def761d81539f73fe96e774e","unresolved":false,"context_lines":[{"line_number":1,"context_line":"The basic requirement we have for running multinode openstack tests is that"},{"line_number":2,"context_line":"tempest must be able to ssh and ping the nested VMs booted by openstack and"},{"line_number":3,"context_line":"these nested VMs need to be able to talk to each other. This is due to how"},{"line_number":4,"context_line":"the tests that are run by tempest."},{"line_number":5,"context_line":""},{"line_number":6,"context_line":"We run devstack-gate on multiple public clouds. In order to meet the above"},{"line_number":7,"context_line":"requirement we need some control over l2 and l3 networking in the test envs,"}],"source_content_type":"text/plain","patch_set":1,"id":"7a2fa921_54fe2f6c","line":4,"updated":"2015-10-06 17:56:37.000000000","message":"Either need to drop \"that\" from this line or \"how\" from the previous line.","commit_id":"dcc582310830c76b77d136e1f75002c09ae4a512"},{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"d6ac814b80edeee947911c363bdb72735e6a5d03","unresolved":false,"context_lines":[{"line_number":73,"context_line":"IP traffic is backhauled over br-int to the primary node where br-ex"},{"line_number":74,"context_line":"exists. br-ex is created on the primary node as it is on the single node"},{"line_number":75,"context_line":"tests and all tempest to floating IP and nested VM to nested VM communication"},{"line_number":76,"context_line":"happens here."},{"line_number":77,"context_line":""},{"line_number":78,"context_line":"           Subnode1                      Primary Node                   Subnode2"},{"line_number":79,"context_line":"+--------------------------+   +--------------------------+   +--------------------------+"}],"source_content_type":"text/plain","patch_set":1,"id":"7a2fa921_de902098","line":76,"updated":"2015-10-06 17:17:39.000000000","message":"This is why I have argued in the past that the non DVR neutron multinode test isn\u0027t that interesting and we should focus on making DVR work instead. We don\u0027t really test anything different here.","commit_id":"dcc582310830c76b77d136e1f75002c09ae4a512"},{"author":{"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"},"change_message_id":"72190fb0f784b990def761d81539f73fe96e774e","unresolved":false,"context_lines":[{"line_number":100,"context_line":"neutron do not configure br-ex except for in the trivial detached from"},{"line_number":101,"context_line":"everything case described earlier. This means we have to configure br-ex"},{"line_number":102,"context_line":"ourselves and the simplest way to do that is to just make br-ex the overlay"},{"line_number":103,"context_line":"itself. Doing this allows neutron to work properly with nesting VMs talking"},{"line_number":104,"context_line":"to nested VMs and it also allows the test nodes to talk to VMs over br-ex as"},{"line_number":105,"context_line":"well."},{"line_number":106,"context_line":""}],"source_content_type":"text/plain","patch_set":1,"id":"7a2fa921_74032b74","line":103,"updated":"2015-10-06 17:56:37.000000000","message":"Did you mean \"nested\" here?","commit_id":"dcc582310830c76b77d136e1f75002c09ae4a512"},{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"130fdaed8b76b573be9a572f24acbb3905385151","unresolved":false,"context_lines":[{"line_number":108,"context_line":"           Subnode1                      Primary Node                   Subnode2"},{"line_number":109,"context_line":"+--------------------------+   +--------------------------+   +--------------------------+"},{"line_number":110,"context_line":"|                +------+  |   |                +------+  |   |                +------+  |"},{"line_number":111,"context_line":"|                |br-tun|--------tunnel---------|br-tun|--------tunnel---------|br-tun|  |"},{"line_number":112,"context_line":"|                +------+  |   |                +------+  |   |                +------+  |"},{"line_number":113,"context_line":"|                |br-int|  |   |                |br-int|  |   |                |br-int|  |"},{"line_number":114,"context_line":"|                +------+  |   |                +------+  |   |                +------+  |"}],"source_content_type":"text/plain","patch_set":3,"id":"7a2fa921_34f61390","line":111,"updated":"2015-10-06 17:56:44.000000000","message":"This is just \"tunnel\" because the underlying tunnel tech is controlled by neutron and could be gre or vxlan or whatever the next thing is.","commit_id":"4548f15460f7e380a8dfd1b39534f567ed989ee7"}]}