Skip to content

Bump migration timestamps after 4.89.2 cherry-picks (4.90 RC)#49850

Merged
georgekarrv merged 1 commit into
rc-minor-fleet-v4.90.0from
bump-migrations-4.90-rc-after-4.89.2
Jul 23, 2026
Merged

Bump migration timestamps after 4.89.2 cherry-picks (4.90 RC)#49850
georgekarrv merged 1 commit into
rc-minor-fleet-v4.90.0from
bump-migrations-4.90-rc-after-4.89.2

Conversation

@georgekarrv

Copy link
Copy Markdown
Member

Related issue: N/A — release-engineering fix for migration forward-compatibility

Summary

Cherry-pick of #49849 into rc-minor-fleet-v4.90.0.

The 4.89.2 patch RC received two cherry-picks carrying DB migrations (#48664 DedupeWindowsProgramTitlesFromUpgradeCode, #49515 FixInstallStatusPrecedence), making 20260717152653 the newest applied migration in 4.89.2. This 4.90 RC contains 11 migrations with timestamps earlier than 20260717152653 that are not in 4.89.2, so a database upgraded through 4.89.2 (max applied = 20260717152653) would encounter them out-of-order when upgrading to 4.90:

20260704213830  DropWindowsMDMCommandResultsCommandUUIDForeignKey
20260706174522  ReparseWindowsHostCertificates
20260707071142  AddWindowsMDMConfigProfilesPriorContent
20260707140752  AddDDMAssetsTable
20260708153912  FixEmbeddedBundleTitleNames
20260708185958  CreateApplePSSOTables
20260708190008  AddPSSODeviceRegistrationTokenFleetVar
20260708192536  AddNanoQueueNextCommandIndex
20260713115453  CreateHostMDMAppleDeviceNames
20260715144547  CustomHostVitals
20260715214831  MultipleCustomPackagesPerTitle

These are bumped to 2026072318140120260723181411 (after 20260717152653), preserving relative order and matching the timestamps used on main in #49849 so the two lines stay consistent.

schema.sql was regenerated against this RC's migration set; DDL is unchanged (only the migration_status_tables version list differs). This RC does not include 20260721090128 / 20260721160351 (they are not part of 4.90), so those are intentionally absent from the regenerated schema.

Checklist for submitter

  • Input data is properly validated, SELECT * is avoided, SQL injection is prevented (using placeholders for values in statements), JS inline code is prevented especially for url redirects, and untrusted data interpolated into shell scripts/commands is validated against shell metacharacters.

Testing

  • QA'd all new/changed functionality manually

Migrations replay cleanly in the new order via make dump-test-schema; regenerated schema.sql differs only in the migration_status_tables INSERT (table definitions byte-identical).

Database migrations

  • Checked schema for all modified table for columns that will auto-update timestamps during migration.
  • Confirmed that updating the timestamps is acceptable, and will not cause unwanted side effects.
  • Ensured the correct collation is explicitly set for character columns (COLLATE utf8mb4_unicode_ci).

No migration logic was changed — only timestamps/filenames were bumped to fix ordering.

The 4.89.2 patch RC (rc-patch-fleet-v4.89.2) received two cherry-picks that
carry DB migrations:

- #48664  20260702164518_DedupeWindowsProgramTitlesFromUpgradeCode
- #49515  20260717152653_FixInstallStatusPrecedence

This makes 20260717152653 the newest (max) applied migration in 4.89.2. But
on main/4.90 there are 11 migrations with timestamps *earlier* than
20260717152653 that are not present in 4.89.2:

  20260704213830  DropWindowsMDMCommandResultsCommandUUIDForeignKey
  20260706174522  ReparseWindowsHostCertificates
  20260707071142  AddWindowsMDMConfigProfilesPriorContent
  20260707140752  AddDDMAssetsTable
  20260708153912  FixEmbeddedBundleTitleNames
  20260708185958  CreateApplePSSOTables
  20260708190008  AddPSSODeviceRegistrationTokenFleetVar
  20260708192536  AddNanoQueueNextCommandIndex
  20260713115453  CreateHostMDMAppleDeviceNames
  20260715144547  CustomHostVitals
  20260715214831  MultipleCustomPackagesPerTitle

A database upgraded through 4.89.2 records 20260717152653 as its max applied
migration, so on upgrade to 4.90 those 11 would be out-of-order. Bump them so
they sort after the current newest migration (20260721160351), preserving
their relative order. Schema DDL is unchanged (schema.sql differs only in the
migration_status_tables version list).

To be cherry-picked into rc-minor-fleet-v4.90.0.
@georgekarrv
georgekarrv requested a review from a team as a code owner July 23, 2026 18:44
@georgekarrv
georgekarrv merged commit 166b673 into rc-minor-fleet-v4.90.0 Jul 23, 2026
23 checks passed
@georgekarrv
georgekarrv deleted the bump-migrations-4.90-rc-after-4.89.2 branch July 23, 2026 18:51
@codecov

codecov Bot commented Jul 23, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 66.66667% with 11 lines in your changes missing coverage. Please review.
⚠️ Please upload report for BASE (rc-minor-fleet-v4.90.0@78677aa). Learn more about missing BASE report.

Files with missing lines Patch % Lines
...opWindowsMDMCommandResultsCommandUUIDForeignKey.go 66.66% 1 Missing ⚠️
...s/20260723181402_ReparseWindowsHostCertificates.go 66.66% 1 Missing ⚠️
...3181403_AddWindowsMDMConfigProfilesPriorContent.go 66.66% 1 Missing ⚠️
...rations/tables/20260723181404_AddDDMAssetsTable.go 66.66% 1 Missing ⚠️
...bles/20260723181405_FixEmbeddedBundleTitleNames.go 66.66% 1 Missing ⚠️
...ons/tables/20260723181406_CreateApplePSSOTables.go 66.66% 1 Missing ⚠️
...23181407_AddPSSODeviceRegistrationTokenFleetVar.go 66.66% 1 Missing ⚠️
...les/20260723181408_AddNanoQueueNextCommandIndex.go 66.66% 1 Missing ⚠️
...es/20260723181409_CreateHostMDMAppleDeviceNames.go 66.66% 1 Missing ⚠️
...grations/tables/20260723181410_CustomHostVitals.go 66.66% 1 Missing ⚠️
... and 1 more
Additional details and impacted files
@@                    Coverage Diff                    @@
##             rc-minor-fleet-v4.90.0   #49850   +/-   ##
=========================================================
  Coverage                          ?   67.87%           
=========================================================
  Files                             ?     3890           
  Lines                             ?   248272           
  Branches                          ?    12967           
=========================================================
  Hits                              ?   168521           
  Misses                            ?    64543           
  Partials                          ?    15208           
Flag Coverage Δ
backend 69.28% <66.66%> (?)

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