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

Add telemetry configuration possibilites #88

Merged
merged 3 commits into from
Jul 2, 2020

Conversation

rndmh3ro
Copy link
Collaborator

No description provided.

@ducminhle
Copy link
Contributor

Hello @rndmh3ro,
Do we need to add telemetry config? I see we have nomad_config_custom, I think we can add telemetry config in this variable.
Ex:

nomad_config_custom:
telemetry:
publish_allocation_metrics: true
publish_node_metrics: true
prometheus_metrics: true

@rndmh3ro
Copy link
Collaborator Author

Hey @ducminhle,

we probably don't need this. However this way it's documented and easier to use.

@rndmh3ro
Copy link
Collaborator Author

Any reasons against merging this?

@lanefu lanefu merged commit ebd9324 into ansible-community:master Jul 2, 2020
@lanefu lanefu mentioned this pull request Jul 13, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants