)]}'
{"nova/network/quantumv2/api.py":[{"author":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"change_message_id":"1cbd175061e46837a698a273173a82fcd87d8a1e","unresolved":false,"context_lines":[{"line_number":570,"context_line":"                subnet[\u0027ips\u0027] \u003d [fixed_ip for fixed_ip in network_IPs"},{"line_number":571,"context_line":"                                 if fixed_ip.is_in_subnet(subnet)]"},{"line_number":572,"context_line":""},{"line_number":573,"context_line":"            bridge \u003d \u0027\u0027"},{"line_number":574,"context_line":"            vif_type \u003d port.get(\u0027binding:vif_type\u0027)"},{"line_number":575,"context_line":"            # TODO(berrange) Quantum should pass the bridge name"},{"line_number":576,"context_line":"            # in another binding metadata field"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAM3%2F%2Fvsw%3D","line":573,"updated":"2013-01-07 21:48:49.000000000","message":"why do you need this line?","commit_id":"d9f7206e57270c84ae2965c81451c6f2473bef2e"},{"author":{"_account_id":1779,"name":"Daniel Berrange","email":"berrange@redhat.com","username":"berrange"},"change_message_id":"0080519b950ee39252f15613e419c6ea4a5cf199","unresolved":false,"context_lines":[{"line_number":570,"context_line":"                subnet[\u0027ips\u0027] \u003d [fixed_ip for fixed_ip in network_IPs"},{"line_number":571,"context_line":"                                 if fixed_ip.is_in_subnet(subnet)]"},{"line_number":572,"context_line":""},{"line_number":573,"context_line":"            bridge \u003d \u0027\u0027"},{"line_number":574,"context_line":"            vif_type \u003d port.get(\u0027binding:vif_type\u0027)"},{"line_number":575,"context_line":"            # TODO(berrange) Quantum should pass the bridge name"},{"line_number":576,"context_line":"            # in another binding metadata field"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAM3%2F%2FvsQ%3D","line":573,"in_reply_to":"AAAAM3%2F%2Fvsw%3D","updated":"2013-01-07 21:52:47.000000000","message":"This was done to be the same as current behaviour which uses \u0027\u0027 instead of None. Looking at the rest of the code I guess we can let it be set to None for other types of VIF without trouble.","commit_id":"d9f7206e57270c84ae2965c81451c6f2473bef2e"}]}