)]}'
{"id":"openstack%2Fneutron~613239","triplet_id":"openstack%2Fneutron~master~Ia22ea54e16cfb4ae423836a06816ec43d9135df6","project":"openstack/neutron","branch":"master","topic":"bug/1799599","hashtags":[],"change_id":"Ia22ea54e16cfb4ae423836a06816ec43d9135df6","subject":"Don\u0027t filter on the TCI bit for flat networks","status":"ABANDONED","created":"2018-10-25 08:29:49.000000000","updated":"2018-11-07 11:27:28.000000000","total_comment_count":2,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"82ca3e489fa5355dbf33a6288400d59e69bccc26","_number":613239,"virtual_id_number":613239,"owner":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"actions":{},"labels":{"Verified":{"recommended":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"value":1,"date":"2018-10-26 16:40:58.000000000","permitted_voting_range":{"min":-2,"max":2},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"date":"2018-10-26 12:41:38.000000000","_account_id":26622,"name":"Cloudbase Neutron Hyper-V CI","email":"neutron_hyperv_ci@cloudbasesolutions.com","username":"neutron_hyperv_ci"},{"_account_id":14270,"name":"Dimitri Savineau","email":"dsavinea@redhat.com","username":"dsavineau"},{"date":"2018-10-26 12:05:09.000000000","_account_id":16376,"name":"Intel NFV CI","email":"openstack-nfv-ci@intel.com","username":"intel-nfv-ci","tags":["SERVICE_USER"]},{"date":"2018-10-31 18:06:50.000000000","_account_id":24042,"name":"Andreas Karis","email":"akaris@redhat.com","username":"akaris"},{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},{"_account_id":13995,"name":"Nate Johnston","email":"nate.johnston@redhat.com","username":"natejohnston"},{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},{"value":1,"date":"2018-10-26 01:59:50.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":9732,"name":"Mellanox CI","email":"mlnx-openstack-ci@dev.mellanox.co.il","username":"mellanox","tags":["SERVICE_USER"]},{"date":"2018-10-26 12:13:57.000000000","_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},{"date":"2018-10-25 21:38:35.000000000","_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]}],"values":{"-2":"Fails","-1":"Doesn\u0027t seem to work"," 0":"No score","+1":"Works for me","+2":"Verified"},"description":"","value":1,"default_value":0,"optional":true},"Code-Review":{"approved":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"all":[{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":26622,"name":"Cloudbase Neutron Hyper-V CI","email":"neutron_hyperv_ci@cloudbasesolutions.com","username":"neutron_hyperv_ci"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":14270,"name":"Dimitri Savineau","email":"dsavinea@redhat.com","username":"dsavineau"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":16376,"name":"Intel NFV CI","email":"openstack-nfv-ci@intel.com","username":"intel-nfv-ci","tags":["SERVICE_USER"]},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":24042,"name":"Andreas Karis","email":"akaris@redhat.com","username":"akaris"},{"value":2,"date":"2018-10-25 18:32:06.000000000","permitted_voting_range":{"min":-2,"max":2},"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},{"value":1,"date":"2018-10-25 15:57:28.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":13995,"name":"Nate Johnston","email":"nate.johnston@redhat.com","username":"natejohnston"},{"value":-1,"date":"2018-10-26 11:25:01.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":9732,"name":"Mellanox CI","email":"mlnx-openstack-ci@dev.mellanox.co.il","username":"mellanox","tags":["SERVICE_USER"]},{"value":0,"permitted_voting_range":{"min":-2,"max":2},"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]}],"values":{"-2":"Do not merge","-1":"This patch needs further work before it can be merged"," 0":"No score","+1":"Looks good to me, but someone else must approve","+2":"Looks good to me (core reviewer)"},"description":"","default_value":0,"optional":true},"Workflow":{"all":[{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"_account_id":26622,"name":"Cloudbase Neutron Hyper-V CI","email":"neutron_hyperv_ci@cloudbasesolutions.com","username":"neutron_hyperv_ci"},{"_account_id":14270,"name":"Dimitri Savineau","email":"dsavinea@redhat.com","username":"dsavineau"},{"_account_id":16376,"name":"Intel NFV CI","email":"openstack-nfv-ci@intel.com","username":"intel-nfv-ci","tags":["SERVICE_USER"]},{"_account_id":24042,"name":"Andreas Karis","email":"akaris@redhat.com","username":"akaris"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},{"_account_id":13995,"name":"Nate Johnston","email":"nate.johnston@redhat.com","username":"natejohnston"},{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},{"_account_id":9732,"name":"Mellanox CI","email":"mlnx-openstack-ci@dev.mellanox.co.il","username":"mellanox","tags":["SERVICE_USER"]},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},{"_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true},"Review-Priority":{"all":[{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"_account_id":26622,"name":"Cloudbase Neutron Hyper-V CI","email":"neutron_hyperv_ci@cloudbasesolutions.com","username":"neutron_hyperv_ci"},{"_account_id":14270,"name":"Dimitri Savineau","email":"dsavinea@redhat.com","username":"dsavineau"},{"_account_id":16376,"name":"Intel NFV CI","email":"openstack-nfv-ci@intel.com","username":"intel-nfv-ci","tags":["SERVICE_USER"]},{"_account_id":24042,"name":"Andreas Karis","email":"akaris@redhat.com","username":"akaris"},{"value":0,"permitted_voting_range":{"min":-1,"max":2},"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},{"_account_id":13995,"name":"Nate Johnston","email":"nate.johnston@redhat.com","username":"natejohnston"},{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},{"_account_id":9732,"name":"Mellanox CI","email":"mlnx-openstack-ci@dev.mellanox.co.il","username":"mellanox","tags":["SERVICE_USER"]},{"value":0,"permitted_voting_range":{"min":-1,"max":2},"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},{"_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]}],"values":{"-1":"Branch Freeze"," 0":"No Priority","+1":"Important Change","+2":"Gate Blocker Fix / Urgent Change"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},{"_account_id":9732,"name":"Mellanox CI","email":"mlnx-openstack-ci@dev.mellanox.co.il","username":"mellanox","tags":["SERVICE_USER"]},{"_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},{"_account_id":13995,"name":"Nate Johnston","email":"nate.johnston@redhat.com","username":"natejohnston"},{"_account_id":14270,"name":"Dimitri Savineau","email":"dsavinea@redhat.com","username":"dsavineau"},{"_account_id":16376,"name":"Intel NFV CI","email":"openstack-nfv-ci@intel.com","username":"intel-nfv-ci","tags":["SERVICE_USER"]},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"_account_id":24042,"name":"Andreas Karis","email":"akaris@redhat.com","username":"akaris"},{"_account_id":26622,"name":"Cloudbase Neutron Hyper-V CI","email":"neutron_hyperv_ci@cloudbasesolutions.com","username":"neutron_hyperv_ci"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2018-10-25 14:46:51.000000000","updated_by":{"_account_id":14270,"name":"Dimitri Savineau","email":"dsavinea@redhat.com","username":"dsavineau"},"reviewer":{"_account_id":14270,"name":"Dimitri Savineau","email":"dsavinea@redhat.com","username":"dsavineau"},"state":"REVIEWER"},{"updated":"2018-10-25 15:57:28.000000000","updated_by":{"_account_id":13995,"name":"Nate Johnston","email":"nate.johnston@redhat.com","username":"natejohnston"},"reviewer":{"_account_id":13995,"name":"Nate Johnston","email":"nate.johnston@redhat.com","username":"natejohnston"},"state":"REVIEWER"},{"updated":"2018-10-25 18:32:06.000000000","updated_by":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"reviewer":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"state":"REVIEWER"},{"updated":"2018-10-25 21:38:35.000000000","updated_by":{"_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},"reviewer":{"_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},"state":"REVIEWER"},{"updated":"2018-10-26 01:59:50.000000000","updated_by":{"_account_id":9732,"name":"Mellanox CI","email":"mlnx-openstack-ci@dev.mellanox.co.il","username":"mellanox","tags":["SERVICE_USER"]},"reviewer":{"_account_id":9732,"name":"Mellanox CI","email":"mlnx-openstack-ci@dev.mellanox.co.il","username":"mellanox","tags":["SERVICE_USER"]},"state":"REVIEWER"},{"updated":"2018-10-26 11:25:01.000000000","updated_by":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"reviewer":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"state":"REVIEWER"},{"updated":"2018-10-26 12:05:09.000000000","updated_by":{"_account_id":16376,"name":"Intel NFV CI","email":"openstack-nfv-ci@intel.com","username":"intel-nfv-ci","tags":["SERVICE_USER"]},"reviewer":{"_account_id":16376,"name":"Intel NFV CI","email":"openstack-nfv-ci@intel.com","username":"intel-nfv-ci","tags":["SERVICE_USER"]},"state":"REVIEWER"},{"updated":"2018-10-26 12:41:38.000000000","updated_by":{"_account_id":26622,"name":"Cloudbase Neutron Hyper-V CI","email":"neutron_hyperv_ci@cloudbasesolutions.com","username":"neutron_hyperv_ci"},"reviewer":{"_account_id":26622,"name":"Cloudbase Neutron Hyper-V CI","email":"neutron_hyperv_ci@cloudbasesolutions.com","username":"neutron_hyperv_ci"},"state":"REVIEWER"},{"updated":"2018-10-26 16:40:58.000000000","updated_by":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"reviewer":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"state":"REVIEWER"},{"updated":"2018-10-31 18:06:50.000000000","updated_by":{"_account_id":24042,"name":"Andreas Karis","email":"akaris@redhat.com","username":"akaris"},"reviewer":{"_account_id":24042,"name":"Andreas Karis","email":"akaris@redhat.com","username":"akaris"},"state":"REVIEWER"}],"messages":[{"id":"e377b85eb2884887142767466d781472e596a38e","author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"date":"2018-10-25 08:29:49.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"107bdbd6f3b4e9d0a8a36e1537ca3bf20123c534","author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"date":"2018-10-25 08:34:30.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"aeba1b80994fec4cf68a5eb8b1be8565fd14a396","author":{"_account_id":26622,"name":"Cloudbase Neutron Hyper-V CI","email":"neutron_hyperv_ci@cloudbasesolutions.com","username":"neutron_hyperv_ci"},"date":"2018-10-25 10:21:08.000000000","message":"Patch Set 2:\n\nBuild succeeded.\n\n- neutron http://cloudbase-ci.com/neutron/613239/2 : SUCCESS in 1h 46m 12s","accounts_in_message":[],"_revision_number":2},{"id":"0b6048db38420e392c5c1920ec02115fd5dd7bf4","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2018-10-25 11:03:32.000000000","message":"Patch Set 2: Verified-1\n\nBuild failed (check pipeline).  For information on how to proceed, see\nhttp://docs.openstack.org/infra/manual/developers.html#automated-testing\n\n\n- neutron-tempest-plugin-api http://logs.openstack.org/39/613239/2/check/neutron-tempest-plugin-api/9a7b58c/ : SUCCESS in 40m 30s\n- neutron-tempest-plugin-designate-scenario http://logs.openstack.org/39/613239/2/check/neutron-tempest-plugin-designate-scenario/c1d89ed/ : SUCCESS in 31m 08s\n- neutron-tempest-plugin-dvr-multinode-scenario http://logs.openstack.org/39/613239/2/check/neutron-tempest-plugin-dvr-multinode-scenario/9b8308d/ : FAILURE in 1h 46m 50s (non-voting)\n- neutron-tempest-plugin-scenario-linuxbridge http://logs.openstack.org/39/613239/2/check/neutron-tempest-plugin-scenario-linuxbridge/3797929/ : SUCCESS in 1h 10m 39s\n- build-openstack-sphinx-docs http://logs.openstack.org/39/613239/2/check/build-openstack-sphinx-docs/0290589/html/ : SUCCESS in 4m 57s\n- openstack-tox-cover http://logs.openstack.org/39/613239/2/check/openstack-tox-cover/e2a1494/cover/ : SUCCESS in 33m 09s\n- openstack-tox-lower-constraints http://logs.openstack.org/39/613239/2/check/openstack-tox-lower-constraints/f3b07f5/ : SUCCESS in 35m 27s\n- openstack-tox-pep8 http://logs.openstack.org/39/613239/2/check/openstack-tox-pep8/1d7092b/ : SUCCESS in 13m 35s\n- openstack-tox-py27 http://logs.openstack.org/39/613239/2/check/openstack-tox-py27/d0cdeef/ : SUCCESS in 21m 34s\n- openstack-tox-py35 http://logs.openstack.org/39/613239/2/check/openstack-tox-py35/31cc627/ : SUCCESS in 23m 06s\n- openstack-tox-py36 http://logs.openstack.org/39/613239/2/check/openstack-tox-py36/75da52c/ : SUCCESS in 23m 26s\n- openstack-tox-docs http://logs.openstack.org/39/613239/2/check/openstack-tox-docs/88766c9/html/ : SUCCESS in 4m 45s\n- tempest-full http://logs.openstack.org/39/613239/2/check/tempest-full/389176e/ : SUCCESS in 1h 37m 33s\n- neutron-grenade http://logs.openstack.org/39/613239/2/check/neutron-grenade/bed5b06/ : SUCCESS in 59m 33s\n- tempest-full-py3 http://logs.openstack.org/39/613239/2/check/tempest-full-py3/e045233/ : SUCCESS in 1h 19m 28s\n- neutron-functional http://logs.openstack.org/39/613239/2/check/neutron-functional/c4b1185/ : FAILURE in 34m 32s\n- neutron-fullstack http://logs.openstack.org/39/613239/2/check/neutron-fullstack/13ed5c4/ : FAILURE in 1h 05m 54s\n- neutron-rally-task http://logs.openstack.org/39/613239/2/check/neutron-rally-task/ccec324/ : SUCCESS in 1h 32m 43s\n- neutron-tempest-dvr http://logs.openstack.org/39/613239/2/check/neutron-tempest-dvr/0bbd853/ : SUCCESS in 1h 42m 05s\n- neutron-tempest-linuxbridge http://logs.openstack.org/39/613239/2/check/neutron-tempest-linuxbridge/303c6ae/ : SUCCESS in 1h 43m 42s\n- neutron-grenade-multinode http://logs.openstack.org/39/613239/2/check/neutron-grenade-multinode/6b6db7f/ : SUCCESS in 1h 09m 03s\n- neutron-grenade-dvr-multinode http://logs.openstack.org/39/613239/2/check/neutron-grenade-dvr-multinode/0935576/ : SUCCESS in 59m 44s\n- tempest-multinode-full http://logs.openstack.org/39/613239/2/check/tempest-multinode-full/1bdcbd4/ : SUCCESS in 1h 31m 40s (non-voting)\n- neutron-tempest-dvr-ha-multinode-full http://logs.openstack.org/39/613239/2/check/neutron-tempest-dvr-ha-multinode-full/729ad35/ : SUCCESS in 1h 46m 34s (non-voting)\n- neutron-tempest-iptables_hybrid http://logs.openstack.org/39/613239/2/check/neutron-tempest-iptables_hybrid/9599b68/ : SUCCESS in 1h 43m 55s\n- neutron-fullstack-python36 http://logs.openstack.org/39/613239/2/check/neutron-fullstack-python36/002b7da/ : FAILURE in 1h 02m 31s\n- ironic-tempest-dsvm-ipa-wholedisk-bios-agent_ipmitool-tinyipa http://logs.openstack.org/39/613239/2/check/ironic-tempest-dsvm-ipa-wholedisk-bios-agent_ipmitool-tinyipa/5149928/ : SUCCESS in 48m 28s (non-voting)\n- tempest-slow http://logs.openstack.org/39/613239/2/check/tempest-slow/6a76c15/ : SUCCESS in 2h 00m 14s","accounts_in_message":[],"_revision_number":2},{"id":"a9592bd002b1e684eed6a1e4b9ac1dc97686df83","author":{"_account_id":9732,"name":"Mellanox CI","email":"mlnx-openstack-ci@dev.mellanox.co.il","username":"mellanox","tags":["SERVICE_USER"]},"date":"2018-10-25 11:13:38.000000000","message":"Patch Set 2: Verified+1\n\nBuild succeeded.\n\n- Neutron-ML2-Sriov http://13.74.249.42/39/613239/2/check-mlnx/Neutron-ML2-Sriov/5c66afc : SUCCESS in 1h 13m 48s\n- Neutron-MACVTAP-ML2-Sriov http://13.74.249.42/39/613239/2/check-mlnx/Neutron-MACVTAP-ML2-Sriov/585bf32 : SUCCESS in 51m 35s\n\nTo re-run the job post \u0027recheck neutron-mlnx\u0027 comment. For more information visit https://wiki.openstack.org/wiki/ThirdPartySystems/Mellanox_CI","accounts_in_message":[],"_revision_number":2},{"id":"85fbcab744dae977f214652fb58b076fc3359c7b","author":{"_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},"date":"2018-10-25 12:48:09.000000000","message":"Patch Set 2:\n\nBuild failed.  To recheck use \u0027xenserver: recheck\u0027.  For 3rd party ci contact info: https://wiki.openstack.org/wiki/ThirdPartySystems\n\n- dsvm-tempest-neutron-network http://dd6b71949550285df7dc-dda4e480e005aaa13ec303551d2d8155.r49.cf1.rackcdn.com/39/613239/2/check/dsvm-tempest-neutron-network/f5e8d3d : FAILURE in 4h 02m 22s","accounts_in_message":[],"_revision_number":2},{"id":"6f9c645eb3a425066ce063773459f52d9738251b","author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"date":"2018-10-25 13:12:53.000000000","message":"Uploaded patch set 3.","accounts_in_message":[],"_revision_number":3},{"id":"f07f3bb7f9f881de24f0e7ddca6c7a5062c9a466","author":{"_account_id":14270,"name":"Dimitri Savineau","email":"dsavinea@redhat.com","username":"dsavineau"},"date":"2018-10-25 14:46:51.000000000","message":"Patch Set 3:\n\nsed -e \u0027s/DCI/TCI/g\u0027 in the commit message ?","accounts_in_message":[],"_revision_number":3},{"id":"d8f927c3b61195d47f2e8caaf7f2b316d16b6534","author":{"_account_id":26622,"name":"Cloudbase Neutron Hyper-V CI","email":"neutron_hyperv_ci@cloudbasesolutions.com","username":"neutron_hyperv_ci"},"date":"2018-10-25 14:53:52.000000000","message":"Patch Set 3:\n\nBuild succeeded.\n\n- neutron http://cloudbase-ci.com/neutron/613239/3 : SUCCESS in 1h 40m 33s","accounts_in_message":[],"_revision_number":3},{"id":"edff74c52b89d180b9c8f7c757471638f8658fac","author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"date":"2018-10-25 15:01:37.000000000","message":"Patch Set 4: Commit message was updated.","accounts_in_message":[],"_revision_number":4},{"id":"51d9e495ebc924484b5d55e1c4437917af72b95a","author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"date":"2018-10-25 15:01:47.000000000","message":"Patch Set 4:\n\n\u003e sed -e \u0027s/DCI/TCI/g\u0027 in the commit message ?\n\nDone","accounts_in_message":[],"_revision_number":4},{"id":"702b84352920cdd2967c019f0ca1f11bc4a5d35d","author":{"_account_id":13995,"name":"Nate Johnston","email":"nate.johnston@redhat.com","username":"natejohnston"},"date":"2018-10-25 15:57:28.000000000","message":"Patch Set 4: Code-Review+1","accounts_in_message":[],"_revision_number":4},{"id":"5a9b8ad7d9102312b9a7e4833e7e3b34f61963bc","author":{"_account_id":26622,"name":"Cloudbase Neutron Hyper-V CI","email":"neutron_hyperv_ci@cloudbasesolutions.com","username":"neutron_hyperv_ci"},"date":"2018-10-25 16:51:01.000000000","message":"Patch Set 4:\n\nBuild succeeded.\n\n- neutron http://cloudbase-ci.com/neutron/613239/4 : SUCCESS in 1h 48m 56s","accounts_in_message":[],"_revision_number":4},{"id":"1b3581debc993bca2255f6d6205dd6797302eeed","author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"date":"2018-10-25 18:32:06.000000000","message":"Patch Set 4: Code-Review+2\n\nLooks correct, thanks Slawek","accounts_in_message":[],"_revision_number":4},{"id":"0c3f11731d7fecbd97be4a7f8144a22bf19736e5","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2018-10-25 19:47:34.000000000","message":"Patch Set 3:\n\nBuild failed (check pipeline).  For information on how to proceed, see\nhttp://docs.openstack.org/infra/manual/developers.html#automated-testing\n\n\n- neutron-tempest-plugin-api http://logs.openstack.org/39/613239/3/check/neutron-tempest-plugin-api/64c6b64/ : SUCCESS in 56m 04s\n- neutron-tempest-plugin-designate-scenario http://logs.openstack.org/39/613239/3/check/neutron-tempest-plugin-designate-scenario/8081e16/ : SUCCESS in 36m 06s\n- neutron-tempest-plugin-dvr-multinode-scenario http://logs.openstack.org/39/613239/3/check/neutron-tempest-plugin-dvr-multinode-scenario/f3343e6/ : SUCCESS in 1h 34m 40s (non-voting)\n- neutron-tempest-plugin-scenario-linuxbridge http://logs.openstack.org/39/613239/3/check/neutron-tempest-plugin-scenario-linuxbridge/8aa521f/ : SUCCESS in 54m 36s\n- build-openstack-sphinx-docs http://logs.openstack.org/39/613239/3/check/build-openstack-sphinx-docs/6d9fead/html/ : SUCCESS in 4m 45s\n- openstack-tox-cover http://logs.openstack.org/39/613239/3/check/openstack-tox-cover/a8ca405/cover/ : SUCCESS in 24m 18s\n- openstack-tox-lower-constraints http://logs.openstack.org/39/613239/3/check/openstack-tox-lower-constraints/9c8c8a3/ : SUCCESS in 24m 44s\n- openstack-tox-pep8 http://logs.openstack.org/39/613239/3/check/openstack-tox-pep8/86aa6e9/ : SUCCESS in 12m 36s\n- openstack-tox-py27 http://logs.openstack.org/39/613239/3/check/openstack-tox-py27/a2b4be1/ : SUCCESS in 26m 11s\n- openstack-tox-py35 http://logs.openstack.org/39/613239/3/check/openstack-tox-py35/27a9699/ : SUCCESS in 29m 43s\n- openstack-tox-py36 http://logs.openstack.org/39/613239/3/check/openstack-tox-py36/1354a89/ : SUCCESS in 22m 59s\n- openstack-tox-docs http://logs.openstack.org/39/613239/3/check/openstack-tox-docs/1a0c81d/html/ : SUCCESS in 4m 11s\n- tempest-full http://logs.openstack.org/39/613239/3/check/tempest-full/41bf5b2/ : SUCCESS in 1h 36m 29s\n- neutron-grenade http://logs.openstack.org/39/613239/3/check/neutron-grenade/858d09b/ : SUCCESS in 55m 07s\n- tempest-full-py3 http://logs.openstack.org/39/613239/3/check/tempest-full-py3/6f9bd0e/ : SUCCESS in 1h 34m 12s\n- neutron-functional http://logs.openstack.org/39/613239/3/check/neutron-functional/c84ecf9/ : SUCCESS in 34m 52s\n- neutron-fullstack http://logs.openstack.org/39/613239/3/check/neutron-fullstack/76f491e/ : FAILURE in 1h 16m 47s\n- neutron-rally-task http://logs.openstack.org/39/613239/3/check/neutron-rally-task/de08b8b/ : SUCCESS in 1h 34m 05s\n- neutron-tempest-dvr http://logs.openstack.org/39/613239/3/check/neutron-tempest-dvr/cde1d61/ : SUCCESS in 1h 34m 36s\n- neutron-tempest-linuxbridge http://logs.openstack.org/39/613239/3/check/neutron-tempest-linuxbridge/118238d/ : SUCCESS in 1h 36m 08s\n- neutron-grenade-multinode http://logs.openstack.org/39/613239/3/check/neutron-grenade-multinode/328ffdf/ : SUCCESS in 1h 09m 22s\n- neutron-grenade-dvr-multinode http://logs.openstack.org/39/613239/3/check/neutron-grenade-dvr-multinode/8254def/ : SUCCESS in 1h 03m 30s\n- tempest-multinode-full http://logs.openstack.org/39/613239/3/check/tempest-multinode-full/7035895/ : SUCCESS in 1h 46m 21s (non-voting)\n- neutron-tempest-dvr-ha-multinode-full http://logs.openstack.org/39/613239/3/check/neutron-tempest-dvr-ha-multinode-full/d123b94/ : SUCCESS in 2h 00m 58s (non-voting)\n- neutron-tempest-iptables_hybrid http://logs.openstack.org/39/613239/3/check/neutron-tempest-iptables_hybrid/2384c3a/ : SUCCESS in 1h 38m 12s\n- neutron-fullstack-python36 http://logs.openstack.org/39/613239/3/check/neutron-fullstack-python36/8fde77f/ : SUCCESS in 1h 23m 27s\n- ironic-tempest-dsvm-ipa-wholedisk-bios-agent_ipmitool-tinyipa http://logs.openstack.org/39/613239/3/check/ironic-tempest-dsvm-ipa-wholedisk-bios-agent_ipmitool-tinyipa/4407a52/ : SUCCESS in 46m 22s (non-voting)\n- tempest-slow http://logs.openstack.org/39/613239/3/check/tempest-slow/872b742/ : SUCCESS in 2h 00m 51s","accounts_in_message":[],"_revision_number":3},{"id":"2a6c33ec4275df7c57fbd6eff8ea3069b88366fe","author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"date":"2018-10-25 20:52:44.000000000","message":"Patch Set 4:\n\nrecheck","accounts_in_message":[],"_revision_number":4},{"id":"a74c0c24700fbd8da9ad598fb98a4699a44fced5","author":{"_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},"date":"2018-10-25 21:38:35.000000000","message":"Patch Set 4:\n\nBuild failed.  To recheck use \u0027xenserver: recheck\u0027.  For 3rd party ci contact info: https://wiki.openstack.org/wiki/ThirdPartySystems\n\n- dsvm-tempest-neutron-network http://dd6b71949550285df7dc-dda4e480e005aaa13ec303551d2d8155.r49.cf1.rackcdn.com/39/613239/4/check/dsvm-tempest-neutron-network/4c1c0fe : FAILURE in 2h 40m 30s","accounts_in_message":[],"_revision_number":4},{"id":"c28ca4c52ae6d57eae48fe1f891d7d28c46a5e3b","author":{"_account_id":26622,"name":"Cloudbase Neutron Hyper-V CI","email":"neutron_hyperv_ci@cloudbasesolutions.com","username":"neutron_hyperv_ci"},"date":"2018-10-25 22:34:11.000000000","message":"Patch Set 4:\n\nBuild succeeded.\n\n- neutron http://cloudbase-ci.com/neutron/613239/4 : SUCCESS in 1h 41m 03s","accounts_in_message":[],"_revision_number":4},{"id":"48cfb00049c62ef669bd79d44c4dc3a47f4f28b8","author":{"_account_id":16376,"name":"Intel NFV CI","email":"openstack-nfv-ci@intel.com","username":"intel-nfv-ci","tags":["SERVICE_USER"]},"date":"2018-10-25 22:51:23.000000000","message":"Patch Set 4:\n\nBuild succeeded (check pipeline).\n\n- tempest-dsvm-ovsdpdk-nfv-networking-xenial http://52.27.155.124/portland/2018-10-25/613239/4/check/tempest-dsvm-ovsdpdk-nfv-networking-xenial/38ad321 : SUCCESS in 1h 00m 50s (non-voting)","accounts_in_message":[],"_revision_number":4},{"id":"362346397880dada083e6650764f923b95ae5b70","author":{"_account_id":9732,"name":"Mellanox CI","email":"mlnx-openstack-ci@dev.mellanox.co.il","username":"mellanox","tags":["SERVICE_USER"]},"date":"2018-10-26 01:59:50.000000000","message":"Patch Set 4: Verified+1\n\nBuild succeeded.\n\n- Neutron-ML2-Sriov http://13.74.249.42/39/613239/4/check-mlnx/Neutron-ML2-Sriov/172c654 : SUCCESS in 57m 24s\n- Neutron-MACVTAP-ML2-Sriov http://13.74.249.42/39/613239/4/check-mlnx/Neutron-MACVTAP-ML2-Sriov/6f83bf1 : SUCCESS in 1h 01m 50s\n\nTo re-run the job post \u0027recheck neutron-mlnx\u0027 comment. For more information visit https://wiki.openstack.org/wiki/ThirdPartySystems/Mellanox_CI","accounts_in_message":[],"_revision_number":4},{"id":"b7cbf008872e2bdbd42f15596b33d9eca4ce2c60","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2018-10-26 03:59:38.000000000","message":"Patch Set 4: Verified-1\n\nBuild failed (check pipeline).  For information on how to proceed, see\nhttp://docs.openstack.org/infra/manual/developers.html#automated-testing\n\n\n- neutron-tempest-plugin-api http://logs.openstack.org/39/613239/4/check/neutron-tempest-plugin-api/aa9e837/ : SUCCESS in 53m 59s\n- neutron-tempest-plugin-designate-scenario http://logs.openstack.org/39/613239/4/check/neutron-tempest-plugin-designate-scenario/eed6a84/ : SUCCESS in 30m 32s\n- neutron-tempest-plugin-dvr-multinode-scenario http://logs.openstack.org/39/613239/4/check/neutron-tempest-plugin-dvr-multinode-scenario/63686c8/ : SUCCESS in 1h 43m 15s (non-voting)\n- neutron-tempest-plugin-scenario-linuxbridge http://logs.openstack.org/39/613239/4/check/neutron-tempest-plugin-scenario-linuxbridge/08a471d/ : SUCCESS in 51m 31s\n- build-openstack-sphinx-docs http://logs.openstack.org/39/613239/4/check/build-openstack-sphinx-docs/7e134c3/html/ : SUCCESS in 4m 15s\n- openstack-tox-cover http://logs.openstack.org/39/613239/4/check/openstack-tox-cover/75ffdf2/cover/ : SUCCESS in 26m 02s\n- openstack-tox-lower-constraints http://logs.openstack.org/39/613239/4/check/openstack-tox-lower-constraints/e700634/ : SUCCESS in 20m 12s\n- openstack-tox-pep8 http://logs.openstack.org/39/613239/4/check/openstack-tox-pep8/c066bea/ : SUCCESS in 14m 01s\n- openstack-tox-py27 http://logs.openstack.org/39/613239/4/check/openstack-tox-py27/4e50ae9/ : SUCCESS in 31m 24s\n- openstack-tox-py35 http://logs.openstack.org/39/613239/4/check/openstack-tox-py35/12cd24f/ : SUCCESS in 38m 06s\n- openstack-tox-py36 http://logs.openstack.org/39/613239/4/check/openstack-tox-py36/1630b94/ : SUCCESS in 22m 35s\n- openstack-tox-docs http://logs.openstack.org/39/613239/4/check/openstack-tox-docs/11d8901/html/ : SUCCESS in 4m 22s\n- tempest-full http://logs.openstack.org/39/613239/4/check/tempest-full/2efda40/ : SUCCESS in 1h 50m 05s\n- neutron-grenade http://logs.openstack.org/39/613239/4/check/neutron-grenade/a45b8cb/ : SUCCESS in 55m 17s\n- tempest-full-py3 http://logs.openstack.org/39/613239/4/check/tempest-full-py3/13f8340/ : SUCCESS in 1h 22m 10s\n- neutron-functional http://logs.openstack.org/39/613239/4/check/neutron-functional/9a5c8e7/ : FAILURE in 48m 23s\n- neutron-fullstack http://logs.openstack.org/39/613239/4/check/neutron-fullstack/45d7f42/ : SUCCESS in 1h 01m 48s\n- neutron-rally-task http://logs.openstack.org/39/613239/4/check/neutron-rally-task/4e4b729/ : SUCCESS in 1h 38m 01s\n- neutron-tempest-dvr http://logs.openstack.org/39/613239/4/check/neutron-tempest-dvr/dfcb466/ : SUCCESS in 1h 37m 43s\n- neutron-tempest-linuxbridge http://logs.openstack.org/39/613239/4/check/neutron-tempest-linuxbridge/6042bd6/ : SUCCESS in 1h 40m 25s\n- neutron-grenade-multinode http://logs.openstack.org/39/613239/4/check/neutron-grenade-multinode/042f6e8/ : SUCCESS in 1h 07m 28s\n- neutron-grenade-dvr-multinode http://logs.openstack.org/39/613239/4/check/neutron-grenade-dvr-multinode/2738caf/ : SUCCESS in 1h 07m 16s\n- tempest-multinode-full http://logs.openstack.org/39/613239/4/check/tempest-multinode-full/0f08ac5/ : SUCCESS in 1h 49m 15s (non-voting)\n- neutron-tempest-dvr-ha-multinode-full http://logs.openstack.org/39/613239/4/check/neutron-tempest-dvr-ha-multinode-full/c74326f/ : SUCCESS in 1h 45m 31s (non-voting)\n- neutron-tempest-iptables_hybrid http://logs.openstack.org/39/613239/4/check/neutron-tempest-iptables_hybrid/6ecbb10/ : SUCCESS in 1h 36m 34s\n- neutron-fullstack-python36 http://logs.openstack.org/39/613239/4/check/neutron-fullstack-python36/5eee05f/ : SUCCESS in 1h 04m 36s\n- ironic-tempest-dsvm-ipa-wholedisk-bios-agent_ipmitool-tinyipa http://logs.openstack.org/39/613239/4/check/ironic-tempest-dsvm-ipa-wholedisk-bios-agent_ipmitool-tinyipa/8a04c4e/ : SUCCESS in 57m 14s (non-voting)\n- tempest-slow http://logs.openstack.org/39/613239/4/check/tempest-slow/ebc59b6/ : SUCCESS in 1h 45m 55s","accounts_in_message":[],"_revision_number":4},{"id":"d4b6cd5be7fd3173617eb1ab4aba6e82761aa4b3","author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"date":"2018-10-26 06:46:25.000000000","message":"Patch Set 4:\n\nrecheck","accounts_in_message":[],"_revision_number":4},{"id":"bc10e227ff0bb8f0f4722da3cf166defbc001855","author":{"_account_id":16376,"name":"Intel NFV CI","email":"openstack-nfv-ci@intel.com","username":"intel-nfv-ci","tags":["SERVICE_USER"]},"date":"2018-10-26 07:49:29.000000000","message":"Patch Set 4:\n\nBuild succeeded (check pipeline).\n\n- tempest-dsvm-ovsdpdk-nfv-networking-xenial http://52.27.155.124/portland/2018-10-26/613239/4/check/tempest-dsvm-ovsdpdk-nfv-networking-xenial/d99bc01 : SUCCESS in 1h 01m 51s (non-voting)","accounts_in_message":[],"_revision_number":4},{"id":"1ee5f9b39aff533ad37e20348767c834419dd0ba","author":{"_account_id":26622,"name":"Cloudbase Neutron Hyper-V CI","email":"neutron_hyperv_ci@cloudbasesolutions.com","username":"neutron_hyperv_ci"},"date":"2018-10-26 08:31:33.000000000","message":"Patch Set 4:\n\nBuild succeeded.\n\n- neutron http://cloudbase-ci.com/neutron/613239/4 : SUCCESS in 1h 44m 41s","accounts_in_message":[],"_revision_number":4},{"id":"074fdd327204246a41a788d03c033d02fb354e23","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2018-10-26 10:24:27.000000000","message":"Patch Set 4:\n\nBuild failed (check pipeline).  For information on how to proceed, see\nhttp://docs.openstack.org/infra/manual/developers.html#automated-testing\n\n\n- neutron-tempest-plugin-api http://logs.openstack.org/39/613239/4/check/neutron-tempest-plugin-api/ddc7086/ : SUCCESS in 48m 06s\n- neutron-tempest-plugin-designate-scenario http://logs.openstack.org/39/613239/4/check/neutron-tempest-plugin-designate-scenario/d376fb3/ : SUCCESS in 27m 58s\n- neutron-tempest-plugin-dvr-multinode-scenario http://logs.openstack.org/39/613239/4/check/neutron-tempest-plugin-dvr-multinode-scenario/083b67e/ : FAILURE in 2h 20m 53s (non-voting)\n- neutron-tempest-plugin-scenario-linuxbridge http://logs.openstack.org/39/613239/4/check/neutron-tempest-plugin-scenario-linuxbridge/9932f24/ : FAILURE in 1h 06m 01s\n- build-openstack-sphinx-docs http://logs.openstack.org/39/613239/4/check/build-openstack-sphinx-docs/391af8d/html/ : SUCCESS in 4m 45s\n- openstack-tox-cover http://logs.openstack.org/39/613239/4/check/openstack-tox-cover/ea2cda9/cover/ : SUCCESS in 31m 50s\n- openstack-tox-lower-constraints http://logs.openstack.org/39/613239/4/check/openstack-tox-lower-constraints/1251dd5/ : SUCCESS in 25m 25s\n- openstack-tox-pep8 http://logs.openstack.org/39/613239/4/check/openstack-tox-pep8/59a9758/ : SUCCESS in 13m 44s\n- openstack-tox-py27 http://logs.openstack.org/39/613239/4/check/openstack-tox-py27/77cc20f/ : SUCCESS in 23m 53s\n- openstack-tox-py35 http://logs.openstack.org/39/613239/4/check/openstack-tox-py35/c4f3671/ : SUCCESS in 23m 37s\n- openstack-tox-py36 http://logs.openstack.org/39/613239/4/check/openstack-tox-py36/11b4150/ : SUCCESS in 22m 07s\n- openstack-tox-docs http://logs.openstack.org/39/613239/4/check/openstack-tox-docs/25f25b7/html/ : SUCCESS in 4m 48s\n- tempest-full http://logs.openstack.org/39/613239/4/check/tempest-full/6046921/ : SUCCESS in 1h 44m 08s\n- neutron-grenade http://logs.openstack.org/39/613239/4/check/neutron-grenade/c4e838f/ : SUCCESS in 53m 02s\n- tempest-full-py3 http://logs.openstack.org/39/613239/4/check/tempest-full-py3/b235d3f/ : SUCCESS in 1h 17m 26s\n- neutron-functional http://logs.openstack.org/39/613239/4/check/neutron-functional/496a17b/ : SUCCESS in 40m 23s\n- neutron-fullstack http://logs.openstack.org/39/613239/4/check/neutron-fullstack/7ab504f/ : SUCCESS in 57m 25s\n- neutron-rally-task http://logs.openstack.org/39/613239/4/check/neutron-rally-task/a545092/ : SUCCESS in 1h 41m 30s\n- neutron-tempest-dvr http://logs.openstack.org/39/613239/4/check/neutron-tempest-dvr/bba5efd/ : SUCCESS in 1h 36m 01s\n- neutron-tempest-linuxbridge http://logs.openstack.org/39/613239/4/check/neutron-tempest-linuxbridge/7617037/ : SUCCESS in 1h 34m 25s\n- neutron-grenade-multinode http://logs.openstack.org/39/613239/4/check/neutron-grenade-multinode/ffcf17f/ : SUCCESS in 55m 03s\n- neutron-grenade-dvr-multinode http://logs.openstack.org/39/613239/4/check/neutron-grenade-dvr-multinode/2519702/ : SUCCESS in 1h 10m 39s\n- tempest-multinode-full http://logs.openstack.org/39/613239/4/check/tempest-multinode-full/ade177d/ : SUCCESS in 1h 48m 08s (non-voting)\n- neutron-tempest-dvr-ha-multinode-full http://logs.openstack.org/39/613239/4/check/neutron-tempest-dvr-ha-multinode-full/1b4aa73/ : SUCCESS in 1h 51m 48s (non-voting)\n- neutron-tempest-iptables_hybrid http://logs.openstack.org/39/613239/4/check/neutron-tempest-iptables_hybrid/afc08ca/ : SUCCESS in 1h 26m 50s\n- neutron-fullstack-python36 http://logs.openstack.org/39/613239/4/check/neutron-fullstack-python36/5f25746/ : SUCCESS in 1h 01m 44s\n- ironic-tempest-dsvm-ipa-wholedisk-bios-agent_ipmitool-tinyipa http://logs.openstack.org/39/613239/4/check/ironic-tempest-dsvm-ipa-wholedisk-bios-agent_ipmitool-tinyipa/52a1db9/ : SUCCESS in 50m 12s (non-voting)\n- tempest-slow http://logs.openstack.org/39/613239/4/check/tempest-slow/3fc6973/ : SUCCESS in 1h 57m 04s","accounts_in_message":[],"_revision_number":4},{"id":"ff3d86b267d1c12f733b1dbf87a0a00c5ba27d65","author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"date":"2018-10-26 11:05:09.000000000","message":"Patch Set 4:\n\nrecheck","accounts_in_message":[],"_revision_number":4},{"id":"64c4f9887d6735a38d698023fff4ba6723f6e44b","author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"date":"2018-10-26 11:25:01.000000000","message":"Patch Set 4: Code-Review-1\n\na vlan of 0 is not the same thing as no vlan\n\ni do not think this is a valid fix.\nif the neutron flat network is not also not set as vlan transparent i dont think this should be allowed.\n\nml2/ovs dose not support vlan tranparancy so sending a vlan tagged packet from the guest is invalid unless using vlanawre vms which is not the case here.\n\nhave you confirmed that this will have no impact on security goups as i woudl suspect the presence of a vlan header should affect both the openvswtich security gorup driver and iptables.","accounts_in_message":[],"_revision_number":4},{"id":"4b394637266025bbf0647fc85711f4f27fd768c8","author":{"_account_id":16376,"name":"Intel NFV CI","email":"openstack-nfv-ci@intel.com","username":"intel-nfv-ci","tags":["SERVICE_USER"]},"date":"2018-10-26 12:05:09.000000000","message":"Patch Set 4:\n\nBuild succeeded (check pipeline).\n\n- tempest-dsvm-ovsdpdk-nfv-networking-xenial http://52.27.155.124/portland/2018-10-26/613239/4/check/tempest-dsvm-ovsdpdk-nfv-networking-xenial/76c4ea1 : SUCCESS in 59m 43s (non-voting)","accounts_in_message":[],"_revision_number":4},{"id":"160433b8ff1abb9f40ad68f4fa32787a56f9ee5e","author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"date":"2018-10-26 12:13:27.000000000","message":"Patch Set 4:\n\n\u003e a vlan of 0 is not the same thing as no vlan\n \u003e \n \u003e i do not think this is a valid fix.\n \u003e if the neutron flat network is not also not set as vlan transparent\n \u003e i dont think this should be allowed.\n \u003e \n \u003e ml2/ovs dose not support vlan tranparancy so sending a vlan tagged\n \u003e packet from the guest is invalid unless using vlanawre vms which is\n \u003e not the case here.\n \u003e \n \u003e have you confirmed that this will have no impact on security goups\n \u003e as i woudl suspect the presence of a vlan header should affect both\n \u003e the openvswtich security gorup driver and iptables.\n\nOnly thing which I\u0027m changing here is that now ovs will not match packets based on DEI bit in 802.1q header. Nothing else. VLAN ID (last 12 bits still need to be the same as it was before this patch).\nIf there is any issue with security groups and such filters as You are suspecting, IMO it should be reported as separate bug and solved in different patch.","accounts_in_message":[],"_revision_number":4},{"id":"d867d8d45cbfb97f250a6839724282fa4f3b4f26","author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"date":"2018-10-26 12:13:57.000000000","message":"Patch Set 4:\n\nOne more thing. Very detailed explanation of reason for this change is in https://bugzilla.redhat.com/show_bug.cgi?id\u003d1635909","accounts_in_message":[],"_revision_number":4},{"id":"73ddd320f6b446a2eb35500123ce259c6bfc389b","author":{"_account_id":26622,"name":"Cloudbase Neutron Hyper-V CI","email":"neutron_hyperv_ci@cloudbasesolutions.com","username":"neutron_hyperv_ci"},"date":"2018-10-26 12:41:38.000000000","message":"Patch Set 4:\n\nBuild succeeded.\n\n- neutron http://cloudbase-ci.com/neutron/613239/4 : SUCCESS in 1h 36m 04s","accounts_in_message":[],"_revision_number":4},{"id":"3fb954c248c8de2e2f9f39397336eedadec7abf8","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2018-10-26 16:40:58.000000000","message":"Patch Set 4: Verified+1\n\nBuild succeeded (check pipeline).\n\n- neutron-tempest-plugin-api http://logs.openstack.org/39/613239/4/check/neutron-tempest-plugin-api/d1789d1/ : SUCCESS in 50m 43s\n- neutron-tempest-plugin-designate-scenario http://logs.openstack.org/39/613239/4/check/neutron-tempest-plugin-designate-scenario/76e2c04/ : SUCCESS in 30m 30s\n- neutron-tempest-plugin-dvr-multinode-scenario http://logs.openstack.org/39/613239/4/check/neutron-tempest-plugin-dvr-multinode-scenario/8138737/ : FAILURE in 2h 09m 04s (non-voting)\n- neutron-tempest-plugin-scenario-linuxbridge http://logs.openstack.org/39/613239/4/check/neutron-tempest-plugin-scenario-linuxbridge/545fa45/ : SUCCESS in 1h 00m 08s\n- build-openstack-sphinx-docs http://logs.openstack.org/39/613239/4/check/build-openstack-sphinx-docs/05eb395/html/ : SUCCESS in 4m 52s\n- openstack-tox-cover http://logs.openstack.org/39/613239/4/check/openstack-tox-cover/562115f/cover/ : SUCCESS in 35m 26s\n- openstack-tox-lower-constraints http://logs.openstack.org/39/613239/4/check/openstack-tox-lower-constraints/d992aef/ : SUCCESS in 21m 17s\n- openstack-tox-pep8 http://logs.openstack.org/39/613239/4/check/openstack-tox-pep8/4e5fabb/ : SUCCESS in 12m 51s\n- openstack-tox-py27 http://logs.openstack.org/39/613239/4/check/openstack-tox-py27/3200c28/ : SUCCESS in 22m 36s\n- openstack-tox-py35 http://logs.openstack.org/39/613239/4/check/openstack-tox-py35/7eccfb9/ : SUCCESS in 21m 04s\n- openstack-tox-py36 http://logs.openstack.org/39/613239/4/check/openstack-tox-py36/1251d17/ : SUCCESS in 24m 50s\n- openstack-tox-docs http://logs.openstack.org/39/613239/4/check/openstack-tox-docs/57ab9e1/html/ : SUCCESS in 4m 49s\n- tempest-full http://logs.openstack.org/39/613239/4/check/tempest-full/e049055/ : SUCCESS in 2h 05m 21s\n- neutron-grenade http://logs.openstack.org/39/613239/4/check/neutron-grenade/36af4a4/ : SUCCESS in 1h 03m 26s\n- tempest-full-py3 http://logs.openstack.org/39/613239/4/check/tempest-full-py3/cef823c/ : SUCCESS in 1h 14m 31s\n- neutron-functional http://logs.openstack.org/39/613239/4/check/neutron-functional/4ae4f3f/ : SUCCESS in 36m 37s\n- neutron-fullstack http://logs.openstack.org/39/613239/4/check/neutron-fullstack/218d735/ : SUCCESS in 1h 03m 43s\n- neutron-rally-task http://logs.openstack.org/39/613239/4/check/neutron-rally-task/cc1ea41/ : SUCCESS in 1h 45m 44s\n- neutron-tempest-dvr http://logs.openstack.org/39/613239/4/check/neutron-tempest-dvr/501d6ef/ : SUCCESS in 1h 44m 25s\n- neutron-tempest-linuxbridge http://logs.openstack.org/39/613239/4/check/neutron-tempest-linuxbridge/0d07917/ : SUCCESS in 2h 11m 07s\n- neutron-grenade-multinode http://logs.openstack.org/39/613239/4/check/neutron-grenade-multinode/e0ad059/ : SUCCESS in 57m 59s\n- neutron-grenade-dvr-multinode http://logs.openstack.org/39/613239/4/check/neutron-grenade-dvr-multinode/a761530/ : SUCCESS in 1h 06m 33s\n- tempest-multinode-full http://logs.openstack.org/39/613239/4/check/tempest-multinode-full/bbe32a0/ : SUCCESS in 1h 50m 44s (non-voting)\n- neutron-tempest-dvr-ha-multinode-full http://logs.openstack.org/39/613239/4/check/neutron-tempest-dvr-ha-multinode-full/dd444cd/ : SUCCESS in 1h 51m 48s (non-voting)\n- neutron-tempest-iptables_hybrid http://logs.openstack.org/39/613239/4/check/neutron-tempest-iptables_hybrid/3199dd9/ : SUCCESS in 1h 37m 15s\n- neutron-fullstack-python36 http://logs.openstack.org/39/613239/4/check/neutron-fullstack-python36/2c53e07/ : SUCCESS in 1h 40m 35s\n- ironic-tempest-dsvm-ipa-wholedisk-bios-agent_ipmitool-tinyipa http://logs.openstack.org/39/613239/4/check/ironic-tempest-dsvm-ipa-wholedisk-bios-agent_ipmitool-tinyipa/24dc54d/ : SUCCESS in 41m 47s (non-voting)\n- tempest-slow http://logs.openstack.org/39/613239/4/check/tempest-slow/560d943/ : SUCCESS in 1h 53m 33s","accounts_in_message":[],"_revision_number":4},{"id":"ecfb9f9f3342073252529843055da1a94f392a3d","author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"date":"2018-10-26 19:45:10.000000000","message":"Patch Set 4:\n\nI would agree with Slawek, according to this comment this change is Ok:\n\nhttps://bugzilla.redhat.com/show_bug.cgi?id\u003d1635909#c14","accounts_in_message":[],"_revision_number":4},{"id":"6e4173c70da44053666a2554210b9a5f7e6f5381","author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"date":"2018-10-30 10:26:13.000000000","message":"Patch Set 4:\n\n(2 comments)","accounts_in_message":[],"_revision_number":4},{"id":"fd3526f24b38bdbb1ccf4176a7f206d201cd01e7","author":{"_account_id":24042,"name":"Andreas Karis","email":"akaris@redhat.com","username":"akaris"},"date":"2018-10-31 17:52:33.000000000","message":"Patch Set 4:\n\nIs it possible in neutron to create VLAN provider networks with tag 0? I guess that this would be the valid and supported workaround for the customer then instead of this patch.\n\nThe customer\u0027s concern is that they otherwise have to go into their end-customer systems every time a support case is generated and change the switch configuration.\n\nImagine the end customer has a trunk port on the switch that connects to our controller node. Somebody on the internet may tag the port as EF. That EF IPv4 packet will be set as COS 5 on all switches in the corporate network. If end customers don\u0027t instruct their switches to drop the IPv4 EF (or COS) somewhere in their network, that tag will make it all the way to the controller into OVS, where it gets dropped. \nShould administrators take care of DSCP and COS tagging as part of best practices? Sure. But there are companies which don\u0027t. I personally ran into a similar issue in the company I worked for before, and this was not easy to troubleshoot. We ended up marking DSCP at the edge of the corporate network. We should of course have done this before, but we didn\u0027t, causing weird issues with some products that ran in the corporate network.\n\nTHe customer\u0027s concern is that their end customers may run into exactly the same scenarios, open a support ticket with the customer, thus creating a whole lot of overhead with the customer\u0027s support.\n\nHence, I see the following possibilities:\n* we tell the customer to document with the end customers that they need to drop COS and thus send untagged traffic for flat networks only\n* we change the neutron spec\n* the customer can change the configuration of neutron to use VLAN provider networks with VLAN tag 0. I did not try this, but I suppose that this would work?\n\nThanks,\n\nAndreas","accounts_in_message":[],"_revision_number":4},{"id":"085a1588cecb078b728e51e70f2ae518e86d5ba8","author":{"_account_id":24042,"name":"Andreas Karis","email":"akaris@redhat.com","username":"akaris"},"date":"2018-10-31 17:59:03.000000000","message":"Patch Set 4:\n\nneutron net-create provider10 --provider:network_type vlan --provider:physical_network datacentre --provider:segmentation_id 0 --shared --router:external\nneutron router-create router10\nneutron router-gateway-set router10 provider10\n\n\n\nSeems to be possible and would IMO do the job:\n\n[root@overcloud-controller-0 ~]# ovs-ofctl dump-flows br-int\n cookie\u003d0xadec411fa0515253, duration\u003d1188297.882s, table\u003d0, n_packets\u003d1304, n_bytes\u003d128986, priority\u003d3,in_port\u003d\"int-br-ex\",dl_vlan\u003d905 actions\u003dmod_vlan_vid:3,NORMAL\n cookie\u003d0xadec411fa0515253, duration\u003d1188291.665s, table\u003d0, n_packets\u003d89041, n_bytes\u003d6945238, priority\u003d3,in_port\u003d\"int-br-ex\",vlan_tci\u003d0x0000/0x1fff actions\u003dmod_vlan_vid:4,NORMAL\n cookie\u003d0xadec411fa0515253, duration\u003d183.053s, table\u003d0, n_packets\u003d0, n_bytes\u003d0, priority\u003d3,in_port\u003d\"int-br-ex\",dl_vlan\u003d0 actions\u003dmod_vlan_vid:5,NORMAL\n cookie\u003d0xadec411fa0515253, duration\u003d1188350.130s, table\u003d0, n_packets\u003d80, n_bytes\u003d4196, priority\u003d2,in_port\u003d\"int-br-ex\" actions\u003ddrop\n cookie\u003d0xadec411fa0515253, duration\u003d1188350.231s, table\u003d0, n_packets\u003d18538, n_bytes\u003d2182045, priority\u003d0 actions\u003dNORMAL\n cookie\u003d0xadec411fa0515253, duration\u003d1188350.234s, table\u003d23, n_packets\u003d0, n_bytes\u003d0, priority\u003d0 actions\u003ddrop\n cookie\u003d0xadec411fa0515253, duration\u003d1188350.230s, table\u003d24, n_packets\u003d0, n_bytes\u003d0, priority\u003d0 actions\u003ddrop","accounts_in_message":[],"_revision_number":4},{"id":"4f49695b5a3b8f0bca2730ba5352f1ab8879d539","author":{"_account_id":24042,"name":"Andreas Karis","email":"akaris@redhat.com","username":"akaris"},"date":"2018-10-31 18:06:50.000000000","message":"Patch Set 4:\n\n\u003e neutron net-create provider10 --provider:network_type vlan\n \u003e --provider:physical_network datacentre --provider:segmentation_id 0\n \u003e --shared --router:external\n \u003e neutron router-create router10\n \u003e neutron router-gateway-set router10 provider10\n \u003e \n \u003e \n \u003e \n \u003e Seems to be possible and would IMO do the job:\n \u003e \n \u003e [root@overcloud-controller-0 ~]# ovs-ofctl dump-flows br-int\n \u003e cookie\u003d0xadec411fa0515253, duration\u003d1188297.882s, table\u003d0,\n \u003e n_packets\u003d1304, n_bytes\u003d128986, priority\u003d3,in_port\u003d\"int-br-ex\",dl_vlan\u003d905\n \u003e actions\u003dmod_vlan_vid:3,NORMAL\n \u003e cookie\u003d0xadec411fa0515253, duration\u003d1188291.665s, table\u003d0,\n \u003e n_packets\u003d89041, n_bytes\u003d6945238, priority\u003d3,in_port\u003d\"int-br-ex\",vlan_tci\u003d0x0000/0x1fff\n \u003e actions\u003dmod_vlan_vid:4,NORMAL\n \u003e cookie\u003d0xadec411fa0515253, duration\u003d183.053s, table\u003d0, n_packets\u003d0,\n \u003e n_bytes\u003d0, priority\u003d3,in_port\u003d\"int-br-ex\",dl_vlan\u003d0\n \u003e actions\u003dmod_vlan_vid:5,NORMAL\n \u003e cookie\u003d0xadec411fa0515253, duration\u003d1188350.130s, table\u003d0,\n \u003e n_packets\u003d80, n_bytes\u003d4196, priority\u003d2,in_port\u003d\"int-br-ex\"\n \u003e actions\u003ddrop\n \u003e cookie\u003d0xadec411fa0515253, duration\u003d1188350.231s, table\u003d0,\n \u003e n_packets\u003d18538, n_bytes\u003d2182045, priority\u003d0 actions\u003dNORMAL\n \u003e cookie\u003d0xadec411fa0515253, duration\u003d1188350.234s, table\u003d23,\n \u003e n_packets\u003d0, n_bytes\u003d0, priority\u003d0 actions\u003ddrop\n \u003e cookie\u003d0xadec411fa0515253, duration\u003d1188350.230s, table\u003d24,\n \u003e n_packets\u003d0, n_bytes\u003d0, priority\u003d0 actions\u003ddrop\n\n\n\n\nBy the way, I don\u0027t know if:\n\nneutron net-create provider10 --provider:network_type vlan --provider:physical_network datacentre --provider:segmentation_id 0 --shared --router:external\nneutron router-create router10\nneutron router-gateway-set router10 provider10\n\nWould work with native VLANs without dot1q tags, it might not. So the current solution which is valid with the neutron contract defined in \nhttps://github.com/openstack/neutron/blob/a388701ddfe628e9a5bd16a78422164799b11ef8/doc/source/admin/archives/adv-features.rst#terminology\n~~~\nflat network \tA virtual network implemented as packets on a specific physical network containing no IEEE 802.1Q header. Each physical network can realize at most one flat network.\n~~~\nis to strip dot1q tags for the native VLAN.\n\nKind regards,\n\nAndreas","accounts_in_message":[],"_revision_number":4},{"id":"5a848577b8590816dc70188c2857e1797a49b3b4","author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"date":"2018-11-07 11:27:28.000000000","message":"Abandoned\n\nas per Sean\u0027s comments it shouldn\u0027t be done this way","accounts_in_message":[],"_revision_number":4}],"current_revision_number":4,"current_revision":"e58e480f2a12bab524109f432d1afc6459eb4e4c","revisions":{"cfe243add943e4103348b292e98ca20389622f57":{"kind":"REWORK","_number":1,"created":"2018-10-25 08:29:49.000000000","uploader":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"ref":"refs/changes/39/613239/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/neutron","ref":"refs/changes/39/613239/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/neutron refs/changes/39/613239/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/neutron refs/changes/39/613239/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/neutron refs/changes/39/613239/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/neutron refs/changes/39/613239/1"}}},"commit":{"parents":[{"commit":"b5fbac8ec8f4070bc9e4496ec327de090a856873","subject":"Merge \"[Functional] Increase test_timeout for db migration tests\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/neutron/commit/b5fbac8ec8f4070bc9e4496ec327de090a856873"}]}],"author":{"name":"Slawek Kaplonski","email":"skaplons@redhat.com","date":"2018-10-25 08:20:33.000000000","tz":120},"committer":{"name":"Slawek Kaplonski","email":"skaplons@redhat.com","date":"2018-10-25 08:20:33.000000000","tz":120},"subject":"Don\u0027t filter on the DCI bit for flat networks","message":"Don\u0027t filter on the DCI bit for flat networks\n\nFor flat networks, openvswitch-agent configure in br-int\nan openflow rule to match packets without vlan id and to set\nlocal vlan_id for such packets.\nThis was done by\n\n    vlan_tci\u003d0x0000/0x1fff\n\nin rule match and this\nworks fine for frames without a dot1q tag.\nAccording to openvswitch manual [1], we should set vlan_tci with\nmask that will wildcard DCI bit to allow match also for dot1q\ntagged frames on VLAN 0, e.g. it COP is modified, and VLAN ID is 0.\n\nTo fix that, this patch changes openflow rule created for such\nnetworks in br-int to use:\n\n    vlan_tci \u003d 0x0000/0x0fff\n\nwhich means that DCI bit is wildcarde in rule match.\n\n[1] http://www.openvswitch.org/support/dist-docs-2.5/ovs-ofctl.8.txt\n\nChange-Id: Ia22ea54e16cfb4ae423836a06816ec43d9135df6\nCloses-Bug: #1799599\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/neutron/commit/cfe243add943e4103348b292e98ca20389622f57"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/neutron/commit/cfe243add943e4103348b292e98ca20389622f57"}]},"branch":"refs/heads/master"},"f8942754a772fa2d266a27045290882fb7a0c4d8":{"kind":"REWORK","_number":2,"created":"2018-10-25 08:34:30.000000000","uploader":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"ref":"refs/changes/39/613239/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/neutron","ref":"refs/changes/39/613239/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/neutron refs/changes/39/613239/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/neutron refs/changes/39/613239/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/neutron refs/changes/39/613239/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/neutron refs/changes/39/613239/2"}}},"commit":{"parents":[{"commit":"b5fbac8ec8f4070bc9e4496ec327de090a856873","subject":"Merge \"[Functional] Increase test_timeout for db migration tests\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/neutron/commit/b5fbac8ec8f4070bc9e4496ec327de090a856873"}]}],"author":{"name":"Slawek Kaplonski","email":"skaplons@redhat.com","date":"2018-10-25 08:20:33.000000000","tz":120},"committer":{"name":"Slawek Kaplonski","email":"skaplons@redhat.com","date":"2018-10-25 08:34:17.000000000","tz":120},"subject":"Don\u0027t filter on the DCI bit for flat networks","message":"Don\u0027t filter on the DCI bit for flat networks\n\nFor flat networks, openvswitch-agent configure in br-int\nan openflow rule to match packets without vlan id and to set\nlocal vlan_id for such packets.\nThis was done by\n\n    vlan_tci\u003d0x0000/0x1fff\n\nin rule match and this\nworks fine for frames without a dot1q tag.\nAccording to openvswitch manual [1], we should set vlan_tci with\nmask that will wildcard DCI bit to allow match also for dot1q\ntagged frames on VLAN 0, e.g. it COP is modified, and VLAN ID is 0.\n\nTo fix that, this patch changes openflow rule created for such\nnetworks in br-int to use:\n\n    vlan_tci \u003d 0x0000/0x0fff\n\nwhich means that DCI bit is wildcarde in rule match.\n\n[1] http://www.openvswitch.org/support/dist-docs-2.5/ovs-ofctl.8.txt\n\nChange-Id: Ia22ea54e16cfb4ae423836a06816ec43d9135df6\nCloses-Bug: #1799599\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/neutron/commit/f8942754a772fa2d266a27045290882fb7a0c4d8"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/neutron/commit/f8942754a772fa2d266a27045290882fb7a0c4d8"}]},"branch":"refs/heads/master"},"aeaec125b26f1ce41885580c8bcb1c36872c7a78":{"kind":"REWORK","_number":3,"created":"2018-10-25 13:12:53.000000000","uploader":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"ref":"refs/changes/39/613239/3","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/neutron","ref":"refs/changes/39/613239/3","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/neutron refs/changes/39/613239/3 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/neutron refs/changes/39/613239/3 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/neutron refs/changes/39/613239/3 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/neutron refs/changes/39/613239/3"}}},"commit":{"parents":[{"commit":"b5fbac8ec8f4070bc9e4496ec327de090a856873","subject":"Merge \"[Functional] Increase test_timeout for db migration tests\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/neutron/commit/b5fbac8ec8f4070bc9e4496ec327de090a856873"}]}],"author":{"name":"Slawek Kaplonski","email":"skaplons@redhat.com","date":"2018-10-25 08:20:33.000000000","tz":120},"committer":{"name":"Slawek Kaplonski","email":"skaplons@redhat.com","date":"2018-10-25 13:12:29.000000000","tz":120},"subject":"Don\u0027t filter on the DCI bit for flat networks","message":"Don\u0027t filter on the DCI bit for flat networks\n\nFor flat networks, openvswitch-agent configure in br-int\nan openflow rule to match packets without vlan id and to set\nlocal vlan_id for such packets.\nThis was done by\n\n    vlan_tci\u003d0x0000/0x1fff\n\nin rule match and this\nworks fine for frames without a dot1q tag.\nAccording to openvswitch manual [1], we should set vlan_tci with\nmask that will wildcard DCI bit to allow match also for dot1q\ntagged frames on VLAN 0, e.g. it COP is modified, and VLAN ID is 0.\n\nTo fix that, this patch changes openflow rule created for such\nnetworks in br-int to use:\n\n    vlan_tci \u003d 0x0000/0x0fff\n\nwhich means that DCI bit is wildcarde in rule match.\n\n[1] http://www.openvswitch.org/support/dist-docs-2.5/ovs-ofctl.8.txt\n\nChange-Id: Ia22ea54e16cfb4ae423836a06816ec43d9135df6\nCloses-Bug: #1799599\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/neutron/commit/aeaec125b26f1ce41885580c8bcb1c36872c7a78"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/neutron/commit/aeaec125b26f1ce41885580c8bcb1c36872c7a78"}]},"branch":"refs/heads/master"},"e58e480f2a12bab524109f432d1afc6459eb4e4c":{"kind":"NO_CODE_CHANGE","_number":4,"created":"2018-10-25 15:01:37.000000000","uploader":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"ref":"refs/changes/39/613239/4","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/neutron","ref":"refs/changes/39/613239/4","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/neutron refs/changes/39/613239/4 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/neutron refs/changes/39/613239/4 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/neutron refs/changes/39/613239/4 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/neutron refs/changes/39/613239/4"}}},"commit":{"parents":[{"commit":"b5fbac8ec8f4070bc9e4496ec327de090a856873","subject":"Merge \"[Functional] Increase test_timeout for db migration tests\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/neutron/commit/b5fbac8ec8f4070bc9e4496ec327de090a856873"}]}],"author":{"name":"Slawek Kaplonski","email":"skaplons@redhat.com","date":"2018-10-25 08:20:33.000000000","tz":120},"committer":{"name":"Slawek Kaplonski","email":"skaplons@redhat.com","date":"2018-10-25 15:01:33.000000000","tz":0},"subject":"Don\u0027t filter on the TCI bit for flat networks","message":"Don\u0027t filter on the TCI bit for flat networks\n\nFor flat networks, openvswitch-agent configure in br-int\nan openflow rule to match packets without vlan id and to set\nlocal vlan_id for such packets.\nThis was done by\n\n    vlan_tci\u003d0x0000/0x1fff\n\nin rule match and this\nworks fine for frames without a dot1q tag.\nAccording to openvswitch manual [1], we should set vlan_tci with\nmask that will wildcard TCI bit to allow match also for dot1q\ntagged frames on VLAN 0, e.g. it COP is modified, and VLAN ID is 0.\n\nTo fix that, this patch changes openflow rule created for such\nnetworks in br-int to use:\n\n    vlan_tci \u003d 0x0000/0x0fff\n\nwhich means that DCI bit is wildcarde in rule match.\n\n[1] http://www.openvswitch.org/support/dist-docs-2.5/ovs-ofctl.8.txt\n\nChange-Id: Ia22ea54e16cfb4ae423836a06816ec43d9135df6\nCloses-Bug: #1799599\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/neutron/commit/e58e480f2a12bab524109f432d1afc6459eb4e4c"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/neutron/commit/e58e480f2a12bab524109f432d1afc6459eb4e4c"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}