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

Chef Automate support should be added #423

Closed
mattray opened this issue Sep 16, 2016 · 6 comments
Closed

Chef Automate support should be added #423

mattray opened this issue Sep 16, 2016 · 6 comments

Comments

@mattray
Copy link

mattray commented Sep 16, 2016

The data_collector.server_url and data_collector.token configuration options should be exposed for management through attributes.

https://docs.chef.io/setup_visibility_chef_automate.html#configure-chef-client-to-send-node-and-converge-data

@yvovandoorn
Copy link

👍

To add to this, it should also handle knife ssl fetch if the Automate Server is using a self-signed certificate.

@tas50
Copy link
Contributor

tas50 commented Oct 25, 2016

@yvovandoorn / @mattray Someone wanna cut a PR for this?

@jerryaldrichiii
Copy link

As far as @mattray's point goes you can just set:

default['chef_client']['config']['data_collector.server_url'] = 'https://my.automate.server/data-collector/v0/'
default['chef_client']['config']['data_collector.token'] = '93a49a4f2482c64126f7b6015e6b0f30284287ee4054ff8807fb63d9cbd1c506'

Adding explicit support would be nice though.

@tas50
Copy link
Contributor

tas50 commented Dec 6, 2016

The best way to do this is to add them to the config hash attribute. I'd be open to adding a documentation section to this cookbook that would give examples of how to do that. We should not be adding more attributes to this cookbook though. There's too many as is.

@tas50 tas50 closed this as completed Dec 6, 2016
@jerryaldrichiii
Copy link

Yuuup which is what I meant by my last comment.

I'd keep this issue open for @yvovandoorn's SSL integration (as part of Automate) or move it to a separate issue.

@tas50
Copy link
Contributor

tas50 commented Dec 6, 2016

@yvovandoorn If you want to open another feature request issue for the ssl fetching stuff that would be helpful to track it

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging a pull request may close this issue.

4 participants