Skip to content
This repository has been archived by the owner on Jul 14, 2021. It is now read-only.

Allow opscode_pushy_server vip to be overridden #172

Merged
merged 1 commit into from Aug 31, 2017

Conversation

jobsolete
Copy link
Contributor

@jobsolete jobsolete commented Aug 30, 2017

Presently, a user cannot leverage push jobs with a highly available chef server
configuration. By allowing a user to set the opscode_pushy_server['vip'] this
pull request allows users to override the default values assigned by Chef, allowing
push jobs to be leveraged with highly available chef server configurations. As
opscode_pushy_server['vip'] defaults to nil, this should only affect users
that have been instructed to make these changes explicitly.

Signed-off-by: Josh O'Brien jobrien@chef.io

Presently, a user cannot leverage push jobs with a highly available chef server
configuration. By allowing a user to set the `opscode_pushy_server['vip']` this
pull request allows users to override the default values assigned by Chef, allowing
push jobs to be leveraged with highly available chef server configurations. As
`opscode_pushy_server['vip'] defaults to `nil`, this should only affect users
that have been instructed to make these changes explicitly.

Signed-off-by: Josh O'Brien <jobrien@chef.io>
Copy link
Contributor

@markan markan left a comment

Choose a reason for hiding this comment

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

This looks very reasonable and safe. +1

@irvingpop irvingpop merged commit 4146a6f into master Aug 31, 2017
@irvingpop irvingpop deleted the OBrienCommaJosh/aws_ha_fix branch August 31, 2017 17:31
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants