djoos-cookbooks / newrelic

Development repository for the newrelic cookbook
https://supermarket.chef.io/cookbooks/newrelic
MIT License
143 stars 245 forks source link

doesn't support strip_command_line parameter #364

Closed brunotag closed 4 years ago

brunotag commented 5 years ago

Hey, so, there's a handy undocumented config parameter for the infrastructure agent.

It's called strip_command_line - it defaults to true, but setting it to false is handy under some circumstances. See for instance https://discuss.newrelic.com/t/is-there-a-way-to-differentiate-linux-processes-by-process-arguments/53632

It appears this recipe doesn't implement it.. right? Is that intentional or would you consider supporting it?

djoos commented 5 years ago

Hi @brunotag,

thanks for reaching out - that sounds like a very interesting addition; if you've got time to send over a PR, I'll get it merged in ASAP!

Thanks in advance!

djoos commented 4 years ago

Fixed by #365