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

Support additional env args #269

Merged
merged 1 commit into from Jun 16, 2015

Conversation

Projects
None yet
3 participants
@krtyyy
Copy link
Contributor

commented May 15, 2015

If attribute node['node']['additional_env_settings'] exists, it is
expected to be an array of strings, which will then be appended
to rabbit-env.conf. This allows users the flexibility to specify
ENV settings that don't have explicit attributes. (E.g. Some
settings may be new to rabbitmq but won't require a recipe update
before people can take advantage of them.)

I also cleaned up a couple of rubocop issues in default_spec.rb

Support additional env args
If attribute node['node']['additional_env_settings'] exists, it is
expected to be an array of strings, which will then be appended
to rabbit-env.conf. This allows users the flexibility to specify
ENV settings that don't have explicit attributes. (E.g. Some
settings may be new to rabbitmq but won't require a recipe update
before people can take advantage of them.)

I also cleaned up a couple of rubocop issues in default_spec.rb
@kramvan1

This comment has been minimized.

Copy link
Contributor

commented May 27, 2015

👍

jjasghar pushed a commit that referenced this pull request Jun 16, 2015

JJ Asghar
Merge pull request #269 from krtyyy/additional_env_args
Support additional env args

@jjasghar jjasghar merged commit e6259c8 into rabbitmq:master Jun 16, 2015

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.