Skip to content

Commit 563df69

Browse files
Fix: Use categorization config in cicd bot environment summary (#5622)
1 parent 596b911 commit 563df69

File tree

2 files changed

+3
-3
lines changed

2 files changed

+3
-3
lines changed

sqlmesh/integrations/github/cicd/controller.py

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -448,10 +448,9 @@ def prod_plan_with_gaps(self) -> Plan:
448448
c.PROD,
449449
# this is required to highlight any data gaps between this PR environment and prod (since PR environments may only contain a subset of data)
450450
no_gaps=False,
451-
# this works because the snapshots were already categorized when applying self.pr_plan so there are no uncategorized local snapshots to trigger a plan error
452-
no_auto_categorization=True,
453451
skip_tests=True,
454452
skip_linter=True,
453+
categorizer_config=self.bot_config.auto_categorize_changes,
455454
run=self.bot_config.run_on_deploy_to_prod,
456455
forward_only=self.forward_only_plan,
457456
)

tests/integrations/github/cicd/test_github_controller.py

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -339,7 +339,8 @@ def test_prod_plan_with_gaps(github_client, make_controller):
339339

340340
assert controller.prod_plan_with_gaps.environment.name == c.PROD
341341
assert not controller.prod_plan_with_gaps.skip_backfill
342-
assert not controller._prod_plan_with_gaps_builder._auto_categorization_enabled
342+
# auto_categorization should now be enabled to prevent uncategorized snapshot errors
343+
assert controller._prod_plan_with_gaps_builder._auto_categorization_enabled
343344
assert not controller.prod_plan_with_gaps.no_gaps
344345
assert not controller._context.apply.called
345346
assert controller._context._run_plan_tests.call_args == call(skip_tests=True)

0 commit comments

Comments
 (0)