Skip to content

Add instance_name metric label to stackdriver monitoring labels#66

Open
cristynkells wants to merge 2 commits into
GoogleCloudPlatform:masterfrom
cristynkells:patch-1
Open

Add instance_name metric label to stackdriver monitoring labels#66
cristynkells wants to merge 2 commits into
GoogleCloudPlatform:masterfrom
cristynkells:patch-1

Conversation

@cristynkells
Copy link
Copy Markdown

Enhance Cloud Monitoring metrics emitted by the MLLP adapter by adding a new
instance_name label derived from GCE metadata.

This change preserves backward compatibility by retaining the existing
instance label (instance ID) and introduces an additional human-readable
identifier for improved observability in Metrics Explorer and dashboards.

No existing labels are modified or removed.

Changes:

  • Add metadata.InstanceName() retrieval during metric exporter startup
  • Introduce optional instance_name label to DefaultMonitoringLabels
  • Fail-safe behavior: absence of instance name does not impact metric export
  • Added instance_name label with instance_name1 value to monitoring_test

Impact:

  • Improves filtering and grouping in Cloud Monitoring
  • Enables easier debugging and per-VM identification
  • Fully backward compatible with existing dashboards and alerting rules

Enhance Cloud Monitoring metrics emitted by the MLLP adapter by adding a new
`instance_name` label derived from GCE metadata.

This change preserves backward compatibility by retaining the existing
`instance` label (instance ID) and introduces an additional human-readable
identifier for improved observability in Metrics Explorer and dashboards.

No existing labels are modified or removed.

Changes:
- Add metadata.InstanceName() retrieval during metric exporter startup
- Introduce optional `instance_name` label to DefaultMonitoringLabels
- Fail-safe behavior: absence of instance name does not impact metric export

Impact:
- Improves filtering and grouping in Cloud Monitoring
- Enables easier debugging and per-VM identification
- Fully backward compatible with existing dashboards and alerting rules
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant