-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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 hostname as part of the subscription substring #15189
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 👍
Codecov Report
Flags with carried forward coverage won't be shown. Click here to find out more. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Could you update the chanelog?
1d8b7ee
to
3a38868
Compare
What does this PR do?
Tries to change the way we subscribe to an ibm mq instance. This will attach the hostname to the subscription, so that the name will be unique per host.
Motivation:
Escalation:
https://datadoghq.atlassian.net/browse/AGENT-9627
Customer sometimes uses multiple host with the same config to monitor ibm_mq instance. This could cause an issue when 2 host with the same config are trying to connect together, they'll get a 2429 error:
https://www.ibm.com/support/pages/multiple-instances-subscriber-same-clientid-are-receiving-reason-code-2429-mqrcsubscriptioninuse
This change aims at making the subscription name unique per host so that this doesn't happen.