-
Notifications
You must be signed in to change notification settings - Fork 1.5k
add source to sqlserver integration dashboard #21022
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
Conversation
This PR does not modify any files shipped with the agent. To help streamline the release process, please consider adding the |
/to-staging |
View all feedbacks in Devflow UI.
This command is disabled. If you need support, contact us on Slack #devflow with those details! |
/integrate |
View all feedbacks in Devflow UI.
No services specified for deployment and BIA didn't detect any impacted services |
/merge |
View all feedbacks in Devflow UI.
This merge request is not mergeable yet, because of pending checks/missing approvals. It will be added to the queue as soon as checks pass and/or get approvals.
devflow unqueued this merge request: It did not become mergeable within the expected time |
/merge |
View all feedbacks in Devflow UI.
This merge request is not mergeable yet, because of pending checks/missing approvals. It will be added to the queue as soon as checks pass and/or get approvals.
devflow unqueued this merge request: It did not become mergeable within the expected time |
/merge |
View all feedbacks in Devflow UI.
PR already in the queue with status waiting |
What does this PR do?
Customer reported their integration dashboard shows data when the widget is closed, but not when it is open. This is because the query does not have a
source
parameter, and the Graph Editor defaults tosource:postgresql
when the widget is opened (source1, source2). We can prevent this by explicitly setting thesource
parameter.Query works (implicit sqlserver):

Query doesn't work (explicit postgresql):

Query works again (explicit sqlserver):

Motivation
Sales prospect noticed broken integration dashboard: https://datadog.zendesk.com/agent/tickets/2210932
Review checklist (to be filled by reviewers)
qa/skip-qa
label if the PR doesn't need to be tested during QA.backport/<branch-name>
label to the PR and it will automatically open a backport PR once this one is merged