Project

General

Profile

Refactor #32258

Change default implementation of to_status to noop

Added by Shimon Shtein 5 months ago. Updated 5 months ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Plugin integration
Target version:
-
Difficulty:
Triaged:
No
Bugzilla link:

Description

Currently to_status method (https://github.com/theforeman/foreman/blob/f5bfce8bc5cab43fa66c362d40ee843e0783fe72/app/models/host_status/status.rb#L29) of status subclasses throws an exception if not implemented.

There are cases that need external event to happen in order to determine the correct status for a host. For such cases to_status method is unnecessary, and probably won't be implemented. We need to change the default implementation to noop

Associated revisions

Revision 13b3ce22 (diff)
Added by Shimon Shtein 5 months ago

Fixes #32258 - Change to_status default to noop

Revision 3f7804fd (diff)
Added by Shimon Shtein 5 months ago

Refs #32258 - Added documentation about status class

History

#1 Updated by The Foreman Bot 5 months ago

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

#2 Updated by The Foreman Bot 5 months ago

  • Fixed in Releases 2.5.0 added

#3 Updated by Shimon Shtein 5 months ago

  • Status changed from Ready For Testing to Closed

#4 Updated by The Foreman Bot 5 months ago

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

Also available in: Atom PDF