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

Added RDS provider. #290

Merged
merged 4 commits into from
Jun 3, 2013
Merged

Added RDS provider. #290

merged 4 commits into from
Jun 3, 2013

Conversation

brandonbell
Copy link
Contributor

Example Cluster Definition :

Ironfan.cluster 'rds' do
  environment           :_default
  cloud(:rds).security_group(:tests).authorize_ip_range "127.0.0.1/32"
  cloud(:rds).security_group(:tests2).authorize_ec2_group "123456789/systemwide"

  facet :mysql do
    cloud(:rds) do
      password              "abc123"
      username              "root"
      flavor                "db.m1.small"
      size                  5
      port                  3601
    end
  end
end

@temujin9
Copy link
Contributor

Huh. I want to be uncomfortable about this, but I'm actually finding little to be uncomfortable about. Definitely needs some further inspection, but I think I like it.

Can we add silverware-style announcements of the underlying DB, saved to the node? That's about the only integration point that I could see missing on a quick and dirty pass through the code. And having those would make cleaning up the various RDS integrations much easier.

@brandonbell
Copy link
Contributor Author

Announcements are the end goal. Unfortunately, chef-client never runs on these servers so it will have to be done at launch.

@temujin9
Copy link
Contributor

temujin9 commented Jun 3, 2013

Your example should say use_cloud :rds inside that facet; if there's more than one cloud defined (and cloud :ec2 is usually in the header of the cluster file), Ironfan has to guess at which to use, and guesses wrong without that. Just FYI for docco purposes.

@temujin9 temujin9 merged commit f369fb0 into master Jun 3, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants