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

Add datasource label to secure_socks_requests_duration #866

Conversation

PoorlyDefinedBehaviour
Copy link
Member

@PoorlyDefinedBehaviour PoorlyDefinedBehaviour commented Jan 19, 2024

What this PR does / why we need it:

Adds the datasource and datasourceType labels to secure_socks_requests_duration.

The operator experience squad wants to expose some Grafana private datasource connect to customers and these labels will make it easier to do so.

Which issue(s) this PR fixes:

Part of https://github.com/grafana/hosted-grafana/issues/5151

Special notes for your reviewer:

@PoorlyDefinedBehaviour PoorlyDefinedBehaviour requested a review from a team as a code owner January 19, 2024 17:41
@PoorlyDefinedBehaviour PoorlyDefinedBehaviour requested review from wbrowne, marefr and xnyo and removed request for a team January 19, 2024 17:41
@PoorlyDefinedBehaviour PoorlyDefinedBehaviour force-pushed the add-datasource-label-to-secure_socks_requests_duration branch 4 times, most recently from 7f92af6 to 5ebee79 Compare January 19, 2024 17:59
Copy link
Member

@xnyo xnyo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great work! Leaving some minor comments:

backend/proxy/secure_socks_proxy.go Outdated Show resolved Hide resolved
backend/proxy/secure_socks_proxy.go Outdated Show resolved Hide resolved
backend/proxy/options.go Outdated Show resolved Hide resolved
Co-authored-by: Giuseppe Guerra <giuseppe.guerra@grafana.com>
Co-authored-by: Giuseppe Guerra <giuseppe.guerra@grafana.com>
Co-authored-by: Giuseppe Guerra <giuseppe.guerra@grafana.com>
@PoorlyDefinedBehaviour PoorlyDefinedBehaviour requested review from xnyo and marefr and removed request for wbrowne, marefr and xnyo January 23, 2024 13:38
Copy link
Contributor

@marefr marefr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Member

@xnyo xnyo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! 🚀

@PoorlyDefinedBehaviour PoorlyDefinedBehaviour merged commit c61c9d0 into main Feb 1, 2024
3 checks passed
@PoorlyDefinedBehaviour PoorlyDefinedBehaviour deleted the add-datasource-label-to-secure_socks_requests_duration branch February 1, 2024 13:07
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.

3 participants