Project

General

Profile

Bug #14874

Path incorrect when using vm_clone method

Added by Francois Herbert almost 4 years ago. Updated over 1 year ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Compute resources - VMware
Target version:
Difficulty:
Triaged:
Bugzilla link:
Fixed in Releases:
Found in Releases:

Description

This is related to the fix for Bug #5630.
The fix for this (http://projects.theforeman.org/projects/foreman/repository/revisions/e7d21bf01f2b9eee19b808c4ea320ddfd9c33b3d) breaks our deployment.

I'm wondering if the path_replace is even required?

So our datacenter is: 'Turitea ITS'
Our original path is '/Datacenters/Turitea/Turitea ITS/vm/Production/Foundation/Infrastructure'

After the path_replace has been done the path is: '/DatacentersProduction/Foundation/Infrastructure' - which obviously doesn't work.

But looking at the original code before the fix for Bug #5630 was applied, after running the original path_replace our path wasn't changed and seemed to deploy via vmware as expected.


Related issues

Related to Foreman - Bug #5630: VmWare clone from template fails if Network Adator has labels in VmWareClosed2014-05-08

Associated revisions

Revision 01fd5003 (diff)
Added by Timo Goebel almost 4 years ago

fixes #14874 - do not run path replace during vmware clone

Revision 4e6eaab6 (diff)
Added by Timo Goebel almost 4 years ago

fixes #14874 - do not run path replace during vmware clone

(cherry picked from commit 01fd5003391d043dda0778ee0cf83abf3debb229)

History

#1 Updated by Francois Herbert almost 4 years ago

The method is clone_vm not vm_clone.

#2 Updated by Dominic Cleal almost 4 years ago

  • Related to Bug #5630: VmWare clone from template fails if Network Adator has labels in VmWare added

#3 Updated by Timo Goebel almost 4 years ago

  • Assignee set to Timo Goebel

I'll take a look shortly.

#4 Updated by Timo Goebel almost 4 years ago

Francois, have you tried removing path_replace completely? I believe cloning should work without this, however I can't test it right now unfortunately.

#5 Updated by Francois Herbert almost 4 years ago

Timo, I've just tested removing path_replace completely and the VM deploys from template as expected.

#6 Updated by The Foreman Bot almost 4 years ago

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

#7 Updated by Anonymous almost 4 years ago

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

#8 Updated by Dominic Cleal almost 4 years ago

  • Legacy Backlogs Release (now unused) set to 155

Also available in: Atom PDF