bitovi / bitops

Automate the provisioning and configuration of cloud infrastructure with BitOps docker image
https://bitops.sh
Other
36 stars 9 forks source link

Switch to "WARNING" logging level by default #398

Closed arm4b closed 1 year ago

arm4b commented 1 year ago

Switch the default logging level to WARNING instead of DEBUG for BitOps.

As we're starting to show the BitOps to others, be mindful of what data we show as logs, - users should be getting only the important messages, excluding all the noise.

Somewhat relevant to https://github.com/bitovi/bitops/issues/270, which highlights a bigger problem with the log formatting standards.

runforesight[bot] commented 1 year ago

Foresight Summary

     Major Impacts Foresight hasn't detected any major impact on your workflows and tests.

View More Details #### :white_check_mark:  [CI](https://app.runforesight.com/repositories/github/bitovi/bitops/workflow-run/874edc71-e244-3c00-aa81-042ac324e045 "See Workflow Run Details") workflow has finished in 43 seconds and finished at 8th Feb, 2023. --- | Job | Failed Steps | Tests | | |---|---|---|---| |lint-pylint|-     [:link:](https://app.runforesight.com/repositories/github/bitovi/bitops/workflow-run/874edc71-e244-3c00-aa81-042ac324e045/11182662115/steps "See Steps Details") | N/A|[See Details](https://app.runforesight.com/repositories/github/bitovi/bitops/workflow-run/874edc71-e244-3c00-aa81-042ac324e045/11182662115/steps)| |lint-black|-     [:link:](https://app.runforesight.com/repositories/github/bitovi/bitops/workflow-run/874edc71-e244-3c00-aa81-042ac324e045/11182662276/steps "See Steps Details") | N/A|[See Details](https://app.runforesight.com/repositories/github/bitovi/bitops/workflow-run/874edc71-e244-3c00-aa81-042ac324e045/11182662276/steps)| ---

🔎 See details on Foresight

*You can configure Foresight comments in your organization settings page.

PhillypHenning commented 1 year ago

Straight forward, looks good to me

I feel like we remove the newline for for combined_output in process.stdout: every PR.... 🤷‍♂️

mcascone commented 1 year ago

YES the output is WAY too noisy!