Bug 1364894

Summary: GCE instance was retired, but was not power off
Product: Red Hat CloudForms Management Engine Reporter: Kyrylo Zvyagintsev <kzvyahin>
Component: AutomateAssignee: Tina Fitzgerald <tfitzger>
Status: CLOSED CURRENTRELEASE QA Contact: Dave Johnson <dajohnso>
Severity: high Docs Contact:
Priority: high    
Version: 5.6.0CC: abellott, cpelland, dajohnso, gblomqui, gmccullo, jfrey, jhardy, mkanoor, obarenbo, simaishi, tfitzger
Target Milestone: GAKeywords: TestOnly, ZStream
Target Release: 5.7.0Flags: kzvyahin: automate_bug+
Hardware: Unspecified   
OS: Unspecified   
Whiteboard: gce:retirement
Fixed In Version: 5.7.0.0 Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of:
: 1368170 (view as bug list) Environment:
Last Closed: 2017-01-11 19:54:39 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: Bug
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 1368170    

Comment 2 Adam Grare 2016-08-11 16:08:21 UTC
Vm.stop is never called during the retirement phase because generic pre_retirement state-machine method, moving to automate.

Comment 3 Adam Grare 2016-08-11 16:10:07 UTC
ETA: because there is no generic pre_retirement state-machine method

Comment 5 CFME Bot 2016-08-16 15:30:47 UTC
New commit detected on ManageIQ/manageiq/master:
https://github.com/ManageIQ/manageiq/commit/5a99030b06f3a1302b65ac53ff335aa83923ab63

commit 5a99030b06f3a1302b65ac53ff335aa83923ab63
Author:     Tina Fitzgerald <tfitzger>
AuthorDate: Thu Aug 11 14:42:14 2016 -0400
Commit:     Tina Fitzgerald <tfitzger>
CommitDate: Thu Aug 11 14:42:14 2016 -0400

    Automate - Cloud Retirement - Add google specific retirement states.
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1364894

 .../StateMachines/Methods.class/__class__.yaml     | 40 ++++++++++++++++++++++
 .../Methods.class/__methods__/pre_retirement.rb    | 10 ++++++
 .../Methods.class/__methods__/pre_retirement.yaml  | 12 +++++++
 .../Methods.class/checkpreretirement.yaml          |  2 ++
 .../StateMachines/Methods.class/preretirement.yaml |  2 ++
 5 files changed, 66 insertions(+)
 create mode 100644 db/fixtures/ae_datastore/ManageIQ/Cloud/VM/Retirement/StateMachines/Methods.class/__methods__/pre_retirement.rb
 create mode 100644 db/fixtures/ae_datastore/ManageIQ/Cloud/VM/Retirement/StateMachines/Methods.class/__methods__/pre_retirement.yaml

Comment 7 Satoe Imaishi 2016-08-29 12:29:37 UTC
Greg, the PR doesn't have darga label, but we have 5.6.z clone for this bug. Are we going to backport this to darga?

Comment 8 Greg McCullough 2016-08-29 13:11:11 UTC
Yes, it should be back-ported.  I added the darga/yes label.