)]}'
{"network/config/single-nic-linux-bridge-vlans/role.role.j2.yaml":[{"author":{"_account_id":21909,"name":"Bob Fournier","email":"bfournie@redhat.com","username":"bfournie"},"change_message_id":"29efffcb945893407e4080ba9d7fe848b1fc4540","unresolved":false,"context_lines":[{"line_number":58,"context_line":"            $network_config:"},{"line_number":59,"context_line":"              network_config:"},{"line_number":60,"context_line":"              - type: linux_bridge"},{"line_number":61,"context_line":"{%- if role.name.startswith(\u0027CephStorage\u0027) %}"},{"line_number":62,"context_line":"                name: br-storage"},{"line_number":63,"context_line":"{%- else %}"},{"line_number":64,"context_line":"                name: bridge_name"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"9f6a8fd7_34830059","line":61,"range":{"start_line":61,"start_character":0,"end_line":61,"end_character":45},"updated":"2018-04-29 18:39:16.000000000","message":"In OSP-10 for single-nic-vlans, both cinder-storage.yaml and swift-storage.yaml also use br-storage.  Should those roles be checked here too?","commit_id":"7c6819e0d7b685120617f3247976a4531d076e5f"},{"author":{"_account_id":21909,"name":"Bob Fournier","email":"bfournie@redhat.com","username":"bfournie"},"change_message_id":"29efffcb945893407e4080ba9d7fe848b1fc4540","unresolved":false,"context_lines":[{"line_number":90,"context_line":"              - type: vlan"},{"line_number":91,"context_line":"                vlan_id:"},{"line_number":92,"context_line":"                  get_param: {{network.name}}NetworkVlanID"},{"line_number":93,"context_line":"                device: bridge_name"},{"line_number":94,"context_line":"                addresses:"},{"line_number":95,"context_line":"                - ip_netmask:"},{"line_number":96,"context_line":"                    get_param: {{network.name}}IpSubnet"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"9f6a8fd7_94b44c37","line":93,"range":{"start_line":93,"start_character":24,"end_line":93,"end_character":35},"updated":"2018-04-29 18:39:16.000000000","message":"I think this would also be bridge-storage for storage roles, no?","commit_id":"7c6819e0d7b685120617f3247976a4531d076e5f"}],"network/config/single-nic-vlans/role.role.j2.yaml":[{"author":{"_account_id":21909,"name":"Bob Fournier","email":"bfournie@redhat.com","username":"bfournie"},"change_message_id":"29efffcb945893407e4080ba9d7fe848b1fc4540","unresolved":false,"context_lines":[{"line_number":58,"context_line":"            $network_config:"},{"line_number":59,"context_line":"              network_config:"},{"line_number":60,"context_line":"              - type: ovs_bridge"},{"line_number":61,"context_line":"{%- if role.name.startswith(\u0027CephStorage\u0027) %}"},{"line_number":62,"context_line":"                name: br-storage"},{"line_number":63,"context_line":"{%- else %}"},{"line_number":64,"context_line":"                name: bridge_name"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"9f6a8fd7_d4aac4d0","line":61,"range":{"start_line":61,"start_character":0,"end_line":61,"end_character":45},"updated":"2018-04-29 18:39:16.000000000","message":"In OSP-10 for single-nic-vlans, both cinder-storage.yaml and swift-storage.yaml also use br-storage.  Should those roles be checked here too?","commit_id":"7c6819e0d7b685120617f3247976a4531d076e5f"}]}