Ensure hasrestart parameter with haproxy service
The haproxy service should have the hasrestart parameter defaulted to true. Previously, this was not the case. This commit establishes that.
Use array in config class default
Previously, I was using the option parameter twice in the default setting for haproxy::config. This commit changes that to passing an array to the option parameter (and fixes the example).
Commit spec tests for the haproxy module
Previously, the haproxy module didn't provide rspec tests for the class or defined resource types. This commit adds the tests, a spec_helper file, and a rakefile so you can do `rake spec` and automatically run the tests. I've also committed a symlink in the spec/fixtures/modules/haproxy directory so the rspec-puppet tests will FIND the haproxy class/defined resource types, but I HAVE NOT committed a symlink so that it will find the concat class (which is a dependency for this module). If you choose to run the tests, you must symlink the concat module in the spec/fixtures/modules directory before the spec tests will pass.
Add the puppet-concat module as a submodule for tests
Rename the README for fancy GitHub formatting
Add a Hacking section to the README
Merge pull request #8 from rtyler/chicken-dinner
Tidy up the place
Add puppet-lint to the Rakefile
Properly handle both Arrays and Strings passed as balancer member opt…
…ions The template cannot rely on deprecated functionality of the String class (`#each` which disappears in 1.9) Fixes #9
Fix some, but by no means all, of puppet-lint's complaining
Remove git submodule fixtures in place of puppetlabs_spec_helper fixt…
…ures behaviour
Merge pull request #1 from hunner/add_spec_helper
Add spec helper
This pull requests performs the minimum amount of effort to add
Debian support.
Adds Debian defaults (which were made to be as similar as possible
to the Redhat defaults)
Adds two Debian specific resources to the haproxy class
- a defaults file to ensure the service is started
- creates the /var/lib/haproxy directory so that the
configuration can be as similar to Redhat as possible.yUpdate specs for haproxy base class with debian support
Many of the tests that apply to the RedHat family also apply to Debian. The tests have been updated to reflect this, as well as to capture the minor differences between the two platforms. This commit incorperates the changes from glarizza#7 and glarizza#12 Not all tests pass because I found it pertinent to update the spec where the actual module code was lacking.
Changes: - Add debian support - Add puppetlabs_spec_helper tasks
Error message should contain the fact actually tested.
Merge pull request #7 from brhelwig/master
Data class error message reports operatingsystem instead of osfamily.
Merge pull request #4 from Mirantis/one-pass
Support for one-pass mode.
Merge pull request #8 from hunner/update_docs
Update hash formatting for readability
Allow multiple ports to be passed
haproxy supports a single port or port range, but also muliples of those. This commit changes the 'virtual_ip_port' parameter to 'ports' and updates the template to accept an array of ports
Merge pull request #9 from hunner/manage_service
Add manage_service parameter for corosync
Merge pull request #10 from hunner/add_ports
Allow multiple ports to be passed
Rename haproxy_config_options parameter to config_options
For great sanity
Merge pull request #11 from hunner/rename_config_options
Rename `haproxy_config_options` parameter to `config_options`
Merge pull request #12 from hunner/rename_listen_ip
Change `virtual_ip` to `listen_ip`
What is renamed:
- Class/define:
- haproxy::config to haproxy::listen
- haproxy::data to haproxy::params
- Parameters:
- haproxy::listen listen_ip to ipaddress
- haproxy::listen config_options to options
- haproxy haproxy_global_options to global_options
- haproxy haproxy_defaults_options to defaults_optionsMerge pull request #13 from hunner/rename_everything
Rename everything! Reviewed by: Cody
Merge pull request #14 from hunner/ports_string
Update ports to also accept a ,-separated string
Use selector for haproxy service
There are no spec tests to verify dependencies on a successful compilation, so I missed this
Merge pull request #15 from hunner/update_manage_service
Use selector for haproxy service
Update the haproxy::balancermember for multiple ports
Also, parameter changes: - `balancer_port` to `ports` - `server_name` to `server_names` - `balancer_ip` to `ipaddresses` - `balancermember_options` to `options`