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

1.39.0 logs have DEBUG log level info and can't be changed to INFO, which makes them 1000x bigger #2128

Open
1 task done
mikamikaWolt opened this issue Nov 8, 2024 · 2 comments
Labels

Comments

@mikamikaWolt
Copy link

Is there an existing issue for this?

  • I have searched the existing issues and didn't find mine.

Steps to reproduce

Run any flow on a local machine on 1.36.0 and 1.39.0, then compare maestro.log from two runs.

Actual results

maestro has test suite logs in /.maestro/tests//maestro.log
Now it contains DEBUG logs, mostly from io.grpc.netty.NettyClientHandler.log.
It can't be turned off, but this makes the log file much bigger.
For example, my short sign in to the app test on 1.36.0 has 8KB log, but with 1.39.0 it is a whopping 4 MB.
This makes it too expensive to store logs as an artifact in CI or send them with test reports somewhere.

Expected results

The default log level should be INFO as before.
Log level should be available to configure.

About app

Any mobile app: problem is with maestro logs, not with the app.

About environment

macOS 15.1

Logs

Logs
<!-- Replace this line with your logs. *DO NOT* remove the backticks! -->

Maestro version

1.39.0

How did you install Maestro?

install script (https://get.maestro.mobile.dev)

Anything else?

No response

@mikamikaWolt mikamikaWolt changed the title 1.39.0 logs have DEBUG log level info and can't chnaged to INFO, which makes them 1000x bigger 1.39.0 logs have DEBUG log level info and can't be changed to INFO, which makes them 1000x bigger Nov 8, 2024
@Fishbowler
Copy link
Contributor

The ask is to make log levels configurable.
Perhaps we can go one better, by redirecting the Netty GRPC logging to a separate file?
Would need a bit of juggling in how LogConfig.kt is called, but looks achievable.

@herval herval added the Maestro label Nov 26, 2024 — with Linear
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants