[CCXDEV-12127] Fix payload-tracker support

[matysek]

Description

Add proper service_name for the payload tracker. Payload tracker is supported out-of-the-box by the insights-ccx-messaging library. This fixes the service name used for payload-tracker kafka messages.

Type of change

Please delete options that are not relevant.

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)

[juandspy]

why do we need to set it explicitly if that's not needed in ccx-data-pipeline? I've seen in sha-extractor we also set it 🤔

[matysek]

Because of this line in ccx-messaging - 'ccx-data-pipeline' is set as default name:
https://github.com/RedHatInsights/insights-ccx-messaging/blob/main/ccx_messaging/watchers/payload_tracker_watcher.py#L34

[juandspy]

Because of this line in ccx-messaging - 'ccx-data-pipeline' is set as default name: https://github.com/RedHatInsights/insights-ccx-messaging/blob/main/ccx_messaging/watchers/payload_tracker_watcher.py#L34

TY! I think it's not a good practice to have a default value there. In any case, LGTM!

[matysek]

in dvo-extractor we have this line:

{"filename": "payload_tracker_watcher.py", "lineno": 58, "process": 1, "levelname": "INFO", "asctime": "2024-01-26 12:18:39,942", "name": "ccx_messaging.watchers.payload_tracker_watcher", "message": "Sending status reports to Payload Tracker on topic platform.payload-status as service ccx-data-pipeline"}

[juandspy]

in dvo-extractor we have this line:

{"filename": "payload_tracker_watcher.py", "lineno": 58, "process": 1, "levelname": "INFO", "asctime": "2024-01-26 12:18:39,942", "name": "ccx_messaging.watchers.payload_tracker_watcher", "message": "Sending status reports to Payload Tracker on topic platform.payload-status as service ccx-data-pipeline"}

Ye... That's a problem IMO. We shouldn't use payload tracker if we don't have enough arguments f.e. missing service name. Let's discuss it some day.