-
Notifications
You must be signed in to change notification settings - Fork 262
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
Add installation info metadata support #707
Conversation
bbda01f
to
a865ced
Compare
recipes/install_info.rb
Outdated
|
||
variables( | ||
install_method: "chef-#{Chef::VERSION}", | ||
install_method_version: "datadog-cookbook-#{Chef::Datadog.cookbook_version(run_context)}" |
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.
Maybe we want to call it datadog_cookbook
so the dash -
separates name from version.
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.
Woops! Good catch
@@ -0,0 +1,6 @@ | |||
<%= JSON.parse(({ |
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.
JSON.parse().to_json.to_yaml
is this the way to print a yaml? 🤔
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.
C/p from other template, I'll improve
436c1cf
to
f0e2a03
Compare
What does this PR do?
It adds the installation info metadata to the chef cookbook so the agent returns that it was installed by the triple chef/chef version/cookbook version
Motivation.
Allow customers to track their chef installation. Allows us to make stats on installation methods.