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

First implementation of Heat Stacks #170

Merged
merged 5 commits into from
Nov 20, 2014
Merged

First implementation of Heat Stacks #170

merged 5 commits into from
Nov 20, 2014

Conversation

julienvey
Copy link
Collaborator

This PR implements a simple workflow for heat stacks.

vagrant up => create stacks
vagrant destroy => delete stacks

Current format of a stack is

os.stacks = [{
   name: 'stack_name',
   template: '/path/to/my/template.yml'
}]

but other attributes should be added later as described in #153

Once this PR is merged, I will create sub-issues for all heat related work, so that we can split the work and have people contribute.

@julienvey julienvey changed the title First simple implementation of Heat Stacks First implementation of Heat Stacks Nov 13, 2014

heat = env[:openstack_client].heat

config.stacks.each do |stack|
Copy link
Owner

Choose a reason for hiding this comment

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

undefined method `each' for nil:NilClass (NoMethodError)

When there's no stack declared in the Vagrantfile. Should be fixed by

config.stacks.each do |stack|
  ...
end unless config.stack.nil?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Done

@ggiamarchi ggiamarchi modified the milestone: 0.6 Nov 16, 2014
@ggiamarchi ggiamarchi added this to the 0.6 milestone Nov 20, 2014
ggiamarchi added a commit that referenced this pull request Nov 20, 2014
First implementation of Heat Stacks
@ggiamarchi ggiamarchi merged commit 3fb23c1 into master Nov 20, 2014
@ggiamarchi ggiamarchi deleted the heat branch November 21, 2014 22:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants