search

ManageIQ / manageiq-providers-vmware

ManageIQ plugin for the VMware vSphere and vCloud providers.
Apache License 2.0
22 stars 70 forks source link

Fix missing log service name for non-rails event catcher #905

Closed agrare closed 5 months ago

agrare commented 5 months ago

The non-rails event catcher was not setting the progname for the ManageIQ::Loggers instance which defaulted to "manageiq".

This meant that the event catcher logging would be missed if you were e.g. journalctl -f -t evm

miq-bot commented 5 months ago

Checked commit https://github.com/agrare/manageiq-providers-vmware/commit/a402622dd6cd4c5f554f0bfc5be9b968bfd3788d with ruby 2.7.8, rubocop 1.56.3, haml-lint 0.51.0, and yamllint 1 file checked, 0 offenses detected Everything looks fine. :+1:

Fryguy commented 5 months ago

Backported to radjabov in commit 32c1ba6b74d8a9113925f9990b988aa584955f5e.

commit 32c1ba6b74d8a9113925f9990b988aa584955f5e
Author: Jason Frey <fryguy9@gmail.com>
Date:   Mon Mar 11 16:56:30 2024 -0400

    Merge pull request #905 from agrare/fix_non_rails_event_catcher_service_name

    Fix missing log service name for non-rails event catcher

    (cherry picked from commit 51a355f131054b7d1a301a602eb91acd439f6497)