Skip to content

Conversation

@tom-pang
Copy link

exports one new metric,
pg_shared_preload_libraries_library_enabled{library=<name}=1 per
library across the fleet

This helps us understand and track rollouts

exports one new metric,
`pg_shared_preload_libraries_library_enabled{library=<name}=1` per
library across the fleet

This helps us understand and track rollouts
Copy link

@JTCunning JTCunning left a comment

Choose a reason for hiding this comment

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

Leaving a comment as I am not trusted to review golang.

"github.com/prometheus/client_golang/prometheus"
)

const sharedPreloadLibrariesSubsystem = "shared_preload_libraries"

Choose a reason for hiding this comment

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

Suggested change
const sharedPreloadLibrariesSubsystem = "shared_preload_libraries"
const sharedPreloadLibrariesSubsystem = "settings_shared_preload_libraries"

If we're only ever going to emit a metric from querying pg_settings I think it's worth either applying that namespace in the metric constant, or in the namespace that's getting passed below, which I'm not sure where that comes from. Follow your heart.

Copy link
Author

Choose a reason for hiding this comment

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

gonna slightly adjust but yeah, agree.

Copy link
Collaborator

@maxenglander maxenglander left a comment

Choose a reason for hiding this comment

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

Lgtm!

@@ -0,0 +1,87 @@
// Copyright 2024 The Prometheus Authors
Copy link
Collaborator

Choose a reason for hiding this comment

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

Nit 2025

@@ -0,0 +1,187 @@
// Copyright 2024 The Prometheus Authors
Copy link
Collaborator

Choose a reason for hiding this comment

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

Same nit

@tom-pang tom-pang merged commit 193afae into main Sep 25, 2025
2 checks passed
@tom-pang tom-pang deleted the export_shared_preload_libraries branch September 25, 2025 12:33
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