)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":28356,"name":"Rafael Weingartner","email":"rafael@apache.org","username":"rafaelweingartner"},"change_message_id":"1be58aac49a6edfca76b69bf5184b4533cc50aa7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"33e25659_9916ae75","updated":"2022-02-10 17:07:56.000000000","message":"\u003e Patch Set 1:\n\u003e \n\u003e I do wonder why we have an identical copy of this file to the one in ceilometer in ceilometer/pipeline/data/event_definitions.yaml. Is there one in the container we can use?\n\n\nIf not used an override mechanism (https://github.com/openstack/ceilometer/blob/788479899b0ea291ac2a2284549b49906e506fea/ceilometer/event/converter.py#L281), Ceilometer will use its default file. The question is then, why overriding it by default in Kolla-ansible? We could use the default that Ceilometer provides, and then if needed, we could create a process to override it with some custom config set by the user via the \"definitions_cfg_file\" option: https://github.com/openstack/ceilometer/blob/788479899b0ea291ac2a2284549b49906e506fea/ceilometer/event/converter.py#L28","commit_id":"147399dbfff2407e985b470c2a3c5ea050c7c1b9"},{"author":{"_account_id":14200,"name":"Maksim Malchuk","email":"maksim.malchuk@gmail.com","username":"mmalchuk"},"change_message_id":"8569293bd4ba33052ccee6522c2e0b652192b0bf","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"a5af5dbf_d9883b49","updated":"2023-08-26 22:50:43.000000000","message":"conflicts with https://review.opendev.org/c/openstack/kolla-ansible/+/872721\nI think this one is useless if there always need to change this file.","commit_id":"147399dbfff2407e985b470c2a3c5ea050c7c1b9"},{"author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"change_message_id":"0d12fa187b2cd5dc7dcedc61bf7cb8df269f63a2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"66ca37cd_a402628d","in_reply_to":"33e25659_9916ae75","updated":"2022-02-11 09:06:32.000000000","message":"If we are not customising the file in kolla-ansible, I agree that using the one in ceilometer by default would be better.","commit_id":"147399dbfff2407e985b470c2a3c5ea050c7c1b9"},{"author":{"_account_id":28356,"name":"Rafael Weingartner","email":"rafael@apache.org","username":"rafaelweingartner"},"change_message_id":"a2900b665cd7a504735df07e68e666c9aadea330","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"9926e498_840f35a2","in_reply_to":"a5af5dbf_d9883b49","updated":"2023-08-27 20:14:36.000000000","message":"I agree. I think that we should abandon it.","commit_id":"147399dbfff2407e985b470c2a3c5ea050c7c1b9"}]}