Add ELB to AWS cookbook

Details

  • Type: New Feature New Feature
  • Status: Closed Closed
  • Priority: Minor Minor
  • Resolution: Fixed
  • Component/s: aws
  • Labels:

Description

I've given this no technical thought, but it would be interesting to have LWRP for ELB, and perhaps a recipe that could be used as a drop in replacement for haproxy::app_lb.

You could pick one of your other nodes to run that recipe and configure an ELB based on it's environment.

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Justin Huff added a comment -

I'm working on this now. My plan is to add a LWRP to allow a node to register itself with the LB. This is similar to how the elastic_ip lwrp works now.

Show
Justin Huff added a comment - I'm working on this now. My plan is to add a LWRP to allow a node to register itself with the LB. This is similar to how the elastic_ip lwrp works now.
Hide
Permalink
Justin Huff added a comment -


Done in https://github.com/jjhuff/cookbooks/tree/CHEF-627

Show
Justin Huff added a comment - Done in https://github.com/jjhuff/cookbooks/tree/CHEF-627
Hide
Permalink
jtimberman added a comment -

This is awesome. Marking triaged so it it gets picked up for merging.

Show
jtimberman added a comment - This is awesome. Marking triaged so it it gets picked up for merging.
Hide
Permalink
jtimberman added a comment -

This patch has been merged to opscode/master and will be included in the next release of the cookbook. Thank you for contributing to our Cookbooks!

Show
jtimberman added a comment - This patch has been merged to opscode/master and will be included in the next release of the cookbook. Thank you for contributing to our Cookbooks!
Hide
Permalink
Iuri Aranda added a comment -

There is a missing 'region' attribute in the ElbInterface constructor. Without it the default region is used, so if you are working with a different region you are screwed
I've already sent a pull request on GitHub. https://github.com/opscode/cookbooks/pull/298

Show
Iuri Aranda added a comment - There is a missing 'region' attribute in the ElbInterface constructor. Without it the default region is used, so if you are working with a different region you are screwed I've already sent a pull request on GitHub. https://github.com/opscode/cookbooks/pull/298
Hide
Permalink
Paul Morton added a comment -

Looks like there is another pull request that covers this same issue region issue.
https://github.com/opscode/cookbooks/pull/299

Show
Paul Morton added a comment - Looks like there is another pull request that covers this same issue region issue. https://github.com/opscode/cookbooks/pull/299
Hide
Permalink
jtimberman added a comment -

I'd love to get this released, and prefer to have the region fix in before we do so. However, we do not have a CLA on file for either Iuri Aranda, nor Jose Fernandez. Just a reminder that we do require a CLA/CCLA on file for all contributions.

Show
jtimberman added a comment - I'd love to get this released, and prefer to have the region fix in before we do so. However, we do not have a CLA on file for either Iuri Aranda , nor Jose Fernandez. Just a reminder that we do require a CLA/CCLA on file for all contributions. http://wiki.opscode.com/display/chef/How+to+Contribute
Hide
Permalink
jtimberman added a comment -

Neither of the authors of the pull request followed up to requests for a CLA, but Justin Huff submitted a pull request with the region fix that is now merged. This will be in the next release of the aws cookbook.

https://github.com/opscode-cookbooks/aws/pull/3

Thanks!

Show
jtimberman added a comment - Neither of the authors of the pull request followed up to requests for a CLA, but Justin Huff submitted a pull request with the region fix that is now merged. This will be in the next release of the aws cookbook. https://github.com/opscode-cookbooks/aws/pull/3 Thanks!
Hide
Permalink
jtimberman added a comment -

This is released in v0.100.0 of the aws cookbook. Thanks!

Show
jtimberman added a comment - This is released in v0.100.0 of the aws cookbook. Thanks!

People

  • Assignee:
    Justin Huff
    Reporter:
    Glenn Pratt
Vote (0)
Watch (3)

Dates

  • Created:
    Updated:
    Resolved: