Project

General

Profile

Bug #18952

Notifications issues prevent host to be destroyed

Added by Ivan Necas over 4 years ago. Updated about 3 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Notifications
Target version:
Difficulty:
Triaged:
Bugzilla link:

Description

When a host doesn't have an owner set (it happened to me, when I created fake host via console,
the failure to send notification prevented the host to be destroyed and no useful message was provided:
everything seemed fine, except the host was still there.

I believe the failures in notifications should not influence the functionality of the operation itself.

2017-03-17T17:41:04 e59c76c5  [app] [D] Setting current user thread-local variable to admin
2017-03-17T17:41:04 e59c76c5 [user:admin] [app] [D] Setting current organization thread-local variable to none
2017-03-17T17:41:04 e59c76c5 [user:admin] [app] [D] Setting current location thread-local variable to none
2017-03-17T17:41:04 e59c76c5 [user:admin] [notifications] [W] Invalid owner for aaron-brangers, unable to send notifications
2017-03-17T17:41:04 e59c76c5 [user:admin] [notifications] [D] Notification event: Hosts::MissingOwner - checking for notifications
2017-03-17T17:41:04 e59c76c5 [user:admin] [app] [I] Failed to save:
2017-03-17T17:41:04 e59c76c5 [user:admin] [app] [I] Redirected to http://localhost:3000/hosts

Related issues

Related to Foreman - Bug #19171: Notification about missing owner of a host is present after host which has been destroyedNew

Associated revisions

Revision 54db95ae (diff)
Added by Ivan Necas over 4 years ago

Fixes #18952 - fix destroy of host without owner

The notifications failure caused the host not being deleted.
I don't think the notifications should interfere with the core
functionality of the objects.

History

#1 Updated by The Foreman Bot over 4 years ago

  • Status changed from New to Ready For Testing
  • Pull request https://github.com/theforeman/foreman/pull/4387 added

#2 Updated by Ivan Necas over 4 years ago

  • Status changed from Ready For Testing to Closed
  • % Done changed from 0 to 100

#3 Updated by The Foreman Bot over 4 years ago

  • Pull request https://github.com/theforeman/foreman/pull/4302 added

#4 Updated by Ohad Levy over 4 years ago

  • Legacy Backlogs Release (now unused) set to 209

#5 Updated by Marek Hulán over 4 years ago

  • Related to Bug #19171: Notification about missing owner of a host is present after host which has been destroyed added

Also available in: Atom PDF