)]}'
{"neutron_taas/services/taas/drivers/linux/ovs_taas.py":[{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"f283f0a36f2e67ca32ee5c1b999a8b990c54ced8","unresolved":false,"context_lines":[{"line_number":219,"context_line":"                             priority\u003d25,"},{"line_number":220,"context_line":"                             in_port\u003dpatch_int_tap_id,"},{"line_number":221,"context_line":"                             dl_vlan\u003dtaas_id,"},{"line_number":222,"context_line":"                             actions\u003d\"pop_vlan,mod_dl_dst:%s,output:%s\" %"},{"line_number":223,"context_line":"                             (port_mac, str(ovs_port_id)))"},{"line_number":224,"context_line":""},{"line_number":225,"context_line":"        # Add flow(s) in br-tap"}],"source_content_type":"text/x-python","patch_set":2,"id":"df33271e_e80c95a6","line":222,"range":{"start_line":222,"start_character":38,"end_line":222,"end_character":46},"updated":"2020-03-30 14:39:31.000000000","message":"Why do you remove the vlan header? If I understand well that is not related to your problem","commit_id":"5870139f7b65513bb441fe3a8fc5c96054cdfa37"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"65aba2b9a11d2c6e9501e56cc88010d02867ec59","unresolved":false,"context_lines":[{"line_number":219,"context_line":"                             priority\u003d25,"},{"line_number":220,"context_line":"                             in_port\u003dpatch_int_tap_id,"},{"line_number":221,"context_line":"                             dl_vlan\u003dtaas_id,"},{"line_number":222,"context_line":"                             actions\u003d\"pop_vlan,mod_dl_dst:%s,output:%s\" %"},{"line_number":223,"context_line":"                             (port_mac, str(ovs_port_id)))"},{"line_number":224,"context_line":""},{"line_number":225,"context_line":"        # Add flow(s) in br-tap"}],"source_content_type":"text/x-python","patch_set":2,"id":"df33271e_7b81ff4a","line":222,"range":{"start_line":222,"start_character":38,"end_line":222,"end_character":46},"in_reply_to":"df33271e_1211fa04","updated":"2020-04-03 16:02:43.000000000","message":"So as I described in the bug report:\n* tcpdump worked for me without this change, I used vxlan network, but if I understand well that is not important.\n\n* Though I think the removal of the host only vlan is a valid need.\n\n* I think the destination mac change is against the intention of traffic mirroring. As I see the original header information must be mirrored and as it works (at least in my environment) I don\u0027t see the reason why to change.","commit_id":"5870139f7b65513bb441fe3a8fc5c96054cdfa37"},{"author":{"_account_id":15907,"name":"Wei Hui","email":"huiweics@gmail.com","username":"huiweics"},"change_message_id":"5a14f8c9392efe0b4d20be6d7d7071cd77613d1c","unresolved":false,"context_lines":[{"line_number":219,"context_line":"                             priority\u003d25,"},{"line_number":220,"context_line":"                             in_port\u003dpatch_int_tap_id,"},{"line_number":221,"context_line":"                             dl_vlan\u003dtaas_id,"},{"line_number":222,"context_line":"                             actions\u003d\"pop_vlan,mod_dl_dst:%s,output:%s\" %"},{"line_number":223,"context_line":"                             (port_mac, str(ovs_port_id)))"},{"line_number":224,"context_line":""},{"line_number":225,"context_line":"        # Add flow(s) in br-tap"}],"source_content_type":"text/x-python","patch_set":2,"id":"df33271e_1211fa04","line":222,"range":{"start_line":222,"start_character":38,"end_line":222,"end_character":46},"in_reply_to":"df33271e_e80c95a6","updated":"2020-04-01 09:32:21.000000000","message":"http://www.openvswitch.org/support/dist-docs/ovs-vswitchd.conf.db.5.txt\nThe VLAN configuration in this column affects Open vSwitch only when it is doing ``normal switching.’’ It does not affect flows set  up  by  an OpenFlow controller, outside of the OpenFlow ``normal action.’’\n\nso output action does not strip vlan header, this vlan is local to br-int，tap service vm receives mirrored packet with this vlan, useless.","commit_id":"5870139f7b65513bb441fe3a8fc5c96054cdfa37"}]}