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

feat!: Add scaling duration dashboard #347

Conversation

henrybell
Copy link
Collaborator

@henrybell henrybell commented Jul 9, 2024

This additionally updates the metrics domain used by the Cloud Functions deployment from custom.googleapis.com to workload.googleapis.com (to match the GCP Metrics exporter for the OpenTelemetry collector).

@henrybell
Copy link
Collaborator Author

/gcbrun

@henrybell henrybell force-pushed the henrybell-scaler-metrics-dashboard branch from 375af47 to 8aec6bb Compare July 9, 2024 15:27
@henrybell henrybell requested a review from nielm July 9, 2024 17:22
@henrybell henrybell added the enhancement New feature or request label Jul 9, 2024
@nielm
Copy link
Collaborator

nielm commented Jul 10, 2024

Consider changing this to a feat! with a BREAKING CHANGE comment, and holding off merging until we decide when to do a cut off to v3.

Reason: if anyone is using the metrics, this will break their existing alerts/dashboards

Copy link
Collaborator

@nielm nielm left a comment

Choose a reason for hiding this comment

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

See other comment about changing to a feat!

BREAKING CHANGE: Update metrics domain for CF to workload.googleapis.com
@henrybell henrybell force-pushed the henrybell-scaler-metrics-dashboard branch from 8aec6bb to f23bc72 Compare July 11, 2024 12:12
@henrybell henrybell changed the title feat: Add scaling duration dashboard feat!: Add scaling duration dashboard Jul 11, 2024
@henrybell henrybell marked this pull request as draft July 11, 2024 12:23
@henrybell
Copy link
Collaborator Author

Converted to draft pending discussion on when to merge.

@henrybell henrybell marked this pull request as ready for review July 23, 2024 13:03
@henrybell henrybell requested a review from nielm July 23, 2024 13:04
@henrybell henrybell merged commit 874593f into cloudspannerecosystem:main Jul 23, 2024
8 checks passed
@henrybell henrybell deleted the henrybell-scaler-metrics-dashboard branch July 23, 2024 13:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants