Skip to content
This repository has been archived by the owner on Jun 14, 2023. It is now read-only.

Feature/yaml parsing switch #26

Merged
merged 2 commits into from
Apr 14, 2016
Merged

Feature/yaml parsing switch #26

merged 2 commits into from
Apr 14, 2016

Conversation

ehealy
Copy link
Contributor

@ehealy ehealy commented Apr 13, 2016

@Tapjoy/eng-group-ops

This drives a wedge into the current top level tass commands that allows us to make calls to cloudformation once we've generated valid stack request json/hash (work to be completed in later tasks).

The following tests were succesfull in my local dev environemnt and can be performed using the example yaml files attached:

bundle exec tass scale -f ~/projects/.tass/config/clusters/v1_nonsense_cluster.yaml 
bundle exec tass audit -f ~/projects/.tass/config/clusters/v2_nonsense_cluster.yaml 
bundle exec tass audit -f ~/projects/.tass/config/clusters/v1_nonsense_cluster.yaml 
bundle exec tass create -f ~/projects/.tass/config/clusters/v1_nonsense_cluster.yaml 
bundle exec tass create -f ~/projects/.tass/config/clusters/v2_nonsense_cluster.yaml 
bundle exec tass update -f ~/projects/.tass/config/clusters/v2_nonsense_cluster.yaml 
bundle exec tass update -f ~/projects/.tass/config/clusters/v1_nonsense_cluster.yaml 

@atayarani
Copy link
Contributor

There's probably some cleanup to do down the road, but this is a good start

@ehealy ehealy force-pushed the feature/yaml_parsing_switch branch from 2a9b7d3 to fc42e66 Compare April 13, 2016 19:38
@ehealy
Copy link
Contributor Author

ehealy commented Apr 13, 2016

@atayarani specs are now inert and will not interact with AWS.

@atayarani
Copy link
Contributor

amazing thanks

@atayarani
Copy link
Contributor

I like how easy the fix makes it look. lgtm

@ehealy ehealy force-pushed the feature/yaml_parsing_switch branch from fff4314 to 64a1905 Compare April 14, 2016 13:26
@ehealy
Copy link
Contributor Author

ehealy commented Apr 14, 2016

@atayarani revised to make version implicit vs. explicit.

@atayarani
Copy link
Contributor

lgtm

@ehealy ehealy merged commit af6d913 into master Apr 14, 2016
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.

2 participants