)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":22954,"name":"Juan Badia Payno","email":"jbadiapa@redhat.com","username":"jbadiapa"},"change_message_id":"b9c94e7cebe641f48edfb3d7db5c7ad55a2b1224","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"4e1af74a_40f3c14a","updated":"2022-10-04 06:17:52.000000000","message":"I think these parameters have never been used [1] or maybe their use was proposed after introduced by the patch.\n\n[1]https://review.opendev.org/c/openstack/tripleo-heat-templates/+/839974/7/deployment/unbound/unbound-container-ansible.yaml","commit_id":"c98f27b167679af646a4aa86dcf137579aa625b8"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"cfb4f2efb6c9244f67e7053cadf2c582a056970f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"b76153ee_f40bcdf8","updated":"2022-10-04 06:28:47.000000000","message":"lgtm but holding workflow - would be good to have a sanity check from the person that added these params ","commit_id":"c98f27b167679af646a4aa86dcf137579aa625b8"}],"deployment/unbound/unbound-container-ansible.yaml":[{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"cfb4f2efb6c9244f67e7053cadf2c582a056970f","unresolved":true,"context_lines":[{"line_number":47,"context_line":"  NeutronDhcpAgentDnsmasqDnsServers:"},{"line_number":48,"context_line":"    default: []"},{"line_number":49,"context_line":"    description: List of servers to use as dnsmasq forwarders"},{"line_number":50,"context_line":"    type: comma_delimited_list"},{"line_number":51,"context_line":""},{"line_number":52,"context_line":"# Unbound specific parameters"},{"line_number":53,"context_line":"  UnboundAllowedCIDRs:"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"09138fec_51ba4812","side":"PARENT","line":50,"updated":"2022-10-04 06:28:47.000000000","message":"these were added with https://opendev.org/openstack/tripleo-heat-templates/commit/78a51efb6c21c2dd3173a0ad55a9fb4873a21766\n\nlooking at this file in that commit indeed they are not referenced anywhere outside this parameter list https://opendev.org/openstack/tripleo-heat-templates/src/commit/78a51efb6c21c2dd3173a0ad55a9fb4873a21766/deployment/unbound/unbound-container-ansible.yaml\n\n\ngoing to add Beagles to the reviewers for sanity check","commit_id":"aa20889f3de9a4722afcce32aed171eb59054e5a"}]}
