)]}'
{"id":"openstack%2Fdevstack~791085","triplet_id":"openstack%2Fdevstack~master~I39ec9ce0a91bea05cf8c446a9767ab879ac8e8f3","project":"openstack/devstack","branch":"master","topic":"ovn-devstack","hashtags":[],"change_id":"I39ec9ce0a91bea05cf8c446a9767ab879ac8e8f3","subject":"OVN : include source compliation functions","status":"MERGED","created":"2021-05-13 03:17:29.000000000","updated":"2021-06-01 22:27:10.000000000","submitted":"2021-06-01 22:24:53.000000000","submitter":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"total_comment_count":7,"unresolved_comment_count":1,"has_review_started":true,"submission_id":"791085-ovn-devstack","meta_rev_id":"31b78dedc2b800a117ecdc896a1652af33033e36","_number":791085,"virtual_id_number":791085,"owner":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"tag":"autogenerated:zuul:gate","value":2,"date":"2021-06-01 22:24:52.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},{"value":0,"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},{"value":0,"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},{"value":0,"_account_id":11952,"name":"Flavio Fernandes","email":"flavio@flaviof.com","username":"ffernand"}],"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":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"all":[{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":2,"date":"2021-06-01 17:05:52.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},{"value":2,"date":"2021-05-28 08:06:43.000000000","_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},{"value":0,"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},{"value":1,"date":"2021-06-01 13:49:06.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":11952,"name":"Flavio Fernandes","email":"flavio@flaviof.com","username":"ffernand"}],"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":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"all":[{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":1,"date":"2021-06-01 17:05:52.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},{"value":0,"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},{"value":0,"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},{"value":0,"_account_id":11952,"name":"Flavio Fernandes","email":"flavio@flaviof.com","username":"ffernand"}],"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":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},{"value":0,"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},{"value":0,"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},{"value":0,"_account_id":11952,"name":"Flavio Fernandes","email":"flavio@flaviof.com","username":"ffernand"}],"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":{"CC":[{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"}],"REVIEWER":[{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},{"_account_id":11952,"name":"Flavio Fernandes","email":"flavio@flaviof.com","username":"ffernand"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2021-05-13 04:56:49.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":"2021-05-13 13:36:04.000000000","updated_by":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"reviewer":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"state":"REVIEWER"},{"updated":"2021-05-13 14:10:48.000000000","updated_by":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"reviewer":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"state":"CC"},{"updated":"2021-05-13 16:59:07.000000000","updated_by":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"reviewer":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"state":"REVIEWER"},{"updated":"2021-05-18 15:43:02.000000000","updated_by":{"_account_id":11952,"name":"Flavio Fernandes","email":"flavio@flaviof.com","username":"ffernand"},"reviewer":{"_account_id":11952,"name":"Flavio Fernandes","email":"flavio@flaviof.com","username":"ffernand"},"state":"REVIEWER"},{"updated":"2021-05-27 08:27:30.000000000","updated_by":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"reviewer":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"state":"CC"},{"updated":"2021-05-27 17:27:52.000000000","updated_by":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"reviewer":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"state":"CC"},{"updated":"2021-05-28 08:06:43.000000000","updated_by":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"reviewer":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"state":"REVIEWER"}],"messages":[{"id":"5a7a7d10b8722a76ce18784f9936db350c9ca205","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"date":"2021-05-13 03:17:29.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"4d694401e40d9a37f6e209bd16ebfbb37e67fd27","author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"date":"2021-05-13 03:19:22.000000000","message":"Patch Set 1:\n\nWe are seeing failures in the nodepool jobs that install devstack, e.g. https://zuul.opendev.org/t/openstack/build/b6b7910f1c6248bf891cb67b0f9c0fe4/console\n\nIt\u0027s an open question as to why the gate testing didn\u0027t pick this up.","accounts_in_message":[],"_revision_number":1},{"id":"e195ec76ac75cc75525efc0a8399f05bef3419d4","author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"date":"2021-05-13 03:25:32.000000000","message":"Patch Set 1:\n\n\u003e Patch Set 1:\n\u003e It\u0027s an open question as to why the gate testing didn\u0027t pick this up.\n\nAhh, because the gate jobs pre-setup the /opt/stack, which other jobs, such as nodepool (or someone downloading it) don\u0027t do.\n\n[1] https://opendev.org/openstack/devstack/src/branch/master/roles/setup-devstack-source-dirs/tasks/main.yaml#L7","accounts_in_message":[],"_revision_number":1},{"id":"a8de3b88170450230db018eea356a95838dd5c1b","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-05-13 04:56:49.000000000","message":"Patch Set 1: Verified-1\n\n(6 comments)\n\nBuild failed (check pipeline).  For information on how to proceed, see\nhttps://docs.opendev.org/opendev/infra-manual/latest/developers.html#automated-testing\n\n\n- grenade https://zuul.opendev.org/t/openstack/build/843ce6d2d0e34444abd088ca1a160874 : SUCCESS in 1h 00m 55s\n- tempest-full-py3 https://zuul.opendev.org/t/openstack/build/bae0aa4c731a405f9eefd1e1083d9ebb : SUCCESS in 1h 11m 33s\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/ee8ec9a5ab744717a9fdd8c2feb192d9 : SUCCESS in 7m 01s\n- devstack https://zuul.opendev.org/t/openstack/build/d677c7c95a8b4f61ad2be37b2569ad7a : SUCCESS in 31m 34s\n- devstack-ipv6 https://zuul.opendev.org/t/openstack/build/914ad0c6ab574474848b1b91109fff8d : SUCCESS in 30m 23s\n- devstack-platform-fedora-latest https://zuul.opendev.org/t/openstack/build/a49bdca1a6a643b3a419313f8248d173 : SUCCESS in 1h 22m 40s (non-voting)\n- devstack-platform-centos-8 https://zuul.opendev.org/t/openstack/build/7a095a2451ae4ceeb82699d05ab53c7d : FAILURE in 7m 20s (non-voting)\n- devstack-platform-bionic https://zuul.opendev.org/t/openstack/build/def5bdee5fe6463e8b1da81933cd06f9 : SUCCESS in 1h 33m 50s (non-voting)\n- devstack-async https://zuul.opendev.org/t/openstack/build/0bc5a91ea6124aef8888337759bd71d1 : SUCCESS in 1h 30m 45s (non-voting)\n- devstack-multinode https://zuul.opendev.org/t/openstack/build/3104c8f1b3e543d1b35eb840cdebdce2 : SUCCESS in 25m 09s\n- devstack-unit-tests https://zuul.opendev.org/t/openstack/build/e92e9dbddbe04bec80a3670f9583bd48 : SUCCESS in 4m 32s\n- openstack-tox-bashate https://zuul.opendev.org/t/openstack/build/0e59d7055d4d4f26874a23f516015633 : FAILURE in 2m 54s\n- ironic-tempest-ipa-wholedisk-bios-agent_ipmitool-tinyipa https://zuul.opendev.org/t/openstack/build/cdeeb52b97d44c539ce9e15f65dea968 : SUCCESS in 1h 22m 50s (non-voting)\n- swift-dsvm-functional https://zuul.opendev.org/t/openstack/build/c6891a47e7b34ef09ef28ecbf0b65b62 : SUCCESS in 36m 30s (non-voting)\n- neutron-grenade-multinode https://zuul.opendev.org/t/openstack/build/7eefdd80a217431dbed3d4f905394b11 : SUCCESS in 1h 04m 42s\n- neutron-tempest-linuxbridge https://zuul.opendev.org/t/openstack/build/37b24ed18cc94173a7f6d012cf32da23 : SUCCESS in 1h 08m 16s\n- neutron-ovn-tempest-ovs-release https://zuul.opendev.org/t/openstack/build/7e1894e297cd4aae80f41338b5b6ca5c : SUCCESS in 1h 12m 43s (non-voting)\n- tempest-multinode-full-py3 https://zuul.opendev.org/t/openstack/build/d43077e313ee493e821ce74de63d0207 : SUCCESS in 1h 32m 59s (non-voting)\n- openstacksdk-functional-devstack https://zuul.opendev.org/t/openstack/build/18a371d695bd4af3b0758617c56c7b5d : SUCCESS in 55m 42s\n- tempest-ipv6-only https://zuul.opendev.org/t/openstack/build/6259813b6b0b48028ba73d962ccc3dae : SUCCESS in 43m 39s\n- nova-ceph-multistore https://zuul.opendev.org/t/openstack/build/86c17d877e244c9e92742738cb6b1cbe : SUCCESS in 1h 22m 11s","accounts_in_message":[],"_revision_number":1},{"id":"4db30013b7a4a9e9a92e0f167bfee2724c788371","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"date":"2021-05-13 06:02:52.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"cb625f3119ae19f06677855ce292ae6e6d90406b","author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"date":"2021-05-13 07:09:51.000000000","message":"Patch Set 2:\n\nhttps://review.opendev.org/791117\n\nshould test this ...","accounts_in_message":[],"_revision_number":2},{"id":"6e9c3b6266088a259f802ccb57030f507c0f9ba9","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-05-13 08:49:36.000000000","message":"Patch Set 2: Verified+1\n\nBuild succeeded (check pipeline).\n\n- grenade https://zuul.opendev.org/t/openstack/build/f826e31c7f6a4b2796171aad89293c10 : SUCCESS in 1h 02m 23s\n- tempest-full-py3 https://zuul.opendev.org/t/openstack/build/41ead491ebef4abc95fe132a834aef3b : SUCCESS in 1h 38m 39s\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/ceb96517e9734dfcbe63f9ac32cd394d : SUCCESS in 9m 50s\n- devstack https://zuul.opendev.org/t/openstack/build/d61bd8527e2f4abd998dd2d0e507871d : SUCCESS in 28m 25s\n- devstack-ipv6 https://zuul.opendev.org/t/openstack/build/e198765c0da34deb9fa611c17fd8d9da : SUCCESS in 29m 33s\n- devstack-platform-fedora-latest https://zuul.opendev.org/t/openstack/build/20663c2627a249419eb6276c9fa0fa75 : SUCCESS in 1h 41m 06s (non-voting)\n- devstack-platform-centos-8 https://zuul.opendev.org/t/openstack/build/0d6c3688a6d149c18c2330b6c66beddd : FAILURE in 10m 37s (non-voting)\n- devstack-platform-bionic https://zuul.opendev.org/t/openstack/build/46635acc083d4a1ebd26791dd6b6217b : SUCCESS in 1h 37m 02s (non-voting)\n- devstack-async https://zuul.opendev.org/t/openstack/build/48c551e1cd9a4c57b28374c2277ebd03 : SUCCESS in 1h 28m 01s (non-voting)\n- devstack-multinode https://zuul.opendev.org/t/openstack/build/be2e397d38a54cfa8add8e58f551b25e : SUCCESS in 36m 00s\n- devstack-unit-tests https://zuul.opendev.org/t/openstack/build/2d29eb199e994cdf8587ca8547a21a25 : SUCCESS in 5m 26s\n- openstack-tox-bashate https://zuul.opendev.org/t/openstack/build/fba08ae9d1b54ba1a46c167a85b96a6c : SUCCESS in 4m 59s\n- ironic-tempest-ipa-wholedisk-bios-agent_ipmitool-tinyipa https://zuul.opendev.org/t/openstack/build/255c475ab99f49899b19dbd265ad8b0c : SUCCESS in 2h 03m 48s (non-voting)\n- swift-dsvm-functional https://zuul.opendev.org/t/openstack/build/25bbd71199584992bb2516e608859326 : SUCCESS in 31m 58s (non-voting)\n- neutron-grenade-multinode https://zuul.opendev.org/t/openstack/build/a8f85258354941b39053eb933f11d295 : SUCCESS in 55m 20s\n- neutron-tempest-linuxbridge https://zuul.opendev.org/t/openstack/build/c5403f06202649b884b9d33e69c305e9 : SUCCESS in 1h 03m 28s\n- neutron-ovn-tempest-ovs-release https://zuul.opendev.org/t/openstack/build/e5dab848528840a4b971ec7e732dd3f4 : SUCCESS in 1h 17m 24s (non-voting)\n- tempest-multinode-full-py3 https://zuul.opendev.org/t/openstack/build/6c09091f62ce4fbea3afc72bb26229a2 : SUCCESS in 1h 45m 55s (non-voting)\n- openstacksdk-functional-devstack https://zuul.opendev.org/t/openstack/build/7e399ec99b4b4d13a402d9313bb2ee57 : SUCCESS in 51m 58s\n- tempest-ipv6-only https://zuul.opendev.org/t/openstack/build/f33c22b23f1e4558b45aeff299a61ff3 : SUCCESS in 57m 06s\n- nova-ceph-multistore https://zuul.opendev.org/t/openstack/build/8b97453e1723402984d5326cfb947d3c : SUCCESS in 1h 21m 29s","accounts_in_message":[],"_revision_number":2},{"id":"f81c8fe65bcce05f7ecc7e56072d42dadb67c2aa","author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"date":"2021-05-13 13:36:04.000000000","message":"Patch Set 2: Code-Review+2 Review-Priority+2\n\nthanks Ian. not sure why it did not occur in https://review.opendev.org/c/openstack/devstack/+/735097","accounts_in_message":[],"_revision_number":2},{"id":"31c7170f930ba8166f30c2b83e4d69c0646d85b8","author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"date":"2021-05-13 14:10:48.000000000","message":"Patch Set 2:\n\n\u003e Patch Set 2: Code-Review+2 Review-Priority+2\n\u003e \n\u003e thanks Ian. not sure why it did not occur in https://review.opendev.org/c/openstack/devstack/+/735097\n\nI believe the reason it works in normal gating is that we \"preclone\" and configure all of the git repos before devstack runs. This ensures that depends-on works. But a side effect is that you can refer to code that won\u0027t be present if someone has cloned devstack and run stack.sh.","accounts_in_message":[],"_revision_number":2},{"id":"15881a9a7dd24dd9ea35672183bda3415fd25c2b","author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"date":"2021-05-13 16:10:37.000000000","message":"Patch Set 2: -Code-Review\n\n\u003e Patch Set 2:\n\u003e \n\u003e \u003e Patch Set 2: Code-Review+2 Review-Priority+2\n\u003e \u003e \n\u003e \u003e thanks Ian. not sure why it did not occur in https://review.opendev.org/c/openstack/devstack/+/735097\n\u003e \n\u003e I believe the reason it works in normal gating is that we \"preclone\" and configure all of the git repos before devstack runs. This ensures that depends-on works. But a side effect is that you can refer to code that won\u0027t be present if someone has cloned devstack and run stack.sh.\n\nThanks, I see. let\u0027s wait for all error to fix.","accounts_in_message":[],"_revision_number":2},{"id":"f646affb80895a2b3841778c4c311d7cd6570a52","author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"date":"2021-05-13 16:59:07.000000000","message":"Patch Set 2: Code-Review+1","accounts_in_message":[],"_revision_number":2},{"id":"48b5eafd6d121c1c253c961bcc79cd525ec7e610","author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"date":"2021-05-17 00:06:00.000000000","message":"Patch Set 2:\n\nNote the revert in I8c2be82f33ed9f6c36f5509b3b210ee1a38e87ca proposes leaving this in neutron and referencing it later.  I tend to think it seems more logical to keep the installation/compilation bits together here, but I\u0027m not hugely fussed.","accounts_in_message":[],"_revision_number":2},{"id":"44565b612b820402cf1808f9980d81e8421ef8e1","author":{"_account_id":11952,"name":"Flavio Fernandes","email":"flavio@flaviof.com","username":"ffernand"},"date":"2021-05-18 15:43:02.000000000","message":"Patch Set 2: Code-Review+1","accounts_in_message":[],"_revision_number":2},{"id":"bc1542c9295b657db4a1f8c0b92134d15d762911","author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"date":"2021-05-25 14:16:06.000000000","message":"Patch Set 2: -Review-Priority\n\nremoving the gate fix priority as we reverted the original patch","accounts_in_message":[],"_revision_number":2},{"id":"cfa612ff53f1e0c04f385ca013d9bf0b11f489c5","author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"date":"2021-05-27 08:27:30.000000000","message":"Patch Set 2:\n\nShouldn\u0027t we rebase this patch now OVN is again the default backend?","accounts_in_message":[],"_revision_number":2},{"id":"7ec5997daa12fc347fee5bcf143926bcebe71fa7","author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"date":"2021-05-27 16:07:28.000000000","message":"Patch Set 2:\n\nI was speaking with gmann today about this patch. This is no longer required BUT, it\u0027s a nice to have.\n\nI think we should try to get these bits into DevStack instead of having it living in a separated repository as now (Neutron).\n\nIan, if you don\u0027t mind, I will fix the Merge Conflict.","accounts_in_message":[],"_revision_number":2},{"id":"f4d63418bdd3a3b92bfd3a6569012878b5f385df","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"date":"2021-05-27 16:17:34.000000000","message":"Uploaded patch set 3.","accounts_in_message":[],"_revision_number":3},{"id":"7b812c3ddd5cb75b57ef76ae88572c44891a466f","author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"date":"2021-05-27 17:27:52.000000000","message":"Patch Set 3:\n\nI only don\u0027t understand why it gets sourced always now, instead of only when the from-source build is requested.","accounts_in_message":[],"_revision_number":3},{"id":"5573b0a6a5e689dbba5ff8ace515c1feb461d05e","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-05-27 18:14:49.000000000","message":"Patch Set 3: Verified+1\n\nBuild succeeded (check pipeline).\n\n- grenade https://zuul.opendev.org/t/openstack/build/bc6b16b363b544b198f725ddfe4df6b3 : SUCCESS in 1h 11m 45s\n- tempest-full-py3 https://zuul.opendev.org/t/openstack/build/b0b9d3dcc4af4dd9a79aa98012bebf7d : SUCCESS in 1h 27m 26s\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/3bc0ec4021b449d0863b4c1aed0865fd : SUCCESS in 7m 47s\n- devstack https://zuul.opendev.org/t/openstack/build/c1737dffb43d44fa9db67cb6ae20541f : SUCCESS in 27m 16s\n- devstack-ipv6 https://zuul.opendev.org/t/openstack/build/c43ed1f75a774f5db293f0bbce227c9a : SUCCESS in 29m 35s\n- devstack-platform-fedora-latest https://zuul.opendev.org/t/openstack/build/7cff8064a2344224911cd1a1dfcf1977 : SUCCESS in 1h 33m 01s (non-voting)\n- devstack-platform-centos-8 https://zuul.opendev.org/t/openstack/build/2a195300bf1445d8a8cc9e237506dff2 : FAILURE in 14m 24s (non-voting)\n- devstack-async https://zuul.opendev.org/t/openstack/build/753ab9a1c1b74f8694375eaa9087d260 : SUCCESS in 1h 27m 37s (non-voting)\n- devstack-multinode https://zuul.opendev.org/t/openstack/build/82382ac006c5477c87a870e047d1310f : SUCCESS in 42m 16s\n- devstack-unit-tests https://zuul.opendev.org/t/openstack/build/7570ac83c12940998b041a61ed25e9fd : SUCCESS in 5m 30s\n- openstack-tox-bashate https://zuul.opendev.org/t/openstack/build/46778cedbac64af580898b5a31aa6d9f : SUCCESS in 3m 43s\n- ironic-tempest-ipa-wholedisk-bios-agent_ipmitool-tinyipa https://zuul.opendev.org/t/openstack/build/1ce7e01edbc9404db83861e1343f4fdf : SUCCESS in 1h 22m 33s (non-voting)\n- swift-dsvm-functional https://zuul.opendev.org/t/openstack/build/fa96283afc56421e82c04144d4247f5e : SUCCESS in 40m 39s (non-voting)\n- neutron-grenade-multinode https://zuul.opendev.org/t/openstack/build/7e6353e962b54f7d86db6fed47e6a767 : SUCCESS in 1h 05m 33s\n- neutron-tempest-linuxbridge https://zuul.opendev.org/t/openstack/build/4cc97bd6bd554158a2fdf427c046c151 : SUCCESS in 56m 16s\n- neutron-ovn-tempest-ovs-release https://zuul.opendev.org/t/openstack/build/6a39b3ba8af045189a41ee0bee35fa19 : SUCCESS in 1h 10m 44s (non-voting)\n- tempest-multinode-full-py3 https://zuul.opendev.org/t/openstack/build/37e297b7703c4d93b37de78937e34f83 : SUCCESS in 1h 45m 47s (non-voting)\n- openstacksdk-functional-devstack https://zuul.opendev.org/t/openstack/build/889999ea0411469c8f5308ecf71d7bda : SUCCESS in 41m 30s\n- tempest-ipv6-only https://zuul.opendev.org/t/openstack/build/2635774c69d34780a96e499ccaf422cf : SUCCESS in 56m 27s\n- nova-ceph-multistore https://zuul.opendev.org/t/openstack/build/636213197d8f4018a21b98fa38e7a062 : SUCCESS in 1h 39m 46s","accounts_in_message":[],"_revision_number":3},{"id":"cee1f61a7f986e274af63c21f640e8b5f25fbc9b","author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"date":"2021-05-27 18:43:01.000000000","message":"Patch Set 3:\n\n\u003e Patch Set 3:\n\u003e \n\u003e I only don\u0027t understand why it gets sourced always now, instead of only when the from-source build is requested.\n\nI thought it was more \"tidy\" to have that include together with the rest of the includes. Also, it was like that before we moved it to inside that OVN_BUILD_FROM_SOURCE \u003d\u003d True check to make the \"ensure-devstack\" job happy.\n\nBut, if reviewers think it\u0027s best to keep it inside the conditional, I\u0027ve no problem with that either.","accounts_in_message":[],"_revision_number":3},{"id":"ba4ec9e3f28424ca7da40d02a4669f487098d188","author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"date":"2021-05-27 18:45:10.000000000","message":"Patch Set 3:\n\n(1 comment)","accounts_in_message":[],"_revision_number":3},{"id":"3457ac27ee03a3c7425957ebff8b1bd6eaa8aae9","author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"date":"2021-05-28 08:06:43.000000000","message":"Patch Set 3: Code-Review+2\n\n\u003e Patch Set 3:\n\u003e \n\u003e \u003e Patch Set 3:\n\u003e \u003e \n\u003e \u003e I only don\u0027t understand why it gets sourced always now, instead of only when the from-source build is requested.\n\u003e \n\u003e I thought it was more \"tidy\" to have that include together with the rest of the includes. Also, it was like that before we moved it to inside that OVN_BUILD_FROM_SOURCE \u003d\u003d True check to make the \"ensure-devstack\" job happy.\n\u003e \n\u003e But, if reviewers think it\u0027s best to keep it inside the conditional, I\u0027ve no problem with that either.\n\nI just needed background, thanks.","accounts_in_message":[],"_revision_number":3},{"id":"df6f3f7a6ee8e47892c8b94e833bd5804c6303ef","author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"date":"2021-05-28 16:35:43.000000000","message":"Patch Set 3:\n\ntesting here https://review.opendev.org/c/zuul/zuul-jobs/+/793623/1","accounts_in_message":[],"_revision_number":3},{"id":"c0498fa4da876f10f0515bb5e47401f933ab2889","author":{"_account_id":11952,"name":"Flavio Fernandes","email":"flavio@flaviof.com","username":"ffernand"},"date":"2021-06-01 13:49:06.000000000","message":"Patch Set 3: Code-Review+1","accounts_in_message":[],"_revision_number":3},{"id":"c1bd938f44738ed088aadf4583847b1864be4f34","author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"date":"2021-06-01 17:05:52.000000000","message":"Patch Set 3: Code-Review+2 Workflow+1\n\nlgtm","accounts_in_message":[],"_revision_number":3},{"id":"633fb2307fef62b13927c3ee51150e17f95c9d9f","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-06-01 17:06:30.000000000","message":"Patch Set 3: -Verified\n\nStarting gate jobs.","accounts_in_message":[],"_revision_number":3},{"id":"1bce903c6de8e9ad98ac8e502cd40a98f21d8137","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-06-01 22:24:52.000000000","message":"Patch Set 3: Verified+2\n\nBuild succeeded (gate pipeline).\n\n- grenade https://zuul.opendev.org/t/openstack/build/bd95152e7e1d4720bf4124a0a272144e : SUCCESS in 58m 20s\n- tempest-full-py3 https://zuul.opendev.org/t/openstack/build/b6e21f6995b74750b31f551406611a77 : SUCCESS in 1h 24m 00s\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/2c8df3fed9f34dcc81cb2ebf4a98d0fb : SUCCESS in 6m 20s\n- devstack https://zuul.opendev.org/t/openstack/build/9d807799a2cd4ac8b2535a5dff71ed49 : SUCCESS in 25m 40s\n- devstack-ipv6 https://zuul.opendev.org/t/openstack/build/74334fe71ec84a77952a4f81cd74c573 : SUCCESS in 24m 53s\n- devstack-multinode https://zuul.opendev.org/t/openstack/build/8bcb2005335147678c41676cd50783ff : SUCCESS in 36m 07s\n- devstack-unit-tests https://zuul.opendev.org/t/openstack/build/5444743f6ac74e58a40f2f0533811724 : SUCCESS in 4m 30s\n- openstack-tox-bashate https://zuul.opendev.org/t/openstack/build/6cf243512b3e4a2083f9c8bd75916e8e : SUCCESS in 3m 55s\n- neutron-grenade-multinode https://zuul.opendev.org/t/openstack/build/ae20b9f804f849e9ac7118df3b7939ed : SUCCESS in 1h 01m 03s\n- neutron-tempest-linuxbridge https://zuul.opendev.org/t/openstack/build/377c81c2edb647298c04c374b390f091 : SUCCESS in 53m 58s\n- openstacksdk-functional-devstack https://zuul.opendev.org/t/openstack/build/2347999e80de4f61a0863aba04d80b3a : SUCCESS in 52m 50s\n- tempest-ipv6-only https://zuul.opendev.org/t/openstack/build/02f3f09d7da24cf8aa8dd19d74f26b7d : SUCCESS in 48m 36s\n- nova-ceph-multistore https://zuul.opendev.org/t/openstack/build/c45d27fae73243fda294199b1d56500b : SUCCESS in 1h 26m 13s","accounts_in_message":[],"_revision_number":3},{"id":"82e51e39119c9387290c7cfa4a370692bde931ba","tag":"autogenerated:gerrit:merged","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-06-01 22:24:53.000000000","message":"Change has been successfully merged by Zuul","accounts_in_message":[],"_revision_number":3},{"id":"31b78dedc2b800a117ecdc896a1652af33033e36","tag":"autogenerated:zuul:promote","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-06-01 22:27:10.000000000","message":"Patch Set 3:\n\nBuild succeeded (promote pipeline).\n\n- promote-openstack-tox-docs https://zuul.opendev.org/t/openstack/build/64061a82792749e990b560483ad3998b : SUCCESS in 1m 08s","accounts_in_message":[],"_revision_number":3}],"current_revision_number":3,"current_revision":"77835633c5d1daba37f453c5bf7c84fad0f2d68e","revisions":{"96a214d8a73f50b9a180719ac6889ef6e8cdb5dc":{"kind":"REWORK","_number":1,"created":"2021-05-13 03:17:29.000000000","uploader":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"ref":"refs/changes/85/791085/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/85/791085/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/85/791085/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/85/791085/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/85/791085/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/85/791085/1"}}},"commit":{"parents":[{"commit":"1769eebed0a90009ad01b16d15eb460f29759433","subject":"Merge \"Change Neutron\u0027s default ML2 driver to OVN\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/1769eebed0a90009ad01b16d15eb460f29759433"}]}],"author":{"name":"Ian Wienand","email":"iwienand@redhat.com","date":"2021-05-13 03:14:42.000000000","tz":600},"committer":{"name":"Ian Wienand","email":"iwienand@redhat.com","date":"2021-05-13 03:14:42.000000000","tz":600},"subject":"OVN : include source compliation functions","message":"OVN : include source compliation functions\n\nIb4194329474e8d68a90886d2a04f027eecd741df moved the OVN module from\nNeutron into devstack, introducting the affected include here.\n\nThis include happens before the trees are cloned.  When the switch was\nmade in I92054ce9d2ab7a42746ed5dececef583b0f8a833 jobs have started\nfailing.\n\nThis brings the ovn file from neutron, which appears to mostly be used\nfor the \"compile from source path\" into devstack as well.  It is\nrenamed \"ovs_source\" to highlight this, and the include is updated.\n\nChange-Id: I39ec9ce0a91bea05cf8c446a9767ab879ac8e8f3\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/96a214d8a73f50b9a180719ac6889ef6e8cdb5dc"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/96a214d8a73f50b9a180719ac6889ef6e8cdb5dc"}]},"parents_data":[{"branch_name":"refs/heads/master","commit_id":"1769eebed0a90009ad01b16d15eb460f29759433","is_merged_in_target_branch":true}],"branch":"refs/heads/master"},"51404444567b7cd44a27cba6f3963cd6384bf3d7":{"kind":"REWORK","_number":2,"created":"2021-05-13 06:02:52.000000000","uploader":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"ref":"refs/changes/85/791085/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/85/791085/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/85/791085/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/85/791085/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/85/791085/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/85/791085/2"}}},"commit":{"parents":[{"commit":"1769eebed0a90009ad01b16d15eb460f29759433","subject":"Merge \"Change Neutron\u0027s default ML2 driver to OVN\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/1769eebed0a90009ad01b16d15eb460f29759433"}]}],"author":{"name":"Ian Wienand","email":"iwienand@redhat.com","date":"2021-05-13 03:14:42.000000000","tz":600},"committer":{"name":"Ian Wienand","email":"iwienand@redhat.com","date":"2021-05-13 06:02:40.000000000","tz":600},"subject":"OVN : include source compliation functions","message":"OVN : include source compliation functions\n\nIb4194329474e8d68a90886d2a04f027eecd741df moved the OVN module from\nNeutron into devstack, introducting the affected include here.\n\nThis include happens before the trees are cloned.  When the switch was\nmade in I92054ce9d2ab7a42746ed5dececef583b0f8a833 jobs have started\nfailing.\n\nThis brings the ovn file from neutron, which appears to mostly be used\nfor the \"compile from source path\" into devstack as well.  It is\nrenamed \"ovs_source\" to highlight this, and the include is updated.\n\nChange-Id: I39ec9ce0a91bea05cf8c446a9767ab879ac8e8f3\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/51404444567b7cd44a27cba6f3963cd6384bf3d7"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/51404444567b7cd44a27cba6f3963cd6384bf3d7"}]},"parents_data":[{"branch_name":"refs/heads/master","commit_id":"1769eebed0a90009ad01b16d15eb460f29759433","is_merged_in_target_branch":true}],"branch":"refs/heads/master"},"77835633c5d1daba37f453c5bf7c84fad0f2d68e":{"kind":"REWORK","_number":3,"created":"2021-05-27 16:17:34.000000000","uploader":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"ref":"refs/changes/85/791085/3","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/85/791085/3","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/85/791085/3 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/85/791085/3 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/85/791085/3 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/85/791085/3"}}},"commit":{"parents":[{"commit":"a634c7f91cd6dd43a8e5d1f280f06f4a5599f504","subject":"Merge \"[CI] Add no-tls-proxy job\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/a634c7f91cd6dd43a8e5d1f280f06f4a5599f504"}]}],"author":{"name":"Ian Wienand","email":"iwienand@redhat.com","date":"2021-05-13 03:14:42.000000000","tz":600},"committer":{"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","date":"2021-05-27 16:10:53.000000000","tz":60},"subject":"OVN : include source compliation functions","message":"OVN : include source compliation functions\n\nThis patch moves the OVS compilation module from Neutron into DevStack.\nIt also renamed it to \"ovs_source\" to highlight its function, and the\ninclude has been moved to where the rest of the includes are located.\n\nAlthough this module is not required since by default DevStack installs\nOVS/OVN from the host OS packages instead of compiling from source,\nthis is a nice to have as it avoids having bits and pieces of the code\nscattered around multiple repositories.\n\nCo-Authored-By: Lucas Alvares Gomes \u003clucasagomes@gmail.com\u003e\nChange-Id: I39ec9ce0a91bea05cf8c446a9767ab879ac8e8f3\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/77835633c5d1daba37f453c5bf7c84fad0f2d68e"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/77835633c5d1daba37f453c5bf7c84fad0f2d68e"}]},"parents_data":[{"branch_name":"refs/heads/master","commit_id":"a634c7f91cd6dd43a8e5d1f280f06f4a5599f504","is_merged_in_target_branch":true}],"branch":"refs/heads/master"}},"requirements":[],"submit_records":[{"status":"CLOSED","labels":[{"label":"Verified","status":"OK","applied_by":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}},{"label":"Code-Review","status":"OK","applied_by":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"}},{"label":"Workflow","status":"OK","applied_by":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"}},{"label":"Review-Priority","status":"MAY"}]}],"submit_requirements":[{"name":"Verified","status":"SATISFIED","is_legacy":true,"submittability_expression_result":{"expression":"label:Verified\u003dCustom-Rule","fulfilled":true,"status":"PASS","passing_atoms":["label:Verified\u003dCustom-Rule"],"failing_atoms":[],"atom_explanations":{}}},{"name":"Workflow","status":"SATISFIED","is_legacy":true,"submittability_expression_result":{"expression":"label:Workflow\u003dCustom-Rule","fulfilled":true,"status":"PASS","passing_atoms":["label:Workflow\u003dCustom-Rule"],"failing_atoms":[],"atom_explanations":{}}},{"name":"Code-Review","status":"SATISFIED","is_legacy":true,"submittability_expression_result":{"expression":"label:Code-Review\u003dCustom-Rule","fulfilled":true,"status":"PASS","passing_atoms":["label:Code-Review\u003dCustom-Rule"],"failing_atoms":[],"atom_explanations":{}}}]}