)]}'
{"id":"openstack%2Fpuppet-ovn~773713","triplet_id":"openstack%2Fpuppet-ovn~master~I66d663971f087d019139b033c032a8e05f71a57a","project":"openstack/puppet-ovn","branch":"master","topic":"fix-multiple-brex","hashtags":[],"change_id":"I66d663971f087d019139b033c032a8e05f71a57a","subject":"Support multiple ovn bridge mappings with the same nic","status":"ABANDONED","created":"2021-02-02 15:47:23.000000000","updated":"2021-08-18 13:20:44.000000000","total_comment_count":4,"unresolved_comment_count":3,"has_review_started":true,"meta_rev_id":"f4756f1b38b9ddf656e9b465a48ff83f305c7a84","_number":773713,"virtual_id_number":773713,"owner":{"_account_id":20172,"name":"Michele Baldessari","email":"michele@acksyn.org","username":"michele"},"actions":{},"labels":{"Verified":{"recommended":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"_account_id":6469,"name":"Carlos Gonçalves","display_name":"Carlos Goncalves","email":"cgoncalves@redhat.com","username":"cgoncalves"},{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},{"_account_id":6681,"name":"Brent Eagles","email":"beagles@redhat.com","username":"beagles"},{"_account_id":11082,"name":"Kamil Sambor","email":"ksambor@redhat.com","username":"ksambor"},{"_account_id":20778,"name":"Damien Ciabrini","email":"dciabrin@redhat.com","username":"dciabrin"},{"tag":"autogenerated:zuul:check","value":1,"date":"2021-02-15 08:58:17.000000000","permitted_voting_range":{"min":-2,"max":2},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"_account_id":23804,"name":"Daniel Alvarez","email":"dalvarez@redhat.com","username":"dalvarez"},{"_account_id":23567,"name":"Luis Tomas Bolivar","email":"ltomasbo@redhat.com","username":"ltomasbo"}],"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":{"recommended":{"_account_id":11082,"name":"Kamil Sambor","email":"ksambor@redhat.com","username":"ksambor"},"disliked":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"all":[{"value":1,"date":"2021-02-15 08:59:58.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":6469,"name":"Carlos Gonçalves","display_name":"Carlos Goncalves","email":"cgoncalves@redhat.com","username":"cgoncalves"},{"value":-1,"date":"2021-02-23 13:03:50.000000000","permitted_voting_range":{"min":-2,"max":2},"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":6681,"name":"Brent Eagles","email":"beagles@redhat.com","username":"beagles"},{"value":1,"date":"2021-02-22 10:50:57.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":11082,"name":"Kamil Sambor","email":"ksambor@redhat.com","username":"ksambor"},{"value":1,"date":"2021-02-15 08:42:08.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":20778,"name":"Damien Ciabrini","email":"dciabrin@redhat.com","username":"dciabrin"},{"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":23804,"name":"Daniel Alvarez","email":"dalvarez@redhat.com","username":"dalvarez"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":23567,"name":"Luis Tomas Bolivar","email":"ltomasbo@redhat.com","username":"ltomasbo"}],"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":"","value":-1,"default_value":0,"optional":true},"Workflow":{"all":[{"_account_id":6469,"name":"Carlos Gonçalves","display_name":"Carlos Goncalves","email":"cgoncalves@redhat.com","username":"cgoncalves"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},{"_account_id":6681,"name":"Brent Eagles","email":"beagles@redhat.com","username":"beagles"},{"_account_id":11082,"name":"Kamil Sambor","email":"ksambor@redhat.com","username":"ksambor"},{"_account_id":20778,"name":"Damien Ciabrini","email":"dciabrin@redhat.com","username":"dciabrin"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"_account_id":23804,"name":"Daniel Alvarez","email":"dalvarez@redhat.com","username":"dalvarez"},{"_account_id":23567,"name":"Luis Tomas Bolivar","email":"ltomasbo@redhat.com","username":"ltomasbo"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":6469,"name":"Carlos Gonçalves","display_name":"Carlos Goncalves","email":"cgoncalves@redhat.com","username":"cgoncalves"},{"_account_id":6681,"name":"Brent Eagles","email":"beagles@redhat.com","username":"beagles"},{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},{"_account_id":11082,"name":"Kamil Sambor","email":"ksambor@redhat.com","username":"ksambor"},{"_account_id":20778,"name":"Damien Ciabrini","email":"dciabrin@redhat.com","username":"dciabrin"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"_account_id":23567,"name":"Luis Tomas Bolivar","email":"ltomasbo@redhat.com","username":"ltomasbo"},{"_account_id":23804,"name":"Daniel Alvarez","email":"dalvarez@redhat.com","username":"dalvarez"}],"CC":[{"_account_id":16347,"name":"Purandhar Mannidi","display_name":"Sai","email":"sai438@gmail.com","username":"mpsairam"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2021-02-02 16:02:54.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-02-02 19:29:58.000000000","updated_by":{"_account_id":20172,"name":"Michele Baldessari","email":"michele@acksyn.org","username":"michele"},"reviewer":{"_account_id":6469,"name":"Carlos Gonçalves","display_name":"Carlos Goncalves","email":"cgoncalves@redhat.com","username":"cgoncalves"},"state":"REVIEWER"},{"updated":"2021-02-02 19:29:58.000000000","updated_by":{"_account_id":20172,"name":"Michele Baldessari","email":"michele@acksyn.org","username":"michele"},"reviewer":{"_account_id":23567,"name":"Luis Tomas Bolivar","email":"ltomasbo@redhat.com","username":"ltomasbo"},"state":"REVIEWER"},{"updated":"2021-02-02 19:29:58.000000000","updated_by":{"_account_id":20172,"name":"Michele Baldessari","email":"michele@acksyn.org","username":"michele"},"reviewer":{"_account_id":23804,"name":"Daniel Alvarez","email":"dalvarez@redhat.com","username":"dalvarez"},"state":"REVIEWER"},{"updated":"2021-02-02 19:30:44.000000000","updated_by":{"_account_id":20172,"name":"Michele Baldessari","email":"michele@acksyn.org","username":"michele"},"reviewer":{"_account_id":20778,"name":"Damien Ciabrini","email":"dciabrin@redhat.com","username":"dciabrin"},"state":"CC"},{"updated":"2021-02-09 08:09:38.000000000","updated_by":{"_account_id":6469,"name":"Carlos Gonçalves","display_name":"Carlos Goncalves","email":"cgoncalves@redhat.com","username":"cgoncalves"},"reviewer":{"_account_id":6681,"name":"Brent Eagles","email":"beagles@redhat.com","username":"beagles"},"state":"CC"},{"updated":"2021-02-09 08:10:15.000000000","updated_by":{"_account_id":6469,"name":"Carlos Gonçalves","display_name":"Carlos Goncalves","email":"cgoncalves@redhat.com","username":"cgoncalves"},"reviewer":{"_account_id":6681,"name":"Brent Eagles","email":"beagles@redhat.com","username":"beagles"},"state":"REVIEWER"},{"updated":"2021-02-15 08:28:57.000000000","updated_by":{"_account_id":20778,"name":"Damien Ciabrini","email":"dciabrin@redhat.com","username":"dciabrin"},"reviewer":{"_account_id":20778,"name":"Damien Ciabrini","email":"dciabrin@redhat.com","username":"dciabrin"},"state":"REVIEWER"},{"updated":"2021-02-22 10:50:57.000000000","updated_by":{"_account_id":11082,"name":"Kamil Sambor","email":"ksambor@redhat.com","username":"ksambor"},"reviewer":{"_account_id":11082,"name":"Kamil Sambor","email":"ksambor@redhat.com","username":"ksambor"},"state":"REVIEWER"},{"updated":"2021-02-23 13:03:50.000000000","updated_by":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"reviewer":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"state":"REVIEWER"},{"updated":"2021-08-17 13:37:09.000000000","updated_by":{"_account_id":16347,"name":"Purandhar Mannidi","display_name":"Sai","email":"sai438@gmail.com","username":"mpsairam"},"reviewer":{"_account_id":16347,"name":"Purandhar Mannidi","display_name":"Sai","email":"sai438@gmail.com","username":"mpsairam"},"state":"CC"}],"messages":[{"id":"720de78d11e8ccdd5e283697cc576e9cfd0a0c11","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":20172,"name":"Michele Baldessari","email":"michele@acksyn.org","username":"michele"},"date":"2021-02-02 15:47:23.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"3110d05f0aa2867bf51458bb80d0282ef9b7328c","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-02-02 16:02:54.000000000","message":"Patch Set 1: Verified+1\n\nBuild succeeded (check pipeline).\n\n- puppet-openstack-lint-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/55963525b7f74c9798d2134a9236fa2c : SUCCESS in 3m 47s\n- puppet-openstack-syntax-6-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/5483255885ab4a2c9fadb4f892039bea : SUCCESS in 3m 46s\n- puppet-openstack-unit-6.14-centos-8 https://zuul.opendev.org/t/openstack/build/f023af0b0784416197bad62a640681d7 : SUCCESS in 8m 52s\n- puppet-openstack-unit-6.14-centos-8-stream https://zuul.opendev.org/t/openstack/build/006941a0550445f6a164183bd6817cfb : SUCCESS in 7m 40s (non-voting)\n- puppet-openstack-unit-6.14-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/8272546ff9ad401ca03298e94fc4dba2 : SUCCESS in 4m 54s\n- puppet-openstack-unit-latest-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/f74a06e06c44406fb51661f799ad6727 : SUCCESS in 5m 45s (non-voting)\n- puppet-openstack-litmus-centos-8 https://zuul.opendev.org/t/openstack/build/3eb3a4bd8bb0418a8293a77f8d48d018 : SUCCESS in 11m 50s\n- puppet-openstack-litmus-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/401d220a5a514214ba51505abdaf55f1 : SUCCESS in 6m 52s (non-voting)","accounts_in_message":[],"_revision_number":1},{"id":"24ad525a81b14337dd099f8327c2aec2182d82e0","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":20172,"name":"Michele Baldessari","email":"michele@acksyn.org","username":"michele"},"date":"2021-02-02 19:29:28.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"359a5fc6c61057f81d466b34b8a0f64f6c323322","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-02-02 19:54:23.000000000","message":"Patch Set 2: Verified-1\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- puppet-openstack-lint-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/299b491b97594d818e5d60489a4f9ab4 : SUCCESS in 3m 55s\n- puppet-openstack-syntax-6-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/a3b4dddb4de54ab1ae6d7b760ba57b59 : SUCCESS in 3m 07s\n- puppet-openstack-unit-6.14-centos-8 https://zuul.opendev.org/t/openstack/build/0e7c71777cf8490589a0e1d3906df44c : FAILURE in 6m 51s\n- puppet-openstack-unit-6.14-centos-8-stream https://zuul.opendev.org/t/openstack/build/e3afd515457a4d95bc3b9ac4854d1978 : FAILURE in 7m 54s (non-voting)\n- puppet-openstack-unit-6.14-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/a0bf9c3ec04748f2b89d555d1005a208 : FAILURE in 5m 51s\n- puppet-openstack-unit-latest-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/83567e560c944e56bd67927e29f92a39 : FAILURE in 5m 28s (non-voting)\n- puppet-openstack-litmus-centos-8 https://zuul.opendev.org/t/openstack/build/d4768b22773b40e0a6eccc3de62e44c8 : SUCCESS in 12m 57s\n- puppet-openstack-litmus-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/bb493bebb0354df993af1b4741207a86 : SUCCESS in 7m 07s (non-voting)","accounts_in_message":[],"_revision_number":2},{"id":"89bc7a9a35270a84c7be966e3aeae2d343c67472","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":20172,"name":"Michele Baldessari","email":"michele@acksyn.org","username":"michele"},"date":"2021-02-03 07:14:21.000000000","message":"Uploaded patch set 3.","accounts_in_message":[],"_revision_number":3},{"id":"6c6ed0330f888aac72c9a97d2065187585e03fe3","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-02-03 07:27:13.000000000","message":"Patch Set 3: Verified-1\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- puppet-openstack-lint-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/36b91d92867d46d8bb2bef4f1de107a1 : SUCCESS in 3m 57s\n- puppet-openstack-syntax-6-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/c799493713cf47fa95b4cea79712aabc : SUCCESS in 2m 43s\n- puppet-openstack-unit-6.14-centos-8 https://zuul.opendev.org/t/openstack/build/67402ea436044a1e882467d46f8e1dbb : FAILURE in 6m 34s\n- puppet-openstack-unit-6.14-centos-8-stream https://zuul.opendev.org/t/openstack/build/6621df4331f849cfa225791c4afa4090 : FAILURE in 6m 53s (non-voting)\n- puppet-openstack-unit-6.14-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/832d0ea0530a440390513b0c14b22e7d : FAILURE in 4m 32s\n- puppet-openstack-unit-latest-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/33027c9e63c744d6aee7eb1a4f71f8e9 : FAILURE in 4m 30s (non-voting)\n- puppet-openstack-litmus-centos-8 https://zuul.opendev.org/t/openstack/build/c17d893a48db4a0aa1c5411515b05e1e : SUCCESS in 10m 51s\n- puppet-openstack-litmus-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/ff72ad19d1eb438a9592227cc266e8b1 : SUCCESS in 5m 24s (non-voting)","accounts_in_message":[],"_revision_number":3},{"id":"8064fee607bac1caba12280237fac0c6157cdcda","author":{"_account_id":23804,"name":"Daniel Alvarez","email":"dalvarez@redhat.com","username":"dalvarez"},"date":"2021-02-03 10:32:05.000000000","message":"Patch Set 3:\n\n(1 comment)","accounts_in_message":[],"_revision_number":3},{"id":"0867d8d525fc3cb7b81c00318ac85e95d0b7d583","author":{"_account_id":20172,"name":"Michele Baldessari","email":"michele@acksyn.org","username":"michele"},"date":"2021-02-03 10:43:05.000000000","message":"Patch Set 3:\n\n(1 comment)","accounts_in_message":[],"_revision_number":3},{"id":"d4991180f2888325632095ba17562ec435aa18c2","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":20172,"name":"Michele Baldessari","email":"michele@acksyn.org","username":"michele"},"date":"2021-02-03 11:20:29.000000000","message":"Uploaded patch set 4.","accounts_in_message":[],"_revision_number":4},{"id":"59004d177405836c49fe1047e407b0583bbdb5b9","author":{"_account_id":20172,"name":"Michele Baldessari","email":"michele@acksyn.org","username":"michele"},"date":"2021-02-03 11:27:58.000000000","message":"Patch Set 4:\n\nAdditional CI test run via https://review.opendev.org/c/openstack/tripleo-heat-templates/+/773846","accounts_in_message":[],"_revision_number":4},{"id":"24c239afaf646b79a093dba46908e7b290f39a03","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-02-03 11:37:09.000000000","message":"Patch Set 4: Verified-1\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- puppet-openstack-lint-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/b846512c604747f4b5f1b24553bf735e : SUCCESS in 3m 42s\n- puppet-openstack-syntax-6-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/820103d1bf5c43d3985af919e5ee6c71 : SUCCESS in 4m 37s\n- puppet-openstack-unit-6.14-centos-8 https://zuul.opendev.org/t/openstack/build/80f89a9eeeac44a7ab0aec39b02e587b : FAILURE in 8m 12s\n- puppet-openstack-unit-6.14-centos-8-stream https://zuul.opendev.org/t/openstack/build/f4057a3be6584f16863b75eb3ba9c9b6 : FAILURE in 8m 11s (non-voting)\n- puppet-openstack-unit-6.14-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/adf2d5ede87b465d89c90346018b501d : FAILURE in 6m 40s\n- puppet-openstack-unit-latest-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/b1e9c2b5703443a89627ed95dfd546a6 : FAILURE in 5m 52s (non-voting)\n- puppet-openstack-litmus-centos-8 https://zuul.opendev.org/t/openstack/build/f9b3b75bd5e343a7a0b8281e5638ff64 : SUCCESS in 10m 43s\n- puppet-openstack-litmus-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/24b7cd652cd64d3ea931e2de912a4a76 : SUCCESS in 6m 30s (non-voting)","accounts_in_message":[],"_revision_number":4},{"id":"ee0c56977936f6d653eb074de1905a146ba51112","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":20172,"name":"Michele Baldessari","email":"michele@acksyn.org","username":"michele"},"date":"2021-02-03 12:24:08.000000000","message":"Uploaded patch set 5.","accounts_in_message":[],"_revision_number":5},{"id":"0f3561b668fd4cafc5c2656ac020c7ec7e77f73b","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-02-03 12:50:33.000000000","message":"Patch Set 5: Verified+1\n\nBuild succeeded (check pipeline).\n\n- puppet-openstack-lint-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/7b1d62aaa36145488cd725f6ff4b6ffa : SUCCESS in 3m 21s\n- puppet-openstack-syntax-6-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/d95af36bf6204047892a87b4f7548db4 : SUCCESS in 3m 40s\n- puppet-openstack-unit-6.14-centos-8 https://zuul.opendev.org/t/openstack/build/917d7c97f93b44c6aa7a6d546318f820 : SUCCESS in 11m 20s\n- puppet-openstack-unit-6.14-centos-8-stream https://zuul.opendev.org/t/openstack/build/aa5e113d5a5146d7b8c89b7ac5b5ad11 : SUCCESS in 7m 37s (non-voting)\n- puppet-openstack-unit-6.14-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/697013433e334fb8bf74816c36c06836 : SUCCESS in 6m 17s\n- puppet-openstack-unit-latest-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/559bb20045874988a2980007fe8a0baa : SUCCESS in 5m 34s (non-voting)\n- puppet-openstack-litmus-centos-8 https://zuul.opendev.org/t/openstack/build/02e02bc29a4748a8ad1592d6ac27cc5a : SUCCESS in 10m 45s\n- puppet-openstack-litmus-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/9f6b6fc140d04386a0aa0bfa5a8d0d56 : SUCCESS in 7m 23s (non-voting)","accounts_in_message":[],"_revision_number":5},{"id":"2db92b54659d9d10101babcb8bdc838f57760a9a","author":{"_account_id":6469,"name":"Carlos Gonçalves","display_name":"Carlos Goncalves","email":"cgoncalves@redhat.com","username":"cgoncalves"},"date":"2021-02-03 12:54:05.000000000","message":"Patch Set 5: Code-Review+1","accounts_in_message":[],"_revision_number":5},{"id":"13cc6a8decf7fd9514f7a121a2fe48a27d1f9717","author":{"_account_id":20172,"name":"Michele Baldessari","email":"michele@acksyn.org","username":"michele"},"date":"2021-02-03 13:13:22.000000000","message":"Patch Set 5:\n\nJust rechecked https://review.opendev.org/c/openstack/tripleo-heat-templates/+/773846 let\u0027s see if it is green there as well","accounts_in_message":[],"_revision_number":5},{"id":"aa3bc5472adf3a3f31bdf820a0b1425ba46cfc4f","author":{"_account_id":20172,"name":"Michele Baldessari","email":"michele@acksyn.org","username":"michele"},"date":"2021-02-08 07:22:51.000000000","message":"Patch Set 5:\n\nhttps://review.opendev.org/c/openstack/tripleo-heat-templates/+/773846 is green and I verified that it indeed has 9ae588e in the version: puppet-ovn-18.2.0-0.20210204065513.9ae588e.el8.noarch so I think this one should be okay at this point","accounts_in_message":[],"_revision_number":5},{"id":"f85700f4cb95a35b231991a5b7e26ebdac10a14c","author":{"_account_id":23804,"name":"Daniel Alvarez","email":"dalvarez@redhat.com","username":"dalvarez"},"date":"2021-02-09 09:17:08.000000000","message":"Patch Set 5: Code-Review+1","accounts_in_message":[],"_revision_number":5},{"id":"b92ce07bd33aef7c7ce69c71e1a9d5833608cee9","tag":"autogenerated:gerrit:setTopic","author":{"_account_id":20172,"name":"Michele Baldessari","email":"michele@acksyn.org","username":"michele"},"date":"2021-02-12 20:52:24.000000000","message":"Topic fix-multiple-brex removed","accounts_in_message":[],"_revision_number":5},{"id":"3712549f14f5b8cef473f937accff76a54fe9b8a","tag":"autogenerated:gerrit:setTopic","author":{"_account_id":20172,"name":"Michele Baldessari","email":"michele@acksyn.org","username":"michele"},"date":"2021-02-12 20:52:34.000000000","message":"Topic set to bgpsupport","accounts_in_message":[],"_revision_number":5},{"id":"54bb6968f137eda51dc26203cc90d0892e0a69a9","author":{"_account_id":20778,"name":"Damien Ciabrini","email":"dciabrin@redhat.com","username":"dciabrin"},"date":"2021-02-15 08:28:57.000000000","message":"Patch Set 5: Code-Review+1","accounts_in_message":[],"_revision_number":5},{"id":"4e3ce3500a2114774ff7bb3e91e06330cd91fcbc","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":20172,"name":"Michele Baldessari","email":"michele@acksyn.org","username":"michele"},"date":"2021-02-15 08:41:16.000000000","message":"Uploaded patch set 6.","accounts_in_message":[],"_revision_number":6},{"id":"62e139705355ef6486c710308513fb213a936c8b","author":{"_account_id":20778,"name":"Damien Ciabrini","email":"dciabrin@redhat.com","username":"dciabrin"},"date":"2021-02-15 08:42:08.000000000","message":"Patch Set 6: Code-Review+1","accounts_in_message":[],"_revision_number":6},{"id":"aab0551dc6d64df940d1c8c8c6cf3f7b139bfe57","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-02-15 08:58:17.000000000","message":"Patch Set 6: Verified+1\n\nBuild succeeded (check pipeline).\n\n- puppet-openstack-lint-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/3be2201c8eab4189903437cb79fb2574 : SUCCESS in 3m 33s\n- puppet-openstack-syntax-6-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/f20792f52fb04581bcc27013e66f3662 : SUCCESS in 3m 04s\n- puppet-openstack-unit-6.14-centos-8 https://zuul.opendev.org/t/openstack/build/e98852ad90434ed1aa4f42a9021398f6 : SUCCESS in 6m 47s\n- puppet-openstack-unit-6.14-centos-8-stream https://zuul.opendev.org/t/openstack/build/e3e7a821b838449e9f361cf3b3878cd9 : SUCCESS in 7m 22s (non-voting)\n- puppet-openstack-unit-6.14-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/67a063f451904662a66b8cf96413fbdd : SUCCESS in 5m 47s\n- puppet-openstack-unit-latest-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/a4345952f1b948659e803a0f3b997a01 : SUCCESS in 5m 44s (non-voting)\n- puppet-openstack-litmus-centos-8 https://zuul.opendev.org/t/openstack/build/381d11f4648f4f5192e175c2bb6e148b : SUCCESS in 11m 49s\n- puppet-openstack-litmus-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/8f775446211143af9a0b0760488bf355 : SUCCESS in 9m 57s (non-voting)","accounts_in_message":[],"_revision_number":6},{"id":"24e37587ccedbca0e3bdbb572e5c1ed094ea6dff","author":{"_account_id":6469,"name":"Carlos Gonçalves","display_name":"Carlos Goncalves","email":"cgoncalves@redhat.com","username":"cgoncalves"},"date":"2021-02-15 08:59:58.000000000","message":"Patch Set 6: Code-Review+1","accounts_in_message":[],"_revision_number":6},{"id":"916600bfcc9651a279ca2bfaca38f240b7435c08","author":{"_account_id":11082,"name":"Kamil Sambor","email":"ksambor@redhat.com","username":"ksambor"},"date":"2021-02-22 10:50:57.000000000","message":"Patch Set 6: Code-Review+1","accounts_in_message":[],"_revision_number":6},{"id":"98b072883c9b7982ce3be02ff881241fd82104b4","author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"date":"2021-02-23 13:03:50.000000000","message":"Patch Set 6: Code-Review-1\n\n(2 comments)\n\nThe change looks reasonable to me, but let me put -1 because of a little concern with the change in resource type interface.","accounts_in_message":[],"_revision_number":6},{"id":"f4756f1b38b9ddf656e9b465a48ff83f305c7a84","tag":"autogenerated:gerrit:abandon","author":{"_account_id":20172,"name":"Michele Baldessari","email":"michele@acksyn.org","username":"michele"},"date":"2021-08-18 13:20:44.000000000","message":"Abandoned\n\nI don\u0027t think we need this one at this time","accounts_in_message":[],"_revision_number":6}],"current_revision_number":6,"current_revision":"1388d9eae38333bfdf998f2aa14be13f237705ad","revisions":{"d914b8b633446348e5c60b058245eb11e13adf14":{"kind":"REWORK","_number":1,"created":"2021-02-02 15:47:23.000000000","uploader":{"_account_id":20172,"name":"Michele Baldessari","email":"michele@acksyn.org","username":"michele"},"ref":"refs/changes/13/773713/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/puppet-ovn","ref":"refs/changes/13/773713/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/puppet-ovn refs/changes/13/773713/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/puppet-ovn refs/changes/13/773713/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/puppet-ovn refs/changes/13/773713/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/puppet-ovn refs/changes/13/773713/1"}}},"commit":{"parents":[{"commit":"3aa6ee0de90a2e5e7f57224c8ea5a641967fba86","subject":"Prepare Wallaby M2","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/puppet-ovn/commit/3aa6ee0de90a2e5e7f57224c8ea5a641967fba86"}]}],"author":{"name":"Michele Baldessari","email":"michele@acksyn.org","date":"2021-02-02 15:39:34.000000000","tz":60},"committer":{"name":"Michele Baldessari","email":"michele@acksyn.org","date":"2021-02-02 15:47:21.000000000","tz":60},"subject":"WIP Support multiple ovn bridge mappings with the same nic","message":"WIP Support multiple ovn bridge mappings with the same nic\n\nCurrently if you pass in with NeutronBridgeMappings: [\"provider1:br-ex\",\n\"provider2:br-ex\"] puppet will fail with duplicate declaration errors\naround the exec because it is using only the nic \u0027br-ex\u0027 as the resource\nname. We fix this by passing both the network and the bridge to the\nname. The command is idempotent so no harm is done.\n\nThe other place is the ovn::controller::bridge define which will complain\nabout duplicate declarations. We fix this by just passing the bridge\nname directly to the ovn::controller::bridge define and we use\nensure_resource() when creating those definitions to avoid duplicates\n\nWIP: as I need to test it in a couple more scenarios\nChange-Id: I66d663971f087d019139b033c032a8e05f71a57a\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/puppet-ovn/commit/d914b8b633446348e5c60b058245eb11e13adf14"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/puppet-ovn/commit/d914b8b633446348e5c60b058245eb11e13adf14"}]},"branch":"refs/heads/master"},"4d205c924e5958ca8462b9dc9d400170420b0265":{"kind":"REWORK","_number":2,"created":"2021-02-02 19:29:28.000000000","uploader":{"_account_id":20172,"name":"Michele Baldessari","email":"michele@acksyn.org","username":"michele"},"ref":"refs/changes/13/773713/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/puppet-ovn","ref":"refs/changes/13/773713/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/puppet-ovn refs/changes/13/773713/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/puppet-ovn refs/changes/13/773713/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/puppet-ovn refs/changes/13/773713/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/puppet-ovn refs/changes/13/773713/2"}}},"commit":{"parents":[{"commit":"3aa6ee0de90a2e5e7f57224c8ea5a641967fba86","subject":"Prepare Wallaby M2","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/puppet-ovn/commit/3aa6ee0de90a2e5e7f57224c8ea5a641967fba86"}]}],"author":{"name":"Michele Baldessari","email":"michele@acksyn.org","date":"2021-02-02 15:39:34.000000000","tz":60},"committer":{"name":"Michele Baldessari","email":"michele@acksyn.org","date":"2021-02-02 19:28:48.000000000","tz":60},"subject":"Support multiple ovn bridge mappings with the same nic","message":"Support multiple ovn bridge mappings with the same nic\n\nCurrently if you pass in [\"provider1:br-ex\", \"provider2:br-ex\"] via the\npuppet ovn::controller::ovn_bridge_mappings hiera parameter, it\nwill fail with duplicate declaration errors around the exec because it\nis using only the nic \u0027br-ex\u0027 as the resource name. We fix this by\npassing both the network and the bridge to the name. The command is\nidempotent so no harm is done.\n\nThe other place is the ovn::controller::bridge define which will complain\nabout duplicate declarations. We fix this by just passing the bridge\nname directly to the ovn::controller::bridge define and we use\nensure_resource() when creating those definitions to avoid duplicates\n\nTested this as follows:\n1) Deployed a train env with this patch with a composable control plane\n with [\"datacentre:br-ex\", \"tenant:br-isolated\"] and obtained:\n\n[root@controller-0 ~]# ovs-vsctl list open . |grep -o -P -e \u0027ovn-bridge-mappings\u003d.*?, \u0027\novn-bridge-mappings\u003d\"datacentre:br-ex,tenant:br-isolated\",\n\n2) Deployed a master env with this patch with\n[\"provider1:br-ex\", \"provider2:br-ex\"] and obtained:\n\n[root@ctrl-1-0 ~]# ovs-vsctl list open . |grep -o -P -e \u0027ovn-bridge-mappings\u003d.*?, \u0027\novn-bridge-mappings\u003d\"provider1:br-ex,provider2:br-ex\",\n\nChange-Id: I66d663971f087d019139b033c032a8e05f71a57a\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/puppet-ovn/commit/4d205c924e5958ca8462b9dc9d400170420b0265"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/puppet-ovn/commit/4d205c924e5958ca8462b9dc9d400170420b0265"}]},"branch":"refs/heads/master"},"ab5516e3082f0508f05e2352b53127fcb34a462a":{"kind":"REWORK","_number":3,"created":"2021-02-03 07:14:21.000000000","uploader":{"_account_id":20172,"name":"Michele Baldessari","email":"michele@acksyn.org","username":"michele"},"ref":"refs/changes/13/773713/3","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/puppet-ovn","ref":"refs/changes/13/773713/3","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/puppet-ovn refs/changes/13/773713/3 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/puppet-ovn refs/changes/13/773713/3 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/puppet-ovn refs/changes/13/773713/3 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/puppet-ovn refs/changes/13/773713/3"}}},"commit":{"parents":[{"commit":"3aa6ee0de90a2e5e7f57224c8ea5a641967fba86","subject":"Prepare Wallaby M2","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/puppet-ovn/commit/3aa6ee0de90a2e5e7f57224c8ea5a641967fba86"}]}],"author":{"name":"Michele Baldessari","email":"michele@acksyn.org","date":"2021-02-02 15:39:34.000000000","tz":60},"committer":{"name":"Michele Baldessari","email":"michele@acksyn.org","date":"2021-02-03 07:13:21.000000000","tz":60},"subject":"Support multiple ovn bridge mappings with the same nic","message":"Support multiple ovn bridge mappings with the same nic\n\nCurrently if you pass in [\"provider1:br-ex\", \"provider2:br-ex\"] via the\npuppet ovn::controller::ovn_bridge_mappings hiera parameter, it\nwill fail with duplicate declaration errors around the exec because it\nis using only the nic \u0027br-ex\u0027 as the resource name. We fix this by\npassing both the network and the bridge to the name. The command is\nidempotent so no harm is done.\n\nThe other place is the ovn::controller::bridge define which will complain\nabout duplicate declarations. We fix this by just passing the bridge\nname directly to the ovn::controller::bridge define and we use\nensure_resource() when creating those definitions to avoid duplicates\n\nTested this as follows:\n1) Deployed a train env with this patch with a composable control plane\n with [\"datacentre:br-ex\", \"tenant:br-isolated\"] and obtained:\n\n[root@controller-0 ~]# ovs-vsctl list open . |grep -o -P -e \u0027ovn-bridge-mappings\u003d.*?, \u0027\novn-bridge-mappings\u003d\"datacentre:br-ex,tenant:br-isolated\",\n\n2) Deployed a master env with this patch with\n[\"provider1:br-ex\", \"provider2:br-ex\"] and obtained:\n\n[root@ctrl-1-0 ~]# ovs-vsctl list open . |grep -o -P -e \u0027ovn-bridge-mappings\u003d.*?, \u0027\novn-bridge-mappings\u003d\"provider1:br-ex,provider2:br-ex\",\n\nChange-Id: I66d663971f087d019139b033c032a8e05f71a57a\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/puppet-ovn/commit/ab5516e3082f0508f05e2352b53127fcb34a462a"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/puppet-ovn/commit/ab5516e3082f0508f05e2352b53127fcb34a462a"}]},"branch":"refs/heads/master"},"bf05a001cc86c70c8b47455540d5a881c687d34f":{"kind":"REWORK","_number":4,"created":"2021-02-03 11:20:29.000000000","uploader":{"_account_id":20172,"name":"Michele Baldessari","email":"michele@acksyn.org","username":"michele"},"ref":"refs/changes/13/773713/4","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/puppet-ovn","ref":"refs/changes/13/773713/4","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/puppet-ovn refs/changes/13/773713/4 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/puppet-ovn refs/changes/13/773713/4 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/puppet-ovn refs/changes/13/773713/4 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/puppet-ovn refs/changes/13/773713/4"}}},"commit":{"parents":[{"commit":"3aa6ee0de90a2e5e7f57224c8ea5a641967fba86","subject":"Prepare Wallaby M2","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/puppet-ovn/commit/3aa6ee0de90a2e5e7f57224c8ea5a641967fba86"}]}],"author":{"name":"Michele Baldessari","email":"michele@acksyn.org","date":"2021-02-02 15:39:34.000000000","tz":60},"committer":{"name":"Michele Baldessari","email":"michele@acksyn.org","date":"2021-02-03 11:20:19.000000000","tz":60},"subject":"Support multiple ovn bridge mappings with the same nic","message":"Support multiple ovn bridge mappings with the same nic\n\nCurrently if you pass in [\"provider1:br-ex\", \"provider2:br-ex\"] via the\npuppet ovn::controller::ovn_bridge_mappings hiera parameter, it\nwill fail with duplicate declaration errors around the exec because it\nis using only the nic \u0027br-ex\u0027 as the resource name. We fix this by\npassing both the network and the bridge to the name. The command is\nidempotent so no harm is done.\n\nThe other place is the ovn::controller::bridge define which will complain\nabout duplicate declarations. We fix this by just passing the bridge\nname directly to the ovn::controller::bridge define and we use\nensure_resource() when creating those definitions to avoid duplicates\n\nTested this as follows:\n1) Deployed a train env with this patch with a composable control plane\n with [\"datacentre:br-ex\", \"tenant:br-isolated\"] and obtained:\n\n[root@controller-0 ~]# ovs-vsctl list open . |grep -o -P -e \u0027ovn-bridge-mappings\u003d.*?, \u0027\novn-bridge-mappings\u003d\"datacentre:br-ex,tenant:br-isolated\",\n\n2) Deployed a master env with this patch with\n[\"provider1:br-ex\", \"provider2:br-ex\"] and obtained:\n\n[root@ctrl-1-0 ~]# ovs-vsctl list open . |grep -o -P -e \u0027ovn-bridge-mappings\u003d.*?, \u0027\novn-bridge-mappings\u003d\"provider1:br-ex,provider2:br-ex\",\n\nChange-Id: I66d663971f087d019139b033c032a8e05f71a57a\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/puppet-ovn/commit/bf05a001cc86c70c8b47455540d5a881c687d34f"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/puppet-ovn/commit/bf05a001cc86c70c8b47455540d5a881c687d34f"}]},"branch":"refs/heads/master"},"9ae588ef69b2b77da704aeef8d4d5c9ddb760905":{"kind":"REWORK","_number":5,"created":"2021-02-03 12:24:08.000000000","uploader":{"_account_id":20172,"name":"Michele Baldessari","email":"michele@acksyn.org","username":"michele"},"ref":"refs/changes/13/773713/5","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/puppet-ovn","ref":"refs/changes/13/773713/5","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/puppet-ovn refs/changes/13/773713/5 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/puppet-ovn refs/changes/13/773713/5 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/puppet-ovn refs/changes/13/773713/5 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/puppet-ovn refs/changes/13/773713/5"}}},"commit":{"parents":[{"commit":"3aa6ee0de90a2e5e7f57224c8ea5a641967fba86","subject":"Prepare Wallaby M2","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/puppet-ovn/commit/3aa6ee0de90a2e5e7f57224c8ea5a641967fba86"}]}],"author":{"name":"Michele Baldessari","email":"michele@acksyn.org","date":"2021-02-02 15:39:34.000000000","tz":60},"committer":{"name":"Michele Baldessari","email":"michele@acksyn.org","date":"2021-02-03 12:23:55.000000000","tz":60},"subject":"Support multiple ovn bridge mappings with the same nic","message":"Support multiple ovn bridge mappings with the same nic\n\nCurrently if you pass in [\"provider1:br-ex\", \"provider2:br-ex\"] via the\npuppet ovn::controller::ovn_bridge_mappings hiera parameter, it\nwill fail with duplicate declaration errors around the exec because it\nis using only the nic \u0027br-ex\u0027 as the resource name. We fix this by\npassing both the network and the bridge to the name. The command is\nidempotent so no harm is done.\n\nThe other place is the ovn::controller::bridge define which will complain\nabout duplicate declarations. We fix this by just passing the bridge\nname directly to the ovn::controller::bridge define and we use\nensure_resource() when creating those definitions to avoid duplicates\n\nTested this as follows:\n1) Deployed a train env with this patch with a composable control plane\n with [\"datacentre:br-ex\", \"tenant:br-isolated\"] and obtained:\n\n[root@controller-0 ~]# ovs-vsctl list open . |grep -o -P -e \u0027ovn-bridge-mappings\u003d.*?, \u0027\novn-bridge-mappings\u003d\"datacentre:br-ex,tenant:br-isolated\",\n\n2) Deployed a master env with this patch with\n[\"provider1:br-ex\", \"provider2:br-ex\"] and obtained:\n\n[root@ctrl-1-0 ~]# ovs-vsctl list open . |grep -o -P -e \u0027ovn-bridge-mappings\u003d.*?, \u0027\novn-bridge-mappings\u003d\"provider1:br-ex,provider2:br-ex\",\n\nChange-Id: I66d663971f087d019139b033c032a8e05f71a57a\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/puppet-ovn/commit/9ae588ef69b2b77da704aeef8d4d5c9ddb760905"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/puppet-ovn/commit/9ae588ef69b2b77da704aeef8d4d5c9ddb760905"}]},"branch":"refs/heads/master"},"1388d9eae38333bfdf998f2aa14be13f237705ad":{"kind":"REWORK","_number":6,"created":"2021-02-15 08:41:16.000000000","uploader":{"_account_id":20172,"name":"Michele Baldessari","email":"michele@acksyn.org","username":"michele"},"ref":"refs/changes/13/773713/6","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/puppet-ovn","ref":"refs/changes/13/773713/6","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/puppet-ovn refs/changes/13/773713/6 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/puppet-ovn refs/changes/13/773713/6 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/puppet-ovn refs/changes/13/773713/6 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/puppet-ovn refs/changes/13/773713/6"}}},"commit":{"parents":[{"commit":"3aa6ee0de90a2e5e7f57224c8ea5a641967fba86","subject":"Prepare Wallaby M2","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/puppet-ovn/commit/3aa6ee0de90a2e5e7f57224c8ea5a641967fba86"}]}],"author":{"name":"Michele Baldessari","email":"michele@acksyn.org","date":"2021-02-02 15:39:34.000000000","tz":60},"committer":{"name":"Michele Baldessari","email":"michele@acksyn.org","date":"2021-02-15 08:40:59.000000000","tz":60},"subject":"Support multiple ovn bridge mappings with the same nic","message":"Support multiple ovn bridge mappings with the same nic\n\nCurrently if you pass in [\"provider1:br-ex\", \"provider2:br-ex\"] via the\npuppet ovn::controller::ovn_bridge_mappings hiera parameter, it\nwill fail with duplicate declaration errors around the exec because it\nis using only the nic \u0027br-ex\u0027 as the resource name. We fix this by\npassing both the network and the bridge to the name. The command is\nidempotent so no harm is done.\n\nThe other place is the ovn::controller::bridge define which will complain\nabout duplicate declarations. We fix this by just passing the bridge\nname directly to the ovn::controller::bridge define and we use\nensure_resource() when creating those definitions to avoid duplicates\n\nTested this as follows:\n1) Deployed a train env with this patch with a composable control plane\n with [\"datacentre:br-ex\", \"tenant:br-isolated\"] and obtained:\n\n[root@controller-0 ~]# ovs-vsctl list open . |grep -o -P -e \u0027ovn-bridge-mappings\u003d.*?, \u0027\novn-bridge-mappings\u003d\"datacentre:br-ex,tenant:br-isolated\",\n\n2) Deployed a master env with this patch with\n[\"provider1:br-ex\", \"provider2:br-ex\"] and obtained:\n\n[root@ctrl-1-0 ~]# ovs-vsctl list open . |grep -o -P -e \u0027ovn-bridge-mappings\u003d.*?, \u0027\novn-bridge-mappings\u003d\"provider1:br-ex,provider2:br-ex\",\n\nChange-Id: I66d663971f087d019139b033c032a8e05f71a57a\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/puppet-ovn/commit/1388d9eae38333bfdf998f2aa14be13f237705ad"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/puppet-ovn/commit/1388d9eae38333bfdf998f2aa14be13f237705ad"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}