acc: add invariant/auto-migrate test - #6806
Merged
Merged
Conversation
New invariant target: deploy on terraform, then deploy on the direct default (auto-migrating the state), then assert no drift - across every resource config. Mirrors invariant/migrate but exercises the deploy-triggered auto path instead of the explicit "bundle deployment migrate" command. schema.yml.tmpl is excluded for now because it drifts on main; the migrate-before-deploy change (#6749) fixes it and drops the exclusion. Co-authored-by: Isaac <no-reply@databricks.com>
The second (auto-migrate) deploy used invariant_deploy, which re-echoes the INPUT_CONFIG_OK fuzzer marker, so it printed twice. The config is already validated by the terraform deploy above; a failure in the auto-migrate deploy is a bug, not an invalid config. Use a plain trace + panic/internal-error check for it, matching invariant/migrate, so the marker fires once. Co-authored-by: Isaac <no-reply@databricks.com>
denik
changed the title
denik
enabled auto-merge
September 23, 2026 09:20
Sync the exclude list to the current migrate/test.toml: schema.yml.tmpl is already excluded via no_schema_custom_retention (custom_max_retention_hours drifts a terraform-first migrate; schema_tf_compat.yml covers schemas), so drop the separate no_schema exclusion I had added. Add the missing comment on no_secret (UC secrets are direct-only, so the terraform seed deploy fails for them). Co-authored-by: Isaac <no-reply@databricks.com>
Co-authored-by: Isaac <no-reply@databricks.com>
…ariant Co-authored-by: Isaac <no-reply@databricks.com>
Collaborator
Integration test reportCommit: 9712345
Top 15 slowest tests (at least 2 minutes):
|
janniklasrose
approved these changes
Sep 23, 2026
Collaborator
Integration test reportCommit: ff85929
320 interesting tests: 196 MISS, 124 FAIL
Top 50 slowest tests (at least 2 minutes):
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Changes
Add an invariant target that deploys on terraform, then deploys on the direct default
(which auto-migrates the state), then asserts no drift — across every resource config.
Mirrors
invariant/migratebut exercises the deploy-triggered auto path rather than theexplicit
bundle deployment migratecommand.Why
Prep for #6749 (move terraform→direct state migration before deploy): landing the
auto-migration drift coverage first, off main, so that PR stays focused on the behavior change.
This pull request and its description were written by Isaac.