Bug 1152659

Summary: [PPC][UX] balloon is sent for PPC VMs when creating VM from snapshot
Product: Red Hat Enterprise Virtualization Manager Reporter: Ori Gofen <ogofen>
Component: ovirt-engineAssignee: Vitor de Lima <vdelima>
Status: CLOSED CURRENTRELEASE QA Contact: Lukas Svaty <lsvaty>
Severity: urgent Docs Contact:
Priority: unspecified    
Version: 3.4.1-1CC: acanan, dfediuck, ecohen, fnovak, gklein, hannsj_uhl, iheim, lpeer, lsurette, mavital, michal.skrivanek, ofrenkel, ogofen, rbalakri, Rhev-m-bugs, sherold, yeylon
Target Milestone: ---Keywords: ZStream
Target Release: 3.5.0   
Hardware: ppc64   
OS: Linux   
Whiteboard: sla
Fixed In Version: org.ovirt.engine-root-3.5.0-19 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
: 1156012 (view as bug list) Environment:
Last Closed: 2015-02-17 17:16:52 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: SLA RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 1122979, 1152664, 1156012    

Description Ori Gofen 2014-10-14 15:52:48 UTC
Description of problem:

operation is Blocked with CDA from UI due to libvirt default decision to add a balloon device upon each VM creation

see BZ #1135939 BZ #1152560

Comment 2 Jiri Moskovcak 2014-10-22 08:50:31 UTC
Please provide specific steps to reproduce

Comment 3 Ori Gofen 2014-10-22 13:03:41 UTC
Have been discussed over irc, the flow is as simple as it sounds.

1.create ppc64 vm,install OS
2.take a snapshot
3.create vm from snapshot

Comment 4 Scott Herold 2014-10-22 18:52:42 UTC
Assuming this actually becomes a non-issue once BZ 1152560 is resolved.  Linking blocking BZ, and leaving this BZ in place for validation after core fix is implemented.

Comment 5 Scott Herold 2014-10-22 18:54:00 UTC
Ignore previous... Getting my Balloon BZs all mixed up...

Comment 6 Jiri Moskovcak 2014-10-23 08:18:26 UTC
I believe we need the same fix in AddVmCommand (which is used when creating vm from snapshot) as Vitor did in ImportVmCommand to fix #1152560. Going to test and will update the bz shortly.

Comment 8 Lukas Svaty 2014-11-11 12:08:14 UTC
verified in vt9

Comment 9 Frank Novak 2015-01-19 21:56:58 UTC
(In reply to Jiri Moskovcak from comment #6)
> I believe we need the same fix in AddVmCommand (which is used when creating
> vm from snapshot) as Vitor did in ImportVmCommand to fix #1152560. Going to
> test and will update the bz shortly.

Did this get added as well.. i.e. handling of the snapshot?

Comment 10 Frank Novak 2015-01-19 21:57:41 UTC
(In reply to Frank Novak from comment #9)
> (In reply to Jiri Moskovcak from comment #6)
> > I believe we need the same fix in AddVmCommand (which is used when creating
> > vm from snapshot) as Vitor did in ImportVmCommand to fix #1152560. Going to
> > test and will update the bz shortly.
> 
> Did this get added as well.. i.e. handling of the snapshot?

that was when creating VM from snapshot...

Comment 11 Eyal Edri 2015-02-17 17:16:52 UTC
rhev 3.5.0 was released. closing.