Skip to content

Commit

Permalink
[8.3](backport #32281) [metricbeat] gcp: check for nil metadata (#32400)
Browse files Browse the repository at this point in the history
* [metricbeat] gcp: check for nil metadata (#32281)

* add check for nil metadata

* add changelog entry

(cherry picked from commit fa8d17f)

* Update CHANGELOG.next.asciidoc

* Update CHANGELOG.next.asciidoc

Co-authored-by: gpop63 <[email protected]>
  • Loading branch information
mergify[bot] and gpop63 authored Aug 25, 2022
1 parent 6b4566e commit 4678fc6
Show file tree
Hide file tree
Showing 2 changed files with 10 additions and 8 deletions.
2 changes: 1 addition & 1 deletion CHANGELOG.next.asciidoc
Original file line number Diff line number Diff line change
Expand Up @@ -52,7 +52,7 @@ https://github.com/elastic/beats/compare/v8.2.0\...main[Check the HEAD diff]

*Metricbeat*

- Fix to ARN parsing for Cloudwatch resource names with leading slashes {pull}32446[32446]
- Check for nil metadata in GCP. {pull}32281[32281]

*Packetbeat*

Expand Down
16 changes: 9 additions & 7 deletions x-pack/metricbeat/module/gcp/metrics/metricset.go
Original file line number Diff line number Diff line change
Expand Up @@ -322,14 +322,16 @@ func (m *MetricSet) getMetadata(out *metric.MetricDescriptor, metricsWithMeta ma
ingestDelay: 0 * time.Second,
}

if out.Metadata.SamplePeriod != nil {
m.Logger().Debugf("For metric type %s: sample period = %s", out.Type, out.Metadata.SamplePeriod)
meta.samplePeriod = time.Duration(out.Metadata.SamplePeriod.Seconds) * time.Second
}
if out.Metadata != nil {
if out.Metadata.SamplePeriod != nil {
m.Logger().Debugf("For metric type %s: sample period = %s", out.Type, out.Metadata.SamplePeriod)
meta.samplePeriod = time.Duration(out.Metadata.SamplePeriod.Seconds) * time.Second
}

if out.Metadata.IngestDelay != nil {
m.Logger().Debugf("For metric type %s: ingest delay = %s", out.Type, out.Metadata.IngestDelay)
meta.ingestDelay = time.Duration(out.Metadata.IngestDelay.Seconds) * time.Second
if out.Metadata.IngestDelay != nil {
m.Logger().Debugf("For metric type %s: ingest delay = %s", out.Type, out.Metadata.IngestDelay)
meta.ingestDelay = time.Duration(out.Metadata.IngestDelay.Seconds) * time.Second
}
}

metricsWithMeta[out.Type] = meta
Expand Down

0 comments on commit 4678fc6

Please sign in to comment.