Skip to content

fix: modify queue gauge to use prometheus naming convention #64

Merged
nuclearcat merged 1 commit intokernelci:mainfrom
gustavobtflores:fix/kcidb-queue-gauge-naming
Mar 18, 2026
Merged

fix: modify queue gauge to use prometheus naming convention #64
nuclearcat merged 1 commit intokernelci:mainfrom
gustavobtflores:fix/kcidb-queue-gauge-naming

Conversation

@gustavobtflores
Copy link
Copy Markdown
Collaborator

Modify kcidb_json_files_total to kcidb_json_files to follow prometheus gauge naming conventions

@gustavobtflores gustavobtflores changed the title fix: modify queue gauge to use convention naming fix: modify queue gauge to use prometheus naming convention Dec 15, 2025
Copy link
Copy Markdown

@AmadeusK525 AmadeusK525 left a comment

Choose a reason for hiding this comment

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

Pre-approving

Comment on lines 147 to +151
metrics.push_str(
"# HELP kcidb_json_files_total Total number of JSON files in the spool directory\n",
);
metrics.push_str("# TYPE kcidb_json_files_total gauge\n");
metrics.push_str(&format!("kcidb_json_files_total {}\n", json_files_num));
metrics.push_str("# TYPE kcidb_json_files gauge\n");
metrics.push_str(&format!("kcidb_json_files {}\n", json_files_num));
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Need to change the help string as well

@gustavobtflores gustavobtflores force-pushed the fix/kcidb-queue-gauge-naming branch from 3edecdc to 69b142c Compare December 15, 2025 13:59
@nuclearcat nuclearcat added this pull request to the merge queue Mar 18, 2026
Merged via the queue into kernelci:main with commit f8cfe5d Mar 18, 2026
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