Skip to content

Commit f8ab0e7

Browse files
[receiver/vcenter] Bump receiver.vcenter.emitPerfMetricsWithObjects feature gate (#30616)
Updates #30615 --------- Co-authored-by: Evan Bradley <[email protected]>
1 parent f90f3b6 commit f8ab0e7

File tree

6 files changed

+1166
-1144
lines changed

6 files changed

+1166
-1144
lines changed

.chloggen/bump-vcenter-gate.yaml

Lines changed: 27 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,27 @@
1+
# Use this changelog template to create an entry for release notes.
2+
3+
# One of 'breaking', 'deprecation', 'new_component', 'enhancement', 'bug_fix'
4+
change_type: breaking
5+
6+
# The name of the component, or a single word describing the area of concern, (e.g. filelogreceiver)
7+
component: vcenterreceiver
8+
9+
# A brief description of the change. Surround your text with quotes ("") if it needs to start with a backtick (`).
10+
note: Bump "receiver.vcenter.emitPerfMetricsWithObjects" feature gate
11+
12+
# Mandatory: One or more tracking issues related to the change. You can use the PR number here if no issue exists.
13+
issues: [30615]
14+
15+
# (Optional) One or more lines of additional information to render under the primary note.
16+
# These lines will be padded with 2 spaces and then inserted directly into the document.
17+
# Use pipe (|) for multiline entries.
18+
subtext:
19+
20+
# If your change doesn't affect end users or the exported elements of any package,
21+
# you should instead start your pull request title with [chore] or use the "Skip Changelog" label.
22+
# Optional: The change log or logs in which this entry should be included.
23+
# e.g. '[user]' or '[user, api]'
24+
# Include 'user' if the change is relevant to end users.
25+
# Include 'api' if there is a change to a library API.
26+
# Default: '[user]'
27+
change_logs: []

receiver/vcenterreceiver/README.md

Lines changed: 2 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -78,17 +78,12 @@ These metrics were incorrectly dimensioned previously but the desired fix is a b
7878

7979
These metrics were improperly dimensioned and needed another metric attribute `object` in order to properly dimension the data.
8080

81-
1. v0.84.0, August 2023:
82-
83-
- The version of the receiver will keep default behavior.
84-
- `receiver.vcenter.emitPerfMetricsWithObjects` is *disabled* by default.
85-
86-
2. v0.86.0, September 2023:
81+
#### v0.93.0, January 2024
8782

8883
- The receiver will emit the `object` metric attribute by default
8984
- `receiver.vcenter.emitPerfMetricsWithObjects` is *enabled* by default.
9085

91-
3. v0.88.0, October 2023:
86+
#### v0.95.0, February 2024
9287

9388
- The feature gates are removed.
9489
- Performance metrics will always include the `object` metric attribute.

receiver/vcenterreceiver/scraper.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,7 @@ const (
2727

2828
var emitPerfMetricsWithObjects = featuregate.GlobalRegistry().MustRegister(
2929
emitPerfMetricsWithObjectsFeatureGateID,
30-
featuregate.StageAlpha,
30+
featuregate.StageBeta,
3131
featuregate.WithRegisterDescription("When enabled, the receiver emits vCenter performance metrics with object metric label dimension."),
3232
)
3333

receiver/vcenterreceiver/scraper_test.go

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,7 @@ func TestScrape(t *testing.T) {
3333
testScrape(ctx, t, cfg)
3434
}
3535

36-
func TestScrapeWithPerfObjects(t *testing.T) {
36+
func TestScrapeWithoutPerfObjects(t *testing.T) {
3737
ctx := context.Background()
3838
mockServer := mock.MockServer(t, false)
3939
defer mockServer.Close()
@@ -46,13 +46,13 @@ func TestScrapeWithPerfObjects(t *testing.T) {
4646
}
4747

4848
scraper := newVmwareVcenterScraper(zap.NewNop(), cfg, receivertest.NewNopCreateSettings())
49-
scraper.emitPerfWithObject = true
49+
scraper.emitPerfWithObject = false
5050

5151
metrics, err := scraper.scrape(ctx)
5252
require.NoError(t, err)
5353
require.NotEqual(t, metrics.MetricCount(), 0)
5454

55-
goldenPath := filepath.Join("testdata", "metrics", "expected_with_object.yaml")
55+
goldenPath := filepath.Join("testdata", "metrics", "expected_without_objects.yaml")
5656
expectedMetrics, err := golden.ReadMetrics(goldenPath)
5757
require.NoError(t, err)
5858

0 commit comments

Comments
 (0)