workflows: Make not to be the latest release for 4.0 branch #10993
Conversation
cosmo0920
requested review from
celalettin1286,
niedbalski and
patrick-stephens
as code owners
WalkthroughStaging-release workflow was split into two release paths: versions starting with "4.1" follow a new "4.1 and latest" path (make_latest: true); "4.0" releases follow a "4.0 - not latest" path (make_latest: false). Docs-related release steps were similarly adjusted. Changes
Sequence Diagram(s)sequenceDiagram
autonumber
actor Dev as Developer
participant GH as GitHub Actions (staging-release)
participant Job as Release Job
participant Rel as create-release Action
Dev->>GH: Dispatch workflow with inputs.version
GH->>Job: Start release job
alt inputs.version startsWith "4.1"
Note over Job,Rel: 4.1 path — make_latest: true
Job->>Rel: Create release (tag: 4.1.x, make_latest: true)
Rel-->>Job: Release created
else
Note over Job,Rel: 4.0 path — make_latest: false
Job->>Rel: Create release (tag: 4.0.x, make_latest: false)
Rel-->>Job: Release created
end
Job-->>GH: Job complete
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Possibly related PRs
Suggested labels
Suggested reviewers
Poem
Pre-merge checks and finishing touches✅ Passed checks (3 passed)
✨ Finishing touches🧪 Generate unit tests (beta)
📜 Recent review detailsConfiguration used: CodeRabbit UI Review profile: CHILL Plan: Pro 📒 Files selected for processing (1)
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 |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 0
Caution
Some comments are outside the diff and can’t be posted inline due to platform limitations.
⚠️ Outside diff range comments (2)
.github/workflows/staging-release.yaml (2)
504-507: Bug: “latest” container tags still update for 4.0.x; restrict to 4.1 onlyCondition matches any 4.x, so 4.0 releases will still publish latest tags. Update condition and comment.
Apply:
- staging-release-images-latest-tags: - # Only update latest tags for 4.0 releases - if: startsWith(github.event.inputs.version, '4.') + staging-release-images-latest-tags: + # Only update latest tags for 4.1 releases + if: startsWith(github.event.inputs.version, '4.1')
964-970: Handle docs checkout for 4.1 and explicit refs
- Rename the 4.0 step to “Release 4.0 – not latest” and add
ref: 4.0.- Add a new “Release 4.1 and latest” step under
startsWith(inputs.version, '4.1')checking outfluent/fluent-bit-docsatmaster.Confirm that the docs repo uses branch
4.0for 4.0.x andmasterfor 4.1+ (adjust refs if different).
🧹 Nitpick comments (2)
.github/workflows/staging-release.yaml (2)
834-844: 4.0 release correctly marked non-latest; add target_commitish for consistencyGood change. For parity with 2.x/3.x blocks, set target_commitish to the 4.0 branch.
- name: Release 4.0 - not latest uses: softprops/action-gh-release@v2 if: startsWith(inputs.version, '4.0') with: body: "https://fluentbit.io/announcements/v${{ inputs.version }}/" draft: false generate_release_notes: true name: "Fluent Bit ${{ inputs.version }}" tag_name: v${{ inputs.version }} + target_commitish: '4.0' make_latest: false
845-855: 4.1 release as latest: add target_commitish for clarityWorks as intended. Consider pinning to master explicitly.
- name: Release 4.1 and latest uses: softprops/action-gh-release@v2 if: startsWith(inputs.version, '4.1') with: body: "https://fluentbit.io/announcements/v${{ inputs.version }}/" draft: false generate_release_notes: true name: "Fluent Bit ${{ inputs.version }}" tag_name: v${{ inputs.version }} + target_commitish: 'master' make_latest: true
|
@cosmo0920 let's ensure the target commitish is set like CodeRabbit says too |
Signed-off-by: Hiroshi Hatake <[email protected]>
cosmo0920
force-pushed
the
cosmo0920-make-only-4.1-as-latest-for-releasing
branch
from
21112c9 to
65a08f3
Compare
Signed-off-by: Hiroshi Hatake <[email protected]>
cosmo0920
force-pushed
the
cosmo0920-make-only-4.1-as-latest-for-releasing
branch
from
65a08f3 to
c37031f
Compare
2 participants
Without this patch, every 4.0 release will be marked as the latest release.
This could be inconvenient for releasing process of 4.0.x.
Enter
[N/A]in the box, if an item is not applicable to your change.Testing
Before we can approve your change; please submit the following in a comment:
If this is a change to packaging of containers or native binaries then please confirm it works for all targets.
ok-package-testlabel to test for all targets (requires maintainer to do).Documentation
Backporting
Fluent Bit is licensed under Apache 2.0, by submitting this pull request I understand that this code will be released under the terms of that license.
Summary by CodeRabbit