Skip to content

Adding changes for Fleet v4.88.1#49038

Open
georgekarrv wants to merge 1 commit into
mainfrom
update-changelog-prepare-4.88.1-main
Open

Adding changes for Fleet v4.88.1#49038
georgekarrv wants to merge 1 commit into
mainfrom
update-changelog-prepare-4.88.1-main

Conversation

@georgekarrv

@georgekarrv georgekarrv commented Jul 9, 2026

Copy link
Copy Markdown
Member

Summary by CodeRabbit

  • New Features

    • Released a new Fleet chart and application version.
    • Updated deployment defaults to use the latest Fleet container image.
  • Tests

    • Adjusted release-candidate branch expectations to match the new version.

Copilot AI review requested due to automatic review settings July 9, 2026 15:29
@georgekarrv georgekarrv requested review from a team, BCTBB, ddribeiro, ksatter and rfairburn as code owners July 9, 2026 15:29
@fleet-release fleet-release requested a review from lukeheath July 9, 2026 15:29

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Updates repo version references to Fleet v4.88.1 across release tooling and deployment artifacts, and adds the corresponding changelog entry.

Changes:

  • Updated Hangar gitrepo tests to reflect the new 4.88.1 RC minor branch naming.
  • Bumped Fleet image tags / versions in fleetctl-npm, dogfood Terraform (AWS/GCP), and the Helm chart (chart version + appVersion + default image tag).
  • Added release notes for Fleet 4.88.1 to CHANGELOG.md.

Reviewed changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
tools/hangar/internal/gitrepo/git_test.go Updates RC branch test data/expectations for the 4.88.1 release line.
tools/fleetctl-npm/package.json Bumps the fleetctl npm package version to v4.88.1.
infrastructure/dogfood/terraform/gcp/variables.tf Updates default Fleet container image tag to v4.88.1 (GCP dogfood).
infrastructure/dogfood/terraform/aws/variables.tf Updates default Fleet container image tag to v4.88.1 (AWS dogfood).
charts/fleet/values.yaml Updates default Helm chart image tag to v4.88.1.
charts/fleet/Chart.yaml Bumps chart version and appVersion for v4.88.1.
CHANGELOG.md Adds Fleet 4.88.1 release notes.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread CHANGELOG.md

- Fixed an issue where a configuration profile could be enqueued multiple times for a single host.
- Fixed recovery lock password being enforced on personally-owned (BYOD) macOS hosts, where it would always fail because personal enrollments have device lock rights stripped. These hosts are now skipped.
- Fixed a bug where a user's BYOD selection was not persisted through IdP authentication
@coderabbitai

coderabbitai Bot commented Jul 9, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 33c7f49d-8528-4237-ac40-a897970835fd

📥 Commits

Reviewing files that changed from the base of the PR and between 232c433 and e213a99.

⛔ Files ignored due to path filters (1)
  • CHANGELOG.md is excluded by !**/*.md
📒 Files selected for processing (6)
  • charts/fleet/Chart.yaml
  • charts/fleet/values.yaml
  • infrastructure/dogfood/terraform/aws/variables.tf
  • infrastructure/dogfood/terraform/gcp/variables.tf
  • tools/fleetctl-npm/package.json
  • tools/hangar/internal/gitrepo/git_test.go

Walkthrough

This PR bumps Fleet's version from v4.88.0 to v4.88.1 (and the chart version from v7.0.10 to v7.0.11) across release-related files: the Helm chart metadata (Chart.yaml, values.yaml), Terraform variable defaults for AWS and GCP dogfood infrastructure, and the fleetctl-npm package version. It also updates the RC branch name fixtures in the git test file to reflect the new version string.

Possibly related PRs

  • fleetdm/fleet#46610: Same code-level version bump pattern across Chart.yaml, values.yaml, Terraform variables, and package.json, for a different version.
  • fleetdm/fleet#47219: Same release-manifest version bump touching the identical set of files.
  • fleetdm/fleet#48446: Same Chart.yaml/values.yaml version bump pattern, from v4.87.1 to v4.88.0.
🚥 Pre-merge checks | ✅ 3 | ❌ 2

❌ Failed checks (1 warning, 1 inconclusive)

Check name Status Explanation Resolution
Description check ⚠️ Warning No pull request description was added, so the required template sections and checklist items are missing. Add the PR description using the repository template, including the related issue, checklist items, testing, and any applicable release notes.
Title check ❓ Inconclusive The title matches the release/version bump, but 'Adding changes' is vague and doesn't clearly state the change. Use a more descriptive title like 'Bump Fleet to v4.88.1' so the main change is immediately clear.
✅ Passed checks (3 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch update-changelog-prepare-4.88.1-main

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@codecov

codecov Bot commented Jul 9, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 68.12%. Comparing base (232c433) to head (e213a99).

Additional details and impacted files
@@           Coverage Diff           @@
##             main   #49038   +/-   ##
=======================================
  Coverage   68.11%   68.12%           
=======================================
  Files        3731     3731           
  Lines      235434   235434           
  Branches    12371    12371           
=======================================
+ Hits       160375   160380    +5     
+ Misses      60670    60666    -4     
+ Partials    14389    14388    -1     
Flag Coverage Δ
backend 69.73% <ø> (+<0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants