Adding changes for Fleet v4.88.0#48446
Merged
Merged
Conversation
georgekarrv
requested review from
BCTBB,
ddribeiro,
ksatter and
rfairburn
as code owners
![claude[bot]](https://avatars.githubusercontent.com/in/1236702?s=60&v=4){kind=small}
There was a problem hiding this comment.
Claude Code Review
This repository is configured for manual code reviews. Comment @claude review to trigger a review and subscribe this PR to future pushes, or @claude review once for a one-time review.
Tip: disable this comment in your organization's Code Review settings.
Contributor
There was a problem hiding this comment.
Pull request overview
Prepares the repository for the Fleet 4.88.0 release by updating Helm chart metadata and adding the release entry to the changelog.
Changes:
- Bumped the Fleet Helm chart version to v7.0.9.
- Added a Fleet 4.88.0 section to
CHANGELOG.mdwith bug fix notes.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| charts/fleet/Chart.yaml | Bumps Helm chart version (but currently leaves appVersion unchanged). |
| CHANGELOG.md | Adds the Fleet 4.88.0 changelog section. |
Comments suppressed due to low confidence (1)
charts/fleet/Chart.yaml:11
- Chart version was bumped, but Chart.yaml still reports appVersion v4.87.0. For the v4.88.0 release, appVersion should be updated so Helm metadata/labels reflect the correct Fleet version.
version: v7.0.9
home: https://github.com/fleetdm/fleet
sources:
- https://github.com/fleetdm/fleet.git
appVersion: v4.87.0
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| - fleet | ||
| - osquery | ||
| version: v7.0.8 | ||
| version: v7.0.9 |
Contributor
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (2)
Walkthrough
Possibly related PRs
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✨ 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 |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
Contributor
There was a problem hiding this comment.
Caution
Some comments are outside the diff and can’t be posted inline due to platform limitations.
⚠️ Outside diff range comments (1)
charts/fleet/Chart.yaml (1)
7-11: 🎯 Functional Correctness | 🟠 Major | ⚡ Quick winBump the chart pins to
v4.88.0charts/fleet/Chart.yamlstill advertisesappVersion: v4.87.0, andcharts/fleet/values.yamlstill defaultsimageTagtov4.87.0, so the release chart would keep labeling and deploying the previous Fleet version.🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@charts/fleet/Chart.yaml` around lines 7 - 11, Update the Fleet chart version pins so the release points to v4.88.0 instead of v4.87.0. In Chart.yaml, change the appVersion field, and in the corresponding values.yaml default for imageTag, update the default tag to match; use the existing Chart.yaml and values.yaml settings to locate the appVersion/imageTag entries and keep them consistent.
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Outside diff comments:
In `@charts/fleet/Chart.yaml`:
- Around line 7-11: Update the Fleet chart version pins so the release points to
v4.88.0 instead of v4.87.0. In Chart.yaml, change the appVersion field, and in
the corresponding values.yaml default for imageTag, update the default tag to
match; use the existing Chart.yaml and values.yaml settings to locate the
appVersion/imageTag entries and keep them consistent.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
Run ID: 2e5cac2a-9e6c-4047-91dd-d165e85de666
⛔ Files ignored due to path filters (1)
CHANGELOG.mdis excluded by!**/*.md
📒 Files selected for processing (1)
charts/fleet/Chart.yaml
lukeheath
previously approved these changes
Jun 29, 2026
georgekarrv
commented
Jul 2, 2026
georgekarrv
commented
Jul 2, 2026
georgekarrv
commented
Jul 2, 2026
Co-authored-by: George Karr <georgekarrv@users.noreply.github.com>
CI Feedback 🧐A test triggered by this PR failed. Here is an AI-generated analysis of the failure:
|
rfairburn
approved these changes
Jul 2, 2026
Leanngove
pushed a commit
that referenced
this pull request
Jul 2, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary by CodeRabbit
v7.0.9tov7.0.10.v4.87.1tov4.88.0.v4.88.0.