Skip to content

Commit d99ad94

Browse files
Restore Ontology to Phase 1 — fabric-cicd requires it before DataAgent
1 parent 6413ac9 commit d99ad94

2 files changed

Lines changed: 12 additions & 13 deletions

File tree

.github/workflows/deploy-test.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,8 @@
11
# Orchestrator workflow for deploying Fabric items to the Test workspace.
22
#
33
# Deploys all supported items via fabric-cicd. The reusable workflow handles
4-
# phased deployment internally (Lakehouse first, then everything else)
5-
# to satisfy parameter.yml dependency resolution.
4+
# phased deployment internally (Lakehouse + Ontology first, then everything
5+
# else) to satisfy fabric-cicd dependency resolution.
66
#
77
# The ETL workflow (etl-test.yml) triggers automatically via workflow_run
88
# once deployment completes successfully.

.github/workflows/reusable-deploy-supported.yml

Lines changed: 10 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -91,22 +91,21 @@ jobs:
9191
environment = os.environ['ENVIRONMENT']
9292
9393
# Deployment uses a 2-phase approach to satisfy item dependencies.
94-
# fabric-cicd resolves \$items dynamic variables (e.g.,
95-
# \$items.Lakehouse.PatternsLakehouse.\$id) by querying the live workspace,
96-
# so the referenced item must already exist.
94+
# fabric-cicd caches workspace state once at the start of each
95+
# publish_all_items() call, so items deployed within the same call
96+
# are not visible to later items' logicalId or \$items resolution.
9797
#
98-
# Phase 1: Lakehouse only — must exist before Phase 2 so that
99-
# parameter.yml rules referencing \$items.Lakehouse resolve.
100-
# Phase 2: All remaining items (Ontology, DataAgent, Notebook, etc.).
101-
# Ontology and DataAgent use logicalId references resolved by
102-
# Fabric at runtime, so they have no deploy-time ordering needs.
98+
# Phase 1: Lakehouse + Ontology
99+
# - Lakehouse must exist so parameter.yml \$items.Lakehouse rules resolve.
100+
# - Ontology must exist so DataAgent's logicalId reference resolves.
101+
# Phase 2: All remaining items (DataAgent, Notebook, SemanticModel, etc.).
103102
#
104-
# On subsequent deployments, all items already exist and phases are
103+
# On subsequent deployments all items already exist and phases are
105104
# idempotent — they simply update in place.
106105
107-
phase1_types = ['Lakehouse']
106+
phase1_types = ['Lakehouse', 'Ontology']
108107
109-
print('Phase 1: Deploying Lakehouse...')
108+
print('Phase 1: Deploying Lakehouse + Ontology...')
110109
phase1_ws = FabricWorkspace(
111110
repository_directory=repo_dir,
112111
workspace_id=workspace_id,

0 commit comments

Comments
 (0)