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.

Sign up for GitHub

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

Jump to bottom

Add host tag to InfluxDB metrics #692

Merged
merged 1 commit into from Jun 10, 2020
Merged

Add host tag to InfluxDB metrics #692

merged 1 commit into from Jun 10, 2020

Conversation

knl
Copy link
Contributor

@knl knl commented Nov 14, 2019

This should help us discern machines in environments with multiple Hydra deployments.

@knl knl requested a review from edolstra November 15, 2019 08:57
@knl
Add host tag to InfluxDB metrics
01ae944 
This should help us discern machines in environments with multiple Hydra deployments.
@knl
Copy link
Contributor Author

knl commented Mar 12, 2020

@edolstra @basvandijk please take another look, I changed the code to use Sys::Hostname::Long.

@Mic92
Copy link
Member

Mic92 commented Jun 10, 2020

ping

@edolstra edolstra merged commit 1831866 into NixOS:master Jun 10, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@edolstra edolstra Awaiting requested review from edolstra

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@knl @Mic92 @edolstra