Kafka, Linux_proc_extras, Directory, and many other small fixes or additions#357
Kafka, Linux_proc_extras, Directory, and many other small fixes or additions#357truthbk merged 19 commits intoDataDog:masterfrom alexfouche:master
Conversation
subrepo: subdir: "modules/datadog_agent" merged: "f1465c9" upstream: origin: "git@github.com:alexfouche/puppet-datadog-agent.git" branch: "master" commit: "7a2ed2f" git-subrepo: version: "0.4.0" origin: "https://github.com/ingydotnet/git-subrepo" commit: "2e78d5d"
subrepo: subdir: "modules/datadog_agent" merged: "91cf4d8" upstream: origin: "git@github.com:alexfouche/puppet-datadog-agent.git" branch: "master" commit: "ebf34de" git-subrepo: version: "0.4.0" origin: "https://github.com/ingydotnet/git-subrepo" commit: "2e78d5d"
…et to false. Yet Datadog defaults are not always set to true, and one must be able to write explicitely a false attribute in the configuration
[update] #357 Kafka, Linux_proc_extras, Directory, and many other small fixes or additions
There was a problem hiding this comment.
This was a really awesome piece of work @alexfouche - I was able to use most of it. There were some changes that needed to be made to accomodate for old puppets. I am aware that this review took a lot longer than it should've so I felt bad asking you to make changes and keep this work in backburner longer. I drafted a separate PR #376 with most of your work here, and some minor changes. Let me know if you have any feedback.
| ) inherits datadog_agent::params { | ||
| include datadog_agent | ||
|
|
||
| file { "${datadog_agent::params::conf_dir}/linux_proc_extras.yaml": |
There was a problem hiding this comment.
should be datadog_agent::conf_dir, as conf_dir is overridable.
| # } | ||
| class datadog_agent::integrations::disk ( | ||
| $use_mount = 'no', | ||
| $use_mount = false, |
There was a problem hiding this comment.
This is exactly what should've been from day one, and I we'll make sure some of this stuff is fixed in the upcoming major release. Unfortunately this would break manifests for current users. I don't think this was currently broken... if it is, please open an issue and I'll tackle it ASAP in the bugfix release.
| $tag_name = '', | ||
| $pattern = '*', | ||
| $recursive = false | ||
| Array[Struct[{ |
There was a problem hiding this comment.
This isn't supported in older puppets, so we're going to have to something suboptimal like we do with haproxy or postgres. I know, its unfortunate thay we have to prescind of a clean elegant solution like this one to accomodate for older puppets. This sort of thing should become the norm soon.
[update] DataDog#357 Kafka, Linux_proc_extras, Directory, and many other small fixes or additions
2 participants
No description provided.