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.

Sign up for GitHub

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

Jump to bottom

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

Open
brettcurtis opened this issue Aug 26, 2023 · 1 comment
Open

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

brettcurtis opened this issue Aug 26, 2023 · 1 comment
Labels
estimate/4h Need 4 hours to be done feature New feature or request
Milestone
Features implemen...

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
@MaxymVlasov MaxymVlasov mentioned this issue Mar 12, 2024
Merged
4 tasks
MaxymVlasov added a commit that referenced this issue Mar 12, 2024
@MaxymVlasov
Merge branch 'master' into GH-638/GH-562/better_logging
fdc3594
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
estimate/4h Need 4 hours to be done feature New feature or request
Projects
None yet
Milestone
Features implementing and cleanup wit...
Development

No branches or pull requests
2 participants
@brettcurtis @MaxymVlasov