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

(MODULES-1508) add support for unix_socket_directories #845

Merged
merged 3 commits into from
Feb 21, 2017

Conversation

eputnam
Copy link
Contributor

@eputnam eputnam commented Feb 13, 2017

unix_socket_directories, new in 9.3, requires a server restart for changes. Only recognized config attributes trigger a restart, so this adds it to the list to be recognized.

This adds a unit test to make sure that unix_socket_directories notifies a service restart

This is an acceptance test to test config entries in general, specifically unix_socket_directories. The manifest should run without failures and the provided list of socket dirs should be present in the config file.

unix_socket_directories, new in 9.3, requires a server restart for changes. Only recognized config attributes trigger a restart, so this adds it to the list to be recognized.
This adds a unit test to make sure that unix_socket_directories notifies a service restart
This is an acceptance test to test config entries in general, specifically unix_socket_directories. The manifest should run without failures and the provided list of socket dirs should be present in the config file.
@eputnam eputnam changed the title (MODULES-1508) WIP - add support for unix_socket_directories (MODULES-1508) add support for unix_socket_directories Feb 14, 2017
@hunner hunner merged commit ea8f093 into puppetlabs:master Feb 21, 2017
cegeka-jenkins pushed a commit to cegeka/puppet-postgresql that referenced this pull request Feb 3, 2022
(MODULES-1508) add support for unix_socket_directories
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants