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.

Sign up for GitHub

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

Jump to bottom

Fixed an inconsistency in the value of the tag sql_instance #6610

Merged
merged 1 commit into from
Nov 5, 2019
Merged

Fixed an inconsistency in the value of the tag sql_instance #6610

merged 1 commit into from
Nov 5, 2019

Conversation

Trovalo
Copy link
Collaborator

@Trovalo Trovalo commented Nov 4, 2019 •
edited
Loading

Added some missing REPLACE(@@ServerName,'\',':') which caused inconsistency in the value of the tag [sql_instance]

Required for all PRs:

  • Signed CLA.
  • Associated README.md updated.
  • Has appropriate unit tests.

@Trovalo
Added some missing REPLACE(@@ServerName,'\',':') which caused inconsi…
73ea18e 
…stency in the value of the tag [sql_instance]
@danielnelson danielnelson added this to the 1.12.5 milestone Nov 5, 2019
@danielnelson danielnelson added area/azure Azure plugins including eventhub_consumer, azure_storage_queue, azure_monitor area/sqlserver fix pr to fix corresponding bug labels Nov 5, 2019
@danielnelson danielnelson merged commit ba57981 into influxdata:master Nov 5, 2019
danielnelson pushed a commit that referenced this pull request Nov 5, 2019
@Trovalo @danielnelson
Add missing character replacement to sql_instance tag (#6610)
6745e07
@Trovalo Trovalo deleted the Input-SqlServer-Fix-sql_instance branch November 8, 2019 08:08
athoune pushed a commit to bearstech/telegraf that referenced this pull request Apr 17, 2020
@Trovalo
Add missing character replacement to sql_instance tag (influxdata#6610)
b64a3a8
idohalevi pushed a commit to idohalevi/telegraf that referenced this pull request Sep 29, 2020
@Trovalo
Add missing character replacement to sql_instance tag (influxdata#6610)
52a61cb
arstercz pushed a commit to arstercz/telegraf that referenced this pull request Mar 5, 2023
@Trovalo @danielnelson
Add missing character replacement to sql_instance tag (influxdata#6610)
a6c412c
arstercz pushed a commit to arstercz/telegraf that referenced this pull request Mar 5, 2023
@Trovalo @danielnelson
Add missing character replacement to sql_instance tag (influxdata#6610)
86986eb
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
area/azure Azure plugins including eventhub_consumer, azure_storage_queue, azure_monitor area/sqlserver fix pr to fix corresponding bug
Projects
None yet
Milestone
1.12.5
Development

Successfully merging this pull request may close these issues.
2 participants
@Trovalo @danielnelson