Project

General

Profile

Actions

Refactor #9546

closed

Discovery should provide a convinient method to convert discovered host into managed host (aka provision)

Added by Marek Hulán about 9 years ago. Updated about 9 years ago.

Status:
Closed
Priority:
Urgent
Category:
Discovery plugin
Target version:
Difficulty:
Triaged:
Fixed in Releases:
Found in Releases:

Description

There's no reusable way to transform discovered host into managed host provided by Discovery, which leads to custom code copied from discovery controller like this Unfortunately this leads to problems when the "official" provision code changes, like someone adds deletion of discovered host facts. I know that there's already perform_auto_provision which is shared among controllers using concern, but it does not belong to controller. I see it as a Host::Discovered responsibility or if not directly this class, then some new HostConverter, not controller.


Related issues 1 (0 open1 closed)

Related to Discovery - Bug #9488: Unable delete the discovered hosts from foremanClosedOri Rabin02/20/2015Actions
Actions #1

Updated by The Foreman Bot about 9 years ago

  • Status changed from New to Ready For Testing
  • Pull request https://github.com/theforeman/foreman_discovery/pull/161 added
  • Pull request deleted ()
Actions #2

Updated by Lukas Zapletal about 9 years ago

  • Assignee set to Lukas Zapletal
  • Target version set to Discovery Plugin 3.0.0

Agreed, pushed first version. Feel free to review.

Actions #3

Updated by Ori Rabin about 9 years ago

  • Related to Bug #9488: Unable delete the discovered hosts from foreman added
Actions #4

Updated by Lukas Zapletal about 9 years ago

  • Priority changed from Normal to Urgent
  • Target version changed from Discovery Plugin 3.0.0 to Discovery Plugin 2.0.1
Actions #5

Updated by Anonymous about 9 years ago

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

Also available in: Atom PDF