)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":23567,"name":"Luis Tomas Bolivar","email":"ltomasbo@redhat.com","username":"ltomasbo"},"change_message_id":"372045084e6cbb9a6884b9e331fbd94af5695b2f","unresolved":true,"context_lines":[{"line_number":7,"context_line":"Configure ovn sb connection for the ovn provider"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"The OVN provider requires connections to the OVN sb db in order to"},{"line_number":10,"context_line":"receive events for associating FIP/VIPs and healthchecks."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: I0b2250aa2d98e6ed02ccb7194183ebdcede97c9e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"6fe0a8c0_99d82a30","line":10,"range":{"start_line":10,"start_character":18,"end_line":10,"end_character":39},"updated":"2022-07-11 14:38:20.000000000","message":"super nit: It requires OVN NB for the FIP/VIPs, and OVN SB for the healtchecks","commit_id":"f03ed3e24468c9bfa237efb76509646bf879829c"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"4fa5499402713e8b9983c20e63d0f98b4b3323d6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"fcfee341_270b4388","updated":"2022-07-12 08:31:29.000000000","message":"Looks good overall, just need few fixes","commit_id":"f03ed3e24468c9bfa237efb76509646bf879829c"},{"author":{"_account_id":23567,"name":"Luis Tomas Bolivar","email":"ltomasbo@redhat.com","username":"ltomasbo"},"change_message_id":"bf07e90af48df4cbc188ad6c0de876e02dbbf714","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"6e8c817a_f7b6cccd","updated":"2022-07-12 08:58:49.000000000","message":"few more missing sb instead of nb","commit_id":"12bc7adc0b512781df26b842627371c0061d295b"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"94f0ba8c565bba80190d0445b05788c5100be31f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"dce7eca0_3689d9ac","updated":"2022-07-13 13:07:40.000000000","message":"The changes I\u0027ve made are all format/unit tests related so I\u0027ll vote my +2 here","commit_id":"555a09de74402ffa53c0b461820e7344537961a3"},{"author":{"_account_id":23567,"name":"Luis Tomas Bolivar","email":"ltomasbo@redhat.com","username":"ltomasbo"},"change_message_id":"645c665a07b097f5c9cf9e77654b2b22a98d3266","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"276fdd1f_5a6d599c","updated":"2022-07-12 13:49:04.000000000","message":"recheck","commit_id":"555a09de74402ffa53c0b461820e7344537961a3"},{"author":{"_account_id":23567,"name":"Luis Tomas Bolivar","email":"ltomasbo@redhat.com","username":"ltomasbo"},"change_message_id":"0722886a8f7a62eb246b9cfcd505b9d94478f60f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"e3243b8b_81574fcc","updated":"2022-07-14 06:03:30.000000000","message":"recheck","commit_id":"555a09de74402ffa53c0b461820e7344537961a3"},{"author":{"_account_id":34451,"name":"Fernando Royo","email":"froyo@redhat.com","username":"froyo"},"change_message_id":"f0016234d01a4490f3aa0ba6dc7bd612b7011e30","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"1f3f8403_4c4672a3","updated":"2022-07-14 08:46:11.000000000","message":"recheck after revert https://review.opendev.org/c/openstack/tripleo-ansible/+/849732 merged","commit_id":"555a09de74402ffa53c0b461820e7344537961a3"},{"author":{"_account_id":6681,"name":"Brent Eagles","email":"beagles@redhat.com","username":"beagles"},"change_message_id":"216ae60da5a3e664c8bf175d4818665ac329654d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"1c432802_420d92b1","in_reply_to":"dce7eca0_3689d9ac","updated":"2022-07-13 13:08:28.000000000","message":"thanks Takashi for the fixes *and* the reviews!","commit_id":"555a09de74402ffa53c0b461820e7344537961a3"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"8ecf78df98da1d224fc9b296ea882852e9158e8e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"2380949c_3d722536","in_reply_to":"e3243b8b_81574fcc","updated":"2022-07-14 06:05:25.000000000","message":"need to wait for https://review.opendev.org/c/openstack/tripleo-ansible/+/849732","commit_id":"555a09de74402ffa53c0b461820e7344537961a3"}],"spec/classes/tripleo_profile_base_octavia_provider_ovn_spec.rb":[{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"4fa5499402713e8b9983c20e63d0f98b4b3323d6","unresolved":true,"context_lines":[{"line_number":132,"context_line":"          :ovn_db_node_ips  \u003d\u003e [\u0027192.0.2.11\u0027, \u0027192.0.2.12\u0027],"},{"line_number":133,"context_line":"          :ovn_db_clustered \u003d\u003e true,"},{"line_number":134,"context_line":"          :ovn_nb_port      \u003d\u003e \u00276641\u0027,"},{"line_number":135,"context_line":"          :ovn_nb_port      \u003d\u003e \u00276642\u0027,"},{"line_number":136,"context_line":"        })"},{"line_number":137,"context_line":"      end"},{"line_number":138,"context_line":""}],"source_content_type":"text/x-ruby","patch_set":1,"id":"265d3250_eec9b2e4","line":135,"range":{"start_line":135,"start_character":15,"end_line":135,"end_character":17},"updated":"2022-07-12 08:31:29.000000000","message":"sb","commit_id":"f03ed3e24468c9bfa237efb76509646bf879829c"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"caab25eb1077dee740af6b39b6e839ca99ffb27a","unresolved":false,"context_lines":[{"line_number":132,"context_line":"          :ovn_db_node_ips  \u003d\u003e [\u0027192.0.2.11\u0027, \u0027192.0.2.12\u0027],"},{"line_number":133,"context_line":"          :ovn_db_clustered \u003d\u003e true,"},{"line_number":134,"context_line":"          :ovn_nb_port      \u003d\u003e \u00276641\u0027,"},{"line_number":135,"context_line":"          :ovn_nb_port      \u003d\u003e \u00276642\u0027,"},{"line_number":136,"context_line":"        })"},{"line_number":137,"context_line":"      end"},{"line_number":138,"context_line":""}],"source_content_type":"text/x-ruby","patch_set":1,"id":"347f1c6f_ba84b995","line":135,"range":{"start_line":135,"start_character":15,"end_line":135,"end_character":17},"in_reply_to":"265d3250_eec9b2e4","updated":"2022-07-12 09:40:03.000000000","message":"Done","commit_id":"f03ed3e24468c9bfa237efb76509646bf879829c"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"4fa5499402713e8b9983c20e63d0f98b4b3323d6","unresolved":true,"context_lines":[{"line_number":138,"context_line":""},{"line_number":139,"context_line":"      it \u0027should set octavia provider ovn nb connection using tcp\u0027 do"},{"line_number":140,"context_line":"        is_expected.to contain_class(\u0027octavia::provider::ovn\u0027).with("},{"line_number":141,"context_line":"          :ovn_nb_connection  \u003d\u003e \u0027tcp:192.0.2.11:6641,tcp:192.0.2.12:6641\u0027"},{"line_number":142,"context_line":"          :ovn_nb_connection  \u003d\u003e \u0027tcp:192.0.2.11:6642,tcp:192.0.2.12:6642\u0027"},{"line_number":143,"context_line":"        )"},{"line_number":144,"context_line":"      end"}],"source_content_type":"text/x-ruby","patch_set":1,"id":"5f5552b1_87e78797","line":141,"range":{"start_line":141,"start_character":73,"end_line":141,"end_character":74},"updated":"2022-07-12 08:31:29.000000000","message":",","commit_id":"f03ed3e24468c9bfa237efb76509646bf879829c"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"caab25eb1077dee740af6b39b6e839ca99ffb27a","unresolved":false,"context_lines":[{"line_number":138,"context_line":""},{"line_number":139,"context_line":"      it \u0027should set octavia provider ovn nb connection using tcp\u0027 do"},{"line_number":140,"context_line":"        is_expected.to contain_class(\u0027octavia::provider::ovn\u0027).with("},{"line_number":141,"context_line":"          :ovn_nb_connection  \u003d\u003e \u0027tcp:192.0.2.11:6641,tcp:192.0.2.12:6641\u0027"},{"line_number":142,"context_line":"          :ovn_nb_connection  \u003d\u003e \u0027tcp:192.0.2.11:6642,tcp:192.0.2.12:6642\u0027"},{"line_number":143,"context_line":"        )"},{"line_number":144,"context_line":"      end"}],"source_content_type":"text/x-ruby","patch_set":1,"id":"1b0220cd_5e72eb5b","line":141,"range":{"start_line":141,"start_character":73,"end_line":141,"end_character":74},"in_reply_to":"5f5552b1_87e78797","updated":"2022-07-12 09:40:03.000000000","message":"Done","commit_id":"f03ed3e24468c9bfa237efb76509646bf879829c"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"4fa5499402713e8b9983c20e63d0f98b4b3323d6","unresolved":true,"context_lines":[{"line_number":139,"context_line":"      it \u0027should set octavia provider ovn nb connection using tcp\u0027 do"},{"line_number":140,"context_line":"        is_expected.to contain_class(\u0027octavia::provider::ovn\u0027).with("},{"line_number":141,"context_line":"          :ovn_nb_connection  \u003d\u003e \u0027tcp:192.0.2.11:6641,tcp:192.0.2.12:6641\u0027"},{"line_number":142,"context_line":"          :ovn_nb_connection  \u003d\u003e \u0027tcp:192.0.2.11:6642,tcp:192.0.2.12:6642\u0027"},{"line_number":143,"context_line":"        )"},{"line_number":144,"context_line":"      end"},{"line_number":145,"context_line":"    end"}],"source_content_type":"text/x-ruby","patch_set":1,"id":"dd76b05f_14eed73a","line":142,"range":{"start_line":142,"start_character":15,"end_line":142,"end_character":17},"updated":"2022-07-12 08:31:29.000000000","message":"sb","commit_id":"f03ed3e24468c9bfa237efb76509646bf879829c"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"caab25eb1077dee740af6b39b6e839ca99ffb27a","unresolved":false,"context_lines":[{"line_number":139,"context_line":"      it \u0027should set octavia provider ovn nb connection using tcp\u0027 do"},{"line_number":140,"context_line":"        is_expected.to contain_class(\u0027octavia::provider::ovn\u0027).with("},{"line_number":141,"context_line":"          :ovn_nb_connection  \u003d\u003e \u0027tcp:192.0.2.11:6641,tcp:192.0.2.12:6641\u0027"},{"line_number":142,"context_line":"          :ovn_nb_connection  \u003d\u003e \u0027tcp:192.0.2.11:6642,tcp:192.0.2.12:6642\u0027"},{"line_number":143,"context_line":"        )"},{"line_number":144,"context_line":"      end"},{"line_number":145,"context_line":"    end"}],"source_content_type":"text/x-ruby","patch_set":1,"id":"ae5128dd_4d02e2a1","line":142,"range":{"start_line":142,"start_character":15,"end_line":142,"end_character":17},"in_reply_to":"dd76b05f_14eed73a","updated":"2022-07-12 09:40:03.000000000","message":"Done","commit_id":"f03ed3e24468c9bfa237efb76509646bf879829c"},{"author":{"_account_id":23567,"name":"Luis Tomas Bolivar","email":"ltomasbo@redhat.com","username":"ltomasbo"},"change_message_id":"bf07e90af48df4cbc188ad6c0de876e02dbbf714","unresolved":true,"context_lines":[{"line_number":102,"context_line":"          :protocol           \u003d\u003e \u0027ssl\u0027,"},{"line_number":103,"context_line":"          :ovn_db_host        \u003d\u003e \u0027192.168.123.111\u0027,"},{"line_number":104,"context_line":"          :ovn_nb_port        \u003d\u003e \u00276641\u0027,"},{"line_number":105,"context_line":"          :ovn_nb_port        \u003d\u003e \u00276642\u0027,"},{"line_number":106,"context_line":"          :ovn_nb_private_key \u003d\u003e \u0027/foo.key\u0027,"},{"line_number":107,"context_line":"          :ovn_nb_certificate \u003d\u003e \u0027/foo.pem\u0027,"},{"line_number":108,"context_line":"          :ovn_nb_ca_cert     \u003d\u003e \u0027/ca_foo.pem\u0027,"}],"source_content_type":"text/x-ruby","patch_set":2,"id":"a24043a6_afec1d5a","line":105,"range":{"start_line":105,"start_character":11,"end_line":105,"end_character":22},"updated":"2022-07-12 08:58:49.000000000","message":"should this be ovn_sb_port instead?","commit_id":"12bc7adc0b512781df26b842627371c0061d295b"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"caab25eb1077dee740af6b39b6e839ca99ffb27a","unresolved":false,"context_lines":[{"line_number":102,"context_line":"          :protocol           \u003d\u003e \u0027ssl\u0027,"},{"line_number":103,"context_line":"          :ovn_db_host        \u003d\u003e \u0027192.168.123.111\u0027,"},{"line_number":104,"context_line":"          :ovn_nb_port        \u003d\u003e \u00276641\u0027,"},{"line_number":105,"context_line":"          :ovn_nb_port        \u003d\u003e \u00276642\u0027,"},{"line_number":106,"context_line":"          :ovn_nb_private_key \u003d\u003e \u0027/foo.key\u0027,"},{"line_number":107,"context_line":"          :ovn_nb_certificate \u003d\u003e \u0027/foo.pem\u0027,"},{"line_number":108,"context_line":"          :ovn_nb_ca_cert     \u003d\u003e \u0027/ca_foo.pem\u0027,"}],"source_content_type":"text/x-ruby","patch_set":2,"id":"504cd256_40c66356","line":105,"range":{"start_line":105,"start_character":11,"end_line":105,"end_character":22},"in_reply_to":"a24043a6_afec1d5a","updated":"2022-07-12 09:40:03.000000000","message":"Done","commit_id":"12bc7adc0b512781df26b842627371c0061d295b"},{"author":{"_account_id":23567,"name":"Luis Tomas Bolivar","email":"ltomasbo@redhat.com","username":"ltomasbo"},"change_message_id":"bf07e90af48df4cbc188ad6c0de876e02dbbf714","unresolved":true,"context_lines":[{"line_number":114,"context_line":""},{"line_number":115,"context_line":"      it \u0027should set octavia provider ovn nb connection using ssl\u0027 do"},{"line_number":116,"context_line":"        is_expected.to contain_class(\u0027octavia::provider::ovn\u0027).with(:ovn_nb_connection  \u003d\u003e \u0027ssl:192.168.123.111:6641\u0027)"},{"line_number":117,"context_line":"        is_expected.to contain_class(\u0027octavia::provider::ovn\u0027).with(:ovn_nb_connection  \u003d\u003e \u0027ssl:192.168.123.111:6642\u0027)"},{"line_number":118,"context_line":"        is_expected.to contain_class(\u0027octavia::provider::ovn\u0027).with(:ovn_nb_private_key \u003d\u003e \u0027/foo.key\u0027)"},{"line_number":119,"context_line":"        is_expected.to contain_class(\u0027octavia::provider::ovn\u0027).with(:ovn_nb_certificate \u003d\u003e \u0027/foo.pem\u0027)"},{"line_number":120,"context_line":"        is_expected.to contain_class(\u0027octavia::provider::ovn\u0027).with(:ovn_nb_ca_cert     \u003d\u003e \u0027/ca_foo.pem\u0027)"}],"source_content_type":"text/x-ruby","patch_set":2,"id":"eb29df92_ffa6e053","line":117,"range":{"start_line":117,"start_character":69,"end_line":117,"end_character":87},"updated":"2022-07-12 08:58:49.000000000","message":"ditto, ovn_sb_connection","commit_id":"12bc7adc0b512781df26b842627371c0061d295b"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"caab25eb1077dee740af6b39b6e839ca99ffb27a","unresolved":false,"context_lines":[{"line_number":114,"context_line":""},{"line_number":115,"context_line":"      it \u0027should set octavia provider ovn nb connection using ssl\u0027 do"},{"line_number":116,"context_line":"        is_expected.to contain_class(\u0027octavia::provider::ovn\u0027).with(:ovn_nb_connection  \u003d\u003e \u0027ssl:192.168.123.111:6641\u0027)"},{"line_number":117,"context_line":"        is_expected.to contain_class(\u0027octavia::provider::ovn\u0027).with(:ovn_nb_connection  \u003d\u003e \u0027ssl:192.168.123.111:6642\u0027)"},{"line_number":118,"context_line":"        is_expected.to contain_class(\u0027octavia::provider::ovn\u0027).with(:ovn_nb_private_key \u003d\u003e \u0027/foo.key\u0027)"},{"line_number":119,"context_line":"        is_expected.to contain_class(\u0027octavia::provider::ovn\u0027).with(:ovn_nb_certificate \u003d\u003e \u0027/foo.pem\u0027)"},{"line_number":120,"context_line":"        is_expected.to contain_class(\u0027octavia::provider::ovn\u0027).with(:ovn_nb_ca_cert     \u003d\u003e \u0027/ca_foo.pem\u0027)"}],"source_content_type":"text/x-ruby","patch_set":2,"id":"13062b42_d6be4c9b","line":117,"range":{"start_line":117,"start_character":69,"end_line":117,"end_character":87},"in_reply_to":"eb29df92_ffa6e053","updated":"2022-07-12 09:40:03.000000000","message":"Done","commit_id":"12bc7adc0b512781df26b842627371c0061d295b"}]}