Skip to content

Revert "Removed switchToModelVersionAt from SO API definition (#219029)"#220063

Merged
kobelb merged 2 commits intoelastic:deploy-fix@1746425571from
afharo:hotfix/revert-19c017a
May 5, 2025
Merged

Revert "Removed switchToModelVersionAt from SO API definition (#219029)"#220063
kobelb merged 2 commits intoelastic:deploy-fix@1746425571from
afharo:hotfix/revert-19c017a

Conversation

@afharo
Copy link
Copy Markdown
Member

@afharo afharo commented May 5, 2025

This reverts commit 19c017a.

Summary

Reverts #219029 as it is causing SO Migration crashes in our QA environment.

It can also be tested locally on main:

  1. On a previous version (git checkout deploy@1745820726)
  2. Start ES: yarn es serverless --projectType es
  3. Start Kibana: yarn start --serverless=es
  4. Wait for migrations to complete
  5. Stop Kibana
  6. Move back to main
  7. Start Kibana: yarn start --serverless=es
  8. Wait for the error

On main, we can see that Kibana fails to start with the fatal error:

 FATAL  Error: Unable to complete saved object migrations for the [.kibana_alerting_cases] index: Model version conflict: inconsistent higher/lower versions

In this branch, it starts successfully.

@afharo afharo requested review from a team as code owners May 5, 2025 12:09
@botelastic botelastic bot added the Team:actionable-obs Formerly "obs-ux-management", responsible for SLO, o11y alerting, significant events, & synthetics. label May 5, 2025
@elasticmachine
Copy link
Copy Markdown
Contributor

Pinging @elastic/obs-ux-management-team (Team:obs-ux-management)

kobelb
kobelb previously approved these changes May 5, 2025
jesuswr
jesuswr previously approved these changes May 5, 2025
legrego
legrego previously approved these changes May 5, 2025
@afharo afharo dismissed stale reviews from legrego, jesuswr, and kobelb via a8e1627 May 5, 2025 12:19
@jesuswr jesuswr self-requested a review May 5, 2025 12:20
@elasticmachine
Copy link
Copy Markdown
Contributor

💛 Build succeeded, but was flaky

Failed CI Steps

Test Failures

  • [job] [logs] FTR Configs #96 / discover - group 1 Discover CSV Export Check Available is available if new

Metrics [docs]

Unknown metric groups

API count

id before after diff
@kbn/core-saved-objects-server 573 574 +1

References to deprecated APIs

id before after diff
@kbn/core 636 638 +2

@kobelb kobelb merged commit 089dfdb into elastic:deploy-fix@1746425571 May 5, 2025
9 checks passed
@afharo afharo deleted the hotfix/revert-19c017a branch May 5, 2025 13:57
@jmikell821 jmikell821 added the release_note:skip Skip the PR/issue when compiling release notes label May 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release_note:skip Skip the PR/issue when compiling release notes Team:actionable-obs Formerly "obs-ux-management", responsible for SLO, o11y alerting, significant events, & synthetics.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants