Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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
Add
logfire.instrument_system_metrics()
#373Add
logfire.instrument_system_metrics()
#373Changes from 14 commits
b713282
7400de8
c1c4e6b
13d0baa
dd05380
ac2b5d7
f882a10
4a9ade1
be7282c
6d7f158
ea40267
7b397d6
48c31bd
68e3f28
7a90d6d
2ab5017
1087dc1
19fca49
a3d5335
c9e889b
34ed55c
fb9a784
44c17c1
3ad0209
142ba70
8c78289
d1dd843
edb3ad8
86073d5
8aa1d1b
5ec79e5
3fc0971
2cc9e1a
8b69741
9bb6b43
f9d86eb
da3e3fa
389b920
e953802
8ea604f
99ada07
900c8e6
e8b0e26
a35d665
cdbcb84
86acff7
d7f4902
be5e9ef
bae2c6d
b2144ad
90df9ca
f0b8d86
c39b697
2102b39
0ba76a7
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The current 'Basic System Metrics' dashboard template won't work with this new set of reduced metrics. I will create a PR with a new template. That PR will need to be ready to merge before these changes can be merged, and it should be deployed approximately when this is released.
When users upgrade the SDK, system metrics will stop being collected, if they were being collected at all. Since their collection was the implicit default, we can't warn them about this in code, and must rely on the changelog and announcements in slack. If the only thing that users do is add
logfire.instrument_system_metrics()
then existing basic system metrics dashboards will still not work, so we need to make it clear that they also need to switch to the new template.The old template is still useful though, because it works with OTEL conventions, so system metrics collected by means other than the logfire SDK can be used with the old template. So we should keep and document both. We need to figure out how to name them to clarify the difference, e.g. 'Basic System Metrics (Logfire)' and 'Basic System Metrics (OpenTelemetry)'.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
PR here: https://github.com/pydantic/platform/pull/3838