Skip to content

Comments

Fix: Evaluation of metadata snapshots with audit changes#5267

Merged
izeigerman merged 2 commits intomainfrom
fix-eval-audit-only-snapshots
Sep 1, 2025
Merged

Fix: Evaluation of metadata snapshots with audit changes#5267
izeigerman merged 2 commits intomainfrom
fix-eval-audit-only-snapshots

Conversation

@izeigerman
Copy link
Member

#5189 introduced a regression in which a plan application would sometimes fail with an exception like:

KeyError: '<model fqn>'

when making metadata-only changes.

This PR fixes the issue as well as addresses the gap in test coverage which ultimately allowed for this regression to take place.

@izeigerman izeigerman requested a review from a team August 31, 2025 02:58
@izeigerman izeigerman merged commit da49d57 into main Sep 1, 2025
36 checks passed
@izeigerman izeigerman deleted the fix-eval-audit-only-snapshots branch September 1, 2025 15:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants