[ci] another fix in the release changelog generation#1077
Conversation
![macroscopeapp[bot]](https://avatars.githubusercontent.com/in/900172?s=60&v=4){kind=small}
There was a problem hiding this comment.
Pull request overview
Updates the release workflow step that generates changelog/release notes, aligning docs-builder inputs/outputs with the repo’s release YAML layout and suppressing unwanted command output to avoid CI failures.
Changes:
- Switch
docs-builder changelog renderto usedocs/releases/<version>.yamlas input. - Suppress
docs-builderstdout during release-notes generation.
Comments suppressed due to low confidence (1)
.github/workflows/release-step-3.yml:240
- There are two consecutive
cat "${tmp}/${{ env.RELEASE_VERSION }}/index.md"lines, and the first one ends with a trailing\. With the continuation, the shell will effectively runcat <file> cat <file> | sed ..., which breaks the pipeline (and typically errors withcat: cat: No such file or directory). Remove the duplicatecatand keep a single pipeline intosed.
cat "${tmp}/${{ env.RELEASE_VERSION }}/index.md" \
| sed -e "/## ${{ env.RELEASE_VERSION }} \[elastic-release-notes-${{ env.RELEASE_VERSION }}/d" \
| sed -e "/###/s/ \[elastic-${{ env.RELEASE_VERSION }}.*$//g" >> $GITHUB_OUTPUT
echo 'RELNOTESEOF' >> $GITHUB_OUTPUT
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Enterprise Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughThe release workflow's changelog rendering step has been updated to source the YAML file from a version-specific path at 🚥 Pre-merge checks | ✅ 2✅ Passed checks (2 passed)✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
should fix https://github.com/elastic/elastic-otel-java/actions/runs/25006685693/job/73231276957
doc-builderinvocation should be removed