Skip to content

Add opt-in per-submission-mode breakdown for user metrics - #1259

Merged
bhou2 merged 1 commit into
masterfrom
user-metrics-split-by-mode
Jul 13, 2026
Merged

Add opt-in per-submission-mode breakdown for user metrics#1259
bhou2 merged 1 commit into
masterfrom
user-metrics-split-by-mode

Conversation

@bhou2

@bhou2 bhou2 commented Jul 10, 2026

Copy link
Copy Markdown
Contributor

Introduce an opt-in genie.tasks.user-metrics.split-by-submission-mode flag that, when enabled, additionally publishes per-user active-jobs and active-memory gauges broken down by job submission mode (submissionMode=api for REST API vs submissionMode=agent for agent/CLI) under new genie.user.active-jobs.by-submission-mode.gauge / genie.user.active-memory.by-submission-mode.gauge metric names.

Unit and Integration Tests:

  • UserMetricsTaskSpec
  • JpaPersistenceServiceImplJobsIntegrationTest

Dev Tests:

New metric [both api mode and agent mode included]:

image

Legacy metric [only api mode included]:

image

@bhou2
bhou2 force-pushed the user-metrics-split-by-mode branch from 3ecf5e6 to 1be4cd5 Compare July 10, 2026 07:39
@bhou2
bhou2 force-pushed the user-metrics-split-by-mode branch from 1be4cd5 to 6f26fad Compare July 11, 2026 00:00
@bhou2
bhou2 force-pushed the user-metrics-split-by-mode branch from 6f26fad to 1ae871b Compare July 13, 2026 17:28
@bhou2
bhou2 marked this pull request as ready for review July 13, 2026 17:52
@bhou2
bhou2 requested review from rdeepak2002 and zdong2 July 13, 2026 17:53
@bhou2
bhou2 force-pushed the user-metrics-split-by-mode branch from 1ae871b to 1d24cd5 Compare July 13, 2026 17:58

@zdong2 zdong2 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM. Thanks for adding it!

@rdeepak2002 rdeepak2002 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

lgtm, thanks!

@bhou2
bhou2 merged commit 17fbef4 into master Jul 13, 2026
2 checks passed
@bhou2
bhou2 deleted the user-metrics-split-by-mode branch July 13, 2026 21:47
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