Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use extra_vars to create a new dialog when editing Ansible playbook service template. #15120

Merged
merged 1 commit into from
May 19, 2017

Conversation

lfu
Copy link
Member

@lfu lfu commented May 17, 2017

Use the new values in extra_vars to create a new dialog, instead of waiting for the updated job template which may take some time.

https://bugzilla.redhat.com/show_bug.cgi?id=1450102

@miq-bot assign @gmcculloug
@miq-bot add_label bug, fine/yes, providers/ansible_tower, services

@gmcculloug
Copy link
Member

@bzwei Is it possible to create the dialog using the options stored in the service_template instead of having to incur more synchronous queue refreshes?

@bzwei
Copy link
Contributor

bzwei commented May 17, 2017

@lfu you can modify https://github.com/ManageIQ/manageiq/blob/master/app/models/dialog/ansible_playbook_service_dialog.rb#L9
to take extra_vars rather than a job_template as argument.

The extra_vars from service_template is no longer a plain hash. Actually it helps better the dialog tool to create appropriate fields based on the metadata.

@lfu lfu force-pushed the svc_dialog_extra_vars_1450102 branch from 7c027f9 to 8821393 Compare May 18, 2017 15:40
@lfu lfu changed the title Let ConfigurationScript.update_in_provider wait until provider refresh is done. Use extra_vars to create a new dialog when editing Ansible playbook service template. May 18, 2017
…ervice template.

Use the new values in extra_vars to create a new dialog, instead of waiting for the updated job template which may take some time.

https://bugzilla.redhat.com/show_bug.cgi?id=1450102
@lfu lfu force-pushed the svc_dialog_extra_vars_1450102 branch from 8821393 to 76bae5b Compare May 18, 2017 16:03
@miq-bot
Copy link
Member

miq-bot commented May 18, 2017

Checked commit lfu@76bae5b with ruby 2.2.6, rubocop 0.47.1, and haml-lint 0.20.0
3 files checked, 0 offenses detected
Everything looks fine. ⭐

@gmcculloug gmcculloug merged commit 0613afb into ManageIQ:master May 19, 2017
@gmcculloug gmcculloug added this to the Sprint 61 Ending May 22, 2017 milestone May 19, 2017
simaishi pushed a commit that referenced this pull request Jun 8, 2017
Use extra_vars to create a new dialog when editing Ansible playbook service template.
(cherry picked from commit 0613afb)

https://bugzilla.redhat.com/show_bug.cgi?id=1459989
@simaishi
Copy link
Contributor

simaishi commented Jun 8, 2017

Fine backport details:

$ git log -1
commit 9832165eb7cd0a3433423ac10aa41594bc89f765
Author: Greg McCullough <gmccullo@redhat.com>
Date:   Fri May 19 10:17:19 2017 -0400

    Merge pull request #15120 from lfu/svc_dialog_extra_vars_1450102
    
    Use extra_vars to create a new dialog when editing Ansible playbook service template.
    (cherry picked from commit 0613afbae8583e679d70aa932962235709aa96e1)
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1459989

@lfu lfu deleted the svc_dialog_extra_vars_1450102 branch October 16, 2017 20:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants