Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Introduce log levels and move unneeded logs to debug log level #562

Open
brettcurtis opened this issue Aug 26, 2023 · 2 comments
Open

Introduce log levels and move unneeded logs to debug log level #562

brettcurtis opened this issue Aug 26, 2023 · 2 comments
Labels
estimate/4h Need 4 hours to be done feature New feature or request

Comments

@brettcurtis
Copy link

Describe the bug

Seeing duplicate or unclear output in regards to Success! The configuration is valid.

Command 'terraform init' successfully done: regional/infra
Success! The configuration is valid.

Success! The configuration is valid.

How can we reproduce it?

gh repo clone osinfra-io/google-cloud-networking
cd google-cloud-networking
 find . -name ".terraform.lock.hcl" | xargs rm -rf
 find . -name ".terraform" | xargs rm -rf
pre-commit run -a
@brettcurtis brettcurtis added area/local_installation bug Something isn't working labels Aug 26, 2023
@MaxymVlasov
Copy link
Collaborator

Okay, need to add a DEBUG log level, set log level to INFO by default, and move mostly all of these logs to the DEBUG lvl

image

@MaxymVlasov MaxymVlasov changed the title Duplicate Success! The configuration is valid. output Introduce log levels and move unnedded logs to debug log level Aug 28, 2023
@MaxymVlasov MaxymVlasov added feature New feature or request and removed bug Something isn't working area/local_installation labels Aug 28, 2023
@MaxymVlasov MaxymVlasov changed the title Introduce log levels and move unnedded logs to debug log level Introduce log levels and move unneeded logs to debug log level Aug 28, 2023
@MaxymVlasov MaxymVlasov added the estimate/4h Need 4 hours to be done label Feb 16, 2024
@pvickery-ParamountCommerce

Another example of unneeded logs are when tflint installs plugins on multiple directories when there is a failure.

Terraform validate with tflint...........................................Failed
- hook id: terraform_tflint
- exit code: 2

Command 'tflint --init' successfully done:
Installing plugins on each working directory...

====================================================
working directory: example0

Plugin "aws" is already installed
====================================================
working directory: example1

Plugin "aws" is already installed
...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
estimate/4h Need 4 hours to be done feature New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants