Skip to content

fix(journald source): Fix typo in metrics name#3252

Merged
bruceg merged 1 commit intomasterfrom
journald-event-fixup
Jul 29, 2020
Merged

fix(journald source): Fix typo in metrics name#3252
bruceg merged 1 commit intomasterfrom
journald-event-fixup

Conversation

@bruceg
Copy link
Copy Markdown
Member

@bruceg bruceg commented Jul 29, 2020

Signed-off-by: Bruce Guenter bruce@timber.io

Looks like we missed this typo @lukesteensen. Thanks to @fanatid for pointing this out.

What do others think about defining constants for these common strings to make typos harder in the future?

Signed-off-by: Bruce Guenter <bruce@timber.io>
@bruceg bruceg added type: bug A code related bug. domain: observability Anything related to monitoring/observing Vector source: journald Anything `journald` source related labels Jul 29, 2020
@bruceg bruceg requested a review from lukesteensen July 29, 2020 16:28
@bruceg bruceg self-assigned this Jul 29, 2020
@bruceg bruceg requested a review from fanatid July 29, 2020 16:29
@binarylogic
Copy link
Copy Markdown
Contributor

What do others think about defining constants for these common strings to make typos harder in the future?

I'm a fan, ref #3247, but I think we're going to learn more as we build out the actual interfaces using this data. So I'm a little hesitant to cement everything riught now.

@bruceg bruceg merged commit d64c8b1 into master Jul 29, 2020
@bruceg bruceg deleted the journald-event-fixup branch July 29, 2020 16:41
mengesb pushed a commit to jacobbraaten/vector that referenced this pull request Dec 9, 2020
Signed-off-by: Bruce Guenter <bruce@timber.io>
Signed-off-by: Brian Menges <brian.menges@anaplan.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

domain: observability Anything related to monitoring/observing Vector source: journald Anything `journald` source related type: bug A code related bug.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants