Bug 1426227 - vm.create_snapshot for rhev vm creates a snapshot with extended name
Summary: vm.create_snapshot for rhev vm creates a snapshot with extended name
Keywords:
Status: CLOSED NOTABUG
Alias: None
Product: Red Hat CloudForms Management Engine
Classification: Red Hat
Component: Providers
Version: 5.7.0
Hardware: Unspecified
OS: Unspecified
unspecified
medium
Target Milestone: GA
: cfme-future
Assignee: Greg Blomquist
QA Contact: Ilanit Stein
URL:
Whiteboard: automate:snapshot:rhev
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2017-02-23 13:23 UTC by Ilanit Stein
Modified: 2017-03-13 14:25 UTC (History)
5 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2017-03-05 14:14:31 UTC
Category: ---
Cloudforms Team: RHEVM
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)

Description Ilanit Stein 2017-02-23 13:23:20 UTC
Description of problem:

when creating snapshot via Automate, the snapshot is created with name other than expected:
For VM with name "test_1", create a snapshot, with name "snp1",
Creates actually snapshot with name "Snapshot:<snp1> for test_1",
though it should be "snp1"

Version-Release number of selected component (if applicable):
CFME-5.7.1.2
RHV-4.0.7

Additional info:
The exact flow in automate, used to create the snapshot is described in bug 1399207 description.

Comment 2 Greg McCullough 2017-02-23 19:33:04 UTC
Looks like the provider create_snapshot method ignores the name passed to the method.

https://github.com/ManageIQ/manageiq/blob/51c1c07fa128b496f480e59f054647492ca38033/app/models/manageiq/providers/redhat/infra_manager/vm/operations/snapshot.rb#L24

Sending to providers team to take a look.

Comment 3 Ilanit Stein 2017-03-05 14:14:31 UTC
After investigating this issue with masayag, it turned out that for RHV provider,
Snapshot name parameter is not used.

Therefore, running create snapshot method with the following line, that includes snap_name, as done in this case:
  
snap_desc = $evm.root['snap_desc'] || "Snapshot:<#{snap_name}> for #{vm.name}"

is not relevant. 

considering the above, I am closing the bug.


Note You need to log in before you can comment on or make changes to this bug.