Repository navigation
docs(databricks): correct three claims about data quality monitoring - #866
Merged
Merged
Conversation
Found while documenting #864 for forge_docs. - Alerts: provider 1.140.0 does have notification_settings, but only under data_profiling_config (on_failure); anomaly_detection_config has only excluded_table_full_names. HONESTLY_TESTED said the provider had none. - A refused monitor's hints do not come first: CLIError's three generic suggestions come before them, and they go ahead of the engine's own tofu state list hint. CHANGELOG, iac/base.py and the _apply_failure docstring now say that. - The Free Edition quota covering the scans was assumed, not measured: the CHANGELOG now says to expect them to count against it.
Merged
1 of 3 tasks
fas89
enabled auto-merge (squash)
November 12, 2026 22:59
This was referenced Nov 13, 2026
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
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.
Follow-up to #864. Wording only: CHANGELOG, HONESTLY_TESTED and two docstrings; no code path changes.
Found while writing the forge_docs page (Agenticstiger/forge_docs#208):
notification_settings, but only underdata_profiling_config(on_failure).anomaly_detection_confighas onlyexcluded_table_full_names, so schema anomaly detection gets no alerts from this provider. HONESTLY_TESTED said the provider had nonotification_settingsat all.CLIErrorputs its three generic suggestions before them, and they go ahead of the engine's owntofu state listhint. The CHANGELOG,iac/base.pyand the_apply_failuredocstring now say that.Validation:
ruff,black==24.10.0 --check,detect-secrets-hookon the changed files;tests/iac/test_iac_databricks_quality.pyandtests/cli/test_apply_reports_to_command_center.pypass./ship steps 5-7 (security review, inspection, defect scan) skipped: wording-only change, no behaviour to review.
Documentation