Bug 1497983

Summary: Ansible playbook service max TTL is always divisible by 100
Product: Red Hat CloudForms Management Engine Reporter: Dmitry Misharov <dmisharo>
Component: AutomateAssignee: William Fitzgerald <wfitzger>
Status: CLOSED CURRENTRELEASE QA Contact: Dmitry Misharov <dmisharo>
Severity: medium Docs Contact:
Priority: medium    
Version: 5.8.0CC: gmccullo, jhardy, mkanoor, obarenbo, simaishi, tfitzger
Target Milestone: GAKeywords: TestOnly, ZStream
Target Release: 5.9.0   
Hardware: All   
OS: All   
Whiteboard: ansible:service:automate
Fixed In Version: 5.9.0.2 Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of:
: 1501524 (view as bug list) Environment:
Last Closed: 2018-03-06 14:45:45 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On: 1506796    
Bug Blocks: 1501524    

Description Dmitry Misharov 2017-10-03 09:37:29 UTC 
Description of problem:
There is no way to set max ttl not divisible by 100. E.g. 150 was entered, but in fact the service check completion will be finished after 100 minutes. So max ttl can be only 100, 200, 300 and so on.

Version-Release number of selected component (if applicable):
5.8.2.1.20170925150507_8770490

How reproducible:
Always

Steps to Reproduce:
1. Open Ansible Playbook catalog item creation dialog.
2. In playbook dropdown pick a playbook that execution time has 120 minutes. 
3. Enter 150 in max ttl field.
4. Order the service.

Actual results:
Service provision failed due timeout error.

Expected results:
The service should be provisioned successfully.
Comment 3 CFME Bot 2017-10-04 22:13:54 UTC 
https://github.com/ManageIQ/manageiq-content/pull/193
Comment 4 CFME Bot 2017-10-05 17:08:13 UTC 
New commit detected on ManageIQ/manageiq-content/master:
https://github.com/ManageIQ/manageiq-content/commit/db4c6efd02217c9ffd6b01b9bad14d65b4bf9258

commit db4c6efd02217c9ffd6b01b9bad14d65b4bf9258
Author:     william fitzgerald <wfitzger>
AuthorDate: Wed Oct 4 17:59:25 2017 -0400
Commit:     william fitzgerald <wfitzger>
CommitDate: Thu Oct 5 12:33:37 2017 -0400

    Fixed calculation for Ansible playbook service max TTL.
    
    Updated method and test.
    https://bugzilla.redhat.com/show_bug.cgi?id=1497983

 .../GenericLifecycle.class/__methods__/check_completed.rb    |  2 +-
 .../__methods__/check_completed_spec.rb                      | 12 ++++++------
 2 files changed, 7 insertions(+), 7 deletions(-)
Comment 6 Dmitry Misharov 2017-10-27 13:27:59 UTC 
Verified in 5.9.0.3.20171017182919_5f3a0ce. Max TTL works as expected.