Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Update stop_event_monitor_queue_on_change to take changes #283

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Update stop_event_monitor_queue_on_change to take changes #283

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 4 additions & 4 deletions app/models/manageiq/providers/nuage/manager_mixin.rb
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -392,14 +392,14 @@ def verify_amqp_credentials(_options = {})
ManageIQ::Providers::Nuage::NetworkManager::EventCatcher::Stream.test_amqp_connection(event_monitor_options)
end

def stop_event_monitor_queue_on_change
if event_monitor_class && !new_record? && (authentications.detect(&:changed?) || endpoints.detect(&:changed?))
def stop_event_monitor_queue_on_change(_changes)
if event_monitor_class && !new_record?
$nuage_log.info("EMS: [#{name}], Credentials or endpoints have changed, stopping Event Monitor. It will be restarted by the WorkerMonitor.")
stop_event_monitor_queue
end
end

def stop_event_monitor_queue_on_credential_change
stop_event_monitor_queue_on_change
def stop_event_monitor_queue_on_credential_change(changes)
stop_event_monitor_queue_on_change(changes)
end
end