)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"e27ac2fa59d260bd78af2af1eeb6d035e877dd81","unresolved":true,"context_lines":[{"line_number":7,"context_line":"Add new paramater \"CeilometerEnableTenantDiscovery\""},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Enabling this parameter will identify user and project"},{"line_number":10,"context_line":"names from the polled metrics [1]."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"These details are collected by making additional requests"},{"line_number":13,"context_line":"to keystone service, depending upon the scale of environment,"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"393b5585_51c03278","line":10,"range":{"start_line":10,"start_character":31,"end_line":10,"end_character":32},"updated":"2022-09-30 06:23:52.000000000","message":"should that be a depends-on then?","commit_id":"9ca1e632a634c5b79b4fb5234a708b3a2c6615db"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"86c75ef3dd337f994ce3b6e4d16951930f06b687","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Add new paramater \"CeilometerEnableTenantDiscovery\""},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Enabling this parameter will identify user and project"},{"line_number":10,"context_line":"names from the polled metrics [1]."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"These details are collected by making additional requests"},{"line_number":13,"context_line":"to keystone service, depending upon the scale of environment,"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"5b0835c5_82b77073","line":10,"range":{"start_line":10,"start_character":31,"end_line":10,"end_character":32},"in_reply_to":"393b5585_51c03278","updated":"2022-10-04 06:05:48.000000000","message":"well, it merged now anyway ;)","commit_id":"9ca1e632a634c5b79b4fb5234a708b3a2c6615db"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":34598,"name":"Manojkatari","email":"mkatari@redhat.com","username":"mkatari"},"change_message_id":"78d1ab60ebd41df476bd7ef3da6a5b3618e58697","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"2021c820_14754ea3","updated":"2022-10-03 05:30:49.000000000","message":"LGTM","commit_id":"7b57f966ccbb1a3909efe0ca97bd343f743df930"}],"deployment/ceilometer/ceilometer-agent-central-container-puppet.yaml":[{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"e27ac2fa59d260bd78af2af1eeb6d035e877dd81","unresolved":false,"context_lines":[{"line_number":109,"context_line":"          - get_attr: [CeilometerServiceBase, role_data, config_settings]"},{"line_number":110,"context_line":"          - ceilometer_redis_password: {get_param: RedisPassword}"},{"line_number":111,"context_line":"            tripleo::profile::base::ceilometer::agent::polling::central_namespace: true"},{"line_number":112,"context_line":"            ceilometer::agent::polling::tenant_name_discovery: {get_param: CeilometerEnableTenantDiscovery}"},{"line_number":113,"context_line":"      service_config_settings:"},{"line_number":114,"context_line":"        map_merge:"},{"line_number":115,"context_line":"          - get_attr: [CeilometerServiceBase, role_data, service_config_settings]"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"c37d23ac_59cb0b73","line":112,"range":{"start_line":112,"start_character":40,"end_line":112,"end_character":61},"updated":"2022-09-30 06:23:52.000000000","message":"looks right compared to what you\u0027re adding in the depends-on puppet-ceilometer","commit_id":"9ca1e632a634c5b79b4fb5234a708b3a2c6615db"},{"author":{"_account_id":34598,"name":"Manojkatari","email":"mkatari@redhat.com","username":"mkatari"},"change_message_id":"e076329d922cb9ca579090038ab815ed4575efec","unresolved":true,"context_lines":[{"line_number":57,"context_line":"    description: The password for the ceilometer service account."},{"line_number":58,"context_line":"    type: string"},{"line_number":59,"context_line":"    hidden: true"},{"line_number":60,"context_line":"  CeilometerEnableTenantDiscovery:"},{"line_number":61,"context_line":"    default: false"},{"line_number":62,"context_line":"    description: Identify user and project names from polled metrics."},{"line_number":63,"context_line":"    type: boolean"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"1f4b0d3f_5bf0026f","line":60,"range":{"start_line":60,"start_character":2,"end_line":60,"end_character":33},"updated":"2022-09-30 07:43:12.000000000","message":"If it is altogether a new parameter, i think even the tht patch requires a release note similar to puppet-ceilometer.","commit_id":"0a8d92aa5bd84e70c24123f9a5ef18454ad2d840"}]}
