Skip to content
This repository has been archived by the owner on Jan 19, 2022. It is now read-only.

Sane default for template locations. #69

Closed
wants to merge 1 commit into from
Closed

Conversation

koikonom
Copy link
Contributor

Since the plan is to use bootstrap-cfn with template-deploy it makes
sense to set the default location to be the location where
template-deploy stores the yaml files.

Since the plan is to use bootstrap-cfn with template-deploy it makes
sense to set the default location to be the location where
template-deploy stores the yaml files.
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.09%) to 53.17% when pulling 8c0328a on sane_defaults into 78707b7 on master.

env.config = None
env.password = None
env.config = os.path.join(path, 'cloudformation', 'template.yaml')
env.password = os.path.join(path, 'cloudformation', 'template-secrets.yaml')
Copy link
Contributor

Choose a reason for hiding this comment

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

These belong in the fabfile.py in template deploy, not in here.

@ashb
Copy link
Contributor

ashb commented Apr 13, 2015

Closing this one - we'll reopen it against template-deploy soon instead.

@ashb ashb closed this Apr 13, 2015
@ashb ashb removed the in progress label Apr 13, 2015
@ashb ashb deleted the sane_defaults branch May 21, 2015 16:31
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants