Skip to content

chore: ensure correlation id is logged in notifier

Jaime Martinez requested to merge log-correlation-id-iomport-notif into master

Looking at the logs I realized we don't print the correlation ID while sending an import notification. We also print path instead of repository. For status, Kibana expects it to be a number and not a string, so it fails to print the correct value.

Screen_Shot_2022-05-17_at_4.46.29_pm

After the change the logs look like this:

INFO[0149] sending import notification                   correlation_id=01G3A7JAA534C4MSC35Z1YK6EY go_version=go1.17.8 migration_detail="pre import completed successfully" migration_status=pre_import_complete name=alpine repository=alpine version=v3.41.1-gitlab-3-g7d639f90
INFO[0149] sent import notification successfully         correlation_id=01G3A7JAA534C4MSC35Z1YK6EY go_version=go1.17.8 migration_detail="pre import completed successfully" migration_status=pre_import_complete name=alpine repository=alpine version=v3.41.1-gitlab-3-g7d639f90
Edited by Jaime Martinez

Merge request reports