Paw/ct 1652 restore default logging

[peterallenwebb]

resolves #6434

Description

Restore logging to stdout before the logging subsystem is fully configured, so that earlier errors are still presented to the user in some form.

Checklist

• I have read the contributing guide and understand what's expected of me
• I have signed the CLA
• I have run this code in development and it appears to resolve the stated issue
• This PR includes tests, or tests are not required/relevant for this PR
• I have opened an issue to add/update docs, or docs changes are not required/relevant for this PR
• I have run changie new to create a changelog entry

[peterallenwebb]

Verified locally the same behavior documented in the issue, and also verified that the propose fix restores the expected behavior:

(env310) ~/Code/dbt-labs/internal-analytics# dbt run
20:42:52  Error importing adapter: No module named 'dbt.adapters.snowflake'
20:42:52  Encountered an error while reading profiles:
20:42:52    ERROR: Runtime Error
  Credentials in profile "garage-snowflake", target "dev" invalid: Runtime Error
    Could not find adapter type snowflake!
20:42:52  Defined profiles:
20:42:52   - default
20:42:52   - peter_test_1
20:42:52   - garage-snowflake
20:42:52   - jaffle_shop
20:42:52
For more information on configuring profiles, please consult the dbt docs:

https://docs.getdbt.com/docs/configure-your-profile

20:42:52  Encountered an error:
Runtime Error
  Could not run dbt

[peterallenwebb]

@MichelleArk You deleted your comment while I was responding, but yes, there is at least the danger that a logger could be double-added. This was my comment:

Our implementation of setup_event_logger() clears the loggers and handlers for this reason, before adding the configured loggers. My feeling is that anyone else who accesses EVENT_MANAGER directly is taking their life in their hands and should know what they are doing. We also hope to get rid of all globals associated with logging, and several other inelegant hacks in this file, in version 1.5. Hopefully we can discuss that further at Brown Bag.

[MichelleArk]

[answered here] Is there a risk of double-adding loggers downstream, leading to extraneous calls to logger.write?

[dbeatty10]

This originally resolved #6436 on accident. I just updated it to #6434 instead.