Manifest upgrade_schema_version runs upgrade script on all older ve…

…rsions (#9488) * Make test fail correctly * Manifest upgrade_schema_version runs on any older version * Add changelog entry
dbt-labs · Jan 30, 2024 · 9c8b28a · 9c8b28a
1 parent 719a50c
commit 9c8b28a
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 2 deletions.
diff --git a/.changes/unreleased/Fixes-20240130-124135.yaml b/.changes/unreleased/Fixes-20240130-124135.yaml
@@ -0,0 +1,7 @@
+kind: Fixes
+body: Run manifest upgrade preprocessing on any older manifest version, including
+  v11
+time: 2024-01-30T12:41:35.899412+01:00
+custom:
+  Author: jtcohen6
+  Issue: "9487"
diff --git a/core/dbt/artifacts/schemas/manifest/v12/manifest.py b/core/dbt/artifacts/schemas/manifest/v12/manifest.py
@@ -165,7 +165,7 @@ def upgrade_schema_version(cls, data):
         """This overrides the "upgrade_schema_version" call in VersionedSchema (via
         ArtifactMixin) to modify the dictionary passed in from earlier versions of the manifest."""
         manifest_schema_version = get_artifact_schema_version(data)
-        if manifest_schema_version <= 10:
+        if manifest_schema_version < cls.dbt_schema_version.version:
             data = upgrade_manifest_json(data, manifest_schema_version)
         return cls.from_dict(data)
 

diff --git a/tests/functional/artifacts/data/state/v11/manifest.json b/tests/functional/artifacts/data/state/v11/manifest.json