Skip to content

chore: bump release 2.18.0#557

Merged
kevin-dimichel merged 6 commits into
release/v2.18.0from
kacey-bump-versions-2.18.0
Apr 28, 2026
Merged

chore: bump release 2.18.0#557
kevin-dimichel merged 6 commits into
release/v2.18.0from
kacey-bump-versions-2.18.0

Conversation

@kaixi-wang

@kaixi-wang kaixi-wang commented Apr 28, 2026

Copy link
Copy Markdown
Member

Rationale

bump from 2.17.2 to 2.18.0

for file in $(ack -l '2\.17\.2'); do perl -pi -e 's/2\.17\.2/2.18.0/g' $file; done
git restore tests/fixtures/**

Changes

Checklist

  • This PR maintains parity between Docker Compose and Helm

Testing

@kaixi-wang kaixi-wang requested a review from a team as a code owner April 28, 2026 02:55
@kaixi-wang kaixi-wang self-assigned this Apr 28, 2026
@coderabbitai

coderabbitai Bot commented Apr 28, 2026

Copy link
Copy Markdown
Contributor

Important

Review skipped

Auto incremental reviews are disabled on this repository.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: aed17dbb-1938-4e2d-a205-0caede802d1e

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

Walkthrough

This pull request updates the FiftyOne platform version from 2.17.2 to 2.18.0 across the codebase. The changes include the Makefile default version variable, Docker and Helm documentation examples, feature documentation, test fixtures, and unit test expectations.

Changes

Cohort / File(s) Summary
Build Configuration
Makefile
Default VERSION variable bumped from 2.17.2 to 2.18.0.
Docker Documentation
docker/README.md, docker/docs/configuring-gpu-workloads.md, docker/docs/upgrading.md
Image tag references and Docker Compose examples updated from v2.17.2 to v2.18.0; upgrading guide documentation references replaced throughout.
Feature Documentation
docs/custom-plugins.md, docs/orchestrators/configuring-databricks-orchestrator.md
Version references in code examples and build arguments updated from 2.17.2 to 2.18.0.
Helm Chart Documentation
helm/docs/upgrading.md, helm/fiftyone-teams-app/README.md
Enterprise upgrade instructions and SDK version recommendations updated from 2.17.2 to 2.18.0; Helm chart README badge and default environment variables bumped accordingly.
Test Fixtures
tests/fixtures/docker/integration_internal_auth.env, tests/fixtures/docker/integration_legacy_auth.env
Environment variables (VERSION, FIFTYONE_APP_VERSION, FIFTYONE_TEAMS_API_VERSION, FIFTYONE_TEAMS_APP_VERSION, FIFTYONE_TEAMS_CAS_VERSION) updated from v2.17.2 to v2.18.0.
Test Files
tests/unit/compose/docker-compose-internal-auth_test.go, tests/unit/compose/docker-compose-legacy-auth_test.go
Test assertions updated to expect service image tags v2.18.0 and FIFTYONE_APP_TEAMS_SDK_RECOMMENDED_VERSION=2.18.0 across multiple compose scenarios.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

🐰 Hop, hop, hooray for version's climb,
From 2.17.2, to 2.18—what a time!
Through Dockerfiles and tests we bound,
Each version string that we have found,
A carrot patch of updates, clean and bright!

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Description check ⚠️ Warning The PR description is incomplete and missing critical information required by the template, including unfilled priority checkbox and missing concrete change descriptions. Fill in the Review Priority section by checking one priority level, provide detailed descriptions of all changes made (version bump details, affected components), and document the testing approach used to validate the version bump.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title 'chore: bump release 2.18.0' clearly and concisely describes the main change: bumping the release version to 2.18.0, which aligns with the changeset that updates version references throughout the codebase.
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.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch kacey-bump-versions-2.18.0

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

@mdecarlo mdecarlo requested review from mdecarlo and removed request for mdecarlo April 28, 2026 13:29
mo-getter
mo-getter previously approved these changes Apr 28, 2026

@kevin-dimichel kevin-dimichel 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.

How were the changes in this PR created? It seems like we didn't follow the docs or our previous PRs. See #551 (comment).

We shouldn't update the tests/fixtures files (yet). That'll be handled during our release automation.

Also please update the PR description.

@lanzhenw

lanzhenw commented Apr 28, 2026

Copy link
Copy Markdown
Member

@kevin-dimichel helm-integration test is failing due to a helm v4 - skaffold compatibility issue. Any suggestions?

@lanzhenw lanzhenw requested a review from mo-getter April 28, 2026 16:43
@kevin-dimichel kevin-dimichel mentioned this pull request Apr 28, 2026
4 tasks
@kevin-dimichel

Copy link
Copy Markdown
Contributor

Thanks @lanzhenw It took a bit, but the fixes in #558 should make the PR checks happy.

@kevin-dimichel kevin-dimichel mentioned this pull request Apr 28, 2026
4 tasks
@kevin-dimichel kevin-dimichel merged commit 88e488a into release/v2.18.0 Apr 28, 2026
12 checks passed
@kevin-dimichel kevin-dimichel deleted the kacey-bump-versions-2.18.0 branch April 28, 2026 19:24
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.

4 participants