Skip to content

Commit

Permalink
x-pack/metricbeat/module/gcp: Fix missing ECS labels when using exclu…
Browse files Browse the repository at this point in the history
…de_labels: true (#40467)

* Issue fix

* Change.log

* resolved comments

* fix
  • Loading branch information
Linu-Elias authored Aug 23, 2024
1 parent e4dec60 commit 973af49
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 1 deletion.
1 change: 1 addition & 0 deletions CHANGELOG.next.asciidoc
Original file line number Diff line number Diff line change
Expand Up @@ -182,6 +182,7 @@ https://github.com/elastic/beats/compare/v8.8.1\...main[Check the HEAD diff]
- Update beat module with apm-server monitoring metrics fields {pull}40127[40127]
- Fix Azure Monitor metric timespan to restore Storage Account PT1H metrics {issue}40376[40376] {pull}40367[40367]
- Remove excessive info-level logs in cgroups setup {pull}40491[40491]
- Add missing ECS Cloud fields in GCP `metrics` metricset when using `exclude_labels: true` {issue}40437[40437] {pull}40467[40467]

*Osquerybeat*

Expand Down
2 changes: 1 addition & 1 deletion x-pack/metricbeat/module/gcp/metrics/metricset.go
Original file line number Diff line number Diff line change
Expand Up @@ -215,7 +215,7 @@ func (m *MetricSet) Fetch(ctx context.Context, reporter mb.ReporterV2) (err erro
func (m *MetricSet) mapToEvents(ctx context.Context, timeSeries []timeSeriesWithAligner, sdc metricsConfig) ([]mb.Event, error) {
mapper := newIncomingFieldMapper(m.Logger(), sdc)

var metadataService = gcp.NewStackdriverMetadataServiceForTimeSeries(nil)
var metadataService gcp.MetadataService
var err error

if !m.config.ExcludeLabels {
Expand Down

0 comments on commit 973af49

Please sign in to comment.