Skip to content

Commit

Permalink
Fix event_catcher blacklisted events logging
Browse files Browse the repository at this point in the history
The event catcher runner was printing the blacklisted events array using
the VMDBLogger#log_hashes method.  Now that this method calls .to_hash
on the argument
(ManageIQ/manageiq-gems-pending#268)
this is throwing an exception for non-hash type arguments.

Instead of using log_hashes just log the elements of the array joined by
newlines directly.
  • Loading branch information
agrare committed Sep 7, 2017
1 parent 9291810 commit fb28b8f
Showing 1 changed file with 1 addition and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -18,8 +18,7 @@ def after_initialize
do_exit("EMS ID [#{@cfg[:ems_id]}] failed authentication check.", 1) unless @ems.authentication_check.first

@filtered_events = @ems.blacklisted_event_names
_log.info("#{log_prefix} Event Catcher skipping the following events:")
$log.log_hashes(@filtered_events)
_log.info("#{log_prefix} Event Catcher skipping the following events:\n#{@filtered_events.to_a.join("\n")}")

configure_event_flooding_prevention if worker_settings.try(:[], :flooding_monitor_enabled)

Expand Down

0 comments on commit fb28b8f

Please sign in to comment.