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

device-metrics: create timeseries per-device with device_uuid tag #88

Merged
merged 1 commit into from Oct 21, 2019

Conversation

xginn8
Copy link
Contributor

@xginn8 xginn8 commented Aug 12, 2019

@xginn8 xginn8 force-pushed the per-device-metrics branch 3 times, most recently from 7513f3a to c399e9e Compare August 12, 2019 20:41
@xginn8 xginn8 force-pushed the per-device-metrics branch 2 times, most recently from 6dfb1a8 to bc0d9ce Compare August 26, 2019 12:44
@xginn8
Copy link
Contributor Author

xginn8 commented Aug 26, 2019

generally inserting label names directly into the metric names is discouraged, however in this case we are covered by the one exception that makes sense (from https://www.robustperception.io/on-the-naming-of-things):

There's one very rare case where this is okay, and that's where you need to distinguish multiple metrics based on the same events but which have different labels for performance reasons.

@xginn8 xginn8 force-pushed the per-device-metrics branch 2 times, most recently from 8b8e717 to 4b19552 Compare August 29, 2019 15:55
@xginn8 xginn8 force-pushed the per-device-metrics branch 2 times, most recently from 585b756 to 6629775 Compare October 2, 2019 09:55
@xginn8 xginn8 force-pushed the per-device-metrics branch 2 times, most recently from bc6673e to 6be4abd Compare October 17, 2019 14:09
@xginn8 xginn8 merged commit db28775 into master Oct 21, 2019
@xginn8 xginn8 deleted the per-device-metrics branch October 21, 2019 15:58
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