Skip to content

chore(metrics): send only new format for pipeline trigger metrics #605

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
Draft
wants to merge 1 commit into
base: main
Choose a base branch
from
Draft

chore(metrics): send only new format for pipeline trigger metrics #605

wants to merge 1 commit into from
+1 −29

Conversation

jvallesm
Copy link
Collaborator

Because

  • The new dashboard design requires breaking changes in the metrics, so until the frontend was implemented both the old and new metrics were sent.

This commit

  • Stops sending the metrics with the old format.

@jvallesm jvallesm self-assigned this Aug 22, 2024
@linear Linear
Copy link

linear bot commented Aug 22, 2024

INS-5379 Credit dashboard endpoints

@jvallesm jvallesm changed the title chore(metrics): Send only new format for pipeline trigger metrics chore(metrics): send only new format for pipeline trigger metrics Aug 22, 2024
@donch1989 donch1989 marked this pull request as ready for review August 22, 2024 09:44
@donch1989 donch1989 marked this pull request as draft August 22, 2024 09:44
@jvallesm jvallesm force-pushed the jvalles/ins-5379-credit-dashboard-endpoints branch from ee8deb6 to 7ae67ee Compare October 4, 2024 11:01
@pinglin pinglin force-pushed the main branch 5 times, most recently from 4357e3d to 61573ac Compare June 29, 2025 10:11
@pinglin pinglin force-pushed the main branch 5 times, most recently from 91eb182 to 349ccd3 Compare July 5, 2025 01:09
@pinglin pinglin force-pushed the main branch 2 times, most recently from 21e92d2 to 34bb299 Compare July 15, 2025 20:41
@pinglin pinglin force-pushed the main branch 6 times, most recently from 2c8422f to fad1c24 Compare September 19, 2025 02:59
@pinglin pinglin force-pushed the main branch 12 times, most recently from 203fb3f to 1153a09 Compare September 27, 2025 21:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@donch1989 donch1989 Awaiting requested review from donch1989 donch1989 is a code owner

@pinglin pinglin Awaiting requested review from pinglin pinglin is a code owner

@xiaofei-du xiaofei-du Awaiting requested review from xiaofei-du

At least 1 approving review is required to merge this pull request.

Assignees

@jvallesm jvallesm

Labels
instill vdp
Projects
No open projects
🔮 Instill Core
Status: 🚧 Planned
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@jvallesm @droplet-bot