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

Remove the local configuration script sources for embedded ansible #400

Merged

Conversation

carbonin
Copy link
Member

We stopped creating these in ManageIQ/manageiq#19056

@carbonin carbonin force-pushed the remove_default_configuration_script_sources branch 2 times, most recently from b156ca9 to 39c2ae7 Compare August 2, 2019 13:21
@carbonin carbonin force-pushed the remove_default_configuration_script_sources branch from 39c2ae7 to 3f42448 Compare August 2, 2019 13:26
@miq-bot
Copy link
Member

miq-bot commented Aug 2, 2019

Checked commit carbonin@3f42448 with ruby 2.4.6, rubocop 0.69.0, haml-lint 0.20.0, and yamllint 1.10.0
2 files checked, 0 offenses detected
Everything looks fine. 👍

@bdunne bdunne merged commit d425885 into ManageIQ:master Aug 2, 2019
@bdunne bdunne assigned bdunne and unassigned Fryguy Aug 2, 2019
@bdunne bdunne added this to the Sprint 117 Ending Aug 5, 2019 milestone Aug 2, 2019
simaishi pushed a commit that referenced this pull request Aug 2, 2019
…ript_sources

Remove the local configuration script sources for embedded ansible

(cherry picked from commit d425885)
@simaishi
Copy link
Contributor

simaishi commented Aug 2, 2019

Ivanchuk backport details:

$ git log -1
commit 12a10190a7f09ff51a58026ab64680340141f0d6
Author: Brandon Dunne <[email protected]>
Date:   Fri Aug 2 11:00:09 2019 -0400

    Merge pull request #400 from carbonin/remove_default_configuration_script_sources
    
    Remove the local configuration script sources for embedded ansible
    
    (cherry picked from commit d4258853decf94144a5f2b1bd23b7b8b08885f58)

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.

5 participants