Bug 1479410 - incorrect value used in stock automation wait_for_completion
Summary: incorrect value used in stock automation wait_for_completion
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Red Hat CloudForms Management Engine
Classification: Red Hat
Component: Automate
Version: 5.7.0
Hardware: All
OS: All
unspecified
medium
Target Milestone: GA
: 5.7.4
Assignee: William Fitzgerald
QA Contact: Milan Falešník
URL:
Whiteboard:
Depends On: 1474748
Blocks:
TreeView+ depends on / blocked
 
Reported: 2017-08-08 14:16 UTC by Satoe Imaishi
Modified: 2022-07-09 08:57 UTC (History)
10 users (show)

Fixed In Version: 5.7.4.0
Doc Type: If docs needed, set a value
Doc Text:
Clone Of: 1474748
Environment:
Last Closed: 2017-12-18 20:28:28 UTC
Category: ---
Cloudforms Team: ---
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Red Hat Product Errata RHSA-2017:3484 0 normal SHIPPED_LIVE Important: Red Hat CloudForms security, bug fix, and enhancement update 2017-12-19 01:24:16 UTC

Comment 3 CFME Bot 2017-08-15 21:50:56 UTC
New commit detected on ManageIQ/manageiq/euwe:
https://github.com/ManageIQ/manageiq/commit/78619b5e97e9c294e6506d078fb4dcda9dc5c323

commit 78619b5e97e9c294e6506d078fb4dcda9dc5c323
Author:     Greg McCullough <gmccullo>
AuthorDate: Tue Jul 25 17:23:48 2017 -0400
Commit:     Satoe Imaishi <simaishi>
CommitDate: Tue Aug 15 17:48:54 2017 -0400

    Merge pull request #151 from billfitzgerald0120/fix_ae_retry_interval
    
    Changed ae_retry_limit = 1.minute to ae_retry_interval = 1.minute.
    (cherry picked from commit 0ee9bd467697aed5cc92f85f1210b4ceed73a87a)
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1479410

 .../StateMachines/Job.class/__methods__/wait_for_completion.rb          | 2 +-
 .../Operations/StateMachines/Job.class/__methods__/wait_for_ip.rb       | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

Comment 4 William Fitzgerald 2017-08-16 15:56:15 UTC
Taeho,

This fix is a one line change in 2 methods. 

1. Create a writeable custom domain (or use an existing custom domain).
2. Use the explorer and Navigate to ManageIQ/ConfigurationManagement/AnsibleTower/Operations/StateMachines/Job class.
3. Select the wait_for_completion method.
4. Copy the method to your custom domain.
5. Use the explorer and Navigate to ManageIQ/ConfigurationManagement/AnsibleTower/Operations/StateMachines/Job class.
6. Select the wait_for_ip method.
7. Copy the method to your custom domain.
8. Use the explorer and Navigate in your custom domain to /ConfigurationManagement/AnsibleTower/Operations/StateMachines/Job class.
9. Select the wait_for_completion method.
10. Edit the method.
11. Change  @handle.root['ae_retry_limit'] = 1.minute to 
            @handle.root['ae_retry_interval'] = 1.minute
12. Save the method.
13. Select the wait_for_ip method.
14. Edit the method.
15. Change  @handle.root['ae_retry_limit'] = 1.minute to 
            @handle.root['ae_retry_interval'] = 1.minute
16. Save the method.


Please let me know if you have any questions.

Billy

Comment 7 William Fitzgerald 2017-08-31 14:29:49 UTC
Taeho,

1. No, The unmodified method will work sometimes.  It will still retry 100 times so your playbook could finish.  The problem is the method won't wait 60 seconds between retries.  Based on how long your playbook runs and how quickly the system retries will determine whether it will work or not.  I run very simple playbooks and never encountered this problem.  

2. I don't understand your question, could you provide more information on this question ?

Thanks

Billy

Comment 9 William Fitzgerald 2017-09-05 19:30:27 UTC
Taeho,

Can you recreate the Service problem and let me have access to your appliance ?

Thanks

Billy

Comment 11 William Fitzgerald 2017-09-08 13:01:47 UTC
Taeho,

It looks like your custom method is getting an error so that is why the process is not retrying.

Thanks

Billy

Comment 13 Milan Falešník 2017-10-03 12:08:22 UTC
Checked the methods in 5.7.4.0 and the fixed root value ae_retry_interval is correctly used and ae_retry_limit is not present.

Comment 16 errata-xmlrpc 2017-12-18 20:28:28 UTC
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

https://access.redhat.com/errata/RHSA-2017:3484


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