Skip to content

Conversation

colleenmcginnis
Copy link
Contributor

@colleenmcginnis colleenmcginnis commented Sep 10, 2025

Contributes to https://github.com/elastic/docs-content-internal/issues/254

Adds missing page-level applies_to to generated exported fields pages across beats. This required updating the automation in libbeat/scripts/generate_fields_docs.py, running the automation, and committing the changes that result.

For the reviewers

  • @elastic/ingest-docs can someone spot check the Markdown files in this PR?
  • @elastic/observablt-ci @elastic/elastic-agent-data-plane you're only the code owner of one file: libbeat/scripts/generate_fields_docs.py

@colleenmcginnis colleenmcginnis self-assigned this Sep 10, 2025
@colleenmcginnis colleenmcginnis added docs Team:Docs Label for the Observability docs team backport-active-9 Automated backport with mergify to all the active 9.[0-9]+ branches labels Sep 10, 2025
@botelastic botelastic bot added needs_team Indicates that the issue/PR needs a Team:* label and removed needs_team Indicates that the issue/PR needs a Team:* label labels Sep 10, 2025
Copy link
Contributor

🤖 GitHub comments

Expand to view the GitHub comments

Just comment with:

  • run docs-build : Re-trigger the docs validation. (use unformatted text in the comment!)

Copy link
Contributor

github-actions bot commented Sep 10, 2025

🔍 Preview links for changed docs

More links …

In total, 199 files changed.

@colleenmcginnis colleenmcginnis marked this pull request as ready for review September 17, 2025 18:57
@colleenmcginnis colleenmcginnis requested review from a team as code owners September 17, 2025 18:57
Copy link
Contributor

@vishaangelova vishaangelova left a comment

Choose a reason for hiding this comment

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

Docs LGTM!

@colleenmcginnis
Copy link
Contributor Author

👋 @elastic/observablt-ci can someone please take a look?

Copy link
Member

@v1v v1v left a comment

Choose a reason for hiding this comment

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

No idea why we own that file, but so far I cannot say much, I only approve to unblock this, but Beats team should be involved in reviewing the changes

@colleenmcginnis colleenmcginnis requested a review from a team September 24, 2025 15:55
@colleenmcginnis colleenmcginnis requested review from rdner and mauri870 and removed request for a team September 24, 2025 15:55
@colleenmcginnis colleenmcginnis requested a review from a team as a code owner September 29, 2025 20:44
@colleenmcginnis
Copy link
Contributor Author

It looks like buildkite/beats is also failing on main. 🤔

@colleenmcginnis colleenmcginnis enabled auto-merge (squash) September 30, 2025 14:59
@colleenmcginnis
Copy link
Contributor Author

@AndersonQ @VihasMakwana can you help me with the failing checks?

@AndersonQ
Copy link
Member

it's a new flaky test (#46904) failing. Let's retry it for now.

@AndersonQ
Copy link
Member

at least locally it passes most of the time. If it's not the case here, we might skip it

@colleenmcginnis colleenmcginnis merged commit eadc27e into elastic:main Oct 2, 2025
212 checks passed
Copy link
Contributor

github-actions bot commented Oct 2, 2025

@Mergifyio backport 9.0 9.1 9.2

Copy link
Contributor

mergify bot commented Oct 2, 2025

mergify bot pushed a commit that referenced this pull request Oct 2, 2025
…46527)

* update logic

* add missing page-level applies_to to auditbeat ga exported fields

* add missing page-level applies_to to filebeat ga exported fields

* add missing page-level applies_to to heartbeat ga exported fields

* add missing page-level applies_to to metricbeat ga exported fields

* add missing page-level applies_to to packetbeat ga exported fields

* add missing page-level applies_to to winlogbeat ga exported fields

* clean up

---------

Co-authored-by: Francisco Ramon <[email protected]>
(cherry picked from commit eadc27e)
mergify bot pushed a commit that referenced this pull request Oct 2, 2025
…46527)

* update logic

* add missing page-level applies_to to auditbeat ga exported fields

* add missing page-level applies_to to filebeat ga exported fields

* add missing page-level applies_to to heartbeat ga exported fields

* add missing page-level applies_to to metricbeat ga exported fields

* add missing page-level applies_to to packetbeat ga exported fields

* add missing page-level applies_to to winlogbeat ga exported fields

* clean up

---------

Co-authored-by: Francisco Ramon <[email protected]>
(cherry picked from commit eadc27e)
mergify bot pushed a commit that referenced this pull request Oct 2, 2025
…46527)

* update logic

* add missing page-level applies_to to auditbeat ga exported fields

* add missing page-level applies_to to filebeat ga exported fields

* add missing page-level applies_to to heartbeat ga exported fields

* add missing page-level applies_to to metricbeat ga exported fields

* add missing page-level applies_to to packetbeat ga exported fields

* add missing page-level applies_to to winlogbeat ga exported fields

* clean up

---------

Co-authored-by: Francisco Ramon <[email protected]>
(cherry picked from commit eadc27e)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport-active-9 Automated backport with mergify to all the active 9.[0-9]+ branches docs skip-changelog Team:Docs Label for the Observability docs team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants