)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":8833,"name":"Rabi Mishra","email":"ramishra@redhat.com","username":"rabi"},"change_message_id":"1d916e9d3e4df31f4e89ac458a9445bf60baa92f","unresolved":true,"context_lines":[{"line_number":7,"context_line":"Ceilometer: Remove snmpd_* parameters"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"These parameters were used by GenericHardwareDeclarativePollster in"},{"line_number":10,"context_line":"undercloud but this setup is no longer valid since Telemetry services"},{"line_number":11,"context_line":"were removed from Undercloud."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"This change removes unused snmpd parameters from the ceilometer service"},{"line_number":14,"context_line":"which is now used only in overcloud."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"e206b8f3_9201cc9e","line":11,"range":{"start_line":10,"start_character":38,"end_line":11,"end_character":29},"updated":"2022-08-08 03:55:29.000000000","message":"I did not know we ever had telemetry services in the undercloud. Also, the patch[1] that added these parameter in THT does not talk about anything like that.\n\nTelemetry team seems to be fine to remove these;)\n\n[1] https://review.opendev.org/c/openstack/tripleo-heat-templates/+/462237","commit_id":"8592fc32a3068a668a96a85c3cad3cc57d69f9b8"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"08caee1493fc8cd1dac17887fb9c01ec51112b80","unresolved":true,"context_lines":[{"line_number":7,"context_line":"Ceilometer: Remove snmpd_* parameters"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"These parameters were used by GenericHardwareDeclarativePollster in"},{"line_number":10,"context_line":"undercloud but this setup is no longer valid since Telemetry services"},{"line_number":11,"context_line":"were removed from Undercloud."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"This change removes unused snmpd parameters from the ceilometer service"},{"line_number":14,"context_line":"which is now used only in overcloud."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"fa4d0c09_a1c83b53","line":11,"range":{"start_line":10,"start_character":38,"end_line":11,"end_character":29},"in_reply_to":"e206b8f3_9201cc9e","updated":"2022-08-08 04:30:36.000000000","message":"AFAIR Telemetry services were enabled by default in Undercloud in Newton.\n\nAlso the usage of snmpd + undercloud temeletry is documented here: https://github.com/openstack/tripleo-heat-templates/blob/master/deployment/snmp/snmp-baremetal-puppet.yaml#L4-L6","commit_id":"8592fc32a3068a668a96a85c3cad3cc57d69f9b8"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":7294,"name":"Leif Madsen","email":"lmadsen@redhat.com","username":"leifmadsen"},"change_message_id":"c7a37fb212d01dc57f2e9fee2ab2eb4714b4fa48","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"3b87af47_80465d11","updated":"2022-06-23 15:27:37.000000000","message":"Will this be backported to Wallaby?","commit_id":"8592fc32a3068a668a96a85c3cad3cc57d69f9b8"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"07bc2538a9cfe56d18e7d0aa75c41ec64f02612c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"28df28b6_88c718a8","updated":"2022-06-23 15:20:28.000000000","message":"recheck","commit_id":"8592fc32a3068a668a96a85c3cad3cc57d69f9b8"},{"author":{"_account_id":4264,"name":"Matthias Runge","email":"mrunge@redhat.com","username":"mrunge"},"change_message_id":"34ea47d8236ef4c26ec6bb8f7ac36cf1cdf43cba","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"1457534d_9bf22099","updated":"2022-06-23 16:02:26.000000000","message":"thank you","commit_id":"8592fc32a3068a668a96a85c3cad3cc57d69f9b8"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"43671d8f7218badd3cdcdb7ed45183bbd9c97646","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"51f69ee6_e79bd952","in_reply_to":"3b87af47_80465d11","updated":"2022-06-23 23:45:41.000000000","message":"Yes. These parameters are set but have been unused since we remove ceilometer from undercloud.\n\nNote that this change does not touch deployment of snmpd itself, which should be covered separately.","commit_id":"8592fc32a3068a668a96a85c3cad3cc57d69f9b8"},{"author":{"_account_id":7294,"name":"Leif Madsen","email":"lmadsen@redhat.com","username":"leifmadsen"},"change_message_id":"d00cc9b055bfcb1c2c88d647075083cd47471b46","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"9e06e5d8_cd5b8e04","in_reply_to":"51f69ee6_e79bd952","updated":"2022-07-04 12:33:30.000000000","message":"Awesome, thanks for following up and clarifying!","commit_id":"8592fc32a3068a668a96a85c3cad3cc57d69f9b8"}]}