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

use nats source as connection label #135

Merged
merged 1 commit into from
Aug 11, 2023

Conversation

mikemrm
Copy link
Contributor

@mikemrm mikemrm commented Aug 11, 2023

Nats supports providing a connection a name.
To help with troubleshooting we should set this name so it's clear which connections are from which service.

We already have the Source field which defines the source in the messages, so we can simply reuse this variable and add a conneciton option to set the name.

Nats supports providing a connection a name.
To help with troubleshooting we should set this name so it's clear which
connections are from which service.

We already have the Source field which defines the source in the
messages, so we can simply reuse this variable and add a conneciton
option to set the name.

Signed-off-by: Mike Mason <mimason@equinix.com>
@mikemrm mikemrm marked this pull request as ready for review August 11, 2023 15:09
@mikemrm mikemrm requested a review from a team as a code owner August 11, 2023 15:09
@mikemrm mikemrm added this pull request to the merge queue Aug 11, 2023
Merged via the queue into infratographer:main with commit f9ec4d4 Aug 11, 2023
@mikemrm mikemrm deleted the nats-conn-label branch August 11, 2023 19:40
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.

2 participants