-
Notifications
You must be signed in to change notification settings - Fork 5.5k
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
Feat: SQL Server - sqlServerRequestsV2- added s.[login_name] #9955
Feat: SQL Server - sqlServerRequestsV2- added s.[login_name] #9955
Conversation
Thanks so much for the pull request! |
!signed-cla |
@denzilribeiro @sspaink Can you have a look at this and merge it? Need this flag urgently for our use case. |
Can you rebase your pull request with master? The CI is out of date and we are now using Go 1.17. The PR itself looks good, thanks for the contribution. |
3914058
to
56bf149
Compare
@sspaink Thanks for the quick revert and approval. I have rebased the branch with master. Can we go ahead with the merge? |
@princebansal before it can be merged it will need to be reviewed by another member of the core Telegraf team, I've labeled the pr with |
The |
56bf149
to
b5fc7d4
Compare
@sspaink Thanks again for the update. I have amended the commit message to the correct format. Thanks! |
📦 Looks like new artifacts were built from this PR. Expand this list to get them here! 🐯Artifact URLs |
@Hipska Can you please review this? |
I cannot approve this PR since the README states this:
So please make updates to the specific database type instead of version2. |
@Hipska Thanks a lot for that pointer. I saw that by specifying I think we can close this PR. :) |
Required for all PRs:
This PR is an extension to #8351. Currently
sqlServerRequestsV2
insqlqueriesV2.go
doesn't give SQL login name and onlynt_user_name
.resolves #
This PR will add
s.login_name
in the output which will be helpful in logging the SQL server login name.