Project

General

Profile

Refactor #19340

new_host vs new_vm variable in compute resource VM partials

Added by Bernhard Suttner about 1 year ago. Updated 10 days ago.

Status:
Closed
Priority:
High
Assignee:
Category:
Compute resources
Target version:
Difficulty:
Triaged:
Bugzilla link:
Team Backlog:
Fixed in Releases:
Found in Releases:

Description

new_host was replaced by new_vm because of:
https://github.com/theforeman/foreman/commit/637da2f28accc1378f0326d97adcbb2efc5d9808

There are still 2 places (vmware name, vmware volume disk_mode) in which new_host is used in:
app/views/compute_resources_vms/form/vmware/_volume.html.erb

Associated revisions

Revision c8efc502 (diff)
Added by Timo Goebel about 1 year ago

fixes #19340 - replace all new_host calls with new_vm

History

#1 Updated by Dominic Cleal about 1 year ago

  • Tracker changed from Bug to Refactor
  • Subject changed from new_host vs new_vm to new_host vs new_vm variable in compute resource VM partials
  • Category set to Compute resources

#2 Updated by Stefan Heil about 1 year ago

  • Related to Bug #19566: Creating a new host VM on xen fails with: undefined method `new_host?' added

#3 Updated by Luke Huxley about 1 year ago

  • Priority changed from Normal to High

Bernhard Suttner wrote:

new_host was replaced by new_vm because of:
https://github.com/theforeman/foreman/commit/637da2f28accc1378f0326d97adcbb2efc5d9808

There are still 2 places (vmware name, vmware volume disk_mode) in which new_host is used in:
app/views/compute_resources_vms/form/vmware/_volume.html.erb

This change seems to have broken provision on XenServer compute resources.

#4 Updated by Timo Goebel about 1 year ago

  • Related to deleted (Bug #19566: Creating a new host VM on xen fails with: undefined method `new_host?' )

#5 Updated by The Foreman Bot about 1 year ago

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

#6 Updated by Timo Goebel about 1 year ago

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

#7 Updated by Marek Hulán about 1 year ago

  • Legacy Backlogs Release (now unused) set to 240

Also available in: Atom PDF