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

should the pull logs be classified with INFO? #122

Open
hellt opened this issue Feb 22, 2024 · 2 comments
Open

should the pull logs be classified with INFO? #122

hellt opened this issue Feb 22, 2024 · 2 comments

Comments

@hellt
Copy link
Member

hellt commented Feb 22, 2024

Currently verbose pull logging (the verbosiness is likely due to the non interactive way the logs are displayed)

But nevertheless, it seems that this pull logs might be rather classified as DEBUG?

    INFO |               clabernetes | ghcr.io/srl-labs/alpine:latest: resolving      |--------------------------------------|                                  │
│ elapsed: 0.5 s                  total:   0.0 B (0.0 B/s)                                                                                                        │
│                                                                                                                                                                 │
│     INFO |               clabernetes | ghcr.io/srl-labs/alpine:latest: resolving      |--------------------------------------|                                  │
│ elapsed: 0.6 s                  total:   0.0 B (0.0 B/s)                                                                                                        │
│                                                                                                                                                                 │
│     INFO |               clabernetes | ghcr.io/srl-labs/alpine:latest:                                                   resolved       |++++++++++++++++++++++ │
│ manifest-sha256:388d48147d080f5d2a2dceb82d9054b65b8e6762111703e78ca5cd625487a691: exists         |++++++++++++++++++++++++++++++++++++++| 
@carlmontanari
Copy link
Contributor

all the stuff that is shell'd out to is put in info rn, so I was just continuing that trend to be consistent, but yea we can put the pull crap into debug if ya want

@hellt
Copy link
Member Author

hellt commented Feb 22, 2024

we can live with that for sure, it just takes a lot of screen estate. We can have this parked for a little while so that it is logged and messes up with the OCD

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants