)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":8655,"name":"Jakub Libosvar","email":"libosvar@redhat.com","username":"jlibosva"},"change_message_id":"b3c458451f261c30a9d10c17fb712a86fa02c253","unresolved":true,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This patch adds support to toggle the liveness report from"},{"line_number":10,"context_line":"the Neutron OVN metadata agent."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Signed-off-by: Daniel Alvarez Sanchez \u003cdalvarez@redhat.com\u003e"},{"line_number":13,"context_line":"Change-Id: Ie66c54a8c20b43bf8359ecad08687290bd422710"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"e25c9734_b31e387c","line":11,"updated":"2021-06-16 11:35:24.000000000","message":"Maybe we shall add \"Related-Bug: #1931133\"","commit_id":"870b64b2e35a614baa035dea3066192cc103ad67"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"db813e276da03388d575d8414555ca65c0207889","unresolved":true,"context_lines":[{"line_number":4,"context_line":"Commit:     Daniel Alvarez Sanchez \u003cdalvarez@redhat.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2021-06-17 10:18:34 +0200"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"[ovn metadata] Add report agent option"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This patch adds support to toggle the liveness report from"},{"line_number":10,"context_line":"the Neutron OVN metadata agent."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"d9f77b64_1c4ce959","line":7,"range":{"start_line":7,"start_character":19,"end_line":7,"end_character":31},"updated":"2021-06-17 08:56:14.000000000","message":"report_agent(actually parmaeter name) would be better.","commit_id":"b974153f34fefef17de465c8ee2c2a2616e8b9d0"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"194fc3fdaa5eb3f1f8bf2c4fd5bcf0b04d13d087","unresolved":true,"context_lines":[{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Depends-On: Ia996ebee183975b38ca101ca78ee33854051f0a2"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Related-Bug: #1931133"},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Signed-off-by: Daniel Alvarez Sanchez \u003cdalvarez@redhat.com\u003e"},{"line_number":17,"context_line":"Change-Id: Ie66c54a8c20b43bf8359ecad08687290bd422710"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"ec464d35_3eb6865f","line":14,"updated":"2021-06-22 13:44:53.000000000","message":"Is this related to 1931133 ? The dependent change closes bug 1929578 and I\u0027m just curious about relationship between these two bugs.","commit_id":"0d000530bb042d9031eb09e2fb51533ac286c1ef"},{"author":{"_account_id":8655,"name":"Jakub Libosvar","email":"libosvar@redhat.com","username":"jlibosva"},"change_message_id":"9d9ab761acbcb9ba59751250bc83e65de66541a6","unresolved":true,"context_lines":[{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Depends-On: Ia996ebee183975b38ca101ca78ee33854051f0a2"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Related-Bug: #1931133"},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Signed-off-by: Daniel Alvarez Sanchez \u003cdalvarez@redhat.com\u003e"},{"line_number":17,"context_line":"Change-Id: Ie66c54a8c20b43bf8359ecad08687290bd422710"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"dbf2793b_f68ac16c","line":14,"in_reply_to":"ec464d35_3eb6865f","updated":"2021-10-01 20:10:41.000000000","message":"1931133 is about read-only option to avoid writing to SB DB while 1929578 is to be able to configure multiple connections","commit_id":"0d000530bb042d9031eb09e2fb51533ac286c1ef"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"d80ebf7879349d4926cee8d36f5069bc1b1cf1e4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"80974a8d_79f0feac","updated":"2022-01-12 04:53:14.000000000","message":"The parent patch is not yet merged so I\u0027ll keep -2 to avoid merging this after any unexpected change in neutron side. The dependent patch has been inactive for a while but do we still need this ?","commit_id":"43ebb400cf47c3bb947af4259752c244f3435f44"},{"author":{"_account_id":8655,"name":"Jakub Libosvar","email":"libosvar@redhat.com","username":"jlibosva"},"change_message_id":"6f666e3f0086074fd0fcb683e54d69ccd3edce93","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"a49a4862_7c361e3e","updated":"2022-04-04 19:36:10.000000000","message":"This can be abandoned as the Neutron patch will not be merged","commit_id":"43ebb400cf47c3bb947af4259752c244f3435f44"},{"author":{"_account_id":6681,"name":"Brent Eagles","email":"beagles@redhat.com","username":"beagles"},"change_message_id":"f10c6672c0509757e95deb1bac9fbb3669fdf1cb","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"7b6ed1c9_cd05af61","updated":"2021-11-04 11:56:37.000000000","message":"lgtm @Daniel @Kamil - what\u0027s the story with the neutron patch this depends on?","commit_id":"43ebb400cf47c3bb947af4259752c244f3435f44"}],"manifests/agents/ovn_metadata.pp":[{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"da3dc101c1c21c2e0b6f19536e77dcbc7dbc2842","unresolved":true,"context_lines":[{"line_number":106,"context_line":"#   in the metadata config."},{"line_number":107,"context_line":"#   Defaults to false."},{"line_number":108,"context_line":"#"},{"line_number":109,"context_line":"# [*ovn_remote_probe_interval*]"},{"line_number":110,"context_line":"#  (optional) Set probe interval, based on user configuration, value is in ms"},{"line_number":111,"context_line":"#  Defaults to 60000"},{"line_number":112,"context_line":"#"},{"line_number":113,"context_line":"# [*agent_report*]"},{"line_number":114,"context_line":"#   (optional) Whether to report liveness of the metadata agent."},{"line_number":115,"context_line":"#   Defaults to true."}],"source_content_type":"text/x-puppet","patch_set":1,"id":"cc57b7df_be1e15a6","line":112,"range":{"start_line":109,"start_character":0,"end_line":112,"end_character":1},"updated":"2021-06-16 11:57:09.000000000","message":"This is a side note but I could not find this parameter in neutron, either...","commit_id":"870b64b2e35a614baa035dea3066192cc103ad67"},{"author":{"_account_id":23804,"name":"Daniel Alvarez","email":"dalvarez@redhat.com","username":"dalvarez"},"change_message_id":"8c7abf99e601d091012037d3d2183ab58be2894b","unresolved":false,"context_lines":[{"line_number":106,"context_line":"#   in the metadata config."},{"line_number":107,"context_line":"#   Defaults to false."},{"line_number":108,"context_line":"#"},{"line_number":109,"context_line":"# [*ovn_remote_probe_interval*]"},{"line_number":110,"context_line":"#  (optional) Set probe interval, based on user configuration, value is in ms"},{"line_number":111,"context_line":"#  Defaults to 60000"},{"line_number":112,"context_line":"#"},{"line_number":113,"context_line":"# [*agent_report*]"},{"line_number":114,"context_line":"#   (optional) Whether to report liveness of the metadata agent."},{"line_number":115,"context_line":"#   Defaults to true."}],"source_content_type":"text/x-puppet","patch_set":1,"id":"9dd9943e_73a29821","line":112,"range":{"start_line":109,"start_character":0,"end_line":112,"end_character":1},"in_reply_to":"0752527a_8d1e4b08","updated":"2021-06-17 07:50:42.000000000","message":"Ack","commit_id":"870b64b2e35a614baa035dea3066192cc103ad67"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"9e60e5f07f1f384b528e7c17b0c14f6fdd1fcf31","unresolved":true,"context_lines":[{"line_number":106,"context_line":"#   in the metadata config."},{"line_number":107,"context_line":"#   Defaults to false."},{"line_number":108,"context_line":"#"},{"line_number":109,"context_line":"# [*ovn_remote_probe_interval*]"},{"line_number":110,"context_line":"#  (optional) Set probe interval, based on user configuration, value is in ms"},{"line_number":111,"context_line":"#  Defaults to 60000"},{"line_number":112,"context_line":"#"},{"line_number":113,"context_line":"# [*agent_report*]"},{"line_number":114,"context_line":"#   (optional) Whether to report liveness of the metadata agent."},{"line_number":115,"context_line":"#   Defaults to true."}],"source_content_type":"text/x-puppet","patch_set":1,"id":"0752527a_8d1e4b08","line":112,"range":{"start_line":109,"start_character":0,"end_line":112,"end_character":1},"in_reply_to":"337dcddb_b5f14cb0","updated":"2021-06-16 12:49:51.000000000","message":"OK. I have submitted the patch to deprecate/remove this.\n\nhttps://review.opendev.org/c/openstack/puppet-neutron/+/796648","commit_id":"870b64b2e35a614baa035dea3066192cc103ad67"},{"author":{"_account_id":11082,"name":"Kamil Sambor","email":"ksambor@redhat.com","username":"ksambor"},"change_message_id":"5a285167720868b3d0b19ee91d801f55fd1dedcf","unresolved":true,"context_lines":[{"line_number":106,"context_line":"#   in the metadata config."},{"line_number":107,"context_line":"#   Defaults to false."},{"line_number":108,"context_line":"#"},{"line_number":109,"context_line":"# [*ovn_remote_probe_interval*]"},{"line_number":110,"context_line":"#  (optional) Set probe interval, based on user configuration, value is in ms"},{"line_number":111,"context_line":"#  Defaults to 60000"},{"line_number":112,"context_line":"#"},{"line_number":113,"context_line":"# [*agent_report*]"},{"line_number":114,"context_line":"#   (optional) Whether to report liveness of the metadata agent."},{"line_number":115,"context_line":"#   Defaults to true."}],"source_content_type":"text/x-puppet","patch_set":1,"id":"337dcddb_b5f14cb0","line":112,"range":{"start_line":109,"start_character":0,"end_line":112,"end_character":1},"in_reply_to":"cc57b7df_be1e15a6","updated":"2021-06-16 12:37:40.000000000","message":"This is some old leftover which we can remove in follow-up patch, because we are setting this value through puppet-ovn https://opendev.org/openstack/puppet-ovn/src/branch/master/manifests/controller.pp#L140","commit_id":"870b64b2e35a614baa035dea3066192cc103ad67"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"da3dc101c1c21c2e0b6f19536e77dcbc7dbc2842","unresolved":true,"context_lines":[{"line_number":109,"context_line":"# [*ovn_remote_probe_interval*]"},{"line_number":110,"context_line":"#  (optional) Set probe interval, based on user configuration, value is in ms"},{"line_number":111,"context_line":"#  Defaults to 60000"},{"line_number":112,"context_line":"#"},{"line_number":113,"context_line":"# [*agent_report*]"},{"line_number":114,"context_line":"#   (optional) Whether to report liveness of the metadata agent."},{"line_number":115,"context_line":"#   Defaults to true."},{"line_number":116,"context_line":"#"},{"line_number":117,"context_line":"class neutron::agents::ovn_metadata ("},{"line_number":118,"context_line":"  $shared_secret,"},{"line_number":119,"context_line":"  $package_ensure            \u003d \u0027present\u0027,"}],"source_content_type":"text/x-puppet","patch_set":1,"id":"e7a132a0_634f3147","line":116,"range":{"start_line":112,"start_character":1,"end_line":116,"end_character":1},"updated":"2021-06-16 11:57:09.000000000","message":"Can we add this parameter just after root_helper_daemon to keep parameter order consistent ?\n\nAlso, I was looking for this parameter incurrent neutron code but I coulnd not find this. Are you adding this parameter to neutron as well ? Then this should have proper depends-on.","commit_id":"870b64b2e35a614baa035dea3066192cc103ad67"},{"author":{"_account_id":23804,"name":"Daniel Alvarez","email":"dalvarez@redhat.com","username":"dalvarez"},"change_message_id":"8c7abf99e601d091012037d3d2183ab58be2894b","unresolved":false,"context_lines":[{"line_number":109,"context_line":"# [*ovn_remote_probe_interval*]"},{"line_number":110,"context_line":"#  (optional) Set probe interval, based on user configuration, value is in ms"},{"line_number":111,"context_line":"#  Defaults to 60000"},{"line_number":112,"context_line":"#"},{"line_number":113,"context_line":"# [*agent_report*]"},{"line_number":114,"context_line":"#   (optional) Whether to report liveness of the metadata agent."},{"line_number":115,"context_line":"#   Defaults to true."},{"line_number":116,"context_line":"#"},{"line_number":117,"context_line":"class neutron::agents::ovn_metadata ("},{"line_number":118,"context_line":"  $shared_secret,"},{"line_number":119,"context_line":"  $package_ensure            \u003d \u0027present\u0027,"}],"source_content_type":"text/x-puppet","patch_set":1,"id":"c3ef72a1_b5238011","line":116,"range":{"start_line":112,"start_character":1,"end_line":116,"end_character":1},"in_reply_to":"5daf3459_69017cb8","updated":"2021-06-17 07:50:42.000000000","message":"Done","commit_id":"870b64b2e35a614baa035dea3066192cc103ad67"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"9e60e5f07f1f384b528e7c17b0c14f6fdd1fcf31","unresolved":true,"context_lines":[{"line_number":109,"context_line":"# [*ovn_remote_probe_interval*]"},{"line_number":110,"context_line":"#  (optional) Set probe interval, based on user configuration, value is in ms"},{"line_number":111,"context_line":"#  Defaults to 60000"},{"line_number":112,"context_line":"#"},{"line_number":113,"context_line":"# [*agent_report*]"},{"line_number":114,"context_line":"#   (optional) Whether to report liveness of the metadata agent."},{"line_number":115,"context_line":"#   Defaults to true."},{"line_number":116,"context_line":"#"},{"line_number":117,"context_line":"class neutron::agents::ovn_metadata ("},{"line_number":118,"context_line":"  $shared_secret,"},{"line_number":119,"context_line":"  $package_ensure            \u003d \u0027present\u0027,"}],"source_content_type":"text/x-puppet","patch_set":1,"id":"5daf3459_69017cb8","line":116,"range":{"start_line":112,"start_character":1,"end_line":116,"end_character":1},"in_reply_to":"c6f06fa0_f25895a2","updated":"2021-06-16 12:49:51.000000000","message":"Ah thanks. Then we need depends on as well as fixing the parameter name.\n\nIn addition, I prefer using $::os_service_default instead of hardcoding the default in puppet layer. This saves us from updating default every time change is made in service side.","commit_id":"870b64b2e35a614baa035dea3066192cc103ad67"},{"author":{"_account_id":11082,"name":"Kamil Sambor","email":"ksambor@redhat.com","username":"ksambor"},"change_message_id":"5a285167720868b3d0b19ee91d801f55fd1dedcf","unresolved":true,"context_lines":[{"line_number":109,"context_line":"# [*ovn_remote_probe_interval*]"},{"line_number":110,"context_line":"#  (optional) Set probe interval, based on user configuration, value is in ms"},{"line_number":111,"context_line":"#  Defaults to 60000"},{"line_number":112,"context_line":"#"},{"line_number":113,"context_line":"# [*agent_report*]"},{"line_number":114,"context_line":"#   (optional) Whether to report liveness of the metadata agent."},{"line_number":115,"context_line":"#   Defaults to true."},{"line_number":116,"context_line":"#"},{"line_number":117,"context_line":"class neutron::agents::ovn_metadata ("},{"line_number":118,"context_line":"  $shared_secret,"},{"line_number":119,"context_line":"  $package_ensure            \u003d \u0027present\u0027,"}],"source_content_type":"text/x-puppet","patch_set":1,"id":"c6f06fa0_f25895a2","line":116,"range":{"start_line":112,"start_character":1,"end_line":116,"end_character":1},"in_reply_to":"e7a132a0_634f3147","updated":"2021-06-16 12:37:40.000000000","message":"true this parameter is used in https://review.opendev.org/c/openstack/neutron/+/792998/3/neutron/agent/ovn/metadata_agent.py#33 (but value there is report_agent)","commit_id":"870b64b2e35a614baa035dea3066192cc103ad67"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"db813e276da03388d575d8414555ca65c0207889","unresolved":true,"context_lines":[{"line_number":168,"context_line":"    \u0027DEFAULT/state_path\u0027:                     value \u003d\u003e $state_path;"},{"line_number":169,"context_line":"    \u0027agent/root_helper\u0027:                      value \u003d\u003e $root_helper;"},{"line_number":170,"context_line":"    \u0027agent/root_helper_daemon\u0027:               value \u003d\u003e $root_helper_daemon;"},{"line_number":171,"context_line":"    \u0027agent/report_agent\u0027:                     value \u003d\u003e $report_agent;"},{"line_number":172,"context_line":"    \u0027ovs/ovsdb_connection\u0027:                   value \u003d\u003e $ovsdb_connection;"},{"line_number":173,"context_line":"    \u0027ovs/ovsdb_connection_timeout\u0027:           value \u003d\u003e $ovsdb_connection_timeout;"},{"line_number":174,"context_line":"    \u0027ovn/ovn_sb_connection\u0027:                  value \u003d\u003e $ovn_sb_connection;"}],"source_content_type":"text/x-puppet","patch_set":3,"id":"c2913249_07766e2d","line":171,"range":{"start_line":171,"start_character":4,"end_line":171,"end_character":69},"updated":"2021-06-17 08:56:14.000000000","message":"Please put this before root_helper to be consistent with parameter definition.","commit_id":"b974153f34fefef17de465c8ee2c2a2616e8b9d0"},{"author":{"_account_id":23804,"name":"Daniel Alvarez","email":"dalvarez@redhat.com","username":"dalvarez"},"change_message_id":"96855933c5c62fbff9d624a04f7f3272ea946bfe","unresolved":true,"context_lines":[{"line_number":168,"context_line":"    \u0027DEFAULT/state_path\u0027:                     value \u003d\u003e $state_path;"},{"line_number":169,"context_line":"    \u0027agent/root_helper\u0027:                      value \u003d\u003e $root_helper;"},{"line_number":170,"context_line":"    \u0027agent/root_helper_daemon\u0027:               value \u003d\u003e $root_helper_daemon;"},{"line_number":171,"context_line":"    \u0027agent/report_agent\u0027:                     value \u003d\u003e $report_agent;"},{"line_number":172,"context_line":"    \u0027ovs/ovsdb_connection\u0027:                   value \u003d\u003e $ovsdb_connection;"},{"line_number":173,"context_line":"    \u0027ovs/ovsdb_connection_timeout\u0027:           value \u003d\u003e $ovsdb_connection_timeout;"},{"line_number":174,"context_line":"    \u0027ovn/ovn_sb_connection\u0027:                  value \u003d\u003e $ovn_sb_connection;"}],"source_content_type":"text/x-puppet","patch_set":3,"id":"b03e7de8_6c6e611c","line":171,"range":{"start_line":171,"start_character":4,"end_line":171,"end_character":69},"in_reply_to":"c2913249_07766e2d","updated":"2021-06-17 09:00:15.000000000","message":"ops right,thanks!","commit_id":"b974153f34fefef17de465c8ee2c2a2616e8b9d0"}],"manifests/agents/releasenotes/notes/ovn-metadata-agent-report-option-6cefdf6fec276267.yaml":[{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"db813e276da03388d575d8414555ca65c0207889","unresolved":true,"context_lines":[{"line_number":2,"context_line":"other:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    Add the option to toggle the liveness reports from the ``Neutron OVN"},{"line_number":5,"context_line":"    metadata agent`` to the server. By default, the reports are enabled."}],"source_content_type":"text/x-yaml","patch_set":3,"id":"da0ed193_c4e71ea2","line":5,"range":{"start_line":5,"start_character":36,"end_line":5,"end_character":72},"updated":"2021-06-17 08:56:14.000000000","message":"This is not behavior of puppet-neutron but one of neutron, so please remove this.","commit_id":"b974153f34fefef17de465c8ee2c2a2616e8b9d0"},{"author":{"_account_id":23804,"name":"Daniel Alvarez","email":"dalvarez@redhat.com","username":"dalvarez"},"change_message_id":"96855933c5c62fbff9d624a04f7f3272ea946bfe","unresolved":true,"context_lines":[{"line_number":2,"context_line":"other:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    Add the option to toggle the liveness reports from the ``Neutron OVN"},{"line_number":5,"context_line":"    metadata agent`` to the server. By default, the reports are enabled."}],"source_content_type":"text/x-yaml","patch_set":3,"id":"6066f5d1_e881c9f0","line":5,"range":{"start_line":5,"start_character":36,"end_line":5,"end_character":72},"in_reply_to":"da0ed193_c4e71ea2","updated":"2021-06-17 09:00:15.000000000","message":"agreed","commit_id":"b974153f34fefef17de465c8ee2c2a2616e8b9d0"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"14ed0e4be9b6e1163f3bcca535a58a7a200d1162","unresolved":true,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"other:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    Add the option to toggle the liveness reports from the ``Neutron OVN"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"253cb94a_faeab00a","line":1,"range":{"start_line":1,"start_character":0,"end_line":1,"end_character":3},"updated":"2021-10-05 01:03:46.000000000","message":"This file is placed in a wrong path(should be placed under releasenotes","commit_id":"0d000530bb042d9031eb09e2fb51533ac286c1ef"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"1da8ea8545ea04c872554164fbdbbac4f707cc6d","unresolved":true,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"other:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    Add the option to toggle the liveness reports from the ``Neutron OVN"},{"line_number":5,"context_line":"    metadata agent`` to the server."}],"source_content_type":"text/x-yaml","patch_set":4,"id":"0b4cd4c5_6ce077e8","line":2,"range":{"start_line":2,"start_character":0,"end_line":2,"end_character":5},"updated":"2021-10-05 01:02:53.000000000","message":"Use \"features\" instead.","commit_id":"0d000530bb042d9031eb09e2fb51533ac286c1ef"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"1da8ea8545ea04c872554164fbdbbac4f707cc6d","unresolved":true,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"other:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    Add the option to toggle the liveness reports from the ``Neutron OVN"},{"line_number":5,"context_line":"    metadata agent`` to the server."}],"source_content_type":"text/x-yaml","patch_set":4,"id":"5d7e82ea_e1c97af6","line":4,"range":{"start_line":4,"start_character":4,"end_line":4,"end_character":28},"updated":"2021-10-05 01:02:53.000000000","message":"I prefer explaining more clearly puppet-neutron is just providing an interface to set this option.","commit_id":"0d000530bb042d9031eb09e2fb51533ac286c1ef"}],"spec/classes/neutron_agents_ovn_metadata_spec.rb":[{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"da3dc101c1c21c2e0b6f19536e77dcbc7dbc2842","unresolved":true,"context_lines":[{"line_number":15,"context_line":"      :ovsdb_connection \u003d\u003e \u0027tcp:127.0.0.1:6640\u0027,"},{"line_number":16,"context_line":"      :root_helper      \u003d\u003e \u0027sudo neutron-rootwrap /etc/neutron/rootwrap.conf\u0027,"},{"line_number":17,"context_line":"      :state_path       \u003d\u003e \u0027/var/lib/neutron/\u0027,"},{"line_number":18,"context_line":"      :agent_report     \u003d\u003e true,"},{"line_number":19,"context_line":"    }"},{"line_number":20,"context_line":"  end"},{"line_number":21,"context_line":""}],"source_content_type":"text/x-ruby","patch_set":1,"id":"3d0b5aac_0e8efa08","line":18,"range":{"start_line":18,"start_character":6,"end_line":18,"end_character":32},"updated":"2021-06-16 11:57:09.000000000","message":"This is the current problem with this unit test but for most of parameters only default is tested. I think it\u0027s better to remove this now and use default.","commit_id":"870b64b2e35a614baa035dea3066192cc103ad67"},{"author":{"_account_id":23804,"name":"Daniel Alvarez","email":"dalvarez@redhat.com","username":"dalvarez"},"change_message_id":"8c7abf99e601d091012037d3d2183ab58be2894b","unresolved":false,"context_lines":[{"line_number":15,"context_line":"      :ovsdb_connection \u003d\u003e \u0027tcp:127.0.0.1:6640\u0027,"},{"line_number":16,"context_line":"      :root_helper      \u003d\u003e \u0027sudo neutron-rootwrap /etc/neutron/rootwrap.conf\u0027,"},{"line_number":17,"context_line":"      :state_path       \u003d\u003e \u0027/var/lib/neutron/\u0027,"},{"line_number":18,"context_line":"      :agent_report     \u003d\u003e true,"},{"line_number":19,"context_line":"    }"},{"line_number":20,"context_line":"  end"},{"line_number":21,"context_line":""}],"source_content_type":"text/x-ruby","patch_set":1,"id":"a4b772a8_041e5598","line":18,"range":{"start_line":18,"start_character":6,"end_line":18,"end_character":32},"in_reply_to":"3d0b5aac_0e8efa08","updated":"2021-06-17 07:50:42.000000000","message":"done","commit_id":"870b64b2e35a614baa035dea3066192cc103ad67"}]}