)]}'
{"id":"openstack%2Fneutron~703813","triplet_id":"openstack%2Fneutron~master~Ifb795626dc9c2ac4f0104f491dd38c9b4cc902c9","project":"openstack/neutron","branch":"master","topic":"bug/1842988-external_mac","hashtags":[],"change_id":"Ifb795626dc9c2ac4f0104f491dd38c9b4cc902c9","subject":"[OVN] Remove VLAN check when setting external_mac","status":"MERGED","created":"2020-01-22 15:03:55.000000000","updated":"2020-01-31 13:39:08.000000000","submitted":"2020-01-31 13:36:51.000000000","submitter":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"submission_id":"703813-1580477811812-71032318","meta_rev_id":"807d632c5697dd3c874d598edfa7cda792276cd2","_number":703813,"virtual_id_number":703813,"owner":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"tag":"autogenerated:zuul:gate","value":2,"date":"2020-01-31 13:36:51.000000000","post_submit":true,"permitted_voting_range":{"min":2,"max":2},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"_account_id":23804,"name":"Daniel Alvarez","email":"dalvarez@redhat.com","username":"dalvarez"},{"value":0,"date":"2020-01-29 11:33:17.000000000","_account_id":26622,"name":"Cloudbase Neutron Hyper-V CI","email":"neutron_hyperv_ci@cloudbasesolutions.com","username":"neutron_hyperv_ci"},{"value":0,"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},{"value":0,"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},{"value":0,"date":"2020-01-29 11:01:07.000000000","permitted_voting_range":{"min":0,"max":1},"_account_id":9845,"name":"Arista CI","email":"arista-openstack-test@aristanetworks.com","username":"arista-test","tags":["SERVICE_USER"]},{"value":0,"_account_id":24791,"name":"Maciej Jozefczyk","email":"jeicam.pl@gmail.com","username":"maciej.jozefczyk"}],"values":{"-2":"Fails","-1":"Doesn\u0027t seem to work"," 0":"No score","+1":"Works for me","+2":"Verified"},"description":"","default_value":0,"optional":true},"Code-Review":{"approved":{"_account_id":23804,"name":"Daniel Alvarez","email":"dalvarez@redhat.com","username":"dalvarez"},"all":[{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":2,"date":"2020-01-31 10:19:50.000000000","_account_id":23804,"name":"Daniel Alvarez","email":"dalvarez@redhat.com","username":"dalvarez"},{"value":0,"_account_id":26622,"name":"Cloudbase Neutron Hyper-V CI","email":"neutron_hyperv_ci@cloudbasesolutions.com","username":"neutron_hyperv_ci"},{"value":2,"date":"2020-01-29 16:29:18.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},{"value":0,"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},{"value":0,"_account_id":9845,"name":"Arista CI","email":"arista-openstack-test@aristanetworks.com","username":"arista-test","tags":["SERVICE_USER"]},{"value":1,"date":"2020-01-29 10:27:43.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":24791,"name":"Maciej Jozefczyk","email":"jeicam.pl@gmail.com","username":"maciej.jozefczyk"}],"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":{"approved":{"_account_id":23804,"name":"Daniel Alvarez","email":"dalvarez@redhat.com","username":"dalvarez"},"all":[{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":1,"date":"2020-01-31 10:19:50.000000000","_account_id":23804,"name":"Daniel Alvarez","email":"dalvarez@redhat.com","username":"dalvarez"},{"value":0,"_account_id":26622,"name":"Cloudbase Neutron Hyper-V CI","email":"neutron_hyperv_ci@cloudbasesolutions.com","username":"neutron_hyperv_ci"},{"value":0,"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},{"value":0,"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},{"value":0,"_account_id":9845,"name":"Arista CI","email":"arista-openstack-test@aristanetworks.com","username":"arista-test","tags":["SERVICE_USER"]},{"value":0,"_account_id":24791,"name":"Maciej Jozefczyk","email":"jeicam.pl@gmail.com","username":"maciej.jozefczyk"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true},"Review-Priority":{"all":[{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"_account_id":23804,"name":"Daniel Alvarez","email":"dalvarez@redhat.com","username":"dalvarez"},{"value":0,"_account_id":26622,"name":"Cloudbase Neutron Hyper-V CI","email":"neutron_hyperv_ci@cloudbasesolutions.com","username":"neutron_hyperv_ci"},{"value":0,"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},{"value":0,"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},{"value":0,"_account_id":9845,"name":"Arista CI","email":"arista-openstack-test@aristanetworks.com","username":"arista-test","tags":["SERVICE_USER"]},{"value":0,"_account_id":24791,"name":"Maciej Jozefczyk","email":"jeicam.pl@gmail.com","username":"maciej.jozefczyk"}],"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":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},{"_account_id":9845,"name":"Arista CI","email":"arista-openstack-test@aristanetworks.com","username":"arista-test","tags":["SERVICE_USER"]},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"_account_id":23804,"name":"Daniel Alvarez","email":"dalvarez@redhat.com","username":"dalvarez"},{"_account_id":24791,"name":"Maciej Jozefczyk","email":"jeicam.pl@gmail.com","username":"maciej.jozefczyk"},{"_account_id":26622,"name":"Cloudbase Neutron Hyper-V CI","email":"neutron_hyperv_ci@cloudbasesolutions.com","username":"neutron_hyperv_ci"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2020-01-29 10:27:43.000000000","updated_by":{"_account_id":24791,"name":"Maciej Jozefczyk","email":"jeicam.pl@gmail.com","username":"maciej.jozefczyk"},"reviewer":{"_account_id":24791,"name":"Maciej Jozefczyk","email":"jeicam.pl@gmail.com","username":"maciej.jozefczyk"},"state":"REVIEWER"},{"updated":"2020-01-29 11:01:07.000000000","updated_by":{"_account_id":9845,"name":"Arista CI","email":"arista-openstack-test@aristanetworks.com","username":"arista-test","tags":["SERVICE_USER"]},"reviewer":{"_account_id":9845,"name":"Arista CI","email":"arista-openstack-test@aristanetworks.com","username":"arista-test","tags":["SERVICE_USER"]},"state":"REVIEWER"},{"updated":"2020-01-29 11:33:17.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":"2020-01-29 16:29:18.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":"2020-01-31 10:19:50.000000000","updated_by":{"_account_id":23804,"name":"Daniel Alvarez","email":"dalvarez@redhat.com","username":"dalvarez"},"reviewer":{"_account_id":23804,"name":"Daniel Alvarez","email":"dalvarez@redhat.com","username":"dalvarez"},"state":"REVIEWER"},{"updated":"2020-01-31 13:36:51.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"}],"messages":[{"id":"ffb17fbbc78e0b87872753fe2bd67bcdcda6b53c","author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"date":"2020-01-22 15:03:55.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"5be8ddc53ecb4a8a81a66932118063b5aeec7ed2","author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"date":"2020-01-22 15:04:18.000000000","message":"Patch Set 1: Workflow-1\n\nNeeds to be tested on a proper environment","accounts_in_message":[],"_revision_number":1},{"id":"c3959e1b5f5badb86a7df38d022f78cb08367a67","author":{"_account_id":9845,"name":"Arista CI","email":"arista-openstack-test@aristanetworks.com","username":"arista-test","tags":["SERVICE_USER"]},"date":"2020-01-22 17:54:11.000000000","message":"Patch Set 1: Verified+1\n\nArista third party testing PASSED [ http://arista-CI.arista.com:8000/January-2020/Arista_Tempest_Testrefs/changes/13/703813/1 ]\n\nFor testing details or any questions please check https://wiki.openstack.org/wiki/Arista-third-party-testing","accounts_in_message":[],"_revision_number":1},{"id":"19edf1d184e4e2dc5686a19d3aae7e0e2a3b4655","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-01-22 18:54:11.000000000","message":"Patch Set 1: 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 https://zuul.opendev.org/t/openstack/build/ee408647254a4e89bd3190f6941f027c : SUCCESS in 1h 00m 53s\n- neutron-tempest-plugin-designate-scenario https://zuul.opendev.org/t/openstack/build/e924315e72144307b015e73485d5a04e : SUCCESS in 45m 39s\n- neutron-tempest-plugin-dvr-multinode-scenario https://zuul.opendev.org/t/openstack/build/483fdafd763b4362aba50f98363e9097 : FAILURE in 1h 36m 54s (non-voting)\n- neutron-tempest-plugin-scenario-linuxbridge https://zuul.opendev.org/t/openstack/build/65684e8bd9a54779bd7ebc8af8363c19 : SUCCESS in 1h 16m 46s\n- neutron-tempest-plugin-scenario-openvswitch https://zuul.opendev.org/t/openstack/build/13e17bbcf6cf4f6196da30a603990140 : SUCCESS in 1h 27m 59s\n- neutron-tempest-plugin-scenario-openvswitch-iptables_hybrid https://zuul.opendev.org/t/openstack/build/78075e5fefbc4d4e91e1220bb1ee7890 : SUCCESS in 1h 13m 10s\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/05296fb009de4c149960c25a12b4e3ca : SUCCESS in 33m 44s\n- openstack-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/2aaa05dfa421466284e0d7f33ac16213 : SUCCESS in 39m 00s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/f3f8d23ca3ee41f29215c154ef721d09 : FAILURE in 7m 28s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/3047b9a395134be7ac20fd9e151e596c : SUCCESS in 23m 20s\n- openstack-tox-py37 https://zuul.opendev.org/t/openstack/build/5e2db88a23d34fb2ad1ff7d32fe40046 : SUCCESS in 26m 00s\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/aaa16fb72b944dcf9ee2819328dfceb3 : SUCCESS in 9m 14s\n- neutron-functional https://zuul.opendev.org/t/openstack/build/96d9bf71d5374b449b7b6cb2e6fd8df8 : SUCCESS in 45m 54s\n- neutron-fullstack https://zuul.opendev.org/t/openstack/build/0a791983c6984a889f5b23c75577f2e2 : SUCCESS in 1h 19m 59s\n- neutron-rally-task https://zuul.opendev.org/t/openstack/build/c0ff9d696f324c0cb51b1cf9fd410e03 : SUCCESS in 1h 54m 49s\n- neutron-tempest-dvr https://zuul.opendev.org/t/openstack/build/81ca5655d84642f3bac5c5de586368e4 : SUCCESS in 1h 18m 33s\n- neutron-tempest-linuxbridge https://zuul.opendev.org/t/openstack/build/83edbc1f942d448c961016807d9f189b : SUCCESS in 1h 18m 02s\n- neutron-grenade-multinode https://zuul.opendev.org/t/openstack/build/821fd9275102426097282502f240d666 : SUCCESS in 1h 33m 04s\n- neutron-grenade-dvr-multinode https://zuul.opendev.org/t/openstack/build/f254a31a229e48df9e93062105f8985b : SUCCESS in 1h 30m 48s\n- tempest-integrated-networking https://zuul.opendev.org/t/openstack/build/ed87faa834584a37a1670ef2a1fda321 : SUCCESS in 1h 44m 14s\n- tempest-multinode-full-py3 https://zuul.opendev.org/t/openstack/build/f2edb9124789461b810ba90a0bbfb18f : SUCCESS in 2h 02m 23s (non-voting)\n- neutron-tempest-dvr-ha-multinode-full https://zuul.opendev.org/t/openstack/build/6d1750b58ab34577b8c6e04a8572f803 : SUCCESS in 1h 56m 31s (non-voting)\n- neutron-tempest-iptables_hybrid https://zuul.opendev.org/t/openstack/build/3b778d1192c04199bcd88c78f16f7f82 : SUCCESS in 1h 26m 01s\n- ironic-tempest-ipa-wholedisk-bios-agent_ipmitool-tinyipa https://zuul.opendev.org/t/openstack/build/1a0af1f189c24a85b5af21ee9f59199d : SUCCESS in 53m 34s (non-voting)\n- tempest-slow-py3 https://zuul.opendev.org/t/openstack/build/f6eed3e83cb14cde867248a00d35401d : SUCCESS in 2h 13m 24s\n- neutron-ovn-tempest-ovs-release https://zuul.opendev.org/t/openstack/build/1e64848069014c2cbb9da844c0753f19 : SUCCESS in 2h 30m 21s\n- networking-midonet-tempest-aio-ml2-centos-7 https://zuul.opendev.org/t/openstack/build/f50d9fada2514cff987ad633c4130b03 : FAILURE in 17m 23s (non-voting)\n- tempest-ipv6-only https://zuul.opendev.org/t/openstack/build/a25ca412915b41c2aafd3981a253e2d9 : SUCCESS in 1h 06m 44s\n- openstacksdk-functional-devstack-networking https://zuul.opendev.org/t/openstack/build/60bc3a220ed5474785bb6cafac7ce943 : SUCCESS in 1h 17m 04s (non-voting)\n- neutron-functional-with-uwsgi https://zuul.opendev.org/t/openstack/build/bfb744148ebe46bf860a304b73205c05 : SUCCESS in 1h 06m 42s (non-voting)\n- neutron-fullstack-with-uwsgi https://zuul.opendev.org/t/openstack/build/a503760db6774db79d1afe66c19a8ad7 : SUCCESS in 1h 11m 22s (non-voting)\n- neutron-tempest-with-uwsgi https://zuul.opendev.org/t/openstack/build/cf27dd73369647e09a29b8d859ad14da : SUCCESS in 1h 16m 59s (non-voting)\n- neutron-ovn-tempest-slow https://zuul.opendev.org/t/openstack/build/b87c0edfe86047d3ae2dffbe689ede8c : FAILURE in 2h 57m 55s (non-voting)\n- neutron-ovn-tempest-full-multinode-ovs-master https://zuul.opendev.org/t/openstack/build/61f4eb513d7e43feacb94cb5834011a3 : FAILURE in 2h 16m 33s (non-voting)","accounts_in_message":[],"_revision_number":1},{"id":"119e7eb7d390f8dd133d087c0e939fdbf2f9bfd4","author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"date":"2020-01-23 10:33:24.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"aab3040d5c34839dd0713571be3a06de6689c0fd","author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"date":"2020-01-23 10:34:06.000000000","message":"Patch Set 2:\n\nThis has now been tested in a local environment","accounts_in_message":[],"_revision_number":2},{"id":"1656d1e59f9d3eabb95c785c2930d62d15a76961","author":{"_account_id":9845,"name":"Arista CI","email":"arista-openstack-test@aristanetworks.com","username":"arista-test","tags":["SERVICE_USER"]},"date":"2020-01-23 11:55:21.000000000","message":"Patch Set 2: Verified+1\n\nArista third party testing PASSED [ http://arista-CI.arista.com:8000/January-2020/Arista_Tempest_Testrefs/changes/13/703813/2 ]\n\nFor testing details or any questions please check https://wiki.openstack.org/wiki/Arista-third-party-testing","accounts_in_message":[],"_revision_number":2},{"id":"b5c18d70594b96796b1ec61c57753141993032aa","author":{"_account_id":26622,"name":"Cloudbase Neutron Hyper-V CI","email":"neutron_hyperv_ci@cloudbasesolutions.com","username":"neutron_hyperv_ci"},"date":"2020-01-23 12:20:45.000000000","message":"Patch Set 2:\n\nBuild succeeded.\n\n- neutron http://cloudbase-ci.com/neutron/703813/2 : SUCCESS in 1h 46m 50s","accounts_in_message":[],"_revision_number":2},{"id":"15044770b2225c89ca682b9e5c990fe85314290b","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-01-23 13:38:39.000000000","message":"Patch Set 2: Verified+1\n\nBuild succeeded (check pipeline).\n\n- neutron-tempest-plugin-api https://zuul.opendev.org/t/openstack/build/e910c5cdd2004234b386dbec0d7fce0f : SUCCESS in 52m 32s\n- neutron-tempest-plugin-designate-scenario https://zuul.opendev.org/t/openstack/build/ffc579b0bc4842c8af6e80ec0e6c8339 : SUCCESS in 33m 05s\n- neutron-tempest-plugin-dvr-multinode-scenario https://zuul.opendev.org/t/openstack/build/79840e9c44b54849b6239d6c64328b44 : FAILURE in 1h 54m 22s (non-voting)\n- neutron-tempest-plugin-scenario-linuxbridge https://zuul.opendev.org/t/openstack/build/97071872d3ec4a98aef9f2a9a3e48f37 : SUCCESS in 1h 33m 29s\n- neutron-tempest-plugin-scenario-openvswitch https://zuul.opendev.org/t/openstack/build/344275cde0ca473faaef9c54e0945b6b : SUCCESS in 59m 04s\n- neutron-tempest-plugin-scenario-openvswitch-iptables_hybrid https://zuul.opendev.org/t/openstack/build/6505cda94ffe4a3aac40c04c5708283a : SUCCESS in 1h 03m 15s\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/7e8b44c847d849e899c89dde93966ebb : SUCCESS in 30m 13s\n- openstack-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/39fa4379961a4fd7b1eee93c4bd46108 : SUCCESS in 28m 06s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/4b655c573b064c03bfeccf441a4b9b67 : SUCCESS in 12m 56s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/89ceae2922794364bca6f14f96b10ebd : SUCCESS in 29m 23s\n- openstack-tox-py37 https://zuul.opendev.org/t/openstack/build/1175655794554445bb640d53e0111696 : SUCCESS in 33m 18s\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/63bf9b2d7ae545b6bac4e16ec8237fd6 : SUCCESS in 8m 10s\n- neutron-functional https://zuul.opendev.org/t/openstack/build/10f4c3ccdffb4e59a8aa97bab6feae0e : SUCCESS in 50m 11s\n- neutron-fullstack https://zuul.opendev.org/t/openstack/build/65abefa6e2134334acac168171abce07 : SUCCESS in 1h 08m 48s\n- neutron-rally-task https://zuul.opendev.org/t/openstack/build/720296f1edb34d998522ba0befd2fe81 : SUCCESS in 2h 04m 46s\n- neutron-tempest-dvr https://zuul.opendev.org/t/openstack/build/0b2aaab3108d4d16925a2cd54dc9352b : SUCCESS in 1h 09m 51s\n- neutron-tempest-linuxbridge https://zuul.opendev.org/t/openstack/build/fae38370f2f147d59271a0c37a19c3a4 : SUCCESS in 1h 06m 09s\n- neutron-grenade-multinode https://zuul.opendev.org/t/openstack/build/c520dd6c1f6d47c091b296164ff9d732 : SUCCESS in 1h 08m 46s\n- neutron-grenade-dvr-multinode https://zuul.opendev.org/t/openstack/build/8d209c08474e4aa7a2596de81cba8151 : SUCCESS in 1h 08m 17s\n- tempest-integrated-networking https://zuul.opendev.org/t/openstack/build/718bfff32f7946eaa5d56f66cb52b14d : SUCCESS in 1h 09m 23s\n- tempest-multinode-full-py3 https://zuul.opendev.org/t/openstack/build/8424462e92784e9b898271c996cfe254 : SUCCESS in 1h 35m 30s (non-voting)\n- neutron-tempest-dvr-ha-multinode-full https://zuul.opendev.org/t/openstack/build/35fb43ea0f3342ad8aa9ca8cf1f9e1fb : SUCCESS in 2h 05m 14s (non-voting)\n- neutron-tempest-iptables_hybrid https://zuul.opendev.org/t/openstack/build/d71aa742dd284131b9a0d86ce9bf1681 : SUCCESS in 1h 34m 46s\n- ironic-tempest-ipa-wholedisk-bios-agent_ipmitool-tinyipa https://zuul.opendev.org/t/openstack/build/31c7fed27434478bae765a71b554ab91 : SUCCESS in 56m 11s (non-voting)\n- tempest-slow-py3 https://zuul.opendev.org/t/openstack/build/f28e7275e77f423288bfd047449686b9 : SUCCESS in 2h 28m 38s\n- neutron-ovn-tempest-ovs-release https://zuul.opendev.org/t/openstack/build/1d1971012607447b86b0a8380efd02cd : SUCCESS in 2h 05m 23s\n- networking-midonet-tempest-aio-ml2-centos-7 https://zuul.opendev.org/t/openstack/build/c04dae79d08245fb98c300c4a67c08b0 : FAILURE in 15m 04s (non-voting)\n- tempest-ipv6-only https://zuul.opendev.org/t/openstack/build/71545039d7044fbfad419de1609480af : SUCCESS in 1h 02m 23s\n- openstacksdk-functional-devstack-networking https://zuul.opendev.org/t/openstack/build/efc02df7b11c40f8abc4402f92cc5bac : SUCCESS in 1h 07m 53s (non-voting)\n- neutron-functional-with-uwsgi https://zuul.opendev.org/t/openstack/build/7c5c091a1ff74031acf6a7a68ad80e54 : SUCCESS in 43m 21s (non-voting)\n- neutron-fullstack-with-uwsgi https://zuul.opendev.org/t/openstack/build/9ccb1798d9d1440a912ef09c7721f84a : SUCCESS in 1h 11m 34s (non-voting)\n- neutron-tempest-with-uwsgi https://zuul.opendev.org/t/openstack/build/351b889413a04825b896cd09f6d70305 : SUCCESS in 1h 25m 20s (non-voting)\n- neutron-ovn-tempest-slow https://zuul.opendev.org/t/openstack/build/05e8b1ab9a194767a212ea2dff573da8 : FAILURE in 2h 55m 34s (non-voting)\n- neutron-ovn-tempest-full-multinode-ovs-master https://zuul.opendev.org/t/openstack/build/130680ef6b7e4cf9a90cf05b2d1ac7e9 : FAILURE in 2h 40m 27s (non-voting)","accounts_in_message":[],"_revision_number":2},{"id":"0b58051fd9d6a52a35ca69eb81d3bcace04ea866","author":{"_account_id":24791,"name":"Maciej Jozefczyk","email":"jeicam.pl@gmail.com","username":"maciej.jozefczyk"},"date":"2020-01-28 13:55:11.000000000","message":"Patch Set 2: Code-Review-1\n\nHum. I\u0027m a bit worry about this. The change that introduced in [1]\nis pretty fresh. Lets ask Brian if he has an idea if that is fine now or not.\n\n[1] https://github.com/openstack/networking-ovn/commit/c5aef51edc9843db605303ec8bd8610b6c55e9c2","accounts_in_message":[],"_revision_number":2},{"id":"dc06dd6a896612cb4d044ee3ac2f93565b50dfda","author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"date":"2020-01-28 14:38:28.000000000","message":"Patch Set 2:\n\n\u003e Hum. I\u0027m a bit worry about this. The change that introduced in [1]\n \u003e is pretty fresh. Lets ask Brian if he has an idea if that is fine\n \u003e now or not.\n \u003e \n \u003e [1] https://github.com/openstack/networking-ovn/commit/c5aef51edc9843db605303ec8bd8610b6c55e9c2\n\n++ for Brian\u0027s input.\n\nIMHO that patch is not correct, for two reasons:\n\n1) That conditional is also blocking VLAN provider networks. In the bug linked with this patch you can see that the user have a geneve tenant network and a VLAN provider network where the FIPs are.\n\n2) We have an event that set/unset the external_mac attribute based on the port getting up and down [0] and that is not checking for any network type. So, in the bug Andreas could workaround the DVR problem with the VLAN provider network by rebooting the node, which would bring the port down and then up again and the external_mac would be set.\n\n[0] https://github.com/openstack/networking-ovn/blob/eda5d7f80d877601170631c5f5485370ea701f42/networking_ovn/ml2/mech_driver.py#L794-L804","accounts_in_message":[],"_revision_number":2},{"id":"05da07746cb35c03e5393362502f5209269944d6","author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"date":"2020-01-28 14:39:25.000000000","message":"Patch Set 2:\n\n\u003e \u003e Hum. I\u0027m a bit worry about this. The change that introduced in\n \u003e [1]\n \u003e \u003e is pretty fresh. Lets ask Brian if he has an idea if that is fine\n \u003e \u003e now or not.\n \u003e \u003e\n \u003e \u003e [1] https://github.com/openstack/networking-ovn/commit/c5aef51edc9843db605303ec8bd8610b6c55e9c2\n \u003e \n \u003e ++ for Brian\u0027s input.\n \u003e \n \u003e IMHO that patch is not correct, for two reasons:\n \u003e \n \u003e 1) That conditional is also blocking VLAN provider networks. In the\n \u003e bug linked with this patch you can see that the user have a geneve\n \u003e tenant network and a VLAN provider network where the FIPs are.\n \u003e \n \u003e 2) We have an event that set/unset the external_mac attribute based\n \u003e on the port getting up and down [0] and that is not checking for\n \u003e any network type. So, in the bug Andreas could workaround the DVR\n \u003e problem with the VLAN provider network by rebooting the node, which\n \u003e would bring the port down and then up again and the external_mac\n \u003e would be set.\n \u003e \n \u003e [0] https://github.com/openstack/networking-ovn/blob/eda5d7f80d877601170631c5f5485370ea701f42/networking_ovn/ml2/mech_driver.py#L794-L804\n\nBy \"The node\" I mean \"the VM\"","accounts_in_message":[],"_revision_number":2},{"id":"cf4c1c8e5e6a9c203740560cc6a15f3fd5804c8c","author":{"_account_id":24791,"name":"Maciej Jozefczyk","email":"jeicam.pl@gmail.com","username":"maciej.jozefczyk"},"date":"2020-01-28 14:45:09.000000000","message":"Patch Set 2: -Code-Review\n\n\u003e \u003e \u003e Hum. I\u0027m a bit worry about this. The change that introduced in\n \u003e \u003e [1]\n \u003e \u003e \u003e is pretty fresh. Lets ask Brian if he has an idea if that is\n \u003e fine\n \u003e \u003e \u003e now or not.\n \u003e \u003e \u003e\n \u003e \u003e \u003e [1] https://github.com/openstack/networking-ovn/commit/c5aef51edc9843db605303ec8bd8610b6c55e9c2\n \u003e \u003e\n \u003e \u003e ++ for Brian\u0027s input.\n \u003e \u003e\n \u003e \u003e IMHO that patch is not correct, for two reasons:\n \u003e \u003e\n \u003e \u003e 1) That conditional is also blocking VLAN provider networks. In\n \u003e the\n \u003e \u003e bug linked with this patch you can see that the user have a\n \u003e geneve\n \u003e \u003e tenant network and a VLAN provider network where the FIPs are.\n \u003e \u003e\n \u003e \u003e 2) We have an event that set/unset the external_mac attribute\n \u003e based\n \u003e \u003e on the port getting up and down [0] and that is not checking for\n \u003e \u003e any network type. So, in the bug Andreas could workaround the DVR\n \u003e \u003e problem with the VLAN provider network by rebooting the node,\n \u003e which\n \u003e \u003e would bring the port down and then up again and the external_mac\n \u003e \u003e would be set.\n \u003e \u003e\n \u003e \u003e [0] https://github.com/openstack/networking-ovn/blob/eda5d7f80d877601170631c5f5485370ea701f42/networking_ovn/ml2/mech_driver.py#L794-L804\n \u003e \n \u003e By \"The node\" I mean \"the VM\"\n\nOk, that makes sense. Lets wait for Brian input.","accounts_in_message":[],"_revision_number":2},{"id":"bd5f1e34c47d1c54065e6eb7eb79d77378f9b4da","author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"date":"2020-01-28 15:18:48.000000000","message":"Patch Set 2:\n\nIt does look like my fix was wrong, I\u0027m fine with this \"revert\", maybe we should call it that in the commit message?  This was also cherry-picked into the stable branches as well.\n\nI guess we\u0027ll have to take another look at the bug it was fixing, https://bugs.launchpad.net/networking-ovn/+bug/1828891 - it also has a bugzilla.","accounts_in_message":[],"_revision_number":2},{"id":"6138d9c0f5eaae640fc778728a32cc6eb8ff2497","author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"date":"2020-01-28 16:17:51.000000000","message":"Patch Set 2:\n\n\u003e It does look like my fix was wrong, I\u0027m fine with this \"revert\",\n \u003e maybe we should call it that in the commit message?  This was also\n \u003e cherry-picked into the stable branches as well.\n \u003e \n \u003e I guess we\u0027ll have to take another look at the bug it was fixing,\n \u003e https://bugs.launchpad.net/networking-ovn/+bug/1828891 - it also\n \u003e has a bugzilla.\n\nThanks a lot for taking a look Brian!\n\nSure, I can state this is some sort of \"revert\" in the commit message.","accounts_in_message":[],"_revision_number":2},{"id":"b648f9055202ed624cea78da443ccd1bf9ee149f","author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"date":"2020-01-29 09:39:42.000000000","message":"Patch Set 2: Workflow-1","accounts_in_message":[],"_revision_number":2},{"id":"61eaf55f6500fa49d798ceaae422d410c8e11331","author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"date":"2020-01-29 09:46:05.000000000","message":"Uploaded patch set 3.","accounts_in_message":[],"_revision_number":3},{"id":"6a6a21826c82d177f48282dcc267f7627527da14","author":{"_account_id":24791,"name":"Maciej Jozefczyk","email":"jeicam.pl@gmail.com","username":"maciej.jozefczyk"},"date":"2020-01-29 10:27:43.000000000","message":"Patch Set 3: Code-Review+1\n\nThanks!","accounts_in_message":[],"_revision_number":3},{"id":"06271bbfa91db12725e518e3d62d5f07e0b360bb","author":{"_account_id":9845,"name":"Arista CI","email":"arista-openstack-test@aristanetworks.com","username":"arista-test","tags":["SERVICE_USER"]},"date":"2020-01-29 11:01:07.000000000","message":"Patch Set 3:\n\nArista third party testing FAILED [ http://arista-CI.arista.com:8000/January-2020/Arista_Tempest_Testrefs/changes/13/703813/3 ]\n\nFor testing details or any questions please check https://wiki.openstack.org/wiki/Arista-third-party-testing","accounts_in_message":[],"_revision_number":3},{"id":"dc8365132164dfaa3f1ac81c489e6718ea177aa1","author":{"_account_id":26622,"name":"Cloudbase Neutron Hyper-V CI","email":"neutron_hyperv_ci@cloudbasesolutions.com","username":"neutron_hyperv_ci"},"date":"2020-01-29 11:33:17.000000000","message":"Patch Set 3:\n\nFor rechecking only on the Cloudbase Neutron Hyper-V CI, add a review comment with run-Cloudbase Neutron Hyper-V CI\n\n- neutron http://cloudbase-ci.com/neutron/703813/3 : FAILURE in 1h 46m 00s","accounts_in_message":[],"_revision_number":3},{"id":"04625dcecadb175d30b1d13990b47d6ec4551490","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-01-29 12:43:34.000000000","message":"Patch Set 3: Verified+1\n\nBuild succeeded (check pipeline).\n\n- neutron-tempest-plugin-api https://zuul.opendev.org/t/openstack/build/b8d1d7b3a4c44cfc82a1b76bbdd7638d : SUCCESS in 58m 53s\n- neutron-tempest-plugin-designate-scenario https://zuul.opendev.org/t/openstack/build/72bb37e0c4c041b69c79322d6e8cd1ee : SUCCESS in 40m 08s\n- neutron-tempest-plugin-dvr-multinode-scenario https://zuul.opendev.org/t/openstack/build/b8533e62798f481683dc778d874abe80 : SUCCESS in 1h 37m 02s (non-voting)\n- neutron-tempest-plugin-scenario-linuxbridge https://zuul.opendev.org/t/openstack/build/e42cabf823d142b4a2dd35bb0fa36e75 : SUCCESS in 1h 11m 26s\n- neutron-tempest-plugin-scenario-openvswitch https://zuul.opendev.org/t/openstack/build/36e115fb1db343138ad55370051047ff : SUCCESS in 1h 00m 26s\n- neutron-tempest-plugin-scenario-openvswitch-iptables_hybrid https://zuul.opendev.org/t/openstack/build/fd0871452329487a815347f1c57f396b : SUCCESS in 1h 09m 53s\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/c6cbd7a8c10646d6838010e4c2ca355f : SUCCESS in 38m 03s\n- openstack-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/109e3b305c134c989917be18d420c4d2 : SUCCESS in 27m 31s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/53f3f1eaa441473b90b48bbccc3a7744 : SUCCESS in 14m 08s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/e6dc16c00a814f91b65316978e3df2ba : SUCCESS in 32m 23s\n- openstack-tox-py37 https://zuul.opendev.org/t/openstack/build/8a728ed8238f4b3e86e2a3c95ce0065e : SUCCESS in 31m 46s\n- openstack-tox-py38 https://zuul.opendev.org/t/openstack/build/6b37cd3c205b42fe93da7c210cdff809 : SUCCESS in 26m 02s (non-voting)\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/cb90f10cd1df47ecab8f628e47c95831 : SUCCESS in 9m 44s\n- neutron-functional https://zuul.opendev.org/t/openstack/build/a953217b8e044cb3a49a862bdfd0c585 : SUCCESS in 43m 49s\n- neutron-fullstack https://zuul.opendev.org/t/openstack/build/390ff90d4ddd47a99a025d328f1e2a72 : SUCCESS in 1h 03m 10s\n- neutron-rally-task https://zuul.opendev.org/t/openstack/build/feef0486e151483085b47ab5b8d9ddbd : SUCCESS in 2h 55m 39s\n- neutron-tempest-dvr https://zuul.opendev.org/t/openstack/build/885b2bfccc914a2a9b4083b57bd15169 : SUCCESS in 1h 44m 02s\n- neutron-tempest-linuxbridge https://zuul.opendev.org/t/openstack/build/7d70000171144ef1a45689be39d21ed5 : SUCCESS in 1h 02m 40s\n- neutron-grenade-multinode https://zuul.opendev.org/t/openstack/build/4f3c6afc7f6c4f0ba89de0d6726bafaf : SUCCESS in 59m 24s\n- neutron-grenade-dvr-multinode https://zuul.opendev.org/t/openstack/build/6e9de326b6eb45bd99eb667ad96d015f : SUCCESS in 1h 02m 36s\n- tempest-integrated-networking https://zuul.opendev.org/t/openstack/build/bf2fe422d96f4a5bbf084c24e9b8a2b3 : SUCCESS in 1h 30m 32s\n- tempest-multinode-full-py3 https://zuul.opendev.org/t/openstack/build/adb0bdb75b004c53b57b549fca384df7 : SUCCESS in 2h 21m 36s (non-voting)\n- neutron-tempest-dvr-ha-multinode-full https://zuul.opendev.org/t/openstack/build/4db5b5d8466b43afab5433c053b41822 : SUCCESS in 1h 55m 51s (non-voting)\n- neutron-tempest-iptables_hybrid https://zuul.opendev.org/t/openstack/build/5af614cf6b334d05ae935072ed35b836 : SUCCESS in 1h 18m 10s\n- ironic-tempest-ipa-wholedisk-bios-agent_ipmitool-tinyipa https://zuul.opendev.org/t/openstack/build/065dd3a7a3944fe4ae4b30d0e465628e : SUCCESS in 59m 58s (non-voting)\n- tempest-slow-py3 https://zuul.opendev.org/t/openstack/build/ace785bbeb5d4821a9603c99f3a99f25 : SUCCESS in 2h 16m 58s\n- neutron-ovn-tempest-ovs-release https://zuul.opendev.org/t/openstack/build/73ae003c4ecb4b709d19dc1bd757397b : SUCCESS in 1h 56m 48s\n- tempest-ipv6-only https://zuul.opendev.org/t/openstack/build/1947b2f6a76d4bf2ae79d698754d99b8 : SUCCESS in 1h 09m 27s\n- openstacksdk-functional-devstack-networking https://zuul.opendev.org/t/openstack/build/646bb8302e404ffda3c71ca0edc82aa7 : SUCCESS in 1h 13m 00s (non-voting)\n- neutron-functional-with-uwsgi https://zuul.opendev.org/t/openstack/build/e5072dca19c2431dbb7d0e5f91ee7bb9 : SUCCESS in 42m 08s (non-voting)\n- neutron-fullstack-with-uwsgi https://zuul.opendev.org/t/openstack/build/f067c036a76746d082fcc80f11cc5f08 : SUCCESS in 1h 09m 10s (non-voting)\n- neutron-tempest-with-uwsgi https://zuul.opendev.org/t/openstack/build/784c6c147ed04f58a8f11b4ff7bc8d0d : SUCCESS in 1h 13m 55s (non-voting)\n- neutron-ovn-tempest-slow https://zuul.opendev.org/t/openstack/build/49c9b6fa189347a79031e30050b4769d : FAILURE in 2h 24m 15s (non-voting)\n- neutron-ovn-tempest-full-multinode-ovs-master https://zuul.opendev.org/t/openstack/build/6a40ec4e95b842e39a38a434243fb361 : FAILURE in 2h 09m 49s (non-voting)","accounts_in_message":[],"_revision_number":3},{"id":"fc835b36d3d226945a04b98e6fd45adf838896bf","author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"date":"2020-01-29 16:29:18.000000000","message":"Patch Set 3: Code-Review+2","accounts_in_message":[],"_revision_number":3},{"id":"b357f86dd0fe76f8222ec55e891a10e38f287326","author":{"_account_id":23804,"name":"Daniel Alvarez","email":"dalvarez@redhat.com","username":"dalvarez"},"date":"2020-01-31 10:19:50.000000000","message":"Patch Set 3: Code-Review+2 Workflow+1","accounts_in_message":[],"_revision_number":3},{"id":"c54db029892bbec120707ea99772f7c60699e7f0","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-01-31 10:20:11.000000000","message":"Patch Set 3: -Verified\n\nStarting gate jobs.","accounts_in_message":[],"_revision_number":3},{"id":"f8bc2a1842de20e7d9bded40658cd2bc6b9c94c8","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-01-31 13:36:51.000000000","message":"Patch Set 3: Verified+2\n\nBuild succeeded (gate pipeline).\n\n- neutron-tempest-plugin-api https://zuul.opendev.org/t/openstack/build/161bf7bb37fa456190c9c58f536bdd92 : SUCCESS in 57m 31s\n- neutron-tempest-plugin-scenario-linuxbridge https://zuul.opendev.org/t/openstack/build/dab6bb45f4a74e17b2daf1de7995bf4b : SUCCESS in 1h 19m 36s\n- neutron-tempest-plugin-scenario-openvswitch https://zuul.opendev.org/t/openstack/build/e9c70b7aaf134bd394e6a995a87d22d1 : SUCCESS in 1h 06m 26s\n- neutron-tempest-plugin-scenario-openvswitch-iptables_hybrid https://zuul.opendev.org/t/openstack/build/098e858d2f6f4f38aa7f13934a891232 : SUCCESS in 57m 37s\n- openstack-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/71cfeaa2b7e746939fa557cb6688c9b6 : SUCCESS in 30m 49s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/005fbc09ba904ff2bab8f3e0d085fba1 : SUCCESS in 13m 02s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/a23e2d3256614f009f32f4211902e1bb : SUCCESS in 28m 42s\n- openstack-tox-py37 https://zuul.opendev.org/t/openstack/build/dec7cc24603440938a01e026576a321d : SUCCESS in 30m 43s\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/d439545496954165a70d15f6d8c86448 : SUCCESS in 8m 27s\n- neutron-functional https://zuul.opendev.org/t/openstack/build/449a825fc7ab47ff91d8d09160a4cff3 : SUCCESS in 52m 21s\n- neutron-fullstack https://zuul.opendev.org/t/openstack/build/b3508c7d45b44c10915e162957b15786 : SUCCESS in 1h 05m 42s\n- tempest-integrated-networking https://zuul.opendev.org/t/openstack/build/7a81756fe8994b3caf3bedcbb5721924 : SUCCESS in 1h 12m 16s\n- neutron-tempest-dvr https://zuul.opendev.org/t/openstack/build/51eed7d5b6c14e9782533230debab0d5 : SUCCESS in 1h 18m 41s\n- neutron-tempest-linuxbridge https://zuul.opendev.org/t/openstack/build/a403a712e27d44f98f786008ba5592b6 : SUCCESS in 1h 06m 16s\n- neutron-tempest-iptables_hybrid https://zuul.opendev.org/t/openstack/build/fbac73b87ebc43ed90af6dfce64a475c : SUCCESS in 1h 17m 07s\n- neutron-grenade-multinode https://zuul.opendev.org/t/openstack/build/49c5e5cd446e4dc8adcf4620dbfbc0c0 : SUCCESS in 1h 07m 09s\n- neutron-grenade-dvr-multinode https://zuul.opendev.org/t/openstack/build/bc82b8a5617542b68be6f9f1b52be825 : SUCCESS in 1h 06m 52s\n- tempest-slow-py3 https://zuul.opendev.org/t/openstack/build/9b53188e2e6d4b2d855cc62edf53814c : SUCCESS in 3h 03m 09s\n- tempest-ipv6-only https://zuul.opendev.org/t/openstack/build/c6263c899b244d7fac010507c89c1ad0 : SUCCESS in 1h 05m 52s\n- neutron-ovn-tempest-ovs-release https://zuul.opendev.org/t/openstack/build/e08affa12cfe4f40b84409d1e8cc8108 : SUCCESS in 1h 40m 03s","accounts_in_message":[],"_revision_number":3},{"id":"f6c82d7c9ff6135d992d6d7e64b9dc6f88db49e7","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-01-31 13:36:51.000000000","message":"Change has been successfully merged by Zuul","accounts_in_message":[],"_revision_number":3},{"id":"d5fc8c2ab832a59342649c3b4b819db55f588149","tag":"autogenerated:zuul:promote","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-01-31 13:39:08.000000000","message":"Patch Set 3:\n\nBuild succeeded (promote pipeline).\n\n- promote-openstack-tox-docs https://zuul.opendev.org/t/openstack/build/23856cc541024564a8f56c8fa1c2a6ae : SUCCESS in 1m 36s","accounts_in_message":[],"_revision_number":3}],"current_revision_number":3,"current_revision":"c471c7330c6c7a642e11ceae5fd604177059a3e8","revisions":{"09b02b7db9aee1f9d8c4e02c24f0ae464658318d":{"kind":"REWORK","_number":1,"created":"2020-01-22 15:03:55.000000000","uploader":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"ref":"refs/changes/13/703813/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/neutron","ref":"refs/changes/13/703813/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/neutron refs/changes/13/703813/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/neutron refs/changes/13/703813/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/neutron refs/changes/13/703813/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/neutron refs/changes/13/703813/1"}}},"commit":{"parents":[{"commit":"4b48de8e8895cce4c5496f41be3679c1f87297fd","subject":"Merge \"Allow to kill keepalived state change monitor process\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/neutron/commit/4b48de8e8895cce4c5496f41be3679c1f87297fd"}]}],"author":{"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","date":"2020-01-22 15:00:32.000000000","tz":0},"committer":{"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","date":"2020-01-22 15:03:34.000000000","tz":0},"subject":"[OVN] Remove VLAN check when setting external_mac","message":"[OVN] Remove VLAN check when setting external_mac\n\nCurrently in the code there\u0027s an incosistency to how we set the\n\"external_mac\" field in the NAT table from OVN NB DB.\n\nWhen we created the FIP, the code is checking for the network type and\nnot setting the \"external_mac\" if its VLAN. I believe this was due to\nthe VLAN type not playing nice with DVR in the past.\n\nNow, when events are generated for the port DOWN/UP and the\n\"external_mac\" field gets cleared [0], this network type is not set.\n\nFor more context see bug #1842988.\n\n[0]\nhttps://github.com/openstack/networking-ovn/blob/eda5d7f80d877601170631c5f5485370ea701f42/networking_ovn/ml2/mech_driver.py#L794-L800\n\nChange-Id: Ifb795626dc9c2ac4f0104f491dd38c9b4cc902c9\nCloses-Bug: #1842988\nSigned-off-by: Lucas Alvares Gomes \u003clucasagomes@gmail.com\u003e\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/neutron/commit/09b02b7db9aee1f9d8c4e02c24f0ae464658318d"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/neutron/commit/09b02b7db9aee1f9d8c4e02c24f0ae464658318d"}]},"branch":"refs/heads/master"},"cf3d6918453779203ede0e700ee10a4bc1a77c8c":{"kind":"REWORK","_number":2,"created":"2020-01-23 10:33:24.000000000","uploader":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"ref":"refs/changes/13/703813/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/neutron","ref":"refs/changes/13/703813/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/neutron refs/changes/13/703813/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/neutron refs/changes/13/703813/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/neutron refs/changes/13/703813/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/neutron refs/changes/13/703813/2"}}},"commit":{"parents":[{"commit":"4b48de8e8895cce4c5496f41be3679c1f87297fd","subject":"Merge \"Allow to kill keepalived state change monitor process\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/neutron/commit/4b48de8e8895cce4c5496f41be3679c1f87297fd"}]}],"author":{"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","date":"2020-01-22 15:00:32.000000000","tz":0},"committer":{"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","date":"2020-01-23 10:33:12.000000000","tz":0},"subject":"[OVN] Remove VLAN check when setting external_mac","message":"[OVN] Remove VLAN check when setting external_mac\n\nCurrently in the code there\u0027s an incosistency to how we set the\n\"external_mac\" field in the NAT table from OVN NB DB.\n\nWhen we created the FIP, the code is checking for the network type and\nnot setting the \"external_mac\" if its VLAN. I believe this was due to\nthe VLAN type not playing nice with DVR in the past.\n\nNow, when events are generated for the port DOWN/UP and the\n\"external_mac\" field gets cleared [0], this network type is not set.\n\nFor more context see bug #1842988.\n\n[0]\nhttps://github.com/openstack/networking-ovn/blob/eda5d7f80d877601170631c5f5485370ea701f42/networking_ovn/ml2/mech_driver.py#L794-L800\n\nChange-Id: Ifb795626dc9c2ac4f0104f491dd38c9b4cc902c9\nCloses-Bug: #1842988\nSigned-off-by: Lucas Alvares Gomes \u003clucasagomes@gmail.com\u003e\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/neutron/commit/cf3d6918453779203ede0e700ee10a4bc1a77c8c"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/neutron/commit/cf3d6918453779203ede0e700ee10a4bc1a77c8c"}]},"branch":"refs/heads/master"},"c471c7330c6c7a642e11ceae5fd604177059a3e8":{"kind":"TRIVIAL_REBASE_WITH_MESSAGE_UPDATE","_number":3,"created":"2020-01-29 09:46:05.000000000","uploader":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"ref":"refs/changes/13/703813/3","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/neutron","ref":"refs/changes/13/703813/3","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/neutron refs/changes/13/703813/3 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/neutron refs/changes/13/703813/3 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/neutron refs/changes/13/703813/3 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/neutron refs/changes/13/703813/3"}}},"commit":{"parents":[{"commit":"f173e2ee089e3bc0aae741e343f47991494c0a97","subject":"Merge \"[OVN] scripts for networking-ovn code migration\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/neutron/commit/f173e2ee089e3bc0aae741e343f47991494c0a97"}]}],"author":{"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","date":"2020-01-22 15:00:32.000000000","tz":0},"committer":{"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","date":"2020-01-29 09:40:20.000000000","tz":0},"subject":"[OVN] Remove VLAN check when setting external_mac","message":"[OVN] Remove VLAN check when setting external_mac\n\nThis patch reverts [0].\n\nThe code wasn\u0027t accounting for VLAN provider networks, as stated in the\nbug #1842988, DVR won\u0027t work if the provider network (where the FIP is\ncreated) is VLAN.\n\nThere was also an incosistency in how the external_mac was set for the\nVLAN networks. Upon creating the FIP the code was checking for the\nnetwork type and not setting the external_mac attribute in case the\nnetwork was VLAN type. But, if the port went down and up again (e.g if\nyou reboot the VM) the event handler that set/unset the external_mac [1]\nwasn\u0027t check for the type. This is how people worked around the DVR\nproblem (as stated in bug #1842988).\n\nFor more information see bug #1842988.\n\n[0]\nhttps://github.com/openstack/networking-ovn/commit/c5aef51edc9843db605303ec8bd8610b6c55e9c2\n[1]\nhttps://github.com/openstack/networking-ovn/blob/eda5d7f80d877601170631c5f5485370ea701f42/networking_ovn/ml2/mech_driver.py#L794-L800\n\nChange-Id: Ifb795626dc9c2ac4f0104f491dd38c9b4cc902c9\nCloses-Bug: #1842988\nSigned-off-by: Lucas Alvares Gomes \u003clucasagomes@gmail.com\u003e\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/neutron/commit/c471c7330c6c7a642e11ceae5fd604177059a3e8"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/neutron/commit/c471c7330c6c7a642e11ceae5fd604177059a3e8"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}