Skip to content

[azure_gateway_otel] Azure Application Gateway Logs OpenTelemetry Assets - #19010

Merged
mykola-elastic merged 12 commits into
elastic:mainfrom
mykola-elastic:azure_gateway_logs
Jun 23, 2026
Merged

[azure_gateway_otel] Azure Application Gateway Logs OpenTelemetry Assets#19010
mykola-elastic merged 12 commits into
elastic:mainfrom
mykola-elastic:azure_gateway_logs

Conversation

@mykola-elastic

@mykola-elastic mykola-elastic commented May 14, 2026

Copy link
Copy Markdown
Contributor

Proposed commit message

Checklist

  • I have reviewed tips for building integrations and this pull request is aligned with them.
  • I have verified that all data streams collect metrics or logs.
  • I have added an entry to my package's changelog.yml file.
  • I have verified that Kibana version constraints are current according to guidelines.
  • I have verified that any added dashboard complies with Kibana's Dashboard good practices

How to test this PR locally

Related issues

Screenshots

Overview

kb-overview-new

WAF

kb-waf-new

Backend health

kb-backend-new
@mykola-elastic mykola-elastic self-assigned this May 14, 2026
@mykola-elastic mykola-elastic added documentation Improvements or additions to documentation. Applied to PRs that modify *.md files. dashboard Relates to a Kibana dashboard bug, enhancement, or modification. New Integration Issue or pull request for creating a new integration package. Team:Obs-InfraObs Observability Infrastructure Monitoring team [elastic/obs-infraobs-integrations] Integration:azure_gateway_otel Azure Application Gateway OpenTelemetry Assets labels May 14, 2026
@github-actions

github-actions Bot commented May 14, 2026

Copy link
Copy Markdown
Contributor

Vale Linting Results

Summary: 1 warning, 4 suggestions found

⚠️ Warnings (1)
File Line Rule Message
packages/azure_gateway_otel/docs/README.md 27 Elastic.DirectionalLanguage Don't use directional language. Use 'in the preceding section' instead of 'listed above'.
💡 Suggestions (4)
File Line Rule Message
packages/azure_gateway_otel/docs/README.md 11 Elastic.WordChoice Consider using 'efficiently' instead of 'simply', unless the term is in the UI.
packages/azure_gateway_otel/docs/README.md 21 Elastic.Semicolons Use semicolons judiciously.
packages/azure_gateway_otel/docs/README.md 23 Elastic.Semicolons Use semicolons judiciously.
packages/azure_gateway_otel/docs/README.md 35 Elastic.Semicolons Use semicolons judiciously.

The Vale linter checks documentation changes against the Elastic Docs style guide.

To use Vale locally or report issues, refer to Elastic style guide for Vale.

@andrewkroh andrewkroh removed the dashboard Relates to a Kibana dashboard bug, enhancement, or modification. label May 14, 2026
@andrewkroh andrewkroh removed the Integration:azure_gateway_otel Azure Application Gateway OpenTelemetry Assets label May 15, 2026
@github-actions

This comment has been minimized.

@github-actions

Copy link
Copy Markdown
Contributor

TL;DR

Buildkite failed in Check go sources because a new package (packages/azure_gateway_otel) is missing a matching owner entry in .github/CODEOWNERS. Add a CODEOWNERS rule for this package and rerun CI.

Remediation

  • Add a package-level owner mapping in .github/CODEOWNERS, e.g. /packages/azure_gateway_otel @elastic/obs-infraobs-integrations`` (consistent with manifest.yml owner and nearby Azure OTel package ownership).
  • Re-run .buildkite/scripts/check_sources.sh (or full Buildkite) to confirm github.com/elastic/integrations/dev/codeowners.Check passes.
Investigation details

Root Cause

dev/codeowners.Check validates that each package has an explicit owner entry in .github/CODEOWNERS. The new package packages/azure_gateway_otel exists in the failing commit (be75362daf8d0525904ea1499167e3be16070fa9), but there is no corresponding /packages/azure_gateway_otel ... rule in CODEOWNERS.

Relevant evidence in repo state:

  • .github/CODEOWNERS:116 has /packages/azure_activity_otel ... and many Azure package entries, but no /packages/azure_gateway_otel entry in that section.
  • packages/azure_gateway_otel/manifest.yml exists at the failing commit and declares:
    • owner.github: elastic/obs-infraobs-integrations

Evidence

Running dependency: github.com/elastic/integrations/dev/codeowners.Check
Error: error validating packages in directory 'packages': error checking manifest 'packages/azure_gateway_otel': there is no owner for "packages/azure_gateway_otel" in ".github/CODEOWNERS"

Verification

  • Not run locally (workspace is not checked out to PR head commit); analysis is based on Buildkite logs + repository content at commit be75362daf8d0525904ea1499167e3be16070fa9.

Follow-up

After adding the CODEOWNERS entry, if the check still fails, verify there are no trailing spaces or pattern typos in the new CODEOWNERS line.

Note

🔒 Integrity filter blocked 5 items

The following items were blocked because they don't meet the GitHub integrity level.

  • #19010 pull_request_read: has lower integrity than agent requires. The agent cannot read data with integrity below "approved".
  • [azure_gateway_otel] Azure Application Gateway Logs OpenTelemetry Assets #19010 pull_request_read: has lower integrity than agent requires. The agent cannot read data with integrity below "approved".
  • #19010 search_pull_requests: has lower integrity than agent requires. The agent cannot read data with integrity below "approved".
  • #19011 search_pull_requests: has lower integrity than agent requires. The agent cannot read data with integrity below "approved".
  • #11933 search_pull_requests: has lower integrity than agent requires. The agent cannot read data with integrity below "approved".

To allow these resources, lower min-integrity in your GitHub frontmatter:

tools:
  github:
    min-integrity: approved  # merged | approved | unapproved | none

What is this? | From workflow: PR Buildkite Detective

Give us feedback! React with 🚀 if perfect, 👍 if helpful, 👎 if not.

@mykola-elastic
mykola-elastic marked this pull request as ready for review May 15, 2026 16:16
@mykola-elastic
mykola-elastic requested a review from a team as a code owner May 15, 2026 16:16
@mykola-elastic

Copy link
Copy Markdown
Contributor Author

/test

@mykola-elastic mykola-elastic changed the title [azure_gateway_otel] [kb-yaml-to-lens] Azure Application Gateway Logs OpenTelemetry Assets - kb-yaml-to-lens May 18, 2026
@mykola-elastic mykola-elastic changed the title [azure_gateway_otel] Azure Application Gateway Logs OpenTelemetry Assets - kb-yaml-to-lens May 18, 2026
@narph
narph requested a review from a team May 19, 2026 12:28
@muthu-mps
muthu-mps requested a review from Copilot May 21, 2026 06:09

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Introduces a new Elastic content package (azure_gateway_otel) that ships Kibana dashboards and supporting assets for analyzing Azure Application Gateway logs encoded as OpenTelemetry (EDOT / azureencodingextension), targeting the logs-azure.application_gateway.otel-* data stream.

Changes:

  • Adds the new azure_gateway_otel package with manifest, license, changelog, docs, images, and logo assets.
  • Ships three Kibana dashboards (Overview, WAF, Backend Health) as saved-object JSON.
  • Adds _dev/shared YAML dashboard sources for authoring/maintenance and updates CODEOWNERS for the new package.

Reviewed changes

Copilot reviewed 11 out of 15 changed files in this pull request and generated 7 comments.

Show a summary per file
File Description
packages/azure_gateway_otel/manifest.yml Declares the new content package, version constraints, discovery dataset, screenshots/icons, and owner.
packages/azure_gateway_otel/LICENSE.txt Adds Elastic License 2.0 for the new package.
packages/azure_gateway_otel/changelog.yml Adds initial changelog entry for v0.1.0.
packages/azure_gateway_otel/docs/README.md Documents prerequisites and what the dashboards expect/cover.
packages/azure_gateway_otel/kibana/dashboard/azure_gateway_otel-overview.json Overview dashboard saved object (ES
packages/azure_gateway_otel/kibana/dashboard/azure_gateway_otel-waf.json WAF dashboard saved object (ES
packages/azure_gateway_otel/kibana/dashboard/azure_gateway_otel-backend-health.json Backend Health dashboard saved object (ES
packages/azure_gateway_otel/_dev/shared/azure_gateway_otel-overview.yaml Dashboard-as-source YAML for Overview (used to generate/maintain JSON).
packages/azure_gateway_otel/_dev/shared/azure_gateway_otel-waf.yaml Dashboard-as-source YAML for WAF.
packages/azure_gateway_otel/_dev/shared/azure_gateway_otel-backend-health.yaml Dashboard-as-source YAML for Backend Health.
packages/azure_gateway_otel/img/* Adds screenshots and the package logo referenced by the manifest.
.github/CODEOWNERS Adds ownership for /packages/azure_gateway_otel.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@zmoog zmoog left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM! Just left a couple of nitpicks, feel free to merge whenever.

Comment thread packages/azure_gateway_otel/docs/README.md Outdated
Comment thread packages/azure_gateway_otel/docs/README.md Outdated
Comment thread packages/azure_gateway_otel/docs/README.md Outdated
Comment thread packages/azure_gateway_otel/manifest.yml Outdated
@elasticmachine

Copy link
Copy Markdown

💚 Build Succeeded

History

cc @mykola-elastic

@muthu-mps

Copy link
Copy Markdown
Contributor

Are we planning to add Assets(SLOs and Alerts) in a separate PR?

@muthu-mps muthu-mps left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Left a comment on assets, Change looks good to me otherwise!

@mykola-elastic

Copy link
Copy Markdown
Contributor Author

Thanks for the review

Are we planning to add Assets(SLOs and Alerts) in a separate PR?

If needed yes, it can be planned. The scope for this one was dashboards

@mykola-elastic

Copy link
Copy Markdown
Contributor Author

@lalit-satapathy ok to merge? needs your approval

@bturquet

Copy link
Copy Markdown

@lalit-satapathy could we have someone from your team giving the green light and merging the PR, as it seems ready ?

@mykola-elastic
mykola-elastic enabled auto-merge (squash) June 23, 2026 13:18

@lalit-satapathy lalit-satapathy left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

code owner approval

@mykola-elastic
mykola-elastic merged commit 11ad485 into elastic:main Jun 23, 2026
12 checks passed
@elastic-vault-github-plugin-prod

Copy link
Copy Markdown
Contributor

Package azure_gateway_otel - 0.1.0 containing this change is available at https://epr.elastic.co/package/azure_gateway_otel/0.1.0/

@andrewkroh andrewkroh added Integration:azure_gateway_otel Azure Application Gateway OpenTelemetry Assets dashboard Relates to a Kibana dashboard bug, enhancement, or modification. labels Jun 23, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dashboard Relates to a Kibana dashboard bug, enhancement, or modification. documentation Improvements or additions to documentation. Applied to PRs that modify *.md files. Integration:azure_gateway_otel Azure Application Gateway OpenTelemetry Assets New Integration Issue or pull request for creating a new integration package. Team:Obs-InfraObs Observability Infrastructure Monitoring team [elastic/obs-infraobs-integrations]

8 participants