We would like to evolve to a new changelog management mechanism for this repository. The goals are to:
In the proposed approach the source of truth for changelog entries will be the original PRs used to introduce the change in the platform. The same entry will be used for all eventual backports.
The changelog entry will be in a section of the description of the PR. Engineers will be responsible for completing this section with an appropriate description and language for the target audience. Tech writers will provide guidelines on how to write good entries and provide reviews/edits but the goal is that the need for edits by the Tech Writers decrease over time as the engineers get used to providing better descriptions. The PR template will be updated to account for this section.
The proposed PR description section for the change log is as follows (the end of the changelog section is delineated by either the next occurrence of a level 2 header or the end of the text itself)
## Changelog
### User Changelog
### Developer Changelog
### Agent Changelog
##) Changelog section marks the start of the section. Some labels to annotate the entry or alter the processing include:
release-note:skip label makes the PR be ignored when processing the PRs.release-note:highlight label marks the content to be included as a release highlight.release-note:breaking label marks the content as a breaking change.release-note:deprecation label marks the content to be included in the deprecation section.release-note:fix label marks the content to be included as a fix.In the user changelog, content should be organized in paragraphs, each one starting with the text [All], [Filebeat], [Metricbeat], etc. This allows a single PR to provide multiple entries for multiple products.
Pinging @elastic/integrations (Team:Integrations)
Pinging @elastic/ingest-management (Team:Ingest Management)
Pinging @elastic/siem (Team:SIEM)
Will there be some protection to block the PR from being merged if there is no CHANGELOG content and release-note:skip is not set? IIRC the Kibana team has something like this for their PRs.
Just curious why Agent Changelog needs a separate section as opposed to using [Agent] like for other products?
Agent documentation may have (now or in the future) a different structure than the current beats, so we are providing separate treatment from the beginning.
Yes, we will need to add some protections.
@ycombinator we also plan at some point to move out of the beats repository.
Most helpful comment
Will there be some protection to block the PR from being merged if there is no CHANGELOG content and
release-note:skipis not set? IIRC the Kibana team has something like this for their PRs.