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

(GH-786) sensu::plugin does not work on windows without specifying install_path #789

Merged
merged 1 commit into from
Aug 3, 2017

Conversation

Phil-Friderici
Copy link
Collaborator

Working on issue #786.

Add Linux/Windows sensitive default values for sensu::plugin::install_path.
Adjust calls of sensu::plugin to not set install_path anymore.

…ng install_path

Add Linux/Windows sensitive default values for sensu::plugin::install_path.
Adjust calls of sensu::plugin to not set install_path anymore.
@Phil-Friderici
Copy link
Collaborator Author

Will continue to work on tests/sensu-client-windows.pp once I understood how to use and modify the Vagrant test suite.

@ghoneycutt
Copy link
Collaborator

ghoneycutt commented Aug 3, 2017

@Phil-Friderici if you put Resolves #123 then when this ticket closes it will automatically close the referenced ticket.

https://help.github.com/articles/closing-issues-using-keywords/

@ghoneycutt
Copy link
Collaborator

ghoneycutt commented Aug 3, 2017

Resolves #786

@ghoneycutt ghoneycutt merged commit 7cc9a9b into sensu:master Aug 3, 2017
@ghoneycutt
Copy link
Collaborator

Released in v2.31.0

@Phil-Friderici Phil-Friderici deleted the GH-768 branch September 27, 2017 07:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants