-
Notifications
You must be signed in to change notification settings - Fork 773
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
[sdk-metrics] Promote cardinality limit view API from experimental to stable #5926
Changes from 3 commits
9c2691a
6b9d182
fcb7ac2
8f110e5
6d771c0
53ed5af
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
This file was deleted.
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -6,6 +6,11 @@ Notes](../../RELEASENOTES.md). | |
|
||
## Unreleased | ||
|
||
* The experimental APIs previously covered by `OTEL1003` | ||
(`MetricStreamConfiguration.CardinalityLimit`) will now be part of the public | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. will be -> is? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I wonder how folks think about https://www.theregister.com/2024/10/08/linus_torvalds_grammar_complaint/? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. This wording is also used in other experimental APIs in this CHANGELOG. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I think "is" is better. Would be good to follow up and make changes everywhere. Not a blocker for this. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Created a PR #5940 for this. |
||
API and supported in stable builds. | ||
([#5926](https://github.com/open-telemetry/opentelemetry-dotnet/pull/5926)) | ||
|
||
* Promoted overflow attribute from experimental to stable and removed the | ||
`OTEL_DOTNET_EXPERIMENTAL_METRICS_EMIT_OVERFLOW_ATTRIBUTE` environment variable. | ||
|
||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -98,14 +98,11 @@ | |
} | ||
} | ||
|
||
#if EXPOSE_EXPERIMENTAL_FEATURES | ||
/// <summary> | ||
/// Gets or sets a positive integer value defining the maximum number of | ||
/// data points allowed for the metric managed by the view. | ||
/// </summary> | ||
/// <remarks> | ||
/// <para><b>WARNING</b>: This is an experimental API which might change or | ||
/// be removed in the future. Use at your own risk.</para> | ||
/// <para>Spec reference: <see | ||
/// href="https://github.com/open-telemetry/opentelemetry-specification/blob/main/specification/metrics/sdk.md#cardinality-limits">Cardinality | ||
/// limits</see>.</para> | ||
|
@@ -116,17 +113,10 @@ | |
/// If not set the default | ||
/// MeterProvider cardinality limit of 2000 will apply. | ||
/// </remarks> | ||
#if NET | ||
[Experimental(DiagnosticDefinitions.CardinalityLimitExperimentalApi, UrlFormat = DiagnosticDefinitions.ExperimentalApiUrlFormat)] | ||
#endif | ||
public | ||
#else | ||
internal | ||
#endif | ||
int? CardinalityLimit | ||
public int? CardinalityLimit | ||
{ | ||
get => this.cardinalityLimit; | ||
Check failure on line 118 in src/OpenTelemetry/Metrics/View/MetricStreamConfiguration.cs GitHub Actions / build-test-project-stable / build-test (ubuntu-latest, net9.0)
Check failure on line 118 in src/OpenTelemetry/Metrics/View/MetricStreamConfiguration.cs GitHub Actions / build-test-project-stable / build-test (ubuntu-latest, net9.0)
Check failure on line 118 in src/OpenTelemetry/Metrics/View/MetricStreamConfiguration.cs GitHub Actions / build-test-project-stable / build-test (ubuntu-latest, net9.0)
Check failure on line 118 in src/OpenTelemetry/Metrics/View/MetricStreamConfiguration.cs GitHub Actions / build-test-project-stable / build-test (ubuntu-latest, net9.0)
Check failure on line 118 in src/OpenTelemetry/Metrics/View/MetricStreamConfiguration.cs GitHub Actions / build-test-project-stable / build-test (ubuntu-latest, net9.0)
Check failure on line 118 in src/OpenTelemetry/Metrics/View/MetricStreamConfiguration.cs GitHub Actions / build-test-project-stable / build-test (ubuntu-latest, net8.0)
Check failure on line 118 in src/OpenTelemetry/Metrics/View/MetricStreamConfiguration.cs GitHub Actions / build-test-project-stable / build-test (ubuntu-latest, net8.0)
Check failure on line 118 in src/OpenTelemetry/Metrics/View/MetricStreamConfiguration.cs GitHub Actions / build-test-project-stable / build-test (ubuntu-latest, net8.0)
Check failure on line 118 in src/OpenTelemetry/Metrics/View/MetricStreamConfiguration.cs GitHub Actions / build-test-project-stable / build-test (ubuntu-latest, net8.0)
Check failure on line 118 in src/OpenTelemetry/Metrics/View/MetricStreamConfiguration.cs GitHub Actions / build-test-project-stable / build-test (ubuntu-latest, net8.0)
Check failure on line 118 in src/OpenTelemetry/Metrics/View/MetricStreamConfiguration.cs GitHub Actions / build-test-project-stable / build-test (otel-linux-arm64, net9.0)
Check failure on line 118 in src/OpenTelemetry/Metrics/View/MetricStreamConfiguration.cs GitHub Actions / build-test-project-stable / build-test (otel-linux-arm64, net9.0)
Check failure on line 118 in src/OpenTelemetry/Metrics/View/MetricStreamConfiguration.cs GitHub Actions / build-test-project-stable / build-test (otel-linux-arm64, net9.0)
Check failure on line 118 in src/OpenTelemetry/Metrics/View/MetricStreamConfiguration.cs GitHub Actions / build-test-project-stable / build-test (otel-linux-arm64, net9.0)
Check failure on line 118 in src/OpenTelemetry/Metrics/View/MetricStreamConfiguration.cs GitHub Actions / build-test-project-stable / build-test (otel-linux-arm64, net9.0)
Check failure on line 118 in src/OpenTelemetry/Metrics/View/MetricStreamConfiguration.cs GitHub Actions / validate-packages / run-package-validation-stable
Check failure on line 118 in src/OpenTelemetry/Metrics/View/MetricStreamConfiguration.cs GitHub Actions / validate-packages / run-package-validation-stable
Check failure on line 118 in src/OpenTelemetry/Metrics/View/MetricStreamConfiguration.cs GitHub Actions / validate-packages / run-package-validation-stable
Check failure on line 118 in src/OpenTelemetry/Metrics/View/MetricStreamConfiguration.cs GitHub Actions / validate-packages / run-package-validation-stable
Check failure on line 118 in src/OpenTelemetry/Metrics/View/MetricStreamConfiguration.cs GitHub Actions / validate-packages / run-package-validation-stable
Check failure on line 118 in src/OpenTelemetry/Metrics/View/MetricStreamConfiguration.cs GitHub Actions / build-test-project-stable / build-test (windows-latest, net8.0)
Check failure on line 118 in src/OpenTelemetry/Metrics/View/MetricStreamConfiguration.cs GitHub Actions / build-test-project-stable / build-test (windows-latest, net8.0)
Check failure on line 118 in src/OpenTelemetry/Metrics/View/MetricStreamConfiguration.cs GitHub Actions / build-test-project-stable / build-test (windows-latest, net8.0)
Check failure on line 118 in src/OpenTelemetry/Metrics/View/MetricStreamConfiguration.cs GitHub Actions / build-test-project-stable / build-test (windows-latest, net8.0)
Check failure on line 118 in src/OpenTelemetry/Metrics/View/MetricStreamConfiguration.cs GitHub Actions / build-test-project-stable / build-test (windows-latest, net8.0)
Check failure on line 118 in src/OpenTelemetry/Metrics/View/MetricStreamConfiguration.cs GitHub Actions / build-test-project-stable / build-test (windows-latest, net462)
Check failure on line 118 in src/OpenTelemetry/Metrics/View/MetricStreamConfiguration.cs GitHub Actions / build-test-project-stable / build-test (windows-latest, net462)
Check failure on line 118 in src/OpenTelemetry/Metrics/View/MetricStreamConfiguration.cs GitHub Actions / build-test-project-stable / build-test (windows-latest, net462)
Check failure on line 118 in src/OpenTelemetry/Metrics/View/MetricStreamConfiguration.cs GitHub Actions / build-test-project-stable / build-test (windows-latest, net462)
Check failure on line 118 in src/OpenTelemetry/Metrics/View/MetricStreamConfiguration.cs GitHub Actions / build-test-project-stable / build-test (windows-latest, net462)
Check failure on line 118 in src/OpenTelemetry/Metrics/View/MetricStreamConfiguration.cs GitHub Actions / build-test-project-stable / build-test (windows-latest, net9.0)
Check failure on line 118 in src/OpenTelemetry/Metrics/View/MetricStreamConfiguration.cs GitHub Actions / build-test-project-stable / build-test (windows-latest, net9.0)
Check failure on line 118 in src/OpenTelemetry/Metrics/View/MetricStreamConfiguration.cs GitHub Actions / build-test-project-stable / build-test (windows-latest, net9.0)
Check failure on line 118 in src/OpenTelemetry/Metrics/View/MetricStreamConfiguration.cs GitHub Actions / build-test-project-stable / build-test (windows-latest, net9.0)
Check failure on line 118 in src/OpenTelemetry/Metrics/View/MetricStreamConfiguration.cs GitHub Actions / build-test-project-stable / build-test (windows-latest, net9.0)
Check warning on line 118 in src/OpenTelemetry/Metrics/View/MetricStreamConfiguration.cs GitHub Actions / lint-dotnet-format / run-dotnet-format-stable
Check warning on line 118 in src/OpenTelemetry/Metrics/View/MetricStreamConfiguration.cs GitHub Actions / lint-dotnet-format / run-dotnet-format-stable
Check warning on line 118 in src/OpenTelemetry/Metrics/View/MetricStreamConfiguration.cs GitHub Actions / lint-dotnet-format / run-dotnet-format-stable
Check warning on line 118 in src/OpenTelemetry/Metrics/View/MetricStreamConfiguration.cs GitHub Actions / lint-dotnet-format / run-dotnet-format-stable
Check warning on line 118 in src/OpenTelemetry/Metrics/View/MetricStreamConfiguration.cs GitHub Actions / lint-dotnet-format / run-dotnet-format-stable
|
||
set | ||
Check failure on line 119 in src/OpenTelemetry/Metrics/View/MetricStreamConfiguration.cs GitHub Actions / build-test-project-stable / build-test (ubuntu-latest, net9.0)
Check failure on line 119 in src/OpenTelemetry/Metrics/View/MetricStreamConfiguration.cs GitHub Actions / build-test-project-stable / build-test (ubuntu-latest, net9.0)
Check failure on line 119 in src/OpenTelemetry/Metrics/View/MetricStreamConfiguration.cs GitHub Actions / build-test-project-stable / build-test (ubuntu-latest, net9.0)
Check failure on line 119 in src/OpenTelemetry/Metrics/View/MetricStreamConfiguration.cs GitHub Actions / build-test-project-stable / build-test (ubuntu-latest, net9.0)
Check failure on line 119 in src/OpenTelemetry/Metrics/View/MetricStreamConfiguration.cs GitHub Actions / build-test-project-stable / build-test (ubuntu-latest, net9.0)
Check failure on line 119 in src/OpenTelemetry/Metrics/View/MetricStreamConfiguration.cs GitHub Actions / build-test-project-stable / build-test (ubuntu-latest, net8.0)
Check failure on line 119 in src/OpenTelemetry/Metrics/View/MetricStreamConfiguration.cs GitHub Actions / build-test-project-stable / build-test (ubuntu-latest, net8.0)
Check failure on line 119 in src/OpenTelemetry/Metrics/View/MetricStreamConfiguration.cs GitHub Actions / build-test-project-stable / build-test (ubuntu-latest, net8.0)
Check failure on line 119 in src/OpenTelemetry/Metrics/View/MetricStreamConfiguration.cs GitHub Actions / build-test-project-stable / build-test (ubuntu-latest, net8.0)
Check failure on line 119 in src/OpenTelemetry/Metrics/View/MetricStreamConfiguration.cs GitHub Actions / build-test-project-stable / build-test (ubuntu-latest, net8.0)
Check failure on line 119 in src/OpenTelemetry/Metrics/View/MetricStreamConfiguration.cs GitHub Actions / build-test-project-stable / build-test (otel-linux-arm64, net9.0)
Check failure on line 119 in src/OpenTelemetry/Metrics/View/MetricStreamConfiguration.cs GitHub Actions / build-test-project-stable / build-test (otel-linux-arm64, net9.0)
Check failure on line 119 in src/OpenTelemetry/Metrics/View/MetricStreamConfiguration.cs GitHub Actions / build-test-project-stable / build-test (otel-linux-arm64, net9.0)
Check failure on line 119 in src/OpenTelemetry/Metrics/View/MetricStreamConfiguration.cs GitHub Actions / build-test-project-stable / build-test (otel-linux-arm64, net9.0)
Check failure on line 119 in src/OpenTelemetry/Metrics/View/MetricStreamConfiguration.cs GitHub Actions / build-test-project-stable / build-test (otel-linux-arm64, net9.0)
Check failure on line 119 in src/OpenTelemetry/Metrics/View/MetricStreamConfiguration.cs GitHub Actions / validate-packages / run-package-validation-stable
Check failure on line 119 in src/OpenTelemetry/Metrics/View/MetricStreamConfiguration.cs GitHub Actions / validate-packages / run-package-validation-stable
Check failure on line 119 in src/OpenTelemetry/Metrics/View/MetricStreamConfiguration.cs GitHub Actions / validate-packages / run-package-validation-stable
Check failure on line 119 in src/OpenTelemetry/Metrics/View/MetricStreamConfiguration.cs GitHub Actions / validate-packages / run-package-validation-stable
Check failure on line 119 in src/OpenTelemetry/Metrics/View/MetricStreamConfiguration.cs GitHub Actions / validate-packages / run-package-validation-stable
Check failure on line 119 in src/OpenTelemetry/Metrics/View/MetricStreamConfiguration.cs GitHub Actions / build-test-project-stable / build-test (windows-latest, net8.0)
Check failure on line 119 in src/OpenTelemetry/Metrics/View/MetricStreamConfiguration.cs GitHub Actions / build-test-project-stable / build-test (windows-latest, net8.0)
Check failure on line 119 in src/OpenTelemetry/Metrics/View/MetricStreamConfiguration.cs GitHub Actions / build-test-project-stable / build-test (windows-latest, net8.0)
Check failure on line 119 in src/OpenTelemetry/Metrics/View/MetricStreamConfiguration.cs GitHub Actions / build-test-project-stable / build-test (windows-latest, net8.0)
Check failure on line 119 in src/OpenTelemetry/Metrics/View/MetricStreamConfiguration.cs GitHub Actions / build-test-project-stable / build-test (windows-latest, net8.0)
Check failure on line 119 in src/OpenTelemetry/Metrics/View/MetricStreamConfiguration.cs GitHub Actions / build-test-project-stable / build-test (windows-latest, net462)
Check failure on line 119 in src/OpenTelemetry/Metrics/View/MetricStreamConfiguration.cs GitHub Actions / build-test-project-stable / build-test (windows-latest, net462)
Check failure on line 119 in src/OpenTelemetry/Metrics/View/MetricStreamConfiguration.cs GitHub Actions / build-test-project-stable / build-test (windows-latest, net462)
Check failure on line 119 in src/OpenTelemetry/Metrics/View/MetricStreamConfiguration.cs GitHub Actions / build-test-project-stable / build-test (windows-latest, net462)
Check failure on line 119 in src/OpenTelemetry/Metrics/View/MetricStreamConfiguration.cs GitHub Actions / build-test-project-stable / build-test (windows-latest, net462)
Check failure on line 119 in src/OpenTelemetry/Metrics/View/MetricStreamConfiguration.cs GitHub Actions / build-test-project-stable / build-test (windows-latest, net9.0)
Check failure on line 119 in src/OpenTelemetry/Metrics/View/MetricStreamConfiguration.cs GitHub Actions / build-test-project-stable / build-test (windows-latest, net9.0)
Check failure on line 119 in src/OpenTelemetry/Metrics/View/MetricStreamConfiguration.cs GitHub Actions / build-test-project-stable / build-test (windows-latest, net9.0)
Check failure on line 119 in src/OpenTelemetry/Metrics/View/MetricStreamConfiguration.cs GitHub Actions / build-test-project-stable / build-test (windows-latest, net9.0)
Check failure on line 119 in src/OpenTelemetry/Metrics/View/MetricStreamConfiguration.cs GitHub Actions / build-test-project-stable / build-test (windows-latest, net9.0)
Check warning on line 119 in src/OpenTelemetry/Metrics/View/MetricStreamConfiguration.cs GitHub Actions / lint-dotnet-format / run-dotnet-format-stable
Check warning on line 119 in src/OpenTelemetry/Metrics/View/MetricStreamConfiguration.cs GitHub Actions / lint-dotnet-format / run-dotnet-format-stable
Check warning on line 119 in src/OpenTelemetry/Metrics/View/MetricStreamConfiguration.cs GitHub Actions / lint-dotnet-format / run-dotnet-format-stable
Check warning on line 119 in src/OpenTelemetry/Metrics/View/MetricStreamConfiguration.cs GitHub Actions / lint-dotnet-format / run-dotnet-format-stable
Check warning on line 119 in src/OpenTelemetry/Metrics/View/MetricStreamConfiguration.cs GitHub Actions / lint-dotnet-format / run-dotnet-format-stable
|
||
{ | ||
if (value != null) | ||
{ | ||
|
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.
Does this need to be moved?
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.
I was simply following the convention of the OTEL1002 above which was released and moved from "Active" to "Inactive". Personally I think it'd also be helpful to have a record of what happened to previous experimental features.