)]}'
{"quantum/plugins/cisco/models/virt_phy_sw_v2.py":[{"author":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"change_message_id":"97836e95385c45e9f9dc72ea5d946a930ce89e6e","unresolved":false,"context_lines":[{"line_number":174,"context_line":"        for v_id in vlan_ids:"},{"line_number":175,"context_line":"            if int(v_id) \u003e 0:"},{"line_number":176,"context_line":"                vlanids \u003d str(v_id) + \u0027,\u0027 + vlanids"},{"line_number":177,"context_line":"        return vlanids.strip(\u0027,\u0027)"},{"line_number":178,"context_line":""},{"line_number":179,"context_line":"    def _validate_vlan_id(self, vlan_id):"},{"line_number":180,"context_line":"        if vlan_id and int(vlan_id) \u003e 1:"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAALH%2F%2FVmc%3D","line":177,"updated":"2012-09-20 11:52:20.000000000","message":"Can you clarify why you add the ID and \u0027,\u0027 and then strip?","commit_id":"abc129701d0271e7838285403a662ce872479c63"},{"author":{"_account_id":490,"name":"Sumit Naiksatam","email":"sumitnaiksatam@gmail.com","username":"snaiksat"},"unresolved":false,"context_lines":[{"line_number":174,"context_line":"        for v_id in vlan_ids:"},{"line_number":175,"context_line":"            if int(v_id) \u003e 0:"},{"line_number":176,"context_line":"                vlanids \u003d str(v_id) + \u0027,\u0027 + vlanids"},{"line_number":177,"context_line":"        return vlanids.strip(\u0027,\u0027)"},{"line_number":178,"context_line":""},{"line_number":179,"context_line":"    def _validate_vlan_id(self, vlan_id):"},{"line_number":180,"context_line":"        if vlan_id and int(vlan_id) \u003e 1:"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAALH%2F%2FVXc%3D","line":177,"in_reply_to":"AAAALH%2F%2FVmc%3D","updated":"2012-09-20 16:27:38.000000000","message":"In this case, we are trying to remove a trailing \u0027,\u0027 character since the nexus switch does not accept that. The above logic was used before (in the create_network method) to generate this string, hence I kept it the way it is to avoid additional churn at this stage (just moved it to a new method so that other methods can also reuse it).","commit_id":"abc129701d0271e7838285403a662ce872479c63"}]}