Drop processor.transform.ConvertBetweenSumAndGaugeMetricContext feature gate #34567
Comments
|
Pinging code owners:
See Adding Labels via Comments if you do not have permissions to add labels yourself. |
|
Thanks for reminding me about this gate. We should continue forward through the standard feature gate process: https://github.com/open-telemetry/opentelemetry-collector/tree/main/featuregate#feature-lifecycle |
…34580) **Description:** promote the `processor.transform.ConvertBetweenSumAndGaugeMetricContext` feature flag from `alpha` to `beta`. **Link to tracking Issue:** #34567 **Testing:** I modified the unit test to check when the feature flag is enabled and disabled. --------- Signed-off-by: Israel Blancas <iblancasa@gmail.com> Co-authored-by: Tyler Helmuth <12352919+TylerHelmuth@users.noreply.github.com>
|
hello, using 0.108 collectors, i can no longer use the is this expected? |
|
That's because the feature gate "processor.transform.ConvertBetweenSumAndGaugeMetricContext" was switched from Alpha to Beta. In Alpha stage, the gate is disabled by default but in Beta stage it is enabled by default. |
|
else you need to change the context. It's not on the "datapoint" context anymore but on the "metric" context that these functions should be used. |
…o_sum (#35022) **Description:** The feature gate "processor.transform.ConvertBetweenSumAndGaugeMetricContext" was updated from Alpha to Beta. This means that it is now enabled by default. The example in the readme should match with the default behavior which is that the functions convert_sum_to_gauge and convert_gauge_to_sum must be used on the "metric" context **Link to tracking Issue:** #34567 Maybe the Readme should state clearly in which context the functions can be used?
…pen-telemetry#34580) **Description:** promote the `processor.transform.ConvertBetweenSumAndGaugeMetricContext` feature flag from `alpha` to `beta`. **Link to tracking Issue:** open-telemetry#34567 **Testing:** I modified the unit test to check when the feature flag is enabled and disabled. --------- Signed-off-by: Israel Blancas <iblancasa@gmail.com> Co-authored-by: Tyler Helmuth <12352919+TylerHelmuth@users.noreply.github.com>
…o_sum (open-telemetry#35022) **Description:** The feature gate "processor.transform.ConvertBetweenSumAndGaugeMetricContext" was updated from Alpha to Beta. This means that it is now enabled by default. The example in the readme should match with the default behavior which is that the functions convert_sum_to_gauge and convert_gauge_to_sum must be used on the "metric" context **Link to tracking Issue:** open-telemetry#34567 Maybe the Readme should state clearly in which context the functions can be used?
|
This issue has been inactive for 60 days. It will be closed in 60 days if there is no activity. To ping code owners by adding a component label, see Adding Labels via Comments, or if you are unsure of which component this issue relates to, please ping Pinging code owners:
See Adding Labels via Comments if you do not have permissions to add labels yourself. |
…ntext` as stable (open-telemetry#36216) #### Description Set the `processor.transform.ConvertBetweenSumAndGaugeMetricContext` feature gate as stable #### Link to tracking issue Fixes open-telemetry#34567 --------- Co-authored-by: Evan Bradley <11745660+evan-bradley@users.noreply.github.com>
…ntext` as stable (open-telemetry#36216) #### Description Set the `processor.transform.ConvertBetweenSumAndGaugeMetricContext` feature gate as stable #### Link to tracking issue Fixes open-telemetry#34567 --------- Co-authored-by: Evan Bradley <11745660+evan-bradley@users.noreply.github.com>
Component(s)
processor/transform
Describe the issue you're reporting
There's an alpha feature gate called
processor.transform.ConvertBetweenSumAndGaugeMetricContext. But the feature seems straightforward and works well, as far as I can tell. Has it been available long enough to drop the feature gate?The text was updated successfully, but these errors were encountered: