Skip to content

Minor move 20 migrations#26236

Merged
TeddyCr merged 3 commits intoopen-metadata:mainfrom
TeddyCr:MINOR-Move-20-Migrations
Mar 5, 2026
Merged

Minor move 20 migrations#26236
TeddyCr merged 3 commits intoopen-metadata:mainfrom
TeddyCr:MINOR-Move-20-Migrations

Conversation

@TeddyCr
Copy link
Collaborator

@TeddyCr TeddyCr commented Mar 4, 2026

Describe your changes:

Fixes

I worked on ... because ...

Type of change:

  • Bug fix
  • Improvement
  • New feature
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation

Checklist:

  • I have read the CONTRIBUTING document.
  • My PR title is Fixes <issue-number>: <short explanation>
  • I have commented on my code, particularly in hard-to-understand areas.
  • For JSON Schema changes: I updated the migration scripts or explained why it is not needed.

Summary by Gitar

  • Migration reorganization:
    • Moved postDataMigrationSQLScript.sql for MySQL and PostgreSQL from version 2.0.0 to 1.13.0 directory

This will update automatically on new commits.

@gitar-bot
Copy link

gitar-bot bot commented Mar 4, 2026

🔍 CI failure analysis for c1e61df: The playwright-ci-postgresql (6, 6) job failure consists of 1 hard failure and 7 flaky tests, all unrelated to this PR's migration file reorganization.

Issue

The playwright-ci-postgresql (6, 6) job failed with 1 hard failure and 7 flaky tests out of 609 total tests (593 passed, 8 skipped).

Root Cause

These failures are unrelated to this PR's changes. This PR only moves postDataMigrationSQLScript.sql files between version directories (2.0.01.13.0) for MySQL and PostgreSQL — no application logic or schema content was modified.

Hard failure (1):

  • Lineage.spec.ts:466Verify table search with special characters as handled — timed out with locator.click: Test timeout of 60000ms exceeded after 2 retries. This is a UI/search flakiness issue unrelated to SQL migration file placement.

Flaky tests (7):

  1. ExplorePageRightPanel.spec.ts:1971 — empty state message visibility assertion
  2. Glossary.spec.ts:2339 — create glossary with optional fields
  3. HyperlinkCustomProperty.spec.ts:131 — URL display assertion
  4. Tag.spec.ts:571 — asset restriction with limited EditTag permission
  5. UserDetails.spec.ts:492Admin user can get all the roles hierarchy and edit rolesTarget page, context or browser has been closed during click
  6. Users.spec.ts:570Check permissions for Data Steward — 180s timeout, Target page, context or browser has been closed while waiting for edit-displayName-button
  7. VersionPages/EntityVersionPages.spec.ts:152SearchIndexTypeError: Cannot read properties of undefined (reading '0') accessing entityResponseData.fields[0].fullyQualifiedName — indicates the SearchIndex entity was created without fields in this run, a known intermittent data race.

The Target page, context or browser has been closed errors in tests #5 and #6 point to browser/server resource exhaustion under concurrent test load — a well-known infrastructure flakiness pattern for this suite. None of these failures are in code paths touched by this PR.

Code Review ✅ Approved

Moves 20 migrations as part of minor refactoring. No issues found.

Tip

Comment Gitar fix CI or enable auto-apply: gitar auto-apply:on

Options

Auto-apply is off → Gitar will not commit updates to this branch.
Display: compact → Showing less information.

Comment with these commands to change:

Auto-apply Compact
gitar auto-apply:on         
gitar display:verbose         

Was this helpful? React with 👍 / 👎 | Gitar

@TeddyCr TeddyCr merged commit 40bf82f into open-metadata:main Mar 5, 2026
16 of 21 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Ingestion safe to test Add this label to run secure Github workflows on PRs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants