Skip to content

feat: Database_observability: support Azure cloud provider config data#5245

Merged
cristiangreco merged 2 commits intomainfrom
cristian/dbo11y-csp-azure-config
Jan 14, 2026
Merged

feat: Database_observability: support Azure cloud provider config data#5245
cristiangreco merged 2 commits intomainfrom
cristian/dbo11y-csp-azure-config

Conversation

@cristiangreco
Copy link
Contributor

@cristiangreco cristiangreco commented Jan 13, 2026

Brief description of Pull Request

Add configuration block database_observability.cloud_provider.azure to support Azure database instances in addition to existing AWS support. When data is provided in this block, the corresponding labels are populated.

Pull Request Details

Issue(s) fixed by this Pull Request

Notes to the Reviewer

PR Checklist

  • Documentation added
  • Tests updated
  • Config converters updated

@cristiangreco cristiangreco force-pushed the cristian/dbo11y-csp-azure-config branch 2 times, most recently from c506575 to 6aaa40b Compare January 14, 2026 09:49
@github-actions
Copy link
Contributor

github-actions bot commented Jan 14, 2026

💻 Deploy preview deleted (feat: Database_observability: support Azure cloud provider config data).

@cristiangreco cristiangreco force-pushed the cristian/dbo11y-csp-azure-config branch 5 times, most recently from c87a267 to e6e4429 Compare January 14, 2026 10:21
@cristiangreco cristiangreco marked this pull request as ready for review January 14, 2026 10:24
@cristiangreco cristiangreco requested review from a team and clayton-cornell as code owners January 14, 2026 10:24

type AzureCloudProviderInfo struct {
Resource string
Resource string
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we call this ServerName to match what is in config?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I see we have this elsewhere

ServerName string `alloy:"server_name,attr,optional"`

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done

Add configuration block `database_observability.cloud_provider.azure` to
support Azure database instances in addition to existing AWS support.
When data is provided in this block, the corresponding labels are
populated.
@cristiangreco cristiangreco force-pushed the cristian/dbo11y-csp-azure-config branch from e6e4429 to e87bee5 Compare January 14, 2026 13:24
@cristiangreco cristiangreco merged commit d7a469f into main Jan 14, 2026
48 checks passed
@cristiangreco cristiangreco deleted the cristian/dbo11y-csp-azure-config branch January 14, 2026 16:41
@grafana-alloybot grafana-alloybot bot mentioned this pull request Jan 14, 2026
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jan 29, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants