fix(ingestion): don't dump variables for config errors

[hsheth2]

Checklist

• The PR conforms to DataHub's Contributing Guideline (particularly Commit Message Format)
• Links to related issues (if applicable)
• Tests for the changes have been added/updated (if applicable)
• Docs related to the changes have been added/updated (if applicable)

[github-actions]

Unit Test Results (build & test)

  45 files  ±0    45 suites  ±0   16m 34s ⏱️ + 2m 23s
505 tests ±0  449 ✔️ ±0  56 💤 ±0  0 ❌ ±0 

Results for commit 386b4eb. ± Comparison against base commit cec541d.

♻️ This comment has been updated with latest results.

[github-actions]

Unit Test Results (metadata ingestion)

       5 files  ±0         5 suites  ±0   36m 19s ⏱️ -40s
   274 tests ±0     273 ✔️ ±0    0 💤 ±0  1 ❌ ±0 
1 261 runs  +2  1 232 ✔️ +8  28 💤  - 6  1 ❌ ±0 

For more details on these failures, see this check.

Results for commit 386b4eb. ± Comparison against base commit cec541d.

♻️ This comment has been updated with latest results.

[jjoyce0510]

Were you able to verify this locally?

[hsheth2]

I was able to verify that it works locally - made one of the config validators throw an exception to test it.

However I think the approach is a bit ugly (particularly the SensitiveError.get_sensitive_cause() method) so open to suggestions/improvements

[shirshanka]

LGTM!