-
-
Notifications
You must be signed in to change notification settings - Fork 98
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
Port to FreeBSD #172
base: master
Are you sure you want to change the base?
Port to FreeBSD #172
Conversation
$service_hasstatus = true | ||
$service_restart = 'pkill -HUP telegraf' | ||
} | ||
'FreeBSD' : { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The metadata should include FreeBSD support at specific versions.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Also README should be updated to list FreeBSD.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Metadata and README fixed in 0391569
Joris
metadata.json
Outdated
"11", | ||
"12", | ||
"13" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"11", | |
"12", | |
"13" | |
"11", | |
"12", | |
"13" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ok fixed
Thanks
@rmdir can you take a look at the failing tests? |
I took a quick look lere. I'm not sure if you want to instal telegraf via an archive or package so I won't be able to adjust the tests accordingly. |
I fixed the tab issue |
On FreeBSD there is a grate package package { 'telegraf': } just works |
@rmdir can you take a look at the failing tests? |
Dear @rmdir, thanks for the PR! This is Vox Pupuli Tasks, your friendly Vox Pupuli GitHub Bot. I noticed that your pull request contains merge conflict. Can you please rebase? You can find my sourcecode at voxpupuli/vox-pupuli-tasks |
Pull Request (PR) description
This PR contains some code that makes puppet-teletgraf works with FreeBSD