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

Ansible support on the Magician #197

Merged

Conversation

rambleraptor
Copy link
Contributor

@rambleraptor rambleraptor commented May 17, 2018

Ansible support on the Magician.

Change-Id: Ifaf460b447f520508589f9048695ae054b11e221


[all]

Ansible support on the Magician.

[terraform]

[puppet]

[puppet-dns]

[puppet-sql]

[puppet-compute]

[chef]

@modular-magician
Copy link
Collaborator

I am a robot that works on MagicModules PRs!
I checked the downstream repositories (see README.md for which ones I can write to), and none of them seem to have any changes.

Once this PR is approved, you can feel free to merge it without taking any further steps.

@modular-magician
Copy link
Collaborator

I am a robot that works on MagicModules PRs!
I checked the downstream repositories (see README.md for which ones I can write to), and none of them seem to have any changes.

Once this PR is approved, you can feel free to merge it without taking any further steps.

@modular-magician
Copy link
Collaborator

modular-magician commented May 17, 2018

I am a robot that works on MagicModules PRs!

I built this PR into one or more PRs on other repositories, and when those are closed, this PR will also be merged and closed.

@rambleraptor rambleraptor changed the title THIS IS A BAD CHANGE - DO NOT COMMIT Ansible support on the Magician May 17, 2018
@modular-magician
Copy link
Collaborator

I am (still) a robot that works on MagicModules PRs!

I just wanted to let you know that your changes (as of commit 7c10f86) have been included in your existing downstream PRs.

.ci/ci.yml.tmpl Outdated
params:
repository: magic-modules-with-comment/build/ansible
branch_file: magic-modules-with-comment/original_pr_branch_name
# Every time a change runs through this pipeline, it will generate a commit with
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can skip the comment the second time - easy for them to get out of sync. "See comment on terraform push" will do.


# This command can crash - if that happens, the script should not fail.
set +e
ANSIBLE_COMMIT_MSG="$(python .ci/magic-modules/extract_from_pr_description.py --tag ansible < .git/body)"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Remember to add the ansible tag to the .github/PULL_REQUEST_TEMPLATE.md.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A document for how to add a new provider is desperately needed. Right now is beyond witchcraft how to add a new product or provider. Any chance we can do that?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I have an issue to write that up, but it's not near the top of my list right now. Do you think it should be?

@nat-henderson nat-henderson self-requested a review May 18, 2018 21:32
@modular-magician
Copy link
Collaborator

I am (still) a robot that works on MagicModules PRs!

I just wanted to let you know that your changes (as of commit e1a5114) have been included in your existing downstream PRs.

Copy link
Contributor

@nat-henderson nat-henderson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, but remember to merge the Ansible change first, then use The Magician to merge this one.

.ci/ci.yml.tmpl Outdated
{% macro names_as_list(repo, names) -%}
{% for name in names %}
build/{{repo}}/{{name}}
{%- endfor %}
{% endmacro -%}
{% set puppet_submodules = names_as_list('puppet', puppet_modules).split() %}
{% set all_submodules = puppet_submodules + ['build/terraform'] if terraform_enabled else [] %}
{% set all_submodules = puppet_submodules + (['build/terraform'] if terraform_enabled else []) + (['build/ansible'] if ansible_enabled else []) %}
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could this fit in less than 500 chars? ;-)


# This command can crash - if that happens, the script should not fail.
set +e
ANSIBLE_COMMIT_MSG="$(python .ci/magic-modules/extract_from_pr_description.py --tag ansible < .git/body)"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A document for how to add a new provider is desperately needed. Right now is beyond witchcraft how to add a new product or provider. Any chance we can do that?

Change-Id: Iba3c1c6226d1f8def660956c37989cfc4cf1999c
* Removed stray comment
* Added ansible to Github PR Template

Change-Id: Id29a52ddd486b8b5f15020646d3209a2388f37f0
@modular-magician
Copy link
Collaborator

I am (still) a robot that works on MagicModules PRs!

I just wanted to let you know that your changes (as of commit 3ad0874) have been included in your existing downstream PRs.

@modular-magician
Copy link
Collaborator

I am (still) a robot that works on MagicModules PRs!

I just wanted to let you know that your changes (as of commit e0d98af) have been included in your existing downstream PRs.

* Made all_submodules < 80 chars
@modular-magician
Copy link
Collaborator

I am a robot that works on MagicModules PRs!

I built this PR into one or more PRs on other repositories, and when those are closed, this PR will also be merged and closed.
depends: modular-magician/ansible#7

@modular-magician modular-magician merged commit 30c649a into GoogleCloudPlatform:master May 29, 2018
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.

4 participants